lmiranda
2e65b60725
Major refactoring of projman plugin architecture: Skills Extraction (17 new files): - Extracted reusable knowledge from commands and agents into skills/ - branch-security, dependency-management, git-workflow, input-detection - issue-conventions, lessons-learned, mcp-tools-reference, planning-workflow - progress-tracking, repo-validation, review-checklist, runaway-detection - setup-workflows, sprint-approval, task-sizing, test-standards, wiki-conventions Command Consolidation (17 → 12 commands): - /setup: consolidates initial-setup, project-init, project-sync (--full/--quick/--sync) - /debug: consolidates debug-report, debug-review (report/review modes) - /test: consolidates test-check, test-gen (run/gen modes) - /sprint-status: absorbs sprint-diagram via --diagram flag Architecture Cleanup: - Remove plugin-level mcp-servers/ symlinks (6 plugins) - Remove plugin README.md files (12 files, ~2000 lines) - Update all documentation to reflect new command structure - Fix documentation drift in CONFIGURATION.md, COMMANDS-CHEATSHEET.md Commands are now thin dispatchers (~20-50 lines) that reference skills. Agents reference skills for domain knowledge instead of inline content. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
52 lines
1.8 KiB
Markdown
52 lines
1.8 KiB
Markdown
---
|
|
description: Pre-sprint-close code quality review
|
|
agent: code-reviewer
|
|
---
|
|
|
|
# Code Review for Sprint Close
|
|
|
|
## Skills Required
|
|
|
|
- skills/review-checklist.md
|
|
|
|
## Purpose
|
|
|
|
Review recent code changes for quality issues before closing the sprint.
|
|
|
|
## Invocation
|
|
|
|
Run `/review` before `/sprint-close` to catch issues.
|
|
|
|
## Workflow
|
|
|
|
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
|
|
|
|
```
|
|
╔══════════════════════════════════════════════════════════════════╗
|
|
║ 📋 PROJMAN ║
|
|
║ 🏁 CLOSING ║
|
|
║ Code Review ║
|
|
╚══════════════════════════════════════════════════════════════════╝
|
|
```
|