Skip to content

Support Claude console subscription auth for model listing#451

Merged
ericdallo merged 1 commit intoeditor-code-assistant:masterfrom
bobrowadam:claude-console-auth
May 8, 2026
Merged

Support Claude console subscription auth for model listing#451
ericdallo merged 1 commit intoeditor-code-assistant:masterfrom
bobrowadam:claude-console-auth

Conversation

@bobrowadam
Copy link
Copy Markdown
Contributor

  • Dispatch x-api-key vs Authorization: Bearer when fetching the provider /models endpoint based on the stored auth :type, so Claude console subscription (token) and OAuth flows both work.

  • Thread auth-type from provider-api-key through fetch-provider-native-models instead of hardcoding :auth/oauth.

  • Add tests for both Anthropic token (x-api-key) and OAuth (Bearer) paths.

  • I added a entry in changelog under unreleased section.

  • This is not an AI slop.

@ericdallo
Copy link
Copy Markdown
Member

@bobrowadam what this fixes? what you see? can you check tests as well?

@ericdallo
Copy link
Copy Markdown
Member

I get the issue now, but can you fix the tests before I test?

@bobrowadam
Copy link
Copy Markdown
Contributor Author

@ericdallo sure will do

@bobrowadam bobrowadam force-pushed the claude-console-auth branch from ed4ea88 to c9a313b Compare May 8, 2026 21:30
Use configurable auth type from provider config (defaulting to :auth/oauth)
so that token-based and OAuth-based providers (e.g. Claude console subscription)
send the correct header (x-api-key vs Authorization: Bearer) when fetching models.

🤖 Generated with [eca](https://eca.dev)

Co-Authored-By: eca <noreply@eca.dev>
@bobrowadam bobrowadam force-pushed the claude-console-auth branch from c9a313b to b6579cb Compare May 8, 2026 21:42
Copy link
Copy Markdown
Member

@ericdallo ericdallo left a comment

Choose a reason for hiding this comment

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

Thank you!

@ericdallo ericdallo merged commit 1eca7af into editor-code-assistant:master May 8, 2026
9 checks passed
@bobrowadam bobrowadam deleted the claude-console-auth branch May 9, 2026 05:36
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