fix: use authoritative is_qa_approved() for batch cleanup

Aligns cleanup logic with status display logic. Previously, cleanup
would delete specs with qa_report.md even if not yet QA-approved,
causing unintended data loss for specs in "qa_in_progress" state.
This commit is contained in:
StillKnotKnown
2026-02-12 14:21:20 +02:00
parent 5bf4514979
commit f6f43e8fa0
+2 -2
View File
@@ -205,10 +205,10 @@ def handle_batch_cleanup_command(project_dir: str, dry_run: bool = True) -> bool
print_status("No specs directory found", "info")
return True
# Find completed specs
# Find completed specs (only QA-approved, matching status display logic)
completed = []
for spec_dir in specs_dir.iterdir():
if spec_dir.is_dir() and (spec_dir / "qa_report.md").exists():
if spec_dir.is_dir() and is_qa_approved(spec_dir):
completed.append(spec_dir.name)
if not completed: