deep-roon-mcp 0.2.0 → 0.2.2

package/README.md

@@ -179,6 +179,8 @@ status; it holds no secrets).
 
 - Short connections, as Roon Labs asked: opened for a question, closed after two minutes of quiet, one at a time, questions
   one after another; a PING every 10 s while open, so the Core doesn't drop the session.
+- Several clients at once (Claude Desktop and Claude Code, or ChatGPT next to them) work: Roon keeps one connection
+  per extension, and the client that asks a question takes it over for that moment.
 - Album, track and artist keys stay valid between calls.
 - The log goes to stderr (your MCP client keeps it).
 - At the start it asks GitHub once whether a newer release is out (`DEEP_ROON_NO_UPDATE_CHECK=1` turns that off). Nothing

package/dist/deep-roon-mcp.js

@@ -5340,7 +5340,7 @@ var require_core2 = __commonJS({
     Object.defineProperty(exports2, "__esModule", { value: true });
     var id_1 = require_id();
     var ref_1 = require_ref();
-    var core = [
+    var core2 = [
       "$schema",
       "$id",
       "$defs",
@@ -5350,7 +5350,7 @@ var require_core2 = __commonJS({
       id_1.default,
       ref_1.default
     ];
-    exports2.default = core;
+    exports2.default = core2;
   }
 });
 
@@ -13239,7 +13239,7 @@ var require_stream = __commonJS({
       };
       duplex._final = function(callback) {
         if (ws.readyState === ws.CONNECTING) {
-          ws.once("open", /* @__PURE__ */ __name(function open3() {
+          ws.once("open", /* @__PURE__ */ __name(function open4() {
             duplex._final(callback);
           }, "open"));
           return;
@@ -13260,7 +13260,7 @@ var require_stream = __commonJS({
       };
       duplex._write = function(chunk, encoding, callback) {
         if (ws.readyState === ws.CONNECTING) {
-          ws.once("open", /* @__PURE__ */ __name(function open3() {
+          ws.once("open", /* @__PURE__ */ __name(function open4() {
             duplex._write(chunk, encoding, callback);
           }, "open"));
           return;
@@ -14277,7 +14277,7 @@ var require_moomsg = __commonJS({
 var require_core3 = __commonJS({
   "node_modules/node-roon-api/core.js"(exports2, module2) {
     "use strict";
-    function Core(moo, roon, registration) {
+    function Core(moo, roon2, registration) {
       this.moo = moo;
       this.core_id = registration.core_id;
       this.display_name = registration.display_name;
@@ -14285,8 +14285,8 @@ var require_core3 = __commonJS({
       this.registration = registration;
       this.services = {};
       var svcs = {};
-      roon.services_opts.required_services.forEach((svcobj) => svcobj.services.forEach((svc) => svcs[svc.name] = svcobj));
-      roon.services_opts.optional_services.forEach((svcobj) => svcobj.services.forEach((svc) => svcs[svc.name] = svcobj));
+      roon2.services_opts.required_services.forEach((svcobj) => svcobj.services.forEach((svc) => svcs[svc.name] = svcobj));
+      roon2.services_opts.optional_services.forEach((svcobj) => svcobj.services.forEach((svc) => svcs[svc.name] = svcobj));
       registration.provided_services.forEach((e) => {
         this.services[svcs[e].name] = new svcs[e](this);
       });
@@ -14903,24 +14903,24 @@ var require_lib = __commonJS({
         });
         this.pairing_service_1 = {
           services: [svc],
-          found_core: /* @__PURE__ */ __name((core) => {
+          found_core: /* @__PURE__ */ __name((core2) => {
             if (!this.paired_core_id) {
               let settings3 = this.get_persisted_state();
-              settings3.paired_core_id = core.core_id;
+              settings3.paired_core_id = core2.core_id;
               this.set_persisted_state(settings3);
-              this.paired_core_id = core.core_id;
-              this.paired_core = core;
+              this.paired_core_id = core2.core_id;
+              this.paired_core = core2;
               this.is_paired = true;
               svc.send_continue_all("subscribe_pairing", "Changed", { paired_core_id: this.paired_core_id });
             }
-            if (core.core_id == this.paired_core_id) {
-              if (this.extension_opts.core_paired) this.extension_opts.core_paired(core);
+            if (core2.core_id == this.paired_core_id) {
+              if (this.extension_opts.core_paired) this.extension_opts.core_paired(core2);
             }
           }, "found_core"),
-          lost_core: /* @__PURE__ */ __name((core) => {
-            if (core.core_id == this.paired_core_id)
+          lost_core: /* @__PURE__ */ __name((core2) => {
+            if (core2.core_id == this.paired_core_id)
               this.is_paired = false;
-            if (this.extension_opts.core_unpaired) this.extension_opts.core_unpaired(core);
+            if (this.extension_opts.core_unpaired) this.extension_opts.core_unpaired(core2);
           }, "lost_core")
         };
         o.provided_services.push(this.pairing_service_1);
@@ -15113,8 +15113,8 @@ var require_lib2 = __commonJS({
       return zo.zone_id;
     }
     __name(zoid, "zoid");
-    function RoonApiTransport2(core) {
-      this.core = core;
+    function RoonApiTransport2(core2) {
+      this.core = core2;
       this._queues = {};
     }
     __name(RoonApiTransport2, "RoonApiTransport");
@@ -15416,7 +15416,7 @@ var require_lib2 = __commonJS({
 var version2;
 var init_package = __esm({
   "package.json"() {
-    version2 = "0.2.0";
+    version2 = "0.2.2";
   }
 });
 
@@ -15443,7 +15443,8 @@ function configDir() {
 function startOfficial(target2) {
   if (started) return;
   started = true;
-  const roon = new import_node_roon_api.default({
+  core = target2;
+  roon = new import_node_roon_api.default({
     extension_id: "nl.renebouwmeester.deep-roon-mcp",
     display_name: "Deep Roon MCP",
     display_version: version2,
@@ -15453,13 +15454,13 @@ function startOfficial(target2) {
     log_level: "none",
     get_persisted_state: readState,
     set_persisted_state: writeState,
-    core_paired(core) {
-      transport = core.services.RoonApiTransport ?? Object.entries(core.services ?? {}).find(([k]) => k.startsWith("RoonApiTransport"))?.[1];
+    core_paired(core2) {
+      transport = core2.services.RoonApiTransport ?? Object.entries(core2.services ?? {}).find(([k]) => k.startsWith("RoonApiTransport"))?.[1];
       if (!transport) {
         log("paired, but the Core offers no transport service");
         return;
       }
-      log(`paired with ${core.display_name} (Roon \u203A Settings \u203A Extensions)`);
+      log(`paired with ${core2.display_name} (Roon \u203A Settings \u203A Extensions)`);
       transport.subscribe_zones((cmd, data) => {
         if (cmd === "Subscribed") {
           zones.clear();
@@ -15490,25 +15491,26 @@ function startOfficial(target2) {
     }
   });
   roon.init_services({ required_services: [import_node_roon_api_transport.default] });
-  let tries = 0;
-  const connect = /* @__PURE__ */ __name(() => {
-    roon.ws_connect({
-      host: target2.host,
-      port: target2.port,
-      onclose: /* @__PURE__ */ __name(() => {
-        transport = null;
-        zones.clear();
-        zonesIn = false;
-        const delay = [5, 15, 60][tries] ?? 300;
-        tries += 1;
-        setTimeout(connect, delay * 1e3).unref();
-      }, "onclose")
-    });
-  }, "connect");
   connect();
 }
+function connect() {
+  if (open2 || !roon || !core) return;
+  open2 = true;
+  roon.ws_connect({
+    host: core.host,
+    port: core.port,
+    onclose: /* @__PURE__ */ __name(() => {
+      open2 = false;
+      transport = null;
+      zones.clear();
+      zonesIn = false;
+      log("the extension connection closed (another copy of Deep Roon MCP may have taken it over) \u2014 it opens again when a tool needs it");
+    }, "onclose")
+  });
+}
 async function paired(ms = 8e3) {
   if (transport) return transport;
+  connect();
   await new Promise((resolve) => {
     waiting.push(resolve);
     setTimeout(resolve, ms).unref();
@@ -15547,7 +15549,7 @@ async function setVolume(name, change) {
   }
   return { zone: z2.display_name, outputs: outs.map((o) => ({ output: o.display_name, scale: `${o.volume.min}\u2013${o.volume.max}` })), volume: change.value, muted: change.mute };
 }
-var import_fs, import_os, import_path, import_node_roon_api, import_node_roon_api_transport, STATE_FILE, readState, writeState, transport, zones, started, waiting, zonesIn, zoneWaiting, officialZones, isPaired;
+var import_fs, import_os, import_path, import_node_roon_api, import_node_roon_api_transport, STATE_FILE, readState, writeState, transport, zones, started, waiting, zonesIn, zoneWaiting, roon, core, open2, officialZones, isPaired;
 var init_official = __esm({
   "src/core/official.ts"() {
     "use strict";
@@ -15578,7 +15580,11 @@ var init_official = __esm({
     waiting = [];
     zonesIn = false;
     zoneWaiting = [];
+    roon = null;
+    core = null;
+    open2 = false;
     __name(startOfficial, "startOfficial");
+    __name(connect, "connect");
     __name(paired, "paired");
     officialZones = /* @__PURE__ */ __name(() => [...zones.values()], "officialZones");
     isPaired = /* @__PURE__ */ __name(() => !!transport, "isPaired");
@@ -15637,7 +15643,7 @@ function close(reason) {
     log(`connection closed (${reason})`);
   }
 }
-async function open2() {
+async function open3() {
   if (client) return client;
   const t = await findTarget();
   await paired();
@@ -15678,7 +15684,7 @@ function withCore(work2) {
       idleTimer = null;
     }
     try {
-      return await work2(await open2());
+      return await work2(await open3());
     } catch (e) {
       if (!(e instanceof RoonError && (e.kind === "not_found" || e.kind === "invalid"))) close("after an error");
       throw e;
@@ -15692,7 +15698,7 @@ function withCore(work2) {
 }
 async function fresh() {
   close("a fresh graph");
-  return open2();
+  return open3();
 }
 function closeSoon(reason) {
   close(reason);
@@ -15722,7 +15728,7 @@ var init_session = __esm({
     __name(findTarget, "findTarget");
     __name(locate, "locate");
     __name(close, "close");
-    __name(open2, "open");
+    __name(open3, "open");
     __name(withCore, "withCore");
     __name(fresh, "fresh");
     __name(closeSoon, "closeSoon");
@@ -48809,7 +48815,7 @@ function registerLibraryTools(server) {
   }, ({ limit: n, offset: o }) => run(() => newForYou(n ?? 30, o ?? 0)));
   server.registerTool("roon_mixes", {
     title: "Daily Mixes",
-    description: "The Daily Mixes from Roon's Home: today's personal mixes, each built around an artist (or genre) the user listens to, with a few `touchstones` and a `key` for roon_play_mix. They change every day.",
+    description: 'Roon\'s own Daily Mixes, from Roon\'s Home \u2014 NOT TIDAL\'s mixes: for "TIDAL mixes", "My Mix", "My Daily Discovery" or TIDAL\'s daily mixes use roon_tidal_playlists with `shelf: "my_mixes"`. These are today\'s personal Roon mixes, each built around an artist (or genre) the user listens to, with a few `touchstones` and a `key` for roon_play_mix. They change every day.',
     inputSchema: {},
     annotations: READ2
   }, () => run(() => mixes()));
@@ -48827,7 +48833,7 @@ function registerLibraryTools(server) {
   }, () => run(() => playlists2()));
   server.registerTool("roon_tidal_playlists", {
     title: "TIDAL playlists",
-    description: "TIDAL's playlists as Roon shows them, when TIDAL is signed in: the editorial shelves `new`, `recommended`, `exclusive`, `masters`, `popular`, `essential`, `hires_classics`, `recently_updated`, `forgotten_favorites`; the user's own TIDAL mixes `my_mixes` (My Daily Discovery, My Mix 1 \u2026) and `history_mixes` (My Most Listened, the yearly ones \u2026); or the playlists of one TIDAL `genre` or `mood` (ids from roon_genres with `service: \"tidal\"`). Each playlist has a `key` for roon_playlist (its tracks) and roon_play_playlist, a description, the number of tracks and a few of its artists.",
+    description: "TIDAL's playlists as Roon shows them, when TIDAL is signed in: the editorial shelves `new`, `recommended`, `exclusive`, `masters`, `popular`, `essential`, `hires_classics`, `recently_updated`, `forgotten_favorites`; the user's own TIDAL mixes `my_mixes` (My Daily Discovery, My Mix 1 \u2026, My New Arrivals \u2014 what the user means by their TIDAL (daily) mixes; not roon_mixes, which are Roon's) and `history_mixes` (My Most Listened, the yearly ones \u2026); or the playlists of one TIDAL `genre` or `mood` (ids from roon_genres with `service: \"tidal\"`). Each playlist has a `key` for roon_playlist (its tracks \u2014 readable without playing) and roon_play_playlist, a description, the number of tracks and a few of its artists.",
     inputSchema: {
       shelf: external_exports.enum(Object.keys(PLAYLIST_SHELVES)).optional().describe("Default new."),
       genre: external_exports.string().optional().describe("A TIDAL genre (roon_genres, service tidal)."),
@@ -48972,7 +48978,7 @@ function registerStatusTool(server, internal) {
     annotations: { readOnlyHint: true, openWorldHint: false }
   }, () => run(async () => {
     const t = await findTarget().catch((e) => ({ error: e.message }));
-    const core = "error" in t ? { error: t.error } : { name: t.info.name, version: t.info.version, address: t.host };
+    const core2 = "error" in t ? { error: t.error } : { name: t.info.name, version: t.info.version, address: t.host };
     if (!("error" in t)) await paired(5e3).catch(() => null);
     const paired2 = isPaired();
     if (internal && paired2 && !selfTestResults()) await selfTest(runSelfTest);
@@ -48980,7 +48986,7 @@ function registerStatusTool(server, internal) {
     return {
       deep_roon_mcp: version2,
       newer_version: updateInfo() ?? void 0,
-      core,
+      core: core2,
       enabled_in_roon: paired2 || 'not yet: Roon \u203A Settings \u203A Extensions \u203A Enable next to "Deep Roon MCP"',
       internal_protocol: internal ? checks2 ? checks2.every((c) => c.ok) ? "works" : "partly broken \u2014 see self_test" : "not tested yet" : "switched off (only the official API)",
       self_test: checks2 ?? void 0
@@ -48998,6 +49004,7 @@ var INSTRUCTIONS = `Deep Roon MCP connects to the user's own Roon Core (music pl
 - Albums, tracks and artists come with a \`key\`; keys stay valid between calls \u2014 no need to search again.
 - Reviews and biographies are Roon's own texts: when asked for them, show them in full with the author's name.
 - What plays changes all the time: read roon_now_playing afresh before describing it.
+- "Daily Mixes" are Roon's own (roon_mixes); the user's TIDAL mixes (My Mix 1 \u2026, My Daily Discovery) are roon_tidal_playlists with shelf my_mixes, and their tracks can be read with roon_playlist.
 - Unofficial: this uses Roon's internal protocol, which a Roon update can change; if a tool says so, tell the user to check for an update.`;
 async function main() {
   const server = new McpServer({ name: "deep-roon-mcp", title: "Deep Roon MCP", version: version2, websiteUrl: "https://github.com/renebouwmeester/deep-roon-mcp" }, { instructions: INSTRUCTIONS });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "deep-roon-mcp",
-  "version": "0.2.0",
+  "version": "0.2.2",
   "description": "An MCP server for Roon that reaches the parts the official API doesn't: reviews, biographies, Qobuz's and TIDAL's shelves, TIDAL's playlists, your history, search across your library and the streaming services",
   "license": "Apache-2.0",
   "author": "René Bouwmeester <21277119+renebouwmeester@users.noreply.github.com>",