feat(templates): deliver on a branch in the current checkout, not a worktree

[protonspy]

What changed

The worktree is out of the implementation procedure. It existed for one case — several Claude Code sessions at once on one repo — and that is the user's own setup, not a step every single-session run should pay for: a directory to create, one to switch into, one to remember to remove, and a stale checkout left behind whenever a run died before the last step.

What survives is the one line the worktree was really carrying: leave the checkout back on main and clean, because that is where the next unit of work starts.

• rules/delivery.md — now Delivery — branch, PR, and the step is git switch -c <type>/<slug> in the checkout you are in. The parallel-session worktree is named once, as the user's to set up. Still 62 lines, inside its grandfathered cap.
• skills/plan-run/SKILL.md — three kickoff questions instead of four. Out go the worktree row, the in-place cost bullet, step 8 ("remove the worktree"), and the debris paragraph in Resuming, which now asks which branch to read rather than which checkout.
• commands/scc-plan-run.md, agents/code-review.md, agents/security-review.md — the two review agents' "in a worktree, read that path" guard is gone; it protected against nothing scc creates any more.
• Code — worktree leaves loopValues / checkLoopAnswers (internal/validate/plan.go) and the map brief frontmatter line (internal/cli/map.go).
• Design — orchestration.md §6 (with the removed key recorded and why), §8 and §9; plan.md, plan-format-v2.md, research.md aligned.
• Template set 14 → 15.

Compatibility

A plan that already recorded worktree: per-group keeps validating. The validator only checks keys it knows, so the key is ignored rather than rejected — it just stops appearing in map brief.

How it was verified

make check — gofmt -l, go vet, and the full suite with -race. TestPlanLoopAnswers now expects two findings; TestSetFrontmatterKeepsTheOrderAsked uses lang in place of the removed key. TestFreshArtifactsPassTheirOwnValidators and TestRulesStayShortEnoughToBePreloaded are the two gates that cover the template edits, and both pass.

🤖 Generated with Claude Code

https://claude.ai/code/session_01ECsTMymG9rUZWKdFvU6wMz