diff-reviewlisted

Language-agnostic structural review of a code diff for correctness bugs and design/cleanup issues, with an adversarial verification pass that suppresses false positives. Collects the diff itself via git, spawns its own reviewers, and emits a severity-ordered report. Use when you want a self-contained, high-confidence review of staged, working-tree, or branch changes before they land.
jedarden/jeds-curated-skills · ★ 0 · Code & Development · score 75

Install: claude install-skill jedarden/jeds-curated-skills

# Diff Review Skill Perform a correctness-first review of a code diff. This skill is opinionated: it biases toward a small set of high-confidence findings and runs an adversarial verifier to drop anything it cannot substantiate from the diff. A clean diff should produce few or zero findings — silence is a valid result, noise is not. ## Step 1: Collect the Diff Run the collector to resolve a base ref and produce the unified diff plus a changed-files summary: ```bash ~/.claude/skills/diff-review/scripts/collect-diff.sh [base-ref] ``` Resolution order when no `base-ref` argument is given: 1. Merge-base of `HEAD` with the default branch (`origin/HEAD`, then `main`, then `master`). 2. Staged changes (`git diff --cached`) if no upstream base resolves. 3. Working-tree changes (`git diff`) as the last fallback. If the repo has multiple plausible base branches and the choice is ambiguous, use AskUserQuestion to confirm the base ref. Otherwise proceed with the resolved default. Note the total changed-line count from the summary. If it exceeds ~800 lines, follow `~/.claude/skills/diff-review/runbooks/LARGE-DIFF.md` instead of the single-pass flow below. ## Step 2: Load Checklists Read ALL of the following checklist files from this skill's directory (`~/.claude/skills/diff-review/`). They define the review criteria: - `CHECKLIST-01-CORRECTNESS.md` — logic errors, boundaries, null handling, error paths, leaks, races, API misuse - `CHECKLIST-02-DESIGN.md` — duplication, leaky abs