@ai-matrx/messaging 0.3.0 → 0.6.0

package/CHANGELOG.md

@@ -1,5 +1,91 @@
 # Changelog — `@ai-matrx/messaging`
 
+## 0.6.0 — 2026-09-07
+
+**The two entries did not agree on their own branded ids.** A consumer that minted a
+`ConversationId` with `asConversationId` from `@ai-matrx/messaging` could not pass it to a hook
+from `@ai-matrx/messaging/react` — the two were different types, with an error message that
+blames the consumer. Found by the first real adoption, which does exactly that on its
+conversation route.
+
+### Fixed
+
+- **The nominal brand is a named marker, not a `unique symbol`.** The package ships two entries
+  built as two bundles, so every shared type is DECLARED TWICE in the shipped `.d.ts` — and two
+  `declare const brand: unique symbol` declarations are two different symbols, hence two
+  incompatible `ConversationId`s. A structural marker (`__mxBrand`) is identical in both
+  declarations, so the brands unify while still being impossible to produce by accident. No
+  runtime change: the marker is types-only.
+- **`verify-tarball` now typechecks a real consumer against BOTH entries** — core-minted ids
+  assigned to `/react` types and back. It fails against the `unique symbol` build, which is the
+  only way this class can be caught: in `src/` there is only ever one declaration, so every
+  source gate stays green while the shipped types are broken.
+- The canary still imported `getInitials` / `avatarPaletteIndex`, which moved to
+  `@ai-matrx/kit/format` in 0.5.0 — it would have failed that release's publish.
+
+### Consumer action (C28)
+
+None. `asConversationId(...)` and friends behave exactly as before; ids from the two entries now
+actually interoperate, which is what the types always claimed.
+
+## 0.5.0 — 2026-09-07
+
+**Consumer action (C28).** Two exports MOVED OUT of this package to
+`@ai-matrx/kit/format`. Update the import; the behaviour is identical.
+
+| was | now |
+|---|---|
+| `import { getInitials } from "@ai-matrx/messaging"` | `import { getInitials } from "@ai-matrx/kit/format"` |
+| `import { avatarPaletteIndex } from "@ai-matrx/messaging"` | `import { avatarPaletteIndex } from "@ai-matrx/kit/format"` |
+
+**Why.** Duplication census finding H1 (2026-09-07): both bodies were
+duplicated in `@ai-matrx/meet`, and while two packages claimed them no host
+twin had a correct owner to point at. They now live in `@ai-matrx/kit`, which
+has zero sibling dependencies.
+
+The conversation-shaped formatters — `formatConversationTime`,
+`formatMessageTime`, `formatDateSeparator`, `formatLastSeen`, `formatTypists`,
+`groupMessages`, `isSameDay`, `participantNames` — all stay here. Nothing is
+re-exported: this package imports kit directly.
+
+New dependency: `@ai-matrx/kit` (`latest`).
+
+## 0.4.0 — 2026-09-07
+
+**A ```matrx fence written in the PLATFORM's dialect was being deleted on render.** Found the
+moment the first consumer's real messages went through this package's bubble. Third and last
+release of the adoption session.
+
+### Fixed
+
+- **A fence this package cannot parse is never dropped.** `splitText` advanced past an
+  unrecognized fence and emitted nothing for it, so a message that named a note or a task lost
+  that paragraph between the sender and the reader — silently, with the surrounding prose intact
+  so nothing looked wrong. This package understands exactly one fence dialect (its own array of
+  `{entityType, entityId}`); Matrx's real fences are two-key `__kind` directive shells whose
+  items are typed per noun and resolvable only by the app's kind registry, i.e. every real
+  reference on the platform hit this path.
+- **The regression test that should have caught it was hollow** — it asserted
+  `segments.every(s => s.type === "text")` on what was, after the drop, an EMPTY array. Replaced
+  with one that asserts the fence survives, and one for the platform's own dialect.
+- **An unreadable fence collapses to the label `Reference` in previews and notifications**, never
+  to nothing. Collapsing to nothing is how a reference-only message becomes an inbox row reading
+  "No messages yet".
+
+### Added
+
+- **`renderFence`** — the host draws ```matrx fences with its own renderer, every fence, parsed
+  or not. A platform's fence dialect is the platform's business.
+- `TextSegment` gains `{ type: "fence", body }`. With no host renderer the bubble draws an inert
+  card labeled "Reference" whose title says what is missing and how to wire it — drawn, never
+  deleted, never a code block of JSON.
+
+### Consumer action (C28)
+
+- **A host on the Matrx reference-fence protocol MUST pass `renderFence`** or its references
+  render as the generic inert card. matrx-frontend passes `MatrxEnvelopeBlock`.
+- Code that switches on `TextSegment.type` gains a third case. TypeScript will point at it.
+
 ## 0.3.0 — 2026-09-07
 
 **Three more seams the first adoption needed** — each one a case where the host

