← ClaudeAtlas

strict-reviewerlisted

Use when reviewing a diff, PR, or set of changes — your own or someone else's. Applies a severe-senior-engineer review checklist (root cause vs. symptom, unrequested abstractions, missing tests/checks, unverified claims) instead of a superficial pass.
beingmartinbmc/jambavan · ★ 2 · AI & Automation · score 66
Install: claude install-skill beingmartinbmc/jambavan
# Strict Reviewer Reviewing is not skimming for typos. Find what a lazy pass would miss: the wrong root cause, the abstraction nobody asked for, the claim with no evidence behind it. ## Step 1 — Get the real diff, not a guess If `jambavan_review_pack` is available, call it against the PR's base branch. It gives you touched files, indexed symbols changed in each, bounded extracted caller candidates, associated tests, past failure records mentioning the same files, and risk flags (open `// rin:` debt, no matching test). Treat graph and test associations as review leads, not proof of completeness. Otherwise: `git diff --stat` first, then the full diff only for files that matter. ## Step 2 — Root cause, not symptom - Is this a fix for the actual bug, or a patch on the path the ticket happened to name? - If a shared function was touched, were the relevant callers and public entry points checked? A guard added to one call site while an identified sibling caller stays broken is a review-blocking finding, not a nit. - Does the change explain *why*, not just *what*? A diff that only patches the symptom should be flagged even if it "works." ## Step 3 — Scope discipline Flag, don't just note, when a diff contains: - An abstraction with exactly one implementation (interface/factory for a single case). - A new dependency where the stdlib or an already-installed package would do. - Boilerplate or scaffolding nobody asked for. - Changes broader than the stated problem — "while I was