dv-pipecat-flows 0.0.22.dev2200__tar.gz → 0.0.22.dev2400__tar.gz

{dv_pipecat_flows-0.0.22.dev2200/src/dv_pipecat_flows.egg-info → dv_pipecat_flows-0.0.22.dev2400}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: dv-pipecat-flows
-Version: 0.0.22.dev2200
+Version: 0.0.22.dev2400
 Summary: Conversation Flow management for Pipecat AI applications
 License: BSD 2-Clause License
 Project-URL: Source, https://github.com/pipecat-ai/pipecat-flows

{dv_pipecat_flows-0.0.22.dev2200 → dv_pipecat_flows-0.0.22.dev2400/src/dv_pipecat_flows.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: dv-pipecat-flows
-Version: 0.0.22.dev2200
+Version: 0.0.22.dev2400
 Summary: Conversation Flow management for Pipecat AI applications
 License: BSD 2-Clause License
 Project-URL: Source, https://github.com/pipecat-ai/pipecat-flows

{dv_pipecat_flows-0.0.22.dev2200 → dv_pipecat_flows-0.0.22.dev2400}/src/pipecat_flows/actions.py

@@ -25,6 +25,7 @@ Actions are used to perform side effects during conversations, such as:
 
 import asyncio
 import inspect
+import re
 from dataclasses import dataclass
 from typing import TYPE_CHECKING, Callable, Dict, List, Optional, Tuple
 
@@ -56,6 +57,48 @@ if TYPE_CHECKING:
 ONGOING_ACTIONS_WAIT_TIMEOUT_S = 60.0
 
 
+# Router/Agent-extracted variable token: ``[{name}]`` (distinct from custom
+# ``{{name}}`` on the FE — see VariableSelect.tsx). Resolved at TTS time from
+# ``flow_manager.state``, where ``_run_router_node`` (and the flattened
+# agent_node→llm_router path) writes the LLM's extracted args. Backend
+# lowering only substitutes compile-time vars (custom_args, api_responses);
+# anything a router extracts at runtime needs this pass to reach TTS.
+#
+# Pattern matches BOTH the FE's dominant mention form ``@[{var}]`` (the
+# ``@``-prefixed visible text inside a Tiptap mention span) AND the raw
+# ``[{var}]`` form — mirrors how ``replace_custom_variables`` handles both
+# ``@{{var}}`` and ``{{var}}`` for custom_args.
+_EXTRACT_VAR_RE = re.compile(
+    r"@?\[\{\s*([A-Za-z_][A-Za-z0-9_.]*)\s*\}\]"
+)
+
+
+def _substitute_extract_vars(text: str, state: dict) -> str:
+    """Replace ``[{var}]`` / ``@[{var}]`` occurrences with ``state[var]``.
+
+    Supports dotted paths (``[{user.name}]``). Unknown variables (key not
+    in state, or any path segment missing) are LEFT LITERAL — the marker
+    survives so downstream debugging can spot the misconfigured extract.
+    """
+    if not text or "[{" not in text or not isinstance(state, dict):
+        return text
+
+    def _resolve(key: str):
+        cur = state
+        for seg in key.split("."):
+            if isinstance(cur, dict) and seg in cur:
+                cur = cur[seg]
+            else:
+                return None
+        return cur
+
+    def _repl(match: "re.Match[str]") -> str:
+        value = _resolve(match.group(1))
+        return str(value) if value is not None else match.group(0)
+
+    return _EXTRACT_VAR_RE.sub(_repl, text)
+
+
 @dataclass
 class FunctionActionFrame(ControlFrame):
     """Frame containing a function action to be executed.
@@ -432,6 +475,16 @@ class ActionManager:
             logger.error("TTS action missing 'text' / 'text_variants' field")
             return
 
+        # Runtime resolution of [{var}] / @[{var}] tokens against
+        # flow_manager.state. Backend lowering substitutes {{var}} (custom
+        # args) and ((tool.field)) (pre-call outputs) at compile time, but
+        # router/agent-extracted vars only exist at runtime — without this
+        # pass, the bot speaks literals like "@[{user_name}]" verbatim.
+        # text_raw stays unrendered so the split-cache's prefix derivation
+        # remains stable across calls.
+        state = self._flow_manager.state if isinstance(self._flow_manager.state, dict) else {}
+        text = _substitute_extract_vars(text, state)
+
         await self._maybe_switch_tts_service_for_action(action)
 
         use_cache = action.get("use_cache", False)

{dv_pipecat_flows-0.0.22.dev2200 → dv_pipecat_flows-0.0.22.dev2400}/src/pipecat_flows/manager.py

@@ -623,6 +623,7 @@ class FlowManager:
         transition_to: Optional[str],
         transition_callback: Optional[Callable] = None,
         state_update_keys: Optional[List[str]] = None,
+        router_node_meta: Optional[Dict[str, Any]] = None,
     ) -> Callable:
         """Create a transition function for the given name and handler.
 
@@ -668,6 +669,7 @@ class FlowManager:
                 # ``_set_node`` after the transition. Mirrors the
                 # ``state.update(extracted)`` that ``_run_router_node`` does
                 # on the non-flattened path.
+                extract_update: Dict[str, Any] = {}
                 if state_update_keys and params.arguments:
                     extract_update = {
                         key: params.arguments[key]
@@ -681,6 +683,53 @@ class FlowManager:
                             f"{list(extract_update.keys())} function={name}"
                         )
 
+                # Flattened-router observability: when this function bypasses
+                # an llm_router (router_node_meta set by backend wiring), the
+                # router was never ``_set_node``'d, so its on_node_entered +
+                # on_variables_extracted events never fired naturally. Emit
+                # them synthetically here so the FE canvas highlights the
+                # bypassed router and displays the extracted variables (e.g.
+                # ``state["language"]``) — same visual continuity as the
+                # non-flattened classifier path. Gated by the same
+                # ``state["observable"]`` flag as the natural fires, so prod
+                # calls (no SSE subscriber) skip the publish overhead.
+                if router_node_meta and self.state.get("observable"):
+                    _router_id = router_node_meta.get("id") or router_node_meta.get("name")
+                    _router_label = router_node_meta.get("name") or _router_id
+                    _router_kind = router_node_meta.get("kind") or "router_node"
+                    if self.on_node_entered and _router_id:
+                        _prev_node_id = self._current_node
+                        _prev_cfg = (
+                            self._nodes.get(_prev_node_id) if _prev_node_id else None
+                        )
+                        _from_label = (_prev_cfg or {}).get("name") if _prev_cfg else None
+                        self._fire_and_forget_observe(
+                            self.on_node_entered(
+                                {
+                                    "from_node_id": _prev_node_id,
+                                    "from_label": _from_label,
+                                    "node_id": _router_id,
+                                    "label": _router_label,
+                                    "kind": _router_kind,
+                                    "synthetic": "flattened_router",
+                                    "ts": time.time(),
+                                }
+                            ),
+                            label=f"flattened_router_entered:{_router_id}",
+                        )
+                    if self.on_variables_extracted and extract_update and _router_id:
+                        self._fire_and_forget_observe(
+                            self.on_variables_extracted(
+                                {
+                                    "node_id": _router_id,
+                                    "variables": dict(extract_update),
+                                    "source": "flattened_router",
+                                    "ts": time.time(),
+                                }
+                            ),
+                            label=f"flattened_router_variables_extracted:{_router_id}",
+                        )
+
                 function_call_id = await start_tracking(
                     transcript_handler=self._transcript_handler,
                     tool_name=name,
@@ -931,6 +980,7 @@ class FlowManager:
         transition_to: Optional[str] = None,
         transition_callback: Optional[Callable] = None,
         state_update_keys: Optional[List[str]] = None,
+        router_node_meta: Optional[Dict[str, Any]] = None,
     ) -> None:
         """Register a function with the LLM if not already registered.
 
@@ -959,6 +1009,7 @@ class FlowManager:
                     transition_to,
                     transition_callback,
                     state_update_keys=state_update_keys,
+                    router_node_meta=router_node_meta,
                 )
 
                 # Register function with LLM (or LLMSwitcher).
@@ -2611,6 +2662,7 @@ In all of these cases, you can provide a `name` in your new node's config for de
                     transition_to=schema.transition_to,
                     transition_callback=schema.transition_callback,
                     state_update_keys=schema.state_update_keys,
+                    router_node_meta=schema.router_node_meta,
                 )
 
             async def register_direct_function(func):

{dv_pipecat_flows-0.0.22.dev2200 → dv_pipecat_flows-0.0.22.dev2400}/src/pipecat_flows/types.py

@@ -279,6 +279,16 @@ class FlowsFunctionSchema:
     # transitions and still propagate the router's ``router_extract``
     # variables (e.g. ``language``) into state. Empty/None = no-op.
     state_update_keys: Optional[List[str]] = None
+    # Identity of the LLM-router this function bypassed in the flatten path
+    # (``agent_node → llm_router``). When set, the engine emits a synthetic
+    # ``on_node_entered`` for the router and an ``on_variables_extracted``
+    # for the LLM's args at function-call time — so the FE canvas still
+    # highlights the bypassed router AND displays the extracted variables
+    # (e.g. "language") even though the router's classifier turn never ran.
+    # Shape: ``{"id": <node_uuid>, "name": <node_name>, "kind": "router_node"}``
+    # None means this function is a direct agent→non-router transition;
+    # no synthetic events fire.
+    router_node_meta: Optional[Dict[str, Any]] = None
 
     def __post_init__(self):
         """Validate the schema configuration.