lmiranda
b302a4237d
- Fix /review command visual header: replace inline CLOSING box with skill reference (Code Reviewer uses 🔍 REVIEW, not 🏁 CLOSING) - Update CLAUDE.md version 5.4.0 → 5.8.0, fix data-platform version in plugin table (1.1.0 → 1.3.0), update Last Updated date - Add Unicode emoji to Phase Registry tables in visual-output.md (now shows 🎯 Target instead of just "Target") Items verified complete: - README.md already shows v5.8.0 - marketplace.json already shows 5.8.0 - CHANGELOG 5.8.0 entry complete (rfc-reject in both Changed/Removed) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
51 lines
1.5 KiB
Markdown
51 lines
1.5 KiB
Markdown
---
|
|
description: Pre-sprint-close code quality review
|
|
agent: code-reviewer
|
|
---
|
|
|
|
# Code Review for Sprint Close
|
|
|
|
## Skills Required
|
|
|
|
- skills/review-checklist.md
|
|
- skills/sprint-lifecycle.md
|
|
|
|
## Purpose
|
|
|
|
Review recent code changes for quality issues before closing the sprint.
|
|
|
|
## Invocation
|
|
|
|
Run `/review` before `/sprint-close` to catch issues.
|
|
|
|
## Workflow
|
|
|
|
0. **Check Lifecycle State** - Execute `skills/sprint-lifecycle.md` check protocol. Expect `Sprint/Executing`. Set `Sprint/Reviewing` after review begins. Warn if in wrong state (allow with `--force`).
|
|
1. **Determine Scope** - Sprint files or recent commits (`git diff --name-only HEAD~5`)
|
|
2. **Read Files** - Use Read tool for each file in scope
|
|
3. **Scan for Patterns** - Check each category from review checklist
|
|
4. **Compile Findings** - Group by severity (Critical, Warning, Recommendation)
|
|
5. **Report Verdict** - READY / NEEDS ATTENTION / BLOCK
|
|
|
|
## Review Categories
|
|
|
|
See `skills/review-checklist.md` for complete patterns:
|
|
- Debug artifacts (TODO, console.log, commented code)
|
|
- Code quality (long functions, deep nesting, duplication)
|
|
- Security (hardcoded secrets, SQL injection, disabled SSL)
|
|
- Error handling (bare except, swallowed exceptions)
|
|
|
|
## DO NOT
|
|
|
|
- Rewrite or refactor code automatically
|
|
- Make changes without explicit approval
|
|
- Review files outside sprint/change scope
|
|
- Spend excessive time on style issues
|
|
|
|
## Visual Output
|
|
|
|
See `skills/visual-output.md`. This command invokes the **Code Reviewer** agent:
|
|
- Phase Emoji: 🔍
|
|
- Phase Name: REVIEW
|
|
- Context: Sprint Name
|