specrails-desktop 2.30.1 → 2.30.3

package/client/dist/index.html

@@ -204,7 +204,7 @@
         .specrails-splash__mark .pill { opacity: 1; }
       }
     </style>
-    <script type="module" crossorigin src="/assets/index-xkzK3E5i.js"></script>
+    <script type="module" crossorigin src="/assets/index-CewRMo0W.js"></script>
     <link rel="modulepreload" crossorigin href="/assets/rolldown-runtime-CNC7AqOf.js">
     <link rel="modulepreload" crossorigin href="/assets/jsx-runtime-zu2_FqZY.js">
     <link rel="modulepreload" crossorigin href="/assets/react-dom-DiWzBDP9.js">
@@ -276,7 +276,7 @@
     <link rel="modulepreload" crossorigin href="/assets/getTimezoneOffsetInMilliseconds-qrcBaLW1.js">
     <link rel="modulepreload" crossorigin href="/assets/formatDistanceToNow-ByKli7Q_.js">
     <link rel="modulepreload" crossorigin href="/assets/spending-DMnCXy84.js">
-    <link rel="modulepreload" crossorigin href="/assets/TicketDetailModal-7EloKP1C.js">
+    <link rel="modulepreload" crossorigin href="/assets/TicketDetailModal-CqQ5gx7O.js">
     <link rel="stylesheet" crossorigin href="/assets/index-Dh9i4CnN.css">
   </head>
   <body>

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "specrails-desktop",
-  "version": "2.30.1",
+  "version": "2.30.3",
   "license": "MIT",
   "repository": {
     "type": "git",

package/server/dist/core-update-manager.js

@@ -116,7 +116,13 @@ class CoreUpdateManager {
         }
         this.updating = true;
         this.emit('downloading', { version: requested });
-        const tmp = fs_1.default.mkdtempSync(path_1.default.join(os_1.default.tmpdir(), 'core-update-'));
+        // realpath the tmp root: on macOS os.tmpdir() is a symlinked path
+        // (/var/folders → /private/var/folders). The staged core's cli.js has an
+        // auto-run guard comparing import.meta.url (realpathed by Node's ESM
+        // loader) against process.argv[1] (the literal spawn arg) — a symlinked
+        // staging path makes them differ, so main() never runs and every
+        // install-framework/swap-current child exits 0 as a silent no-op.
+        const tmp = fs_1.default.mkdtempSync(path_1.default.join(realTmpDir(), 'core-update-'));
         try {
             this.npmInstallFn(`${exports.CORE_PACKAGE}@${requested}`, tmp);
             const coreRoot = path_1.default.join(tmp, 'node_modules', exports.CORE_PACKAGE);
@@ -136,9 +142,16 @@ class CoreUpdateManager {
             if (mat.errors.length > 0) {
                 throw new Error(`framework materialize failed: ${mat.errors.map((e) => `${e.provider}: ${e.message}`).join('; ')}`);
             }
-            const swapped = fm.swapCurrent(installed);
-            if (!swapped) {
-                throw new Error('framework swap-current failed');
+            // A materialize that reports no errors but ALSO no materialized providers
+            // wrote nothing — swapping `current` at a nonexistent version dir would
+            // fail downstream with a misleading "swap-current failed". Surface the
+            // real condition instead (mirrors FrameworkManager.versionCheck's guard).
+            if (mat.providers.length === 0) {
+                throw new Error(`framework materialize completed without materializing any provider (requested: ${providers.join(', ') || 'none'})`);
+            }
+            const swapped = fm.swapCurrentDetailed(installed);
+            if (!swapped.ok) {
+                throw new Error(`framework swap-current failed${swapped.detail ? `: ${swapped.detail}` : ''}`);
             }
             this.latestVersion = installed;
             this.lastCheckedAt = Date.now();
@@ -180,6 +193,16 @@ function uniqueProviders(values) {
     const out = Array.from(new Set(values.filter((v) => typeof v === 'string' && v.length > 0)));
     return out.length > 0 ? out : ['claude'];
 }
+/** os.tmpdir() with symlinks resolved (macOS /var/folders → /private/var/folders). */
+function realTmpDir() {
+    const dir = os_1.default.tmpdir();
+    try {
+        return fs_1.default.realpathSync(dir);
+    }
+    catch {
+        return dir;
+    }
+}
 /** GET the latest published version from the npm registry (no npm binary needed). */
 async function fetchLatestFromRegistry() {
     const controller = new AbortController();

package/server/dist/framework-manager.js

@@ -26,6 +26,22 @@ const path_resolver_1 = require("./path-resolver");
 function nodeInterpreter() {
     return (0, path_resolver_1.resolveBundledNodeExe)() ?? process.execPath;
 }
+/**
+ * Resolve symlinks in the CLI path we pass as the child's argv[1]. Core's
+ * cli.js auto-run guard compares import.meta.url (realpathed by Node's ESM
+ * loader) against pathToFileURL(process.argv[1]) — a symlinked argv[1] (e.g.
+ * a staging dir under macOS /var/folders) makes them differ, so main() never
+ * runs and the child exits 0 as a silent no-op. Realpathing here keeps the
+ * guard true regardless of where the core package lives.
+ */
+function realpathOrSelf(p) {
+    try {
+        return fs_1.default.realpathSync(p);
+    }
+    catch {
+        return p;
+    }
+}
 /** `~/.specrails/framework` — same home as the registry. */
 function frameworkRoot(home) {
     return path_1.default.join((0, artifact_registry_1.resolveHome)(home), '.specrails', 'framework');
@@ -121,9 +137,10 @@ class FrameworkManager {
     resolveCli() {
         if (this.coreRoot) {
             const cli = path_1.default.join(this.coreRoot, 'dist', 'installer', 'cli.js');
-            return fs_1.default.existsSync(cli) ? cli : null;
+            return fs_1.default.existsSync(cli) ? realpathOrSelf(cli) : null;
         }
-        return (0, bundled_core_1.getBundledCoreCli)();
+        const bundled = (0, bundled_core_1.getBundledCoreCli)();
+        return bundled ? realpathOrSelf(bundled) : null;
     }
     /** True when a usable core (override or bundled) is present (else methods no-op). */
     isAvailable() {
@@ -204,16 +221,30 @@ class FrameworkManager {
      * `swap-current` only moves the version pointer (provider-invariant).
      */
     swapCurrent(version, _provider = 'claude') {
+        return this.swapCurrentDetailed(version).ok;
+    }
+    /**
+     * Like `swapCurrent`, but on failure carries WHY it failed (`detail`) — the
+     * subprocess stderr/exit or the missing-CLI/verify-mismatch case — so callers
+     * (the core update channel) can surface an actionable error instead of the
+     * bare "swap-current failed".
+     */
+    swapCurrentDetailed(version) {
         const cli = this.resolveCli();
         if (!cli)
-            return false;
+            return { ok: false, detail: 'no usable core CLI to run swap-current with' };
         if (readCurrentFrameworkVersion(this.home) === version)
-            return true;
+            return { ok: true, detail: null };
         const fwDir = frameworkRoot(this.home);
         const res = (0, child_process_1.spawnSync)(nodeInterpreter(), [cli, 'swap-current', '--framework-dir', fwDir, '--version', version], { env: this.childEnv(), encoding: 'utf-8', timeout: 120_000 });
-        if (res.error || (res.status ?? 1) !== 0)
-            return false;
-        return readCurrentFrameworkVersion(this.home) === version;
+        if (res.error || (res.status ?? 1) !== 0) {
+            const detail = (res.error?.message ?? `${res.stderr || res.stdout || ''}`.trim()) || `exit ${res.status ?? 'unknown'}`;
+            return { ok: false, detail };
+        }
+        if (readCurrentFrameworkVersion(this.home) !== version) {
+            return { ok: false, detail: `swap-current exited 0 but current still resolves to ${readCurrentFrameworkVersion(this.home) ?? 'nothing'}` };
+        }
+        return { ok: true, detail: null };
     }
     /**
      * Compare the bundled core version with `framework/current`. When they differ
@@ -266,8 +297,8 @@ class FrameworkManager {
             // No errors yet nothing materialized (empty/duplicate-only provider list).
             return { swapped: false, version: current };
         }
-        const ok = this.swapCurrent(bundled);
-        if (ok) {
+        const swap = this.swapCurrentDetailed(bundled);
+        if (swap.ok) {
             try {
                 this.broadcast?.({ type: 'framework.updated', version: bundled });
             }
@@ -281,7 +312,7 @@ class FrameworkManager {
             this.broadcast?.({
                 type: 'framework.update_failed',
                 version: bundled,
-                errors: [{ provider: providers[0] ?? 'claude', message: 'swap-current failed' }],
+                errors: [{ provider: providers[0] ?? 'claude', message: `swap-current failed${swap.detail ? `: ${swap.detail}` : ''}` }],
             });
         }
         catch {