Skip to content

Commit c06c13f

Browse files
committed
skill: collapse next-pr rebases to one commit
1 parent 51a8034 commit c06c13f

1 file changed

Lines changed: 6 additions & 6 deletions

File tree

.agents/skills/next-pr/SKILL.md

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
---
22
name: next-pr
3-
description: Review and advance the next easy-to-merge pull request in this repository, then continue to the next simplest PR. Use when the user wants Codex to list open PRs, pick the most trivial low-risk PR, establish or reuse a clean baseline on the default branch, rebase the PR, rerun tests and benchmarks when needed, summarize findings for approval, and only merge and push after explicit user approval.
3+
description: Review and advance the next easy-to-merge pull request in this repository, then continue to the next simplest PR. Use when the user wants Codex to list open PRs, pick the most trivial low-risk PR, establish or reuse a clean baseline on the default branch, collapse the rebased PR into a single commit on top of that branch, rerun tests and benchmarks when needed, summarize findings for approval, and only merge and push after explicit user approval.
44
---
55

66
# Next PR
@@ -64,9 +64,9 @@ If any of these changed since the previous cycle, do not reuse the old baseline:
6464
- the local checkout or environment in a way that could affect results
6565

6666
5. Check out the PR only after the baseline is green.
67-
Create a dedicated local review branch for the PR in the current checkout. Use `gh pr checkout` or an equivalent `gh`-driven fetch to materialize the PR locally. Rebase it on the latest default branch tip. Resolve conflicts carefully without discarding upstream or user work.
67+
Create a dedicated local review branch for the PR in the current checkout. Use `gh pr checkout` or an equivalent `gh`-driven fetch to materialize the PR locally. Rebase it on the latest default branch tip and collapse the PR changes into a single commit on top of that tip. Resolve conflicts carefully without discarding upstream or user work.
6868

69-
6. Validate the rebased PR branch.
69+
6. Validate the collapsed rebased PR branch.
7070
Run the same tests and benchmarks used for the baseline so the comparison is meaningful. Call out:
7171
- passes or failures
7272
- benchmark regressions or improvements
@@ -82,7 +82,7 @@ Report:
8282
- PR number and title
8383
- why it was chosen
8484
- what changed at a high level
85-
- rebase result
85+
- rebase/collapse result
8686
- baseline validation result
8787
- PR validation result
8888
- recommendation and any risks
@@ -95,13 +95,13 @@ If the review shows the PR is cosmetic or otherwise a no-op, and it does not fix
9595
- use `gh pr close` with a polite comment after the user approves closing it
9696

9797
8. Merge only after explicit user approval.
98-
After approval, integrate the rebased PR into the default branch in the way the repo expects. Preserve the PR commits unless the user asks to squash or rewrite them.
98+
After approval, integrate the single collapsed PR commit into the default branch in the way the repo expects.
9999

100100
9. Push only after the merge is complete and the default branch is in the intended state.
101101
Push the default branch to the appropriate remote and report what was pushed. If the pushed default branch tip matches the rebased PR branch you just validated, treat that validation run as the baseline for the next cycle.
102102

103103
10. Close the PR manually if GitHub does not auto-close it after the merge push.
104-
When a PR was merged by rebasing or otherwise rewriting commits, verify whether it is still open. If it is, close it with `gh pr close` and leave a short comment explaining that the changes were merged onto the default branch via rebased commit(s).
104+
When a PR was merged by rebasing or otherwise rewriting commits, verify whether it is still open. If it is, close it with `gh pr close` and leave a short comment explaining that the changes were merged onto the default branch via the collapsed rebased commit.
105105

106106
11. Immediately start the workflow again on the next simplest remaining PR unless the user explicitly asked to stop after one PR.
107107
Go back to step 2, pick the next easiest yes/no decision, and keep working through the queue. When step 9 left you on the same validated default branch tip, use that just-finished validation run as the baseline for the new cycle instead of rerunning baseline tests and benchmarks.

0 commit comments

Comments
 (0)