lsm / neokai / 27226163813

82%

Build Type

push

github

Committed by web-flow

Commit Message

feat: alias Anthropic 1M model IDs for Codex to prevent premature prompt-too-long (#2086)

* feat: alias Anthropic 1M model IDs for Codex to prevent premature prompt-too-long

The Claude Agent SDK has a hard-coded model database. When it sees unknown
Codex IDs (gpt-5.5, gpt-5.3-codex, etc.) it falls back to ~200k context and
rejects requests at ~175k tokens — before NeoKai's compaction at 231k fires.

Fix: present Anthropic model IDs with known large context windows to the SDK:
- claude-opus-4-20250918 (1M context) for frontier models
- claude-sonnet-4-20250514 (200k context) for mini models

The bridge then maps these Anthropic IDs back to real Codex IDs via
modelAliases before forwarding to OpenAI.

Changes:
- codex-models.ts: add CODEX_TO_SDK_ANTHROPIC_MODEL and
  SDK_ANTHROPIC_TO_CODEX_MODEL mappings
- anthropic-to-codex-bridge-provider.ts: use Anthropic IDs in env vars,
  translateModelIdForSdk, modelAliases, and bridge models list
- tests: update assertions to match new Anthropic ID behavior

* fix: use SDK-recognized Opus ID and add per-session model override

Review feedback:
- Change Anthropic Opus ID from claude-opus-4-20250918 (unrecognized by
  SDK 0.2.141) to claude-opus-4-1-20250805 (known by current SDK).
- Add per-session model alias override via setSessionModelConfig() on the
  bridge server so the originally-selected Codex model is preserved upstream.
  Without this, gpt-5.3-codex and gpt-5.4 collapse to gpt-5.5 because they
  share the same Anthropic SDK alias.
- Update stale comment in context-manager.test.ts.
- Add test verifying Anthropic entries in responsesBridgeModels().
- Add test verifying per-session model override in bridge server.

* fix: key session model overrides by (sessionId, aliasModelId)

Previous override was keyed by sessionId alone, causing two bugs:
1. P2: Override applied to ALL requests in a session regardless of
   incoming model. Haiku/sonnet calls from SDK sub-agents were
   incorrectly rewritten to the se... (continued)

Coverage Stats

10123 of 14808 branches covered (68.36%)

Branch coverage included in aggregate %.

64 of 69 new or added lines in 3 files covered. (92.75%)

1 existing line in 1 file now uncovered.

86924 of 103165 relevant lines covered (84.26%)

291.47 hits per line