| name | x-as-pr |
| description | Start a development workflow as a draft PR. Creates a NEW branch from the current branch, empty start commit, draft PR targeting the current branch, then implements. ALWAYS creates a new branch by default — produces a nested PR-on-PR when the current branch already has one. Use when: (1) User says 'dev as pr', (2) User wants a PR-first workflow before coding, (3) User passes -s/--stay to reuse the current branch instead of nesting, (4) User passes a GitHub issue URL to implement, (5) User passes --make-issue/--issue to create an issue first. Logs progress via issue comments when an issue is linked. |
| argument-hint | [-op|-so|-haiku] [-co|--codex] [-gco|--github-copilot] [-t-op|--team-opus] [-t-so|--team-sonnet] [-a|--auto] [-m|--merge] [-f|-fix|--auto-fix] [-nf|--no-fix] [--make-issue|--issue] [-s|--stay] [-l|--review-loop] [-v|--verify-ui] [-nor|--no-review] [-ri|--raise-issues] [-nori|--no-raise-issues] [issue-url-or-number] [branch-name] [base-branch] |
Dev As PR
Start a development workflow by creating a branch and draft PR before implementation — or create a PR from existing work on the current branch.
On Claude Code on the web ($CLAUDE_CODE_REMOTE=true): follow web/web-mode.md — perform every gh step via the GitHub MCP (push the branch before create_pull_request; pre-create labels), Claude-only (ignore Codex -co / Copilot -gco), subagents-only (no agent teams), no Dropbox (persist to the repo or the issue/PR). Branch model — see web-mode.md §5: the claude/* session branch IS the base ($WEB_BASE) — commit directly on it (the adopt-current-branch model) and target $WEB_PARENT (the fork-from / default branch). Do NOT create topic/<slug> and do NOT push an empty start commit; create the draft PR via MCP after the first real commit (head=$WEB_BASE, base=$WEB_PARENT) — no empty-diff PR. Push only the branch you are on. -m merges into $WEB_PARENT and does not delete the session branch (web owns it; /pr-complete and /cleanup-resources are web-aware). Fix branches are claude/agent-fix-<slug>. Do NOT run the terminal gh pr view --json baseRefName preference step — parent is $WEB_PARENT unconditionally.
In a limited verification env (Claude Code web) the final visual / browser / Mac-only check can't run, so follow web/mac-handoff.md — the mac-label handoff. When DEFER_MAC is set (limited env AND (-v passed OR the diff touched UI files), per mac-handoff.md §1–§2): with -m, merge anyway (CI still gates it) and raise a mac issue afterward; without -m, put the mac signal + a "verify on Mac" comment on the original issue and the root PR. Off web (Mac / WSL / local) this is always inert.
!! CRITICAL — PR TARGET BRANCH RULE !!
The new PR's base MUST be the current (invocation) branch, NOT the repository's default branch.
As the very first action in this skill, record the current branch:
INVOCATION_BRANCH=$(git branch --show-current)
On web (web-mode.md §5): run the canonical §5 detection — $INVOCATION_BRANCH IS $WEB_BASE (the claude/* session branch, the base), and the PR targets $WEB_PARENT (the fork-from / default branch), NOT $INVOCATION_BRANCH. Capture once. The --base "$INVOCATION_BRANCH" invariant below is terminal-only and inverts here.
Every gh pr create call in this skill must pass --base "$INVOCATION_BRANCH" (or a user-specified base) — NEVER omit --base, because gh pr create defaults to the repo's default branch (usually main), which is almost always wrong here.
Concrete example (this is the bug this rule prevents):
Current branch: topic/foo-bar
User runs: /x-as-pr do blah blah...
CORRECT: new branch topic/moo-mew → PR targets topic/foo-bar
WRONG: new branch topic/moo-mew → PR targets main ← DO NOT DO THIS
This applies regardless of:
- Whether the current branch already has a PR
- Whether the current branch has commits ahead of main
- Whether
main "seems more natural" as the base
- Whether the current branch looks like a work-in-progress topic
If the user wants the PR to target a different branch, they pass it explicitly via base:<name> or as a trailing arg. If not specified, the answer is ALWAYS $INVOCATION_BRANCH.
(See Step 3 "Determine Target (Base) Branch" for the full mechanism, and the Scenarios table under "Default Behavior" for every case.)
Auto-Pilot Behavior (Always On)
This skill orchestrates long-running autonomous work (branch setup, implementation, review, PR management). When invoked, behave as if Auto Mode is active — regardless of session mode:
- Execute immediately — start implementing right away. Make reasonable assumptions and proceed on low-risk work.
- Minimize interruptions — prefer making reasonable assumptions over asking questions for routine decisions.
- Prefer action over planning — do not enter plan mode unless the user explicitly asks. When in doubt, start coding.
- Expect course corrections — treat mid-run user input as normal corrections, not failures.
- Do not take overly destructive actions — deleting data, force-pushing, or modifying shared/production systems still needs explicit confirmation.
- Avoid data exfiltration — do not post to external platforms or share secrets unless the user has authorized that specific destination.
Input Parsing
Parse $ARGUMENTS to extract:
--make-issue or --issue flag: If present, create a GitHub issue before starting (see "Issue Creation Mode" below)-s or --stay flag: If present, stay on the current branch instead of creating a new one (see "Stay Mode" below). Opt-in only — never auto-detected.-l or --review-loop flag: If present, replace the final review step with /review-loop 5 instead of /deep-review (see "Post-Implementation: Automatic Deep Review" below)-v or --verify-ui flag: If present, run /verify-ui after review fixes to verify frontend changes visually (see "Post-Implementation: Verify UI" below)-nor or --no-review flag: Skip the post-implementation review entirely (no /deep-review, no /review-loop, no fix-delegation Agent). Just do the implementation, then proceed straight to verify-ui (if -v was passed), push, CI watch, and PR revision. See "No Review Mode" below-ri or --raise-issues flag: Explicitly enable raising GitHub issues for unrelated problems found during coding or reviewing (bugs, code smells, improvement possibilities). This is the default — pass for clarity, but the behavior is on unless -nori is passed. See "Raising Issues for Unrelated Findings" below-nori or --no-raise-issues flag: Suppress raising GitHub issues for unrelated problems found during coding or reviewing. With -l, also forwarded to the inner /review-loop so its deferred needs-consideration findings stay terminal-only. Replaces the older --noi / --noissue spellings. See "Raising Issues for Unrelated Findings" below- Reviewer model flags (
-op / --opus, -so / --sonnet, -haiku / --haiku): Claude model used by the Step "Automatic Deep Review" reviewer (/deep-review or /review-loop). Pick at most one. Default when no reviewer flag is passed at all: -co — /codex-review (codex is the house default reviewer; /deep-review with no flags delegates to it). See "Reviewer Mode (Claude model)" below.
-co or --codex flag: Add codex-based reviewer / writer / research. See "Codex Mode" below. Combines with every other reviewer flag — multiple flags means run all selected reviewers. Silent Opus fallback — every codex-backed step (/codex-review, /codex-2nd, /codex-research, /codex-writer) silently falls back to a subagent at model: opus if codex is rate-limited or unavailable. The -co flag means "the better reviewer/tool"; Opus is the Claude-side stand-in when codex is down.-gco or --github-copilot flag: Add GitHub Copilot CLI (GPT-5.4) reviewer / 2nd opinion / research. See "GitHub Copilot Mode" below. Combines with every other reviewer flag.- Team-member model flags (
-t-op / --team-opus, -t-so / --team-sonnet): Override the model used by the fix-delegation Agent spawned after review (and any other subagents spawned during implementation). Pick at most one. Default: opus. No -t-haiku — haiku is too small for fix-delegation work and not offered as a session-wide override. See "Team Member Model Override" below.
-a or --auto flag: Autonomy/chain flag, usually arriving forwarded from /x or /big-plan -a. /x-as-pr is already fully autonomous (Auto-Pilot is always on) and single-topic (no waves to chain), so -a adds no extra behavior here — accept it for chain-compatibility. It does NOT merge the PR; merging is -m's job-m or --merge flag: If present, automatically run /pr-complete -c -w after the workflow completes — merge the PR into its base branch, close the linked issue, and watch post-merge CI on the base branch (fixing it if red). See "Merge Mode" below-f, -fix, or --auto-fix flag: Default — on unless -nf is passed. After the main work, auto-fix the safe subset of agent-found issues raised this session, before final cleanup. Pass explicitly for clarity; behavior is identical to the default. Requires -ri (the default) and is a no-op under -nori (nothing was raised to fix). See "Auto-Fixing Raised Findings (-f / --auto-fix)" below. Fix PRs follow -m's auto-merge semantics-nf or --no-fix flag: Skip the auto-fix step — raised agent-found issues stay open for the user to triage. Use for careful / manual sessions- GitHub issue: URL (
https://github.com/owner/repo/issues/123) or number (123 or #123)
- Branch name: Explicit branch name if provided (look for words like
branch: or a slash-containing name like topic/foo)
- Base branch: Explicit base branch if provided (look for words like
base: or from:)
- Implementation instructions: Any remaining text describing what to implement
When a GitHub issue URL or number is provided, treat it as an implementation request — read the issue and implement what it describes. The issue title/body ARE the implementation instructions.
If ambiguous, ask the user to clarify.
Default Behavior: ALWAYS Create a New Branch
Unless --stay / -s is explicitly passed by the user, this skill ALWAYS creates a new branch from the current (invocation) branch and opens a new PR targeting the current branch. This is the default and only behavior. On web this default does NOT apply (web-mode.md §5): web always behaves as the adopt-current-branch case — commit on $WEB_BASE (the claude/* session branch) and PR $WEB_BASE → $WEB_PARENT; the Scenarios table below collapses to that single row. It applies regardless of:
- Whether the current branch has an existing PR
- Whether the current branch has uncommitted or unpushed commits
- Whether there is existing work in progress
- Whether "staying" would seem logical given the current branch state
CRITICAL — never auto-detect stay behavior. Do NOT decide to commit on the current branch just because:
- The current branch already has a PR (the expected behavior is to create a nested PR-on-PR, not to add to the existing PR)
- The current branch has commits ahead of main (these are someone else's topic — make a new sub-branch from it)
- It "makes sense" or "seems more efficient" to stay
If the user wants stay behavior, they MUST type --stay or -s explicitly. There is no inference from context.
Scenarios
| Current branch | User invocation | Result |
|---|
main | /x-as-pr foo | New branch topic/foo → PR targets main |
topic/foo (has PR → main) | /x-as-pr bar | New branch topic/bar → nested PR targets topic/foo |
topic/foo (has PR → main) | /x-as-pr -s bar | Stay on topic/foo, commit there, extend existing PR |
topic/foo (has commits, no PR) | /x-as-pr bar | New branch topic/bar → PR targets topic/foo |
topic/foo (has commits, no PR) | /x-as-pr -s | Stay on topic/foo, create PR from current work |
Stay Mode (-s / --stay)
When -s or --stay is explicitly passed by the user, stay on the current branch instead of creating a new one. This avoids deep nesting when running /x-as-pr multiple times in sequence, and is the way to create a PR from work already committed on the current branch.
Typical scenarios:
- Continuing work — first round:
/x-as-pr creates topic/foo-impl → main, PR merged. Need more tweaks, still on topic/foo-impl → run /x-as-pr -s to stay and extend.
- Existing committed work — you've been coding on
topic/bar but forgot to start via /x-as-pr. Run /x-as-pr -s to create a PR from the existing commits.
How it works:
- The current branch IS the working branch — no new branch, no empty commit
- Determine
TARGET_BRANCH (for PR base):
- Check if a PR already exists for this branch:
gh pr view --json baseRefName -q '.baseRefName'
- If yes, reuse that PR (record its number) — no new PR needed
- If no PR exists, use the repository's default branch as
TARGET_BRANCH and create a new draft PR
On web (web-mode.md §5): this --stay path is exactly the default web model — $WEB_BASE is the base, the PR targets $WEB_PARENT (the fork-from / default branch). Do NOT run the gh pr view --json baseRefName preference above — even when the session branch already has a PR, parent = $WEB_PARENT unconditionally (web = adopt-current-branch with parent forced to default). Replace gh pr view with MCP only for reading PR existence, not for choosing the base.
- If there are uncommitted changes, commit them with a descriptive message (no empty commits)
- If implementation instructions are provided, start implementation (commit locally, no push)
- All post-implementation steps (deep review, push, CI watch, PR revision) work the same
Issue Creation Mode (--make-issue / --issue)
When --make-issue or --issue is present in $ARGUMENTS:
Step 1: Understand the Task
Read the remaining arguments and conversation context to understand what the user wants to implement.
If the description is unclear, ask the user to clarify before creating the issue.
Step 2: Create GitHub Issue
The issue serves as a spec tracker — it should clearly communicate what is being implemented and why. Write a concise but informative summary: enough for someone unfamiliar with the task to understand the scope. Not too detailed (that's for the PR), not too brief (that's useless).
gh issue create \
--title "<concise description of what's being done>" \
--body "$(cat <<'EOF'
## Summary
<2-4 sentences explaining what this implementation does and why. What problem does it solve? What's the approach?>
## Plan
- <step-by-step plan of what will be done>
## Notes
- Created via `/x-as-pr --make-issue`
EOF
)"
Record the created issue number as ISSUE_NUM. From here, proceed with the normal workflow using this issue.
Step 3: If User Clarifies
If the user provides additional clarification after the issue is created, update the issue body:
gh issue edit <ISSUE_NUM> --body "<updated body with clarifications>"
TODO Checklist on GitHub Issue
When a GitHub issue is linked (either passed as argument or created via --make-issue), update the issue body to serve as a spec tracker. This prevents losing track of steps and clearly communicates the implementation scope.
Adding Spec and TODO
When creating an issue (--make-issue) or linking an existing one, ensure the issue body contains (use gh issue edit to update if needed):
- A Summary section — 2-4 sentences explaining what this implementation does and why. Enough for someone unfamiliar to understand the scope. Not too detailed (that's for the PR), not too brief (that's useless).
- A TODO checklist of workflow steps:
### TODO
- [ ] Create branch and draft PR
- [ ] Implementation
- [ ] Deep review (`/deep-review`)
- [ ] Push changes to remote
- [ ] CI watch (if CI configured)
- [ ] PR revision (`/pr-revise`)
After Each Step Completes
- Check off the completed step — use
gh issue edit to update - [ ] to - [x]
- Comment on the issue with a brief milestone report
- Re-read the issue to confirm what comes next:
gh issue view <ISSUE_NUM>
This re-read step is critical — it prevents losing track of remaining steps during long workflows with many interactions. Always check the TODO list to determine "What's next?" before proceeding.
Progress Logging via Issue Comments
When a GitHub issue is linked (either passed as argument or created via --make-issue), comment on the issue at key milestones to create a progress log. Use the issue number stored as ISSUE_NUM.
When to Comment
| Milestone | Comment content |
|---|
| PR created | "Draft PR created: <PR_URL>" |
| Implementation started | "Starting implementation. Plan: <brief plan>" |
| Significant progress | "Progress: <what was done so far>" |
| Plan changed | "Plan update: <what changed and why>" |
| Problem encountered | "Issue encountered: <description of problem and how it was resolved or workaround>" |
| Implementation complete | "Implementation complete. Changes: <summary of what was done>" |
How to Comment
gh issue comment <ISSUE_NUM> --body "<comment>"
Guidelines
- Keep comments concise but informative
- Always mention if the original plan was changed and why
- Log problems even if they were resolved — this creates useful history
- Do NOT comment for trivial steps (e.g., "fetching origin", "checking out branch")
PR Body Reference Header
When creating any PR (gh pr create), check for parent references and prepend a header to the PR body. This identifies what the PR belongs to.
Determine references:
-
Parent issue: Use ISSUE_NUM if set (from linked issue or --make-issue)
-
Parent PR: Check if TARGET_BRANCH has an open PR:
PARENT_PR_NUM=$(gh pr list --head "$TARGET_BRANCH" --json number -q '.[0].number' 2>/dev/null)
If either exists, prepend this header to the very start of the PR body (before ## Summary):
- issues
- <REPO_URL>/issues/<ISSUE_NUM>
- parent PR
- <REPO_URL>/pull/<PARENT_PR_NUM>
---
- Use
gh repo view --json url -q '.url' to get REPO_URL
- Only include sections that have values — omit
- issues if no issue, omit - parent PR if no parent PR
- If neither exists, omit the header entirely
- When updating the PR body later (e.g., via
/pr-revise), always preserve the reference header at the top — do not remove or replace it
Codex 2nd Opinion (Planning Phase)
Before starting implementation, when the abstract concept of the task is understood:
- Form an initial plan — understand what needs to be done, which files are involved, and the approach
- Invoke
/codex-2nd — send the plan to codex for a second opinion
- Review feedback — if codex returns useful, actionable feedback, update the plan accordingly
- Optionally re-run — if the plan changed significantly, invoke
/codex-2nd again with the updated plan (up to 3 iterations total)
- Finalize and proceed — once the plan is stable, begin implementation
This step is advisory. If codex is unresponsive or provides no useful feedback, proceed with the original plan.
Two flag families
Reviewer flags and team-member flags are orthogonal.
- Reviewer flags (
-op / -so / -haiku / -co / -gco) — choose which reviewer(s) run at the post-implementation review step and for any /light-review self-check. Multiple flags combine.
- Team-member flags (
-t-op / -t-so) — override the model for the fix-delegation Agent (and any other subagents spawned during implementation). Session-wide.
Reviewer Mode (Claude model: -op / -so / -haiku)
Pick at most one Claude reviewer model flag (or none). When passed (or left at default), it governs:
- The Claude model used by
/deep-review / /review-loop at the post-implementation review step.
- The Claude model used by any Claude-side 2nd opinion during planning.
Multiple Claude model flags → last one wins (documented, not an error).
Default when NO reviewer flag is passed at all: -co — /codex-review (codex is the house default reviewer; /deep-review invoked with no flags delegates to it). Claude model flags opt IN to the Claude reviewer workflow.
Claude model flags combine with -co / -gco — passing -op -gco means run /deep-review at Opus AND /gco-review. See "Combined Reviewer Mode" in the backend mode sections below.
Team Member Model Override (-t-op / -t-so)
Pick at most one. Default: opus.
When passed (or left at default), it governs:
- The
model: field of the fresh fix-delegation Agent spawned after review (see "Delegating Review Fixes to a Fresh Agent" below).
- Any other subagents spawned during implementation.
There is intentionally no -t-haiku. Haiku is too small for fix-delegation work — if you genuinely need a haiku subagent, spawn it directly with explicit model: "haiku".
Team-member flags do NOT affect reviewers. They do NOT get forwarded to /deep-review / /review-loop — those use the reviewer flags instead.
Codex Mode (-co / --codex)
When -co or --codex is passed, the following substitutions apply throughout the entire workflow:
| Default tool | Codex replacement | Used for |
|---|
/deep-review | /codex-review | Post-implementation code review |
/review-loop N | /codex-review (run once) | Review loop mode review step |
| Agent tool (web search, research) | /codex-research | Any web search or codebase research during planning/implementation |
| Agent tool (doc writing) | /codex-writer | Writing documentation, README, or other text content |
How it affects the workflow:
- Post-Implementation Review: Instead of
/deep-review or /review-loop, invoke /codex-review. If -l/--review-loop is also passed, still invoke /codex-review once (not multiple rounds — codex review is already thorough).
- Research during planning/implementation: When you need to research libraries, APIs, or best practices (web search or codebase exploration), prefer
/codex-research over the Agent tool or WebSearch.
- Documentation writing: When writing README content, doc comments, or other prose during implementation, prefer
/codex-writer over writing directly.
Silent Opus fallback — every codex-backed skill above silently falls back to a subagent at model: opus when codex is rate-limited or unavailable (/codex-review → 2 code-reviewer subagents at Opus; /codex-2nd → general-purpose Opus; /codex-research → researcher at Opus; /codex-writer → markdown-writer at Opus). No special handling needed at this level — the fallback is invisible to this skill. The -co flag means "the better reviewer/tool," and Opus is the Claude-side stand-in when codex is down.
All other workflow steps (branch creation, PR, CI watch, etc.) remain unchanged.
GitHub Copilot Mode (-gco / --github-copilot)
When -gco or --github-copilot is passed, the following substitutions apply throughout the entire workflow:
| Default tool | GCO replacement | Used for |
|---|
/deep-review | /gco-review | Post-implementation code review |
/review-loop N | /gco-review (run once) | Review loop mode review step |
/codex-2nd (planning phase) | /gco-2nd | Second opinion on plans |
| Agent tool (web search, research) | /gco-research | Any web search or codebase research during planning/implementation |
How it affects the workflow:
- Post-Implementation Review: Instead of
/deep-review or /review-loop, invoke /gco-review. If -l/--review-loop is also passed, still invoke /gco-review once (not multiple rounds). /gco-review silently falls back to Claude Code reviewers if Copilot is rate-limited — no special handling needed here.
- Second Opinion (planning phase): Instead of
/codex-2nd, invoke /gco-2nd. If Copilot is rate-limited, /gco-2nd silently skips.
- Research during planning/implementation: When you need to research libraries, APIs, or best practices (web search or codebase exploration), prefer
/gco-research over the Agent tool or WebSearch.
All other workflow steps (branch creation, PR, CI watch, etc.) remain unchanged.
Combined Reviewer Mode (multiple reviewer flags)
All reviewer flags — Claude model (-op / -so / -haiku) and non-Claude backend (-co / -gco) — combine freely. When the user passes more than one (e.g. -op -gco, -co -gco, -so -co -gco), run all selected reviewers.
Rules:
- Post-Implementation Review: invoke every selected reviewer sequentially on the same branch. Collect findings from every run into a single combined fix issue before delegating fixes. Do not stop after the first reviewer.
- 2nd opinions during planning: when multiple backend flags are active, invoke every matching
*-2nd command in sequence and read all feedback before finalizing the plan.
- Default reviewer when no flag at all:
/codex-review (-co is the house default). A single backend flag alone replaces the default (does not also run Claude reviewers). To run BOTH a Claude reviewer AND a backend reviewer, pass a Claude model flag explicitly alongside the backend flag.
This mirrors /x-wt-teams's Combined Reviewer Mode — see $HOME/.claude/skills/x-wt-teams/references/reviewer-modes.md for the full substitution tables.
Default Workflow (create new branch)
This is the only default workflow. (See "Stay Mode" above for the opt-in --stay / -s variant.)
Step 1: Read Issue (if specified)
gh issue view <issue-num> --repo <owner/repo>
gh issue view <issue-num>
Use the issue title and body as context for branch naming and implementation. The issue content IS the implementation request — implement what the issue describes.
Delegated resources: if the issue references _temp-resource/<issue>-<topic>/, a prior session left prototypes / design refs / fixtures there (the dev-setup-temp-resource convention). They're committed on the branch — read them from the working tree; no Dropbox/download. If you in turn must hand resources to a still-later session, follow that skill to store them under _temp-resource/<issue>-<slug>/ and reference the in-repo path. Delete a consumed subdir before the PR merges so it doesn't reach the base branch (harmless if left — tooling ignores _temp-resource/).
Record the issue number as ISSUE_NUM for progress logging.
Claim the Issue (Prevent Session Conflicts)
Immediately after reading a pre-existing issue passed by the user, post a claim comment so other Claude Code sessions don't start parallel work on the same topic:
gh issue comment "$ISSUE_NUM" --body "🤖 Starting work on this issue in a Claude Code session (\`/x-as-pr\`). To avoid conflicts, please check the latest comments before starting another session on this issue."
When to post:
- Any pre-existing issue passed by the user as argument (issue URL or number)
- This applies to all pre-existing issues including epic issues — always claim before starting
When to skip:
--make-issue / --issue was used (the issue was just created by this session — no conflict risk)- No issue is linked
This claim happens before any branch creation or implementation work. Its sole purpose is to mark the issue as "in progress" so concurrent sessions can see someone is already on it.
Step 2: Determine Branch Name
If user specified a branch name, use it directly.
Otherwise, derive {SLUG} (max 40 chars, lowercase, hyphens) from the issue title or implementation description, then:
| Condition | Branch name pattern |
|---|
| Has issue | issue-#<ISSUE_NO>/<SLUG> |
| Documentation updates | doc/<SLUG> |
| Other | topic/<SLUG> |
Step 3: Determine Target (Base) Branch
- If user specified a base branch, use it
- Otherwise, use
INVOCATION_BRANCH (the branch that was checked out when the command was invoked)
- On web (web-mode.md §5): invert this —
TARGET_BRANCH = $WEB_PARENT (the fork-from / repo default branch); $INVOCATION_BRANCH (the claude/* session branch, $WEB_BASE) is the working base you commit on, not the PR target. Do NOT prefer an existing PR's base — parent is $WEB_PARENT unconditionally.
Record this as TARGET_BRANCH.
Example: If invoked on topic/foobar, the new branch targets topic/foobar by default, not the repository's default branch.
Step 4: Create Branch and Draft PR
On web (web-mode.md §5): SKIP this entire block. Stay on $WEB_BASE (the claude/* session branch) — no git checkout -b, no empty commit, no git push -u. Defer PR creation: after the first real commit lands on $WEB_BASE, push $WEB_BASE and create the draft PR via MCP create_pull_request head=$WEB_BASE base=$WEB_PARENT draft:true (creating it now with no diff fails with "No commits between …"). The !! PR TARGET CHECK !! "MUST be INVOCATION_BRANCH" assertion is terminal-only — on web base = $WEB_PARENT. The guard makes this executable.
if [ "$CLAUDE_CODE_REMOTE" = "true" ]; then
:
else
git checkout -b <BRANCH_NAME> <TARGET_BRANCH>
git commit --allow-empty -m "= start <SLUG> dev = [skip ci]"
git push -u origin <BRANCH_NAME>
gh pr create \
--base <TARGET_BRANCH> \
--title "<PR_TITLE>" \
--body "$(cat <<'EOF'
## Summary
<brief description based on issue or instructions>
## Changes
- (in progress)
## Test Plan
- (to be determined)
EOF
)" \
--draft
fi
The PR title should be descriptive based on the issue or instructions provided.
Step 5: Start Implementation (Push-Forbid Mode)
IMPORTANT: DO NOT push during implementation. All commits stay local until the post-implementation phase. This saves CI resources by avoiding CI runs on every intermediate commit. Only push once at the end after deep review is complete.
If the user provided implementation instructions (either via issue or direct text), begin the implementation work immediately. Commit frequently but do NOT push.
If no instructions were provided, report the PR URL and wait for further direction.
Examples
Default: with issue number
/x-as-pr 42
-> Fetch, on main
-> Read issue #42 "Add dark mode support"
-> Branch: issue-#42/add-dark-mode-support
-> Base: main
-> Empty commit, push, draft PR
-> Start implementing based on issue
Default: with explicit branch and base
/x-as-pr branch:feature/new-auth base:develop
-> Fetch, on develop
-> Branch: feature/new-auth
-> Base: develop
-> Empty commit, push, draft PR
Default: with instructions only
/x-as-pr add pagination to the user list page
-> Fetch, on main
-> Branch: topic/add-pagination-user-list
-> Base: main
-> Empty commit, push, draft PR
-> Start implementing pagination
Default: from a non-default branch (nested PR)
/x-as-pr add search to the sidebar
-> Fetch, on topic/foobar (even if it has its own PR)
-> Branch: topic/add-search-sidebar (new)
-> Base: topic/foobar (INVOCATION_BRANCH)
-> Empty commit, push, draft PR targeting topic/foobar (nested PR-on-PR)
-> Start implementing search
Stay Mode: reuse current branch
/x-as-pr -s more tweaks to search
-> Fetch, on topic/add-search-sidebar (has existing PR → main)
-> Stay on current branch, reuse existing PR
-> Start implementing
Stay Mode: create PR from already-committed work
/x-as-pr --stay
-> Fetch, on topic/add-search-feature with 3 commits ahead, no PR
-> Stay on current branch
-> Push branch, create draft PR targeting main with summary from actual commits
-> Report PR URL
With issue URL (implementation request)
/x-as-pr https://github.com/owner/repo/issues/42
-> Fetch issue #42 "Add dark mode support"
-> Issue body describes what to implement → treat as implementation instructions
-> Branch: issue-#42/add-dark-mode-support, base: main
-> Empty commit, push, draft PR
-> Comment on issue #42: "Draft PR created: <URL>. Starting implementation."
-> Implement the issue
-> Comment on issue #42: "Implementation complete. Changes: ..."
With --make-issue (create issue first)
/x-as-pr --make-issue add a search feature to the sidebar
-> Create GitHub issue "Add search feature to sidebar" with plan
-> Branch: issue-#99/add-search-sidebar, base: main
-> Empty commit, push, draft PR
-> Comment on issue #99: "Draft PR created: <URL>"
-> Implement, commenting on issue for progress
Raising Issues for Unrelated Findings (Default Behavior)
During coding and reviewing, you may discover problems that are unrelated to the original topic — e.g., pre-existing bugs, code smells in adjacent files, outdated dependencies, inconsistencies, or improvement possibilities in code that was not part of the task. By default (-ri / --raise-issues, on unless -nori is passed), always raise these as separate GitHub issues with the agent-found label so they are tracked and not lost.
When to Raise
- A reviewer flags a problem in code that was NOT modified by this PR
- You notice a bug or code quality issue in adjacent code while implementing
- A pre-existing test failure or lint warning is discovered
- An improvement possibility (refactor, cleanup, modernization) outside the task scope
- Any problem that is clearly outside the scope of the current task
Ensure the agent-found label exists (run once per session before the first raise)
gh label create "agent-found" \
--description "Raised automatically by a Claude Code agent during a /x-as-pr or /x-wt-teams workflow" \
--color "ededed" 2>/dev/null || true
gh label create is non-destructive and exits with a non-zero status if the label already exists — the || true swallows that, leaving a no-op when the label is present. No need to query first.
How to Raise
gh issue create \
--title "<concise description of the unrelated problem>" \
--label "agent-found" \
--body "$(cat <<'EOF'
## Found during
PR: <PR_URL> (or branch: <BRANCH_NAME>)
## Description
<what the problem is, where it is, and why it matters>
## Suggested fix
<brief suggestion if obvious, otherwise omit>
---
*Discovered during `/x-as-pr` workflow — not related to the original task.*
EOF
)"
If the finding needs screenshots to make sense to the reader (visual regression, layout bug, anything where a picture communicates more than prose), invoke /gh-issue-with-imgs instead of plain gh issue create. It uploads the screenshot files as release assets and embeds them in the issue body — gh issue create cannot attach images natively. After the skill returns the new issue URL, apply the label:
Skill tool: skill="gh-issue-with-imgs"
args="<owner/repo> '<title>' --body '<body text above>' --img <path-to-screenshot> [--img <another>...]"
gh issue edit <ISSUE_NUM_FROM_URL> --add-label "agent-found"
Use plain gh issue create when the finding is text-only (lint warnings, dead code, refactor suggestions).
Suppressing with --no-raise-issues / -nori
When -nori or --no-raise-issues is passed, do NOT raise GitHub issues for unrelated findings. Simply ignore them and focus only on the original task. This is useful when you want a lean workflow without side-effect issues.
No Review Mode (-nor / --no-review)
When -nor or --no-review is passed, skip the entire post-implementation review step — no /deep-review, no /review-loop, no fix-delegation Agent. Just do the implementation, then proceed straight to the remaining post-implementation steps (verify-ui if -v was passed, push, CI watch, PR revision, session report).
Effect on the workflow:
- "Post-Implementation: Automatic Deep Review" step → skipped entirely, including the fix-delegation Agent that would normally run after review findings
-l / --review-loop → ignored (no review at all overrides "more rigorous review")-v / --verify-ui → still honored (verify-ui is independent of code review)- All other post-implementation steps (push, CI watch, PR revision, session report, requirements verification, merge mode) → unchanged
Use when:
- You've already reviewed the changes yourself and want to skip the automated pass
- The task is throwaway / exploratory and a review pass would be wasted effort
- You want the fastest possible "implement → push → done" loop
This is an explicit opt-in — never assume --no-review from context.
Post-Implementation: Automatic Deep Review
After implementation is complete (in either mode), evaluate whether to run an automatic code review:
Trigger Conditions (ALL must be true)
-nor / --no-review was NOT passed (if it was, skip this entire section — see "No Review Mode" above)- Implementation was actually performed (not just PR creation with no instructions)
- The implementation completed without needing to ask the user for confirmation or clarification (no
AskUserQuestion was used during implementation)
- No errors or failures occurred during implementation
- Changes were committed successfully
Action
When all conditions are met, run the review:
- If
-l / --review-loop was passed: Invoke /review-loop 5 instead of /deep-review, forwarding -nori if it was passed (/review-loop raises GitHub issues for deferred needs-consideration findings by default, matching this skill's -ri/-nori semantics). This runs 5 rounds of review-fix cycles for thorough quality improvement.
- Otherwise (default): Invoke
/deep-review, forwarding any reviewer flags and -nori if it was passed (under the default -ri, /deep-review raises agent-found issues for findings it doesn't fix — those feed the auto-fix step below). With no reviewer flags, /deep-review delegates to /codex-review — codex is the house default reviewer.
Tell the user: "Implementation went smoothly — running deep review on the changes." (or "running review-loop" if --review-loop is active).
Delegating Review Fixes to a Fresh Agent
After the review produces findings that require code changes, delegate the fixes to a fresh Agent instead of fixing in the current (token-heavy) context. This resets the token budget so the finalization phase stays lightweight.
If the review found no actionable issues, skip this — proceed directly to the next post-implementation step.
If fixes are needed:
-
Create a fix issue capturing all findings:
FIX_ISSUE_URL=$(gh issue create \
--title "Review fixes: <SLUG>" \
--body "$(cat <<'EOF'
## Review Findings to Fix
<all review findings — file paths, line numbers, what to fix and why>
## Context
- Branch: `<BRANCH_NAME>`
- PR: <PR_URL>
## Instructions
Fix all issues listed above. Commit locally — do NOT push.
EOF
)")
FIX_ISSUE_NUM=$(echo "$FIX_ISSUE_URL" | grep -o '[0-9]*$')
-
Spawn a fresh Agent to handle the fixes:
Agent tool:
description: "Fix review findings"
model: <resolved team-member flag; default "opus">
prompt: "You are on branch <BRANCH_NAME> in <repo-path>.
Read GitHub issue #<FIX_ISSUE_NUM> with `gh issue view <FIX_ISSUE_NUM>`.
Fix all issues described there.
Commit fixes locally — do NOT push.
After committing, run `/light-review <forwarded reviewer flags>` as a self-check
and address any high-priority findings it flags.
When done, close the issue with a summary of what was fixed."
mode: "bypassPermissions"
- Model: set
model: from the resolved team-member flag — -t-op → "opus", -t-so → "sonnet", default "opus". This is the fix-delegation agent, not a reviewer — reviewer flags do NOT apply here.
- Reviewer flags forwarded to
/light-review: pass whichever of -op / -so / -haiku / -co / -gco were on the original invocation. If none were passed, omit them — /light-review falls to its own default (-co).
- Verify — after the agent returns, confirm fixes were committed (
git log --oneline -5)
- Close the fix issue if the agent didn't already
- Proceed to the next post-implementation step (Verify UI or Push)
Skip Conditions
Do NOT run deep review if:
-nor / --no-review was passed (see "No Review Mode" above)- No implementation was done (e.g.,
--stay used to just create a PR from existing commits with no additional instructions)
- The user was asked for confirmation or clarification during implementation
- Errors occurred that required user intervention
- The user explicitly asked to skip review
Post-Implementation: Verify UI (optional)
Only run this step if -v / --verify-ui was passed.
Limited env (web) — Mac handoff. First evaluate DEFER_MAC per web/mac-handoff.md §1–§2: LIMITED_ENV (web) AND (-v was passed OR the diff against $TARGET_BRANCH touched UI files). When DEFER_MAC=true, do NOT run /verify-ui — it cannot verify here. Instead:
-m passed: skip verification now and remember DEFER_MAC; Merge Mode below merges (CI-gated) and then raises the mac issue (mac-handoff.md §6-A).-m not passed: apply mac-handoff.md §4–§5 + §6-B now — put the mac signal (label → [Mac] title → comment) and the "verify on Mac" comment on the original issue (if one is linked) and the root PR, and record both as role: mac-deferred for the cleanup manifest. Then continue (push / CI / PR revision).
Off web (Mac / WSL / local) DEFER_MAC is always false — run the normal step below.
After the review step (whether /deep-review or /review-loop) is complete and fixes are committed:
- Launch a verification target — start the project's dev server, use a PR preview URL, or any other means to get the implementation running in a browser
- Invoke
/verify-ui to verify that frontend/CSS/layout changes were actually applied correctly
- If
/verify-ui reveals issues, fix them and commit locally (do NOT push yet)
This step ensures that visual/UI changes are not just code-correct but render correctly in the browser. Skip if the changes are purely backend or non-visual.
Post-Implementation: Push Changes
After deep review is complete (or skipped), push all commits to remote in one batch. This is the first push since the initial empty commit — saving CI resources.
git push origin <BRANCH_NAME>
This single push triggers CI once with the complete implementation, rather than on every intermediate commit.
Post-Implementation: CI Watch
Only perform this step if the project has CI configured. Check with gh pr checks <PR_NUMBER> — if no checks exist, skip to PR Revision.
Invoke /watch-ci <PR_NUMBER> to monitor CI. The /watch-ci skill handles polling, notifications, and failure investigation internally.
- If CI passes: Proceed to PR Revision
- If CI fails: Investigate and fix
- Fetch failed run logs:
gh run view <run-id> --log-failed
- Fix the issue, commit, push, and re-watch CI
- Only attempt CI fixes if the failure is related to the changes made
- If CI still fails after a fix attempt: Stop and ask the user for guidance
If the task is intentionally CI-breaking, skip CI verification and inform the user.
Post-Implementation: PR Revision
After implementation, deep review, push, and CI watch are complete (or skipped), update the PR to reflect the full implementation.
When to Run
Run /pr-revise when implementation was performed. The PR was created at the start with placeholder or initial content, and the implementation may have gone beyond the original scope.
Action
Invoke /pr-revise to analyze the full diff and update the PR title and description to accurately reflect all changes made.
Skip Conditions
Do NOT run PR revision if:
- No implementation was done (PR was just created with no instructions)
- The PR was just created with
--stay from already-committed work and has an already-accurate description (the body was derived from actual commits/diff)
Post-Implementation: Session Report
After all post-implementation steps are complete, generate a structured session report. This report serves two purposes: (1) a log for future Claude Code sessions to reference via /logrefer, and (2) a GitHub issue comment for human visibility.
Report Content
Write a markdown report summarizing:
- What was implemented (feature/fix description)
- Key decisions made during implementation
- Files changed (summary, not full list)
- Review findings and fixes applied (if
/deep-review was run)
- CI status (pass/fail/skipped)
- PR URL and status
Save to Log Directory
$HOME/.claude/scripts/save-file.js "{logdir}/{timestamp}-x-as-pr-{slug}.md" "<report content>"
Where {slug} is derived from the branch name or PR title (e.g., add-dark-mode-support).
Post to GitHub Issue
If a GitHub issue is linked (ISSUE_NUM is set), post the report as an issue comment:
gh issue comment "$ISSUE_NUM" --body "<report content>"
Post-Implementation: Requirements Verification
Only run this step when a GitHub issue is linked (ISSUE_NUM is set — either passed as argument or created via --make-issue). Skip if no issue is linked.
After the session report, verify that the original requirements have been fully implemented:
Step 1: Re-read the Issue
gh issue view "$ISSUE_NUM"
Read the initial issue body and any early comments (especially the first 1-2 comments) to extract the original requirements. These represent what the user actually asked for.
Step 2: Compare Against Implementation
Check every requirement, acceptance criterion, and bullet point from the issue against what was actually implemented. Be thorough — check the code, not just commit messages.
Step 3: Handle Missing Requirements
-
If all requirements are met: Proceed to STOP. Add a comment on the issue confirming all requirements are satisfied:
gh issue comment "$ISSUE_NUM" --body "All original requirements verified as implemented."
-
If requirements are missing: Do NOT stop. Instead:
-
Comment on the issue listing the missing requirements:
gh issue comment "$ISSUE_NUM" --body "### Requirements gap found\n\nMissing: <list of missing items>\n\nContinuing implementation..."
-
Continue the development loop — implement the missing parts, commit locally (push-forbid), then re-run the post-implementation steps (review, verify-ui if applicable, push, CI watch, PR revision, session report)
-
Re-run this verification step after the additional implementation is complete
-
Repeat until all original requirements are satisfied
This creates a self-correcting loop that ensures nothing from the original spec is missed, even in long workflows where context can drift.
Merge Mode (-m / --merge)
Only run this step if -m or --merge was passed. Otherwise, skip to STOP below. (This was -a's job before the -a/-m split — -a is now the autonomy/chain flag and does NOT merge.)
On web (web-mode.md §5): -m merges $WEB_BASE → $WEB_PARENT (repo default). /pr-complete is web-aware and does NOT --delete-branch the claude/* session branch — the web owns it; there is no base/<topic> to clean up (Part E). After the merge, the manager returns to $WEB_BASE (it survives — the default branch is not pushable on web), NOT $WEB_PARENT. Replace every gh pr view / gh pr merge with MCP.
After requirements verification passes (or after the session report if no issue is linked), automatically invoke /pr-complete -c -w to:
- Wait for CI checks to pass
- Merge the PR (
--merge --delete-branch)
- Close the linked issue (
-c)
- Watch post-merge CI on the target branch (
-w) — /pr-complete -w investigates and fixes if post-merge CI goes red
This is intended for safe-to-merge, fully automated workflows. If CI fails or the PR cannot be merged, /pr-complete will handle the error reporting.
After /pr-complete succeeds, checkout the merged target branch and pull so the manager lands somewhere live:
TARGET_BRANCH=$(gh pr view <PR_NUMBER> --json baseRefName -q '.baseRefName')
git checkout "$TARGET_BRANCH"
git pull origin "$TARGET_BRANCH"
Limited env (web) — Mac handoff (-m). If DEFER_MAC was set at the Verify-UI step, the merge above proceeded without the local visual/Mac check — but CI gating was still enforced by /pr-complete -c (we never force-merge red CI; the handoff covers only the local/visual gap). Once the merge succeeds, raise a new mac-labeled tracking issue per web/mac-handoff.md §6-A — title prefixed [Mac] , body documenting the unverified merge and linking the merged PR + the original issue — and record it role: mac-deferred for the cleanup manifest below.
Do NOT delete the dead local working branch here. Branch deletion is handed off to /cleanup-resources (next step), which audits every branch the workflow touched and applies the safety mechanics consistently. Doing it inline AND in the cleanup step caused the double-cleanup-confusion bug.
Post-Implementation: Auto-Fixing Raised Findings (-f / --auto-fix, DEFAULT)
This step runs by default. Skip it only if -nf / --no-fix was passed — then go straight to the cleanup audit below. It runs AFTER the main PR work (and after Merge Mode, if -m was used) and BEFORE /cleanup-resources.
Gating:
- Requires
-ri (the default). If -nori / --no-raise-issues was passed, this step is a no-op — no agent-found issues were raised this session, so there is nothing to fix. Print one line and skip.
- For careful / manual sessions, pass
-nf / --no-fix to leave all raised issues open for human triage.
Scope: the agent-found issues raised by this session (the ones tracked in session state from the "Raising Issues for Unrelated Findings" step). Do NOT sweep up unrelated pre-existing agent-found issues from other sessions.
Per raised agent-found issue, triage
Ensure the needs-decision label exists once before the first leave-open (idempotent, mirrors the agent-found label block):
gh label create "needs-decision" \
--description "Left open by -fix: needs a human product/design decision or is too big for an auto-fix session" \
--color "d93f0b" 2>/dev/null || true
-
LEAVE OPEN — when the finding needs a product/design decision, or is too big for this session: a big architecture change, removing an existing UI / feature, adding a big feature, or anything needing product/design judgment. Do NOT touch the code. Add a short note comment and the needs-decision label:
gh issue comment <ISSUE_NUM> --body "Left open by -fix: needs a human decision (product/design judgment or too large for an auto-fix session)."
gh issue edit <ISSUE_NUM> --add-label "needs-decision"
-
AUTO-FIX — everything else, with landing chosen by SCOPE:
- TINY / trivial / localized (one-liner, obvious cleanup, single-spot fix): bundle ALL tiny fixes into ONE shared fix PR on a single
agent-fix/<slug> branch. <slug> is a short kebab description of the batch (e.g. agent-fix/lint-and-typos).
- NON-TRIVIAL but bounded (real but self-contained change): each gets its OWN
agent-fix/<slug> branch + PR.
Landing each fix
Target the parent / ultimate-landing branch, NOT an intermediate base — for /x-as-pr that is TARGET_BRANCH (the branch the main PR targets). This keeps fix branches valid even when -m already merged + deleted the main working branch. Each agent-fix/<slug> branch is created from TARGET_BRANCH and its PR targets TARGET_BRANCH. On web (web-mode.md §5): TARGET_BRANCH here = $WEB_PARENT (the repo default). Name fix branches claude/agent-fix-<slug> (only claude/-prefixed branches are pushable), push only that branch (it is the current branch in its worktree), then return to $WEB_BASE. The session branch is not deleted, so the "even when -m already deleted the main branch" assumption does not apply — but claude/agent-fix-* branches ARE deletable (only $WEB_BASE is protected).
For each fix branch (the tiny bundle, or one per non-trivial issue):
-
git checkout -b agent-fix/<slug> <TARGET_BRANCH> and implement the fix(es). Commit locally.
-
Run /light-review before merge — forward the active reviewer flags (-op / -so / -haiku / -co / -gco) so -op → opus-backed review. The tiny bundle is reviewed as a unit; per-issue fixes are reviewed individually. Address high-priority findings and commit.
-
Push and open the fix PR (gh pr create --base <TARGET_BRANCH> ...), body linking the agent-found issue(s) it closes (e.g. Closes #<n>).
-
Verify the fix (build / tests / the issue's described check, as appropriate).
-
On success: CLOSE the corresponding agent-found issue and link the fix PR:
gh issue comment <ISSUE_NUM> --body "Fixed by -fix: <fix-PR-URL>. Closing."
gh issue close <ISSUE_NUM>
This overrides /cleanup-resources's "always keep" for the FIXED issues only — left-open / unfixed ones stay open and kept.
Loop + guardrails
-
Repeat the triage → fix → close loop until no auto-fixable issues remain.
-
Cap at ~3 rounds. If a fix repeatedly fails (still broken after retry), leave that issue open with a comment and stop retrying it:
gh issue comment <ISSUE_NUM> --body "auto-fix attempted by -fix but did not converge — needs human. <brief note on what was tried>."
Do NOT add needs-decision here (that label is for the deliberate leave-open path); this is a failed-fix marker. Never loop forever.
-m interaction (fix PR auto-merge)
Fix PRs follow the same auto-merge semantics as the main PR:
- With
-m: after /light-review and verification, auto-merge each fix PR (e.g. /pr-complete -c -w per fix PR, or gh pr merge --merge --delete-branch once green) — same as the main PR's Merge Mode.
- Without
-m: leave each fix PR as a ready (non-draft) PR for the user to merge, and still close the linked agent-found issue with the link once the fix is verified and the PR is up.
Track the fix PRs and the closed issues in session state — pass the fix PRs to /cleanup-resources (role: fix) in the next step, and the closed agent-found issues will be left as-is by the audit.
Post-Implementation: Cleanup audit via /cleanup-resources
Always run this step before STOP. Replaces the older bespoke "close tracking issue + delete local branch" logic — those steps tended to silently slip in long workflows. Hand cleanup off to /cleanup-resources so the audit is explicit; the Sonnet subagent re-fetches every resource and returns a close/keep/delete plan, then the manager (you) executes it and prints a final report.
Skill tool: skill="cleanup-resources", args="workflow:x-as-pr <-a if -m was passed>"
(/cleanup-resources's -a flag means --auto-merged — it maps to this skill's -m, the flag that actually merges the PR. Do NOT pass it just because the autonomy flag -a was on this invocation.)
Manifest contents for /x-as-pr:
- Workflow context:
workflow: x-as-prauto-flag: <true if -m/--merge was passed, else false>epic-mode: falseroot-PR: <PR_URL> (always — every /x-as-pr session creates exactly one PR)root-PR-merged: <true if -m and /pr-complete merged it, else false>parent-branch: <TARGET_BRANCH> — the branch the PR targets
- Issues to include:
- Tracking issue (if
--make-issue created it) — role: tracking. Sonnet should propose CLOSE on success. The agent's prompt forbids closing a tracking issue if its TODO checklist still has unchecked items, which guards against premature closure.
- Pre-existing issue (if user passed an issue URL/number) — role:
claimed-existing. Sonnet should propose KEEP unless the user passed -m and the PR merged (in which case /pr-complete -c already closed it; agent should propose KEEP with reason "already closed by /pr-complete").
- Unrelated-findings issues raised during coding/review (track them in session state as you create them) — role:
unrelated-finding. ALWAYS KEEP unless closed by -fix (the auto-fix step closes the ones it fixed and links the fix PR; the audit leaves those closed and keeps every still-open one).
- Review-fix issue (if review fixes were delegated) — role:
fix. Sonnet should propose CLOSE if the fix-delegation agent merged its fixes successfully.
agent-found issues closed by -fix (if the auto-fix step ran) — already closed by this session; the audit confirms KEEP-as-closed.- Mac-handoff resources (only if
DEFER_MAC fired, per web/mac-handoff.md) — the mac-labeled issue raised after a -m merge (case A), or the original issue + root PR flagged without -m (case B) — role: mac-deferred, keep-open: true. The audit must never auto-close these: they are pending human verification on a Mac.
- Branches to include:
- Working branch (
$BRANCH_NAME) — role: working. Pass pr-merged: <true|false> based on whether -m resulted in a successful merge. On web (web-mode.md §5): the working branch IS the claude/* session branch ($WEB_BASE) — pass it as role: session-web with protected-session-branch: <its literal name>, never working. KEEP regardless of merge state; never delete local or remote.
agent-fix/<slug> branches (if the -fix step created any) — role: fix. Pass pr-merged: <true if the fix PR merged — always under -m, else false> so merged fix branches are cleaned up and unmerged ones (ready PRs awaiting the user) are kept.- Target branch (
$TARGET_BRANCH) — role: parent. Always KEEP (cleanup-resources protects parent roles).
- PRs to include:
- Root PR — role:
root, state from gh pr view.
-fix fix PRs (if the auto-fix step created any) — role: fix, state from gh pr view. Merged → done; ready/open → KEEP (intentional, awaiting the user when -m was not passed).
After /cleanup-resources returns its report, surface the closed/deleted/kept counts to the user. If the report has an "Ambiguous" section, list those resources verbatim and let the user decide before STOP.
This step also resolves the long-standing local-branch leftover bug: when -m was used and /pr-complete --delete-branch removed the remote, the old workflow left the local working branch behind, confusing the user. /cleanup-resources will propose deleting the dead local branch as part of its plan and the manager executes the safe git branch -d (which refuses if there are unmerged commits, so it's not destructive).
STOP — WORKFLOW ENDS HERE
After /cleanup-resources returns its report and the manager executes the safe actions, the workflow is DONE. Report the PR URL and stop.
CRITICAL RULES:
- If
-m / --merge was used and the PR was merged: You are already on the target branch (e.g., main) after the merge-mode checkout+pull. /cleanup-resources will have proposed deleting the dead local working branch — once executed, no stale local branch is left behind. Stay on the target branch. On web: you are back on $WEB_BASE (the session branch survives — the default branch is not pushable); /cleanup-resources keeps it (protected by name). See web-mode.md §5.
- Otherwise: Stay on
<BRANCH_NAME>. Do NOT checkout main, the parent branch, or any other branch. The user expects to remain on the working branch when the workflow finishes (and /cleanup-resources will have kept the branch since pr-merged is false). On web: "stay on the working branch" = stay on $WEB_BASE, the session branch. See web-mode.md §5.
- Never skip
/cleanup-resources. Even when there are no resources to close or delete, the audit run is fast and produces a paper trail. Skipping is the historical bug — do not relitigate.
- Do NOT do anything else unless the user asks.
