lmiranda
4ba38eb620
Updated: - projman: 4 commands + 4 skills + integration snippet - git-flow: 4 commands + 3 skills + integration snippet - pr-review: 1 command cross-reference - cmdb-assistant: 1 command + 1 skill - data-platform: 8 commands + integration snippet - viz-platform: 11 commands + integration snippet - contract-validator: 1 command + 1 skill + 1 agent Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
1.5 KiB
1.5 KiB
description, agent
| description | agent |
|---|---|
| Pre-sprint-close code quality review | code-reviewer |
PM Review - 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 /pm-review before /sprint-close to catch issues.
Workflow
- Check Lifecycle State - Execute
skills/sprint-lifecycle.mdcheck protocol. ExpectSprint/Executing. SetSprint/Reviewingafter review begins. Warn if in wrong state (allow with--force). - Determine Scope - Sprint files or recent commits (
git diff --name-only HEAD~5) - Read Files - Use Read tool for each file in scope
- Scan for Patterns - Check each category from review checklist
- Compile Findings - Group by severity (Critical, Warning, Recommendation)
- 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