@flamingo-stack/openframe-frontend-core 0.0.724 → 0.0.726

package/dist/utils/index.js

@@ -3573,11 +3573,11 @@ var SOURCE_ICON_NAMES = {
   vendors: "package-search",
   // Financials
   "investor-updates": "mail",
-  "financial-kpis": "activity",
-  "financial-cap-table": "table",
-  "financial-pnl": "trending-up",
-  "financial-balance-sheet": "dollar-sign",
-  "financial-cash-flow": "banknote",
+  "financial-kpis": "chart-bar-01-ver",
+  "financial-cap-table": "chart-pie",
+  "financial-pnl": "money-bill-dollar",
+  "financial-balance-sheet": "bank",
+  "financial-cash-flow": "coins-exchange-currency",
   // ClickUp
   "clickup-roadmap": "clickup",
   "clickup-delivery": "clickup",
@@ -3596,22 +3596,22 @@ var SOURCE_ICON_NAMES = {
   // Communications
   "slack-messages": "slack",
   // People-hub employee content
-  "what-i-shipped": "package",
-  "how-i-work": "book-open",
-  "ai-prompts": "message-square",
+  "what-i-shipped": "rocket-02",
+  "how-i-work": "clipboard-list",
+  "ai-prompts": "chat-quote",
   // Product-hub internal
   "design-docs": "file-text",
   "openframe-tenants": "openframe",
-  "prospect-calls": "phone",
+  "prospect-calls": "call",
   // Code intelligence
-  "code-rules": "check-square",
-  "code-graph-repos": "github",
+  "code-rules": "bracket-curly-check",
+  "code-graph-repos": "coding-branch",
   "code-graph-deployments": "package",
-  "code-graph-files": "file-text",
-  "code-symbols": "box",
-  "code-duplicates": "search",
-  "code-impact": "activity",
-  "change-sets": "coding-branch"
+  "code-graph-files": "file-code",
+  "code-symbols": "code-square",
+  "code-duplicates": "copy-01",
+  "code-impact": "coding-pull-request",
+  "change-sets": "coding-merge"
 };
 function getSourceIconName(tableId) {
   if (!tableId) return void 0;
@@ -3785,8 +3785,7 @@ function typeOf(source) {
   return source.documentType || defaultDocumentTypeForTableId(source.sourceRepo) || "";
 }
 function joinsTableChip(source) {
-  if (!source.sourceRepo) return false;
-  return groupsByTable(typeOf({ ...source, sourceRepo: source.sourceRepo }));
+  return !!source.sourceRepo;
 }
 function rowsOf(source) {
   if (isGroupedChip(source)) return source.items ?? [];

package/dist/utils/source-grouping.d.ts

@@ -25,13 +25,22 @@ import type { ChatSource } from '../components/chat/types/message.types';
 /**
  * Doc-table documentTypes: whole documents with their own viewer (markdown =
  * product docs, data_room_doc = data room). They carry an in-app `path`, an
- * embedder keys `docPlatformTargets` by them, and they NEVER group: one chip
- * per document. The hub's per-source contract test pins this list to its
+ * embedder keys `docPlatformTargets` by them, and a server never folds them
+ * into one citation (`groupsByTable`); the strip still draws them as one chip
+ * per table. The hub's per-source contract test pins this list to its
  * doc-table configs.
  */
 export declare const DOC_TABLE_TYPES: readonly ["markdown", "data_room_doc"];
-/** THE rule: rows of a knowledge table share one chip unless they are whole
- *  documents. Decided by `documentType`, the one fact every producer has. */
+/**
+ * Which tables a SERVER folds into one citation: rows of a knowledge table
+ * share one number (and reach the model as one table) unless they are whole
+ * documents, which each keep their own number and are read in full. Decided by
+ * `documentType`, the one fact every producer has.
+ *
+ * This is not the strip's rule. The strip (`groupSourcesByTable`) draws ONE chip
+ * per table for every source, documents included: a document keeps its own
+ * citation number inside its table's chip.
+ */
 export declare function groupsByTable(documentType: string | null | undefined): boolean;
 /** `(1 record)` / `(14 records)`: the count every grouped chip states. */
 export declare function recordCountLabel(count: number): string;
@@ -71,7 +80,8 @@ export declare function buildGroupedSource(spec: {
  * table's only source. When flat rows of the same table arrive beside it (a
  * hub-injected chip plus MCP-cited rows in one answer), they merge into it:
  * its rows first, its chip-level link kept, the count restated. Whole
- * documents and rows naming no table pass through as their own chips.
+ * documents group under their table too, each row keeping its own citation
+ * number; only a row naming no table passes through as its own chip.
  */
 export declare function groupSourcesByTable(sources: ChatSource[]): ChatSource[];
 /**

package/dist/utils/source-grouping.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"source-grouping.d.ts","sourceRoot":"","sources":["../../src/utils/source-grouping.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;;;;GAsBG;AAEH,OAAO,KAAK,EAAE,UAAU,EAAE,MAAM,wCAAwC,CAAC;AAGzE;;;;;;GAMG;AACH,eAAO,MAAM,eAAe,wCAAyC,CAAC;AAEtE;6EAC6E;AAC7E,wBAAgB,aAAa,CAAC,YAAY,EAAE,MAAM,GAAG,IAAI,GAAG,SAAS,GAAG,OAAO,CAE9E;AAED,0EAA0E;AAC1E,wBAAgB,gBAAgB,CAAC,KAAK,EAAE,MAAM,GAAG,MAAM,CAEtD;AAED,KAAK,WAAW,GAAG,WAAW,CAAC,UAAU,CAAC,OAAO,CAAC,CAAC,CAAC,MAAM,CAAC,CAAC;AAE5D;2EAC2E;AAC3E,MAAM,MAAM,gBAAgB,GAAG,WAAW,CAAC;AAE3C;;;;;;;;GAQG;AACH,wBAAgB,kBAAkB,CAAC,IAAI,EAAE;IACvC,kDAAkD;IAClD,KAAK,EAAE,MAAM,CAAC;IACd,UAAU,EAAE,MAAM,CAAC;IACnB,IAAI,EAAE,gBAAgB,EAAE,CAAC;IACzB,2DAA2D;IAC3D,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,cAAc,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;CAChC,GAAG,UAAU,CAcb;AA0DD;;;;;;;;;;;;;;;GAeG;AACH,wBAAgB,mBAAmB,CAAC,OAAO,EAAE,UAAU,EAAE,GAAG,UAAU,EAAE,CAqCvE;AAED;;;;;;GAMG;AACH,wBAAgB,qBAAqB,CAAC,OAAO,EAAE,MAAM,EAAE,GAAG,MAAM,CAU/D"}
+{"version":3,"file":"source-grouping.d.ts","sourceRoot":"","sources":["../../src/utils/source-grouping.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;;;;GAsBG;AAEH,OAAO,KAAK,EAAE,UAAU,EAAE,MAAM,wCAAwC,CAAC;AAGzE;;;;;;;GAOG;AACH,eAAO,MAAM,eAAe,wCAAyC,CAAC;AAEtE;;;;;;;;;GASG;AACH,wBAAgB,aAAa,CAAC,YAAY,EAAE,MAAM,GAAG,IAAI,GAAG,SAAS,GAAG,OAAO,CAE9E;AAED,0EAA0E;AAC1E,wBAAgB,gBAAgB,CAAC,KAAK,EAAE,MAAM,GAAG,MAAM,CAEtD;AAED,KAAK,WAAW,GAAG,WAAW,CAAC,UAAU,CAAC,OAAO,CAAC,CAAC,CAAC,MAAM,CAAC,CAAC;AAE5D;2EAC2E;AAC3E,MAAM,MAAM,gBAAgB,GAAG,WAAW,CAAC;AAE3C;;;;;;;;GAQG;AACH,wBAAgB,kBAAkB,CAAC,IAAI,EAAE;IACvC,kDAAkD;IAClD,KAAK,EAAE,MAAM,CAAC;IACd,UAAU,EAAE,MAAM,CAAC;IACnB,IAAI,EAAE,gBAAgB,EAAE,CAAC;IACzB,2DAA2D;IAC3D,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,cAAc,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;CAChC,GAAG,UAAU,CAcb;AA0DD;;;;;;;;;;;;;;;;GAgBG;AACH,wBAAgB,mBAAmB,CAAC,OAAO,EAAE,UAAU,EAAE,GAAG,UAAU,EAAE,CAqCvE;AAED;;;;;;GAMG;AACH,wBAAgB,qBAAqB,CAAC,OAAO,EAAE,MAAM,EAAE,GAAG,MAAM,CAU/D"}

package/package.json

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

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

@@ -61,29 +61,14 @@ 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';
 import { SlackLogoGreyIcon } from '../../icons-v2-generated/brand-logos/slack-logo-grey-icon';
-import { ChartBar01VerIcon } from '../../icons-v2-generated/charts/chart-bar-01-ver-icon';
-import { ChartPieIcon } from '../../icons-v2-generated/charts/chart-pie-icon';
 import { PresentationBarIcon } from '../../icons-v2-generated/charts/presentation-bar-icon';
 import { PresentationLineIcon } from '../../icons-v2-generated/charts/presentation-line-icon';
-import { BracketCurlyCheckIcon } from '../../icons-v2-generated/coding/bracket-curly-check-icon';
 import { CodeIcon } from '../../icons-v2-generated/coding/code-icon';
-import { CodeSquareIcon } from '../../icons-v2-generated/coding/code-square-icon';
-import { CodingBranchIcon } from '../../icons-v2-generated/coding/coding-branch-icon';
 import { CodingCommitIcon } from '../../icons-v2-generated/coding/coding-commit-icon';
-import { CodingMergeIcon } from '../../icons-v2-generated/coding/coding-merge-icon';
 import { CodingPullRequestIcon } from '../../icons-v2-generated/coding/coding-pull-request-icon';
-import { PackageIcon } from '../../icons-v2-generated/coding/package-icon';
-import { CallIcon } from '../../icons-v2-generated/communication/call-icon';
-import { ChatQuoteIcon } from '../../icons-v2-generated/communication/chat-quote-icon';
 import { CalendarIcon } from '../../icons-v2-generated/date-and-time/calendar-icon';
-import { ClipboardListIcon } from '../../icons-v2-generated/documents/clipboard-list-icon';
-import { Copy01Icon } from '../../icons-v2-generated/documents/copy-01-icon';
-import { FileCodeIcon } from '../../icons-v2-generated/documents/file-code-icon';
 import { FileContentIcon } from '../../icons-v2-generated/documents/file-content-icon';
 import { NewspaperIcon } from '../../icons-v2-generated/documents/newspaper-icon';
-import { BankIcon } from '../../icons-v2-generated/finance/bank-icon';
-import { CoinsExchangeCurrencyIcon } from '../../icons-v2-generated/finance/coins-exchange-currency-icon';
-import { MoneyBillDollarIcon } from '../../icons-v2-generated/finance/money-bill-dollar-icon';
 import { MicrophoneIcon } from '../../icons-v2-generated/household/microphone-icon';
 import { AlertTriangleIcon } from '../../icons-v2-generated/interface/alert-triangle-icon';
 import { EyeIcon } from '../../icons-v2-generated/interface/eye-icon';
@@ -113,10 +98,9 @@ import { resolveCardDestination } from '../utils/card-destination';
 import { resolveHrefForRuntime } from '../utils/chat-nav-resolution';
 import { executeNavigation } from '../utils/execute-navigation';
 import { clickupTaskUrl } from '../utils/external-app-urls';
-import { resolveIcon } from '../utils/icon-library';
 import { computeIsNewTab, buildAnchorProps } from '../utils/nav-anchor-props';
 import { readFetchedCardTitle } from '../utils/resolve-fetched-card-href';
-import { getSourceLabel, SOURCE_ICON_NAMES } from '../utils/source-icons';
+import { defaultTableIdForDocumentType, getSourceLabel, SOURCE_ICON_NAMES } from '../utils/source-icons';
 import { resolveSourceIcon, sourceRowCtxFromRuntime } from '../utils/source-row-cta';
 import { BlockCard } from './block-card';
 import { BlogCardSkeleton } from './blog-card';
@@ -1393,18 +1377,16 @@ function refHydratedEntry(
   };
 }
 
+/** The leading glyph carries the type signal (the badge pill is dropped); it is the SOURCE's icon (`sourceGlyph`). */
 interface FinancialCardConfig {
   label: string;
-  /** Per-type leading glyph — carries the type signal now that the
-   *  badge pill is dropped. */
-  icon: () => React.ReactNode;
 }
 const FINANCIAL_CARD_CONFIGS: Record<string, FinancialCardConfig> = {
-  financial_kpi: { label: 'Financial KPI', icon: () => <ChartBar01VerIcon size={24} /> },
-  cap_table: { label: 'Cap table entry', icon: () => <ChartPieIcon size={24} /> },
-  profit_loss: { label: 'P&L period', icon: () => <MoneyBillDollarIcon size={24} /> },
-  balance_sheet: { label: 'Balance sheet', icon: () => <BankIcon size={24} /> },
-  cash_flow: { label: 'Cash flow', icon: () => <CoinsExchangeCurrencyIcon size={24} /> },
+  financial_kpi: { label: 'Financial KPI' },
+  cap_table: { label: 'Cap table entry' },
+  profit_loss: { label: 'P&L period' },
+  balance_sheet: { label: 'Balance sheet' },
+  cash_flow: { label: 'Cash flow' },
 };
 /** Expand a per-family config map into registry entries — ONE loop shell
  *  for every card family (financial, github, program, roadmap); adding a
@@ -1423,7 +1405,7 @@ function financialRegistryEntries(): Record<string, ChatCardRegistryEntry> {
     refHydratedEntry(docType, cfg.label, (displayRef, opts) => (
       <GenericFinancialChatCard
         chatRef={displayRef}
-        icon={cfg.icon()}
+        icon={sourceGlyph(docType)}
         isNewTab={opts.isNewTab}
         discuss={opts.discuss}
       />
@@ -1463,11 +1445,13 @@ const GITHUB_CARD_CONFIGS: Record<string, GitHubCardConfig> = {
 interface GlyphCardConfig {
   label: string;
   /**
-   * The card's leading glyph. It is handed the hydrated ref, so a type whose
-   * records carry their own mark (a vendor's logo, a page's menu icon) draws
-   * that and keeps a generic glyph for a record without one.
+   * ONLY for a type whose records carry their own mark (a vendor's logo, a
+   * page's menu icon): it is handed the hydrated ref and draws that mark.
+   * Every other type leaves it out and gets its SOURCE's icon
+   * (`sourceGlyph`): the one `SOURCE_ICON_NAMES` entry its chip draws too, so
+   * a card and its chip can never show different icons.
    */
-  icon: (ref?: ChatRef) => React.ReactNode;
+  icon?: (ref?: ChatRef) => React.ReactNode;
   /**
    * Video-bearing ref types: render the media card (cover from the ref's
    * `metadata.videoPoster`, else the host's OG placeholder; the type label as the
@@ -1481,41 +1465,51 @@ interface GlyphCardConfig {
  *  (`/api/design-docs`, `/api/openframe-tenants`, `/api/prospect-calls` — ChatRef-shaped items,
  *  same preset as github / slack). Adding one = one line here + the
  *  `list-url.ts` builder + the `source-icons.ts` label / icon / type entries. */
-/** The OpenFrame logo every OpenFrame surface uses (the `openframe` icon name). */
-const OpenFrameGlyph = resolveIcon('openframe');
 const REF_GLYPH_CARD_CONFIGS: Record<string, GlyphCardConfig> = {
-  openframe_tenant: { label: 'OpenFrame tenant', icon: () => <OpenFrameGlyph size={24} /> },
-  prospect_call: { label: 'Prospect call', icon: () => <CallIcon size={24} />, media: true },
+  openframe_tenant: { label: 'OpenFrame tenant' },
+  prospect_call: { label: 'Prospect call', media: true },
   // Code intelligence (product-hub internal): the review rules a repository is
   // held to, the repositories themselves, and one row per deployment
   // environment. Glyph cards like their siblings — none of the three has a
   // cover image, and a rule is read in the admin screen it links to.
-  code_rule: { label: 'Code rule', icon: () => <BracketCurlyCheckIcon size={24} /> },
-  code_repo: { label: 'Repository', icon: () => <CodingBranchIcon size={24} /> },
-  code_deployment: { label: 'Deployment', icon: () => <PackageIcon size={24} /> },
+  code_rule: { label: 'Code rule' },
+  code_repo: { label: 'Repository' },
+  code_deployment: { label: 'Deployment' },
   // The graph's own objects: one source file's public surface, one public
   // symbol, a near-duplicate cluster, and a pull request's blast radius.
   // Glyph cards for the same reason as their three siblings — none carries a
   // cover image, and each links to the admin screen that renders it in full.
-  code_file: { label: 'Code file', icon: () => <FileCodeIcon size={24} /> },
-  code_symbol: { label: 'Code symbol', icon: () => <CodeSquareIcon size={24} /> },
-  code_duplicate: { label: 'Duplicate code', icon: () => <Copy01Icon size={24} /> },
-  code_impact: { label: 'Change impact', icon: () => <CodingPullRequestIcon size={24} /> },
+  code_file: { label: 'Code file' },
+  code_symbol: { label: 'Code symbol' },
+  code_duplicate: { label: 'Duplicate code' },
+  code_impact: { label: 'Change impact' },
   // A change set: pull requests across repositories declared as one change, with the ClickUp tasks and design
   // docs its pull requests are attached to. ONE card for it wherever a set is shown (chat, a design doc's page).
-  change_set: { label: 'Change set', icon: () => <CodingMergeIcon size={24} /> },
+  change_set: { label: 'Change set' },
   // A page of the public website (pricing, product, legal, and every page the navigation lists), read live
   // from the data the page shows. A glyph card: a page has no cover of its own, and the card opens the page.
   site_page: { label: 'Website page', icon: ref => <SitePageGlyph iconName={refText(ref, 'icon_name')} /> },
   // What OpenFrame charges, read live from its billing: the plan in force, or one AI model's token exchange
   // rate. A glyph card; it opens the pricing page.
-  openframe_price: { label: 'OpenFrame price', icon: () => <OpenFrameGlyph size={24} /> },
+  openframe_price: { label: 'OpenFrame price' },
   // A vendor (product) of the OpenMSP directory. A glyph card; it opens the vendor's page there.
   vendor: { label: 'Vendor', icon: ref => <VendorGlyph title={ref?.title ?? ''} logoUrl={refText(ref, 'logo_url')} /> },
 };
+/**
+ * A card type's icon: its SOURCE's, resolved exactly as the source's chip
+ * resolves it (`resolveSourceIcon` over `SOURCE_ICON_NAMES`). One name per
+ * source, one resolver, so a card and its chip always match.
+ */
+function sourceGlyph(docType: string): React.ReactNode {
+  const { Icon } = resolveSourceIcon({ sourceRepo: defaultTableIdForDocumentType(docType), documentType: docType });
+  return <Icon className="size-6" />;
+}
+
 function refGlyphRegistryEntries(): Record<string, ChatCardRegistryEntry> {
-  return registryEntries(REF_GLYPH_CARD_CONFIGS, (cfg, docType) =>
-    refHydratedEntry(docType, cfg.label, (displayRef, opts) =>
+  return registryEntries(REF_GLYPH_CARD_CONFIGS, (config, docType) => {
+    const glyph = config.icon ?? (() => sourceGlyph(docType));
+    const cfg = config;
+    return refHydratedEntry(docType, cfg.label, (displayRef, opts) =>
       cfg.media ? (
         <EntityMingoCard
           title={displayRef.title}
@@ -1524,7 +1518,7 @@ function refGlyphRegistryEntries(): Record<string, ChatCardRegistryEntry> {
             displayRef.metadata?.videoPoster,
             opts.extras?.buildOgPlaceholderUrl?.(displayRef.title ?? '') ?? null,
           )}
-          fallbackIcon={cfg.icon(displayRef)}
+          fallbackIcon={glyph(displayRef)}
           status={{ label: cfg.label, variant: 'grey' }}
           chatRef={displayRef}
           isNewTab={opts.isNewTab}
@@ -1532,15 +1526,10 @@ function refGlyphRegistryEntries(): Record<string, ChatCardRegistryEntry> {
           menuAriaLabel={`${cfg.label} actions`}
         />
       ) : (
-        <GlyphChatCard
-          chatRef={displayRef}
-          icon={cfg.icon(displayRef)}
-          isNewTab={opts.isNewTab}
-          discuss={opts.discuss}
-        />
+        <GlyphChatCard chatRef={displayRef} icon={glyph(displayRef)} isNewTab={opts.isNewTab} discuss={opts.discuss} />
       ),
-    ),
-  );
+    );
+  });
 }
 
 /** People-hub employee feeds hydrate from their EXISTING list APIs
@@ -1558,9 +1547,9 @@ function fetchedEmployeeEntryDisplayRef(item: unknown, chatRef: ChatRef): ChatRe
   return { ...chatRef, title, preview: preview.length > 0 ? preview : chatRef.preview };
 }
 const EMPLOYEE_ENTRY_CARD_CONFIGS: Record<string, GlyphCardConfig> = {
-  what_i_shipped: { label: 'What I Shipped', icon: () => <Rocket02Icon size={24} /> },
-  how_i_work: { label: 'How I Work', icon: () => <ClipboardListIcon size={24} /> },
-  ai_prompt: { label: 'Squawkbox', icon: () => <ChatQuoteIcon size={24} /> },
+  what_i_shipped: { label: 'What I Shipped' },
+  how_i_work: { label: 'How I Work' },
+  ai_prompt: { label: 'Squawkbox' },
 };
 function employeeEntryRegistryEntries(): Record<string, ChatCardRegistryEntry> {
   return registryEntries(EMPLOYEE_ENTRY_CARD_CONFIGS, (cfg, docType) => ({
@@ -1571,7 +1560,7 @@ function employeeEntryRegistryEntries(): Record<string, ChatCardRegistryEntry> {
     render: (item, chatRef, opts) => (
       <GlyphChatCard
         chatRef={fetchedEmployeeEntryDisplayRef(item, chatRef)}
-        icon={cfg.icon()}
+        icon={sourceGlyph(docType)}
         isNewTab={opts.isNewTab}
         discuss={opts.discuss}
       />

package/src/components/chat/utils/icon-library.tsx

@@ -57,7 +57,7 @@ import { MiamiCyberGangLogoFaceOnly } from '../../icons/miami-cyber-gang-logo-fa
 import { MlgLogo } from '../../icons/mlg-logo';
 import { OpenFrameLogo } from '../../icons/openframe-logo';
 import { OpenmspLogo } from '../../icons/openmsp-logo';
-import { getIconComponent } from './icon-registry';
+import { ICON_REGISTRY, getIconComponent, normalizeIconKey } from './icon-registry';
 
 /** Shape used by every icon registered here (matches `icons-v2-generated`). */
 export type IconComponent = ComponentType<{
@@ -288,8 +288,13 @@ export function resolveIcon(
   opts?: { variant?: 'design' | 'brand' },
 ): IconComponent {
   if (opts?.variant === 'brand') {
-    // Brand variant — delegate to the registry resolver (same glyph as before).
-    return getIconComponent(iconName) as IconComponent;
+    // The brand set first. A name it does not hold is drawn from the design
+    // set, so a source whose icon lives only there is never shown as the file
+    // glyph; a name neither holds keeps the registry's own fallback.
+    const brand = iconName ? ICON_REGISTRY[normalizeIconKey(iconName)] : undefined;
+    if (brand) return brand as IconComponent;
+    const design = iconName ? (ICON_ALIASES[iconName] ?? resolveFromLibrary(iconName)) : undefined;
+    return design ?? (getIconComponent(iconName) as IconComponent);
   }
   if (!iconName) return FileIcon;
   return ICON_ALIASES[iconName] ?? resolveFromLibrary(iconName) ?? FileIcon;

package/src/components/chat/utils/source-row-cta.ts

@@ -29,7 +29,7 @@ import { canonicalContentRefType } from '../../../utils/list-url';
 import { DOC_TABLE_TYPES } from '../../../utils/source-grouping';
 import type { ChatRef } from '../chat-ref.types';
 import { safeHref } from './compact-card-classes';
-import { getIconComponent } from './icon-registry';
+import { resolveIcon } from './icon-library';
 import { getSourceIconName } from './source-icons';
 
 /** Path sanitization — keep alphanumerics, slash, dash, dot, underscore;
@@ -146,11 +146,11 @@ export function sourceRowCtxFromRuntime(
 /**
  * Pure icon-resolution helper — `sourceRepo` (RagTableConfig.id) →
  * icon_name via `SOURCE_ICON_NAMES`, then icon_name → React component
- * via `ICON_REGISTRY`.
+ * via `resolveIcon` (the brand set, else the design set).
  */
 function pickSourceIcon(sourceRepo: string | null, documentType: string | null | undefined) {
   const iconName = sourceRepo ? getSourceIconName(sourceRepo) : undefined;
-  const icon = iconName ? getIconComponent(iconName) : FileText;
+  const icon = iconName ? resolveIcon(iconName, { variant: 'brand' }) : FileText;
   const iconLabel = documentType ?? 'Source';
   return { icon, iconLabel };
 }

package/src/utils/source-grouping.ts

@@ -28,14 +28,23 @@ import { defaultDocumentTypeForTableId, getSourceLabel } from './source-icons';
 /**
  * Doc-table documentTypes: whole documents with their own viewer (markdown =
  * product docs, data_room_doc = data room). They carry an in-app `path`, an
- * embedder keys `docPlatformTargets` by them, and they NEVER group: one chip
- * per document. The hub's per-source contract test pins this list to its
+ * embedder keys `docPlatformTargets` by them, and a server never folds them
+ * into one citation (`groupsByTable`); the strip still draws them as one chip
+ * per table. The hub's per-source contract test pins this list to its
  * doc-table configs.
  */
 export const DOC_TABLE_TYPES = ['markdown', 'data_room_doc'] as const;
 
-/** THE rule: rows of a knowledge table share one chip unless they are whole
- *  documents. Decided by `documentType`, the one fact every producer has. */
+/**
+ * Which tables a SERVER folds into one citation: rows of a knowledge table
+ * share one number (and reach the model as one table) unless they are whole
+ * documents, which each keep their own number and are read in full. Decided by
+ * `documentType`, the one fact every producer has.
+ *
+ * This is not the strip's rule. The strip (`groupSourcesByTable`) draws ONE chip
+ * per table for every source, documents included: a document keeps its own
+ * citation number inside its table's chip.
+ */
 export function groupsByTable(documentType: string | null | undefined): boolean {
   return !(DOC_TABLE_TYPES as readonly string[]).includes(documentType ?? '');
 }
@@ -98,16 +107,16 @@ function typeOf(source: TableSource): string {
   return source.documentType || defaultDocumentTypeForTableId(source.sourceRepo) || '';
 }
 
-/** A source that belongs to its table's chip: it names its table and is not a
- *  whole document. Both shapes qualify, a flat row AND a chip the server
- *  already grouped, so one table is never drawn twice. Naming the table is the
- *  ONLY requirement: a row with no `id`, or no `documentType`, still joins (as
- *  the hub's server-side chips do). Leaving it out would split one table
- *  across a group AND a stray chip. What it lacks only limits its dropdown
- *  row: no `id` or no type means Open-only, no Ask. */
+/** A source that belongs to its table's chip: it names its table. Whole
+ *  documents join their table's chip like every other record (each keeps its
+ *  own citation number as a dropdown row). Both shapes qualify, a flat row AND
+ *  a chip the server already grouped, so one table is never drawn twice.
+ *  Naming the table is the ONLY requirement: a row with no `id`, or no
+ *  `documentType`, still joins (as the hub's server-side chips do). Leaving it
+ *  out would split one table across a group AND a stray chip. What it lacks
+ *  only limits its dropdown row: no `id` or no type means Open-only, no Ask. */
 function joinsTableChip(source: ChatSource): source is TableSource {
-  if (!source.sourceRepo) return false;
-  return groupsByTable(typeOf({ ...source, sourceRepo: source.sourceRepo }));
+  return !!source.sourceRepo;
 }
 
 /** A source's dropdown rows: a grouped chip's own items, or the flat row itself. */
@@ -154,7 +163,8 @@ function uniqueRows(rows: GroupedSourceRow[]): GroupedSourceRow[] {
  * table's only source. When flat rows of the same table arrive beside it (a
  * hub-injected chip plus MCP-cited rows in one answer), they merge into it:
  * its rows first, its chip-level link kept, the count restated. Whole
- * documents and rows naming no table pass through as their own chips.
+ * documents group under their table too, each row keeping its own citation
+ * number; only a row naming no table passes through as its own chip.
  */
 export function groupSourcesByTable(sources: ChatSource[]): ChatSource[] {
   const membersByTable = new Map<string, TableSource[]>();

package/src/utils/source-icons.ts

@@ -64,11 +64,11 @@ export const SOURCE_ICON_NAMES: Record<string, string> = {
 
   // Financials
   'investor-updates': 'mail',
-  'financial-kpis': 'activity',
-  'financial-cap-table': 'table',
-  'financial-pnl': 'trending-up',
-  'financial-balance-sheet': 'dollar-sign',
-  'financial-cash-flow': 'banknote',
+  'financial-kpis': 'chart-bar-01-ver',
+  'financial-cap-table': 'chart-pie',
+  'financial-pnl': 'money-bill-dollar',
+  'financial-balance-sheet': 'bank',
+  'financial-cash-flow': 'coins-exchange-currency',
 
   // ClickUp
   'clickup-roadmap': 'clickup',
@@ -92,24 +92,24 @@ export const SOURCE_ICON_NAMES: Record<string, string> = {
   'slack-messages': 'slack',
 
   // People-hub employee content
-  'what-i-shipped': 'package',
-  'how-i-work': 'book-open',
-  'ai-prompts': 'message-square',
+  'what-i-shipped': 'rocket-02',
+  'how-i-work': 'clipboard-list',
+  'ai-prompts': 'chat-quote',
 
   // Product-hub internal
   'design-docs': 'file-text',
   'openframe-tenants': 'openframe',
-  'prospect-calls': 'phone',
+  'prospect-calls': 'call',
 
   // Code intelligence
-  'code-rules': 'check-square',
-  'code-graph-repos': 'github',
+  'code-rules': 'bracket-curly-check',
+  'code-graph-repos': 'coding-branch',
   'code-graph-deployments': 'package',
-  'code-graph-files': 'file-text',
-  'code-symbols': 'box',
-  'code-duplicates': 'search',
-  'code-impact': 'activity',
-  'change-sets': 'coding-branch',
+  'code-graph-files': 'file-code',
+  'code-symbols': 'code-square',
+  'code-duplicates': 'copy-01',
+  'code-impact': 'coding-pull-request',
+  'change-sets': 'coding-merge',
 };
 
 /** Lookup an icon name by RagTableConfig.id. Returns undefined when