fraim-framework 2.0.194 → 2.0.195

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

@@ -2252,9 +2252,20 @@ class AiHubServer {
                 if (!Array.isArray(body.conversations)) {
                     return res.status(400).json({ error: 'conversations array required' });
                 }
+                // The client strips server-owned heavy fields (messages/events/artifacts/run/
+                // delegation) from its PUT payload to stay under the body-size limit. Merge them
+                // back from the stored record so a client-owned change (e.g. reviewApproved) never
+                // erases run history. List membership still comes from the incoming array, so
+                // deletes are honored (a conversation absent here is dropped).
+                const prior = this.conversationStore.loadProject(key);
+                const priorById = new Map(prior.conversations.map((entry) => [entry.id, entry]));
+                const conversations = body.conversations.map((incoming) => {
+                    const existing = incoming && incoming.id ? priorById.get(incoming.id) : undefined;
+                    return existing ? { ...existing, ...incoming } : incoming;
+                });
                 const saved = this.conversationStore.replaceProject(key, {
                     activeId: body.activeId ?? null,
-                    conversations: body.conversations,
+                    conversations,
                 });
                 return res.json({ projectPath: key, scope: scope ?? 'project', ...saved, source: 'disk' });
             }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "fraim-framework",
-  "version": "2.0.194",
+  "version": "2.0.195",
   "description": "FRAIM: AI Workforce Infrastructure — the organizational capability that turns AI agents into an accountable workforce, their operators into capable AI managers, and executives into leaders with clear optics on AI proficiency.",
   "main": "index.js",
   "bin": {

package/public/ai-hub/index.html

@@ -33,18 +33,18 @@
             <div class="am-email" id="am-email"></div>
           </div>
         </div>
-        <button class="am-item" id="am-account" type="button">
+        <button class="am-item" id="am-account" type="button">
           <span class="am-ico">👤</span>
           <div><div class="am-label">Account</div><div class="am-sub">API key, sessions, billing</div></div>
-        </button>
+        </button>
         <button class="am-item" id="am-brain" type="button">
           <span class="am-ico">🧠</span>
           <div><div class="am-label">Brain</div><div class="am-sub">Everything your team knows</div></div>
         </button>
-        <button class="am-item" id="am-analytics" type="button">
+        <button class="am-item" id="am-analytics" type="button">
           <span class="am-ico">📊</span>
           <div><div class="am-label">Analytics</div><div class="am-sub">Usage, cost, and ROI</div></div>
-        </button>
+        </button>
         <button class="am-item am-theme" id="am-theme-toggle" type="button" role="switch" aria-checked="false">
           <span class="am-ico" id="am-theme-ico">🌙</span>
           <div><div class="am-label">Dark mode</div><div class="am-sub" id="am-theme-sub">Off</div></div>
@@ -141,21 +141,21 @@
     </div>
   </section>
 
-  <section class="hub-area" id="area-connected" hidden>
-    <div class="connected-shell">
-      <div class="connected-toolbar">
-        <div class="connected-title-block">
-          <div class="connected-kicker" id="connected-kicker">Connected FRAIM</div>
-          <div class="connected-title" id="connected-title">Account</div>
-        </div>
-        <div class="connected-origin" id="connected-origin"></div>
-        <button class="connected-close" id="connected-close" type="button">Back</button>
-      </div>
-      <iframe id="connected-frame" class="connected-frame" title="FRAIM connected surface" allow="clipboard-read; clipboard-write"></iframe>
-    </div>
-  </section>
-
-  <!-- Issue #512: Projects area wraps the existing single-surface Hub. -->
+  <section class="hub-area" id="area-connected" hidden>
+    <div class="connected-shell">
+      <div class="connected-toolbar">
+        <div class="connected-title-block">
+          <div class="connected-kicker" id="connected-kicker">Connected FRAIM</div>
+          <div class="connected-title" id="connected-title">Account</div>
+        </div>
+        <div class="connected-origin" id="connected-origin"></div>
+        <button class="connected-close" id="connected-close" type="button">Back</button>
+      </div>
+      <iframe id="connected-frame" class="connected-frame" title="FRAIM connected surface" allow="clipboard-read; clipboard-write"></iframe>
+    </div>
+  </section>
+
+  <!-- Issue #512: Projects area wraps the existing single-surface Hub. -->
   <section class="hub-area on" id="area-projects">
     <div class="proj-tabs" id="proj-tabs">
       <button class="ptab on" id="ptab-overview" type="button" data-view="overview">Overview</button>
@@ -868,6 +868,6 @@
     </div>
   </div>
 
-  <script src="./script.js"></script>
+  <script src="./script.js?v=conv-persist-20260705"></script>
 </body>
 </html>

package/public/ai-hub/script.js

@@ -507,11 +507,29 @@ function normalizeGeminiConversationMessages(conv) {
   return changed;
 }
 
+// Server-owned conversation fields. The server re-derives these from the run
+// registry and writes them to the conversation store on every run update
+// (persistRunConversation), so the client must NOT re-upload them: a large run's
+// messages/events/artifacts push the PUT body past the server's size limit, the
+// request 413s, the error is swallowed, and the client-owned flags in the SAME
+// payload (e.g. reviewApproved from "Mark complete") never persist. Strip them
+// here; the server's PUT handler merges them back from the stored record.
+const SERVER_OWNED_CONV_FIELDS = ['messages', 'events', 'artifacts', 'run', 'delegation'];
+function slimConversationForPersist(conv) {
+  if (!conv || typeof conv !== 'object') return conv;
+  const slim = { ...conv };
+  for (const field of SERVER_OWNED_CONV_FIELDS) delete slim[field];
+  return slim;
+}
+function slimConversationsForPersist(list) {
+  return Array.isArray(list) ? list.map(slimConversationForPersist) : list;
+}
+
 function projectConversationPayload() {
   return {
     projectPath: state.projectPath || '',
     activeId: state.activeId || null,
-    conversations: projectConversations(),
+    conversations: slimConversationsForPersist(projectConversations()),
   };
 }
 
@@ -537,7 +555,7 @@ function scheduleConversationDiskPersist() {
         await requestJson('/api/ai-hub/conversations', {
           method: 'PUT',
           headers: { 'Content-Type': 'application/json' },
-          body: JSON.stringify({ scope, activeId: null, conversations: bucket }),
+          body: JSON.stringify({ scope, activeId: null, conversations: slimConversationsForPersist(bucket) }),
         });
       }
       state.conversationDiskAvailable = true;