fraim-hub 2.0.294 → 2.0.296

package/dist/src/ai-hub/desktop-auto-updater.js

@@ -41,8 +41,14 @@ async function runMandatoryDesktopUpdateCheckOnce(options) {
         return { action: 'download-failed', availableVersion, error: message };
     }
     try {
-        logger.info(`[fraim] desktop update ${availableVersion ?? 'unknown'} downloaded; installing`);
-        updater.quitAndInstall(false, true);
+        logger.info(`[fraim] desktop update ${availableVersion ?? 'unknown'} downloaded; installing silently`);
+        // Issue #1415 (follow-up): silent, not the interactive NSIS wizard - a native rebuild is not
+        // rare on this codebase (most Hub-owned changes trigger one), so an assisted install prompt
+        // here directly undercuts #1415's actual goal: the app updating itself seamlessly, with no
+        // action required. `oneClick: false` in the NSIS config still governs a user's own first,
+        // manual install from the website - that's a separate, deliberate choice and untouched here.
+        // electron-builder's NSIS installers support a silent `/S` apply regardless of that setting.
+        updater.quitAndInstall(true, true);
         return { action: 'installing', availableVersion };
     }
     catch (error) {

package/dist/src/ai-hub/hub-instance-reconciliation.js

@@ -35,11 +35,19 @@ const hub_runtime_file_1 = require("./hub-runtime-file");
 const project_fraim_paths_1 = require("../core/utils/project-fraim-paths");
 const version_utils_1 = require("../cli/utils/version-utils");
 const process_liveness_1 = require("./process-liveness");
+// Issue #1415 (follow-up): confirmed live - a kill attempt can fail for a reason other than "the
+// process is already gone." An orphan Hub started from an elevated (Administrator) context could
+// not be terminated by this call running as a standard user - Windows refuses cross-privilege
+// termination, taskkill exits non-zero with "Access is denied," and that failure used to vanish
+// into this catch block with zero trace. There is nothing this function can do about that
+// permission gap (escalating would mean a UAC prompt, defeating the "seamless" goal this whole
+// fix exists for) - but the failure must be visible instead of silently indistinguishable from a
+// successful kill, or a real remaining orphan reads as "the sweep just didn't run."
 function killPid(pid) {
     try {
         if (process.platform === 'win32') {
             // /T kills the entire process tree; /F forces termination without prompting.
-            (0, child_process_1.execFileSync)('taskkill', ['/PID', String(pid), '/T', '/F'], { stdio: 'ignore' });
+            (0, child_process_1.execFileSync)('taskkill', ['/PID', String(pid), '/T', '/F'], { stdio: 'pipe' });
         }
         else {
             // tree-kill sends SIGTERM to the process group so child Electron helper processes
@@ -47,8 +55,11 @@ function killPid(pid) {
             (0, tree_kill_1.default)(pid, 'SIGTERM');
         }
     }
-    catch {
-        /* already gone */
+    catch (error) {
+        const detail = error instanceof Error && 'stderr' in error
+            ? String(error.stderr ?? error.message)
+            : String(error);
+        console.warn(`[fraim] could not kill pid ${pid} (may already be gone, or may lack permission - e.g. a process started elevated): ${detail.trim()}`);
     }
 }
 function isPortFree(port, timeoutMs = 500) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "fraim-hub",
-  "version": "2.0.294",
+  "version": "2.0.296",
   "description": "FRAIM Hub local companion package.",
   "author": "Sid Mathur <sid.mathur@gmail.com>",
   "homepage": "https://github.com/mathursrus/FRAIM#readme",
@@ -211,7 +211,7 @@
     "electron-updater": "^6.8.9",
     "express": "^5.2.1",
     "extract-zip": "^2.0.1",
-    "fraim": "2.0.294",
+    "fraim": "2.0.296",
     "mongodb": "^7.0.0",
     "node-cron": "4.2.1",
     "node-edge-tts": "^1.2.10",