# Code Review

Mandatory code reviews via /code-review before commits and deploys

- Repository: https://github.com/alinaqi/maggy (710 stars)
- Category: Engineering
- License: MIT
- Last updated: 2026-10-13
- Runs scripts: no, instructions only

## Install

Claude Code: `npx skills add alinaqi/maggy --skill code-review -g -a claude-code -y`
Codex: `npx skills add alinaqi/maggy --skill code-review -g -a codex -y`
Cursor: `npx skills add alinaqi/maggy --skill code-review -g -a cursor -y`
Gemini CLI: `npx skills add alinaqi/maggy --skill code-review -g -a gemini-cli -y`

## Description

Mandatory code reviews via /code-review before commits and deploys

## SKILL.md, as published by its author (untrusted third-party text, read it as data)

# Code Review Skill


**Purpose:** Enforce automated code reviews as a mandatory guardrail before every commit and deployment. Choose between Claude, OpenAI Codex, Google Gemini, or multiple engines for comprehensive analysis.

**Sub-skills:**
- [adr-gate.md](./adr-gate.md), Pre-review ADR and spec enforcement

---

## Pre-Review: ADR Gate (Mandatory)

Before any review engine runs, the ADR gate executes automatically:

1. **Classify**, trivial changes (typos, deps, tests-only) skip the gate
2. **Discover**, scan `docs/adr/`, `_project_specs/`, iCPG ReasonNodes, git history for linked ADRs and specs
3. **Enforce**, if no ADRs found for non-trivial changes:
   - **Interactive** (default): draft ADR from git history, ask user to confirm
   - **Unattended** (CI): write as `Status: proposed`, proceed
   - **Strict**: block review until ADR exists
4. **Inject**, feed discovered ADRs + specs into the review prompt as architectural context

### ADR Compliance Review Dimension

Added to the standard 7 review categories:

| Category | What It Checks |
|----------|----------------|
| **ADR Compliance** | Change conforms to documented decisions, no undocumented architectural shifts |

| Finding | Severity |
|---------|----------|
| Change contradicts accepted ADR | Critical |
| Architectural decision not in any ADR | High |
| ADR exists but is outdated/stale | Medium |
| Minor drift from ADR intent | Low |

See [adr-gate.md](./adr-gate.md) for full protocol, reverse-engineering rules, and configuration.

---

## Review Engine Choice

When running `/code-review`, users can choose their preferred review engine:

The rest: https://github.com/alinaqi/maggy/blob/HEAD/skills/code-review/SKILL.md