chore(web): Added option to specify bearer tokens for the anthropic sdk#968
chore(web): Added option to specify bearer tokens for the anthropic sdk#968brendan-kellam merged 4 commits intomainfrom
Conversation
WalkthroughAdds support for using bearer tokens with the Anthropic provider via a new optional Changes
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~25 minutes Possibly related PRs
Suggested labels
Suggested reviewers
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches
🧪 Generate unit tests (beta)
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. Comment |
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@CHANGELOG.md`:
- Line 11: The changelog entry for the upgrade of `@ai-sdk/anthropic` to
`^3.0.22` incorrectly links to PR `#967`; update that entry so the trailing PR
reference is `[`#968`](https://github.com/sourcebot-dev/sourcebot/pull/968)`
instead of `#967` so the release note points to the correct pull request.
In `@packages/web/package.json`:
- Line 18: The package manifest added/updated the dependency
"@ai-sdk/anthropic": "^3.0.22" but the lockfile still pins v2.0.57; regenerate
the lockfile by running your package manager to install the new version (e.g.,
run `yarn install`) so yarn.lock is updated to reflect
"@ai-sdk/anthropic"@^3.0.22 and commit the updated yarn.lock alongside the
package.json change.
There was a problem hiding this comment.
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
packages/web/src/features/chat/actions.ts (1)
761-773:⚠️ Potential issue | 🔴 CriticalRemove unsupported
authTokenparameter—not available in@ai-sdk/anthropic.The
authTokenoption is not supported by@ai-sdk/anthropic@^3.0.22. Authentication with this provider is handled exclusively viaapiKey(sent as thex-api-keyheader) or custom headers. TheauthTokenparameter exists only in the official@anthropic-ai/sdkpackage, not the AI SDK provider wrapper.Either remove the
authTokenconfiguration or switch to using@anthropic-ai/sdkdirectly if that feature is required.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@packages/web/src/features/chat/actions.ts` around lines 761 - 773, The code passes an unsupported authToken option to createAnthropic; remove the authToken branch in the createAnthropic call (references: createAnthropic, config.authToken, getTokenFromConfig) and rely on apiKey and/or headers for authentication (use await getTokenFromConfig(config.token) for apiKey or include any custom auth value in config.headers via extractLanguageModelKeyValuePairs); if authToken behavior is required, replace createAnthropic with the official `@anthropic-ai/sdk` usage instead of the `@ai-sdk/anthropic` wrapper.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Outside diff comments:
In `@packages/web/src/features/chat/actions.ts`:
- Around line 761-773: The code passes an unsupported authToken option to
createAnthropic; remove the authToken branch in the createAnthropic call
(references: createAnthropic, config.authToken, getTokenFromConfig) and rely on
apiKey and/or headers for authentication (use await
getTokenFromConfig(config.token) for apiKey or include any custom auth value in
config.headers via extractLanguageModelKeyValuePairs); if authToken behavior is
required, replace createAnthropic with the official `@anthropic-ai/sdk` usage
instead of the `@ai-sdk/anthropic` wrapper.
ℹ️ Review info
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
⛔ Files ignored due to path filters (1)
yarn.lockis excluded by!**/yarn.lock,!**/*.lock
📒 Files selected for processing (10)
docs/docs/configuration/language-model-providers.mdxdocs/snippets/schemas/v3/index.schema.mdxdocs/snippets/schemas/v3/languageModel.schema.mdxpackages/schemas/src/v3/index.schema.tspackages/schemas/src/v3/index.type.tspackages/schemas/src/v3/languageModel.schema.tspackages/schemas/src/v3/languageModel.type.tspackages/shared/src/env.server.tspackages/web/src/features/chat/actions.tsschemas/v3/languageModel.json
Co-authored-by: Brendan Kellam <brendan-kellam@users.noreply.github.com>
Co-authored-by: Brendan Kellam <brendan-kellam@users.noreply.github.com>
0529f5a to
4f1de65
Compare
There was a problem hiding this comment.
Actionable comments posted: 1
♻️ Duplicate comments (1)
CHANGELOG.md (1)
12-12:⚠️ Potential issue | 🟡 MinorFix incorrect PR reference in changelog entry.
Line 12 links to
#967, but this change is in PR#968, so the release note currently points to the wrong PR.Suggested fix
-- Added support for using bearer tokens with anthropic. [`#967`](https://github.com/sourcebot-dev/sourcebot/pull/967) +- Added support for using bearer tokens with anthropic. [`#968`](https://github.com/sourcebot-dev/sourcebot/pull/968)As per coding guidelines, entries must include the GitHub pull request id at the end of the line, formatted as
[#<id>](<url>).🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@CHANGELOG.md` at line 12, Update the changelog entry string "Added support for using bearer tokens with anthropic. [`#967`](https://github.com/sourcebot-dev/sourcebot/pull/967)" to reference the correct pull request id and URL for PR `#968` so the GitHub link at the end of that line is "[`#968`](https://github.com/sourcebot-dev/sourcebot/pull/968)"; edit the same line content to replace `#967` and its URL with `#968` and the corresponding URL.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@docs/docs/configuration/language-model-providers.mdx`:
- Around line 92-99: Fix the grammar in the Anthropic auth comments: change "a
API key" to "an API key" in the comment near the "token" property and change "a
auth token" to "an auth token" in the comment near the "authToken" property so
the comments read "Auth can be provided via an API key..." and "...or via an
auth token sent using the `Authorization: Bearer` header." Reference the "token"
and "authToken" keys when making the edits.
---
Duplicate comments:
In `@CHANGELOG.md`:
- Line 12: Update the changelog entry string "Added support for using bearer
tokens with anthropic.
[`#967`](https://github.com/sourcebot-dev/sourcebot/pull/967)" to reference the
correct pull request id and URL for PR `#968` so the GitHub link at the end of
that line is "[`#968`](https://github.com/sourcebot-dev/sourcebot/pull/968)"; edit
the same line content to replace `#967` and its URL with `#968` and the
corresponding URL.
ℹ️ Review info
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (11)
CHANGELOG.mddocs/docs/configuration/language-model-providers.mdxdocs/snippets/schemas/v3/index.schema.mdxdocs/snippets/schemas/v3/languageModel.schema.mdxpackages/schemas/src/v3/index.schema.tspackages/schemas/src/v3/index.type.tspackages/schemas/src/v3/languageModel.schema.tspackages/schemas/src/v3/languageModel.type.tspackages/shared/src/env.server.tspackages/web/src/features/chat/actions.tsschemas/v3/languageModel.json
🚧 Files skipped from review as they are similar to previous changes (4)
- packages/web/src/features/chat/actions.ts
- docs/snippets/schemas/v3/languageModel.schema.mdx
- packages/schemas/src/v3/index.type.ts
- packages/schemas/src/v3/index.schema.ts
Adds the
authTokenoption to the anthropic language model config for specifying bearer tokens.Fixes #966
Generated with Claude Code
Summary by CodeRabbit
New Features
Documentation