@naxodev/opencode-music-player 0.3.0 → 0.3.1

package/README.md

@@ -8,7 +8,7 @@ It supports browsers, Spotify, Apple Music, Kaset, and other apps exposed throug
 
 One reconnecting music-session client supplies replayed and live state, provider status, transport, and daemon-owned native artwork bytes. The shared same-user daemon owns provider discovery, provider events and polling, the playback clock, and global transport ordering.
 
-OpenCode keeps plugin/controller lifecycle, the Solid compact and sidebar UI, optimistic transport presentation, seek coalescing, notifications, waveform projection, iTunes catalog fallback and downloads, conversion, bounded presentation cache/jobs, and Kitty or half-block rendering. Plugin disposal removes local listeners and presentation work, then disposes only its session client. Other clients keep the shared daemon alive.
+OpenCode keeps plugin/controller lifecycle, the Solid compact and sidebar UI, transport loading feedback, seek coalescing, notifications, waveform projection, iTunes catalog fallback and downloads, conversion, bounded presentation cache/jobs, and Kitty or half-block rendering. Plugin disposal removes local listeners and presentation work, then disposes only its session client. Other clients keep the shared daemon alive.
 
 Read the [music session architecture field guide](../../docs/music-session-architecture.html) for the daemon protocol, replay, reconnect, and cleanup model.
 
@@ -38,13 +38,12 @@ set -g allow-passthrough on
 ```
 
 > [!IMPORTANT]
-> This source targets stable `opencode v2.0.3`. The previous beta host is unsupported. See the [compatibility contract](../../docs/opencode-compatibility.md) for the exact dependency set and release requirements.
+> This source targets the exact stable host in the [validated compatibility metadata](../../docs/package-compatibility.md). The previous beta host is unsupported. See the [compatibility contract](../../docs/opencode-compatibility.md) for runtime ownership and release requirements.
 
 ## Requirements
 
 - macOS
-- OpenCode `v2.0.3`
-- The Bun-based OpenCode runtime; Bun 1.3.7 for workspace development
+- The exact tested OpenCode host and workspace Bun pin listed in the compatibility metadata
 - [`media-control`](https://github.com/ungive/media-control), recommended:
 
   ```sh
@@ -97,6 +96,8 @@ Start `opencode` and run `/plugins` in the TUI. The list should include `music-p
 
 ## Controls
 
+Play/pause uses the shared [playback toggle contract](../music-core/README.md#playback-toggle-contract). Each activation toggles the daemon's accepted state in queue order, even when this view's icon is stale. Rapid clicks remain separate commands.
+
 The compact bar appears below the active route whenever a current track exists, including while playback is paused. It remains visible when the session sidebar is collapsed. Wide terminals show the playback marker, title, and artist. Medium terminals omit the artist. Narrow terminals truncate the title, then keep only the playback marker when metadata cannot fit safely. The bar always stays on one row.
 
 | Input              | Action         |

package/dist/index.js

@@ -561,6 +561,9 @@ function createSessionSystemMedia(overrides = {}) {
     async play() {
       await (await activeClient()).play();
     },
+    async toggle() {
+      await (await activeClient()).toggle();
+    },
     async pause() {
       await (await activeClient()).pause();
     },
@@ -1037,6 +1040,10 @@ function AlbumArtwork(props) {
       paintPending = false;
       if (!disposed && ownership.isCurrent())
         paintNativeImage();
+    }).catch((error) => {
+      paintPending = false;
+      if (!disposed && ownership.isCurrent())
+        console.error("Failed to paint music artwork", error);
     });
   };
   onMount(() => props.context.renderer.on("frame", scheduleNativeImage));
@@ -1899,8 +1906,6 @@ function createController(context, dependencies = controllerDependencies) {
   };
   let transportError = null;
   let receivedSnapshot = false;
-  let snapshotEpoch = 0;
-  let playbackIntent;
   let activeSeek = null;
   let latestSeek = null;
   const pending = new Set;
@@ -1958,9 +1963,7 @@ function createController(context, dependencies = controllerDependencies) {
       } catch (error) {
         result = Promise.reject(error);
       }
-      let succeeded = false;
       Promise.resolve(result).then(() => {
-        succeeded = true;
         if (!isActive() || generation !== lifecycleGeneration)
           return;
         if (transportError !== null)
@@ -1977,10 +1980,12 @@ function createController(context, dependencies = controllerDependencies) {
         });
       }).finally(() => {
         try {
-          onSettled?.(succeeded);
+          onSettled?.();
         } finally {
           settle(resolve);
         }
+      }).catch((error) => {
+        console.error("Failed to settle music command", error);
       });
     });
   };
@@ -1999,6 +2004,8 @@ function createController(context, dependencies = controllerDependencies) {
         runSeek(next);
       else
         next?.resolves.splice(0).forEach((resolve) => resolve());
+    }).catch((error) => {
+      console.error("Failed to start music seek", error);
     });
   };
   const refreshAll = async () => {
@@ -2017,26 +2024,7 @@ function createController(context, dependencies = controllerDependencies) {
         setTransportError(errMsg(error));
     }
   };
-  const playPause = () => {
-    const nextPlaying = !(playbackIntent?.target ?? !!session.player?.is_playing);
-    const intent = {
-      target: nextPlaying,
-      startedAtEpoch: snapshotEpoch,
-      settled: false
-    };
-    playbackIntent = intent;
-    return runCommand(() => nextPlaying ? media.play() : media.pause(), (succeeded) => {
-      if (playbackIntent !== intent)
-        return;
-      if (!succeeded) {
-        playbackIntent = undefined;
-        return;
-      }
-      intent.settled = true;
-      if (snapshotEpoch > intent.startedAtEpoch && session.player?.is_playing === intent.target)
-        playbackIntent = undefined;
-    });
-  };
+  const playPause = () => runCommand(() => media.toggle());
   const seek = (positionMs) => {
     const target = seekTarget(positionMs, session.player?.track?.duration_ms);
     if (!session.player?.track || target === null || !isActive())
@@ -2065,18 +2053,13 @@ function createController(context, dependencies = controllerDependencies) {
       return;
     if (event?.type === "snapshot") {
       receivedSnapshot = true;
-      snapshotEpoch++;
       setSession((draft) => {
         draft.player = mergePlayerSnapshot(draft.player, event.state);
       });
-      if (playbackIntent?.settled && snapshotEpoch > playbackIntent.startedAtEpoch && event.state.is_playing === playbackIntent.target)
-        playbackIntent = undefined;
       return;
     }
     if (event?.type === "lifecycle") {
       setLifecycleError(event.message, event.source);
-      if (event.source === "connection" && event.message !== null)
-        playbackIntent = undefined;
       if (event.message !== null && latestSeek) {
         latestSeek.resolves.splice(0).forEach((resolve) => resolve());
         latestSeek = null;
@@ -2090,7 +2073,9 @@ function createController(context, dependencies = controllerDependencies) {
       draft.player = mergeArtworkCompletion(draft.player, event);
     });
   });
-  refreshAll();
+  refreshAll().catch((error) => {
+    console.error("Failed to publish initial music state", error);
+  });
   return {
     session,
     subscribe(listener) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@naxodev/opencode-music-player",
-  "version": "0.3.0",
+  "version": "0.3.1",
   "description": "macOS system Now Playing controls for the OpenCode 2 TUI",
   "license": "MIT",
   "author": {