Skip to content

Fix Codex runner variant dispatcher - #25

Merged
KeyffMS merged 2 commits into
mainfrom
fix/codex-runner-variant-dispatch
Sep 4, 2026
Merged

Fix Codex runner variant dispatcher#25
KeyffMS merged 2 commits into
mainfrom
fix/codex-runner-variant-dispatch

Conversation

@KeyffMS

@KeyffMS KeyffMS commented Sep 4, 2026

Copy link
Copy Markdown
Owner

Summary

Fixes the diagnostic runner crash from qualification run #13:

TypeError: _hook_variant() takes 3 positional arguments but 8 were given

The matrix dispatcher stores variants as tuples and expands them positionally. The hook, compaction, and subagent helpers all declared their variant fields as keyword-only, so all three families were vulnerable to the same runtime failure.

Changes

  • align _hook_variant, _compact_variant, and _subagent_variant signatures with the tuple-based dispatcher;
  • add a regression test that binds the exact dispatcher call shapes;
  • add a no-Codex dispatcher test that executes all 14 matrix entries and verifies all 14 per-variant JSON files are emitted.

No capability contract, runner policy, workflow, sandbox, or release gate behavior is changed.

@KeyffMS
KeyffMS merged commit b3a8adc into main Sep 4, 2026
7 checks passed
KeyffMS added a commit that referenced this pull request Sep 7, 2026
…rotocol (#34)

Preserve the exact source-bound full run #25 archive, provenance and capability results. Make the C08 missing-checkpoint stop and same-run repaired completion finite and fail closed on timeout, incomplete observation or changed files. Separate fresh unexecuted templates from historical qualified evidence, strengthen production provenance/product-identity checks, add scoped c08 live mode and protected real-CLI conformance. Keep product payload, C09/C10/C13 runtimes and self-hosted runner security unchanged. PR CI #118 passed all eight jobs including actual Codex 0.153.4 loopback tests; the new model-backed C08 result remains pending.
KeyffMS added a commit that referenced this pull request Sep 7, 2026
…ure (#35)

Documentation-only alignment with the existing full-archive production validator. Record the merged finite C08 repair and the automated run #26 rejected by the local initiating-actor policy before Codex. Require a fresh main/full run by an allowed account; retain runner security and distinguish offline conformance from live evidence. Preserve archived run #25 and leave all product, runtime, tests and release guards unchanged. PR CI #120 passed.
KeyffMS added a commit that referenced this pull request Sep 8, 2026
Preserve full self-hosted run #27 (34140846679) exactly, with its actual source a9cdcdc and all C01-C16 REPRODUCED. C08 now proves expected stop and repaired completion without timeout; C09/C10 regressions passed and C13 retains the explicit non-ephemeral fallback. Update current evidence/provenance and status/release documentation while keeping original #25, product/runtime/security and release guards unchanged. Strict release validation passed on the exact reviewed tree; PR CI #124 passed all eight jobs. No new live run or production tag is created.

Closes #7.
KeyffMS added a commit that referenced this pull request Sep 8, 2026
Reconcile compliance and release notes with immutable full #27 evidence, preserve #25 history and the C13 non-ephemeral limitation, and add three documentation consistency regressions. PR CI #126 passed all eight jobs. VERSION and qualified product/runtime bytes remain unchanged. Production publication still requires strict validation and a verified signed annotated tag.

Closes #17.
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