Skip to content

Fix cleanup run scheduling context#468

Merged
chubes4 merged 2 commits into
mainfrom
fix/cleanup-run-id
May 29, 2026
Merged

Fix cleanup run scheduling context#468
chubes4 merged 2 commits into
mainfrom
fix/cleanup-run-id

Conversation

@chubes4
Copy link
Copy Markdown
Member

@chubes4 chubes4 commented May 29, 2026

Summary

  • Pass the active agent slug when DMC schedules workspace cleanup runs so Data Machine accepts the queued system task.
  • Treat empty scheduler results as explicit failures instead of returning unusable cleanup-run-0 IDs.
  • Add smoke coverage for agent-context scheduling and empty-schedule error handling.

Testing

  • php -l inc/Abilities/WorkspaceAbilities.php
  • php tests/smoke-workspace-cleanup-run-context.php
  • php tests/smoke-worktree-cleanup-cli.php

AI assistance

  • AI assistance: Yes
  • Tool(s): OpenCode (GPT-5.5)
  • Used for: Diagnosed the cleanup scheduling failure, drafted the code/test changes, and ran local verification. Chris reviewed/approved the release and merge path.

@homeboy-ci
Copy link
Copy Markdown
Contributor

homeboy-ci Bot commented May 29, 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 30d9ff3

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

Test

test — passed

ℹ️ Set --setting require_phpunit_tests=true when this component is expected to contain PHPUnit tests.
ℹ️ 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 30d9ff3

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

Audit

audit — passed

  • dead_code — 5 finding(s)
  • intra-method-duplication — 5 finding(s)
  • repeated_literal_shape — 1 finding(s)
  • requested_detectors — 1 finding(s)
  • structural — 1 finding(s)
  • Total: 13 finding(s)

Deep dive: homeboy audit data-machine-code --changed-since 30d9ff3

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/26642489823
Tooling versions
  • Homeboy CLI: homeboy 0.204.1+f96bc59
  • Extension: wordpress from https://github.com/Extra-Chill/homeboy-extensions
  • Extension revision: a2ee1909
  • Action: unknown@unknown

@chubes4 chubes4 merged commit bbe59dd into main May 29, 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