Skip to content

fix: skip unused worker status checks#2360

Merged
chubes4 merged 6 commits into
mainfrom
fix-job-worker-skip-unused-status
May 29, 2026
Merged

fix: skip unused worker status checks#2360
chubes4 merged 6 commits into
mainfrom
fix-job-worker-skip-unused-status

Conversation

@chubes4
Copy link
Copy Markdown
Member

@chubes4 chubes4 commented May 29, 2026

Summary

  • Avoid computing the expensive worker status snapshot unless --stop-on-pending-actions is enabled.
  • Keeps job-mode workers focused on claiming/draining jobs during catch-up runs.

Tests

  • php tests/worker-cli-smoke.php

AI assistance

  • AI assistance: Yes
  • Tool(s): OpenCode (GPT-5.5)
  • Used for: Identified the hot-path status overhead, drafted the code/test change, and ran smoke verification.

@homeboy-ci
Copy link
Copy Markdown
Contributor

homeboy-ci Bot commented May 29, 2026

Homeboy Results — data-machine

Lint

lint — failed

  • formatting — 1 finding(s)
  • yoda — 1 finding(s)
  • Total: 2 finding(s)

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

Artifacts and drill-down
  • CI results artifact: homeboy-ci-results-data-machine-lint-quality-Linux-node24 contains immediate command JSON for this action invocation.
  • Observation artifact: homeboy-observations-data-machine-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/actions/runs/26637913464

Test

test — passed

  • 865 passed
  • 3 skipped

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

Artifacts and drill-down
  • CI results artifact: homeboy-ci-results-data-machine-test-quality-Linux-node24 contains immediate command JSON for this action invocation.
  • Observation artifact: homeboy-observations-data-machine-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/actions/runs/26637913464

Audit

audit — passed

  • dead_code — 158 finding(s)
  • requested_detectors — 150 finding(s)
  • intra-method-duplication — 66 finding(s)
  • repeated_literal_shape — 13 finding(s)
  • field_patterns — 9 finding(s)
  • structural — 6 finding(s)
  • Taxonomy — 5 finding(s)
  • dead_guard — 5 finding(s)
  • Abilities — 4 finding(s)
  • Flow — 4 finding(s)
  • Total: 431 finding(s)

Deep dive: homeboy audit data-machine --changed-since 424fe47

Artifacts and drill-down
  • CI results artifact: homeboy-ci-results-data-machine-audit-quality-Linux-node24 contains immediate command JSON for this action invocation.
  • Observation artifact: homeboy-observations-data-machine-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/actions/runs/26637913464
Tooling versions
  • Homeboy CLI: homeboy 0.204.0+fe10a555
  • Extension: wordpress from https://github.com/Extra-Chill/homeboy-extensions
  • Extension revision: 55b43c47
  • Action: unknown@unknown

@chubes4 chubes4 merged commit 30f47a9 into main May 29, 2026
4 of 5 checks passed
@chubes4 chubes4 deleted the fix-job-worker-skip-unused-status branch May 29, 2026 18:25
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