harness-sdk-python 0.31.1__tar.gz → 0.32.0__tar.gz

{harness_sdk_python-0.31.1 → harness_sdk_python-0.32.0}/PKG-INFO

@@ -1,11 +1,11 @@
 Metadata-Version: 2.5
 Name: harness-sdk-python
-Version: 0.31.1
+Version: 0.32.0
 Summary: RunManager: the harness-sdk runs subsystem for Python Statewire hosts
 Project-URL: Repository, https://github.com/assistant-ui/harness-sdk
 License-Expression: MIT
 Requires-Python: <4.0,>=3.12
-Requires-Dist: pinned<0.15,>=0.14.0
+Requires-Dist: pinned<0.15,>=0.14.6
 Requires-Dist: statewire<0.13,>=0.12.3
 Provides-Extra: deepagents
 Requires-Dist: deepagents>=0.6.12; extra == 'deepagents'

{harness_sdk_python-0.31.1 → harness_sdk_python-0.32.0}/pyproject.toml

@@ -1,11 +1,11 @@
 [project]
 name = "harness-sdk-python"
-version = "0.31.1"
+version = "0.32.0"
 description = "RunManager: the harness-sdk runs subsystem for Python Statewire hosts"
 readme = "README.md"
 license = "MIT"
 requires-python = ">=3.12,<4.0"
-dependencies = ["statewire>=0.12.3,<0.13", "pinned>=0.14.0,<0.15"]
+dependencies = ["statewire>=0.12.3,<0.13", "pinned>=0.14.6,<0.15"]
 
 [project.optional-dependencies]
 postgres = ["langgraph-checkpoint-postgres>=2.0.0"]

{harness_sdk_python-0.31.1 → harness_sdk_python-0.32.0}/src/harness_sdk/voice.py

@@ -7,9 +7,10 @@ lives in its own ephemeral tray, minted by ``voice/start`` and deleted when
 the session ends. The host implements ``VoiceManager.Host``: the seed read
 and the turn commit both stay host-owned.
 
-``voice/start {offer, tools?}`` seeds the provider from the thread
-transcript (at most 128 messages, tail taken), hands the client's frontend
-function tool definitions to the provider, and projects ``state["voice"] =
+``voice/start {offer}`` seeds the provider from the thread
+transcript (at most 128 messages, tail taken), hands the caller's frontend
+function tool definitions (the ``tools`` map the client's synced model
+context stores at ``ctx.caller.context``) to the provider, and projects ``state["voice"] =
 {clientId, status, toolCalls, activeTask}`` — the SDP answer rides only the command's
 result payload ``{answer}``. A named call matching a client-declared tool
 projects into ``toolCalls`` as ``{callId, name, arguments}``; the owning
@@ -74,28 +75,32 @@ def _reject(reason: str, message: str) -> StatewireReject:
     return StatewireReject(message, payload={"reason": reason})
 
 
-def _validated_tools(tools: Any) -> "list[VoiceManager.ToolDef]":
+def _context_tools(context: Any) -> "list[VoiceManager.ToolDef]":
+    tools = context.get("tools") if isinstance(context, dict) else None
     if tools is None:
         return []
-    if not isinstance(tools, list):
-        raise _reject("invalid-message", "tools must be a list")
-    names: set[str] = set()
-    for tool in tools:
+    if not isinstance(tools, dict):
+        raise _reject("invalid-message", "context tools must be a name-keyed object")
+    defs: "list[VoiceManager.ToolDef]" = []
+    for name, spec in tools.items():
         if (
-            not isinstance(tool, dict)
-            or not isinstance(tool.get("name"), str)
-            or tool["name"] == ""
-            or not isinstance(tool.get("description"), str)
-            or not isinstance(tool.get("parameters"), dict)
+            name == ""
+            or not isinstance(spec, dict)
+            or not isinstance(spec.get("description", ""), str)
+            or not isinstance(spec.get("parameters"), dict)
         ):
             raise _reject(
                 "invalid-message",
-                f"tools entries must be {{name, description, parameters}}, got {tool!r}",
+                f"context tools entries must be name -> {{description?, parameters}}, got {name!r}: {spec!r}",
             )
