Skip to content

fix(models): route all chat models through OpenRouter#14

Merged
AndrewKirkovski merged 1 commit into
feat/facade-theming-e2efrom
fix/openrouter-model-routing
Jun 11, 2026
Merged

fix(models): route all chat models through OpenRouter#14
AndrewKirkovski merged 1 commit into
feat/facade-theming-e2efrom
fix/openrouter-model-routing

Conversation

@AndrewKirkovski

@AndrewKirkovski AndrewKirkovski commented Jun 10, 2026

Copy link
Copy Markdown
Contributor

Problem

Chat fails with "Error occurred: Claude API key is required". This repo is OpenRouter-only.env provides OPENAI_COMPAT_API_KEY / OPENAI_COMPAT_BASE_URL (OpenRouter), and ANTHROPIC_API_KEY / OPENAI_API_KEY are intentionally unset. But the default chat model claude-sonnet-4-6 (and claude-opus-4-6 + the gpt-5.x models) routed to the native wippy.llm.claude / wippy.llm.openai providers, which require their own keys.

Root cause: merge 2c767de ("Merge origin/master into feat/facade-theming-e2e") resolved the models/_index.yaml conflict by "took master's refreshed model set", which reverted the OpenRouter routing that commit f4fceb3 had applied.

Fix

Route all chat/LLM models through wippy.llm.openai_compat:provider with OpenRouter model ids:

model provider_model
claude-opus-4-6 anthropic/claude-opus-4.6
claude-sonnet-4-6 anthropic/claude-sonnet-4.6
claude-haiku-4-5 anthropic/claude-haiku-4.5 (already)
gpt-5.5 / gpt-5.4 / gpt-5.4-nano / gpt-5.3-codex openai/…

This repo is OpenRouter-only: .env sets OPENAI_COMPAT_API_KEY /
OPENAI_COMPAT_BASE_URL, while ANTHROPIC_API_KEY / OPENAI_API_KEY are not
set. A prior merge ("took master's refreshed model set") reverted the
Claude/OpenAI models back to the native wippy.llm.claude / wippy.llm.openai
providers, which broke chat with "Claude API key is required".

Re-route opus / sonnet / haiku / gpt-5.x to wippy.llm.openai_compat with
OpenRouter model ids (anthropic/claude-*, openai/gpt-5.*). Verified each
id against the live OpenRouter model list. text-embedding-3-small stays on
native OpenAI (OpenRouter has no embedding models).

Added a DO NOT REVERT comment documenting the OpenRouter-only constraint.
@AndrewKirkovski AndrewKirkovski merged commit c9fa82d into feat/facade-theming-e2e Jun 11, 2026
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