Repository navigation
docs(skills): say which package provides the experimental entry point - #153
Merged
Merged
Conversation
The launchdarkly-ai-python README said launchdarkly_ai_server.experimental is something "this package installs". The dependency on launchdarkly-ai-server is unpinned, so upgrading only launchdarkly-ai-python can keep an older launchdarkly-ai-server (0.2.4 has no experimental subpackage). Say the features need a launchdarkly-ai-server release that includes them, and to upgrade that package if the import fails. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
XieX
marked this pull request as ready for review
October 9, 2026 17:30
jeffdupont
approved these changes
Oct 9, 2026
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
2 participants
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.
Corrects one sentence in the "Experimental features" section of
packages/ai/README.md(launchdarkly-ai-python).The README said
launchdarkly_ai_server.experimentalis something "this package installs". That isn't guaranteed.packages/ai/pyproject.tomldepends on an unpinnedlaunchdarkly-ai-server. A user who upgrades onlylaunchdarkly-ai-pythonkeeps whateverlaunchdarkly-ai-serverthey already have installed or locked, because that version still satisfies the requirement. The latest published release, 0.2.4, has noexperimentalsubpackage, so on itimport launchdarkly_ai_server.experimental.skillsraisesModuleNotFoundError. Nothing else breaks:launchdarkly-ai-pythonisfrom launchdarkly_ai_server import *and never importsexperimental.The new text:
launchdarkly-ai-serverand need a release of it that includes themlaunchdarkly-ai-serveritself if the import fails🤖 Generated with Claude Code, updated by @XieX
Note
Overview
Documentation-only update to the Experimental features section in
packages/ai/README.md.The README no longer implies that installing
launchdarkly-ai-pythonalways installs alaunchdarkly-ai-serverbuild withexperimental. It now states that experimental APIs live inlaunchdarkly-ai-server, depend on a server release that actually ships them, and thatimport launchdarkly_ai_server.experimentalfailures should be fixed by upgradinglaunchdarkly-ai-server, not only the barrel package—because the dependency is unpinned and an older server can remain after upgradinglaunchdarkly-ai-python.Reviewed by Cursor Bugbot for commit 167b370. Bugbot is set up for automated code reviews on this repo. Configure here.