fraim-hub 2.0.324 → 2.0.325

package/dist/src/ai-hub/server.js

@@ -7189,9 +7189,10 @@ class AiHubServer {
                         // job that just received coaching has not reached retrospective
                         // again yet. Clear it at send-time so a coached Done job stops
                         // offering next-job recommendations immediately, not only once the
-                        // resumed session's next seekMentoring signal arrives.
+                        // resumed session's next seekMentoring signal arrives. The active
+                        // reviewHandoff deliberately survives conversational coaching; its
+                        // lifecycle is owned by applySeekMentoringSignal (issue #1763).
                         current.nextJobRecommendations = null;
-                        current.reviewHandoff = null;
                     });
                     const startedFresh = this.runRegistry.get(run.id);
                     if (startedFresh)
@@ -7260,9 +7261,10 @@ class AiHubServer {
                         current.pendingDeliveryActionId = selectedReviewAction.deliveryActionId || null;
                     }
                     // Issue #1373 (Defect 2): see the no-session branch above for why
-                    // this can only be stale at coaching-send time.
+                    // next-job recommendations can only be stale at coaching-send time.
+                    // applySeekMentoringSignal remains the sole reviewHandoff lifecycle
+                    // owner so a conversational coaching turn does not decide the review.
                     current.nextJobRecommendations = null;
-                    current.reviewHandoff = null;
                 });
                 const started = this.runRegistry.get(run.id);
                 if (started)

package/package.json

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

package/public/ai-hub/review.css

@@ -140,6 +140,10 @@
   color: var(--muted, #888);
   filter: none;
 }
+:root[data-theme="dark"] .review-actions .rb-approve:not(:disabled):not(.managed-disabled),
+:root[data-theme="dark"] .review-actions .rb-secondary:not(:disabled):not(.managed-disabled) {
+  color: var(--bg, #1e1e1e);
+}
 .review-actions .rb-handoff {
   font-size: 12px;
   color: var(--muted);

package/public/ai-hub/script.js

@@ -7713,15 +7713,17 @@ async function approveReview(options) {
   const opts = options || {};
   const conv = activeConversation();
   if (!conv) return;
+  const shouldResume = !!opts.resumeAfterApproval
+    && !!conv.runId
+    && convAwaitingReview(conv);
+  if (opts.resumeAfterApproval && !shouldResume) {
+    showStatus('Approve could not resume this run. The review gate is no longer active. Use Mark complete if you intend to close it manually.', true);
+    return;
+  }
   if (state.pollHandle) {
     window.clearInterval(state.pollHandle);
     state.pollHandle = null;
   }
-  const shouldResume = !!opts.resumeAfterApproval
-    && !!conv.runId
-    && !!conv.sessionId
-    && !!reviewHandoffForConversation(conv)
-    && convAwaitingReview(conv);
   if (shouldResume) {
     const selectedReviewAction = opts.reviewAction && typeof opts.reviewAction === 'object'
       ? opts.reviewAction