fraim-hub 2.0.295 → 2.0.297

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/dist/src/ai-hub/server.js

@@ -3328,9 +3328,17 @@ class AiHubServer {
     conversationRecordFromRun(run) {
         const lastUpdatedAt = run.updatedAt || new Date().toISOString();
         const stages = deriveStages(run, run.projectPath);
+        const rawScope = run.scope
+            ?? run.invokedArea;
+        const recordScope = rawScope === 'manager' || rawScope === 'company' ? rawScope : 'project';
+        const recordProjectPath = recordScope === 'manager'
+            ? conversation_store_1.MANAGER_SCOPE_KEY
+            : recordScope === 'company'
+                ? conversation_store_1.COMPANY_SCOPE_KEY
+                : path_1.default.resolve(run.projectPath);
         const record = {
             id: run.conversationId || run.id,
-            projectPath: path_1.default.resolve(run.projectPath),
+            projectPath: recordProjectPath,
             title: run.conversationTitle || run.jobTitle || run.jobId,
             jobId: run.jobId,
             jobTitle: run.jobTitle || run.jobId,
@@ -3392,9 +3400,7 @@ class AiHubServer {
             sourceTrigger: run.sourceTrigger,
             // Issue #708: carry the invocation scope so the record lands in (and is keyed to)
             // the right bucket. Falls back to the legacy client `invokedArea` when present.
-            scope: run.scope
-                ?? run.invokedArea
-                ?? 'project',
+            scope: recordScope,
             run: {
                 stages,
                 currentPhase: run.currentPhase || null,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "fraim-hub",
-  "version": "2.0.295",
+  "version": "2.0.297",
   "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.295",
+    "fraim": "2.0.297",
     "mongodb": "^7.0.0",
     "node-cron": "4.2.1",
     "node-edge-tts": "^1.2.10",

package/public/ai-hub/script.js

@@ -8515,6 +8515,12 @@ function convNeedsPolling(conv) {
 }
 
 function switchToConversation(id) {
+  const conv = findConversation(id);
+  const scope = convScope(conv);
+  const targetArea = scope === 'manager' || scope === 'company' ? scope : 'projects';
+  if (conv && tf.area !== targetArea) {
+    tfShowArea(targetArea);
+  }
   state.activeId = id;
   persistActiveConversationSelection(id);
   // Issue #1065: a find session belongs to one thread. srchOpenResult re-opens it with the search
@@ -8522,23 +8528,23 @@ function switchToConversation(id) {
   if (typeof convFindClose === 'function') convFindClose();
   // For Company/Manager: move the shared .page into this area's conv-host and
   // switch from the info view to the full coaching panel.
-  if (tf.area === 'company' || tf.area === 'manager') {
-    tfEnsurePageInArea(tf.area);
-    const host = document.getElementById(tf.area + '-conv-host');
-    const info = document.getElementById(tf.area + '-info-view');
+  if (targetArea === 'company' || targetArea === 'manager') {
+    tfEnsurePageInArea(targetArea);
+    const host = document.getElementById(targetArea + '-conv-host');
+    const info = document.getElementById(targetArea + '-info-view');
     if (host) host.hidden = false;
     if (info) info.hidden = true;
     // #1397: an area selection represents every explicit destination, not only
     // the Info view. Poll-driven renders must preserve this conversation until
     // the user chooses another destination or the conversation disappears.
-    if (tf.areaSelection) tf.areaSelection[tf.area] = id;
+    if (tf.areaSelection) tf.areaSelection[targetArea] = id;
   }
   renderRail();
   renderActive();
-  hydrateConversationBody(state.projectPath, id).catch((error) =>
+  hydrateConversationBody(conv ? convBucketKey(conv) : state.projectPath, id).catch((error) =>
     console.warn('Could not hydrate switched conversation body:', error));
-  const conv = activeConversation();
-  if (convNeedsPolling(conv)) {
+  const active = activeConversation();
+  if (convNeedsPolling(active)) {
     startPolling();
   } else if (state.pollHandle) {
     window.clearInterval(state.pollHandle);