Steps
Confirm that implementation scope, approach, and trade-offs are agreed in the conversation. A plan approved through the
issueskill counts as agreed. If agreement is missing, run thegrill-meskill and reach it before continuing.Branch check: ensure you are in a Claude Code–managed worktree on a topic branch (Branch & Worktree Policy in the root
CLAUDE.md). Fresh worktree setup:yarn install, thenNX_WORKSPACE_ROOT_PATH=<worktree-absolute-path> yarn build.Read the governing constraints before writing code: the package-level
CLAUDE.mdof every package you will touch, plus any matching.claude/rules/*.md.Implement exactly what was agreed, test-first: add failing spec tests that define the expected behavior (never throwaway reproduction scripts), then make them pass.
/code-review medium(user-executed):/code-reviewcannot be invoked by Claude. Present the following to the user, ask them to run it, and wait here for the results. Then fix all findings./code-review mediumWriting a substitute review yourself, or skipping this step because you cannot invoke it, is forbidden.
Run the
qa-engineerskill; fix all findings.Run the
product-managerskill; fix all findings (includes documentation consistency — JSDoc placement, comment policy).Run
yarn lint; fix all errors.Run
yarn test; fix all failures.Commit following the
gitskill.Create the PR following the
prskill.
Rules
- Fix-all scope: code-level fixes, added tests, and documentation fixes are applied on the spot. Findings that imply a spec change or scope expansion require the user's approval first.
- Loop back: if any step causes a major change of direction or a large code change, restart from step 5.
- No skipping steps: step 5 in particular — being un-invocable by Claude is not a reason to skip it; asking the user and waiting is the correct path. Skip a step only when the user explicitly says to.
- Strict scope: never expand beyond the agreed scope on your own.