Skip to content

docs: document ANTHROPIC_BASE_URL and GOOGLE_GENERATIVE_AI_BASE_URL in .env.example - #79

Merged
jerelvelarde merged 2 commits into
CopilotKit:mainfrom
kuishou68:cocoon-fix-openmuse
Oct 6, 2026
Merged

jerelvelarde merged 2 commits into
CopilotKit:mainfrom
kuishou68:cocoon-fix-openmuse

Conversation

@kuishou68

Copy link
Copy Markdown
Contributor

What

.env.example documents the optional OPENAI_BASE_URL endpoint override, but the analogous overrides for the other two providers are missing, even though the model adapter reads them:

  • ANTHROPIC_BASE_URL — apps/server/src/engine/tanstack-agent.ts:35 (a trailing /v1 is stripped, because the Anthropic SDK adds /v1 itself)
  • GOOGLE_GENERATIVE_AI_BASE_URL — apps/server/src/engine/tanstack-agent.ts:44 (a trailing /v1beta is stripped, because @google/genai adds the API version itself)

Why

Anyone routing Anthropic or Google traffic through a gateway/proxy endpoint has no way to discover these variables from the sample env file, and the suffix handling (/v1, /v1beta) is easy to get wrong silently. This adds two commented lines mirroring the existing OPENAI_BASE_URL entry, in the same style.

Docs-only change; no code touched.

Verification

  • Grepped the source tree (excluding node_modules/dist): the only readers of these two variables are the adapter lines quoted above.
  • pnpm exec biome check . still passes.

@jerelvelarde jerelvelarde left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Value: makes supported Anthropic and Google endpoint overrides discoverable. Verified both environment variable readers and exact /v1 and /v1beta suffix normalization against current main. Template: concise problem, resulting behavior and checks; no provider/device execution needed for comments-only change. No secrets or runtime permissions added. Updated-main diff reviewed; merge after required CI.

@jerelvelarde
jerelvelarde merged commit 968108c into CopilotKit:main Oct 6, 2026
7 checks passed
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