Skip to content

feat(credentials): Add support for Anthropic credentials#245

Merged
Ayush8923 merged 1 commit into
mainfrom
fix/added-antropic-credentials
Jul 6, 2026
Merged

feat(credentials): Add support for Anthropic credentials#245
Ayush8923 merged 1 commit into
mainfrom
fix/added-antropic-credentials

Conversation

@Ayush8923

@Ayush8923 Ayush8923 commented Jul 6, 2026

Copy link
Copy Markdown
Collaborator

Issue

Closes #242

Summary

  • Added support for Anthropic credentials.

Checklist

Before submitting a pull request, please ensure that you mark these task.

  • Ran npm run dev and npm run build in the repository root and test.
  • If you've fixed a bug or added code that is tested

@coderabbitai

coderabbitai Bot commented Jul 6, 2026

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

Added a new anthropic provider entry to the PROVIDERS list in app/lib/types/credentials.ts, including display metadata and a password-type api_key credential field with placeholder text, inserted before the existing elevenlabs provider.

Changes

Anthropic Provider Configuration

Layer / File(s) Summary
Add anthropic provider entry
app/lib/types/credentials.ts
New anthropic object added to PROVIDERS array with id, name, description, credentialKey, and an api_key password field (placeholder sk-ant-xxxxx).

Estimated code review effort: 1 (Trivial) | ~3 minutes

Related issues: #242 - Enables the frontend to accept an Anthropic API key by adding the corresponding provider configuration.

Suggested labels: enhancement, frontend

Suggested reviewers: (none identified from provided context)

🐰 A whisker twitch, a new key in tow,
Anthropic joins the credentials row,
Password hidden, placeholder bright,
sk-ant-xxxxx, tucked in just right,
Hop along, one more provider to know!

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Linked Issues check ⚠️ Warning The API key field and password handling are added, but there is no evidence of the required API key format validation [#242]. Add Anthropic API key format validation in the credentials flow and verify storage/security behavior still meets #242.
✅ Passed checks (4 passed)
Check name Status Explanation
Out of Scope Changes check ✅ Passed The change stays focused on Anthropic credentials and does not दिख evidence of unrelated modifications.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly matches the main change: adding Anthropic credential support in the credentials types.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/added-antropic-credentials

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.

@Ayush8923 Ayush8923 self-assigned this Jul 6, 2026
@github-actions github-actions Bot changed the title fix(credentials): Added the Anthropic credentials supoort feat(auth): Add support for Anthropic credentials Jul 6, 2026
@Ayush8923 Ayush8923 requested a review from vprashrex July 6, 2026 07:15
@Ayush8923 Ayush8923 changed the title feat(auth): Add support for Anthropic credentials feat(credentials): Add support for Anthropic credentials Jul 6, 2026

@coderabbitai coderabbitai Bot 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.

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@app/lib/types/credentials.ts`:
- Around line 76-89: The Anthropic credential entry in the ProviderDef list is
structurally fine, but it still lacks the required API key format validation.
Extend FieldDef in credentials.ts with an optional validation hook such as
pattern or validate, then update the credentials form flow that consumes
FieldDef to enforce the Anthropic api_key format (for example the sk-ant-
prefix) before saving. Use the existing Anthropic provider entry and the
credentials page/form logic as the integration points.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: fb91bc6b-d242-48ff-8b99-139af20b43a2

📥 Commits

Reviewing files that changed from the base of the PR and between 0829826 and e4f1111.

📒 Files selected for processing (1)
  • app/lib/types/credentials.ts

Comment thread app/lib/types/credentials.ts
@Ayush8923 Ayush8923 merged commit 2458d95 into main Jul 6, 2026
3 checks passed
@Ayush8923 Ayush8923 deleted the fix/added-antropic-credentials branch July 6, 2026 10:00
@github-actions

github-actions Bot commented Jul 6, 2026

Copy link
Copy Markdown

🎉 This PR is included in version 0.3.0-main.10 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Credentials: Enable Anthropic API key

2 participants