gloomberb 0.10.4 → 0.11.0

package/src/components/command-bar/routes/root/results.ts

@@ -13,6 +13,7 @@ import {
 } from "../../assist/model";
 import { matchPrefix, type Command } from "../../commands/registry";
 import { isCollectionCommand } from "../../helpers";
+import { dedupeById } from "../../view-model";
 import type { ResultItem } from "../../list/model";
 import type { parseRootShortcutIntent } from "./shortcuts";
 import type { CommandBarRoute } from "../../workflow/types";
@@ -21,6 +22,13 @@ import { buildRootShortcutItem } from "./shortcut-items";
 
 type RootShortcutIntent = ReturnType<typeof parseRootShortcutIntent>;
 
+/** The ART plugin command claims the prefix, so article rows must still be shown. */
+export function isArticleLookupShortcut(intent: RootShortcutIntent): boolean {
+  return intent.kind !== "none"
+    && intent.source === "plugin-command"
+    && intent.command.id === "open-news-article";
+}
+
 interface PaneTemplateItemOptions {
   category?: string;
   createOptions?: PaneTemplateCreateOptions;
@@ -67,6 +75,7 @@ export interface RootResultModelOptions {
   pluginCommandResultItems: (command: CommandDef, shortcutArg: string) => ResultItem[];
   rootQuery: string;
   rootShortcutIntent: RootShortcutIntent;
+  articleResultItems?: ResultItem[];
   runDirectCommand: (command: Command, arg: string) => void;
   runSecurityDescriptionShortcut: (query?: string) => void | Promise<void>;
   state: AppState;
@@ -74,9 +83,17 @@ export interface RootResultModelOptions {
 }
 
 /** An in-flight or answered request keeps its rows even if the heuristic lapses. */
-function isAssistSectionVisible(assist: AssistRowHandlers, query: string, resultCount: number): boolean {
+function isAssistSectionVisible(
+  assist: AssistRowHandlers,
+  query: string,
+  resultCount: number,
+  hasShortcutIntent: boolean,
+): boolean {
   if (!query.trim()) return false;
   if (assist.state.status !== "idle" && assist.state.query === query.trim()) return true;
+  // A resolved shortcut is the user speaking the command language, so nothing is
+  // asked of the AI, and a sign-up offer must not outrank that exact match.
+  if (hasShortcutIntent) return false;
   // Signed out there is nothing to wait for, so the older heuristic still picks
   // the queries worth offering a sign-up row for.
   if (!assist.enabled) return shouldShowAssistRow({ query, resultCount });
@@ -108,6 +125,7 @@ export function buildRootResultModel(options: RootResultModelOptions): RootResul
     pluginCommandResultItems,
     rootQuery,
     rootShortcutIntent,
+    articleResultItems = [],
     runDirectCommand,
     runSecurityDescriptionShortcut,
     state,
@@ -157,19 +175,18 @@ export function buildRootResultModel(options: RootResultModelOptions): RootResul
         includePromptableTickerTemplates: true,
       })
       : [];
-    const seenItemIds = new Set<string>();
-    for (const item of [...templateItems, ...relatedTemplateItems]) {
-      if (seenItemIds.has(item.id)) continue;
-      seenItemIds.add(item.id);
-      items.push(item);
-    }
+    items.push(...templateItems, ...relatedTemplateItems);
   } else if (
     rootShortcutIntent.kind !== "none"
     && rootShortcutIntent.source === "plugin-command"
     && shortcutItem
   ) {
-    const dynamicItems = pluginCommandResultItems(rootShortcutIntent.command, rootShortcutIntent.argText);
-    items.push(...(dynamicItems.length > 0 ? dynamicItems : [shortcutItem]));
+    if (isArticleLookupShortcut(rootShortcutIntent)) {
+      items.push(shortcutItem);
+    } else {
+      const dynamicItems = pluginCommandResultItems(rootShortcutIntent.command, rootShortcutIntent.argText);
+      items.push(...(dynamicItems.length > 0 ? dynamicItems : [shortcutItem]));
+    }
   } else if (match && match.command.id === "plugins") {
     items.push(...buildPluginItems(match.arg));
   } else if (match && match.command.id === "layout") {
@@ -227,12 +244,22 @@ export function buildRootResultModel(options: RootResultModelOptions): RootResul
     items.push(...matchedItems);
   }
 
+  if (rootShortcutIntent.kind === "none" || isArticleLookupShortcut(rootShortcutIntent)) {
+    items.push(...articleResultItems);
+  }
+
   // Built from the local matches, then moved above them: the AI answers the
   // question the user typed, so it leads the list. Rows landing here renumber
   // everything below, which the root selection effect absorbs by identity.
-  const assistItems = assist && isAssistSectionVisible(assist, rootQuery, items.length)
+  const assistItems = assist
+    && isAssistSectionVisible(
+      assist,
+      rootQuery,
+      items.length,
+      rootShortcutIntent.kind !== "none" && !isArticleLookupShortcut(rootShortcutIntent),
+    )
     ? buildAssistResultItems({ ...assist, query: rootQuery })
     : [];
 
-  return { items: [...assistItems, ...items], initialIdx };
+  return { items: dedupeById([...assistItems, ...items]), initialIdx };
 }

