code-review

Reviews code changes in an isolated subagent context and writes a structured findings report.

Updated May 3, 2017
One-click install
npx skills add https://github.com/arnonmoscona/dot_files --skill code-review-arnonmoscona
Or copy as Structured Prompt for Agent
Please help me install this Agent Skill.
Skill: code-review
Source: https://github.com/arnonmoscona/dot_files/tree/main/wsl/.claude/skills/code-review
Command: npx skills add https://github.com/arnonmoscona/dot_files --skill code-review-arnonmoscona

SYSTEM DOCUMENTATION & REQUIREMENTS

What problem does it solve? Code reviews done in the main conversation context lose objectivity and miss project-wide issues like duplication, architectural drift, and convention breaches. This Skill runs the review in a forked subagent with no conversation history, loads project conventions and task context first, and produces a persistent written report instead of ephemeral chat output. ## Core Features & Use Cases - Scoped Reviews: Review staged changes, all changed files, all Python files, a specific package, or an explicit file list, with an optional ticket ID to load task context. - Architectural Drift Analysis: For larger change sets, runs a separate pass detecting blast-radius imbalance, logical coupling via git co-change history, unassigned new files, boundary crossings, and test-cost trends. - Structured Report Output: Writes findings grouped by Critical / Major / Minor / Suggestions to a dated report file in the project's basic-memory directory, returning only a short summary to the main agent. - Use Case: After finishing a feature branch touching eight files, invoke the review with scope "changed TOO-14" to get correctness, security, and duplication findings plus an architectural drift assessment tied to the ticket's intent. ## Quick Start Ask the agent to review the staged changes with the code-review skill and write the findings report.

Frequently Asked Questions about code-review

High-intent search queries and answers about installing and using this skill.

FAQPage Schema
How do I review only staged git changes with an AI agent?

Pass the scope argument "staged" and the skill resolves the file list using git diff --cached --name-only. It confirms the resolved files with you before starting the review.

What does an automated code review check for?

It checks correctness bugs, edge cases, duplication across the whole project, unnecessary complexity, efficiency problems, security issues like injection and credential handling, and adherence to project conventions read during bootstrap.

Can a code review detect architectural drift, not just bugs?

Yes, when the change set touches five or more production files or a ticket ID is given, a separate pass analyzes blast radius, logical coupling from git co-change history, unassigned new files, boundary crossings, and test-cost trends.

Where does the code review report get saved?

The full report is written to a note titled latest-code-review-report.md in the project's basic-memory directory, with the current date at the top. Only a short summary under 200 words returns to the main agent.

When should I skip the architectural drift analysis?

Skip it for small or single-file reviews, since the drift pass costs extra commands and reveals nothing useful about a two-line fix. It only runs for change sets of five or more production files or when a ticket ID is provided.