Skip to content

Add native Codex session-record evidence adapter for configuration-verified routing - #45

Merged
GrillerGeek merged 5 commits into
mainfrom
copilot/add-native-codex-session-record-adapter
Sep 23, 2026
Merged

GrillerGeek merged 5 commits into
mainfrom
copilot/add-native-codex-session-record-adapter

Conversation

Copilot AI commented Sep 23, 2026 •

Copy link
Copy Markdown
Contributor

Guildhall supported configuration_verified routing but only through full app-server JSON-RPC captures, which are not exposed by Codex desktop worker dispatch. This change adds a native session-record evidence path that can qualify configuration (not execution identity) from explicitly supplied, task-owned records.

  • Native Codex evidence adapter (codex-native-session-v1)

    • Adds a new capture format in routing_evidence.py for native session records.
    • Validates worker/session/turn/response correlation, parent association, runtime-version consistency, completion matching, and substitution signals.
    • Preserves separation of concerns: requested vs configured vs observed; observed model/effort remain unknown for this lane.
  • Usage normalization and fail-closed attribution rules

    • Normalizes usage from response records and token-count snapshots without double counting.
    • Handles retries, duplicate records, multi-turn captures, and snapshot-only captures (including empty response_ids inventories).
    • Returns unknown with explicit evidence gaps on conflicts/missing identity/configuration/completion data.
  • Preflight and docs integration

    • Extends preflight record_access with codex-native-session-records mapped to candidate lane configuration_verified.
    • Updates host-evidence guidance to document supported native-record requirements and weaker guarantee boundaries.
    • Adds a redacted synthetic native capture example for reviewers/study inputs.
  • Regression coverage

    • Adds native-path tests for happy path, parent/config conflicts, snapshot-only mode, per-turn multi-snapshot/multi-turn accounting, and fail-closed behavior.
# preflight lane selection now supports native Codex session records
lane = {
    'none': None,
    'claude-owned-transcript': 'execution_observed',
    'codex-owned-jsonrpc': 'configuration_verified',
    'codex-native-session-records': 'configuration_verified',
}[packet['record_access']]

Copilot AI and others added 4 commits September 23, 2026 15:06
Co-authored-by: GrillerGeek <66818+GrillerGeek@users.noreply.github.com>
Co-authored-by: GrillerGeek <66818+GrillerGeek@users.noreply.github.com>
Co-authored-by: GrillerGeek <66818+GrillerGeek@users.noreply.github.com>
Co-authored-by: GrillerGeek <66818+GrillerGeek@users.noreply.github.com>
Copilot AI changed the title [WIP] Add native Codex session-record adapter for configuration-verified routing Add native Codex session-record evidence adapter for configuration-verified routing Sep 23, 2026
Copilot AI requested a review from GrillerGeek September 23, 2026 15:15
@GrillerGeek
GrillerGeek marked this pull request as ready for review September 23, 2026 15:49
@coldtea-pr-lens

Copy link
Copy Markdown

◈ PR Lens

Note

Copilot opened this pull request, so PR Lens left it undrawn. Comment @pr-lens draw to draw it

github.comment.notice: false in .github/pr-lens.yml turns this note off

@GrillerGeek
GrillerGeek merged commit 1c25444 into main Sep 23, 2026
3 checks passed
@GrillerGeek
GrillerGeek deleted the copilot/add-native-codex-session-record-adapter branch September 23, 2026 15:49
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.

Add native Codex session-record adapter for configuration-verified routing

2 participants