@genesislcap/ai-assistant 15.2.1 → 15.3.1

package/dist/react.cjs

@@ -102,6 +102,16 @@ const AiChatBubble = React.forwardRef(function AiChatBubble(props, ref) {
   return React.createElement(customElements.getName(AiChatBubbleWC) ?? 'foundation-ai-chat-bubble', { ...rest, ref }, children);
 });
 
+const AiChatMarkdown = React.forwardRef(function AiChatMarkdown(props, ref) {
+  const { children, ...rest } = props;
+  return React.createElement(customElements.getName(AiChatMarkdownWC) ?? 'ai-chat-markdown', { ...rest, ref }, children);
+});
+
+const FoundationAiPopoutManager = React.forwardRef(function FoundationAiPopoutManager(props, ref) {
+  const { children, ...rest } = props;
+  return React.createElement(customElements.getName(FoundationAiPopoutManagerWC) ?? 'foundation-ai-popout-manager', { ...rest, ref }, children);
+});
+
 const AiChatInteractionWrapper = React.forwardRef(function AiChatInteractionWrapper(props, ref) {
   const { onRequestScrollToBottom, onInteractionCompleted, children, ...rest } = props;
   const _innerRef = React.useRef(null);
@@ -124,16 +134,6 @@ const AiChatInteractionWrapper = React.forwardRef(function AiChatInteractionWrap
   return React.createElement(customElements.getName(AiChatInteractionWrapperWC) ?? 'ai-chat-interaction-wrapper', { ...rest, ref: _mergeRefs(_innerRef, ref) }, children);
 });
 
-const AiChatMarkdown = React.forwardRef(function AiChatMarkdown(props, ref) {
-  const { children, ...rest } = props;
-  return React.createElement(customElements.getName(AiChatMarkdownWC) ?? 'ai-chat-markdown', { ...rest, ref }, children);
-});
-
-const FoundationAiPopoutManager = React.forwardRef(function FoundationAiPopoutManager(props, ref) {
-  const { children, ...rest } = props;
-  return React.createElement(customElements.getName(FoundationAiPopoutManagerWC) ?? 'foundation-ai-popout-manager', { ...rest, ref }, children);
-});
-
 module.exports = {
   AiFlowingWavesIndicator,
   AiHaloOverlay,
@@ -144,7 +144,7 @@ module.exports = {
   AiActivityHalo,
   AgentPicker,
   AiChatBubble,
-  AiChatInteractionWrapper,
   AiChatMarkdown,
   FoundationAiPopoutManager,
+  AiChatInteractionWrapper,
 };

package/dist/react.mjs

@@ -100,6 +100,16 @@ export const AiChatBubble = React.forwardRef(function AiChatBubble(props, ref) {
   return React.createElement(customElements.getName(AiChatBubbleWC) ?? 'foundation-ai-chat-bubble', { ...rest, ref }, children);
 });
 
+export const AiChatMarkdown = React.forwardRef(function AiChatMarkdown(props, ref) {
+  const { children, ...rest } = props;
+  return React.createElement(customElements.getName(AiChatMarkdownWC) ?? 'ai-chat-markdown', { ...rest, ref }, children);
+});
+
+export const FoundationAiPopoutManager = React.forwardRef(function FoundationAiPopoutManager(props, ref) {
+  const { children, ...rest } = props;
+  return React.createElement(customElements.getName(FoundationAiPopoutManagerWC) ?? 'foundation-ai-popout-manager', { ...rest, ref }, children);
+});
+
 export const AiChatInteractionWrapper = React.forwardRef(function AiChatInteractionWrapper(props, ref) {
   const { onRequestScrollToBottom, onInteractionCompleted, children, ...rest } = props;
   const _innerRef = React.useRef(null);
@@ -121,13 +131,3 @@ export const AiChatInteractionWrapper = React.forwardRef(function AiChatInteract
   }, []);
   return React.createElement(customElements.getName(AiChatInteractionWrapperWC) ?? 'ai-chat-interaction-wrapper', { ...rest, ref: _mergeRefs(_innerRef, ref) }, children);
 });
-
-export const AiChatMarkdown = React.forwardRef(function AiChatMarkdown(props, ref) {
-  const { children, ...rest } = props;
-  return React.createElement(customElements.getName(AiChatMarkdownWC) ?? 'ai-chat-markdown', { ...rest, ref }, children);
-});
-
-export const FoundationAiPopoutManager = React.forwardRef(function FoundationAiPopoutManager(props, ref) {
-  const { children, ...rest } = props;
-  return React.createElement(customElements.getName(FoundationAiPopoutManagerWC) ?? 'foundation-ai-popout-manager', { ...rest, ref }, children);
-});

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@genesislcap/ai-assistant",
   "description": "Genesis AI Assistant micro-frontend",
-  "version": "15.2.1",
+  "version": "15.3.1",
   "license": "SEE LICENSE IN license.txt",
   "main": "dist/esm/index.js",
   "types": "dist/ai-assistant.d.ts",
@@ -73,26 +73,26 @@
     }
   },
   "devDependencies": {
-    "@genesislcap/foundation-testing": "15.2.1",
-    "@genesislcap/genx": "15.2.1",
-    "@genesislcap/rollup-builder": "15.2.1",
-    "@genesislcap/ts-builder": "15.2.1",
-    "@genesislcap/uvu-playwright-builder": "15.2.1",
-    "@genesislcap/vite-builder": "15.2.1",
-    "@genesislcap/webpack-builder": "15.2.1",
+    "@genesislcap/foundation-testing": "15.3.1",
+    "@genesislcap/genx": "15.3.1",
+    "@genesislcap/rollup-builder": "15.3.1",
+    "@genesislcap/ts-builder": "15.3.1",
+    "@genesislcap/uvu-playwright-builder": "15.3.1",
+    "@genesislcap/vite-builder": "15.3.1",
+    "@genesislcap/webpack-builder": "15.3.1",
     "@types/dompurify": "^3.0.5",
     "@types/marked": "^5.0.2",
     "esbuild": "0.25.12"
   },
   "dependencies": {
-    "@genesislcap/foundation-ai": "15.2.1",
-    "@genesislcap/foundation-logger": "15.2.1",
-    "@genesislcap/foundation-notifications": "15.2.1",
-    "@genesislcap/foundation-redux": "15.2.1",
-    "@genesislcap/foundation-ui": "15.2.1",
-    "@genesislcap/foundation-utils": "15.2.1",
-    "@genesislcap/rapid-design-system": "15.2.1",
-    "@genesislcap/web-core": "15.2.1",
+    "@genesislcap/foundation-ai": "15.3.1",
+    "@genesislcap/foundation-logger": "15.3.1",
+    "@genesislcap/foundation-notifications": "15.3.1",
+    "@genesislcap/foundation-redux": "15.3.1",
+    "@genesislcap/foundation-ui": "15.3.1",
+    "@genesislcap/foundation-utils": "15.3.1",
+    "@genesislcap/rapid-design-system": "15.3.1",
+    "@genesislcap/web-core": "15.3.1",
     "dompurify": "^3.3.1",
     "marked": "^17.0.3"
   },
