@ai-matrx/messaging 0.1.0 → 0.1.2

package/CHANGELOG.md

@@ -1,5 +1,62 @@
 # Changelog — `@ai-matrx/messaging`
 
+## 0.1.2 — 2026-08-31
+
+**Typing, presence and message broadcast work.** The root defect was in `@ai-matrx/realtime`
+(every channel got a unique instance topic, and for broadcast/presence the topic IS the room);
+this package found it, and it is fixed in realtime 0.2.0. These are the consumer-side changes
+that fix came with — both of them real defects this package was carrying.
+
+### Fixed
+
+- **One presence publisher per conversation.** One channel carries ONE presence entry per key, so
+  `useTypists` and `useOnlineUserIds` now read the roster with `presence: { publish: false }`.
+  The composer — which actually knows whether this user is typing — is the single publisher. Two
+  publishers overwrote each other on every heartbeat.
+- **One composer per conversation, so REPLY works.** `ConversationView` called `useComposer` and
+  also rendered `<Composer>`, which called it again: two independent drafts, two independent
+  reply targets — picking "reply" on a message set it on an instance the composer's reply bar
+  never read — and two typing publishers. The view now owns the state and renders `ComposerView`;
+  the exported `Composer` is unchanged for standalone use.
+- **A broadcast message one field short no longer takes down the thread.** `useMessageAction`
+  reads `message.action ?? null` instead of assuming the field is present; an off-shape wire
+  payload used to throw `undefined.kind` and unmount the conversation.
+
+### Added
+
+- `src/react/realtime-rooms.test.tsx` — renders the real `ConversationView` against a shared,
+  topic-deduping fake Supabase client and proves every holder of the conversation topic lands on
+  ONE live channel named as declared (0.1.1 produced five), a remote typist reaches the
+  indicator, a remote broadcast reaches the thread, and exactly one holder publishes presence.
+  All four fail against `@ai-matrx/realtime` 0.1.1.
+
+### Consumer action (C28)
+
+None. No public API changed. `@ai-matrx/realtime` >= 0.2.0 is required and is picked up
+automatically by the `"latest"` spec.
+
+## 0.1.1 — 2026-08-31
+
+Caught in review immediately after 0.1.0 published.
+
+### Fixed
+
+- **Sending a message no longer costs a conversation-list read.** `onSent` and the inbox
+  channel's message handler both called `listConversations` to refresh one row — a full RPC page
+  read per sent message, i.e. one per burst at a busy org, and a regression against the origin
+  (`matrx-frontend` updated the row optimistically via `updateConversationLastMessage`). The row's
+  new preview and sort position are entirely derivable from the message already in hand, so the
+  store gained `applyLastMessage`, which also refuses to move a row BACKWARDS on an out-of-order
+  arrival, ignores an optimistic (still-sending) message, and never badges the conversation being
+  read. Five regression tests.
+- **The inbox no longer flashes "No conversations yet" before it has looked.** An empty list meant
+  two different things — "you have none" and "we have not read yet" — and the surface guessed.
+  `MessagingSnapshot.hasLoadedConversations` makes it a fact.
+
+### Consumer action
+
+None. Both fixes are internal to the package; no API changed and no host wiring is affected.
+
 ## 0.1.0 — 2026-08-31
 
 First release. Enterprise, AI-native in-app messaging for Matrx clients, built to the ratified
@@ -40,7 +97,7 @@ work order at `common-docs/systems/communications/messaging/HANDOFF.md`, on top
   insets, 44px touch targets, a 16px composer minimum so iOS Safari does not zoom on focus) and
   `@ai-matrx/messaging/tokens.css` (a complete default palette, light and dark, host-overridable).
   Hardcoded colors in the structural sheet fail the release gate.
-- 113 tests, each named for the failure it prevents, plus a packed-tarball canary that exercises
+- 118 tests, each named for the failure it prevents, plus a packed-tarball canary that exercises
   ESM + CJS, both entries, the `"use client"` banner per chunk, the `globalThis` slots across
   module graphs, the stylesheets' presence and export-map resolution, and the stability of the
   table/RPC names.

package/dist/index.cjs

@@ -1111,6 +1111,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
@@ -1181,7 +1204,7 @@ function createMessagingEngine(options) {
     onSent: (entry, message) => {
       store.ingest(message);
       broadcastMessage(message);
-      void refreshConversationRow(message.conversationId);
+      store.applyLastMessage(message);
       void entry;
     },
     onChange: (entries) => {
@@ -1207,15 +1230,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;
@@ -1266,10 +1280,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);
             }
           },
           {