Skip to content

fix: harden cleanup apply resumability#619

Merged
chubes4 merged 2 commits into
mainfrom
fix/cleanup-operator-papercuts
Jun 10, 2026
Merged

fix: harden cleanup apply resumability#619
chubes4 merged 2 commits into
mainfrom
fix/cleanup-operator-papercuts

Conversation

@chubes4

@chubes4 chubes4 commented Jun 10, 2026

Copy link
Copy Markdown
Member

Summary

  • Mark DB-backed cleanup rows as applying before destructive artifact/worktree cleanup so interrupted operator runs leave visible resumable state.
  • Treat applying rows as remaining work, prepend current-run resume guidance, and cap DB-backed worktree-removal apply batches to one row.
  • Separate preview commands from destructive apply commands in remaining-work recommendations and add remediation text for skipped buckets.

Testing

  • php tests/smoke-cleanup-run-storage.php
  • php tests/smoke-worktree-cleanup-cli.php
  • php -l inc/Workspace/CleanupRunService.php
  • php -l inc/Cleanup/CleanupRemainingWorkSummary.php
  • php -l inc/Cli/Commands/WorkspaceCommand.php

AI assistance

  • AI assistance: Yes
  • Tool(s): OpenCode (GPT-5.5)
  • Used for: Drafted the cleanup resumability patch, updated smoke coverage, and ran targeted verification; Chris remains responsible for review and merge.

@homeboy-ci

homeboy-ci Bot commented Jun 10, 2026

Copy link
Copy Markdown
Contributor

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 c474155

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/27277290739

Test

test — passed

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

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/27277290739

Audit

audit — passed

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

Deep dive: homeboy audit data-machine-code --changed-since c474155

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/27277290739
Tooling versions
  • Homeboy CLI: homeboy 0.227.9+0808078
  • Extension: wordpress from https://github.com/Extra-Chill/homeboy-extensions
  • Extension revision: 8758c05d
  • Action: unknown@unknown

@chubes4 chubes4 merged commit 351ac29 into main Jun 10, 2026
5 checks passed
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