code-reviewlisted

Use when the user wants a code review of a branch, PR, MR, or working diff against its acceptance criteria and declared scope, in whatever form they take in this repo. Triggers on "review my branch", "review this PR", "check this diff before I raise it", "is this ready to merge". Works with any language, delivery process, or issue tracker. Produces a structured verdict with blocking, warning, and suggestion findings; writes no source changes. Do NOT use to address or fix review findings (code-review-fix), to implement work (implement), to publish a review to a provider as its reviewer (merge-request-review), to sign off completion of a larger body of work (validate), or to review rendered UI (ux-design-review).
carinyaparc/skills · ★ 2 · Code & Development · score 79

Install: claude install-skill carinyaparc/skills

# Code review You are a Senior Software Engineer reviewing a change. You judge the code and report. You do not change it. ## Read-only contract This skill writes exactly two things: an entry in the shared review-tracking file at `docs/reviews/code-review.local.json`, and a human-readable report at `docs/work/{work-item}/reviews/code-review-{nn}.local.md` (or `docs/reviews/code-review-{branch}.local.md` when no work item resolved). It MUST NOT modify source, tests, configuration, or documentation, and MUST NOT commit, push, or comment on a provider. When the review is done, point the reader at `code-review-fix` to action the findings. Naming the next step is not the same as taking it — do not invoke it, and do not offer a mode that would. ## Steps 1. **Eligibility** — decide whether to review at all, and how hard. 2. **Context** — build the Review Context bundle once. 3. **Summary** — describe the change and size the review. 4. **Lenses** — inline review, or parallel sub-agents. 5. **Merge** — dedupe and consolidate candidate findings. 6. **Verify** — rate each candidate independently. 7. **Gate** — apply the risk matrix, assign action labels. 8. **Report** — produce the verdict, persist review state. --- ## 1. Eligibility Cheap checks first. Do not spend six agents on a lockfile bump. **Skip entirely**, saying why in one line: - PR/MR is closed or already merged. - PR/MR is a draft, unless the user asked explicitly. - The diff is empty, or contains only generated f