Skip to content

fix: reference correct PyPI package names in provider load error messages#164

Merged
jsonbailey merged 3 commits intomainfrom
jb/fix-provider-error-message
May 5, 2026
Merged

fix: reference correct PyPI package names in provider load error messages#164
jsonbailey merged 3 commits intomainfrom
jb/fix-provider-error-message

Conversation

@jsonbailey
Copy link
Copy Markdown
Contributor

@jsonbailey jsonbailey commented May 5, 2026

Summary

  • The provider load error message previously referenced internal Python module names (ldai_openai, ldai_langchain) rather than the pip-installable PyPI package names, making the error unhelpful to end users.
  • Added a _PYPI_PACKAGE_NAMES mapping in runner_factory.py that maps internal module names to their corresponding PyPI package names.
  • _pkg_exists now raises an ImportError that includes the PyPI package name and an actionable pip install command (e.g. pip install launchdarkly-server-sdk-ai-openai).
  • Simplified the warning log in _get_provider_factory — it now forwards the ImportError message directly, which already contains the install guidance.
  • Added tests/test_runner_factory.py covering _pkg_exists, _PYPI_PACKAGE_NAMES, and _get_provider_factory warning content.

Before:

WARNING:ldclient.util:Could not load provider 'openai': Package 'ldai_openai' not found. Make sure the corresponding package is installed.

After:

WARNING:ldclient.util:Could not load provider 'openai': Package 'launchdarkly-server-sdk-ai-openai' not found. Run: pip install launchdarkly-server-sdk-ai-openai

Test plan

  • All existing tests pass (uv run pytest tests/ -x -q — 185 passed)
  • New tests/test_runner_factory.py verifies the PyPI name appears in both the ImportError and the warning log for openai and langchain providers
  • Verified the old boilerplate "Make sure the corresponding package is installed" text is no longer present in warnings

🤖 Generated with Claude Code


Note

Low Risk
Low risk: only adjusts ImportError/warning messaging when optional provider dependencies are absent and adds unit tests; no runtime behavior changes when providers are installed.

Overview
Improves provider-load failure messaging in RunnerFactory by introducing a _PYPI_PACKAGE_NAMES mapping and emitting errors that reference the pip-installable package names (instead of internal module names) when ldai_openai/ldai_langchain aren’t importable.

Simplifies _get_provider_factory’s ImportError warning to log the exception message directly, and adds test_runner_factory.py to validate the mapping and that warnings/errors include PyPI names and remain package-manager agnostic.

Reviewed by Cursor Bugbot for commit a872281. Bugbot is set up for automated code reviews on this repo. Configure here.

jsonbailey and others added 2 commits May 5, 2026 10:04
…ages

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…ssing

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@jsonbailey jsonbailey marked this pull request as ready for review May 5, 2026 15:40
@jsonbailey jsonbailey requested a review from a team as a code owner May 5, 2026 15:40
@jsonbailey jsonbailey merged commit 48761c9 into main May 5, 2026
47 checks passed
@jsonbailey jsonbailey deleted the jb/fix-provider-error-message branch May 5, 2026 15:50
@github-actions github-actions Bot mentioned this pull request May 5, 2026
jsonbailey added a commit that referenced this pull request May 6, 2026
🤖 I have created a release *beep* *boop*
---


<details><summary>launchdarkly-server-sdk-ai: 0.19.0</summary>

##
[0.19.0](launchdarkly-server-sdk-ai-0.18.0...launchdarkly-server-sdk-ai-0.19.0)
(2026-05-05)


### ⚠ BREAKING CHANGES

