Skip to content

Phase 1: Codex native queue delivery - #1846

Closed
khaliqgant wants to merge 3 commits into
feat/native-delivery-phase-0-seamfrom
feat/native-delivery-phase-1-codex-queue
Closed

khaliqgant wants to merge 3 commits into
feat/native-delivery-phase-0-seamfrom
feat/native-delivery-phase-1-codex-queue

Conversation

@khaliqgant

@khaliqgant khaliqgant commented Sep 22, 2026

Copy link
Copy Markdown
Member

Stacked on #1825.

Phase 1 adds native Codex delivery through codex queue, persistent route/settlement handling, self-registration for already-running Codex threads, mutation-backed seam invariants, and an unlaunched-session E2E gate.

Current evidence on finn-mini:

  • parity gates green
  • real Codex evaluation gate green
  • unlaunched Codex E2E 7/7 green
  • Rust clippy/tests and TypeScript build green

Still pending before ready-for-review:

  • fresh full 71/74-step dual-provider campaign as applicable
  • independent Claude and Codex sealed signoff
  • final commit-if-green normalization

Claude CLI capacity is currently the blocking external dependency; keep this PR in draft until final acceptance.

Review in cubic

Stacked on PR #1825. Deterministic parity, real Codex evaluation, and unlaunched-session E2E are green on finn-mini. Dual-provider final signoff remains pending Claude CLI capacity.
@coderabbitai

coderabbitai Bot commented Sep 22, 2026

Copy link
Copy Markdown
Contributor

Important

Draft PR not reviewed

Draft PRs are not automatically reviewed by default.

  • Trigger a manual review

To automatically review draft PRs, update your CodeRabbit configuration:

reviews:
  auto_review:
    drafts: true

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@khaliqgant khaliqgant closed this Sep 23, 2026
@khaliqgant
khaliqgant deleted the feat/native-delivery-phase-1-codex-queue branch September 23, 2026 06:56
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.

2 participants