@@ -105,5 +105,5 @@
     "access": "public"
   },
   "customElements": "dist/custom-elements.json",
-  "gitHead": "3917f3dc0bb5f4487bd30b37360fa0cddda92f0c"
+  "gitHead": "1c3a33f7d56acec166f832b9610e83343a8873e6"
 }

package/src/state/persistence/diagnostics.test.ts

@@ -1,5 +1,5 @@
 import { assert, createLogicSuite } from '@genesislcap/foundation-testing';
-import { assembleDebugLog, type DiagnosticEntry } from './diagnostics';
+import { assembleDebugLog, diagnosticsMessageKey, type DiagnosticEntry } from './diagnostics';
 
 // GENC-1351 §5.8: `assembleDebugLog` reassembles the forward-only diagnostics
 // stream back into the `{ readme, timeline, meta }` debug-log shape — timeline
@@ -93,3 +93,47 @@ Suite('collapses exact-duplicate entries but keeps genuinely-distinct ones', ()
 });
 
 Suite.run();
+
+// GENC-1461: the reasoning/narration/answer split (GENC-1411) emits several messages from ONE model
+// response, so they share `timestamp` + `role` + `subAgentOf`. The message identity key must still
+// tell them apart — otherwise the forward-capture dedup treats them as one and drops all but the
+// first (a server generation's answer message vanished from the persisted download while its
+// reasoning survived).
+const KeySuite = createLogicSuite('diagnosticsMessageKey');
+
+KeySuite('gives co-timestamped split messages (reasoning vs answer) distinct keys', () => {
+  const base = {
+    kind: 'message' as const,
+    role: 'assistant',
+    timestamp: '2026-01-01T00:00:05.000Z',
+  };
+  const reasoning: DiagnosticEntry = { ...base, category: 'reasoning', content: 'thinking…' };
+  const answer: DiagnosticEntry = { ...base, content: 'the generated code' }; // no category = the answer
+  const kReasoning = diagnosticsMessageKey(reasoning);
+  const kAnswer = diagnosticsMessageKey(answer);
+  assert.ok(kReasoning, 'reasoning message has a key');
+  assert.ok(kAnswer, 'answer message has a key');
+  assert.not.equal(
+    kReasoning,
+    kAnswer,
+    'a reasoning and an answer split from one response must not collide',
+  );
+});
+
+KeySuite('still collapses a genuine re-append of the same message (same key)', () => {
+  const m: DiagnosticEntry = {
+    kind: 'message',
+    role: 'assistant',
+    timestamp: '2026-01-01T00:00:05.000Z',
+    category: 'reasoning',
+    content: 'thinking…',
+  };
+  // Two instances / a reload re-appending the SAME message must still share one key.
+  assert.equal(diagnosticsMessageKey(m), diagnosticsMessageKey({ ...m }));
+});
+
+KeySuite('returns null when the entry has no timestamp (treated as always-new)', () => {
+  assert.is(diagnosticsMessageKey({ kind: 'message', role: 'assistant' }), null);
+});
+
+KeySuite.run();

