Merge pull request #31 from pipecat-ai/mb/serializer-deprecation-warnings

Fix deprecation warnings from the frame serializer

Author: markbackman

changelog/31.fixed.md

@@ -0,0 +1 @@
+- Fixed deprecation warnings being emitted for every frame sent to Whisker. Deprecated frame fields, such as `StartFrame.audio_in_sample_rate` and its siblings, warn when read, and the serializer read each dataclass field twice while walking every frame. It now reads the raw instance state instead, so no warning is emitted.

pipecat/src/pipecat_whisker/sink.py

@@ -166,12 +166,11 @@ def whisker_obj_serializer(obj: Any) -> Any:
     Returns:
         A JSON-shaped representation of the input.
     """
-    if is_dataclass(obj):
-        return {
-            f.name: whisker_obj_serializer(getattr(obj, f.name))
-            for f in fields(obj)
-            if getattr(obj, f.name) is not None
-        }
+    if is_dataclass(obj) and not isinstance(obj, type):
+        # Read the raw instance state: a field can intercept reads to warn that
+        # it is deprecated, and this walks every field of every frame.
+        values = {f.name: object.__getattribute__(obj, f.name) for f in fields(obj)}
+        return {k: whisker_obj_serializer(v) for k, v in values.items() if v is not None}
     elif isinstance(obj, (list, tuple, set)):
         return [whisker_obj_serializer(v) for v in obj if v is not None]
     elif isinstance(obj, dict):

ui/.gitignore

@@ -1 +1,2 @@
 .vercel
+.env*