feat: add built-in /code-review workflow command - #42
Conversation
|
Thanks for this. There are two /code-review PRs open on #32 (this and #46), so I'm consolidating on one. I'm keeping #46's generator — it does real small/medium/big tier routing, dedups candidates, and takes The part worth keeping here is the tests: #46 ships none and breaks the existing One heads-up: the |
|
Superseded by #46, which includes this PR's test coverage (builtin-workflows.test.ts, 4→5 count bump) along with auto-resume and the scoped code-review generator. |
Summary
Implements upstream #32 by adding a built-in
/code-reviewworkflow command.generateCodeReviewWorkflow()with 7 parallel review angles, verification, and final report phases/code-reviewalongside the existing built-in workflow commandsCloses #32.
Review notes
Full scoped review completed before publishing. One issue was found and fixed before PR creation: the gather phase now handles
null/missing output instead of dereferencinggathered.diff.Verification
npx biome check src/code-review.ts src/builtin-commands.ts tests/builtin-commands.test.ts tests/builtin-workflows.test.tsnpm run buildnpm run test:unit -- tests/builtin-workflows.test.ts(project script ran full unit suite: 745 passing, 0 failing)