-        if tool["name"] in names:
-            raise _reject("invalid-message", f"duplicate tool name {tool['name']!r}")
-        names.add(tool["name"])
-    return tools
+        defs.append(
+            {
+                "name": name,
+                "description": spec.get("description", ""),
+                "parameters": spec["parameters"],
+            }
+        )
+    return defs
 
 
 def _capped_seed(seed: Any) -> "list[VoiceManager.SeedMessage]":
@@ -156,7 +161,7 @@ class VoiceManager(Tray):
         text: str
 
     class ToolDef(TypedDict):
-        """One frontend function tool the client declares at ``voice/start``."""
+        """One frontend function tool from the caller's synced model context."""
 
         name: str
         description: str
@@ -593,7 +598,7 @@ class VoiceManagerNamespace(TrayMapNamespace):
         offer = params.get("offer")
         if not isinstance(offer, str) or offer == "":
             raise _reject("invalid-message", "offer must be a non-empty SDP string")
-        tools = _validated_tools(params.get("tools"))
+        tools = _context_tools(ctx.caller.context)
         if self._starting:
             raise _reject("voice-active", "a voice session is already live")
         session = self.live()
@@ -672,11 +677,11 @@ def voice_manager() -> Callable[[Callable[..., Any]], VoiceManagerDescriptor]:
     return decorate
 
 
-def _voice_of(host: Any) -> VoiceManagerNamespace | None:
+def _voice_of(host: Any, *, create: bool = True) -> VoiceManagerNamespace | None:
     for cls in type(host).__mro__:
         for name, member in vars(cls).items():
             if isinstance(member, VoiceManagerDescriptor):
-                return getattr(host, name)
+                return getattr(host, name) if create else member.peek(host)
     return None
 
 
@@ -698,7 +703,8 @@ async def end_voice(host: Any, *, ctx: Any) -> None:
 
 def voice_client_departed(host: Any, client_id: str) -> None:
     """The Statewire ``client_departed`` hook body: routes to the host's declared voice manager."""
-    namespace = _voice_of(host)
+    # peek: instantiating here would stamp voice state after the attach snapshot
+    namespace = _voice_of(host, create=False)
     if namespace is not None:
         namespace.client_departed(client_id)
 

{harness_sdk_python-0.31.1 → harness_sdk_python-0.32.0}/tests/run_helpers.py

@@ -162,6 +162,7 @@ class RunDriver:
         self.client_id = client_id
         self._side_seqs: dict[str, int] = {}
         self._seq = 0
+        self._ctx_seq = 0
         self._res: dict[int, list[dict[str, Any]]] = {}
         self._posts: list["asyncio.Task[None]"] = []
         self.replica: dict[str, Any] = {}
@@ -180,6 +181,20 @@ class RunDriver:
     def _spawn(self, frame: dict[str, Any]) -> None:
         self._posts.append(asyncio.create_task(self._submit(frame)))
 
+    async def set_context(self, value: dict[str, Any]) -> None:
+        """Sync this client's stored context (one root replace), as contextSync does."""
+        self._ctx_seq += 1
+        await self._submit(
+            {
+                "ctx": [
+                    {
+                        "ops": [{"op": "replace", "path": [], "value": value}],
+                        "seq": self._ctx_seq,
+                    }
+                ]
+            }
+        )
+
     def post(self, method: str, params: Any = None) -> int:
         """Background frame POST; settle via ``res(seq)``."""
         self._seq += 1

{harness_sdk_python-0.31.1 → harness_sdk_python-0.32.0}/tests/test_voice.py

@@ -4,7 +4,7 @@
 import asyncio
 
 from run_helpers import _statement, assert_rejected
