gloomberb 0.11.2 → 0.12.0

package/src/components/command-bar/surface/index.test.tsx

@@ -81,10 +81,74 @@ function mutablePaneRegistryMap(map: ReadonlyMap<string, unknown>): Map<string,
 }
 
 describe("CommandBar", () => {
-  test("keeps generic command filtering separate from ticker search", async () => {
+  test("runs symbol search for plain text and folds the hits under the local matches", async () => {
     const searchQueries: string[] = [];
     testSetup = await testRender(<CommandBarHarness
-      query="MSFT"
+      query="msf"
+      dataProvider={makeDataProvider(async (query) => {
+        searchQueries.push(query);
+        return [
+          { providerId: "yahoo", symbol: "MSFT", name: "Microsoft Corp", exchange: "NASDAQ", type: "EQUITY" },
+          { providerId: "yahoo", symbol: "MSF", name: "MFS Municipal Fund", exchange: "NYSE", type: "ETF" },
+        ];
+      })}
+    />, {
+      width: 80,
+      height: 24,
+    });
+
+    await testSetup.renderOnce();
+    const frame = await waitForFrameToContain("Instruments");
+    expect(searchQueries.length).toBeGreaterThan(0);
+    // The listing split of the DES route is collapsed into one section, with
+    // one row per symbol, and a symbol the query spells out is promoted.
+    expect(frame).not.toContain("Other Listings");
+    // Rows lead with a class badge, so the symbol is the second token, not the line start.
+    expect(frame.split("\n").filter((line) => /^\s*\S+\s+MSFT\b/.test(line))).toHaveLength(1);
+    expect(frame).toContain("Exact Match");
+    expect(frame.indexOf("Exact Match")).toBeLessThan(frame.indexOf("Instruments"));
+    // Signed out, the AI section is a sign-up offer, which sits under the answers.
+    expect(frame.indexOf("Instruments")).toBeLessThan(frame.indexOf("Ask AI"));
+  });
+
+  test("keeps the row the user picked when an exact symbol lands above it", async () => {
+    const created: Array<{ templateId: string; options?: PaneTemplateCreateOptions }> = [];
+    let releaseSearch = () => {};
+    const held = new Promise<void>((resolve) => { releaseSearch = resolve; });
+    testSetup = await testRender(<CommandBarHarness
+      query="list"
+      configurePluginRegistry={(pluginRegistry) => {
+        pluginRegistry.createPaneFromTemplateAsyncFn = async (templateId, options) => {
+          created.push({ templateId, options });
+        };
+      }}
+      dataProvider={makeDataProvider(async () => {
+        await held;
+        return [{ providerId: "yahoo", symbol: "LIST", name: "List Corp", exchange: "NYSE", type: "EQUITY" }];
+      })}
+    />, {
+      width: 100,
+      height: 20,
+    });
+
+    await testSetup.renderOnce();
+    const before = testSetup.captureCharFrame();
+    expect(before).not.toContain("Exact Match");
+    // Down moves off the first pane match onto the second one.
+    await emitKeypress(testSetup, { name: "down" });
+    releaseSearch();
+    await waitForFrameToContain("Exact Match");
+
+    // The symbol row renumbered everything under it; Enter still runs the
+    // pane the user had picked, not whatever now sits at its old index.
+    await emitKeypress(testSetup, { name: "return", sequence: "\r" });
+    expect(created).toEqual([{ templateId: "new-watchlist-pane", options: undefined }]);
+  });
+
+  test("keeps symbol search out of a query a prefix claims", async () => {
+    const searchQueries: string[] = [];
+    testSetup = await testRender(<CommandBarHarness
+      query="PF"
       dataProvider={makeDataProvider(async (query) => {
         searchQueries.push(query);
         return [];
@@ -98,9 +162,7 @@ describe("CommandBar", () => {
     await Bun.sleep(260);
     await testSetup.renderOnce();
 
-    const frame = testSetup.captureCharFrame();
-    expect(frame).not.toContain("Tickers");
-    expect(frame).not.toContain("Microsoft Corp.");
+    expect(testSetup.captureCharFrame()).toContain("Shortcut: Portfolio");
     expect(searchQueries).toEqual([]);
   });
 
@@ -120,7 +182,7 @@ describe("CommandBar", () => {
     await testSetup.renderOnce();
 
     expect(calls).toHaveLength(1);
-    expect(testSetup.captureCharFrame()).not.toContain("Commands");
+    expect(testSetup.captureCharFrame()).toContain("Search or run a command");
   });
 
   test("shows one account management result when searching profile", async () => {
@@ -269,52 +331,6 @@ describe("CommandBar", () => {
     expect(frame).toContain("SA");
   });
 
-  test("treats plugin route trigger words as an empty plugin filter", async () => {
-    testSetup = await testRender(<CommandBarHarness query="PL plugins" />, {
-      width: 80,
-      height: 18,
-    });
-
-    await testSetup.renderOnce();
-
-    const frame = testSetup.captureCharFrame();
-    expect(frame).toContain("News");
-    expect(frame).toContain("Notes");
-    expect(frame).not.toContain("No plugins match");
-  });
-
-  test("finds grouped plugins through their owned feature terms", async () => {
-    testSetup = await testRender(<CommandBarHarness
-      query="PL kelly"
-      configurePluginRegistry={(pluginRegistry) => {
-        (pluginRegistry.allPlugins as Map<string, any>).set("portfolio", {
-          id: "portfolio",
-          name: "Portfolio",
-          version: "1.0.0",
-          description: "Portfolio management",
-          toggleable: true,
-        });
-        (pluginRegistry.panes as Map<string, any>).set("kelly-sizer", {
-          id: "kelly-sizer",
-          name: "Position Sizer",
-          component: () => null,
-          defaultPosition: "right",
-        });
-        const getPluginPaneIds = pluginRegistry.getPluginPaneIds;
-        pluginRegistry.getPluginPaneIds = (pluginId: string) => (
-          pluginId === "portfolio" ? ["kelly-sizer"] : getPluginPaneIds(pluginId)
-        );
-      }}
-    />, {
-      width: 80,
-      height: 18,
-    });
-
-    await testSetup.renderOnce();
-
-    expect(testSetup.captureCharFrame()).toContain("Portfolio");
-  });
-
   test("opens plugin command shortcut arguments in the wizard for confirmation", async () => {
     const calls: Array<Record<string, string> | undefined> = [];
 
@@ -543,7 +559,9 @@ describe("CommandBar", () => {
     const frame = await waitForFrameToContain("Security Description");
     expectSingleBackControl(frame);
     expect(frame).toContain("BRK.B");
-    expect(frame).toContain("Equity NYSE");
+    // The class moved into the left badge; the trailing text carries only the venue.
+    expect(frame).toMatch(/EQ\s+BRK\.B.*NYSE/);
+    expect(frame).not.toContain("Equity NYSE");
   });
 
   test("opens ticker search when activating the Ticker Research pane item without a ticker", async () => {
@@ -557,7 +575,8 @@ describe("CommandBar", () => {
     const tickerResearchRow = rootFrame
       .split("\n")
       .find((line) => line.includes("Ticker Research"));
-    expect(tickerResearchRow).toMatch(/\bT\s*$/);
+    // The shortcut sits in the badge column left of the label, not on the right.
+    expect(tickerResearchRow).toMatch(/^\s*T\s+Ticker Research\s*$/);
 
     await act(async () => {
       testSetup!.mockInput.pressEnter();
@@ -579,7 +598,6 @@ describe("CommandBar", () => {
     await testSetup.renderOnce();
 
     let frame = testSetup.captureCharFrame();
-    expect(frame).toContain("Commands");
     expect(frame).toContain("DES");
     expect(frame).toContain("Type a ticker symbol");
     expect(frame).not.toContain("Back");
@@ -753,14 +771,10 @@ describe("CommandBar", () => {
     await testSetup.renderOnce();
     expect(testSetup.captureCharFrame()).toContain("DES AMD");
 
-    await act(async () => {
-      testSetup!.mockInput.pressKey("backspace", { meta: true });
-      await testSetup!.renderOnce();
-    });
+    await emitKeypress(testSetup, { name: "backspace", meta: true });
 
     const frame = testSetup.captureCharFrame();
-    expect(frame).toContain("Commands");
-    expect(frame).toContain("Search");
+    expect(frame).toContain("Type a ticker symbol");
     expect(frame).not.toContain("DES AMD");
   });
 
@@ -777,7 +791,7 @@ describe("CommandBar", () => {
       await testSetup!.renderOnce();
     });
 
-    expect(testSetup.captureCharFrame()).not.toContain("Commands");
+    expect(testSetup.captureCharFrame()).toContain("Search or run a command");
   });
 
   test("DES MSFT opens an exact ticker directly", async () => {
@@ -886,13 +900,15 @@ describe("CommandBar", () => {
 
     await testSetup.renderOnce();
 
+    // Below the sheet; the header row above it hosts the input and is not
+    // click-away territory.
     await act(async () => {
-      await testSetup!.mockMouse.click(0, 0);
+      await testSetup!.mockMouse.click(0, 22);
       await testSetup!.renderOnce();
     });
     await testSetup.renderOnce();
 
-    expect(testSetup.captureCharFrame()).not.toContain("Commands");
+    expect(testSetup.captureCharFrame()).toContain("Search or run a command");
   });
 
   test("groups ticker search sections and keeps saved matches above looser provider results", async () => {
@@ -917,8 +933,8 @@ describe("CommandBar", () => {
     const rows = frame.split("\n");
     const savedHeadings = frame.split("\n").filter((line) => line.trim() === "Saved");
     const otherListingsHeadings = frame.split("\n").filter((line) => line.trim() === "Other Listings");
-    const aaplRow = rows.findIndex((line) => line.trimStart().startsWith("AAPL"));
-    const appRow = rows.findIndex((line) => line.trimStart().startsWith("APP"));
+    const aaplRow = rows.findIndex((line) => /^\s*\S+\s+AAPL\b/.test(line));
+    const appRow = rows.findIndex((line) => /^\s*\S+\s+APP\b/.test(line));
     expect(savedHeadings).toHaveLength(1);
     expect(otherListingsHeadings).toHaveLength(1);
     expect(aaplRow).toBeGreaterThanOrEqual(0);
@@ -983,8 +999,8 @@ describe("CommandBar", () => {
     await testSetup.renderOnce();
     const frame = await waitForFrameToContain("APLY");
     const rows = frame.split("\n");
-    const aaplRow = rows.findIndex((line) => line.trimStart().startsWith("AAPL"));
-    const apcRow = rows.findIndex((line) => line.trimStart().startsWith("APC"));
+    const aaplRow = rows.findIndex((line) => /^\s*\S+\s+AAPL\b/.test(line));
+    const apcRow = rows.findIndex((line) => /^\s*\S+\s+APC\b/.test(line));
 
     expect(aaplRow).toBeGreaterThanOrEqual(0);
     expect(apcRow).toBeGreaterThanOrEqual(0);

package/src/components/command-bar/surface/index.tsx

@@ -7,13 +7,10 @@ import { usePlanAccess } from "../../../plugins/builtin/shared/plan-access";
 import { buildAssistCommandInventory } from "../assist/inventory";
 import { useCommandBarAssist } from "../assist/runtime";
 import { shouldAutoAskAssist, type AssistRowHandlers } from "../assist/model";
-import { useNewsArticles } from "../../../news/hooks";
 import {
-  ARTICLE_SEARCH_QUERY,
-  cachedNewsArticles,
-  looksLikeArticleQuery,
-} from "../../../plugins/builtin/news/wire/article-search";
-import { buildArticleSearchResultItems } from "../routes/root/article-results";
+  getAvailableCommandBarSearchProviders,
+  useCommandBarSearchProviders,
+} from "../routes/root/search-providers";
 import { openUrl } from "../../ui/external-link";
 import { useRouteListState } from "../routing/list-state";
 import { useCommandBarRootRuntime } from "../routes/root/runtime";
@@ -57,6 +54,7 @@ export function CommandBar({
     getCommittedThemeId,
     nativeListScrollRef,
     nativePaneChrome,
+    nativeWindowChrome,
     persistConfig,
     skipTickerSearchDebounceRef,
     state,
@@ -113,7 +111,6 @@ export function CommandBar({
     adaptTickerSearchRouteResult,
     buildLayoutItems,
     buildPaneSettingItems,
-    buildPluginItems,
     buildTickerSearchResultItems,
     buildWindowModeItems,
     collectionWorkflowActions,
@@ -258,31 +255,30 @@ export function CommandBar({
     ),
   }), [askAssistNow, assistActive, assistAutoAsk, assistState, planAccess.emailVerified, startAssistSignUp]);
 
-  const newsState = useNewsArticles(looksLikeArticleQuery(rootQuery) ? ARTICLE_SEARCH_QUERY : null);
-  const articleResultItems = useMemo(() => {
-    const cached = cachedNewsArticles();
-    const seen = new Set<string>();
-    const articles = [];
-    for (const article of [...cached, ...newsState.articles]) {
-      if (seen.has(article.id)) continue;
-      seen.add(article.id);
-      articles.push(article);
-    }
-    const localReady = cached.length > 0
-      || newsState.phase === "ready"
-      || newsState.phase === "refreshing"
-      || newsState.phase === "error";
-    const stillLoading = !localReady && (newsState.phase === "idle" || newsState.phase === "loading");
-    return buildArticleSearchResultItems({
-      articles,
-      query: rootQuery,
-      phase: stillLoading ? "loading" : "ready",
-      onOpen: (article) => {
-        openUrl(article.url);
-        closeAll({ revertThemePreview: false });
-      },
-    });
-  }, [closeAll, newsState.articles, newsState.phase, rootQuery]);
+  const searchProviders = useMemo(
+    () => getAvailableCommandBarSearchProviders(pluginRegistry, state.config.disabledPlugins),
+    [pluginRegistry, state.config.disabledPlugins],
+  );
+  const searchProviderContext = useMemo(() => ({
+    activeTicker: activeTickerSymbol,
+    activeCollectionId,
+  }), [activeCollectionId, activeTickerSymbol]);
+  const closeAfterProviderResult = useCallback(() => {
+    closeAll({ revertThemePreview: false });
+  }, [closeAll]);
+  const { providerResultItems, providerSearching } = useCommandBarSearchProviders({
+    providers: searchProviders,
+    query: rootQuery,
+    // A resolved prefix means the user is running a command, so free-text
+    // providers neither ask the network nor add rows.
+    enabled: !currentRoute && rootShortcutIntent.kind === "none",
+    context: searchProviderContext,
+    onExecuted: closeAfterProviderResult,
+  });
+  const providerCategoryPriorities = useMemo(
+    () => new Map(searchProviders.map((provider) => [provider.category, provider.priority ?? 0])),
+    [searchProviders],
+  );
 
   const {
     activeMatch,
@@ -302,7 +298,6 @@ export function CommandBar({
     availableCommands,
     buildLayoutItems,
     buildPaneSettingItems,
-    buildPluginItems,
     buildTickerSearchResultItems,
     buildWindowModeItems,
     createPaneTemplateItem,
@@ -320,7 +315,9 @@ export function CommandBar({
     paneShortcutItems,
     pluginCommandItems,
     pluginCommandResultItems,
-    articleResultItems,
+    providerResultItems,
+    providerCategoryPriorities,
+    providerSearching,
     readTickerSearchCache,
     rootModeKind: rootModeInfo.kind,
     rootQuery,
@@ -384,10 +381,10 @@ export function CommandBar({
     adaptTickerSearchRouteResult,
     buildLayoutItems,
     buildPaneSettingItems,
-    buildPluginItems,
     currentRoute,
     orderedRootResults,
     pluginRegistry,
+    rootCategoryPriorities: providerCategoryPriorities,
     rootHoveredIdx,
     rootModeKind: rootModeInfo.kind,
     rootQuery,
@@ -432,6 +429,7 @@ export function CommandBar({
     moveWorkflowFocus,
     nativeListScrollRef,
     nativePaneChrome,
+    nativeWindowChrome,
     onNativeOccluderChange,
     openWorkflowFieldPicker,
     persistConfig,
@@ -440,7 +438,6 @@ export function CommandBar({
     resetAssist,
     rootModeKind: rootModeInfo.kind,
     rootGhostSuffix,
-    rootQueryLength: rootQuery.length,
     rootShortcutFeedback,
     routeListState,
     setActiveListQuery,

package/src/components/command-bar/surface/test-harness.tsx

@@ -18,7 +18,8 @@ import type { TickerRecord } from "../../../types/ticker";
 import type { PluginRegistry } from "../../../plugins/registry";
 import type { PaneSettingField } from "../../../types/plugin";
 import { useShortcut } from "../../../react/input";
-import { Text } from "../../../ui";
+import { Box, Text } from "../../../ui";
+import { Header } from "../../layout/header";
 
 export async function emitKeypress(
   renderer: Awaited<ReturnType<typeof testRender>>,
@@ -213,6 +214,8 @@ function makePluginRegistry(hasPaneSettings: (paneId: string) => boolean = () =>
         execute: async () => {},
       }],
     ]),
+    commandBarSearchProviders: new Map<string, any>(),
+    getCommandBarSearchProviderPluginId: () => undefined,
     tickerActions: new Map<string, any>([
       ["pin", {
         id: "pin",
@@ -385,8 +388,17 @@ export function CommandBarHarness({
     <ThemeProvider themeId={getEffectiveThemeId(currentState)}>
       <AppContext value={{ state: currentState, dispatch: currentDispatch }}>
         <TestDialogProvider>
-          {live && <ThemeProbe />}
-          {showQueryState && <Text>{`query:${currentState.commandBarQuery}`}</Text>}
+          {/* The header hosts the bar's input while it is open, so typing in a
+              test needs the real header on the first row. */}
+          <Header />
+          <Box flexGrow={1} />
+          {/* The sheet spans the full width and can reach as far down as
+              termHeight - 6 (panel/layout.ts), so probes live on the bottom row,
+              the one place no sheet height can cover. */}
+          <Box flexDirection="row" gap={1} height={1}>
+            {live && <ThemeProbe />}
+            {showQueryState && <Text>{`query:${currentState.commandBarQuery}`}</Text>}
+          </Box>
           {currentState.commandBarOpen && (
             <CommandBar
               dataProvider={dataProvider}

package/src/components/command-bar/theme-picker.tsx

@@ -13,14 +13,42 @@ import { Box, Text } from "../../ui";
 import { TextAttributes } from "../../ui";
 import { ListView, type ListViewItem } from "../ui";
 import type { ListRowState } from "../ui/list-view";
-import { getThemeIds, themes as themeRegistry } from "../../theme/themes";
+import { getThemeIds, isDarkTheme, themes as themeRegistry } from "../../theme/themes";
 import { truncateText } from "./view-model";
 
 const THEME_PREVIEW_DEBOUNCE_MS = 120;
-const THEME_OPTIONS = getThemeIds().map((id) => ({
-  id,
-  name: themeRegistry[id]!.name,
-}));
+/**
+ * Marks the dark half of the list. The registry's own order groups themes by
+ * family, which only helps if you already know which family you want; sorted by
+ * name you can find one by reading, and the glyph carries the grouping the
+ * order used to.
+ */
+const DARK_THEME_GLYPH = "☾";
+/** The glyph plus the space that keeps names on one left edge, dark or light. */
+const GLYPH_GUTTER_WIDTH = 2;
+
+export interface ThemeOption {
+  id: string;
+  name: string;
+  dark: boolean;
+}
+
+const THEME_OPTIONS: ThemeOption[] = getThemeIds()
+  .map((id) => ({ id, name: themeRegistry[id]!.name, dark: isDarkTheme(id) }))
+  .sort((a, b) => a.name.localeCompare(b.name));
+
+/**
+ * Shared with the panel layout, which sizes the sheet to whatever this returns
+ * so the picker never opens taller than the themes it can show.
+ */
+export function matchThemeOptions(filter: string): ThemeOption[] {
+  const normalized = filter.trim().toLowerCase();
+  if (!normalized) return THEME_OPTIONS;
+  return THEME_OPTIONS.filter((theme) => (
+    theme.name.toLowerCase().includes(normalized)
+    || theme.id.toLowerCase().includes(normalized)
+  ));
+}
 
 interface ThemePickerScrollEvent {
   stopPropagation: () => void;
@@ -83,13 +111,7 @@ export const ThemePicker = memo(forwardRef<ThemePickerHandle, ThemePickerProps>(
   const onPreviewRef = useRef(onPreview);
   const onCommitRef = useRef(onCommit);
   const normalizedFilter = filter.trim().toLowerCase();
-  const themes = useMemo(() => {
-    if (!normalizedFilter) return THEME_OPTIONS;
-    return THEME_OPTIONS.filter((theme) => (
-      theme.name.toLowerCase().includes(normalizedFilter)
-      || theme.id.toLowerCase().includes(normalizedFilter)
-    ));
-  }, [normalizedFilter]);
+  const themes = useMemo(() => matchThemeOptions(normalizedFilter), [normalizedFilter]);
   const [selectedIndex, setSelectedIndex] = useState(() => (
     Math.max(0, themes.findIndex((theme) => theme.id === committedThemeId))
   ));
@@ -199,8 +221,9 @@ export const ThemePicker = memo(forwardRef<ThemePickerHandle, ThemePickerProps>(
     onCommitRef.current(selected.id);
   }, [cancelPreview]);
 
+  const nameWidth = Math.max(1, labelWidth - GLYPH_GUTTER_WIDTH);
   const renderRow = useCallback((item: ListViewItem, state: ListRowState) => {
-    const label = truncateText(item.label, labelWidth);
+    const label = truncateText(item.label, nameWidth);
     const trailing = item.current ? "current" : "";
     return (
       <Box
@@ -211,7 +234,12 @@ export const ThemePicker = memo(forwardRef<ThemePickerHandle, ThemePickerProps>(
         data-command-bar-row-selected={nativePaneChrome && state.selected ? "true" : undefined}
         style={nativePaneChrome ? { borderRadius: 6 } : undefined}
       >
-        <Box width={labelWidth}>
+        <Box width={GLYPH_GUTTER_WIDTH}>
+          <Text fg={state.selected ? paletteSelectedText : paletteSubtleText}>
+            {isDarkTheme(item.id) ? DARK_THEME_GLYPH : ""}
+          </Text>
+        </Box>
+        <Box width={nameWidth}>
           <Text
             fg={state.selected ? paletteSelectedText : paletteText}
             attributes={item.current ? TextAttributes.BOLD : undefined}
@@ -228,7 +256,7 @@ export const ThemePicker = memo(forwardRef<ThemePickerHandle, ThemePickerProps>(
     );
   }, [
     contentPadding,
-    labelWidth,
+    nameWidth,
     nativePaneChrome,
     paletteSelectedText,
     paletteSubtleText,

package/src/components/command-bar/view-model.test.ts

@@ -13,7 +13,6 @@ describe("command bar view model helpers", () => {
     expect(resolveCommandBarMode("DES NVDA")).toMatchObject({ kind: "search", badge: "DES" });
     expect(resolveCommandBarMode("T NVDA")).toMatchObject({ kind: "search", badge: "T" });
     expect(resolveCommandBarMode("TH ")).toMatchObject({ kind: "themes", badge: "THEMES" });
-    expect(resolveCommandBarMode("PL notes")).toMatchObject({ kind: "plugins", badge: "PLUGINS" });
     expect(resolveCommandBarMode("LAY")).toMatchObject({ kind: "direct-command", badge: "COMMAND" });
     expect(resolveCommandBarMode("LMA ")).toMatchObject({ kind: "layout", badge: "LAYOUT" });
     expect(resolveCommandBarMode("NP ")).toMatchObject({ kind: "default", badge: "FILTER" });
@@ -48,23 +47,69 @@ describe("command bar view model helpers", () => {
     expect(sections.map((section) => section.category)).toEqual(["Tickers", "Config", "Danger", "Debug"]);
   });
 
-  test("drops an offer-only section below real matches even when its category leads", () => {
+  test("drops an offer-only section below real matches whatever its category priority", () => {
     const sections = buildSections([
-      { id: "assist:sign-up", category: "Ask AI", disabled: true, defaultSelectable: false },
+      { id: "exact", category: "Exact Match", disabled: true, defaultSelectable: false },
       { id: "holders", category: "Panes" },
     ]);
 
-    expect(sections.map((section) => section.category)).toEqual(["Panes", "Ask AI"]);
+    expect(sections.map((section) => section.category)).toEqual(["Panes", "Exact Match"]);
     expect(sections[0]?.items[0]?.id).toBe("holders");
   });
 
-  test("keeps an answered AI section leading the list", () => {
-    const sections = buildSections([
+  test("leads with an exact symbol, then the AI, then the async sections in arrival order", () => {
+    // An exactly matching symbol is the most certain answer available, so it
+    // outranks the AI's reading of the same query. Below it the AI leads even
+    // though it answers last, since it turns the typed sentence into commands;
+    // its placeholder holds the rows meanwhile. Symbol search and documents
+    // arrive in that order under the local matches, so each only pushes rows
+    // below itself. Documents come from a provider, which contributes its own
+    // priority.
+    const items = [
+      { id: "doc", category: "Documents" },
+      { id: "nvda-mx", category: "Instruments" },
+      { id: "nvda", category: "Exact Match" },
+      { id: "holders", category: "Panes" },
+      { id: "help", category: "Commands" },
+      { id: "quit", category: "Application" },
+      { id: "plugin-row", category: "Portfolio" },
       { id: "assist:candidate:0", category: "Ask AI" },
+    ];
+    const categoryPriorities = new Map([["Documents", 200]]);
+
+    for (const sectionOrder of ["default", "app-first"] as const) {
+      const sections = buildSections(items, { sectionOrder, categoryPriorities });
+      expect(sections.map((section) => section.category)).toEqual([
+        "Exact Match",
+        "Ask AI",
+        "Panes",
+        "Commands",
+        "Application",
+        "Portfolio",
+        "Instruments",
+        "Documents",
+      ]);
+    }
+  });
+
+  test("drops the AI's sign-up offer under the async sections", () => {
+    const sections = buildSections([
+      { id: "assist:sign-up", category: "Ask AI", defaultSelectable: false },
+      { id: "doc", category: "Documents" },
+      { id: "nvda-mx", category: "Instruments" },
       { id: "holders", category: "Panes" },
-    ]);
+    ], { categoryPriorities: new Map([["Documents", 200]]) });
+
+    expect(sections.map((section) => section.category)).toEqual(["Panes", "Instruments", "Documents", "Ask AI"]);
+  });
+
+  test("lets a provider's contributed priority override the built-in band", () => {
+    const sections = buildSections([
+      { id: "nvda-mx", category: "Instruments" },
+      { id: "doc", category: "Documents" },
+    ], { categoryPriorities: new Map([["Documents", -200]]) });
 
-    expect(sections.map((section) => section.category)).toEqual(["Ask AI", "Panes"]);
+    expect(sections.map((section) => section.category)).toEqual(["Documents", "Instruments"]);
   });
 
   test("keeps non-exact ticker suggestions behind app sections in app-first order", () => {