Skip to content

Fix cleanup resume limit batching#651

Merged
chubes4 merged 1 commit into
mainfrom
fix/issue-645-cleanup-resume-limit
Jun 12, 2026
Merged

Fix cleanup resume limit batching#651
chubes4 merged 1 commit into
mainfrom
fix/issue-645-cleanup-resume-limit

Conversation

@chubes4

@chubes4 chubes4 commented Jun 12, 2026

Copy link
Copy Markdown
Member

Summary

  • Fixes cleanup resume limit does not drain multiple worktree removal rows #645.
  • Lets DB-backed cleanup apply/resume pass the full remaining --limit capacity to worktree-removal direct apply instead of hard-capping each invocation to one row.
  • Adds explicit JSON evidence fields: processed, applied, skipped, and next_command.
  • Updates focused cleanup-run smoke coverage for multi-row resume while preserving limit=1 behavior.

Verification

  • composer install
  • php -l inc/Workspace/CleanupRunService.php
  • php -l tests/smoke-cleanup-run-storage.php
  • php tests/smoke-cleanup-run-storage.php
  • php tests/smoke-worktree-cleanup-remove-guard.php

AI assistance

  • AI assistance: Yes
  • Tool(s): OpenCode (openai/gpt-5.5)
  • Used for: Implemented the cleanup resume batching fix, updated focused smoke coverage, and ran targeted verification. Chris remains responsible for review and merge.

@homeboy-ci

homeboy-ci Bot commented Jun 12, 2026

Copy link
Copy Markdown
Contributor

Homeboy Results — data-machine-code

Lint

lint — failed

ℹ️ Auto-fix: homeboy lint data-machine-code --path /home/runner/work/data-machine-code/data-machine-code --changed-since f9eac72 --fix (or homeboy refactor data-machine-code --path /home/runner/work/data-machine-code/data-machine-code --changed-since f9eac72 --from lint --write)
ℹ️ Some issues may require manual fixes
ℹ️ Full options: homeboy docs commands/lint
ℹ️ Save lint baseline: homeboy lint data-machine-code --baseline
Deep dive: homeboy lint data-machine-code --changed-since f9eac72

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

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 f9eac72

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

Audit

audit — passed

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

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

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/27391997189
Tooling versions
  • Homeboy CLI: homeboy 0.228.13+ad443b1
  • Extension: wordpress from https://github.com/Extra-Chill/homeboy-extensions
  • Extension revision: 19cdd6a5
  • Action: unknown@unknown

@chubes4 chubes4 merged commit 7001ddd into main Jun 12, 2026
4 of 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.

cleanup resume limit does not drain multiple worktree removal rows

1 participant