review-local-branch
Installation
SKILL.md
Review the local branch (review-local-branch)
The sibling of review-pr, for work that has not become a PR yet. Same agents, same proof-of-work contract, same calibrations — the only real differences are how scope is discovered and that the code is yours.
Drafts only. This skill never commits, never pushes, never edits your working tree.
Trust model — what the sandbox is and is not here
review-pr puts an untrusted PR in a container because the code is a stranger's. Here the code is yours, already on your disk, and there is nothing to isolate it from. Be honest about that rather than implying a boundary that does not exist:
sandbox start --localregisters your checkout with no isolation.sandbox doctorwill say so.- What it still buys is uniformity: the agents speak only the CLI, so the same definitions work in both skills with no transport branch and no fallback path to fall down.
- What it also buys is a guardrail on your branch. A local sandbox is
exec=screened: commands that obviously write —git checkout,rm,pnpm install,>redirects — are rejected, so a review cannot mutate the thing it was asked to review. It is a guardrail, not a security boundary. sandbox worktreestill works. It cuts a peer worktree outside the repo, under~/.nx-sandboxes/worktrees/<id>-<agent>, so an agent proving a test can fail never touches your files or your branch — the only thing it writes into your repo is the.git/worktreesregistration, whichsandbox stopremoves.- That worktree carries your uncommitted work, not just the last commit.
git worktree add … HEADwould check out the committed state, which is not what is under review here; the CLI appliesgit diff HEADon top so the tree matches the diff the agents were given. Untracked files stay out, exactly as they do from the diff. - It is installed on creation, which on this repo is minutes rather than seconds. That cost is why agents are told to reach for one only when static reading genuinely cannot settle the question — not as a matter of course.
If you want an agent to run the repo's own build or test commands, start with --allow-exec and say so in the charter. No screen can contain those anyway — node -e is arbitrary code by construction.