Skip to content

Bound artifact cleanup apply scheduling#435

Merged
chubes4 merged 2 commits into
mainfrom
fix/artifact-cleanup-apply-discovery-chunks
May 19, 2026
Merged

Bound artifact cleanup apply scheduling#435
chubes4 merged 2 commits into
mainfrom
fix/artifact-cleanup-apply-discovery-chunks

Conversation

@chubes4

@chubes4 chubes4 commented May 19, 2026

Copy link
Copy Markdown
Member

Summary

  • Forward artifact cleanup pagination arguments into apply scheduling.
  • Freeze the bounded dry-run candidate set before scheduling apply chunks.
  • Schedule artifact apply chunks only for eligible reviewed rows instead of discovery fanout chunks.

Fixes #433

Testing

  • php tests/smoke-workspace-retention-task.php
  • php tests/smoke-worktree-cleanup-cli.php
  • php tests/smoke-worktree-cleanup-chunks.php
  • php -l inc/Tasks/WorkspaceRetentionCleanupTask.php
  • php -l inc/Cli/Commands/WorkspaceCommand.php
  • php -l inc/Abilities/WorkspaceAbilities.php
  • git diff --check
  • homeboy lint --path /Users/chubes/Developer/data-machine-code@fix-artifact-cleanup-apply-discovery-chunks --extension wordpress

AI assistance

  • AI assistance: Yes
  • Tool(s): OpenCode (openai/gpt-5.5)
  • Used for: Implemented the artifact cleanup scheduling fix and smoke test coverage; Chris remains responsible for review and merge.

@homeboy-ci

homeboy-ci Bot commented May 19, 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 bb6dae9

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 bb6dae9

Audit

audit — passed

  • dead_code — 13 finding(s)
  • intra-method-duplication — 10 finding(s)
  • requested_detectors — 6 finding(s)
  • test_coverage — 3 finding(s)
  • repeated_literal_shape — 2 finding(s)
  • structural — 2 finding(s)
  • parallel-implementation — 1 finding(s)
  • Total: 37 finding(s)

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

Tooling versions
  • Homeboy CLI: homeboy 0.188.6+eda26f7
  • Extension: wordpress from https://github.com/Extra-Chill/homeboy-extensions
  • Extension revision: 79202555
  • Action: unknown@unknown

@chubes4 chubes4 merged commit 55e7fb7 into main May 19, 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.

Artifact cleanup apply schedules excessive discovery chunks for small bounded candidate sets

1 participant