Skip to content

fix(anthropic): preserve routed response model metadata - #1121

Draft
Ingwannu wants to merge 1 commit into
devfrom
agent/fix-1117-anthropic-response-model
Draft

fix(anthropic): preserve routed response model metadata#1121
Ingwannu wants to merge 1 commit into
devfrom
agent/fix-1117-anthropic-response-model

Conversation

@Ingwannu

@Ingwannu Ingwannu commented Aug 6, 2026

Copy link
Copy Markdown
Owner

Summary

  • Preserve the canonical provider-qualified Anthropic selector in Codex-facing streaming and JSON Responses while keeping the bare model id on the upstream Messages request.
  • Thread the client-facing identity through the normal Responses bridge, image bridge, and web-search loop without changing non-Anthropic or OpenAI virtual-model identities.
  • Generate hidden, marker-owned bare Anthropic compatibility catalog rows with the canonical context/compaction/capability metadata, and remove those rows safely during sync/restore.
  • Document the response-identity and cleanup invariant.

Closes #1117

Verification

  • bun run typecheck - passed on latest dev (bbd82e73).
  • bun test tests/anthropic-response-model.test.ts tests/openai-api-virtual-models.test.ts tests/server-combo-failover-e2e.test.ts tests/images/loop.test.ts tests/web-search.test.ts tests/codex-catalog.test.ts tests/codex-catalog-restore.test.ts - 291 passed, 0 failed.
  • bun run privacy:scan - passed.
  • Full suite run - 9,227 passed / 10 skipped / 2 failed out of 9,239. The two failures were unrelated local-environment cases: the shim fixture read this host's existing persisted service token instead of its temporary fixture token, and one native-profile timing test passed immediately when rerun in isolation. Required GitHub CI remains the merge authority.

Checklist

  • Scope stays focused and avoids unrelated cleanup.
  • Docs or release notes were updated when needed.
  • Security-sensitive changes were reviewed for secrets, auth, and unsafe defaults.

@coderabbitai

coderabbitai Bot commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: 292034ef-2cc8-47ad-b61a-2732cf275167

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

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.

@github-actions

github-actions Bot commented Aug 6, 2026

Copy link
Copy Markdown

Deterministic PR hygiene checks passed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant