Conversation
Open pull requests stall on things triage never looked at: runs held for first-time contributors, outdated review threads, reviews left unanswered and branches that no longer merge. Rename the skill to codetrial-triage and add a read-only sweep script that lists these, plus pull requests closing the same issue. A sweep applies the mechanical fixes itself: run approvals, thread resolutions, and the fixed rebase request, hiding the copies it replaces only once it lands. Drafted nudges still wait for the exact text to be confirmed.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Open pull requests stall on things the triage skill never looked at: workflow runs held for first-time contributors, outdated review threads, reviews left unanswered for over a week, branches that no longer merge, and two pull requests closing the same issue. This renames
codetrial-issue-triagetocodetrial-triageand adds a pull request sweep:pr-sweep.shgathers all five in one read-only call, andpulls.mdsays what to do with each. A sweep applies the mechanical actions itself (approving a first-time contributor's held run, resolving an outdated thread, posting the fixed rebase request inrebase.mdon a conflicting non-draft pull request and hiding the copies it supersedes), while a drafted nudge or overlap notice still waits for its exact text to be confirmed. A hand-written review that mentions rebase is never hidden automatically, since it may ask for more than the rebase.Verified with shellcheck, shfmt and
scripts/indent.sh --check; by runningpr-sweep.shagainst a stubghserving fixtures for drafts, the viewer's own pull request, a stacked branch, a workflow-touching run, a stale run SHA, a trimmed fixed-text match and a fixed-text review carrying inline comments; and read-only against this repository, where it matched held runs to their pull requests by head SHA and found the #128/#143 overlap on #93 before #128 merged.Summary by cubic
Adds a pull request sweep to the triage skill, which previously stalled on five things it never looked at: workflow runs held for first-time contributors, outdated review threads, reviews unanswered for over a week, branches that no longer merge, and two pull requests closing the same issue. Renames
codetrial-issue-triagetocodetrial-triage.Sweep behavior
pr-sweep.shgathers all five conditions in one read-only call;pulls.mdsays what to do with each.rebase.mdwhile hiding the copies it supersedes.Written for commit be33df4. Summary will update on new commits.