Skip to content

[codex] fix artifact registration payload#269

Merged
hopeatina merged 1 commit intomainfrom
codex/fix-artifact-registration-payload
Apr 3, 2026
Merged

[codex] fix artifact registration payload#269
hopeatina merged 1 commit intomainfrom
codex/fix-artifact-registration-payload

Conversation

@hopeatina
Copy link
Copy Markdown
Contributor

Summary

  • stop sending top-level initiative_id on artifact create payloads for both the canonical client contract and the legacy fallback path
  • keep initiative/workstream scope in artifact metadata and entity linkage instead of the rejected top-level field
  • add regression coverage for the payload contract and update the lifecycle assertion that encoded the old shape

Why

The plugin-side artifact registration path still emitted initiative_id on artifact create requests. Current OrgX artifact writes reject that field for entity type artifact, which forced registrations into the local outbox fallback even when connectivity was healthy.

Impact

Artifact registration now stays on the normal upstream path for initiative-scoped artifacts instead of degrading to deferred local buffering.

Test plan

  • npm run typecheck
  • npm run build
  • npm run test:file -- tests/artifacts/register-artifact-client-contract.test.mjs tests/http/autopilot-slice-lifecycle.test.mjs
  • npm run test:hooks (known unrelated local baseline failure remains in tests/http/workspace-scope-live-surfaces.test.mjs, specifically workspace scope filters legacy live activity pages)

@hopeatina hopeatina merged commit 21d582e into main Apr 3, 2026
0 of 3 checks passed
@hopeatina hopeatina deleted the codex/fix-artifact-registration-payload branch April 3, 2026 02:16
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