Skip to content

Roadmap item 6: Add inactive Codex CLI producer normalizer payload - #240

Merged
yihanzhu merged 2 commits into
mainfrom
ystack/roadmap-6/codex-cli-producer-normalizer-v1
Sep 6, 2026
Merged

Roadmap item 6: Add inactive Codex CLI producer normalizer payload#240
yihanzhu merged 2 commits into
mainfrom
ystack/roadmap-6/codex-cli-producer-normalizer-v1

Conversation

@yihanzhu

@yihanzhu yihanzhu commented Sep 5, 2026

Copy link
Copy Markdown
Owner

Summary

  • Roadmap item 6, first alternative harness. adapters/codex-cli-producer/v1/normalize.jq is the Claude Code producer normalizer with only the harness identity swapped: snapshot kind and content id, manifest id, recorded snapshot fact, adapter id, and the model provider the binding must name (openai). Every trust relation, snapshot relation, state, reason, and the generic observation are the same, so a profile can select either harness under one core contract.
  • The test proves it: the Claude program with those six tokens substituted is byte-identical to this one; the full producer contract suite passes against the new identity; a snapshot, recorded fact, or provider from the other harness is refused.
  • Payload only, no manifest. Branches from main; not stacked.

Inactive boundary

No Codex or model call, credential, network, target write, publish, or profile activation. Pure jq over the core modules.

Exact candidate

  • Head: 7d4a0e4
  • Base: main at a637451
  • Roadmap: item 6, Alternative adapters, unit 2 (alternative harness payload; eval wiring for both alternatives follows once Roadmap item 6: Add inactive GitLab forge normalizer payload #239 and this land)
  • Owned paths (4): adapters/codex-cli-producer/v1/normalize.jq (new), scripts/test/default-codex-cli-producer-adapter.test.sh (new), README.md, ci/required-files.txt

Current-head local proof

  • bash scripts/test/default-codex-cli-producer-adapter.test.sh — 46/46 checks passed
  • ShellCheck 0.11.0 — clean
  • Schema activation guard and rename gate — clean on the committed tree

Fresh CI and a fresh independent review are required for this head.

First alternative harness. adapters/codex-cli-producer/v1/normalize.jq is
the Claude Code producer normalizer with only the harness identity
swapped: the snapshot kind and content id, the manifest id, the recorded
snapshot fact, the adapter id, and the model provider the binding must
name (openai). Every trust relation, snapshot relation, state, reason,
and the generic observation are the same, so a profile can select either
harness under one core contract.

The test proves the two programs differ only in those six tokens (the
Claude program with the tokens substituted is byte-identical to this
one), runs the full producer contract suite against the new identity,
and refuses a snapshot, recorded fact, or provider from the other
harness.

Inactive and unqualified: no Codex or model call, credential, network,
target write, publish, or profile activation. No manifest ships here.

Proof: scripts/test/default-codex-cli-producer-adapter.test.sh 46/46,
shellcheck 0.11.0 clean.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Sep 5, 2026

Copy link
Copy Markdown

Deploying ystack with  Cloudflare Pages  Cloudflare Pages

Latest commit: 1e1dc98
Status: ✅  Deploy successful!
Preview URL: https://815cb4d3.fabrica-6yx.pages.dev
Branch Preview URL: https://ystack-roadmap-6-codex-cli-p.fabrica-6yx.pages.dev

View logs

@yihanzhu

yihanzhu commented Sep 5, 2026

Copy link
Copy Markdown
Owner Author

Codex reviewer (cross-vendor, read-only)

Reviewed-head: 7d4a0e4
Reviewed-base: a637451
reviewer: gpt-5.5 @ high

Posted verbatim by codex-review.sh (codex exec review --json --base refs/codex-review/240-34181/base in an isolated temp worktree, sandbox forced read-only). Comments only — Codex never pushes, approves, or merges.

The new inactive Codex CLI producer normalizer mirrors the existing Claude producer contract with the intended identity substitutions, is covered by a focused test, and is included in the required-files manifest. I did not find a discrete correctness issue introduced by this patch.

@yihanzhu

yihanzhu commented Sep 6, 2026

Copy link
Copy Markdown
Owner Author

Codex reviewer (cross-vendor, read-only)

Reviewed-head: 1e1dc98
Reviewed-base: a670fbc
reviewer: gpt-5.5 @ high

Posted verbatim by codex-review.sh (codex exec review --json --base refs/codex-review/240-80947/base in an isolated temp worktree, sandbox forced read-only). Comments only — Codex never pushes, approves, or merges.

The added Codex CLI producer normalizer mirrors the existing Claude Code producer contract with the intended identity substitutions, and the new test covers the adapter states, provenance, stale/malformed inputs, and cross-harness rejection. I did not identify a discrete correctness issue in the changed files.

@yihanzhu
yihanzhu merged commit d31d6ad into main Sep 6, 2026
2 checks passed
@yihanzhu
yihanzhu deleted the ystack/roadmap-6/codex-cli-producer-normalizer-v1 branch September 6, 2026 03:51
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