package/README.md

@@ -262,6 +262,11 @@ its own registry, hands those in rather than rebuilding the thread:
 package's card — deliberately, because an app that draws references everywhere else must not
 draw them two ways.
 
+**If your platform has its own ```matrx fence dialect, pass `renderFence` too.** This package
+parses one shape (an array of `{entityType, entityId}`); anything else reaches your renderer
+verbatim as the fence body. Without it those fences draw an honest inert "Reference" card — they
+are never deleted, and never shown to a reader as JSON.
+
 ### Theming
 
 Structural CSS ships in the package; the token **contract** is enforced; token **values** are

package/dist/index.cjs

@@ -30,7 +30,6 @@ __export(src_exports, {
   asMessageId: () => asMessageId,
   asOrganizationId: () => asOrganizationId,
   asUserId: () => asUserId,
-  avatarPaletteIndex: () => avatarPaletteIndex,
   composeFence: () => composeFence,
   conversationTopic: () => conversationTopic,
   createActionRegistry: () => createActionRegistry,
@@ -48,7 +47,6 @@ __export(src_exports, {
   formatLastSeen: () => formatLastSeen,
   formatMessageTime: () => formatMessageTime,
   formatTypists: () => formatTypists,
-  getInitials: () => getInitials,
   groupMessages: () => groupMessages,
   inboxTopic: () => inboxTopic,
   invalidResponse: () => invalidResponse,
@@ -319,21 +317,32 @@ function splitText(content) {
     if (start > cursor) {
       segments.push({ type: "text", value: content.slice(cursor, start) });
     }
-    parseFenceBody(match[1] ?? "").forEach((reference) => {
-      segments.push({ type: "reference", reference });
-    });
+    const body = match[1] ?? "";
+    const references = parseFenceBody(body);
+    if (references.length === 0) {
+      segments.push({ type: "fence", body });
+    } else {
+      references.forEach((reference) => {
+        segments.push({ type: "reference", reference });
+      });
+    }
     cursor = start + match[0].length;
   }
   if (cursor < content.length) {
     segments.push({ type: "text", value: content.slice(cursor) });
   }
   return segments.filter(
-    (segment) => segment.type === "reference" || segment.value.trim().length > 0
+    (segment) => segment.type !== "text" || segment.value.trim().length > 0
   );
 }
 function summarizeText(content, maxLength = 140) {
   const parts = splitText(content).map(
-    (segment) => segment.type === "text" ? segment.value : segment.reference.label
+    (segment) => segment.type === "text" ? segment.value : segment.type === "reference" ? segment.reference.label : (
+      // A fence we cannot read still SAYS something. Collapsing it to
+      // nothing is how a reference-only message becomes an inbox row that
+      // reads "No messages yet" — a screen telling a lie.
+      "Reference"
+    )
   );
   const flattened = parts.join(" ").replace(/\s+/g, " ").trim();
   if (flattened.length <= maxLength) return flattened;
@@ -1485,20 +1494,6 @@ function formatDateSeparator(isoString, now = Date.now(), locale) {
     day: "numeric"
   });
 }
-function getInitials(name) {
-  const parts = name.trim().split(/\s+/).filter(Boolean);
-  if (parts.length === 0) return "?";
-  const first = parts[0]?.[0] ?? "";
-  const last = parts.length > 1 ? parts.at(-1)?.[0] ?? "" : "";
-  return `${first}${last}`.toUpperCase() || "?";
-}
-function avatarPaletteIndex(seed, buckets = 8) {
-  let hash = 0;
-  for (let index = 0; index < seed.length; index += 1) {
-    hash = hash * 31 + seed.charCodeAt(index) | 0;
-  }
-  return Math.abs(hash) % buckets;
-}
 function groupMessages(messages, args = {}) {
   const windowMs = args.windowMs ?? 5 * MINUTE;
   const now = args.now ?? Date.now();