@sym-bot/mesh-channel 0.3.14 → 0.3.15

package/.mcp.json

@@ -4,7 +4,7 @@
       "command": "npx",
       "args": [
         "-y",
-        "@sym-bot/mesh-channel@0.3.10"
+        "@sym-bot/mesh-channel@0.3.14"
       ],
       "env": {
         "SYM_RELAY_URL": "${user_config.relay_url}",

package/CHANGELOG.md

@@ -1,5 +1,12 @@
 # Changelog
 
+## 0.3.15
+
+### Fixed
+
+- **`start` now finds `claude` on Windows.** The launch used `spawnSync('claude', …)` with no shell, which does an exact-filename lookup that ignores Windows `PATHEXT` — so `start` failed with `ENOENT` even when `claude` ran fine in the shell (there it resolves a `.cmd`/`.ps1` shim or `.exe`, never bare `claude`). The launch now routes through a shell on Windows so `PATHEXT` resolution applies; whitespace args are quoted since `shell: true` forwards them unquoted. The POSIX launch path is unchanged.
+- **`start --name` no longer silently reverts identity on a stale entry.** `start` auto-injected `--force` only on a *live* entry mismatch, but `npx` rotates its cached `server.js` path on every version resolve, so the persisted entry is routinely stale yet still holds the node's name/group. On a re-run, `start` saw no live entry, pushed no `--force`, and `init`'s preserve-over-request precedence dropped the requested `--name` — reverting the node's identity to the stale name. `start` now reconciles against the persisted entry whether or not it's stale and forces the rewrite when an explicit `--name`/`--group` differs. The group is preserved when no `--group` is passed.
+
 ## 0.3.14
 
 ### Added

package/bin/install.js

@@ -151,19 +151,26 @@ if (cmd === 'start') {
 
   // Is the scope Claude Code will actually read already configured with a
   // LIVE entry?  --project → <cwd>/.mcp.json ; otherwise → ~/.claude.json
-  function liveEntryInScope() {
+  // Reconcile identity against the persisted entry whether or not its
+  // server.js path is stale. npx rotates the cached server.js path on every
+  // version resolve (…/_npx/<hash>/…), so the entry is routinely stale yet
+  // still carries the node's name/group. Comparing only against a *live*
+  // entry let an explicit --name lose to the stale name on the heal path:
+  // start saw no live entry, pushed no --force, and init's non-force
+  // precedence (preserve-over-request) kept the old identity.
+  function rawEntryInScope() {
     try {
       const p = isProject
         ? path.join(launchDir, '.mcp.json')
         : path.join(os.homedir(), '.claude.json');
       if (!fs.existsSync(p)) return null;
       const j = JSON.parse(fs.readFileSync(p, 'utf8'));
-      const e = j && j.mcpServers && j.mcpServers['claude-sym-mesh'];
-      return e && !isStaleEntry(e) ? e : null;
+      return (j && j.mcpServers && j.mcpServers['claude-sym-mesh']) || null;
     } catch { return null; }
   }
 
-  const existing = liveEntryInScope();
+  const existing = rawEntryInScope();
+  const stale = existing ? isStaleEntry(existing) : false;
   const wantName = nameArg || null;
   const wantGroup = groupArg || null;
   const mismatch = !!existing && (
@@ -171,13 +178,19 @@ if (cmd === 'start') {
     (wantGroup && (preserveGroup(existing) || 'default') !== wantGroup)
   );
 
-  // Configure only when there's nothing live yet, an explicit --name/--group
-  // differs from the current entry, or --force was passed. Otherwise launch
-  // straight away — `start` stays cheap to run every session.
-  if (!existing || mismatch || force) {
+  // Configure when there's nothing persisted yet, the persisted server.js
+  // path is stale (heal), an explicit --name/--group differs, or --force.
+  // Otherwise launch straight away — `start` stays cheap to run every
+  // session.
+  if (!existing || stale || mismatch || force) {
     const initArgs = ['init'];
     if (isProject) initArgs.push('--project');
     if (groupArg) initArgs.push('--group', groupArg);
+    // --force makes init honor the explicit --name/--group over the
+    // persisted value. Required on the rename path AND on stale-heal with
+    // an explicit --name, where init would otherwise preserve the old name
+    // and silently drop the request. Force alone never clobbers an
+    // unspecified group: resolveGroup() still preserves when no --group.
     if (existing && (mismatch || force)) initArgs.push('--force');
     const childEnv = Object.assign({}, process.env);
     if (nameArg) childEnv.SYM_NODE_NAME = nameArg;
@@ -188,7 +201,17 @@ if (cmd === 'start') {
   }
 
   console.log(`\n▶ Launching Claude Code on the SYM mesh — real-time push on.\n  (channel: ${handle}; the dev flag is temporary until Anthropic allowlists it)\n`);
-  const run = spawnSync('claude', claudeArgs, { stdio: 'inherit', cwd: launchDir });
+  // On Windows the `claude` CLI is a `.cmd`/`.ps1` shim (npm) or `.exe`
+  // (native installer). Node's spawn does an exact-filename lookup that
+  // ignores PATHEXT, so bare `spawnSync('claude', …)` returns ENOENT even
+  // when `claude` runs fine in the user's shell. Route through a shell on
+  // Windows so PATHEXT resolution kicks in; quote args that contain spaces
+  // since shell:true forwards the args unquoted.
+  const isWindows = process.platform === 'win32';
+  const spawnArgs = isWindows
+    ? claudeArgs.map((a) => (/\s/.test(a) ? `"${a}"` : a))
+    : claudeArgs;
+  const run = spawnSync('claude', spawnArgs, { stdio: 'inherit', cwd: launchDir, shell: isWindows });
   if (run.error && run.error.code === 'ENOENT') {
     process.stderr.write('ERROR: `claude` was not found on your PATH.\n');
     process.stderr.write('Install Claude Code (https://claude.com/code), make sure `claude` runs in your terminal, then re-run `sym-mesh-channel start`.\n');

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sym-bot/mesh-channel",
-  "version": "0.3.14",
+  "version": "0.3.15",
   "description": "MCP server — real-time agent-to-agent cognition for Claude Code remote teams via the SYM mesh.",
   "main": "server.js",
   "bin": {