harness-sdk-python 0.32.1__tar.gz → 0.32.2__tar.gz

{harness_sdk_python-0.32.1 → harness_sdk_python-0.32.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: harness-sdk-python
-Version: 0.32.1
+Version: 0.32.2
 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.32.1 → harness_sdk_python-0.32.2}/pyproject.toml

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

{harness_sdk_python-0.32.1 → harness_sdk_python-0.32.2}/src/harness_sdk/voice.py

@@ -20,7 +20,10 @@ clears the entry. The session is bound to the issuing client's connection: ``voi
 provider ``closed`` event, or the owner staying detached past the
 ``departure_grace`` window (the host's ``client_departed`` hook starts the
 timer, the re-check runs at expiry) ends it; ``voice/start`` while the live
-session's owner is detached ends the orphan and starts the new session.
+session's owner is detached ends the orphan and starts the new session, and
+``voice/start`` from the owner itself supersedes its own session — the owner
+holds one media leg, so a fresh start is the definitive signal the old leg
+died.
 Provider turn
 events render live in ``state["messages"]`` through
 ``Host.upsert_ephemeral_message`` (message id = turn id), tagged
@@ -45,8 +48,8 @@ same ``toolCalls`` projection: the client's ``voice/temp-frontend-tool-result``
 resolves the await instead of forwarding to the provider, and session end
 fails outstanding awaits.
 
-Rejections: ``voice-active`` (a session is already live and its owner
-attached), ``no-voice``
+Rejections: ``voice-active`` (another client's session is live and its
+owner attached), ``no-voice``
 (no voice manager declared, or ``voice/end`` with no session),
 ``not-voice-owner`` (``voice/end`` from another client),
 ``voice-unavailable`` (the factory or ``provider.start`` raised).
@@ -602,12 +605,20 @@ class VoiceManagerNamespace(TrayMapNamespace):
         if self._starting:
             raise _reject("voice-active", "a voice session is already live")
         session = self.live()
-        if session is not None:
-            if self._api.has_subscriber(session.clientId.value):
-                raise _reject("voice-active", "a voice session is already live")
-            await session._end("connection-drop")
+        if (
+            session is not None
+            and session.clientId.value != ctx.caller.client_id
+            and self._api.has_subscriber(session.clientId.value)
+        ):
+            raise _reject("voice-active", "a voice session is already live")
         self._starting = True
         try:
+            if session is not None:
+                await session._end(
+                    "superseded"
+                    if session.clientId.value == ctx.caller.client_id
+                    else "connection-drop"
+                )
             seed = _capped_seed(await self._api.get_voice_seed())
             tray = await self.get_or_create(uuid4().hex)
             assert isinstance(tray, VoiceManager)

{harness_sdk_python-0.32.1 → harness_sdk_python-0.32.2}/tests/test_voice.py

@@ -48,12 +48,53 @@ async def test_start_seeds_provider_and_projects_session():
         assert bench.contexts[0].client_id == drv.client_id
 
 
-async def test_start_while_active_rejects_voice_active():
+async def test_start_from_the_owner_supersedes_its_own_session():
     bench = Bench()
     async with voice_host(bench) as (drv, _app):
         await drv.command("voice/start", {"offer": OFFER})
+        superseded = bench.provider
+        bench.provider = FakeVoiceProvider()
+        rsp = await drv.command("voice/start", {"offer": OFFER})
+        assert rsp["type"] == "accepted", rsp
+        assert superseded.closed
+        await drv.wait(
+            lambda s: (voice_of(s) or {}).get("clientId") == drv.client_id
+            and (voice_of(s) or {}).get("status") == "live"
+        )
+
+
+async def test_overlapping_start_during_supersede_rejects_voice_active():
+    class SuspendedCloseProvider(FakeVoiceProvider):
+        def __init__(self):
+            super().__init__()
+            self.closing = asyncio.Event()
+            self.release = asyncio.Event()
+
+        async def close(self):
+            self.closing.set()
+            await self.release.wait()
+            await super().close()
+
+    bench = Bench(provider=SuspendedCloseProvider())
+    async with voice_host(bench) as (drv, app):
+        await drv.command("voice/start", {"offer": OFFER})
+        old = bench.provider
+        bench.provider = FakeVoiceProvider()
+        supersede = asyncio.ensure_future(
+            drv.command("voice/start", {"offer": OFFER})
+        )
+        await old.closing.wait()
         assert_rejected(
-            await drv.command("voice/start", {"offer": OFFER}), "voice-active"
+            await side_command(app, drv._client, "voice/start", {"offer": OFFER}),
+            "voice-active",
+        )
+        old.release.set()
+        rsp = await supersede
+        assert rsp["type"] == "accepted", rsp
+        assert [call[0] for call in bench.provider.calls] == ["start"]
+        await drv.wait(
+            lambda s: (voice_of(s) or {}).get("clientId") == drv.client_id
+            and (voice_of(s) or {}).get("status") == "live"
         )