Remove dead code; nits

Author: kradkfl

examples/avatar_agents/keyframe/README.md

@@ -37,7 +37,7 @@ await avatar.set_emotion("happy")  # "neutral", "happy", "sad", "angry"
 To let the LLM control the avatar's expression, register it as a tool:
 
 ```python
-@session.tool()
+@function_tool()
 async def set_avatar_emotion(emotion: keyframe.Emotion):
     """Set the avatar's facial expression and demeanor."""
     await avatar.set_emotion(emotion)

livekit-plugins/livekit-plugins-keyframe/livekit/plugins/keyframe/__init__.py

@@ -14,14 +14,13 @@
 
 from .avatar import AvatarSession
 from .errors import KeyframeException
-from .types import Emotion, PersonaConfig
+from .types import Emotion
 from .version import __version__
 
 __all__ = [
     "AvatarSession",
     "Emotion",
     "KeyframeException",
-    "PersonaConfig",
     "__version__",
 ]
 

livekit-plugins/livekit-plugins-keyframe/livekit/plugins/keyframe/avatar.py

@@ -54,14 +54,14 @@ def __init__(
         )
 
         # Resolve persona
-        self._persona_id = persona_id if utils.is_given(persona_id) else None
-        self._persona_slug = persona_slug if utils.is_given(persona_slug) else None
-
-        has_id = bool(self._persona_id)
-        has_slug = bool(self._persona_slug)
+        has_id = utils.is_given(persona_id)
+        has_slug = utils.is_given(persona_slug)
         if has_id == has_slug:
             raise KeyframeException("Provide exactly one of persona_id or persona_slug")
 
+        self._persona_id = persona_id if has_id else None
+        self._persona_slug = persona_slug if has_slug else None
+
         # Resolve API config
         api_url_val = (
             api_url if utils.is_given(api_url) else os.getenv("KEYFRAME_API_URL", DEFAULT_API_URL)

livekit-plugins/livekit-plugins-keyframe/livekit/plugins/keyframe/types.py

@@ -1,21 +1,3 @@
-from dataclasses import dataclass
 from typing import Literal
 
 Emotion = Literal["neutral", "happy", "sad", "angry"]
-
-
-@dataclass
-class PersonaConfig:
-    """Configuration for Keyframe Labs avatar persona.
-
-    Provide exactly one of persona_id or persona_slug.
-    """
-
-    persona_id: str | None = None
-    persona_slug: str | None = None
-
-    def __post_init__(self) -> None:
-        has_id = bool(self.persona_id and self.persona_id.strip())
-        has_slug = bool(self.persona_slug and self.persona_slug.strip())
-        if has_id == has_slug:
-            raise ValueError("Provide exactly one of persona_id or persona_slug")

livekit-plugins/livekit-plugins-keyframe/livekit/plugins/keyframe/version.py

@@ -1 +1 @@
-__version__ = "0.1.0"
+__version__ = "1.4.3"