fraim 2.0.138 → 2.0.140

package/dist/src/first-run/session-service.js

@@ -762,7 +762,6 @@ class FirstRunSessionService {
             const hubServer = new AiHubServer({ projectPath: this.state.workspacePath });
             await hubServer.start(port);
             const hubUrl = `http://127.0.0.1:${port}/ai-hub/?firstRun=true`;
-            this.openBrowser(hubUrl);
             appendInstallLog(`hub-opened ${hubUrl}`);
             return {
                 ok: true,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "fraim",
-  "version": "2.0.138",
+  "version": "2.0.140",
   "description": "FRAIM CLI - Framework for Rigor-based AI Management (alias for fraim-framework)",
   "main": "index.js",
   "bin": {

package/public/ai-hub/script.js

@@ -366,13 +366,20 @@ function renderActive() {
 
 // Render the inline employee selector shown in the coach section of an
 // active conversation. Allows switching agents without reopening the modal.
+//
+// Two invariants keep agent-switching working correctly:
+//   1. sel.value is only reset when the active conversation changes — not on
+//      every poll tick. This preserves a user's in-progress selection.
+//   2. conv.employeeId is NOT updated by the change handler, only by the send
+//      handler after a restart completes. This keeps the comparison in the send
+//      handler accurate: sel.value !== conv.employeeId ↔ "user changed agent".
 function renderActiveEmployeeSelect(conv) {
   const sel = els['active-employee-select'];
   if (!sel) return;
   const employees = state.bootstrap?.employees || [];
   if (employees.length === 0) { sel.hidden = true; return; }
   sel.hidden = false;
-  // Only rebuild the options when the employee list changes (avoid reset on every poll).
+  // Rebuild options when the employee list changes.
   const newKey = employees.map((e) => `${e.id}:${e.available}`).join('|');
   if (sel.dataset.optionsKey !== newKey) {
     sel.innerHTML = '';
@@ -385,8 +392,14 @@ function renderActiveEmployeeSelect(conv) {
     }
     sel.dataset.optionsKey = newKey;
   }
-  const current = (conv && conv.employeeId) || state.selectedEmployeeId || 'claude';
-  if (sel.value !== current) sel.value = current;
+  // Only reset the selected value when the conversation itself changes.
+  // Never reset it during normal poll ticks — that would clobber the user's
+  // in-progress agent selection before they click Send.
+  const convId = (conv && conv.id) || '';
+  if (sel.dataset.convId !== convId) {
+    sel.dataset.convId = convId;
+    sel.value = (conv && conv.employeeId) || state.selectedEmployeeId || 'claude';
+  }
 }
 
 // Issue #347 R1 — render the pizza tracker. Reads conv.run.stages and
@@ -1479,13 +1492,11 @@ function wireEvents() {
 
   if (els['active-employee-select']) {
     els['active-employee-select'].addEventListener('change', () => {
-      const conv = activeConversation();
-      const newAgent = els['active-employee-select'].value;
-      state.selectedEmployeeId = newAgent;
-      if (conv) {
-        conv.employeeId = newAgent;
-        persistConversations();
-      }
+      // Only update the global preference here. Do NOT update conv.employeeId —
+      // the send handler compares sel.value vs conv.employeeId to detect a
+      // switch; updating conv here would make them equal and the restart would
+      // never fire.
+      state.selectedEmployeeId = els['active-employee-select'].value;
     });
   }