-
Notifications
You must be signed in to change notification settings - Fork 0
Step 5 Review AI
Command: /review-ai [files or diff] · Habit: H4 Think Win-Win · Previous: Step 4 · Build Brief · Next: Step 6 · Deploy Guide
Audit AI-generated code before commit. This is the non-negotiable step — never skip it, no matter how small the change feels.
- Always — after any code generation, before any commit
- Before opening a PR
- Before merging someone else's AI-generated contribution
- Never. Every other step has legitimate skip cases. Review does not.
- Get the diff:
git diff --name-only HEADandgit diff HEAD - Review against the brief from Step 4 — did the implementation match?
- Check for AI hallucinations: nonexistent APIs, wrong function signatures, fabricated imports
- Security pass: secrets, injection, auth bypass, unsafe deserialization, SSRF
- Error handling: null/empty inputs, permission denied, partial failures
- Scope creep: did Claude add features nobody asked for?
- Test coverage: are new code paths tested?
Findings categorized CRITICAL · HIGH · MEDIUM · LOW with actionable fixes. Fix all CRITICAL and HIGH before committing.
Run these in addition to /review-ai for deeper coverage:
-
/security-check— focused OWASP Top 10 -
/cross-verify— 17-question 8-Habit checklist -
/whole-person-check— Body/Mind/Heart/Spirit balance
-
Expects: Implemented code + brief from
/build-brief -
Produces for
/deploy-guide: Reviewed, fix-applied code ready to deploy
"Does this interaction leave the next developer better informed and more capable?"
Source of truth: this wiki is generated from docs/wiki/. Edits made through the GitHub Wiki web UI may be overwritten by the next sync. To change a page, open a PR against the repository source file.
Repository · Issues · README · License
Workflow discipline for AI-assisted development
Start
Workflow
- Overview
- 0 · Research
- 1 · Requirements
- 2 · Design
- 3 · Breakdown
- 4 · Build Brief
- 5 · Review AI
- 6 · Deploy Guide
- 7 · Monitor Setup
Operations
Reference
- Habits Reference
- Maturity Model
- Architecture
- Limitations
- Vibe Coding vs Structured
- Harness Engineering
Project