Prompt file imported from grimm00/work-prod (
.cursor/commands/fix-review.md). Copyright stays with the author.
Fix Review Command
Review old deferred issues to identify candidates for addressing. Helps prioritize which old issues are worth fixing now.
Workflow Overview
When to use:
- After several phases/PRs completed
- When looking for quick wins or accumulated issues
- Before starting new feature work
- To clean up technical debt
Key principle: Review deferred issues periodically to identify opportunities for improvement and technical debt reduction.
Usage
Command: @fix-review [options]
Examples:
@fix-review- Review all old deferred issues@fix-review --priority MEDIUM- Review only MEDIUM priority issues@fix-review --min-age 30- Review issues older than 30 days@fix-review --accumulated- Find issues that have accumulated (similar issues)@fix-review --quick-wins- Find LOW effort issues (quick wins)
Options:
--priority LEVEL- Filter by priority (CRITICAL, HIGH, MEDIUM, LOW)--min-effort LEVEL- Filter by minimum effort (LOW, MEDIUM, HIGH)--min-age DAYS- Only review issues older than N days--accumulated- Find issues that appear multiple times (similar issues)--quick-wins- Find LOW effort issues that can be fixed quickly--by-pr- Group issues by PR number--by-file- Group issues by affected file
Step-by-Step Process
1. Scan All Deferred Issues
Sources:
- Fix tracking document:
docs/maintainers/planning/features/projects/fix/README.md(main hub) - PR hub files:
docs/maintainers/planning/features/projects/fix/pr##/README.md - Sourcery review files:
docs/maintainers/feedback/sourcery/pr##.md - Individual fix plans:
docs/maintainers/planning/features/projects/fix/pr##/batch-*.mdorpr##/issue-*.md - Archived fixes:
docs/maintainers/planning/features/projects/fix/archived/pr##/
Extract:
- Issue ID (PR##-#N)
- Priority, Impact, Effort
- Description
- File location
- Age (days since PR merged)
- Status (deferred, not fixed, etc.)
Checklist:
- All deferred issues identified
- Age calculated for each issue
- Status verified (not already fixed)
2. Filter and Group Issues
Apply filters:
- Priority filter (if specified)
- Effort filter (if specified)
- Age filter (if specified)
- Accumulation filter (if specified)
Group by:
- By PR: All issues from same PR together
- By File: Issues affecting same file together
- By Type: Similar issues (e.g., all test improvements)
- By Priority: Highest priority first
Checklist:
- Filters applied
- Issues grouped logically
- Duplicates identified
3. Identify Candidates
Criteria for addressing:
-
Accumulated Issues
- Same issue appears multiple times
- Similar issues that can be batched
- Example: Multiple "use click.Choice" suggestions
-
Quick Wins
- LOW effort, LOW priority
- Can be fixed quickly
- Example: Style improvements, minor refactors
-
Blocking Issues
- Fixing enables other improvements
- Reduces technical debt
- Example: Code duplication that blocks refactoring
-
Old Issues
- Been deferred for a while
- Code may have changed making fix easier
- Example: Issues from early PRs
-
Related to Current Work
- Fixing while working on related code
- Reduces context switching
- Example: CLI improvements while adding CLI features
Checklist:
- Accumulated issues identified
- Quick wins identified
- Blocking issues identified
- Old issues reviewed
- Related issues noted
4. Generate Review Report
Report Format:
# Fix Review Report
**Date:** YYYY-MM-DD
**Total Deferred Issues:** [N]
**Candidates for Addressing:** [M]
---
## Summary
- **Accumulated Issues:** [X] (can batch together)
- **Quick Wins:** [Y] (LOW effort, can fix quickly)
- **Blocking Issues:** [Z] (enable other improvements)
- **Old Issues:** [W] (deferred for [N] days)
---
## Accumulated Issues
### Issue Type: [Description]
**Occurrences:** [N] times
**PRs:** #[N1], #[N2], #[N3]
**Total Effort:** [Effort Level]
**Issues:**
- PR##-#N: [Description] ([Priority], [Effort])
- PR##-#M: [Description] ([Priority], [Effort])
- PR##-#K: [Description] ([Priority], [Effort])
**Recommendation:** Batch together in single fix plan
---
## Quick Wins
| Issue | Priority | Effort | Age | Description |
|-------|----------|--------|-----|-------------|
| PR##-#N | š¢ LOW | š¢ LOW | [N] days | [Description] |
**Recommendation:** Create quick-wins batch
---
## Blocking Issues
| Issue | Priority | Effort | Blocks | Description |
|-------|----------|--------|--------|-------------|
| PR##-#N | š” MEDIUM | š” MEDIUM | [What it blocks] | [Description] |
**Recommendation:** Address before [blocked work]
---
## Old Issues (30+ days)
| Issue | Priority | Effort | Age | Description |
|-------|----------|--------|-----|-------------|
| PR##-#N | š” MEDIUM | š¢ LOW | [N] days | [Description] |
**Recommendation:** Review if still relevant
---
## Recommendations
1. **Immediate:** [Top recommendation]
2. **Next:** [Second recommendation]
3. **Future:** [Long-term recommendation]
Checklist:
- Report generated
- Candidates identified
- Recommendations provided
- Report saved for reference
5. Create Fix Plans (Optional)
If issues are worth addressing:
-
Use
/fix-plan --from-review-reportcommand:# Create batches from latest review report /fix-plan --from-review-report fix-review-report-2025-12-05.md # Create specific batch from report /fix-plan --from-review-report fix-review-report-2025-12-05.md --batch "Quick Wins" # Create only Quick Wins batch /fix-plan --from-review-report --quick-wins -
The command will:
- Parse review report recommendations
- Resolve issue details from source PRs
- Create cross-PR batches
- Update fix tracking
-
Or manually create batches:
- Group accumulated issues
- Create quick-wins batch
- Address blocking issues first
Checklist:
- Review report saved
- Fix plans created using
/fix-plan --from-review-report(if addressing) - Fix tracking updated
- Issues marked as planned
Common Scenarios
Scenario 1: Accumulated Similar Issues
Situation: Multiple PRs have similar issues (e.g., "use click.Choice")
Action:
- Identify all occurrences
- Create single batch for all
- Fix once, addresses multiple PRs
Example:
- PR #10: Use click.Choice for status
- PR #12: Use click.Choice for status
- PR #15: Use click.Choice for classification ā Batch together: "CLI validation improvements"
Scenario 2: Quick Wins
Situation: Many LOW effort, LOW priority issues accumulated
Action:
- Create quick-wins batch
- Fix all in single PR
- Clean up technical debt
Example:
- 5 LOW/LOW issues from different PRs ā Batch: "Code quality quick wins"
Scenario 3: Blocking Refactoring
Situation: Code duplication issue blocking other improvements
Action:
- Prioritize fixing duplication
- Enables cleaner refactoring
- Reduces future technical debt
Example:
- PR #8 Overall: Code duplication ā Fix before major refactoring work
Tips
When to Review
- After each phase: Quick review of new deferred issues
- Monthly: Comprehensive review of all deferred issues
- Before major refactoring: Check for blocking issues
- When looking for work: Find quick wins or accumulated issues
What to Look For
- Patterns: Similar issues across PRs
- Accumulation: Many small issues that can batch
- Dependencies: Issues that block other work
- Age: Old issues that may be easier now
Prioritization
- Blocking issues (enable other work)
- Accumulated issues (fix once, helps many)
- Quick wins (easy, builds momentum)
- Old issues (if still relevant)
Reference
Fix Tracking:
docs/maintainers/planning/features/projects/fix/README.md(main hub)docs/maintainers/planning/features/projects/fix/pr##/README.md(PR hubs)docs/maintainers/planning/features/projects/fix/archived/README.md(archive hub)
Sourcery Reviews:
docs/maintainers/feedback/sourcery/pr##.md
Related Commands:
/fix-plan- Create fix plans from review/fix-implement- Implement fix batches
Last Updated: 2025-12-04
Status: ā
Active
Next: Run periodically to identify fix opportunities