← ClaudeAtlas

vibe-reviewlisted

Evidence-based code review skill — mandatory gate after every phase completes. Triggers on "review:" prefix, "review the code", "code review", "audit the codebase", "check code quality", "review phase N", "is the code ready", "quality check". Mandatory gate: Phase N cannot proceed to Phase N+1 without review passing with no P0 issues. Final phase review blocks deploy until all P0 and P1 issues are resolved. Acts as Senior Engineer, Architect, and Code Quality Auditor. Gracefully handles missing ARCHITECTURE.md — reviews against PLAN.md patterns instead. Automated checks adapt to project stack (not npm-only). Every finding backed by file path and line number. No vague feedback.
aakashdhar/vibe-skill · ★ 7 · Code & Development · score 68
Install: claude install-skill aakashdhar/vibe-skill
# Vibe Review Skill Mandatory quality gate after every phase. Acts as **Senior Engineer, Architect, and Code Quality Auditor**. Evidence-based. Every finding backed by file path and line number. **Always run in Plan Mode (Shift+Tab). Never writes feature code.** --- ## The O'Reilly principle this enforces Issues caught at Phase 1: cheap — fix once, never propagates. Issues caught at Phase 3: expensive — retrofit across every feature. Issues caught at deploy: very expensive — production risk. Without a review gate, bad patterns from Phase 1 get copied into every feature. Architecture drifts silently. Duplication builds. Quality degrades. --- ## When this skill runs **Mandatory — phase gates:** ``` Phase 1 complete → review: phase 1 → gate passed? → Phase 2 begins Phase 2 complete → review: phase 2 → gate passed? → Phase 3 begins Final phase → review: final → gate passed? → deploy unlocked ``` **On demand:** ``` review: phase 2 invoice management review: audit the Supabase integration review: full codebase before new team member ``` **Gate rules:** - P0 → blocks progression, fix tasks in TASKS.md immediately - P1 → logged to backlog.md, must resolve before deploy - Final phase: P0 AND P1 both must be zero before deploy - No P0s → gate passes, next phase begins --- ## Documents this skill reads Read in this order. Note if any are missing. 1. `vibe/ARCHITECTURE.md` — primary reference for drift detection 2. `vibe/CODEBASE.md` — exact file paths as-built 3. `v