Stacked Pr Review

You are a staff engineer addressing code review feedback on a stack of jj branches, keeping the fix as a clean, separate revision rather than squashing it into the parent.

Process

1. Read PR review comments using gh api:

``bash gh api repos/{owner}/{repo}/pulls/{pr_number}/comments `` Identify which branch the PR targets (this is the "target bookmark").

1. Navigate to the target branch by creating a new revision on top of it:

``bash jj new `` This places the working copy directly on the target branch, ready for the fix.

1. Make code changes to address the review comments. Only change what's needed to satisfy the reviewer — don't refactor or clean up unrelated code.

1. Describe the fix with a concise commit message:

``bash jj describe -m "address review: " ``

1. Identify downstream stacked branches — these are any branches that were originally built on the target bookmark and now need to be rebased onto the fix:

``bash jj log --revisions 'descendants()' --no-graph `` Note the revision IDs of any downstream branches.

1. Rebase downstream branches onto the fix:

``bash jj rebase -s -d @ ` Run this for each downstream branch that needs to move. @` refers to the current fix revision.

1. Resolve any rebase conflicts, if they appear:

• jj log will show conflicted revisions with a ! marker.
• For each conflicted revision:

1. jj new — create a resolution revision on top of it
2. Edit the conflicted files to resolve the markers (>>)
3. Verify the file looks correct
4. jj squash — fold the resolution into the conflicted revision

• Repeat until jj log shows no ! markers.

1. Retag bookmarks to point to the updated revisions:

``bash jj bookmark set -r `` Do this for each downstream bookmark that moved during the rebase.

1. Push all affected branches in one command:

``bash jj git push --bookmark --bookmark ... `` Include the fix revision's bookmark AND all downstream bookmarks.

1. Verify: visit the PRs on GitHub (or run gh pr view ) to confirm the pushes updated the right branches. Check that CI is triggered.

Key Rules

• Never squash the fix into the parent unless the reviewer explicitly asks for it — keep it as its own revision. This preserves the review history.
• Never force-push main/master — only push to feature branches.
• If you're unsure which revision a bookmark points to, use jj bookmark list to inspect.

Completion Criteria

• All review comments are addressed in code
• The fix is a separate revision with a clear description
• All downstream branches have been rebased and their bookmarks updated
• All affected bookmarks are pushed to the remote
• CI is triggered on the updated PRs

Source & license

This open-source skill is cataloged on AgentStack and links to its original source — we do not rehost the code.

• Author: jack-michaud
• Source: jack-michaud/faire
• License: MIT

Install and usage instructions live in the source repository linked above.