@flamingo-stack/openframe-frontend-core 0.0.670 → 0.0.671-snapshot.20260928023432

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@flamingo-stack/openframe-frontend-core",
-  "version": "0.0.670",
+  "version": "0.0.671-snapshot.20260928023432",
   "description": "Shared design system and components for all Flamingo platforms",
   "type": "module",
   "main": "./dist/index.js",

package/src/__tests__/search-input-submit-on-clear.test.tsx

@@ -0,0 +1,32 @@
+import { fireEvent, render, screen } from '@testing-library/react';
+import { useState } from 'react';
+import { describe, expect, it, vi } from 'vitest';
+import { SearchInput } from '../components/ui/search-input';
+
+// An enter-to-search host: a submitted search must not stay applied once its text is gone.
+function Host({ onSubmit }: { onSubmit: (v: string) => void }) {
+  const [value, setValue] = useState('sso');
+  return (
+    <SearchInput value={value} onChange={setValue} onSubmit={onSubmit} showDropdown={false} placeholder="Search" />
+  );
+}
+
+describe('SearchInput submits the empty search when the box is emptied', () => {
+  it('by the clear button', () => {
+    const onSubmit = vi.fn();
+    render(<Host onSubmit={onSubmit} />);
+    fireEvent.click(screen.getByLabelText('Clear search'));
+    expect(onSubmit).toHaveBeenCalledWith('');
+  });
+
+  it('by deleting the text, once (not on every keystroke, and not when it was already empty)', () => {
+    const onSubmit = vi.fn();
+    render(<Host onSubmit={onSubmit} />);
+    const input = screen.getByPlaceholderText('Search');
+    fireEvent.change(input, { target: { value: 'ss' } });
+    expect(onSubmit).not.toHaveBeenCalled();
+    fireEvent.change(input, { target: { value: '' } });
+    expect(onSubmit).toHaveBeenCalledTimes(1);
+    expect(onSubmit).toHaveBeenCalledWith('');
+  });
+});

package/src/chat-protocol/__tests__/card-marker.test.ts

@@ -16,7 +16,7 @@
  */
 
 import { describe, expect, it } from 'vitest';
-import { CARD_REFERENCE, createCardMarkerScanner } from '../card-marker';
+import { CARD_REFERENCE, createCardMarkerScanner, extractCardReferences } from '../card-marker';
 
 /** The id half as the backend validates it: anything but `]` and newlines. */
 const PUNCTUATED_IDS = [
@@ -61,3 +61,29 @@ describe('card marker grammar', () => {
     expect(CARD_REFERENCE.exec('[card://faq:a] plus a note')).toBeNull();
   });
 });
+
+describe('extractCardReferences', () => {
+  it('returns each reference once, in order of first appearance', () => {
+    const text =
+      'See [card://hubspot_ticket_self:48720777129] and [card://faq:a], ' +
+      'then [card://hubspot_ticket_self:48720777129] again ([card://podcast:9b2f).';
+
+    expect(extractCardReferences(text)).toEqual([
+      { type: 'hubspot_ticket_self', id: '48720777129' },
+      { type: 'faq', id: 'a' },
+      { type: 'podcast', id: '9b2f' },
+    ]);
+  });
+
+  it('finds nothing in text without markers, or with a malformed one', () => {
+    expect(extractCardReferences('no cards here')).toEqual([]);
+    expect(extractCardReferences('[card://faq:]')).toEqual([]);
+  });
+
+  it('is served from the server-safe `chat-protocol` entry point', async () => {
+    const chatProtocol = await import('../index');
+    expect(chatProtocol.extractCardReferences).toBe(extractCardReferences);
+    expect(chatProtocol.createCardMarkerScanner).toBe(createCardMarkerScanner);
+    expect(chatProtocol.CARD_REFERENCE).toBe(CARD_REFERENCE);
+  });
+});

package/src/chat-protocol/card-marker.ts

@@ -38,3 +38,27 @@ export const CARD_REFERENCE = new RegExp(`^\\[card://(${CARD_TYPE}):(${CARD_ID})
 export function createCardMarkerScanner(): RegExp {
   return new RegExp(`\\[card://(${CARD_TYPE}):(${CARD_ID})[\\])]`, 'g');
 }
+
+/** One `[card://<type>:<id>]` reference, split into its two halves. */
+export interface CardReference {
+  type: string;
+  id: string;
+}
+
+/**
+ * Every card reference EMBEDDED in a text, in order of first appearance,
+ * each once. The server reads the references an answer cites through this
+ * (the hub's MCP `ask_guide` returns them as `cards[]`), so a host and the
+ * body renderer can never disagree about which markers a text carries.
+ */
+export function extractCardReferences(text: string): CardReference[] {
+  const seen = new Set<string>();
+  const out: CardReference[] = [];
+  for (const match of text.matchAll(createCardMarkerScanner())) {
+    const key = `${match[1]}:${match[2]}`;
+    if (seen.has(key)) continue;
+    seen.add(key);
+    out.push({ type: match[1], id: match[2] });
+  }
+  return out;
+}

package/src/chat-protocol/index.ts

@@ -23,3 +23,6 @@ export * from './ip-normalize';
 // forwarding above must be parsed IDENTICALLY on both sides of the seam.
 export * from './env-flag';
 export { isRecord, unwrapEnvelope } from './wire-narrow';
+// The `[card://<type>:<id>]` grammar — the renderer's and the server's one
+// definition, so a reference the hub emits is one the client expands.
+export * from './card-marker';

package/src/components/ui/search-input.tsx

@@ -51,7 +51,11 @@ export interface SearchInputProps {
   defaultValue?: string;
   /** Called when input value changes (raw, not debounced) */
   onChange?: (value: string) => void;
-  /** Called when user presses Enter */
+  /**
+   * Called when the user presses Enter, AND with `''` when the box is emptied (the clear button, or deleting the
+   * text): a search that was submitted must not stay applied after its text is gone. Read the argument, never the
+   * host's own state, which has not re-rendered yet when this fires.
+   */
   onSubmit?: (value: string) => void;
   /** Search results to display in the dropdown */
   results?: SearchResult[];
@@ -292,6 +296,7 @@ export function SearchInput({
     } else {
       setInternalValue(newVal);
     }
+    if (newVal === '' && currentValue !== '') onSubmit?.('');
     if (!isOpen) setIsOpen(true);
     setHighlightedIndex(-1);
   };
@@ -304,6 +309,7 @@ export function SearchInput({
     } else {
       setInternalValue('');
     }
+    if (currentValue !== '') onSubmit?.('');
     inputRef.current?.focus();
   };
 

package/src/types/design-doc.ts

@@ -376,7 +376,11 @@ export interface DesignDocListResponse {
   count: number;
   /** Server-computed spec-mode counts over the scope, MINUS the spec-mode
    *  filter itself — so a filtered row still shows what the other value holds. */
-  facets: { spec_mode: Record<string, number> };
+  facets: {
+    spec_mode: Record<string, number>;
+    /** The same over readiness (`ready` / `not_ready`), MINUS the ready filter itself: the filter row's options. */
+    ready: Record<'ready' | 'not_ready', number>;
+  };
 }
 
 export interface DesignDocStats {