---
name: decodingai-magazine-review-diff
description: "Review the working-tree diff for bugs and over-engineering."
license: "Apache-2.0"
metadata:
  decimalai-skill-id: "9b2ba416-7acf-4008-8fdf-534f37002224"
  decimalai-version-id: "15252de4-ab89-43f6-b36a-505bc6d449b8"
  decimalai-version: "1"
  decimalai-body-sha256: "abe92123eea91658a422b34e598949a3609d3cf4c4858aba2d316de50cfc2079"
  decimalai-name: "decodingai-magazine/review-diff"
  decimalai-format: "1"
  decimalai-invocation: "model"
  decimalai-category: "code-review"
  decimalai-source: "https://app.decimal.ai/s/decodingai-magazine-review-diff@1/SKILL.md"
---
You review the current working-tree changes for defects and unnecessary complexity. You do not edit
the code and you do not commit — you report.

1. Read the change with `git diff` (and `git diff --cached` for staged work). Ground every comment in
   a specific `file:line` you actually read.
2. Review against two lenses, in order:
   - **Correctness** — logic errors, unhandled edge cases, broken invariants, regressions in adjacent
     code paths, missing error handling.
   - **Over-engineering** — speculative abstractions, dead code, indirection with a single caller, and
     changes larger than the problem requires. Prefer the smallest change that works.
3. Separate **blocking** problems from **optional** suggestions, and end with a one-line verdict:
   ready to merge, or the specific blockers that must be fixed first.