@smartspace/chat-ui 1.13.1-dev.aa44752 → 1.13.1-dev.b2f9256

package/dist/index.d.ts

@@ -238,6 +238,14 @@ type MessageThread = {
     totalMessages: number;
     pinned: boolean;
     workSpaceId: string;
+    /**
+     * Monotonic version of when this summary was emitted (epoch ms). Used by
+     * `applyThreadToCache` to reject stale writes — e.g. a SignalR summary
+     * landing after a fresher SSE thread frame because the server's DB write
+     * lagged Redis. Mappers derive this from `lastUpdatedAt`; client-side
+     * writers like `ensureDraftThread` use `Date.now()`.
+     */
+    summaryEmittedAt: number;
 };
 type ThreadsResponse = {
     data: MessageThread[];
@@ -969,20 +977,6 @@ declare function mapMentionUserDtoToModel(dto: MentionUserDto): MentionUser;
 declare function mapWorkspaceDtoToModel(dto: WorkspaceDto): Workspace;
 declare const mapWorkspacesDtoToModels: (arr: WorkspacesListItemDto[]) => Workspace[];
 
-/**
- * Write a freshly-observed thread (from SignalR or an SSE thread frame)
- * directly into the relevant query caches so subscribers paint without a
- * refetch roundtrip.
- *
- * - Merges into `threadsKeys.detail(workspaceId, thread.id)`.
- * - Splices into every threads-list cache for the workspace, handling both
- *   finite `ThreadsResponse` and infinite `{ pages, pageParams }` shapes.
- *
- * Returns `true` when the thread was found in at least one list cache.
- * Callers that need to surface brand-new threads (e.g. another user just
- * created one) can fall back to invalidating the list queries when this
- * returns `false`.
- */
 declare function applyThreadToCache(qc: QueryClient, thread: MessageThread): boolean;
 /**
  * Invalidate every threads-list cache for a workspace. Use as a fallback when

package/dist/index.js

@@ -3233,7 +3233,20 @@ var threadsKeys = {
 };
 
 // src/domains/threads/cache.ts
+function isStaleSummary(incoming, existing) {
+  if (!existing) return false;
+  if (typeof existing.summaryEmittedAt !== "number") return false;
+  if (typeof incoming.summaryEmittedAt !== "number") return false;
+  if (incoming.summaryEmittedAt >= existing.summaryEmittedAt) return false;
+  return existing.isFlowRunning === false && incoming.isFlowRunning === true;
+}
 function applyThreadToCache(qc, thread) {
+  const existingDetail = qc.getQueryData(
+    threadsKeys.detail(thread.workSpaceId, thread.id)
+  );
+  if (isStaleSummary(thread, existingDetail)) {
+    return false;
+  }
   qc.setQueryData(
     threadsKeys.detail(thread.workSpaceId, thread.id),
     (old) => ({ ...old ?? thread, ...thread })
@@ -3254,6 +3267,7 @@ function applyThreadToCache(qc, thread) {
           if (!page?.data) return page;
           const idx2 = page.data.findIndex((t) => t.id === thread.id);
           if (idx2 === -1) return page;
+          if (isStaleSummary(thread, page.data[idx2])) return page;
           changed = true;
           foundInList = true;
           const nextData2 = page.data.slice();
@@ -3266,6 +3280,7 @@ function applyThreadToCache(qc, thread) {
       if (!list2.data) return old;
       const idx = list2.data.findIndex((t) => t.id === thread.id);
       if (idx === -1) return old;
+      if (isStaleSummary(thread, list2.data[idx])) return old;
       foundInList = true;
       const nextData = list2.data.slice();
       nextData[idx] = { ...nextData[idx], ...thread };
@@ -3335,36 +3350,40 @@ var {
   messageThreadsGetMessageThreadWorkspacesWorkspaceIdMessagethreadsIdResponse: threadResponseSchema
 } = ChatZod;
 function mapThreadDtoToModel(dto) {
+  const lastUpdatedAt = utcDate(dto.lastUpdatedAt);
   return {
     id: dto.id,
     createdAt: utcDate(dto.createdAt),
     createdBy: dto.createdBy ?? "",
     createdByUserId: dto.createdByUserId,
     isFlowRunning: dto.isFlowRunning,
-    lastUpdatedAt: utcDate(dto.lastUpdatedAt),
+    lastUpdatedAt,
     lastUpdatedByUserId: dto.lastUpdatedByUserId,
     name: dto.name ?? "",
     totalMessages: dto.totalMessages,
     pinned: dto.favorited,
-    workSpaceId: dto.workSpaceId
+    workSpaceId: dto.workSpaceId,
+    summaryEmittedAt: lastUpdatedAt.getTime()
   };
 }
 function mapThreadsResponseDtoToModel(dto) {
   return { data: dto.data.map(mapThreadDtoToModel), total: dto.total };
 }
 function mapSignalRThreadSummaryToModel(summary) {
+  const lastUpdatedAt = utcDate(summary.lastUpdatedAt);
   return {
     id: summary.id,
     createdAt: utcDate(summary.createdAt),
     createdBy: summary.createdBy ?? "",
     createdByUserId: summary.createdByUserId,
     isFlowRunning: summary.isFlowRunning,
-    lastUpdatedAt: utcDate(summary.lastUpdatedAt),
+    lastUpdatedAt,
     lastUpdatedByUserId: summary.lastUpdatedByUserId,
     name: summary.name ?? "",
     totalMessages: summary.totalMessages,
     pinned: summary.favorited,
-    workSpaceId: summary.workSpaceId
+    workSpaceId: summary.workSpaceId,
+    summaryEmittedAt: lastUpdatedAt.getTime()
   };
 }
 var threadDetailOptions = ({
@@ -3457,6 +3476,15 @@ var messagesMutationsKeys = {
 };
 
 // src/domains/messages/mutations.ts
+function reconcileWithMessage(old, incoming, onDuplicate = "keep-existing") {
+  const stable = old.filter((m) => !m.optimistic);
+  const idx = stable.findIndex((m) => m.id === incoming.id);
+  if (idx === -1) return [...stable, incoming];
+  if (onDuplicate === "keep-existing") return stable;
+  const copy = stable.slice();
+  copy[idx] = incoming;
+  return copy;
+}
 function useSendMessage() {
   const qc = useQueryClient();
   const { userId, displayName: userName } = useChatIdentity();
@@ -3540,7 +3568,10 @@ function useSendMessage() {
         toast.error("There was an error posting your message");
         throw err;
       }
-      qc.setQueryData(messagesKeys.list(threadId), [realMessage]);
+      qc.setQueryData(
+        messagesKeys.list(threadId),
+        (old = []) => reconcileWithMessage(old, realMessage)
+      );
       qc.setQueryData(
         threadsKeys.detail(workspaceId, threadId),
         (old) => old ? { ...old, isFlowRunning: true } : old
@@ -3596,14 +3627,10 @@ function useAddInputToMessage() {
       });
     },
     onSuccess: (message, { threadId }) => {
-      qc.setQueryData(messagesKeys.list(threadId), (old = []) => {
-        const stable = old.filter((x) => !x.optimistic);
-        const idx = stable.findIndex((x) => x.id === message.id);
-        if (idx === -1) return [...stable, message];
-        const copy = stable.slice();
-        copy[idx] = message;
-        return copy;
-      });
+      qc.setQueryData(
+        messagesKeys.list(threadId),
+        (old = []) => reconcileWithMessage(old, message, "replace")
+      );
     },
     onError: (_e, { threadId }) => {
       qc.setQueryData(
@@ -19804,6 +19831,10 @@ function MessageList({
   const messagesEndRef = useRef(null);
   const prevMessageCountRef = useRef(0);
   const hasInitialScrollRef = useRef(false);
+  const everHadMessagesRef = useRef({
+    threadId: "",
+    had: false
+  });
   const isMobile = useIsMobile();
   const { data: activeWorkspace } = useWorkspace(workspaceId);
   const [isAtBottom, setIsAtBottom] = useState(true);
@@ -19875,8 +19906,15 @@ function MessageList({
     ro.observe(content);
     return () => ro.disconnect();
   }, [isAtBottom, scrollToBottom]);
+  const safeMessages = messages ?? [];
+  if (everHadMessagesRef.current.threadId !== threadId) {
+    everHadMessagesRef.current = { threadId, had: safeMessages.length > 0 };
+  } else if (safeMessages.length > 0) {
+    everHadMessagesRef.current.had = true;
+  }
+  const hadMessagesBefore = everHadMessagesRef.current.had;
   const isLoading = isChoosingThread || (threadPending || threadFetching) && !thread || (messagesPending || messagesFetching) && messages === void 0;
-  if (isLoading) {
+  if (isLoading && !hadMessagesBefore) {
     return /* @__PURE__ */ jsx(
       "div",
       {
@@ -19892,7 +19930,7 @@ function MessageList({
       }
     );
   }
-  if (threadError || messagesError) {
+  if ((threadError || messagesError) && !hadMessagesBefore) {
     return /* @__PURE__ */ jsx("div", { className: "flex flex-1 items-center justify-center p-6", children: /* @__PURE__ */ jsxs("div", { className: "w-full max-w-md space-y-3", children: [
       threadError && /* @__PURE__ */ jsxs("div", { className: "flex items-center gap-2 rounded-md border border-destructive/30 bg-destructive/10 px-3 py-2 text-destructive", children: [
         /* @__PURE__ */ jsx(AlertTriangle, { className: "h-4 w-4" }),
@@ -19904,8 +19942,7 @@ function MessageList({
       ] })
     ] }) });
   }
-  const safeMessages = messages ?? [];
-  if (safeMessages.length === 0) {
+  if (safeMessages.length === 0 && !hadMessagesBefore) {
     return /* @__PURE__ */ jsxs("div", { className: "flex overflow-auto flex-shrink-10 flex-col p-8 text-center", children: [
       /* @__PURE__ */ jsx("h3", { className: "text-lg font-medium mb-2", children: activeWorkspace?.name ?? "No messages yet" }),
       activeWorkspace?.firstPrompt && /* @__PURE__ */ jsx("div", { className: "max-w-3xl mx-auto p-4", children: /* @__PURE__ */ jsx(MessageMarkdown, { value: activeWorkspace.firstPrompt }) })