Skip to content

feat: allow anthropic and openai api key providers - #26

Merged
icco merged 2 commits into
mainfrom
feat/anthropic-openai
Oct 17, 2026
Merged

icco merged 2 commits into
mainfrom
feat/anthropic-openai

Conversation

@icco

@icco icco commented Oct 17, 2026

Copy link
Copy Markdown
Owner

Why

Allows the `anthropic` and `openai` providers in the deny-all provider policy and passes `ANTHROPIC_API_KEY` and `OPENAI_API_KEY` through Compose.

Check the smoke test's assertion that both providers expose an API-key login method on 2.0.29.

Risks

Claude Pro and Max subscriptions are not supported; Anthropic only permits API keys in third-party tools. The default model stays on Copilot.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@icco
icco marked this pull request as ready for review October 17, 2026 12:35
@icco
icco requested a balanced review from Copilot October 17, 2026 12:35

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🔵 Needs a closer look

The API-key method assertions still need verification against OpenCode 2.0.29.

No open issues 🎉

What changed in this PR

Adds Anthropic and OpenAI to the allowed providers while keeping Copilot as the default model.

Changes:

  • Allows both providers and passes their API keys through Compose.
  • Documents authentication and extends provider-policy and login-method checks.
File Description
tests/​routing.test.mjs Checks the expanded provider allowlist.
scripts/​test-router-integration.mjs Checks API-key methods and permitted models.
README.md Documents provider login and API keys.
opencode.jsonc Allows Anthropic and OpenAI.
compose.yml Passes both API-key environment variables.

Review effort: Balanced

@icco
icco merged commit a86df02 into main Oct 17, 2026
8 checks passed
@icco
icco deleted the feat/anthropic-openai branch October 17, 2026 12:39
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