Review Before Shipping

Software Development with Claude Code

Dani Zysman

AI Curriculum Manager @ DataCamp

Why Review Matters

recraft: half: Developer with magnifying glass examining code on screen

AI code looks right but may have:
- Missing error handling
- Unvalidated inputs
- Hard coded values
- Edge cases that crash
- Imports that don't exist

Review catches what tests miss.

The 7-Item Checklist

layers: Secrets, Error Handling, Edge Cases, Input Validation, Test Coverage, Logging, Hallucinated Imports

Using the Code-Reviewer Agent

> @code-reviewer review reporter.py

Code Review: 5 issues found

1. SQL case-insensitivity: use COLLATE NOCASE
2. Connection cleanup: close before sys.exit()
3. NULL handling: consolidate with COALESCE()
4. Silent edge case: document or use COALESCE()
5. Lost error details: include str(e) in output

> fix the top issue

Fixed: Added COLLATE NOCASE
Artist search now matches regardless of how the name is typed.

Development Lifecycle Complete

Development lifecycle: Plan, Build, Test, Review, Ship

Let's Practice!

Software Development with Claude Code