Skip to content

Improve cleanup blocker follow-up reports#657

Merged
chubes4 merged 2 commits into
mainfrom
fix/issue-646-cleanup-blocker-reports-fresh
Jun 12, 2026
Merged

Improve cleanup blocker follow-up reports#657
chubes4 merged 2 commits into
mainfrom
fix/issue-646-cleanup-blocker-reports-fresh

Conversation

@chubes4

@chubes4 chubes4 commented Jun 12, 2026

Copy link
Copy Markdown
Member

Summary

  • Add bucket-specific, non-destructive follow-up commands for cleanup blockers: dirty_worktree, unpushed_commits, stale_worktree_marker, and primary_missing.
  • Surface narrow evidence commands in both task-backed cleanup status summaries and direct worktree cleanup skipped_next_commands.
  • Extend focused cleanup CLI smoke coverage for status, evidence, JSON, and human output paths.

Fixes #646

Verification

  • php -l inc/Cleanup/CleanupRemainingWorkSummary.php
  • php -l inc/Workspace/WorkspaceWorktreeCleanupEngine.php
  • php -l tests/smoke-worktree-cleanup-cli.php
  • php tests/smoke-worktree-cleanup-cli.php

Notes

  • The new recommended commands are read-only or dry-run previews by default.
  • Broader php tests/smoke-worktree-cleanup.php was not used as a gate for this slice because it currently reports duplicate skipped-row fixture assertions unrelated to these focused output changes.

AI assistance

  • AI assistance: Yes
  • Tool(s): OpenCode (openai/gpt-5.5)
  • Used for: Implementing the cleanup follow-up command mapping, updating focused smoke coverage, and running targeted verification. Chris remains responsible for review and merge.

@chubes4 chubes4 merged commit eb2d332 into main Jun 12, 2026
4 of 5 checks passed
@chubes4 chubes4 deleted the fix/issue-646-cleanup-blocker-reports-fresh branch June 12, 2026 04:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

cleanup safety buckets need operator-specific follow-up commands and reports

1 participant