npm - specrails-desktop - Versions diffs - 2.30.1 → 2.30.2 - Mend

specrails-desktop 2.30.1 → 2.30.2

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.

Files changed (28) hide show

package/client/dist/index.html CHANGED Viewed

@@ -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-F9r8mWP_.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-CI47Dq9E.js">
     <link rel="stylesheet" crossorigin href="/assets/index-Dh9i4CnN.css">
   </head>
   <body>

package/package.json CHANGED Viewed

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

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

@@ -136,9 +136,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();

package/server/dist/framework-manager.js CHANGED Viewed

@@ -204,16 +204,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 +280,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 +295,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 {