package/src/state/persistence/diagnostics.ts

@@ -42,18 +42,26 @@ export interface DebugLog {
 }
 
 /**
- * Stable identity for a message diagnostic entry — `timestamp::role::subAgentOf`.
+ * Stable identity for a message diagnostic entry — `timestamp::role::subAgentOf::category`.
  * Used by the forward-capture delta (skip already-emitted messages) and by the
  * restore watermark (skip messages already present in a prior load's stream).
  * `null` when the entry has no timestamp → treat as always-new. Shared so the
  * element's diagnostics capture and the `SessionPersister`'s restore watermark
  * compute identity identically (GENC-1351 §5.8/§5.10).
  *
+ * `category` is part of the key because the reasoning/narration/answer split (GENC-1411) emits
+ * SEVERAL messages from one model response — the thought summary, any interstitial narration, and
+ * the answer/tool-call message — which share `timestamp` + `role` + `subAgentOf`. Keying on those
+ * alone collapsed them to one, so the forward-capture dropped all but the first: a server
+ * generation's answer message vanished from the persisted debug log while its reasoning survived
+ * (GENC-1461). `category` is stable across a message's in-place annotation (only `toolCalls` change,
+ * which is deliberately excluded), so a genuine re-append of the same message still dedups.
+ *
  * @internal
  */
 export function diagnosticsMessageKey(entry: DiagnosticEntry): string | null {
   return typeof entry.timestamp === 'string'
-    ? `${entry.timestamp}::${(entry.role as string) ?? ''}::${(entry.subAgentOf as string) ?? ''}`
+    ? `${entry.timestamp}::${(entry.role as string) ?? ''}::${(entry.subAgentOf as string) ?? ''}::${(entry.category as string) ?? ''}`
     : null;
 }
 

package/src/state/persistence/session-persister.test.ts

