review-loop

Review Loop — review → fix → re-review until clean

Proceed only when the surrounding user request explicitly asks for the loop (including direct /tgc-skills:review-loop invocation) or tgc-skills:ship/tgc-skills:checkpoint invokes it. It commits fixes and pushes once, so stop without that authorization.

Review the branch diff with independent agents, fix legitimate findings, and repeat until clean or a stop condition is reached. Never merge the PR.

Keep the loop out of the main context

Create STATUS=$(mktemp "${TMPDIR:-/tmp}/tgc-review-loop.XXXXXX") and tell the user that progress is visible there. Then launch one general-purpose subagent with the repo path, optional PR/branch argument, status path, and the complete workflow below. The main context manages only the dispatch and final report. Subagents may start in the background; explicitly wait for this one to finish before returning or letting a parent workflow report completion.

Subagent workflow

Setup and heartbeat

1. Resolve the local nameWithOwner, current branch, and GitHub default branch. Stop if the current branch is the default branch. Require a supplied branch argument to equal the current branch. If a PR is supplied or detected, require its URL repository to match nameWithOwner and its headRefName to match the current branch. Stop on any mismatch. Use its baseRefName; without a PR, use the default branch.

2. Require a clean working tree. Stop and report staged, unstaged, or untracked files rather than silently excluding them.

3. Fetch the resolved base and review its diff consistently throughout the loop. Capture the base ref from command output into a shell variable and always reference it quoted — a valid Git ref can contain shell metacharacters ($(...), backticks), and command-substitution output is not re-evaluated, so this cannot inject commands:

   BASE_REF=$(gh pr view <pr> --json baseRefName --jq .baseRefName \
     || gh repo view --json defaultBranchRef --jq .defaultBranchRef.name)
   git fetch origin "$BASE_REF"
   LOOP_BASE=$(git rev-parse HEAD)
   git diff "origin/$BASE_REF...HEAD"
   

4. Create/truncate the passed status file outside the repo: