deep-roon-mcp 0.2.1 → 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.1";
+    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");
@@ -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

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "deep-roon-mcp",
-  "version": "0.2.1",
+  "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>",