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

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

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: harness-sdk-python
-Version: 0.31.2
+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

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

@@ -1,6 +1,6 @@
 [project]
 name = "harness-sdk-python"
-version = "0.31.2"
+version = "0.32.0"
 description = "RunManager: the harness-sdk runs subsystem for Python Statewire hosts"
 readme = "README.md"
 license = "MIT"

{harness_sdk_python-0.31.2 → 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()

{harness_sdk_python-0.31.2 → 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.2 → 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(