@flamingo-stack/openframe-frontend-core 0.0.584-1973.5461.1 → 0.0.584

package/src/components/chat/utils/__tests__/process-historical-messages-golden.test.ts

@@ -391,60 +391,3 @@ describe('processHistoricalMessages — ASK rows', () => {
     expect(processHistoricalMessages(dialog).messages).toEqual([]);
   });
 });
-
-describe('processHistoricalMessages — Mingo source metadata', () => {
-  it.each(['GUIDE', 'SOURCES'] as const)('restores %s metadata on the same assistant turn', type => {
-    const dialog: HistoricalMessage[] = [
-      {
-        id: 'a1',
-        chatType: 'ADMIN_AI_CHAT',
-        createdAt: '2026-07-19T09:00:05Z',
-        owner: { type: 'ASSISTANT' },
-        messageData: [
-          {
-            type,
-            payload: {
-              sources: [
-                {
-                  index: 1,
-                  name: 'Install agent',
-                  path: 'docs/install.md',
-                  documentType: 'markdown',
-                },
-              ],
-              videos: [
-                {
-                  ref: '[card://video:mux-9b6586b494]',
-                  title: 'Community demo',
-                  url: 'https://stream.mux.com/playback-id.m3u8',
-                  metadata: { videoUrl: 'https://stream.mux.com/playback-id.m3u8' },
-                },
-              ],
-            },
-          },
-          { type: 'TEXT', text: 'Install the agent [1]. [card://video:mux-9b6586b494]' },
-        ],
-      },
-    ];
-
-    expect(processHistoricalMessages(dialog).messages[0]).toMatchObject({
-      role: 'assistant',
-      sources: [
-        {
-          index: 1,
-          name: 'Install agent',
-          path: 'docs/install.md',
-          documentType: 'markdown',
-        },
-      ],
-      refs: [
-        {
-          type: 'video',
-          id: 'mux-9b6586b494',
-          metadata: { videoUrl: 'https://stream.mux.com/playback-id.m3u8' },
-        },
-      ],
-      content: [{ type: 'text', text: 'Install the agent [1]. [card://video:mux-9b6586b494]' }],
-    });
-  });
-});

package/src/components/chat/utils/process-historical-messages.ts

@@ -33,8 +33,7 @@ import {
 } from '../../../chat-protocol/events';
 // One normalizer for ask rows, shared with the live decoder — history and the
 // stream must agree on which options are usable.
-import { decodeNatsChunk, normalizeAskOptions } from '../../../chat-protocol/nats-decoder';
-import type { ChatRef } from '../chat-ref.types';
+import { normalizeAskOptions } from '../../../chat-protocol/nats-decoder';
 import { applyApprovalStatusToSegment } from '../stream/message-mutations';
 import {
   MESSAGE_TYPE,
@@ -47,10 +46,8 @@ import {
   type MessageProcessingOptions,
   type MessageData,
   type MessageOwner,
-  type ChatSource,
 } from '../types';
 import { approvalDisplaysInline } from './approval-display';
-import { mergeChatRefs, mergeChatSources } from './chat-source-metadata';
 import { type MessageSegmentAccumulator, createMessageSegmentAccumulator } from './message-segment-accumulator';
 import { getCommandText } from './tool-call-helpers';
 
@@ -124,10 +121,6 @@ export function decodeHistoricalMessageData(data: MessageData): ChatStreamEvent
       }
       return null;
 
-    case MESSAGE_TYPE.GUIDE:
-    case MESSAGE_TYPE.SOURCES:
-      return decodeNatsChunk({ type: data.type, payload: data.payload });
-
     // Same completeness gate as the live decoder (`decodeNatsChunk`): a
     // persisted row without a question or without options is not a card the
     // user can answer, so it replays as nothing rather than as empty chrome.