-from statewire_helpers import post_frame, stream_of
+from statewire_helpers import apply_ops, post_frame, stream_of
 from voice_helpers import (
     ANSWER,
     OFFER,
@@ -316,6 +316,18 @@ async def test_reattach_within_grace_keeps_the_session(monkeypatch):
             assert not bench.provider.closed
 
 
+async def test_departure_before_any_voice_command_leaves_state_untouched():
+    bench = Bench()
+    async with voice_host(bench) as (drv, app):
+        async with stream_of(app, client_id="c2") as stream:
+            await stream.next_event()
+        await asyncio.sleep(0.05)
+        assert "voice" not in drv.replica
+        async with stream_of(app, client_id="c3") as stream:
+            snapshot = apply_ops({}, await stream.next_event())
+        assert "voice" not in snapshot
+
+
 async def test_start_over_a_detached_owner_replaces_the_session():
     bench = Bench()
     async with voice_host(bench) as (drv, app):

{harness_sdk_python-0.31.1 → harness_sdk_python-0.32.0}/tests/test_voice_frontend_tools.py

@@ -1,5 +1,5 @@
-"""Frontend function tools contract: ``voice/start {tools}`` reaches the
-provider, matching calls project into ``state["voice"]["toolCalls"]``,
+"""Frontend function tools contract: the caller's synced context ``tools``
+map reaches the provider at ``voice/start``, matching calls project into ``state["voice"]["toolCalls"]``,
 ``voice/temp-frontend-tool-result`` answers them, and the delegated-task
 handler's ``call_frontend_tool`` rides the same projection."""
 
@@ -23,11 +23,13 @@ async def until(predicate, timeout: float = 5) -> None:
         assert asyncio.get_running_loop().time() < deadline, "condition never held"
         await asyncio.sleep(0.01)
 
-TOOL = {
-    "name": "get_local_time",
-    "description": "Read the user's local date and time.",
-    "parameters": {"type": "object", "properties": {}},
+TOOLS = {
+    "get_local_time": {
+        "description": "Read the user's local date and time.",
+        "parameters": {"type": "object", "properties": {}},
+    },
 }
+TOOL = {"name": "get_local_time", **TOOLS["get_local_time"]}
 
 
 def call_of(events, call_id: str = "call-1") -> None:
@@ -38,25 +40,34 @@ def call_of(events, call_id: str = "call-1") -> None:
     )
 
 
-async def test_start_passes_tools_to_the_provider():
+async def test_start_reads_tools_from_the_caller_context():
     bench = Bench()
     async with voice_host(bench) as (drv, _app):
-        rsp = await drv.command("voice/start", {"offer": OFFER, "tools": [TOOL]})
+        await drv.set_context(
+            {"tools": {**TOOLS, "no_desc": {"parameters": {"type": "object"}}}}
+        )
+        rsp = await drv.command("voice/start", {"offer": OFFER})
         assert rsp["type"] == "accepted", rsp
-        assert bench.provider.calls[0] == ("start", OFFER, [], [TOOL])
+        assert bench.provider.calls[0] == (
+            "start",
+            OFFER,
+            [],
+            [TOOL, {"name": "no_desc", "description": "", "parameters": {"type": "object"}}],
+        )
 
 
-async def test_malformed_tools_reject_invalid_message():
+async def test_malformed_context_tools_reject_invalid_message():
     bench = Bench()
     async with voice_host(bench) as (drv, _app):
         for tools in (
-            "not-a-list",
-            [{"name": "", "description": "", "parameters": {}}],
-            [{"name": "a", "description": "d"}],
-            [TOOL, TOOL],
+            "not-a-map",
+            {"": {"description": "", "parameters": {}}},
+            {"a": {"description": "d"}},
+            {"a": "not-a-spec"},
         ):
+            await drv.set_context({"tools": tools})
             assert_rejected(
-                await drv.command("voice/start", {"offer": OFFER, "tools": tools}),
+                await drv.command("voice/start", {"offer": OFFER}),
                 "invalid-message",
             )
         assert bench.provider.calls == []
@@ -65,7 +76,8 @@ async def test_malformed_tools_reject_invalid_message():
 async def test_client_tool_call_projects_into_state():
     bench = Bench()
     async with voice_host(bench) as (drv, _app):
-        await drv.command("voice/start", {"offer": OFFER, "tools": [TOOL]})
+        await drv.set_context({"tools": TOOLS})
+        await drv.command("voice/start", {"offer": OFFER})
         call_of(bench.events())
         await drv.wait(
             lambda s: (voice_of(s) or {}).get("toolCalls")
@@ -78,7 +90,8 @@ async def test_client_tool_call_projects_into_state():
 async def test_tool_result_forwards_and_clears_the_pending_call():
     bench = Bench()
     async with voice_host(bench) as (drv, _app):
-        await drv.command("voice/start", {"offer": OFFER, "tools": [TOOL]})
+        await drv.set_context({"tools": TOOLS})
+        await drv.command("voice/start", {"offer": OFFER})
         call_of(bench.events())
         await drv.wait(lambda s: (voice_of(s) or {}).get("toolCalls") != [])
         rsp = await drv.command(
@@ -93,7 +106,8 @@ async def test_tool_result_forwards_and_clears_the_pending_call():
 async def test_unknown_call_id_rejects_unknown_tool_call():
     bench = Bench()
     async with voice_host(bench) as (drv, _app):
-        await drv.command("voice/start", {"offer": OFFER, "tools": [TOOL]})
+        await drv.set_context({"tools": TOOLS})
+        await drv.command("voice/start", {"offer": OFFER})
         assert_rejected(
             await drv.command(
                 "voice/temp-frontend-tool-result",
@@ -113,7 +127,8 @@ async def test_executor_call_projects_and_the_result_resolves_the_await():
 
     bench.task_handler = handler
     async with voice_host(bench) as (drv, _app):
-        await drv.command("voice/start", {"offer": OFFER, "tools": [TOOL]})
+        await drv.set_context({"tools": TOOLS})
+        await drv.command("voice/start", {"offer": OFFER})
         bench.events().task_requested("d1", "what time is it locally?")
         await drv.wait(lambda s: (voice_of(s) or {}).get("toolCalls") != [])
         [call] = drv.replica["voice"]["toolCalls"]
@@ -142,7 +157,8 @@ async def test_executor_call_with_undeclared_tool_raises_in_the_handler():
 
     bench.task_handler = handler
     async with voice_host(bench) as (drv, _app):
-        await drv.command("voice/start", {"offer": OFFER, "tools": [TOOL]})
+        await drv.set_context({"tools": TOOLS})
+        await drv.command("voice/start", {"offer": OFFER})
         bench.events().task_requested("d1", "open up")
         await until(
             lambda: any(
@@ -170,7 +186,8 @@ async def test_session_end_fails_a_pending_executor_call():
 
     bench.task_handler = handler
     async with voice_host(bench) as (drv, _app):
-        await drv.command("voice/start", {"offer": OFFER, "tools": [TOOL]})
+        await drv.set_context({"tools": TOOLS})
+        await drv.command("voice/start", {"offer": OFFER})
         bench.events().task_requested("d1", "what time is it?")
         await drv.wait(lambda s: (voice_of(s) or {}).get("toolCalls") != [])
         rsp = await drv.command("voice/end")
@@ -183,7 +200,8 @@ async def test_session_end_fails_a_pending_executor_call():
 async def test_tool_result_from_other_client_rejects_not_voice_owner():
     bench = Bench()
     async with voice_host(bench) as (drv, app):
-        await drv.command("voice/start", {"offer": OFFER, "tools": [TOOL]})
+        await drv.set_context({"tools": TOOLS})
+        await drv.command("voice/start", {"offer": OFFER})
         call_of(bench.events())
         await drv.wait(lambda s: (voice_of(s) or {}).get("toolCalls") != [])
         rsp = await side_command(