@flamingo-stack/openframe-frontend-core 0.0.587-1973.5528.1 → 0.0.587-2000.5532.1

package/src/components/chat/embeddable-chat.tsx

@@ -808,14 +808,13 @@ function SourceChips({
   const [expanded, setExpanded] = useState(false);
 
   if (cited.length === 0) {
-    const visible = expanded ? uncited : uncited.slice(0, FALLBACK_TOP_RETRIEVED);
-    const hiddenCount = Math.max(0, uncited.length - FALLBACK_TOP_RETRIEVED);
-    if (visible.length === 0) return null;
+    const fallback = uncited.slice(0, FALLBACK_TOP_RETRIEVED);
+    if (fallback.length === 0) return null;
     return (
       <div className="mt-2 flex flex-col gap-1.5 border-t border-ods-border pt-2">
         <span className="uppercase text-ods-text-muted text-h6">Top retrieved sources</span>
-        <div className={`flex flex-wrap gap-1.5 ${expanded ? 'max-h-[200px] overflow-y-auto overscroll-contain' : ''}`}>
-          {visible.map(src => (
+        <div className="flex flex-wrap gap-1.5">
+          {fallback.map(src => (
             <SourceChip
               key={src.index}
               src={src}
@@ -825,16 +824,6 @@ function SourceChips({
               onDiscuss={onDiscuss}
             />
           ))}
-          {hiddenCount > 0 && (
-            <button
-              onClick={() => setExpanded(!expanded)}
-              className="inline-flex cursor-pointer items-center gap-1 rounded border border-ods-accent bg-ods-card px-2 py-0.5 text-ods-accent transition-colors text-h6 hover:bg-ods-accent/10"
-              aria-expanded={expanded}
-              aria-label={expanded ? 'Show fewer sources' : `Show ${hiddenCount} additional retrieved sources`}
-            >
-              {expanded ? 'Show less' : `+${hiddenCount} more retrieved ${hiddenCount === 1 ? 'source' : 'sources'}`}
-            </button>
-          )}
         </div>
       </div>
     );
@@ -1590,10 +1579,6 @@ function EmbeddableChatInner({
         // explicitly — see `Message.hidden` and `chat-message-list`'s skip.
         ...(m.hidden ? { hidden: true } : {}),
         ...(m.scrollAnchor ? { scrollAnchor: m.scrollAnchor } : {}),
-        // Preserve same-turn rich metadata. Source chips and entity-card
-        // renderers consume it after this raw-to-Message projection.
-        ...(m.sources && m.sources.length > 0 ? { sources: m.sources } : {}),
-        ...(m.refs && m.refs.length > 0 ? { refs: m.refs } : {}),
         // Forward attached context items so the user bubble renders its chips.
         ...(m.contextItems && m.contextItems.length > 0 ? { contextItems: m.contextItems } : {}),
       };
@@ -1844,41 +1829,24 @@ function EmbeddableChatInner({
     return out;
   }, [commandsById, enabledSet]);
 
-  const renderSourcesAfterMessage = useCallback(
-    (message: Message, index: number) => {
-      if (message.role !== 'assistant' || !message.sources?.length) return null;
-      if (chatLoading && index === messages.length - 1) return null;
-      const content =
-        typeof message.content === 'string'
-          ? message.content
-          : message.content
-              .filter(segment => segment.type === 'text')
-              .map(segment => segment.text)
-              .join('\n');
-      const citationOrder = [...content.matchAll(/\[(\d+)\]/g)].map(m => parseInt(m[1], 10));
-      const seenOrder = new Map<number, number>();
-      citationOrder.forEach(idx => {
-        if (!seenOrder.has(idx)) seenOrder.set(idx, seenOrder.size);
-      });
-      const cited = message.sources
-        .filter(s => seenOrder.has(s.index))
-        .sort((a, b) => (seenOrder.get(a.index) ?? 0) - (seenOrder.get(b.index) ?? 0));
-      const uncited = message.sources.filter(s => !seenOrder.has(s.index));
-      return (
-        <div className="flex-shrink-0 pb-2">
-          <SourceChips
-            cited={cited}
-            uncited={uncited}
-            baseRoute={resolvedBaseRoute}
-            chipBasePlatform={chipBasePlatform}
-            onClose={handleNavigationClose}
-            onDiscuss={discussRef}
-          />
-        </div>
-      );
-    },
-    [chatLoading, messages.length, resolvedBaseRoute, chipBasePlatform, handleNavigationClose, discussRef],
-  );
+  // Find sources for the last assistant message; split into cited / uncited.
+  const lastAssistantMsg = [...rawMessages].reverse().find(m => m.role === 'assistant');
+  const lastSources = useMemo(() => {
+    if (chatLoading) return undefined;
+    const sources = lastAssistantMsg?.sources;
+    if (!sources || sources.length === 0) return undefined;
+    const content = lastAssistantMsg?.content || '';
+    const citationOrder = [...content.matchAll(/\[(\d+)\]/g)].map(m => parseInt(m[1], 10));
+    const seenOrder = new Map<number, number>();
+    citationOrder.forEach(idx => {
+      if (!seenOrder.has(idx)) seenOrder.set(idx, seenOrder.size);
+    });
+    const cited = sources
+      .filter(s => seenOrder.has(s.index))
+      .sort((a, b) => (seenOrder.get(a.index) ?? 0) - (seenOrder.get(b.index) ?? 0));
+    const uncited = sources.filter(s => !seenOrder.has(s.index));
+    return { cited, uncited };
+  }, [lastAssistantMsg, chatLoading]);
 
   // Host node for in-panel Radix portals (see the body wrapper below).
   const [portalHost, setPortalHost] = useState<HTMLDivElement | null>(null);
@@ -2422,7 +2390,6 @@ function EmbeddableChatInner({
                                   resolveContextIcon={resolveContextIcon}
                                   renderContextItem={renderContextItem}
                                   renderMention={renderMention}
-                                  renderAfterMessage={renderSourcesAfterMessage}
                                   // Gated on `chatLoading` for the same reason the composer
                                   // is: no second send while a turn is in flight. Passive
                                   // demo hosts (previewMode) stay read-only.
@@ -2460,6 +2427,20 @@ function EmbeddableChatInner({
                                   onLoadMore={loadMoreMessages}
                                 />
                               )}
+                              {lastSources &&
+                                (lastSources.cited.length > 0 || lastSources.uncited.length > 0) &&
+                                !chatLoading && (
+                                  <div className="flex-shrink-0 pb-2">
+                                    <SourceChips
+                                      cited={lastSources.cited}
+                                      uncited={lastSources.uncited}
+                                      baseRoute={resolvedBaseRoute}
+                                      chipBasePlatform={chipBasePlatform}
+                                      onClose={handleNavigationClose}
+                                      onDiscuss={discussRef}
+                                    />
+                                  </div>
+                                )}
                             </div>
                           ) : activeMode === 'mingo' ? (
                             /* Figma node 7532:222444 — default (Mingo-mode) empty state:

package/src/components/chat/entity-cards/dispatch.tsx

@@ -30,7 +30,6 @@
  */
 
 import React, { type ReactNode } from 'react';
-import { isSourceMetadataCardRef } from '../../../chat-protocol/source-metadata';
 import { useRequiredChatRuntime } from '../../../contexts/chat-runtime-context';
 import Image from '../../../embed-shims/next-image';
 import { useRouter } from '../../../embed-shims/next-navigation';
@@ -42,7 +41,6 @@ import {
   formatTimeWithTimezone,
   formatDurationFromRange,
 } from '../../../utils/format';
-import { extractYouTubeId } from '../../features/video';
 import { MingoIcon } from '../../icons';
 import { ArrowRightUpIcon } from '../../icons-v2-generated/arrows/arrow-right-up-icon';
 import { ClickupLogoIcon } from '../../icons-v2-generated/brand-logos/clickup-logo-icon';
@@ -424,15 +422,6 @@ function hubspotStatusToVariant(status: string | undefined): MingoInfoCardStatus
  * which renders the marker as plain text (same anti-hallucination
  * posture as every other card type).
  */
-function validatedHttpsUrl(value: unknown): string | null {
-  if (typeof value !== 'string') return null;
-  try {
-    return new URL(value).protocol === 'https:' ? value : null;
-  } catch {
-    return null;
-  }
-}
-
 function decodeVideoMarkerId(id: string): { videoUrl?: string; youtubeUrl?: string } | null {
   // Prefix checks FIRST: an id like `mux-1234567` (11 chars) would also
   // satisfy the bare-YouTube-id pattern below.
@@ -450,8 +439,8 @@ function decodeVideoMarkerId(id: string): { videoUrl?: string; youtubeUrl?: stri
       const url = atob(b64);
       // https-only — the id is model-emitted text; never let a marker
       // smuggle a non-https src into a player.
-      const safeUrl = validatedHttpsUrl(url);
-      return safeUrl ? { videoUrl: safeUrl } : null;
+      if (!/^https:\/\//.test(url)) return null;
+      return { videoUrl: url };
     } catch {
       return null;
     }
@@ -460,29 +449,6 @@ function decodeVideoMarkerId(id: string): { videoUrl?: string; youtubeUrl?: stri
   return null;
 }
 
-function validatedVideoMetadata(chatRef: ChatRef): Record<string, unknown> | null {
-  const input = chatRef.metadata ?? {};
-  const metadata: Record<string, unknown> = {};
-  for (const key of ['videoUrl', 'highlightVideoUrl', 'videoPoster', 'highlightVideoPoster'] as const) {
-    const url = validatedHttpsUrl(input[key]);
-    if (url) metadata[key] = url;
-  }
-  const youtubeUrl = input.youtubeUrl;
-  if (typeof youtubeUrl === 'string' && /^[A-Za-z0-9_-]{11}$/.test(youtubeUrl)) {
-    metadata.youtubeUrl = youtubeUrl;
-  }
-
-  const safeUrl = validatedHttpsUrl(chatRef.url);
-  if (safeUrl) {
-    const youtubeId = extractYouTubeId(safeUrl);
-    delete metadata.videoUrl;
-    delete metadata.youtubeUrl;
-    metadata[youtubeId ? 'youtubeUrl' : 'videoUrl'] = youtubeId ?? safeUrl;
-  }
-
-  return Object.keys(metadata).length > 0 ? metadata : null;
-}
-
 /** Read hero-video metadata off a FETCHED item — the API-driven
  *  replacement for the retired refs-frame `metadata.videoUrl` check. Two item
  *  shapes: ChatRef-shaped items (per-object hydration routes) carry the
@@ -1262,10 +1228,6 @@ interface ChatCardRegistryEntry {
    *  point — a host that re-homes a type must be able to say so even when
    *  the row already carries the content host's own url. */
   noComposedHref?: boolean;
-  /** Render a document card from same-turn SOURCES metadata when its optional
-   *  hydration endpoint has no row. Only doc refs opt in: their source record
-   *  already carries the complete title/path/navigation fallback. */
-  renderFromSourceMetadata?: boolean;
 }
 
 /**
@@ -1288,7 +1250,6 @@ function refHydratedEntry(
   docType: string,
   label: string,
   renderRef: (displayRef: ChatRef, opts: ChatCardRenderOptions) => React.ReactNode,
-  renderFromSourceMetadata = false,
 ): ChatCardRegistryEntry {
   return {
     label,
@@ -1298,19 +1259,6 @@ function refHydratedEntry(
     fallbackHref: (item: { url?: string | null }) => item?.url ?? null,
     skeleton: () => <MingoInfoCardSkeleton />,
     render: (item, chatRef, opts) => renderRef(fetchedItemDisplayRef(item, chatRef), opts),
-    renderFromSourceMetadata,
-  };
-}
-
-function sourceMetadataCardItem(chatRef: ChatRef): ChatCardItem {
-  return {
-    type: chatRef.type,
-    id: chatRef.id,
-    title: chatRef.title,
-    url: chatRef.url,
-    targetPlatform: chatRef.targetPlatform,
-    sourceRepo: chatRef.sourceRepo,
-    metadata: chatRef.metadata,
   };
 }
 
@@ -1506,18 +1454,12 @@ const CHAT_CARD_REGISTRY: Record<string, ChatCardRegistryEntry> = {
   hubspot_ticket_self: refHydratedEntry('hubspot_ticket_self', 'HubSpot ticket (self)', (displayRef, opts) => (
     <HubspotTicketChatCard chatRef={displayRef} isNewTab={opts.isNewTab} discuss={opts.discuss} />
   )),
-  data_room_doc: refHydratedEntry(
-    'data_room_doc',
-    'Data-room doc',
-    (displayRef, opts) => <DataRoomDocChatCard chatRef={displayRef} isNewTab={opts.isNewTab} discuss={opts.discuss} />,
-    true,
-  ),
-  markdown: refHydratedEntry(
-    'markdown',
-    'Doc page (markdown)',
-    (displayRef, opts) => <DataRoomDocChatCard chatRef={displayRef} isNewTab={opts.isNewTab} discuss={opts.discuss} />,
-    true,
-  ),
+  data_room_doc: refHydratedEntry('data_room_doc', 'Data-room doc', (displayRef, opts) => (
+    <DataRoomDocChatCard chatRef={displayRef} isNewTab={opts.isNewTab} discuss={opts.discuss} />
+  )),
+  markdown: refHydratedEntry('markdown', 'Doc page (markdown)', (displayRef, opts) => (
+    <DataRoomDocChatCard chatRef={displayRef} isNewTab={opts.isNewTab} discuss={opts.discuss} />
+  )),
   // Body-synthesized video ids (`shortVideoId` content hashes / YouTube
   // ids) have no backing table row to fetch — ref-only by nature.
   video: {
@@ -1960,13 +1902,6 @@ export function ChatCardLoader({
   }
   if (isLoading) return <>{fetchEntry.skeleton?.() ?? null}</>;
   if (!item) {
-    const hasSourceMetadata =
-      fetchEntry.renderFromSourceMetadata &&
-      isSourceMetadataCardRef(chatRef) &&
-      typeof finalChatRef.metadata?.path === 'string';
-    if (hasSourceMetadata) {
-      return finish(fetchEntry.render(sourceMetadataCardItem(finalChatRef), finalChatRef, renderOpts));
-    }
     // FAIL LOUD (explicit product decision 2026-08-13): a card that
     // cannot hydrate renders a VISIBLE error card — silent removal made
     // broken answers look fine. Three distinguishable states:
@@ -2048,28 +1983,24 @@ export function renderChatInlineEntityCard(
     />
   );
 
-  // SOURCES metadata may enrich an opaque video marker with its playable URL.
-  // Self-describing marker ids remain transport-independent through
-  // `decodeVideoMarkerId`; undecodable ids without matching metadata fall
-  // through to the loader and degrade to plain text.
+  // Embedded videos are SELF-DESCRIBING: the marker id alone recovers the
+  // playable URL (`decodeVideoMarkerId`), so the player renders on ANY
+  // transport with zero refs/fetch. Undecodable ids (legacy sha1 hashes,
+  // hallucinations) fall through to the loader → plain-text degrade.
   // Hero videos on FETCHED entity types (webinars, releases, …) are
   // handled post-fetch inside ChatCardLoader (`itemVideoMetadata`) — the
-  // marker metadata is not required for that post-fetch path.
+  // marker's ref plays no part in either path.
   if (chatRef.type === 'video') {
-    const enrichedMetadata = validatedVideoMetadata(chatRef);
     const decoded = decodeVideoMarkerId(chatRef.id);
-    const playableMetadata = enrichedMetadata ?? decoded;
-    if (playableMetadata) {
+    if (decoded) {
       const displayRef: ChatRef = {
         ...chatRef,
         title: chatRef.title && chatRef.title !== chatRef.id ? chatRef.title : 'Video',
         url:
           chatRef.url ??
-          (typeof playableMetadata.videoUrl === 'string' ? playableMetadata.videoUrl : null) ??
-          (typeof playableMetadata.youtubeUrl === 'string'
-            ? `https://www.youtube.com/watch?v=${playableMetadata.youtubeUrl}`
-            : null),
-        metadata: playableMetadata,
+          decoded.videoUrl ??
+          (decoded.youtubeUrl ? `https://www.youtube.com/watch?v=${decoded.youtubeUrl}` : null),
+        metadata: { ...(chatRef.metadata ?? {}), ...decoded },
       };
       return (
         <BlockCard inline={<ChatInlineVideoPill chatRef={displayRef} />}>

package/src/components/chat/hooks/use-nats-chat-adapter.ts

@@ -61,7 +61,6 @@ import type {
   MessageSegment,
   NatsMessageType,
   ChatApprovalStatus,
-  ChatSource,
 } from '../types';
 import type { DialogItem } from '../types/component.types';
 import type {
@@ -353,8 +352,6 @@ export function mapProcessedToUnified(
     role: 'user' | 'assistant' | 'error';
     content: string | MessageSegment[];
     name?: string;
-    sources?: ChatSource[];
-    refs?: ChatRef[];
   }>,
 ): UnifiedChatMessage[] {
   const out: UnifiedChatMessage[] = [];
@@ -366,16 +363,12 @@ export function mapProcessedToUnified(
         role: m.role,
         content: '',
         segments: m.content,
-        ...(m.sources ? { sources: m.sources } : {}),
-        ...(m.refs ? { refs: m.refs } : {}),
       });
     } else {
       out.push({
         id: m.id,
         role: m.role,
         content: m.content,
-        ...(m.sources ? { sources: m.sources } : {}),
-        ...(m.refs ? { refs: m.refs } : {}),
       });
     }
   }

package/src/components/chat/hooks/use-slash-commands.ts

@@ -24,12 +24,10 @@ import type { SlashCommandActionId, SlashCommandSummaryAction, SlashCommandSumma
 // canonical declaration.
 export type { SlashCommandActionId, SlashCommandSummaryAction, SlashCommandSummary };
 
-const COMMANDS_MAX_RESULTS = 10;
-
 /**
- * Fetch the full slash-command registry for the CURRENT deployment platform,
- * then match id-prefix `prefix` locally. The chat source is server-resolved —
- * the client sends NO `source` param on the URL.
+ * Fetch slash-command suggestions for the CURRENT deployment platform,
+ * matching id-prefix `prefix`. The chat source is server-resolved — the
+ * client sends NO `source` param on the URL.
  *
  * `commandsUrl` is required — supplied by `useChatRuntime().endpoints.commandsUrl`
  * at the call site. NO default value.
@@ -58,15 +56,13 @@ export async function fetchSlashCommands(
 ): Promise<SlashCommandSummary[]> {
   try {
     const url = new URL(commandsUrl, window.location.origin);
+    if (prefix) url.searchParams.set('q', prefix);
     // `headers: {}` opts out of the default `Content-Type: application/json`
     // — this is a bare GET with no body, so no content-type is needed.
     const res = await embedAuthedFetch(url.toString(), { signal, headers: {} });
     if (!res.ok) return [];
     const data = (await res.json()) as { commands?: SlashCommandSummary[] };
-    const commands = data.commands ?? [];
-    const normalizedPrefix = prefix.trim().toLowerCase();
-    if (!normalizedPrefix) return commands;
-    return commands.filter(({ id }) => id.startsWith(normalizedPrefix)).slice(0, COMMANDS_MAX_RESULTS);
+    return data.commands ?? [];
   } catch (err) {
     // Cancellation (unmount / dep change) MUST propagate so react-query treats
     // it as cancelled, not as a successful empty result. Every OTHER failure

package/src/components/chat/hooks/use-sse-chat-adapter.ts

@@ -63,7 +63,7 @@ import type { ChatDialogStore } from '../stream/chat-dialog-store';
 import type { ChatStreamReducerOptions } from '../stream/chat-stream-reducer';
 import { useChatStreamReducer } from '../stream/use-chat-stream-reducer';
 import type { DialogItem } from '../types/component.types';
-import type { ChatSource, Message, MessageSegment } from '../types/message.types';
+import type { Message, MessageSegment } from '../types/message.types';
 import type {
   UnifiedChatState,
   UnifiedChatMessage,
@@ -81,7 +81,6 @@ import { useSlashCommandRegistry, type SlashCommandSummary } from './use-slash-c
 // Canonical home of the per-turn meta row moved to the stream module in
 // Phase 3; re-exported here to keep the legacy import path stable.
 export type { ChatTurnMeta } from '../stream/chat-stream-reducer';
-export type { ChatSource } from '../types/message.types';
 
 // ─── Stable no-op references for the Guide-mode dialog-management stubs ──
 // Plain module-scope constants so the adapter's return identity stays
@@ -129,6 +128,34 @@ const noopBumpMetaTick = (): void => {};
  *  hub-side platform-utils, not in lib). */
 export type DocSource = string;
 
+export interface ChatSource {
+  index: number;
+  name: string;
+  path: string;
+  documentType: string;
+  externalUrl?: string;
+  /** Platform that owns the destination at `externalUrl`. */
+  targetPlatform?: string | null;
+  /** Primary-key value for single-row chips. */
+  id?: string;
+  /** Per-row items for grouped chips. */
+  items?: Array<{
+    id: string;
+    documentType: string;
+    name: string;
+    externalUrl?: string;
+    targetPlatform?: string | null;
+    /** In-app doc-tree path for markdown / data-room-doc rows so the
+     *  grouped chip's anchor can trigger an in-page doc-tree swap via
+     *  `handleChatNavClick` (parity with single-row chips + cards). */
+    path?: string | null;
+  }>;
+  /** RagTableConfig.id for this source. */
+  sourceRepo?: string;
+  /** Optional display label override returned by the chat API. */
+  label?: string;
+}
+
 export interface DocChatMessage {
   id: string;
   role: 'user' | 'assistant';

package/src/components/chat/remark-strip-citations.ts

@@ -22,16 +22,13 @@ import type { Plugin } from 'unified';
 import { visit } from 'unist-util-visit';
 
 const CITATION_REGEX = /(^|\s)\[\d{1,2}(?:\s*,\s*\d{1,2})*\](?=[\s.,;:!?)\]]|$)/g;
-const CITATION_LIST_REGEX =
-  /(^|\s)(?:\[\d{1,2}(?:\s*,\s*\d{1,2})*\]\s*,\s*)+(?:(?:and|or)\s+)?\[\d{1,2}(?:\s*,\s*\d{1,2})*\](?=[\s.,;:!?)\]]|$)/g;
 
 export const remarkStripCitations: Plugin<[], Root> = () => {
   return (tree: Root) => {
     visit(tree, 'text', (node: Text, index, parent) => {
       if (!parent || typeof index !== 'number') return;
       if (!node.value || !node.value.includes('[')) return;
-      const withoutCitationLists = node.value.replace(CITATION_LIST_REGEX, '$1');
-      const withoutCitations = withoutCitationLists.replace(CITATION_REGEX, '$1');
+      const withoutCitations = node.value.replace(CITATION_REGEX, '$1');
       // No citation matched → leave the node untouched, INCLUDING its
       // whitespace (collapsing unconditionally would rewrite deliberate
       // spacing in prose like `array[1]  value`).
@@ -44,7 +41,8 @@ export const remarkStripCitations: Plugin<[], Root> = () => {
       // leaf and moves on to the next sibling, never re-entering this slot.
       parent.children.splice(index, 1, {
         ...node,
-        value: withoutCitations.replace(/\s+([.,;:!?])/g, '$1').replace(/ {2,}/g, ' '),
+        // Collapse doubled spaces the removal can leave mid-sentence.
+        value: withoutCitations.replace(/ {2,}/g, ' '),
       });
     });
   };

package/src/components/chat/stream/__tests__/chat-stream-reducer.test.ts

@@ -1289,59 +1289,3 @@ describe('createChatStreamReducer — ask cards', () => {
     expect(last.segments?.map(s => s.type)).toEqual(['ask', 'ask']);
   });
 });
-
-describe('createChatStreamReducer — Mingo source metadata', () => {
-  const sources = [
-    {
-      index: 1,
-      name: 'Enroll devices',
-      path: 'docs/enroll.md',
-      documentType: 'markdown',
-    },
-  ];
-
-  it('attaches metadata received before text to the same assistant turn', () => {
-    const reducer = createChatStreamReducer({ transport: 'nats' });
-    const refs = [
-      {
-        type: 'video',
-        id: 'mux-9b6586b494',
-        title: 'Community demo',
-        url: 'https://stream.mux.com/playback-id.m3u8',
-        metadata: { videoUrl: 'https://stream.mux.com/playback-id.m3u8' },
-      },
-    ];
-
-    reducer.apply({ type: 'turn-start', seq: 1 });
-    reducer.apply({ type: 'sources', sources, refs, seq: 2 });
-    reducer.apply({ type: 'text-delta', text: 'Follow the guide [1].', seq: 3 });
-
-    expect(reducer.state.messages).toHaveLength(1);
-    expect(reducer.state.messages[0]).toMatchObject({ sources, refs });
-  });
-
-  it('deduplicates cumulative metadata and does not leak it into the next turn', () => {
-    const reducer = createChatStreamReducer({ transport: 'nats' });
-
-    reducer.apply({ type: 'turn-start', seq: 1 });
-    reducer.apply({ type: 'sources', sources, seq: 2 });
-    reducer.apply({
-      type: 'sources',
-      sources: [
-        { ...sources[0], name: 'Duplicate' },
-        { index: 2, name: 'Install agent', path: 'docs/install.md', documentType: 'markdown' },
-      ],
-      seq: 3,
-    });
-    reducer.apply({ type: 'text-delta', text: 'First answer [1] [2].', seq: 4 });
-    reducer.apply({ type: 'turn-end', seq: 5 });
-    reducer.apply({ type: 'turn-start', seq: 6 });
-    reducer.apply({ type: 'text-delta', text: 'Second answer.', seq: 7 });
-
-    expect(reducer.state.messages[0].sources).toEqual([
-      sources[0],
-      { index: 2, name: 'Install agent', path: 'docs/install.md', documentType: 'markdown' },
-    ]);
-    expect(reducer.state.messages[1].sources).toBeUndefined();
-  });
-});

package/src/components/chat/stream/chat-stream-reducer.ts

@@ -60,7 +60,6 @@ import {
   type ParticipantEvent,
   type UsageEvent,
 } from '../../../chat-protocol/events';
-import type { ChatRef } from '../chat-ref.types';
 import type {
   ApprovalBatchSegment,
   ApprovalRequestSegment,
@@ -71,7 +70,6 @@ import type {
   ToolExecutionSegment,
   ExecutingToolState,
 } from '../types';
-import type { ChatSource } from '../types/message.types';
 import type { PendingApproval } from '../types/processing.types';
 import type {
   DialogTokenUsage,
@@ -80,7 +78,6 @@ import type {
   UnifiedUsageBreakdown,
 } from '../types/unified-chat-state.types';
 import { approvalDisplaysInline } from '../utils/approval-display';
-import { mergeChatRefs, mergeChatSources } from '../utils/chat-source-metadata';
 import {
   type MessageSegmentAccumulator,
   createMessageSegmentAccumulator,
@@ -679,9 +676,6 @@ export function createChatStreamReducer(options: ChatStreamReducerOptions = {}):
   let sendCount = 0;
   let sseCurrentText = '';
 
-  let pendingSources: ChatSource[] | undefined;
-  let pendingRefs: ChatRef[] | undefined;
-
   // ── Snapshot cache ─────────────────────────────────────────────────────
   let stateCache: ChatReducerState | null = null;
   function invalidate(): void {
@@ -779,18 +773,6 @@ export function createChatStreamReducer(options: ChatStreamReducerOptions = {}):
     setMessagesInternal(stampTrailingAssistantSeq(updateTrailingAssistant(messages, segments), seq));
   }
 
-  function attachPendingSourceMetadata(): void {
-    if (!pendingSources && !pendingRefs) return;
-    const last = messages[messages.length - 1];
-    if (!last || last.role !== 'assistant') return;
-    const next = {
-      ...last,
-      ...(pendingSources ? { sources: pendingSources } : {}),
-      ...(pendingRefs ? { refs: pendingRefs } : {}),
-    };
-    setMessagesInternal([...messages.slice(0, -1), next]);
-  }
-
   function applyStreamStartToState(): void {
     setPhaseInternal('streaming');
     // A new stream must never overwrite a COMPLETED trailing assistant
@@ -984,8 +966,6 @@ export function createChatStreamReducer(options: ChatStreamReducerOptions = {}):
 
     switch (event.type) {
       case 'turn-start': {
-        pendingSources = undefined;
-        pendingRefs = undefined;
         isInStream = true;
         hasEverStreamed = true;
         turnStartedAt = Date.now();
@@ -1001,15 +981,6 @@ export function createChatStreamReducer(options: ChatStreamReducerOptions = {}):
         emit('onStreamEnd');
         setPhaseInternal('idle');
         accumulator.resetSegments();
-        pendingSources = undefined;
-        pendingRefs = undefined;
-        break;
-      }
-
-      case 'sources': {
-        pendingSources = mergeChatSources(pendingSources, event.sources);
-        if (event.refs) pendingRefs = mergeChatRefs(pendingRefs, event.refs);
-        attachPendingSourceMetadata();
         break;
       }
 
@@ -1046,12 +1017,10 @@ export function createChatStreamReducer(options: ChatStreamReducerOptions = {}):
         if (isInStream || !hasEverStreamed) {
           emitSegments(segments);
           applySegmentsToState(segments, withSeqMeta(undefined));
-          attachPendingSourceMetadata();
         } else {
           const delta: MessageSegment[] = [{ type: kind, text: event.text }];
           emitSegments(delta, { append: true });
           applySegmentsToState(delta, withSeqMeta({ append: true }));
-          attachPendingSourceMetadata();
         }
         break;
       }

package/src/components/chat/tool-execution-display.tsx

@@ -54,9 +54,6 @@ const ToolExecutionDisplay = forwardRef<HTMLDivElement, ToolExecutionDisplayProp
 
     const hasResult = isExecuted && typeof message.result === 'string' && message.result.length > 0;
     const hasBody = argEntries.length > 0 || hasResult || isExecuting;
-    const isConciseRemoteRead =
-      !message.integratedToolType && !!message.toolTitle?.trim() && argEntries.length === 0 && !hasResult;
-    const isStaticRow = isClientView || isConciseRemoteRead;
 
     const renderStatusIcon = () => {
       if (isExecuting) return <DotsLoaderIcon size={16} className="text-ods-text-secondary" />;
@@ -85,7 +82,7 @@ const ToolExecutionDisplay = forwardRef<HTMLDivElement, ToolExecutionDisplayProp
           {previewText}
         </div>
         <div className="flex h-5 w-5 shrink-0 items-center justify-center">{renderStatusIcon()}</div>
-        {!isStaticRow && (
+        {!isClientView && (
           <div className="flex h-5 w-5 shrink-0 items-center justify-center">
             <ExpandChevron expanded={expanded} />
           </div>
@@ -104,7 +101,7 @@ const ToolExecutionDisplay = forwardRef<HTMLDivElement, ToolExecutionDisplayProp
         )}
         {...props}
       >
-        {isStaticRow ? (
+        {isClientView ? (
           // Client (Fae end-user): static, non-expandable row — no chevron, no
           // body. Just the explanation + status; command/args/result are hidden.
           <div className="flex w-full items-start gap-[var(--spacing-system-xs)] p-[var(--spacing-system-s)] text-left">