chore: add ANTHROPIC_API_KEY to langchain example .env.example - #40
Draft
devin-ai-integration[bot] wants to merge 1 commit into
Draft
chore: add ANTHROPIC_API_KEY to langchain example .env.example#40devin-ai-integration[bot] wants to merge 1 commit into
devin-ai-integration[bot] wants to merge 1 commit into
Conversation
Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
Contributor
Author
🤖 Devin AI EngineerI'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
Note: I can only respond to comments from users who have write access to this repository. ⚙️ Control Options:
|
Contributor
Author
|
@cursor review |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
The LangChain example advertises Anthropic support (README: "supporting multiple providers including OpenAI, Anthropic, Google GenAI, and AWS Bedrock") and ships
langchain-anthropicas a dependency since #34, but.env.examplenever listed the key that provider needs, so anyone pointing an AI Config at an Anthropic model gets an auth error with no hint from the example config.OPENAI_API_KEY= +ANTHROPIC_API_KEY= GOOGLE_API_KEY=No dependency changes needed
Every manifest uses floor/caret constraints and
poetry.lockis gitignored, sopoetry installalready resolves to the latest within-major releases; per repo hygiene, already-satisfied constraints are not bumped. Resolved today in the LangChain example env:launchdarkly-server-sdk-ai1.2.0 (latest),launchdarkly-server-sdk-ai-langchain0.8.0 (latest),launchdarkly-server-sdk-ai-openai0.7.0 (latest),launchdarkly-observability1.3.2 (latest),launchdarkly-server-sdk9.16.1,langchain1.3.15,langgraph1.2.11,openai3.1.0,boto31.43.72.google-genaistays at^1.30.0(latest is 2.18.1 — a major jump). Thelaunchdarkly-server-sdk-ai = ">=0.20.0"floors in thegetting_startedexamples are left alone: 0.20.0 already exposes every API these examples use (completion_config,AICompletionConfigDefault,track_metrics_of_async,summary.tokens), so the declared minimum is not wrong, and the constraint installs 1.2.0 anyway.How it was validated
poetry installin all 10 example directories (the repo's only CI check) — all succeed against today's latest published deps.DeprecationWarnings enabled: no deprecated LaunchDarkly API usage. (The OpenAI example constructsOpenAI()at import time, so it needsOPENAI_API_KEY; unrelated to this change.)ANTHROPIC_API_KEYfrom the environment — this is exactly the path the new env var documents:Limitations
sample-completion,sample-agent, ...) do not exist in the available LaunchDarkly project, socompletion_configreturns a disabled config; the end-to-end run above supplied anAICompletionConfigDefaultlocally instead. The other providers (OpenAI, Gemini, Bedrock) could not be exercised — no provider credentials available here.launchdarkly-observability's generated GraphQL client imports the deprecatedwebsockets.client/websockets.legacyAPI, emittingDeprecationWarningon import of every example. That belongs to the observability SDK.AIConfigreference in the commented resiliency defaults is already handled by open PR chore: correct commented default class name to AICompletionConfigDefault #38; not duplicated here.Link to Devin session: https://app.devin.ai/sessions/6616c6d9b1164addb8900bde78197a14