Skip to content

fix(hume, simli): make every __all__ entry resolve - #7981

Merged
longcw merged 1 commit into
livekit:mainfrom
omChauhanDev:fix/hume-simli-all-exports
Oct 20, 2026
Merged

longcw merged 1 commit into
livekit:mainfrom
omChauhanDev:fix/hume-simli-all-exports

Conversation

@omChauhanDev

Copy link
Copy Markdown
Contributor

Problem

from livekit.plugins.hume import * and from livekit.plugins.simli import * both fail with AttributeError, because each package lists a name in __all__ that it never binds. In hume that name is ModelVersion: it is defined in hume/tts.py and types the model_version argument of TTS, but __init__.py never imports it, so hume.ModelVersion raises AttributeError: module 'livekit.plugins.hume' has no attribute 'ModelVersion' and from livekit.plugins.hume import ModelVersion raises ImportError. In simli the name is Exception, which the package doesn't define, and module attribute lookup does not fall back to builtins. Tools that read __all__, such as the pdoc docs build, are handed the same missing names.

Changes

Import ModelVersion in hume's from .tts import (...) block, and remove "Exception" from simli's __all__.

hume listed ModelVersion in __all__ without importing it, and simli listed Exception, which it never defines, so star imports of both packages raised AttributeError.
@omChauhanDev
omChauhanDev requested a review from a team as a code owner October 20, 2026 10:16

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.

Devin Review

@longcw
longcw merged commit a24dbb0 into livekit:main Oct 20, 2026
17 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