bro-review-before-i-screw-up

Complete Pre-Commit Review

Exhaustively analyze all pending changes before committing. Detect critical issues, evaluate code quality and suggest concrete improvements. Multi-language support.

Step 1: Identify Pending Changes

Run these commands to get the current state:

# Modified files (tracked)
git diff --name-only HEAD

# New files (untracked)
git ls-files --others --exclude-standard

# Staged files
git diff --cached --name-only

If there are no changes

If all commands return empty, stop and respond:

✅ Clean working directory

No pending changes to review. Nothing to audit.

Don't continue if there are no files to review.

Step 2: Get Complete Context

# Full diff of unstaged changes
git diff HEAD

# Diff of staged changes (if any)
git diff --cached

# See project structure for context
ls -la

# Detect stack
cat package.json pyproject.toml go.mod Cargo.toml composer.json Gemfile 2>/dev/null

# Project standards
cat CLAUDE.md AGENTS.md 2>/dev/null

If CLAUDE.md, AGENTS.md, README.md or project configuration files (.eslintrc, tsconfig.json, etc.) exist, read them to understand project standards.

Step 3: Analysis by Categories

Review each modified file looking for problems in these categories, ordered by priority:

---

P0 - CRITICAL: Bugs and Logic Errors

Problems that will cause production failures:

• Uninitialized variables or access to null/undefined properties
• Impossible conditions or inverted logic
• Off-by-one errors in iterations and boundaries
• Race conditions in asynchronous code
• Uncaught exceptions that can crash the application
• Memory leaks or unreleased resources (connections, listeners, timers)
• Incorrect types that will pass at runtime but fail
• Inconsistent states that corrupt data

Examples by language:

Language	Problem	Example
JS/TS	null/undefined without check	user.profile.name without optional chaining
Python	Mutable default args	def f(lst=[])
Go	Ignored error	result, _ := fn()
Rust	unwrap() in production	value.unwrap()
PHP	Weak comparison	$a == "0" vs $a === "0"