@@ -598,8 +591,6 @@ export function processHistoricalMessages(
   // MAX persisted seq across the rows grouped into the current assistant turn
   // — carried onto the flushed message's streamSeq for per-role merge coverage.
   let currentAssistantStreamSeq: number | undefined;
-  let currentAssistantSources: ChatSource[] | undefined;
-  let currentAssistantRefs: ChatRef[] | undefined;
 
   /**
    * Flush the current assistant message to processedMessages.
@@ -617,8 +608,6 @@ export function processHistoricalMessages(
         authorType: assistantType,
         timestamp: currentAssistantTimestamp || new Date(),
         avatar: assistantAvatar,
-        ...(currentAssistantSources ? { sources: currentAssistantSources } : {}),
-        ...(currentAssistantRefs ? { refs: currentAssistantRefs } : {}),
         ...(currentAssistantStreamSeq !== undefined ? { streamSeq: currentAssistantStreamSeq } : {}),
       });
       accumulator.resetSegments();
@@ -634,8 +623,6 @@ export function processHistoricalMessages(
     currentAssistantTimestamp = null;
     lastAssistantId = null;
     currentAssistantStreamSeq = undefined;
-    currentAssistantSources = undefined;
-    currentAssistantRefs = undefined;
   };
 
   messages.forEach((msg, index) => {
@@ -702,11 +689,6 @@ export function processHistoricalMessages(
       messageDataArray.forEach(data => {
         const event = decodeHistoricalMessageData(data);
         if (!event) return;
-        if (event.type === 'sources') {
-          currentAssistantSources = mergeChatSources(currentAssistantSources, event.sources);
-          if (event.refs) currentAssistantRefs = mergeChatRefs(currentAssistantRefs, event.refs);
-          return;
-        }
         applyHistoryEvent(
           event,
           accumulator,

package/src/components/navigation/navigation-sidebar-item.tsx

@@ -52,21 +52,31 @@ export const NavigationSidebarItemButton = memo(function NavigationSidebarItemBu
   const hasUnread = unreadCount > 0;
 
   const className = cn(
-    'relative flex w-full items-center justify-start',
+    // `isolate` scopes the hover layer's negative z-index to this row, so it
+    // paints over the row's own background and under its content instead of
+    // sliding behind the sidebar.
+    'relative isolate flex w-full items-center justify-start',
     'h-14 p-[var(--spacing-system-m)]',
-    // 150ms, not 300: the outgoing and incoming entries cross-fade, so for the
-    // whole duration TWO rows read as half-active. At 300ms that window is long
-    // enough to look like a flicker rather than a transition.
-    'transition-colors duration-150',
-    '[&_svg]:transition-colors [&_svg]:duration-150',
+    // The hover/pending wash is its own LAYER, and that separation is the whole
+    // point. Hover and active both want the row's background-color, so a
+    // `transition-colors` that smooths hover necessarily smooths the accent too
+    // — and it runs on the OUTGOING row as well as the incoming one. Measured on
+    // a real navigation: for 150ms two rows carried the accent background AND
+    // the accent bar, mid-fade, with nothing to say which one was the page you
+    // were actually on. Shortening it (300ms -> 150ms) only shortened the lie.
+    //
+    // Split apart, each gets the timing its meaning calls for: the wash fades,
+    // because it is decoration reacting to a cursor; the accent switches in a
+    // single frame, because it is a statement of fact — exactly one row is the
+    // current page, at every instant, including the instant the route commits.
+    "after:pointer-events-none after:absolute after:inset-0 after:-z-10 after:bg-ods-bg-hover after:opacity-0 after:transition-opacity after:duration-150 after:content-['']",
     "before:absolute before:bottom-0 before:left-0 before:top-0 before:w-1 before:content-['']",
-    'before:transition-colors before:duration-150',
     // Pending borrows the HOVER weight and nothing else — no accent bar, no
     // accent text, no `aria-current`. It says "this row is working", not "this
     // is where you are", and it reads as the row staying pressed under the
     // cursor, which is exactly what happened.
-    !isActive && !disabled && 'text-ods-text-primary hover:bg-ods-bg-hover [&_svg]:fill-ods-text-secondary',
-    !isActive && !disabled && isPending && 'bg-ods-bg-hover',
+    !isActive && !disabled && 'text-ods-text-primary hover:after:opacity-100 [&_svg]:fill-ods-text-secondary',
+    !isActive && !disabled && isPending && 'after:opacity-100',
     !isActive && disabled && 'text-ods-text-secondary [&_svg]:fill-ods-text-secondary',
     isActive &&
       !disabled && [

package/src/components/ui/hover-dropdown.tsx

@@ -106,7 +106,6 @@ export function HoverDropdown({
 }: HoverDropdownProps) {
   const [open, setOpen] = useState(false);
   const timeoutRef = useRef<ReturnType<typeof setTimeout>>(undefined);
-  const clickPinnedRef = useRef(false);
   const containerRef = useRef<HTMLDivElement>(null);
   const panelRef = useRef<HTMLDivElement>(null);
 
@@ -157,22 +156,18 @@ export function HoverDropdown({
   }, []);
 
   const hide = useCallback(() => {
-    if (clickPinnedRef.current) return;
     timeoutRef.current = setTimeout(() => setOpen(false), hideDelay);
   }, [hideDelay]);
 
   // Click handler — touch / mobile (no hover) + desktop users who
-  // expect clicking a chip to open a menu. The first desktop click can arrive
-  // after mouse-enter already opened the panel; pin it open instead of blindly
-  // toggling it closed. A second click closes it. `stopPropagation` prevents the click
+  // expect clicking a chip to open a menu. Toggles the dropdown state
+  // so a second click closes it. `stopPropagation` prevents the click
   // from bubbling to ancestor handlers (the chip's parent message
   // bubble shouldn't react to chip clicks).
   const toggle = useCallback((e: React.MouseEvent) => {
     e.stopPropagation();
     clearTimeout(timeoutRef.current);
-    const pinned = !clickPinnedRef.current;
-    clickPinnedRef.current = pinned;
-    setOpen(pinned);
+    setOpen(prev => !prev);
   }, []);
 
   // Every hook is above this line: an `items` list that goes from empty to

package/dist/chat-protocol/source-metadata.d.ts

@@ -1,5 +0,0 @@
-import type { ChatRef } from '../components/chat/chat-ref.types';
-import type { SourcesEvent } from './events';
-export declare function isSourceMetadataCardRef(ref: ChatRef): boolean;
-export declare function sourceMetadataEvent(payload: unknown): SourcesEvent | null;
-//# sourceMappingURL=source-metadata.d.ts.map

package/dist/chat-protocol/source-metadata.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"source-metadata.d.ts","sourceRoot":"","sources":["../../src/chat-protocol/source-metadata.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,OAAO,EAAE,MAAM,mCAAmC,CAAC;AAEjE,OAAO,KAAK,EAAE,YAAY,EAAE,MAAM,UAAU,CAAC;AAK7C,wBAAgB,uBAAuB,CAAC,GAAG,EAAE,OAAO,GAAG,OAAO,CAE7D;AAmLD,wBAAgB,mBAAmB,CAAC,OAAO,EAAE,OAAO,GAAG,YAAY,GAAG,IAAI,CAazE"}