@ai-matrx/messaging 0.10.0 → 0.10.2

package/CHANGELOG.md

@@ -1,5 +1,43 @@
 # Changelog — `@ai-matrx/messaging`
 
+## 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
+`sourceApp` / `sourceFeature` to the core, so every run went out stamped with this package's
+placeholder `ai-matrx` / `messaging.<capability>` — and the AI Matrx server keeps a REGISTER of
+legal producers and answers an unregistered one with a 422. Found by putting a real request on the
+wire, not by reading the code; the same shape as the 0.7.0 wrong-schema defect.
+
+### Added
+
+- **`<MessagingProvider sourceApp sourceFeature>`**, forwarded to `createMessagingAi`. Only the
+  host knows its registered slugs, so the host passes them; the package's defaults are documented
+  as placeholders that will not survive a real server.
+
 ## 0.10.0 — 2026-09-07
 
 **The four AI-native intelligences rendered nothing in the first host that shipped them, and

package/README.md

@@ -172,6 +172,8 @@ messages, keyset pagination, and the whole polished UI. There is nothing else to
     draftReply:  { agentId: "…" },
   }}
   maxTranscriptMessages={200}         // your knob, not our taste
+  sourceApp="matrx-frontend"          // YOUR registered producer slugs
+  sourceFeature="messages"
 >
 ```
 
@@ -190,6 +192,10 @@ names the job it is waiting on. If you render the output yourself, render `parti
 `isRunning` — a placeholder that sits until the run finishes is a spinner standing in for an
 answer.
 
+🚨 **`sourceApp` / `sourceFeature` are not analytics garnish.** The AI Matrx server keeps a register
+of legal producers and answers an unregistered one with a **422** — so every AI call fails until you
+pass slugs your server knows. This package's defaults are placeholders; only you know yours.
+
 **"Catch me up" means the unread tail**, derived from the store's own unread count. Nothing
 unread, or a loaded window that does not reach back to the read mark, means there is no honest
 cutoff and the whole window applies — never a guessed timestamp.

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,36 @@ 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,
+      ...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,
+    ...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;
@@ -1517,18 +1535,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
     };