@ai-matrx/messaging 0.10.1 → 0.10.3

package/CHANGELOG.md

@@ -1,5 +1,45 @@
 # Changelog — `@ai-matrx/messaging`
 
+## 0.10.3 — 2026-09-08
+
+### Fixed
+
+- Flat platform agent-event envelopes no longer display the human audit principal as a visible
+  “via” author. The principal remains immutable in `sender_id` and `principalUserId`; it is not
+  presentation identity for an automated event.
+- The engine declares the conversation room's read-only presence shape on the first holder,
+  before subscribe. Typing and online-roster hooks can now attach without forcing a late-binding
+  room rebuild and its `channel.room.binding.refused` diagnostics.
+
+### Consumer action (C28)
+
+None. Upgrade `@ai-matrx/realtime` alongside this package so presence/typing holders also receive
+the ephemeral-roster backfill fix.
+
+## 0.10.2 — 2026-09-08
+
+**Agent-authored review events wore the human audit principal's name, avatar, alignment, and
+delivery tick.** The event rows already carried the canonical flat
+`actor_kind: "agent"` / `actor_label` envelope, but effective-actor resolution recognized only
+the older nested `metadata.actor` shape. The UI therefore presented an automated review as if
+the signed-in reviewer had written it.
+
+### Fixed
+
+- Flat canonical actor envelopes now resolve to an agent presentation without inheriting the
+  human principal's name or avatar. Adjacent agent and human events sharing one audit principal
+  are grouped separately, and agent events never receive the human's right-aligned/delivery-tick
+  treatment.
+- `<ConversationView showHeader showAi>` lets an embedding workspace remove duplicate thread
+  chrome and capabilities it does not offer.
+- `<ConversationView renderComposerInput>` lets a host supply its canonical textarea component
+  while this package retains draft, reply, typing, Enter-key, and send ownership.
+
+### Consumer action (C28)
+
+None for existing screens. Embedded workspaces may set `showHeader={false}`, `showAi={false}`,
+or inject their canonical composer through `renderComposerInput`.
+
 ## 0.10.1 — 2026-09-07
 
 **Every AI call would have been refused by the real server.** `<MessagingProvider>` never forwarded

package/dist/index.cjs

@@ -42,6 +42,7 @@ __export(src_exports, {
   createOutbox: () => createOutbox,
   createReadCache: () => createReadCache,
   createWebOutboxStorage: () => createWebOutboxStorage,
+  effectiveActorKey: () => effectiveActorKey,
   extractReferences: () => extractReferences,
   formatConversationTime: () => formatConversationTime,
   formatDateSeparator: () => formatDateSeparator,
@@ -141,19 +142,38 @@ function createActionRegistry() {
 
 // src/core/actor.ts
 function readActorHint(metadata) {
-  const raw = metadata["actor"];
-  if (typeof raw !== "object" || raw === null || Array.isArray(raw)) return null;
-  const record = raw;
-  const agentId = record["agentId"] ?? record["agent_id"];
-  if (typeof agentId !== "string" || agentId.length === 0) return null;
-  const name = record["agentName"] ?? record["agent_name"];
-  const avatar = record["agentAvatarUrl"] ?? record["agent_avatar_url"];
+  const envelope = metadata;
+  const raw = envelope["actor"];
+  if (typeof raw === "object" && raw !== null && !Array.isArray(raw)) {
+    const record = raw;
+    const agentId = record["agentId"] ?? record["agent_id"];
+    if (typeof agentId !== "string" || agentId.length === 0) return null;
+    const name2 = record["agentName"] ?? record["agent_name"];
+    const avatar2 = record["agentAvatarUrl"] ?? record["agent_avatar_url"];
+    return {
+      agentId,
+      showPrincipalAttribution: true,
+      ...typeof name2 === "string" && name2.length > 0 ? { agentName: name2 } : {},
+      ...typeof avatar2 === "string" && avatar2.length > 0 ? { agentAvatarUrl: avatar2 } : {}
+    };
+  }
+  const actorKind = envelope["actor_kind"] ?? envelope["actorKind"];
+  if (actorKind !== "agent") return null;
+  const id = envelope["actor_id"] ?? envelope["actorId"];
+  const name = envelope["actor_label"] ?? envelope["actorLabel"];
+  const avatar = envelope["actor_avatar_url"] ?? envelope["actorAvatarUrl"];
   return {
-    agentId,
+    showPrincipalAttribution: false,
+    ...typeof id === "string" && id.length > 0 ? { agentId: id } : {},
     ...typeof name === "string" && name.length > 0 ? { agentName: name } : {},
     ...typeof avatar === "string" && avatar.length > 0 ? { agentAvatarUrl: avatar } : {}
   };
 }
+function effectiveActorKey(message) {
+  const hint = readActorHint(message.metadata);
+  if (hint === null) return `principal:${message.senderId}`;
+  return `agent:${hint.agentId ?? hint.agentName ?? "anonymous"}`;
+}
 function resolveActor(message, sender) {
   const hint = readActorHint(message.metadata);
   const humanName = sender?.displayName ?? message.senderId;
@@ -165,7 +185,11 @@ function resolveActor(message, sender) {
       displayName: hint.agentName ?? "Agent",
       avatarUrl: hint.agentAvatarUrl ?? null,
       isAgent: true,
-      onBehalfOfName: humanName,
+      // The nested envelope explicitly models an agent acting on behalf of a
+      // session principal. Platform event rows use the flat actor envelope as
+      // the complete presentation identity; surfacing their audit writer as
+      // "via Human" wrongly re-attaches that person to an automated event.
+      onBehalfOfName: hint.showPrincipalAttribution ? humanName : null,
       principalUserId: message.senderId
     };
   }
@@ -1352,6 +1376,11 @@ function createMessagingEngine(options) {
       if (openChannels.has(id) || disposed) return;
       const handle = manager.open({
         topic: conversationTopic(id),
+        // Presence callbacks are JOIN-TIME in supabase-js. Declare the
+        // read-only room shape on the first holder (the engine) so the
+        // composer/typist/online hooks can attach without forcing a rebuild
+        // after this durable channel has already subscribed.
+        presence: { publish: false, state: {} },
         postgresChanges: [
           {
             event: "*",
@@ -1517,18 +1546,20 @@ function groupMessages(messages, args = {}) {
   let current = null;
   let previousDay = null;
   for (const message of messages) {
+    const actorKey = args.actorKey?.(message) ?? `principal:${message.senderId}`;
     const day = message.createdAt.slice(0, 10);
     const startsNewDay = day !== previousDay;
     previousDay = day;
     const last = current?.messages.at(-1);
     const withinWindow = last !== void 0 && Math.abs(Date.parse(message.createdAt) - Date.parse(last.createdAt)) <= windowMs;
-    if (current !== null && current.senderId === message.senderId && withinWindow && !startsNewDay) {
+    if (current !== null && current.senderId === message.senderId && current.actorKey === actorKey && withinWindow && !startsNewDay) {
       current.messages.push(message);
       continue;
     }
     if (current !== null) groups.push(current);
     current = {
       senderId: message.senderId,
+      actorKey,
       messages: [message],
       dateSeparator: startsNewDay ? formatDateSeparator(message.createdAt, now, args.locale) : null
     };