Skip to content

fix: add job-claiming worker mode#2359

Merged
chubes4 merged 1 commit into
mainfrom
fix/job-claiming-worker-2359
May 29, 2026
Merged

fix: add job-claiming worker mode#2359
chubes4 merged 1 commit into
mainfrom
fix/job-claiming-worker-2359

Conversation

@chubes4
Copy link
Copy Markdown
Member

@chubes4 chubes4 commented May 29, 2026

Summary

  • add wp datamachine worker run --mode=job to claim one Data Machine job at a time with per-job worker locks
  • drain claimed jobs through the existing datamachine/drain-job primitive instead of hopping across the shared queue
  • keep the existing queue/lane mode as the default for compatibility
  • expand worker CLI smoke coverage for job mode, per-job locks, and job-id extraction

Testing

  • php -l inc/Cli/Commands/WorkerCommand.php
  • php tests/worker-cli-smoke.php
  • php tests/worker-lock-smoke.php
  • homeboy lint --path /Users/chubes/Developer/data-machine@fix-job-claiming-worker-2359 --extension wordpress --file inc/Cli/Commands/WorkerCommand.php

AI assistance

  • AI assistance: Yes
  • Tool(s): OpenCode (GPT-5.5)
  • Used for: Drafted and verified the job-claiming worker mode and smoke coverage; Chris remains responsible for review and merge.

@homeboy-ci
Copy link
Copy Markdown
Contributor

homeboy-ci Bot commented May 29, 2026

Homeboy Results — data-machine

Lint

lint — passed

ℹ️ Full options: homeboy docs commands/lint
Deep dive: homeboy lint data-machine --changed-since 3d2c021

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

Test

test — passed

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

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

Audit

audit — passed

  • duplication — 1 finding(s)
  • requested_detectors — 1 finding(s)
  • Total: 2 finding(s)

Deep dive: homeboy audit data-machine --changed-since 3d2c021

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/26613388217
Tooling versions
  • Homeboy CLI: homeboy 0.201.4+9eab332a
  • Extension: wordpress from https://github.com/Extra-Chill/homeboy-extensions
  • Extension revision: 281e511d
  • Action: unknown@unknown

@chubes4 chubes4 merged commit 424fe47 into main May 29, 2026
5 checks passed
@chubes4 chubes4 deleted the fix/job-claiming-worker-2359 branch May 29, 2026 02:03
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