lmiranda
c62e0dbd2c
- Add review.md command for pre-sprint-close code quality review - Add code-reviewer.md agent for structured review workflow - Covers debug artifacts, code quality, security, error handling - Integrates with projman sprint context when available - Provides structured output with severity levels Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2.3 KiB
2.3 KiB
name, description
| name | description |
|---|---|
| review | Pre-sprint-close code quality review |
Code Review for Sprint Close
Review the recent code changes for quality issues before closing the sprint.
Review Checklist
Analyze the changes and report on:
1. Debug Artifacts
- TODO/FIXME comments that should be resolved or converted to issues
- Console.log, print(), debug statements left in code
- Commented-out code blocks
2. Code Quality
- Functions exceeding 50 lines (complexity smell)
- Deeply nested conditionals (>3 levels)
- Duplicate code patterns
- Missing docstrings/comments on public functions
3. Security Scan (Lightweight)
- Hardcoded strings that look like secrets (API keys, passwords, tokens)
- SQL strings with concatenation (injection risk)
- Disabled SSL verification
- Overly permissive file permissions in code
4. Error Handling
- Bare except/catch blocks
- Swallowed exceptions (catch with pass/empty block)
- Missing null/undefined checks on external data
Output Format
Provide a structured report:
## Sprint Review Summary
### Critical Issues (Block Sprint Close)
- [file:line] Description
### Warnings (Should Address)
- [file:line] Description
### Recommendations (Nice to Have)
- [file:line] Description
### Clean Files
- List of files with no issues found
Scope
If sprint context is available from projman, limit review to files touched in current sprint. Otherwise, review staged changes or changes in the last 5 commits.
How to Determine Scope
- Check for sprint context: Look for
.projman/current-sprint.jsonor similar - Fall back to git changes: Use
git diff --name-only HEAD~5or staged files - Filter by file type: Focus on code files (.py, .js, .ts, .go, .rs, etc.)
Execution Steps
- Determine scope (sprint files or recent commits)
- For each file in scope:
- Read the file content
- Scan for patterns in each category
- Record findings with file:line references
- Compile findings into the structured report
- Provide recommendation: READY / NEEDS ATTENTION / BLOCK
Do NOT
- Rewrite or refactor code automatically
- Make changes without explicit approval
- Review files outside the sprint/change scope
- Spend excessive time on style issues (assume formatters handle this)