Skip to content

Delete marked worktrees before abandoned cleanup classification#584

Merged
chubes4 merged 1 commit into
mainfrom
fix-abandoned-cleanup-delete-first
Jun 7, 2026
Merged

Delete marked worktrees before abandoned cleanup classification#584
chubes4 merged 1 commit into
mainfrom
fix-abandoned-cleanup-delete-first

Conversation

@chubes4
Copy link
Copy Markdown
Member

@chubes4 chubes4 commented Jun 7, 2026

Summary

  • Drain already-marked cleanup-eligible worktrees at the start of applied abandoned cleanup runs.
  • Treat abandoned --stage=bounded as an explicit delete-only fast path.
  • Keep the post-classifier bounded drain so newly marked rows are still removed in the same run.

Tests

  • php -l inc/Cli/Commands/WorkspaceCommand.php && php -l tests/smoke-worktree-cleanup-cli.php
  • php tests/smoke-worktree-cleanup-cli.php
  • git diff --check

AI assistance

  • AI assistance: Yes
  • Tool(s): OpenCode (GPT-5.5)
  • Used for: Drafted the delete-first cleanup orchestration change and updated smoke coverage; Chris directed the behavior and remains responsible for review/testing.

@homeboy-ci
Copy link
Copy Markdown
Contributor

homeboy-ci Bot commented Jun 7, 2026

Homeboy Results — data-machine-code

Lint

lint — passed

ℹ️ Full options: homeboy docs commands/lint
ℹ️ Save lint baseline: homeboy lint data-machine-code --baseline
Deep dive: homeboy lint data-machine-code --changed-since 5f4cfbc

Artifacts and drill-down
  • CI results artifact: homeboy-ci-results-data-machine-code-lint-quality-Linux-node24 contains immediate command JSON for this action invocation.
  • Observation artifact: homeboy-observations-data-machine-code-lint-quality-Linux-node24 contains exported Homeboy run history for deeper queries.
  • Drill-down: download the observation artifact, then run homeboy runs import <dir>, homeboy runs list, and homeboy runs findings <run-id>.
  • Artifacts are attached to the workflow run: https://github.com/Extra-Chill/data-machine-code/actions/runs/27091928923

Test

test — passed

ℹ️ Auto-fix lint issues: homeboy refactor data-machine-code --from lint --write
ℹ️ Collect coverage: homeboy test data-machine-code --coverage
ℹ️ Pass args to test runner: homeboy test -- [args]
ℹ️ Full options: homeboy docs commands/test
Deep dive: homeboy test data-machine-code --changed-since 5f4cfbc

Artifacts and drill-down
  • CI results artifact: homeboy-ci-results-data-machine-code-test-quality-Linux-node24 contains immediate command JSON for this action invocation.
  • Observation artifact: homeboy-observations-data-machine-code-test-quality-Linux-node24 contains exported Homeboy run history for deeper queries.
  • Drill-down: download the observation artifact, then run homeboy runs import <dir>, homeboy runs list, and homeboy runs findings <run-id>.
  • Artifacts are attached to the workflow run: https://github.com/Extra-Chill/data-machine-code/actions/runs/27091928923

Audit

audit — passed

  • audit — 20 finding(s)
  • Total: 20 finding(s)

Deep dive: homeboy audit data-machine-code --changed-since 5f4cfbc

Artifacts and drill-down
  • CI results artifact: homeboy-ci-results-data-machine-code-audit-quality-Linux-node24 contains immediate command JSON for this action invocation.
  • Observation artifact: homeboy-observations-data-machine-code-audit-quality-Linux-node24 contains exported Homeboy run history for deeper queries.
  • Drill-down: download the observation artifact, then run homeboy runs import <dir>, homeboy runs list, and homeboy runs findings <run-id>.
  • Artifacts are attached to the workflow run: https://github.com/Extra-Chill/data-machine-code/actions/runs/27091928923
Tooling versions
  • Homeboy CLI: homeboy 0.222.4+6a1948b
  • Extension: wordpress from https://github.com/Extra-Chill/homeboy-extensions
  • Extension revision: 7e31b663
  • Action: unknown@unknown

@chubes4 chubes4 merged commit dc57daf into main Jun 7, 2026
5 checks passed
@chubes4 chubes4 deleted the fix-abandoned-cleanup-delete-first branch June 7, 2026 12:05
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.

1 participant