What problem does it solve? Code reviews of agent-written or human-written changes tend to be either rubber-stamps or nitpick floods, missing real bugs while drowning reviewers in style noise. This Skill runs a strict, senior-engineer-grade review that focuses on correctness, contract breaks, and structural regressions, and saves every finding with exact file:line citations and pasteable fixes. ## Core Features & Use Cases - Diff-driven review: Pulls a PR/MR diff via gh or glab, or diffs the current branch against its merge-base, then reads every changed file in full plus surrounding context. - Structured findings report: Groups findings into Issues, Rules & Conventions, Refactor Opportunities, Quick Wins, and Bigger Picture, each with verbatim current code and a compilable suggested replacement, saved to ~/.agents/code-reviews/. - Resolution gate: Ends every run with a fixed lettered disposition menu (grill on ambiguous findings, hand off to planning, apply best judgment, or do nothing) and records all decisions in the review file. - Use Case: After finishing a feature branch, ask for a review before opening the PR. The Skill audits both sides of every touched boundary, flags a contract break in your API handler with a concrete fix, and lets you approve each remediation one by one. ## Quick Start Ask the agent to review the current branch or a specific PR number, for example: review the changes on this branch and flag any real bugs or design problems.