package/src/components/command-bar/routes/root/runtime.ts

@@ -1,4 +1,4 @@
-import { useEffect, useMemo, useRef, type Dispatch, type RefObject, type SetStateAction } from "react";
+import { useEffect, useLayoutEffect, useMemo, useRef, type Dispatch, type RefObject, type SetStateAction } from "react";
 import type { ScrollBoxRenderable } from "../../../../ui";
 import type { AppState } from "../../../../state/app/context";
 import type { DataProvider } from "../../../../types/data-provider";
@@ -63,6 +63,7 @@ interface UseCommandBarRootRuntimeOptions {
   }): ResultItem[];
   pluginCommandItems(): ResultItem[];
   pluginCommandResultItems(command: CommandDef, shortcutArg: string): ResultItem[];
+  articleResultItems?: ResultItem[];
   readTickerSearchCache(
     query: string,
     brokerId?: string | null,
@@ -114,6 +115,7 @@ export function useCommandBarRootRuntime({
   paneShortcutItems,
   pluginCommandItems,
   pluginCommandResultItems,
+  articleResultItems = [],
   readTickerSearchCache,
   rootModeKind,
   rootQuery,
@@ -187,6 +189,7 @@ export function useCommandBarRootRuntime({
     pluginCommandResultItems,
     rootQuery,
     rootShortcutIntent,
+    articleResultItems,
     runDirectCommand,
     runSecurityDescriptionShortcut,
     state,
@@ -215,6 +218,7 @@ export function useCommandBarRootRuntime({
     pluginCommandResultItems,
     rootQuery,
     rootShortcutIntent,
+    articleResultItems,
     runDirectCommand,
     runSecurityDescriptionShortcut,
     state,
@@ -246,7 +250,7 @@ export function useCommandBarRootRuntime({
     writeTickerSearchCache,
   });
 
-  useEffect(() => {
+  useLayoutEffect(() => {
     if (currentRoute) return;
 
     const resultIds = orderedRootResults.map((item) => item.id);

package/src/components/command-bar/routes/ticker-search/actions.ts

@@ -43,7 +43,7 @@ export function useCommandBarTickerSearchActions({
       focusTicker(ticker.metadata.ticker, options);
       closeAll({ revertThemePreview: false });
     })();
-  }, [tickerRepository, dispatch, pluginRegistry.events, focusTicker, closeAll]);
+  }, [closeAll, dispatch, focusTicker, pluginRegistry.events, tickerRepository]);
 
   const mapTickerSearchCandidateToResultItem = useCallback((candidate: TickerSearchCandidate): ResultItem => {
     const detail = formatTickerSearchDetail(candidate);

package/src/components/command-bar/routes/ticker-search/results.test.ts

@@ -0,0 +1,34 @@
+import { expect, test } from "bun:test";
+import type { ResultItem } from "../../list/model";
+import { mergeTickerSearchResultItems } from "./results";
+
+function resultItem(id: string, label: string, right: string, kind: ResultItem["kind"] = "ticker"): ResultItem {
+  return {
+    id,
+    label,
+    detail: "Apple Inc.",
+    category: kind === "info" ? "Search" : "Saved",
+    kind,
+    right,
+    action: () => {},
+  };
+}
+
+test("keeps completed ticker-search results authoritative over provisional rows", () => {
+  const authoritative = [
+    resultItem("ranked:AAPL", "AAPL", "Equity NASDAQ"),
+    resultItem("ranked:APC", "APC", "Equity XETRA"),
+  ];
+  const provisional = [
+    resultItem("local:APC", "APC", "Equity XETRA"),
+    resultItem("local:AAPL", "AAPL", "Equity NASDAQ"),
+  ];
+
+  expect(mergeTickerSearchResultItems("Apple", authoritative, provisional).map((item) => item.id)).toEqual([
+    "ranked:AAPL",
+    "ranked:APC",
+  ]);
+
+  const noResults = resultItem("no-results", "No matches for Apple", "", "info");
+  expect(mergeTickerSearchResultItems("Apple", [noResults], [])).toEqual([noResults]);
+});

package/src/components/command-bar/routes/ticker-search/results.ts

@@ -1,7 +1,6 @@
 import type { TickerRecord } from "../../../../types/ticker";
 import {
   createLocalTickerSearchCandidates,
-  rankTickerSearchItems,
   type TickerSearchCandidate,
 } from "../../../../tickers/search";
 import type { ResultItem } from "../../list/model";
@@ -33,7 +32,7 @@ function isExactTickerResultMatch(item: ResultItem, query: string): boolean {
 
 export function mergeTickerSearchResultItems(
   query: string,
-  preferredItems: ResultItem[],
+  rankedItems: ResultItem[],
   fallbackItems: ResultItem[],
 ): ResultItem[] {
   const merged: ResultItem[] = [];
@@ -45,12 +44,11 @@ export function mergeTickerSearchResultItems(
     seen.add(key);
     merged.push(item);
   };
-  preferredItems.forEach(addItem);
+  rankedItems.forEach(addItem);
   fallbackItems.forEach(addItem);
-  if (merged.length === 0) return fallbackItems;
+  if (merged.length === 0) return rankedItems.length > 0 ? rankedItems : fallbackItems;
 
-  return rankTickerSearchItems(merged, query)
-    .map((item) => isExactTickerResultMatch(item, query) && item.category !== "Saved"
+  return merged.map((item) => isExactTickerResultMatch(item, query) && item.category !== "Saved"
       ? { ...item, category: "Exact Match" }
       : item);
 }

package/src/components/command-bar/routes/ticker-search/route.ts

@@ -80,7 +80,7 @@ export function useTickerSearchRouteResults(options: {
       brokerInstanceId,
     );
     setTickerSearchResults(cachedCandidates
-      ? mergeTickerSearchResultItems(searchQuery, localItems, buildTickerSearchResultItems(cachedCandidates, searchQuery))
+      ? mergeTickerSearchResultItems(searchQuery, buildTickerSearchResultItems(cachedCandidates, searchQuery), localItems)
       : localItems);
     const requestId = ++searchRequestIdRef.current;
     const searchDelay = skipTickerSearchDebounceRef.current ? 0 : 200;
@@ -109,8 +109,8 @@ export function useTickerSearchRouteResults(options: {
         );
         setTickerSearchResults(mergeTickerSearchResultItems(
           searchQuery,
-          localTickerSearchResultItems(searchQuery, { limit: 6 }),
           buildTickerSearchResultItems(combined, searchQuery),
+          localTickerSearchResultItems(searchQuery, { limit: 6 }),
         ));
       } catch {
         if (requestId !== searchRequestIdRef.current) return;

package/src/components/command-bar/routing/list-state.ts

@@ -135,12 +135,13 @@ function buildRouteListState(options: BuildRouteListStateOptions): ListScreenSta
           query: currentRoute.query,
           selectedIdx: currentRoute.selectedIdx,
           hoveredIdx: currentRoute.hoveredIdx,
-          results: orderListResults(results),
+          results: orderListResults(results, { sectionOrder: "ranked" }),
           searching: tickerSearchPending,
           emptyLabel: emptyState.label,
           emptyDetail: emptyState.detail,
           footerLeft: getScreenFooterLeft(currentRoute),
           footerRight: getScreenFooterRight(currentRoute),
+          sectionOrder: "ranked",
         };
       }
       default:

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

@@ -144,12 +144,17 @@ describe("CommandBar AI assist", () => {
 
   test("keeps the row the user picked when an answer lands above it", async () => {
     signInVerified();
-    mockAssistTransport(() => jsonResponse({
-      candidates: [
-        { input: "CHAT #general", title: "Open the general channel", prefix: "CHAT", confidence: 0.9 },
-        { input: "CHAT #random", title: "Open the random channel", prefix: "CHAT", confidence: 0.4 },
-      ],
-    }));
+    let releaseResponse = () => {};
+    const held = new Promise<void>((resolve) => { releaseResponse = resolve; });
+    const requests = mockAssistTransport(async () => {
+      await held;
+      return jsonResponse({
+        candidates: [
+          { input: "CHAT #general", title: "Open the general channel", prefix: "CHAT", confidence: 0.9 },
+          { input: "CHAT #random", title: "Open the random channel", prefix: "CHAT", confidence: 0.4 },
+        ],
+      });
+    });
     const created: Array<{ templateId: string; options?: PaneTemplateCreateOptions }> = [];
 
     testSetup = await testRender(
@@ -161,8 +166,10 @@ describe("CommandBar AI assist", () => {
     );
 
     await testSetup.renderOnce();
+    await waitForRequest(requests);
     // Down lands on the local match while a single "Thinking…" row sits above.
     await emitKeypress(testSetup, { name: "down" });
+    releaseResponse();
     await waitForFrameToContain("CHAT #random — Open the random channel", ASSIST_WAIT_ATTEMPTS);
 
     // Two answers replaced that one row, so the chosen row moved down by one —

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

@@ -926,6 +926,71 @@ describe("CommandBar", () => {
     expect(aaplRow).toBeLessThan(appRow);
   });
 
+  test("keeps the provider-ranked canonical listing ahead of provisional saved order", async () => {
+    testSetup = await testRender(
+      <CommandBarHarness
+        query=""
+        extraTickers={[makeTicker("APC", "Apple Inc.", {
+          exchange: "XETRA",
+          currency: "EUR",
+          assetCategory: "STK",
+        })]}
+        configureState={(state) => ({
+          ...state,
+          commandBarLaunchRequest: {
+            kind: "ticker-search",
+            query: "Apple",
+            sequence: 1,
+          },
+          tickers: new Map([
+            ["APC", state.tickers.get("APC")!],
+            ["AAPL", state.tickers.get("AAPL")!],
+            ["MSFT", state.tickers.get("MSFT")!],
+          ]),
+        })}
+        dataProvider={makeDataProvider(async () => [
+          {
+            providerId: "yahoo",
+            symbol: "AAPL",
+            name: "Apple Inc.",
+            exchange: "NASDAQ",
+            primaryExchange: "NASDAQ",
+            type: "EQUITY",
+            currency: "USD",
+          },
+          {
+            providerId: "broker",
+            symbol: "APC",
+            name: "Apple Inc.",
+            exchange: "XETRA",
+            primaryExchange: "XETRA",
+            type: "EQUITY",
+            currency: "EUR",
+          },
+          {
+            providerId: "yahoo",
+            symbol: "APLY",
+            name: "Apple Yield Shares ETF",
+            exchange: "NYSE Arca",
+            type: "ETF",
+            currency: "USD",
+          },
+        ])}
+      />,
+      { width: 100, height: 24 },
+    );
+
+    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"));
+
+    expect(aaplRow).toBeGreaterThanOrEqual(0);
+    expect(apcRow).toBeGreaterThanOrEqual(0);
+    expect(aaplRow).toBeLessThan(apcRow);
+  });
+
   test("renders form-layout wizard fields together on one screen", async () => {
     testSetup = await testRender(
       <CommandBarHarness

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

@@ -7,6 +7,14 @@ 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";
+import { openUrl } from "../../ui/external-link";
 import { useRouteListState } from "../routing/list-state";
 import { useCommandBarRootRuntime } from "../routes/root/runtime";
 import { parseRootShortcutIntent } from "../routes/root/shortcuts";
@@ -247,6 +255,32 @@ 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 {
     activeMatch,
     orderedRootResults,
@@ -283,6 +317,7 @@ export function CommandBar({
     paneShortcutItems,
     pluginCommandItems,
     pluginCommandResultItems,
+    articleResultItems,
     readTickerSearchCache,
     rootModeKind: rootModeInfo.kind,
     rootQuery,

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

@@ -47,6 +47,25 @@ 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", () => {
+    const sections = buildSections([
+      { id: "assist:sign-up", category: "Ask AI", disabled: true, defaultSelectable: false },
+      { id: "holders", category: "Panes" },
+    ]);
+
+    expect(sections.map((section) => section.category)).toEqual(["Panes", "Ask AI"]);
+    expect(sections[0]?.items[0]?.id).toBe("holders");
+  });
+
+  test("keeps an answered AI section leading the list", () => {
+    const sections = buildSections([
+      { id: "assist:candidate:0", category: "Ask AI" },
+      { id: "holders", category: "Panes" },
+    ]);
+
+    expect(sections.map((section) => section.category)).toEqual(["Ask AI", "Panes"]);
+  });
+
   test("keeps non-exact ticker suggestions behind app sections in app-first order", () => {
     const sections = buildSections([
       { id: "pane", category: "Panes" },
@@ -67,6 +86,30 @@ describe("command bar view model helpers", () => {
     ]);
   });
 
+  test("orders ticker sections by their best-ranked candidate", () => {
+    const sections = buildSections([
+      { id: "primary", category: "Primary Listing" },
+      { id: "saved-alternate", category: "Saved" },
+      { id: "other", category: "Other Listings" },
+      { id: "saved-fallback", category: "Saved" },
+      { id: "fund", category: "Funds & Derivatives" },
+    ], { sectionOrder: "ranked" });
+
+    expect(sections.map((section) => section.category)).toEqual([
+      "Primary Listing",
+      "Saved",
+      "Other Listings",
+      "Funds & Derivatives",
+    ]);
+    expect(sections.flatMap((section) => section.items.map((item) => item.id))).toEqual([
+      "primary",
+      "saved-alternate",
+      "saved-fallback",
+      "other",
+      "fund",
+    ]);
+  });
+
   test("derives row presentation for toggles and current rows", () => {
     expect(getRowPresentation({
       id: "plugin:news",

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

@@ -4,6 +4,23 @@ import { truncateToDisplayWidth } from "../../utils/format";
 
 export { rankTickerSearchItems } from "../../tickers/search";
 
+/**
+ * Drop repeats of an id, keeping the first. The root list is assembled from
+ * overlapping sources (pane shortcuts, commands, plugin commands, ticker
+ * actions, layout items) in priority order, so the first hit is the one whose
+ * category and detail fit best.
+ */
+export function dedupeById<T extends { id: string }>(items: T[]): T[] {
+  const seen = new Set<string>();
+  const result: T[] = [];
+  for (const item of items) {
+    if (seen.has(item.id)) continue;
+    seen.add(item.id);
+    result.push(item);
+  }
+  return result;
+}
+
 export type CommandBarMode = "default" | "search" | "themes" | "plugins" | "layout" | "direct-command";
 
 export interface CommandBarModeInfo {
@@ -17,7 +34,7 @@ export interface CommandBarSection<T> {
   items: T[];
 }
 
-export type CommandBarSectionOrder = "default" | "app-first";
+export type CommandBarSectionOrder = "default" | "app-first" | "ranked";
 
 export interface CommandBarItemView {
   id: string;
@@ -66,7 +83,24 @@ export function resolveCommandBarMode(query: string, commandList?: Command[]): C
   }
 }
 
-export function buildSections<T extends { category: string }>(
+/**
+ * A section whose every row is disabled or unselectable is an offer, not an
+ * answer, so it sorts below real matches however its category is prioritized.
+ */
+const OFFER_SECTION_DEMOTION = 200;
+
+interface SectionSortableItem {
+  category: string;
+  disabled?: boolean;
+  defaultSelectable?: boolean;
+}
+
+function isOfferOnlySection<T extends SectionSortableItem>(items: T[]): boolean {
+  return items.length > 0
+    && items.every((item) => item.disabled === true || item.defaultSelectable === false);
+}
+
+export function buildSections<T extends SectionSortableItem>(
   items: T[],
   options?: { sectionOrder?: CommandBarSectionOrder },
 ): Array<CommandBarSection<T>> {
@@ -82,8 +116,10 @@ export function buildSections<T extends { category: string }>(
   return sections
     .map((section, index) => ({ section, index }))
     .sort((a, b) => {
-      const leftPriority = getCategoryPriority(a.section.category, options?.sectionOrder);
-      const rightPriority = getCategoryPriority(b.section.category, options?.sectionOrder);
+      const leftPriority = getCategoryPriority(a.section.category, options?.sectionOrder)
+        + (isOfferOnlySection(a.section.items) ? OFFER_SECTION_DEMOTION : 0);
+      const rightPriority = getCategoryPriority(b.section.category, options?.sectionOrder)
+        + (isOfferOnlySection(b.section.items) ? OFFER_SECTION_DEMOTION : 0);
       const priorityDiff = leftPriority - rightPriority;
       return priorityDiff !== 0 ? priorityDiff : a.index - b.index;
     })
@@ -138,6 +174,7 @@ export function truncateText(text: string, width: number): string {
 
 function getCategoryPriority(category: string, sectionOrder: CommandBarSectionOrder = "default"): number {
   const normalized = category.trim().toLowerCase();
+  if (sectionOrder === "ranked") return 0;
   // The AI answers the question the user actually typed, so it leads the list.
   if (normalized === "ask ai") return -100;
   if (normalized === "exact match") return -50;

package/src/components/command-bar/workflow/collection-actions.test.ts

@@ -0,0 +1,94 @@
+import { describe, expect, test } from "bun:test";
+import type { Dispatch } from "react";
+import { createInitialState, type AppAction, type AppState } from "../../../state/app/context";
+import { createDefaultConfig } from "../../../types/config";
+import type { DataProvider } from "../../../types/data-provider";
+import type { TickerRecord } from "../../../types/ticker";
+import { EventBus } from "../../../plugins/event-bus";
+import type { PluginRegistry } from "../../../plugins/registry";
+import { createCommandBarCollectionWorkflowActions } from "./collection-actions";
+
+function ticker(symbol: string, portfolios: string[] = []): TickerRecord {
+  return {
+    metadata: {
+      ticker: symbol,
+      exchange: "NASDAQ",
+      currency: "USD",
+      name: symbol,
+      portfolios,
+      watchlists: [],
+      positions: [],
+      custom: {},
+      tags: [],
+    },
+  };
+}
+
+function createHarness(initialTicker: TickerRecord) {
+  const state: AppState = {
+    ...createInitialState(createDefaultConfig(":memory:")),
+    tickers: new Map([[initialTicker.metadata.ticker, initialTicker]]),
+  };
+  const order: string[] = [];
+  const payloads: Array<{ symbol: string; portfolioId: string }> = [];
+  const events = new EventBus();
+  const pluginRegistry = { events } as PluginRegistry;
+  events.on("command-bar:portfolio-membership-persisted", (payload) => {
+    order.push("event");
+    payloads.push(payload);
+  });
+
+  const actions = createCommandBarCollectionWorkflowActions({
+    activeCollectionId: "main",
+    activeTickerSymbol: initialTicker.metadata.ticker,
+    dataProvider: {} as DataProvider,
+    dispatch: ((action: AppAction) => {
+      expect(action.type).toBe("UPDATE_TICKER");
+      order.push("dispatch");
+    }) as Dispatch<AppAction>,
+    getState: () => state,
+    notify: () => {},
+    persistConfig: () => {},
+    pluginRegistry,
+    setActiveCollection: () => {},
+    tickerRepository: {
+      loadAllTickers: async () => [],
+      loadTicker: async () => null,
+      saveTicker: async () => {
+        order.push("save:start");
+        await Promise.resolve();
+        order.push("save:end");
+      },
+      createTicker: async () => initialTicker,
+      deleteTicker: async () => {},
+    },
+  });
+
+  return { actions, order, payloads };
+}
+
+describe("command-bar collection workflow actions", () => {
+  test("emits portfolio membership after the ticker is saved and dispatched", async () => {
+    const { actions, order, payloads } = createHarness(ticker("AAPL"));
+
+    await actions.addTickerMembershipFromWorkflow({
+      portfolioId: "main",
+      ticker: "AAPL",
+    });
+
+    expect(order).toEqual(["save:start", "save:end", "dispatch", "event"]);
+    expect(payloads).toEqual([{ symbol: "AAPL", portfolioId: "main" }]);
+  });
+
+  test("emits portfolio membership when a resumed AP finds it already durable", async () => {
+    const { actions, order, payloads } = createHarness(ticker("AAPL", ["main"]));
+
+    await actions.addTickerMembershipFromWorkflow({
+      portfolioId: "main",
+      ticker: "AAPL",
+    });
+
+    expect(order).toEqual(["event"]);
+    expect(payloads).toEqual([{ symbol: "AAPL", portfolioId: "main" }]);
+  });
+});