Skip to content

feat: add /speckit.cleanup command for post-implementation review #1617

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
dsrednicki wants to merge 3 commits into
base: main
Choose a base branch
from
+549 −2
Open

feat: add /speckit.cleanup command for post-implementation review #1617

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
232f7c2
feat: add /speckit.cleanup command for post-implementation review
dsrednicki Feb 17, 2026
eace261
feat: integrate cleanup phase into implementation workflow
dsrednicki Feb 17, 2026
34665bb
docs: add cleanup command to README
dsrednicki Feb 17, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
33 changes: 33 additions & 0 deletions README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -132,6 +132,14 @@ Use **`/speckit.implement`** to execute all tasks and build your feature accordi
/speckit.implement
```

### 7. Review and clean up

Use **`/speckit.cleanup`** to review all implementation changes, identify tech debt, and ensure code quality.

```bash
/speckit.cleanup
```

For detailed step-by-step instructions, see our [comprehensive guide](./spec-driven.md).

## 📽️ Video Overview
Expand Down Expand Up @@ -258,6 +266,7 @@ Essential commands for the Spec-Driven Development workflow:
| `/speckit.plan` | Create technical implementation plans with your chosen tech stack |
| `/speckit.tasks` | Generate actionable task lists for implementation |
| `/speckit.implement` | Execute all tasks to build the feature according to the plan |
| `/speckit.cleanup` | Review implementation, fix small issues, identify tech debt |

#### Optional Commands

Expand Down Expand Up @@ -614,6 +623,30 @@ The `/speckit.implement` command will:
Once the implementation is complete, test the application and resolve any runtime errors that may not be visible in CLI logs (e.g., browser console errors). You can copy and paste such errors back to your AI agent for resolution.

### **STEP 8:** Clean up and review

After implementation is complete, use the `/speckit.cleanup` command to perform a final quality review:

```text
/speckit.cleanup
```

The `/speckit.cleanup` command will:

- Review all files modified during implementation
- **Fix small issues immediately** (Scout Rule): Remove debugging artifacts (console.log, print statements), unused imports, dead code, and other mechanical issues
- **Create follow-up tasks** for medium issues: Missing error handling, code duplication, incomplete documentation
- **Generate detailed analysis** for large issues: Architectural concerns, security vulnerabilities, performance bottlenecks - each with multiple options and recommendations

The cleanup phase produces:

- **Direct fixes** committed with clear messages for small issues
- **Tech Debt Tasks section** appended to `tasks.md` for medium issues (can be addressed by re-running `/speckit.implement`)
- **`tech-debt-report.md`** for large issues requiring team discussion, with problem descriptions, impact assessments, and recommended approaches

> [!NOTE]
> The cleanup command will halt on critical security issues (hardcoded credentials, exposed secrets) and require you to fix them before proceeding.
</details>

---
Expand Down
Loading