@ai-matrx/messaging 0.1.0 → 0.1.2

package/dist/react.d.cts

@@ -672,6 +672,15 @@ interface MessagingStore {
     ingestMany(messages: readonly Message[]): void;
     removeMessage(conversationId: ConversationId, messageId: MessageId): void;
     markConversationRead(id: ConversationId): void;
+    /**
+     * Move a conversation's preview + sort position to reflect a message, with NO
+     * refetch. Sending a message must not cost a conversation-list RPC — at a
+     * busy org that is one full page read per keystroke-burst, and the row's new
+     * state is entirely derivable from the message we already hold.
+     */
+    applyLastMessage(message: Message, args?: {
+        incrementUnread?: boolean;
+    }): void;
     setUnreadCount(id: ConversationId, count: number): void;
 }
 declare function createMessagingStore(): MessagingStore;
@@ -934,6 +943,13 @@ declare function MessageBubble(props: {
 declare function MessageActionChips(props: {
     message: Message;
 }): ReactNode;
+/**
+ * The standalone composer: it owns its own state. Inside `ConversationView` the
+ * VIEW owns it instead (`ComposerView` below) — two `useComposer` instances on
+ * one conversation are two independent drafts, two independent reply targets,
+ * and — since typing rides presence — two publishers fighting over ONE presence
+ * entry, which `@ai-matrx/realtime` warns about by name.
+ */
 declare function Composer(props: {
     conversationId: ConversationId | null;
 }): ReactNode;

package/dist/react.d.ts

@@ -672,6 +672,15 @@ interface MessagingStore {
     ingestMany(messages: readonly Message[]): void;
     removeMessage(conversationId: ConversationId, messageId: MessageId): void;
     markConversationRead(id: ConversationId): void;
+    /**
+     * Move a conversation's preview + sort position to reflect a message, with NO
+     * refetch. Sending a message must not cost a conversation-list RPC — at a
+     * busy org that is one full page read per keystroke-burst, and the row's new
+     * state is entirely derivable from the message we already hold.
+     */
+    applyLastMessage(message: Message, args?: {
+        incrementUnread?: boolean;
+    }): void;
     setUnreadCount(id: ConversationId, count: number): void;
 }
 declare function createMessagingStore(): MessagingStore;
@@ -934,6 +943,13 @@ declare function MessageBubble(props: {
 declare function MessageActionChips(props: {
     message: Message;
 }): ReactNode;
+/**
+ * The standalone composer: it owns its own state. Inside `ConversationView` the
+ * VIEW owns it instead (`ComposerView` below) — two `useComposer` instances on
+ * one conversation are two independent drafts, two independent reply targets,
+ * and — since typing rides presence — two publishers fighting over ONE presence
+ * entry, which `@ai-matrx/realtime` warns about by name.
+ */
 declare function Composer(props: {
     conversationId: ConversationId | null;
 }): ReactNode;

package/dist/react.js

@@ -966,6 +966,29 @@ function createMessagingStore() {
       });
       emit();
     },
+    applyLastMessage(message, args = {}) {
+      const existing = conversations.find(
+        (item) => item.conversation.id === message.conversationId
+      );
+      if (existing === void 0) return;
+      if (message.deliveryState === "sending" || message.deletedAt !== null) return;
+      if (existing.lastMessageAt !== null && message.createdAt < existing.lastMessageAt) return;
+      const isActive = activeConversationId === message.conversationId;
+      const shouldCount = args.incrementUnread === true && !isActive;
+      conversations = normalizeConversations(
+        conversations.map(
+          (item) => item.conversation.id === message.conversationId ? {
+            ...item,
+            lastMessageContent: message.content,
+            lastMessageSenderId: message.senderId,
+            lastMessageAt: message.createdAt,
+            sortAt: message.createdAt,
+            unreadCount: shouldCount ? item.unreadCount + 1 : item.unreadCount
+          } : item
+        )
+      );
+      emit();
+    },
     markConversationRead(id) {
       conversations = conversations.map(
         (item) => item.conversation.id === id ? { ...item, unreadCount: 0 } : item
@@ -1036,7 +1059,7 @@ function createMessagingEngine(options) {
     onSent: (entry, message) => {
       store.ingest(message);
       broadcastMessage(message);
-      void refreshConversationRow(message.conversationId);
+      store.applyLastMessage(message);
       void entry;
     },
     onChange: (entries) => {
@@ -1062,15 +1085,6 @@ function createMessagingEngine(options) {
   function broadcastMessage(message) {
     openChannels.get(message.conversationId)?.send(MESSAGING_EVENTS.message, message);
   }
-  async function refreshConversationRow(id) {
-    try {
-      const page = await repository.listConversations({ limit: conversationPageSize });
-      const row = page.items.find((item) => item.conversation.id === id);
-      if (row !== void 0) store.upsertConversation(row);
-    } catch (error) {
-      reportError(error, "refreshConversationRow");
-    }
-  }
   async function reloadInbox() {
     const page = await repository.listConversations({ limit: conversationPageSize });
     conversationCursor = page.nextCursor;
@@ -1121,10 +1135,9 @@ function createMessagingEngine(options) {
                 return;
               }
               store.ingest(message);
-              if (message.senderId !== identity.userId) {
-                options.onIncoming?.(message);
-              }
-              void refreshConversationRow(message.conversationId);
+              const isMine = message.senderId === identity.userId;
+              store.applyLastMessage(message, { incrementUnread: !isMine });
+              if (!isMine) options.onIncoming?.(message);
             }
           },
           {
@@ -2016,7 +2029,12 @@ function useTypists(conversationId, participants) {
   const typing = useTyping(
     {
       topic: conversationId === null ? "" : conversationTopic(conversationId),
-      identity: { userId: host?.identity.userId ?? "" }
+      identity: { userId: host?.identity.userId ?? "" },
+      // READ-ONLY. This label and `useComposer` sit on the SAME topic, which is
+      // one channel carrying ONE presence entry per key — so exactly one of
+      // them publishes, and it is the composer that actually knows whether this
+      // user is typing. Publishing here would overwrite that on every heartbeat.
+      publish: false
     },
     { enabled: conversationId !== null && host !== null }
   );
@@ -2043,7 +2061,10 @@ function useOnlineUserIds(conversationId) {
   const presence = usePresence(
     {
       topic: conversationId === null ? "" : conversationTopic(conversationId),
-      presence: { state: { userId: host?.identity.userId ?? "" } }
+      // READ-ONLY, for the same reason: `useComposer`'s typing presence already
+      // puts this user in the roster under this key with their `userId`, and a
+      // second publisher on one entry just fights it.
+      presence: { publish: false, state: { userId: host?.identity.userId ?? "" } }
     },
     { enabled: conversationId !== null && host !== null }
   );
@@ -2124,7 +2145,7 @@ function useMessagingAi(conversationId) {
 }
 function useMessageAction(message) {
   const host = useRequiredMessagingHost();
-  const action = message.action;
+  const action = message.action ?? null;
   const [receipt, setReceipt] = useState(
     () => action === null ? null : host.actions.receiptFor(action.kind, message.id, host.identity.userId)
   );
@@ -2656,7 +2677,7 @@ function ConversationView(props) {
       /* @__PURE__ */ jsx4(TypingDots, {}),
       typists.label
     ] }) : null }),
-    /* @__PURE__ */ jsx4(Composer, { conversationId })
+    /* @__PURE__ */ jsx4(ComposerView, { composer, disabled: false })
   ] });
 }
 function MessageGroupView(props) {
@@ -2789,6 +2810,10 @@ function MessageActionChips(props) {
 }
 function Composer(props) {
   const composer = useComposer(props.conversationId);
+  return /* @__PURE__ */ jsx4(ComposerView, { composer, disabled: props.conversationId === null });
+}
+function ComposerView(props) {
+  const composer = props.composer;
   const textareaRef = useRef3(null);
   useEffect3(() => {
     const node = textareaRef.current;
@@ -2822,7 +2847,7 @@ function Composer(props) {
           value: composer.value,
           placeholder: "Message",
           "aria-label": "Message",
-          disabled: props.conversationId === null,
+          disabled: props.disabled,
           onChange: (event) => {
             composer.setValue(event.target.value);
             composer.onKeystroke();