Skip to content

refactor: consume public run artifact ability - #1008

Merged
chubes4 merged 1 commit into
mainfrom
refactor-1006-public-run-artifacts
Aug 6, 2026
Merged

refactor: consume public run artifact ability#1008
chubes4 merged 1 commit into
mainfrom
refactor-1006-public-run-artifacts

Conversation

@chubes4

@chubes4 chubes4 commented Aug 6, 2026

Copy link
Copy Markdown
Member

Summary

  • replace direct Data Machine artifact and job-storage access with datamachine/get-run-artifacts
  • preserve normalized artifacts, egress policy, and downstream PR/bundle behavior through one DMC-owned result contract
  • distinguish unavailable integration, invalid job, missing run, unauthorized access, owner error, malformed response, valid empty, and populated results
  • add architecture coverage preventing artifact-storage internals from returning

Verification

  • php tests/run-artifact-public-ability.php
  • php tests/run-artifact-repository-boundary.php
  • php tests/run-artifact-architecture.php
  • changed-file PHPCS and PHPStan level 7 via homeboy review lint --changed-only
  • Homeboy architecture audit
  • complete standalone tests/*.php suite: all pass except pre-existing bootstrap failure tracked in test: load GitCheckout in worktree cleanup patch equivalence smoke #1007
  • WP-CLI workspace command registration and managed worktree show smokes

Depends on Data Machine PR Extra-Chill/data-machine#3060.

Closes #1006

@chubes4
chubes4 merged commit 206840b into main Aug 6, 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.

refactor: consume Data Machine run artifacts through a public contract

1 participant