@@ -367,6 +367,16 @@ const metaSnap = (dedupSignature: string, meta: unknown): DiagnosticEntry => ({
   dedupSignature,
 });
 
+// A diagnostic MESSAGE entry as the reasoning/narration/answer split (GENC-1411) produces them —
+// several messages from ONE response, sharing timestamp + role, differing by `category` / content.
+const diagMsg = (over: Partial<DiagnosticEntry> = {}): DiagnosticEntry => ({
+  kind: 'message',
+  role: 'assistant',
+  timestamp: '2026-01-01T00:00:05.000Z',
+  content: '',
+  ...over,
+});
+
 Suite('scheduleDiagnostics() appends the collected delta once per window', async () => {
   const { p, f } = makePersister({ diagnosticEntries: [evt(0), evt(1)] });
   p.scheduleDiagnostics();
@@ -425,4 +435,38 @@ Suite(
   },
 );
 
+Suite(
+  'flushDiagnostics() keeps co-timestamped split messages (reasoning + answer), not deduped as one',
+  async () => {
+    // GENC-1461: the reasoning/narration/answer split emits several messages from ONE response, so
+    // they share timestamp + role + subAgentOf. The forward-capture keyed only on those, so it
+    // emitted the reasoning message and dropped the answer as a "duplicate" — a server generation's
+    // answer message vanished from the persisted download while its reasoning survived. Both must be
+    // captured.
+    const reasoning = diagMsg({ category: 'reasoning', content: 'thinking…' });
+    const answer = diagMsg({ content: 'the generated code' }); // no category = the answer/tool-call slot
+    const { p, f } = makePersister({ diagnosticEntries: [reasoning, answer] });
+    await p.flushDiagnostics();
+    assert.is(f.appended.length, 1, 'one flush');
+    assert.is(
+      f.appended[0].length,
+      2,
+      'both the reasoning and the answer message are captured, not collapsed to one',
+    );
+  },
+);
+
+Suite(
+  'flushDiagnostics() still dedups a genuine re-append of the same message across flushes',
+  async () => {
+    // The dedup must still stop the SAME message being re-appended on every throttled flush.
+    const m = diagMsg({ category: 'reasoning', content: 'thinking…' });
+    const { p, f } = makePersister({ diagnosticEntries: [m] });
+    await p.flushDiagnostics();
+    assert.is(f.appended.length, 1, 'first flush appends the message');
+    await p.flushDiagnostics();
+    assert.is(f.appended.length, 1, 'unchanged message → no re-append');
+  },
+);
+
 Suite.run();

package/src/state/persistence/session-persister.ts

@@ -638,6 +638,18 @@ export class SessionPersister {
    * (§5.8). Skips messages present at restore or already appended; emits each turn
    * (by turn-index) and meta-event (by index) once; appends a `meta-snapshot` only
    * when the block changed. Advances the shared per-session cursors as it goes.
+   *
+   * TODO(GENC-1461, longer-term): this whole delta step exists only because the element *pulls* the
+   * ENTIRE debug log on every throttled flush (`getDiagnosticEntries()` returns the full timeline),
+   * so we must diff it against per-session cursors to append only the new tail — and messages, unlike
+   * events (monotonic `index`) and turns (`turnIndex`), have no stable positional cursor (compaction
+   * rewrites ranges, tool-call messages are annotated in place, synthetic messages are spliced in,
+   * restore reloads them), forcing a content-derived identity (`diagnosticsMessageKey`). That
+   * identity is inherently fragile: it was too coarse for the reasoning/answer/tool-call split and
+   * silently dropped co-timestamped messages (GENC-1461; fixed for now by adding `category`). The
+   * robust fix is to invert the flow — have the driver *push* each diagnostic entry exactly once at
+   * creation (an append-only event stream) instead of the element pulling-and-diffing — which removes
+   * the message-identity key and this dedup entirely. Bigger change; deferred.
    */
   private collectDiagnosticsDelta(): DiagnosticEntry[] {
     const cursors = getDiagnosticsCursors(this.stateKey);