review-code

/review-code — read the rules, fix the changed files

This command reads every relevant code rule and best practice, finds what changed, and edits those files until they comply. Unlike /scan and /review-changes (read-only reports), /review-code applies the fixes.

Scope: $ARGUMENTS (default: uncommitted changes + branch diff vs base).

Procedure

1. Load the rules (read, don't skim)

Read the full enforceable rule set before touching code:

• ai_rules/rules/frequent_rules.md ← the most-violated rules + pre-commit checklist (READ FIRST)
• ai_rules/globalRules.md (routing) and ai_rules/rules/best-practices.md
• Then, per changed-file category, load only the matching packs (same mapping as .claude/skills/best-practices-scan/SKILL.md §2):
  • screens/components → design-system.md (+ Callstack RN: lists, re-renders, animations)
  • hooks → best-practices.md (+ Callstack RN: atomic-state, profile-react)
  • store → state-rtk.md · services → best-practices.md (+ Callstack RN: memory-leaks)
  • theme/tokens → design-system.md §10 · tests → testing.md
• Best-practices source: Callstack react-native-best-practices skill at ~/.claude/plugins/marketplaces/callstack-agent-skills/skills/react-native-best-practices/references/

2. Gather scope

Same scoping as the scan skill:

BASE=$(git remote show origin 2>/dev/null | awk '/HEAD branch/{print $NF}')   # often development or main
git diff --name-only "${BASE:-main}...HEAD"
git status --porcelain | awk '{print $2}'

Combine + dedupe. Exclude node_modules/, ios/, android/, dist/, coverage/, .expo/, lock files, generated files. Overrides: staged → git diff --cached; session → files touched this session; file:<path> → one file; branch → branch diff only; all → full tree (slow, ask first). If scope is empty → say "no changes to review" and stop.

3. Detect, then FIX (surgical only)

For each changed file, find violations (static greps from the scan skill §3 + AI-judgment §4) and edit the file to fix them. Touch only what the rule requires — do not reformat or refactor unrelated lines. Apply at minimum: