code-review

Code Review Skill

Purpose

Systematically review code against the full antigravity rule set. Catches issues that linters miss: architectural violations, missing observability, business logic errors, pattern inconsistencies.

When to Invoke

• During the /audit skill (as part of parallel subagent dispatch)
• When user asks for a code review outside any workflow
• Best practice: Invoke in a fresh conversation (not the same one that authored the code) to avoid confirmation bias

Review Process

1. Scope the Review

Identify the files/features to review. Determine the review scope:

• Feature review — all files in a feature directory
• PR review — only changed files
• Full codebase audit — all features

2. Load the Rule Set

Read all applicable rules from .agents/rules/. Use rule-priority.md for severity classification.

3. Review Categories (Priority Order)

Review each file/feature against these categories, in order from rule-priority.md:

Critical (Must Fix)

• Security — injection, hardcoded secrets, broken auth
• Data loss — missing error handling on writes, no transaction boundaries
• Resource leaks — unclosed connections, missing cleanup

Major (Should Fix)

• Testability — I/O not behind interfaces, untested error paths
• Observability — missing logging on operations, no correlation IDs
• Error handling — empty catch blocks, swallowed errors
• Architecture — circular dependencies, wrong layer access

Minor (Nice to Fix)

• Pattern consistency — deviation from established codebase patterns
• Naming — unclear variable/function names
• Code organization — functions too long, mixed responsibilities

Enhancement (Backlog)

• Style — formatting issues or non-critical refactorings
• Documentation — missing comments on complex logic

4. Produce Findings

Output a structured findings document:

# Code Review: {Feature/Module Name}
Date: {date}
Reviewer: AI Agent (fresh context)

## Summary
- **Files reviewed:** N
- **Issues found:** N (X critical, Y major, Z minor, W enhancement)

## Critical Issues
- [ ] **[SEC]** {description} — [{file}:{line}](file:///path)
- [ ] **[DATA]** {description} — [{file}:{line}](file:///path)

## Major Issues
- [ ] **[TEST]** {description} — [{file}:{line}](file:///path)
- [ ] **[OBS]** {description} — [{file}:{line}](file:///path)

## Minor Issues
- [ ] **[PAT]** {description} — [{file}:{line}](file:///path)

## Enhancement Issues
- [ ] {description} — [{file}:{line}](file:///path)

## Rules Applied
List of rules referenced during this review.