Skip to content

chore: correct commented default class name to AICompletionConfigDefault - #38

Draft
devin-ai-integration[bot] wants to merge 1 commit into
mainfrom
devin/1786352880-fix-default-comment
Draft

chore: correct commented default class name to AICompletionConfigDefault#38
devin-ai-integration[bot] wants to merge 1 commit into
mainfrom
devin/1786352880-fix-default-comment

Conversation

@devin-ai-integration

Copy link
Copy Markdown
Contributor

Summary

The four getting_started completion examples show a commented-out resiliency default that references AIConfig, a class that no longer exists in launchdarkly-server-sdk-ai 1.x. Anyone uncommenting it hits ImportError/NameError. The current name is AICompletionConfigDefault, which features/create_model already uses.

-# default = AIConfig(+# default = AICompletionConfigDefault(

No dependency changes: every manifest uses floor/caret constraints and poetry.lock is gitignored, so poetry install already resolves to the latest within-major versions (launchdarkly-server-sdk-ai 1.2.0, launchdarkly-observability 1.3.1, launchdarkly-server-sdk-ai-langchain 0.8.0, langchain 1.3.14). google-genai stays on ^1.30.0 since 2.x would be a major jump.

How it was tested

  • poetry install in all 10 example directories against latest published deps, then imported each example module — all succeed (the OpenAI example instantiates OpenAI() at import time, so it needs OPENAI_API_KEY, which is unrelated to this change).
  • Ran the LangChain path end-to-end against a real LaunchDarkly SDK key with an AICompletionConfigDefault default and an Anthropic model, confirming the corrected class name works and metrics are tracked:
init True
enabled True claude-sonnet-4-5 anthropic
response: I can help you with a wide variety of tasks! ...
duration 6328 success True tokens (20, 198, 218)

Limitations

  • The examples could not be run against real AI Configs: none of the example config keys (sample-completion, sample-agent, sample-judge, ...) exist in the hello-app LaunchDarkly project (FLAG_NOT_FOUND), and provider keys for OpenAI/Gemini/Bedrock are not available here. The end-to-end run above used a local default plus an Anthropic key instead.
  • Pre-existing, not fixable here: launchdarkly-observability's generated GraphQL client imports the deprecated websockets.client API, emitting DeprecationWarning on import. That belongs to the observability SDK.

Link to Devin session: https://app.devin.ai/sessions/d518388a79344d6ebcdaaeedbef361d0
Requested by: @jsonbailey

Co-Authored-By: jbailey@launchdarkly.com <accounts@sidewaysgravity.com>
@devin-ai-integration

Copy link
Copy Markdown
ContributorAuthor

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment, CI, and merge conflict monitoring

@devin-ai-integrationdevin-ai-integrationBot added the devin-pr PR created by Devin label Aug 10, 2026
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

devin-prPR created by Devin

Projects

None yet

Development

Successfully merging this pull request may close these issues.

0 participants