* StructuredResponse replaced by RunnerResult with new "parsed" property
* AgentResult replaced by RunnerResult and Managed Result
* Removed ModelRunner and AgentRunner protocols
* Removed invoke_method, invoke_structured_model from AIProvider base
class.
* ModelResponse was replaced by RunnerResult
* Add ManagedResult, RunnerResult, and Runner protocol; rename invoke()
to run()
([#148](#148))
* Swap track_metrics_of parameter order to match spec
([#144](#144))

### Features

* Add evaluations support to ManagedAgent.run()
([#153](#153))
([442f46a](442f46a))
* Add judge evaluation support to agent graphs
([#142](#142))
([3d5a6a9](3d5a6a9))
* Add ManagedGraphResult, GraphMetricSummary, and AgentGraphRunnerResult
types
([#151](#151))
([301e24c](301e24c))
* Add ManagedResult, RunnerResult, and Runner protocol; rename invoke()
to run()
([#148](#148))
([88d4ddc](88d4ddc))
* Add root-level tools map with customParameters to AI Config types
([#141](#141))
([f17c535](f17c535))
* bake sampling_rate into Judge at construction; simplify Evaluator to
List[Judge]
([#159](#159))
([86c79e6](86c79e6))
* Update LangChain runners to implement Runner protocol returning
RunnerResult
([#150](#150))
([62a8e25](62a8e25))


### Bug Fixes

* Add runtime DeprecationWarnings to deprecated methods
([#145](#145))
([2189b81](2189b81))
* AgentResult replaced by RunnerResult and Managed Result
([fbb0b4b](fbb0b4b))
* build judge input as string; strip legacy judge config messages
([#165](#165))
([e6942a6](e6942a6))
* Fall back to model.parameters.tools when root tools absent
([#146](#146))
([2c30d75](2c30d75))
* Graph tracking refactor — ManagedAgentGraph drives tracking for new
runner shape
([#154](#154))
([20a5020](20a5020))
* ModelResponse was replaced by RunnerResult
([fbb0b4b](fbb0b4b))
* parse model.parameters.tools as list
([#160](#160))
([fb53e99](fb53e99))
* reference correct PyPI package names in provider load error messages
([#164](#164))
([48761c9](48761c9))
* Removed invoke_method, invoke_structured_model from AIProvider base
class.
([fbb0b4b](fbb0b4b))
* Removed ModelRunner and AgentRunner protocols
([fbb0b4b](fbb0b4b))
* Replace done_callback with coroutine chain for judge tracking
([#147](#147))
([1e1f36b](1e1f36b))
* StructuredResponse replaced by RunnerResult with new "parsed" property
([fbb0b4b](fbb0b4b))
* Swap track_metrics_of parameter order to match spec
([#144](#144))
([53db736](53db736))
</details>

<details><summary>launchdarkly-server-sdk-ai-langchain: 0.6.0</summary>

##
[0.6.0](launchdarkly-server-sdk-ai-langchain-0.5.0...launchdarkly-server-sdk-ai-langchain-0.6.0)
(2026-05-05)


### Features

* Add judge evaluation support to agent graphs
([#142](#142))
([3d5a6a9](3d5a6a9))
* Migrate LangGraph runner to AgentGraphRunnerResult; clean up legacy
shape detection
([#156](#156))
([efa8e00](efa8e00))
* Support conversation history directly in AI Provider model runners
([#166](#166))
([4bb3e78](4bb3e78))
* Update LangChain runners to implement Runner protocol returning
RunnerResult
([#150](#150))
([62a8e25](62a8e25))


### Bug Fixes

* build judge input as string; strip legacy judge config messages
([#165](#165))
([e6942a6](e6942a6))
</details>

<details><summary>launchdarkly-server-sdk-ai-openai: 0.5.0</summary>

##
[0.5.0](launchdarkly-server-sdk-ai-openai-0.4.0...launchdarkly-server-sdk-ai-openai-0.5.0)
(2026-05-05)


### Features

* Add judge evaluation support to agent graphs
([#142](#142))
([3d5a6a9](3d5a6a9))
* Support conversation history directly in AI Provider model runners
([#166](#166))
([4bb3e78](4bb3e78))
* Update OpenAI graph runner to return AgentGraphRunnerResult with
GraphMetrics
([#155](#155))
([388b7af](388b7af))
* Update OpenAI runners to implement Runner protocol returning
RunnerResult
([#149](#149))
([382e662](382e662))


### Bug Fixes

* build judge input as string; strip legacy judge config messages
([#165](#165))
([e6942a6](e6942a6))
</details>

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

<!-- CURSOR_SUMMARY -->
---

> [!NOTE]
> **Medium Risk**
> Primarily a release/version bump, but it publishes **breaking API
changes** (move to unified `Runner.run()`/`RunnerResult` and removal of
`invoke_*` methods), which can break downstream integrations.
> 
> **Overview**
> Cuts a new release across the core SDK and provider packages:
`launchdarkly-server-sdk-ai` to `0.19.0`, LangChain provider to `0.6.0`,
and OpenAI provider to `0.5.0`, updating the release manifest and
package metadata accordingly.
> 
> Changelogs document the shipped breaking API surface changes (notably
removing `invoke_model()`/`invoke_structured_model()` in favor of
`run(...)` and standardizing returns on `RunnerResult`) plus
accompanying feature/fix entries; the core package version
constants/docs (`__version__`, `PROVENANCE.md`) are updated to match.
> 
> <sup>Reviewed by [Cursor Bugbot](https://cursor.com/bugbot) for commit
a20d7a5. Bugbot is set up for automated
code reviews on this repo. Configure
[here](https://www.cursor.com/dashboard/bugbot).</sup>
<!-- /CURSOR_SUMMARY -->

---------

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: jsonbailey <jbailey@launchdarkly.com>
Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
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