fraim-hub 2.0.218 → 2.0.219

package/dist/src/ai-hub/conversation-store.js

@@ -15,10 +15,11 @@ const conversation_store_lock_1 = require("./conversation-store-lock");
 // They are NOT filesystem paths and must never be passed through path.resolve.
 exports.MANAGER_SCOPE_KEY = '@manager';
 exports.COMPANY_SCOPE_KEY = '@company';
-// Heavy fields stripped from a header (the light shape the list UI renders). Everything else
+// Fields stripped from a header (the light shape the list UI renders). Everything else
 // (id, title, jobId, jobTitle, agentName, personaKey, status, runId, createdAt, lastUpdatedAt,
-// scope, reviewHandoff, ...) is kept.
-const HEAVY_FIELDS = ['messages', 'events', 'artifacts', 'run', 'delegation'];
+// scope, reviewHandoff, ...) is kept. Client-only runtime flags are omitted too so stale index
+// cache state cannot make the frontend skip lazy body hydration.
+const HEADER_OMITTED_FIELDS = ['messages', 'events', 'artifacts', 'run', 'delegation', '_bodyLoaded', '_stopping'];
 /**
  * Issue #708: resolve the conversation store bucket key for a given scope.
  * - 'manager'/'company' → a stable sentinel key (project-independent home).
@@ -107,6 +108,44 @@ function conversationRichness(conv) {
     const events = Array.isArray(value?.events) ? value.events.length : 0;
     return messages + events;
 }
+function normalizedOptionalString(value) {
+    return typeof value === 'string' ? value.trim() : '';
+}
+function sameOptionalField(left, right) {
+    const leftValue = normalizedOptionalString(left);
+    const rightValue = normalizedOptionalString(right);
+    return !leftValue || !rightValue || leftValue === rightValue;
+}
+function sameOptionalProjectPath(left, right) {
+    const leftValue = normalizedOptionalString(left);
+    const rightValue = normalizedOptionalString(right);
+    if (!leftValue || !rightValue)
+        return true;
+    return canonicalProjectPathKey(leftValue) === canonicalProjectPathKey(rightValue);
+}
+function conversationContinuityCompatible(left, right) {
+    const a = left;
+    const b = right;
+    if (!a || !b)
+        return true;
+    const existingSwitches = Array.isArray(a.agentSwitches) ? a.agentSwitches.length : 0;
+    const incomingSwitches = Array.isArray(b.agentSwitches) ? b.agentSwitches.length : 0;
+    if (incomingSwitches > existingSwitches)
+        return true;
+    if (!sameOptionalField(a.scope, b.scope))
+        return false;
+    if (!sameOptionalProjectPath(a.projectPath, b.projectPath))
+        return false;
+    if (!sameOptionalField(a.jobId, b.jobId))
+        return false;
+    if (!sameOptionalField(a.issueNumber, b.issueNumber))
+        return false;
+    if (!sameOptionalField(a.configuredAgentId, b.configuredAgentId))
+        return false;
+    if (!sameOptionalField(a.baseHostId, b.baseHostId))
+        return false;
+    return true;
+}
 // Place a raw conversation into a bucket, deduping by id. When the id already exists, keep the
 // richer copy (more run history); tie-break on newer lastUpdatedAt. Order-independent.
 function placeConversationInBucket(bucket, conv) {
@@ -122,6 +161,9 @@ function placeConversationInBucket(bucket, conv) {
         return;
     }
     const existing = bucket.conversations[idx];
+    if (!conversationContinuityCompatible(existing, conv)) {
+        return;
+    }
     const existingScore = conversationRichness(existing);
     const incomingScore = conversationRichness(conv);
     if (incomingScore > existingScore) {
@@ -223,6 +265,8 @@ function normalizeConversation(projectPath, raw) {
         jobId: value.jobId,
         agentName: value.agentName,
         status: value.status,
+        // Issue #904: carry pauseReason explicitly so it survives the ...value spread round-trip.
+        ...(value.pauseReason !== undefined && { pauseReason: value.pauseReason }),
         createdAt: value.createdAt ?? new Date().toISOString(),
         lastUpdatedAt: value.lastUpdatedAt ?? value.createdAt ?? new Date().toISOString(),
     };
@@ -246,7 +290,7 @@ function newestFirst(a, b) {
 }
 function toHeader(conv) {
     const header = { ...conv };
-    for (const field of HEAVY_FIELDS)
+    for (const field of HEADER_OMITTED_FIELDS)
         delete header[field];
     return header;
 }