Skip to content

feat: add GitSync pull system task#442

Closed
chubes4 wants to merge 1 commit into
mainfrom
feat/gitsync-pull-system-task
Closed

feat: add GitSync pull system task#442
chubes4 wants to merge 1 commit into
mainfrom
feat/gitsync-pull-system-task

Conversation

@chubes4

@chubes4 chubes4 commented May 20, 2026

Copy link
Copy Markdown
Member

Summary

  • Adds a gitsync_pull Data Machine system task that delegates to the existing API-first GitSync::pull() primitive.
  • Registers the task so scheduled or webhook-triggered flows can refresh GitSync bindings without shelling out to WP-CLI.
  • Documents the system-task usage in the GitSync docs.

Verification

  • php -l inc/Tasks/GitSyncPullTask.php
  • git diff --check

AI assistance

  • AI assistance: Yes
  • Tool(s): OpenCode (GPT-5.5)
  • Used for: Implemented the task wrapper and documentation based on Chris's requested docs publishing workflow; Chris remains responsible for review and testing.

@homeboy-ci

homeboy-ci Bot commented May 20, 2026

Copy link
Copy Markdown
Contributor

Homeboy Results — data-machine-code

Lint

lint — failed

  • phpstan — 3 finding(s)
  • Total: 3 finding(s)

ℹ️ Auto-fix: homeboy lint data-machine-code --path /home/runner/work/data-machine-code/data-machine-code --changed-since ca0546f --fix (or homeboy refactor data-machine-code --path /home/runner/work/data-machine-code/data-machine-code --changed-since ca0546f --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 ca0546f

Test

test — passed

ℹ️ No impacted tests found for --changed-since ca0546f
ℹ️ Run full suite if needed: homeboy test data-machine-code
Deep dive: homeboy test data-machine-code --changed-since ca0546f

Audit

audit — passed

  • test_coverage — 1 finding(s)
  • Total: 1 finding(s)

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

Tooling versions
  • Homeboy CLI: homeboy 0.189.0+fb8db38
  • Extension: wordpress from https://github.com/Extra-Chill/homeboy-extensions
  • Extension revision: 8541a487
  • Action: unknown@unknown

@chubes4

chubes4 commented May 26, 2026

Copy link
Copy Markdown
Member Author

Closing this because the architecture is pointed in the wrong direction. GitSync pull/publish should be modeled as part of the normal Data Machine flow/pipeline surface, not as a standalone scheduled system task. The low-level GitSync primitive and WP-CLI operator command are fine, but the automation/product surface should be a pipeline step/handler or flow scaffold.

@chubes4 chubes4 closed this May 26, 2026
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