Skip to content

Updated the Claude Model and Google Vertex Commented Out#4

Open
BboyAkers wants to merge 1 commit into
mainfrom
env-fix
Open

Updated the Claude Model and Google Vertex Commented Out#4
BboyAkers wants to merge 1 commit into
mainfrom
env-fix

Conversation

@BboyAkers

Copy link
Copy Markdown
Member

Summary:

  • Updated the claude model in the env file
  • Commented out Google Vertex AI key/value in the env to prevent an error

@gemini-code-assist gemini-code-assist 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.

Code Review

This pull request updates the dot-env.example file by changing the CLAUDE_MODEL to claude-sonnet-4-6 and commenting out the Google Vertex AI configuration variables. Feedback indicates that claude-sonnet-4-6 is a Vertex AI model identifier and using it for CLAUDE_MODEL (direct Anthropic API) may cause API errors, suggesting to revert it to a valid Anthropic model identifier.

Comment thread dot-env.example
# Anthropic (Optional)
ANTHROPIC_API_KEY=sk-ant-your-key-here
CLAUDE_MODEL=claude-sonnet-4-5-20250514
CLAUDE_MODEL=claude-sonnet-4-6

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

high

The model identifier claude-sonnet-4-6 appears to be a Google Vertex AI model name (configured as the default for VERTEX_MODEL in lib/config.js). Setting CLAUDE_MODEL (which is used for the direct Anthropic API provider) to claude-sonnet-4-6 will likely result in an API error because the direct Anthropic API expects standard Anthropic model identifiers (or the claude-sonnet-4-5-20250929 default defined in lib/config.js). Consider reverting this to a valid Anthropic model identifier or keeping it consistent with the default in lib/config.js.

CLAUDE_MODEL=claude-sonnet-4-5-20250514

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