@ai-matrx/associations 0.13.137 → 0.13.138

package/CHANGELOG.md

@@ -1,5 +1,18 @@
 # Changelog — @ai-matrx/associations
 
+
+## 0.13.138
+
+**A conversation's file inventory is read once per tab.** `useContainerLinks` kept the
+`conversation_files` answer in its own state, so a chat tile that slept and woke, a Remove +
+Undo and every second view of the same conversation read it again. It now lives in one cache
+entry per conversation per store: the first view reads, later views and remounts render the
+kept list, a read in flight is joined, a store write to the conversation still re-reads it,
+and an older answer never lands over a newer one. Guard: `react/__tests__/hooks.test.tsx`
+"reads a conversation's files once for a remount and a second view" (red on 0.13.137).
+
+Consumer action: none.
+
 ## Unreleased
 
 **Resource groups are a dot and a name.** `AssociationCardGrid` no longer renders the one-line group descriptions ("Incoming sources of truth (Knowledge In).", "Filed under"'s line). Everything else of the restored tiles look stays. Arman 2026-09-30: descriptions are banned on in-app pages (`common-docs/policies/interface-text-is-layout.md`). `CONTENT_ROLES[].tagline` stays in the data for now. No consumer action required.

package/dist/react/index.cjs

@@ -1823,6 +1823,66 @@ function metadataRecord(metadata) {
   }
   return Object.fromEntries(Object.entries(metadata));
 }
+var conversationFileCaches = /* @__PURE__ */ new WeakMap();
+function conversationFilesFor(store) {
+  const existing = conversationFileCaches.get(store);
+  if (existing) return existing;
+  const entries = /* @__PURE__ */ new Map();
+  const inflight = /* @__PURE__ */ new Map();
+  const seqs = /* @__PURE__ */ new Map();
+  const listeners = /* @__PURE__ */ new Set();
+  const set = (entry) => {
+    entries.set(entry.key, entry);
+    for (const listener of listeners) listener();
+  };
+  const cache = {
+    get: (key) => entries.get(key) ?? null,
+    subscribe(listener) {
+      listeners.add(listener);
+      return () => {
+        listeners.delete(listener);
+      };
+    },
+    load(key, conversationId, force) {
+      const kept = entries.get(key);
+      if (!force) {
+        const running = inflight.get(key);
+        if (running) return running;
+        if (kept && !kept.loading && kept.error === null) return Promise.resolve();
+      }
+      const seq = (seqs.get(key) ?? 0) + 1;
+      seqs.set(key, seq);
+      set({ key, files: kept?.files ?? [], error: kept?.error ?? null, loading: true });
+      const request = store.services.associations.listConversationFiles(conversationId).then((result) => {
+        if (seqs.get(key) !== seq) return;
+        if (isAssociationsRpcErr(result)) {
+          set({ key, files: entries.get(key)?.files ?? [], error: result.error.message, loading: false });
+          return;
+        }
+        set({
+          key,
+          files: result.data.files.map((file) => ({
+            edgeId: `conversation-file:${file.fileId}`,
+            resourceId: file.fileId,
+            token: "file",
+            role: null,
+            label: file.label,
+            metadata: file.metadata
+          })),
+          error: null,
+          loading: false
+        });
+      });
+      const tracked = request.finally(() => {
+        if (inflight.get(key) === tracked) inflight.delete(key);
+      });
+      inflight.set(key, tracked);
+      return tracked;
+    }
+  };
+  conversationFileCaches.set(store, cache);
+  return cache;
+}
 function useContainerLinks(args) {
   const { containerType, containerId, orgId } = args;
   const outgoing = args.direction === "outgoing";
@@ -1838,58 +1898,21 @@ function useContainerLinks(args) {
     id: containerId
   });
   const conversationKey = !outgoing && containerType === "conversation" && containerId ? `conversation:${containerId}` : null;
-  const activeConversationKey = (0, import_react3.useRef)(conversationKey);
-  const conversationGeneration = (0, import_react3.useRef)(0);
-  const [conversationResult, setConversationResult] = (0, import_react3.useState)(null);
-  (0, import_react3.useLayoutEffect)(() => {
-    activeConversationKey.current = conversationKey;
-    conversationGeneration.current += 1;
-  }, [conversationKey]);
-  const loadConversationFiles = (0, import_react3.useCallback)(async () => {
-    const key = conversationKey;
-    if (!key || !containerId) return;
-    if (activeConversationKey.current !== key) return;
-    const generation = ++conversationGeneration.current;
-    setConversationResult((current) => ({
-      key,
-      files: current?.key === key ? current.files : [],
-      // Keep the visible failure context while retrying so the same control
-      // can disable itself and show progress instead of disappearing.
-      error: current?.key === key ? current.error : null,
-      loading: true
-    }));
-    const result = await store.services.associations.listConversationFiles(containerId);
-    if (generation !== conversationGeneration.current || activeConversationKey.current !== key) {
-      return;
-    }
-    if (isAssociationsRpcErr(result)) {
-      setConversationResult((current) => ({
-        key,
-        files: current?.key === key ? current.files : [],
-        error: result.error.message,
-        loading: false
-      }));
-      return;
-    }
-    setConversationResult({
-      key,
-      files: result.data.files.map((file) => ({
-        edgeId: `conversation-file:${file.fileId}`,
-        resourceId: file.fileId,
-        token: "file",
-        role: null,
-        label: file.label,
-        metadata: file.metadata
-      })),
-      error: null,
-      loading: false
-    });
-  }, [containerId, conversationKey, store]);
+  const files = conversationFilesFor(store);
+  const conversationResult = (0, import_react3.useSyncExternalStore)(
+    files.subscribe,
+    () => conversationKey ? files.get(conversationKey) : null,
+    () => null
+  );
+  const loadConversationFiles = (0, import_react3.useCallback)(
+    async (force = true) => {
+      if (!conversationKey || !containerId) return;
+      await files.load(conversationKey, containerId, force);
+    },
+    [containerId, conversationKey, files]
+  );
   (0, import_react3.useEffect)(() => {
-    const timeout = setTimeout(() => {
-      void loadConversationFiles();
-    }, 0);
-    return () => clearTimeout(timeout);
+    void loadConversationFiles(false);
   }, [loadConversationFiles]);
   const lastGenericFetch = (0, import_react3.useRef)({
     key: conversationKey,