gloomberb 0.18.2 → 0.18.4

package/src/components/command-bar/commands/collection-list.ts

@@ -1,6 +1,8 @@
 import type { AppState } from "../../../state/app/context";
 import type { TickerRecord } from "../../../types/ticker";
 import { splitSymbolArgs, splitSymbolWords, symbolInputKey, uniqueSymbols } from "../../../cli/symbol-args";
+import { protectFigureCommas } from "../../../cli/symbol-cells";
+import { readSymbolList } from "../../../cli/symbol-list";
 import { readNameEntries } from "../../../cli/name-entries";
 import {
   addSymbols,
@@ -42,17 +44,17 @@ export interface PastedSymbolList {
 }
 
 /**
- * The entries of text typed or pasted after `AW`, split as `gloomberb quote`
+ * The entries of text typed or pasted after `AW`, split as `gloom quote`
  * splits its arguments (`splitSymbolArgs`): on commas, semicolons, tabs and
  * lines when it has them, where an entry with spaces may be a company's name
- * or several symbols, else on spaces. Null for one word or none, which keep
- * the single add. Spaces alone also separate a company's name (`Palo Alto
- * Networks`), which is why `separated` says when the text left no doubt it is
- * a list.
+ * or several symbols, else on spaces. A decimal comma (`2.582,52`) splits
+ * nothing. Null for one word or none, which keep the single add. Spaces alone
+ * also separate a company's name (`Palo Alto Networks`), which is why
+ * `separated` says when the text left no doubt it is a list.
  */
 export function pastedSymbolList(raw: string | undefined): PastedSymbolList | null {
   if (!raw || splitSymbolWords([raw]).length < 2) return null;
-  return { ...uniqueSymbols(splitSymbolArgs([raw])), separated: /[,;\t\r\n]/.test(raw) };
+  return { ...uniqueSymbols(splitSymbolArgs([raw])), separated: /[,;\t\r\n]/.test(protectFigureCommas(raw)) };
 }
 
 /**
@@ -86,30 +88,45 @@ function reasonLine(row: SymbolRow): string | null {
   if (row.hint && row.status === "added") return row.hint;
   // Another company had the symbol: say which, beside the listing added.
   if (row.changeNote && row.status === "added") return row.changeNote;
+  // Listed only abroad: the venue and currency it was added on.
+  if (row.abroad && row.status === "added") return row.abroad;
   if (row.readAs && (row.status === "added" || row.status === "already_listed")) return row.readAs;
   // Why the match was left out, and the symbol it trades as now when there is one.
   if (row.status === "needs_choice") return row.message ?? typed;
+  // Not a symbol, or not looked up: the summary names them, and the notice says what to do about the second.
   if (row.status !== "failed" && !(row.status === "not_found" && row.message && !NO_MATCH.test(row.message))) return null;
   const text = reason(row.message ?? "", typed);
   return text.includes(typed) ? text : `${typed}: ${text}`;
 }
 
+/** Rows that did not join the watchlist and say why, which keep the report up longer. */
+const UNRESOLVED = new Set<SymbolRow["status"]>(["not_found", "needs_choice", "failed", "not_a_symbol", "not_looked_up"]);
+
 /**
  * What a list add says, in the words the CLI's summary uses: the counts, the
  * symbols that found nothing or were not added, and under it the reason for
- * each that has one. Skipped symbols keep the report up longer.
+ * each that has one, what the reader left out (the prices beside the
+ * symbols, in one line), and one line for the symbols Gloom Cloud did not
+ * answer for. Skipped symbols keep the report up longer.
  */
 function describeListAdd(
   rows: readonly SymbolRow[],
   watchlistName: string,
   repeated: number,
+  notes: readonly string[] = [],
 ): { body: string; type: "success" | "info" | "error"; duration?: number } {
   const summary = `Watchlist "${watchlistName}": ${summarizeSymbolRows(rows, "add", repeated)}`;
   const reasons = rows.map(reasonLine).filter((line): line is string => !!line);
+  const unanswered = rows.filter((row) => row.status === "not_looked_up").length;
   const added = rows.some((row) => row.status === "added");
-  const unresolved = rows.some((row) => row.status === "not_found" || row.status === "needs_choice" || row.status === "failed");
+  const unresolved = rows.some((row) => UNRESOLVED.has(row.status));
   return {
-    body: [summary, ...reasons].join("\n"),
+    body: [
+      summary,
+      ...reasons,
+      ...notes,
+      ...unanswered > 0 ? [`Gloom Cloud did not answer for ${unanswered === 1 ? "one symbol" : `${unanswered} symbols`}; add ${unanswered === 1 ? "it" : "them"} again in a minute.`] : [],
+    ].join("\n"),
     type: added && !unresolved ? "success" : unresolved && !added ? "error" : "info",
     duration: unresolved ? SKIPPED_REPORT_MS : undefined,
   };
@@ -118,12 +135,14 @@ function describeListAdd(
 /**
  * A symbol typed to add on its own that a pasted list would leave for a
  * choice (`typedSymbolChoice`: its company moved or merged away, the match is
- * abroad or stopped trading) is not added either. The notice says why, and
- * what to add instead or to keep the match. Resolves true when it said so.
+ * abroad while the symbol names a US listing, or it stopped trading) is not
+ * added either. The notice says why, and what to add instead or to keep the
+ * match; `refused` says it did. One listed only abroad is added, and `abroad`
+ * is the line naming its venue and currency, for the add to say.
  */
-export async function refuseTypedChoice(rawInput: string, deps: SharedWorkflowDeps, notify: Notify): Promise<boolean> {
+export async function refuseTypedChoice(rawInput: string, deps: SharedWorkflowDeps, notify: Notify): Promise<{ refused: boolean; abroad: string | null }> {
   const row = await typedSymbolChoice(rawInput, deps.tickerRepository, deps.dataProvider);
-  if (!row) return false;
+  if (row?.status !== "needs_choice") return { refused: false, abroad: row?.abroad ?? null };
   const message = row.message ?? symbolInputKey(row.input);
   const offers = [
     ...row.tradesAs ? [`${row.tradesAs} for what it trades as now`] : [],
@@ -131,12 +150,12 @@ export async function refuseTypedChoice(rawInput: string, deps: SharedWorkflowDe
   ];
   notify([/[.!?]$/.test(message) ? message : `${message}.`, ...offers.length > 0 ? [`Add ${offers.join(", or ")}.`] : []].join("\n"),
     { type: "error", duration: SKIPPED_REPORT_MS });
-  return true;
+  return { refused: true, abroad: null };
 }
 
 /**
  * Adds every symbol of a pasted list to a watchlist in one step. Each symbol
- * resolves as `gloomberb watchlist add` resolves it (Gloom Cloud search and
+ * resolves as `gloom watchlist add` resolves it (Gloom Cloud search and
  * quote, the venue its own quote is on), except that a symbol several listings
  * carry is never given the first: it is left out and named, for the single add
  * to ask which. One that is unknown or already listed does not stop the rest.
@@ -159,10 +178,13 @@ export async function addPastedListToWatchlist(options: {
   const deps = options.buildWorkflowDeps();
   // Spaces alone also separate the words of a name, which belongs to search.
   if (!list.separated && await namesOneCompany(list.symbols, deps.dataProvider)) return false;
-  // An entry with spaces is a company's name search finds, or the symbols it holds.
-  const read = uniqueSymbols(await readNameEntries(list.symbols, deps.dataProvider));
+  // Read as `watchlist add` reads a list: an entry with spaces is a company's name search finds, or the symbols it
+  // holds, and a price beside a symbol (`ED 2.582,52`) is left out and named.
+  const read = await readSymbolList({
+    args: [options.rawInput], stdinPiped: false, readEntries: (entries) => readNameEntries(entries, deps.dataProvider),
+  });
   const symbols = read.symbols;
-  const repeated = list.repeated + read.repeated;
+  const repeated = read.repeated;
   const state = options.getState();
   const known = options.explicitTargetId && state.config.watchlists.some((entry) => entry.id === options.explicitTargetId)
     ? options.explicitTargetId
@@ -233,7 +255,7 @@ export async function addPastedListToWatchlist(options: {
       recordAttention(publicTickerKey(ticker.metadata.ticker, ticker.metadata.exchange), "watchlist_add");
     }
   }
-  const report = describeListAdd(rows, watchlistName, repeated);
+  const report = describeListAdd(rows, watchlistName, repeated, read.notes);
   notify(report.body, { type: report.type, duration: report.duration });
   options.closeAll({ revertThemePreview: false });
   return true;

package/src/components/command-bar/commands/collection.ts

@@ -62,8 +62,10 @@ export async function executeCollectionCommandAction(options: {
   });
   if (list?.separated && await addList(list)) return;
   // One typed symbol is decided as a symbol of a pasted list is: a match the list would leave for a choice is not added.
-  if (action === "add" && !options.selectedTicker && !list && options.rawInput?.trim()
-    && await refuseTypedChoice(options.rawInput, deps, options.notify)) {
+  const typed = action === "add" && !options.selectedTicker && !list && options.rawInput?.trim()
+    ? await refuseTypedChoice(options.rawInput, deps, options.notify)
+    : null;
+  if (typed?.refused) {
     options.closeAll({ revertThemePreview: false });
     return;
   }
@@ -246,7 +248,11 @@ export async function executeCollectionCommandAction(options: {
       `${action === "add" ? "Added" : "Removed"} ${resolvedTicker.symbol} ${action === "add" ? "to" : "from"} "${targetName}".`,
       { type: "success" },
     );
-    if (action === "add") notifyTickerChange(deps.dataProvider, resolvedTicker.ticker, options.notify);
+    if (action === "add") {
+      // Listed only abroad: the venue and currency it went to, as `watchlist add` says it.
+      if (typed?.abroad) options.notify(typed.abroad, { type: "info" });
+      notifyTickerChange(deps.dataProvider, resolvedTicker.ticker, options.notify);
+    }
   } else {
     options.notify(
       action === "add"

package/src/components/command-bar/commands/direct/index.ts

@@ -220,7 +220,7 @@ export function runDirectCommandAction(options: {
         title: "Reset All Data",
         body: [
           "This will permanently delete all portfolios, tickers, notes, broker credentials, and settings.",
-          "Gloomberb will quit and show the setup wizard on next launch.",
+          "Gloom will quit and show the setup wizard on next launch.",
         ],
         confirmLabel: "Reset Everything",
         cancelLabel: "Back",

package/src/components/command-bar/commands/direct/runtime.ts

@@ -81,12 +81,12 @@ export function useCommandBarDirectCommandRuntime({
   // A confirm it opens (Reset All Data) runs after the bar closed.
   const getState = useAppGetState();
   const rendererHost = useRendererHost();
-  const runSecurityDescriptionShortcut = useCallback(async (query?: string) => {
+  const runSecurityDescriptionShortcut = useCallback(async (query?: string, options?: { forceNewPane?: boolean }) => {
     const trimmed = query?.trim() || "";
     if (!trimmed) {
       const inferred = normalizeTickerInput(activeTickerSymbol, query);
       if (inferred) {
-        focusTicker(inferred);
+        focusTicker(inferred, options);
         closeAll({ revertThemePreview: false });
         return;
       }
@@ -103,6 +103,7 @@ export function useCommandBarDirectCommandRuntime({
     );
     if (resolvedTicker) {
       focusTicker(resolvedTicker.symbol, {
+        ...options,
         instrument: resolvedTicker.instrument,
         listing: resolvedTicker.listing,
       });

package/src/components/command-bar/pane-templates/items.ts

@@ -3,6 +3,7 @@ import type {
   PaneTemplateContext,
   PaneTemplateCreateOptions,
   PaneTemplateDef,
+  ShortcutArgError,
 } from "../../../types/plugin";
 import { fuzzyFilter } from "../../../utils/fuzzy-search";
 import { summarizeError } from "../helpers";
@@ -38,6 +39,16 @@ export function recentPaneTemplateArg(template: PaneTemplateDef, arg: string | u
   return arg?.trim().toUpperCase() || undefined;
 }
 
+/** Why a typed argument opens nothing for the template (`PF Nope`), or null. */
+export function paneTemplateArgError(
+  template: PaneTemplateDef,
+  arg: string | undefined,
+  context: PaneTemplateContext,
+): ShortcutArgError | null {
+  const trimmed = arg?.trim();
+  return trimmed ? template.shortcut?.argError?.(trimmed, context) ?? null : null;
+}
+
 export function canPromptForPaneTemplateArg(template: PaneTemplateDef): boolean {
   const argKind = getPaneTemplateArgKind(template);
   return argKind === "ticker"
@@ -95,6 +106,22 @@ export function getAvailablePaneTemplatesForState(options: {
     });
 }
 
+/**
+ * "ETF flows" spells out ETFF (Crypto ETF Flows), not ETF for a ticker called
+ * FLOWS: a query that is word for word a keyword of a template whose own code
+ * extends the typed one is no ticker argument, and is left to the search,
+ * which ranks that template first. "JOBS DAY" stays Jobs for Dayforce, since
+ * the template that lists "jobs day" (MDAY) does not extend JOBS.
+ */
+function spellsOutLongerCode(query: string, code: string, templates: readonly PaneTemplateDef[]): boolean {
+  const phrase = query.toLowerCase().replace(/\s+/g, " ");
+  return templates.some((template) => {
+    const longer = template.shortcut?.prefix?.trim().toUpperCase() ?? "";
+    return longer.length > code.length && longer.startsWith(code)
+      && !!template.keywords?.some((keyword) => keyword.toLowerCase() === phrase);
+  });
+}
+
 export function getAvailablePaneShortcutTemplatesForQuery(options: {
   pluginRegistry: PluginRegistry;
   disabledPlugins: readonly string[];
@@ -105,9 +132,11 @@ export function getAvailablePaneShortcutTemplatesForQuery(options: {
   const trimmed = options.query.trim();
   const upper = trimmed.toUpperCase();
   const disabledPluginIds = new Set(options.disabledPlugins);
-  return [...options.pluginRegistry.paneTemplates.values()].filter((template) => {
+  const enabled = [...options.pluginRegistry.paneTemplates.values()].filter((template) => {
     const pluginId = options.pluginRegistry.getPaneTemplatePluginId(template.id);
-    if (pluginId && disabledPluginIds.has(pluginId)) return false;
+    return !pluginId || !disabledPluginIds.has(pluginId);
+  });
+  return enabled.filter((template) => {
     const argKind = getPaneTemplateArgKind(template);
     // The longest matching mnemonic wins, so an alias sharing a stem cannot shadow the typed one.
     const prefix = paneTemplateShortcutPrefixes(template).map((value) => value.toUpperCase())
@@ -115,14 +144,16 @@ export function getAvailablePaneShortcutTemplatesForQuery(options: {
       .sort((a, b) => b.length - a.length)[0];
     if (!prefix) return false;
     const arg = trimmed.slice(prefix.length).trim();
+    if (argKind === "ticker" && arg && spellsOutLongerCode(trimmed, prefix, enabled)) return false;
     if (!template.canCreate) return true;
     try {
       const canCreate = template.canCreate(options.context, arg ? { arg } : undefined);
       if (canCreate) return true;
-      if (!arg && !options.context.activeTicker && canPromptForPaneTemplateArg(template)) {
-        return true;
-      }
-      return false;
+      // A code typed with an argument it cannot open (`CMP AAPL` compares two
+      // or more) is still that function: its row opens the form or says why,
+      // where dropping it would leave the text to search and Ask AI.
+      if (arg) return canPromptForPaneTemplateArg(template);
+      return !options.context.activeTicker && canPromptForPaneTemplateArg(template);
     } catch (error) {
       options.logError?.("Pane shortcut canCreate failed", {
         templateId: template.id,
@@ -141,6 +172,8 @@ export function buildPaneTemplateItem(options: {
   createOptions?: PaneTemplateCreateOptions;
   showShortcut?: boolean;
   shortcutExecution?: boolean;
+  /** Why `createOptions.arg` opens nothing; the row then says so instead of opening the pane. */
+  argError?: ShortcutArgError | null;
   runPaneTemplateShortcut: (template: PaneTemplateDef, rawArg?: string) => void;
   shouldOpenTemplateConfig: (template: PaneTemplateDef, arg?: string) => boolean;
   openPaneTemplateWorkflow: (template: PaneTemplateDef, options?: { arg?: string }) => void;
@@ -169,6 +202,26 @@ export function buildPaneTemplateItem(options: {
     pluginName || "",
   ].filter(Boolean).join(" ");
 
+  if (options.argError) {
+    return {
+      id: `pane-template-arg-error:${options.template.id}:${arg || ""}`,
+      label: options.argError.message,
+      detail: options.argError.detail ?? "",
+      // Under the label, where the bar shows a row's second line: what to type instead.
+      ...(options.argError.detail ? { lines: [{ segments: [{ text: options.argError.detail }] }] } : {}),
+      category: options.category ?? (pluginName ? `${pluginName} Panes` : "Panes"),
+      // The function's own row, disabled: its code sits in the code column as on
+      // the row that opens it, where an info row would carry it on the right.
+      kind: "action",
+      right: options.showShortcut ? options.template.shortcut?.prefix : undefined,
+      shortcutQuery: options.template.shortcut?.prefix,
+      searchText,
+      defaultSelectable: false,
+      disabled: true,
+      action: () => {},
+    };
+  }
+
   const action = () => {
     if (
       options.template.shortcut

package/src/components/command-bar/pane-templates/workflow.ts

@@ -6,6 +6,7 @@ import type {
   PaneTemplateDef,
 } from "../../../types/plugin";
 import { normalizeTickerInput } from "../../../tickers/search";
+import { commandPromptTicker } from "../../../layout/selected-ticker";
 import { debugLog } from "../../../utils/debug-log";
 import {
   isCollectionCommand,
@@ -20,6 +21,7 @@ import {
   getAvailablePaneShortcutTemplatesForQuery,
   getAvailablePaneTemplatesForState,
   getPaneTemplateDisplayLabel,
+  paneTemplateArgError,
   recentPaneTemplateArg,
 } from "./items";
 import type { FormModalRequest } from "../../form-modal";
@@ -126,13 +128,18 @@ export function useCommandBarPaneTemplateActions({
   ) => {
     const trimmedArg = rawArg?.trim() || "";
     const argKind = template.shortcut?.argKind ?? template.shortcut?.argPlaceholder;
+    // Required ticker panes use the instrument already in the command prompt
+    // when nothing was typed. Optional panes stay unbound.
+    const promptSymbol = template.shortcut?.argOptional
+      ? null
+      : commandPromptTicker(buildWorkflowDeps().getState())?.symbol ?? null;
     if (argKind === "ticker") {
       if (!trimmedArg && template.shortcut?.argOptional) {
         await openPaneTemplateDirect(template);
         return;
       }
       const resolvedTicker = await resolveTickerInput(
-        trimmedArg || undefined,
+        trimmedArg || promptSymbol || undefined,
         activeTickerSymbol,
         activeCollectionId,
         buildWorkflowDeps(),
@@ -285,6 +292,7 @@ export function useCommandBarPaneTemplateActions({
       shortcutExecution?: boolean;
     },
   ): ResultItem => buildPaneTemplateItem({
+    argError: paneTemplateArgError(template, options?.createOptions?.arg, getPaneTemplateContext()),
     category: options?.category,
     createOptions: options?.createOptions,
     openPaneTemplateDirect,
@@ -296,6 +304,7 @@ export function useCommandBarPaneTemplateActions({
     showShortcut: options?.showShortcut,
     template,
   }), [
+    getPaneTemplateContext,
     openPaneTemplateDirect,
     openPaneTemplateWorkflow,
     pluginRegistry,

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

@@ -3,11 +3,26 @@ import type { TickerRecord } from "../../../../types/ticker";
 import { isManualPortfolio } from "../../../../portfolio/mutations";
 import type { Command } from "../../commands/registry";
 import type { ResultItem } from "../../list/model";
-import { canRestartToApply } from "../../../../updater";
+import { canRestartToApply, type UpdateProgress } from "../../../../updater";
 import { NIGHTLY_CHANNEL_SUMMARY, resolveUpdateChannel } from "../../../../updater/channel";
+import { updateFailureReason } from "../../../../updater/failure";
+import { t, tf } from "../../../../i18n";
 import { isPrereleaseVersion } from "../../../../utils/semver";
 import { VERSION } from "../../../../version";
 
+/** "Update failed: download interrupted", as the header says it. */
+function failedUpdateLabel(progress: UpdateProgress): string {
+  const reason = progress.failure ? t(updateFailureReason(progress.failure)) : (progress.error ?? "").replace(/\s+/g, " ").trim();
+  return tf("Update failed: {error}", { error: reason });
+}
+
+/** The whole error under the failed update's row, unless the label already says all of it. */
+function failedUpdateLines(progress: UpdateProgress): ResultItem["lines"] {
+  const error = (progress.error ?? "").replace(/\s+/g, " ").trim();
+  if (!progress.failure || !error) return undefined;
+  return [{ segments: [{ text: error, emphasis: "muted" }] }];
+}
+
 interface RootCommandItemBuilderOptions {
   activeCollectionId: string | null;
   activeTickerData: TickerRecord | null | undefined;
@@ -88,9 +103,12 @@ export function createRootCommandItemBuilder({
         return activeTickerSymbol ? `Remove ${activeTickerSymbol} from Portfolio` : command.label;
       case "set-portfolio-position":
         return activeTickerSymbol ? `Set Position for ${activeTickerSymbol}` : command.label;
-      // The row only shows its label, so a downloaded update is announced there.
+      // The row only shows its label, so a downloaded update is announced there,
+      // and a failed one with its reason; Enter restarts into the one and retries the other.
       case "check-for-updates":
-        return state.updateProgress?.phase === "ready" ? "Update ready, restart to apply" : command.label;
+        if (state.updateProgress?.phase === "ready") return "Update ready, restart to apply";
+        if (state.updateProgress?.phase === "error") return failedUpdateLabel(state.updateProgress);
+        return command.label;
       default:
         return command.label;
     }
@@ -140,7 +158,7 @@ export function createRootCommandItemBuilder({
           return state.updateProgress.message ?? "Update ready, restart to apply";
         }
         if (state.updateProgress?.phase === "error") {
-          return `Update failed: ${state.updateProgress.error}`;
+          return `${failedUpdateLabel(state.updateProgress)}. ${state.updateProgress.error ?? ""}`.trim();
         }
         if (state.updateCheckInProgress) return "Checking releases now";
         if (state.updateAvailable) {
@@ -198,8 +216,11 @@ export function createRootCommandItemBuilder({
       // "update" is a whole word here so it lands on this command before Set
       // Portfolio Position (whose keywords say "update position") and Release
       // Channel, which keeps the word further back.
+      // Ready and failed rename the row; its own name still finds it.
       case "check-for-updates":
-        return state.updateProgress?.phase === "ready" ? "update check for updates restart relaunch" : "update";
+        if (state.updateProgress?.phase === "ready") return "update check for updates restart relaunch";
+        if (state.updateProgress?.phase === "error") return "update check for updates retry";
+        return "update";
       case "release-channel":
         return "nightly stable beta prerelease update updates channel";
       default:
@@ -218,9 +239,14 @@ export function createRootCommandItemBuilder({
       right: smartRight(command, arg),
       shortcutQuery: command.prefix || undefined,
       searchText: smartSearchText(command),
+      ...(command.id === "check-for-updates" && state.updateProgress?.phase === "error"
+        ? { lines: failedUpdateLines(state.updateProgress) }
+        : {}),
+      // Busy while it checks, downloads or installs; a failed update is retried from here.
       disabled:
         command.id === "check-for-updates" &&
-        (state.updateCheckInProgress || (!!state.updateProgress && !canRestartToApply(state.updateProgress))),
+        (state.updateCheckInProgress
+          || (!!state.updateProgress && !canRestartToApply(state.updateProgress) && state.updateProgress.phase !== "error")),
       action: () => runDirectCommand(command, arg),
     };
   };

package/src/components/command-bar/routes/root/plugin-install.ts

@@ -195,7 +195,7 @@ function buildPluginInstallItem(
   const actionLine = !installable
     ? `Get it in ${whereItRuns(plugin)}: ${pageUrl.replace(/^https:\/\//, "")}`
     : required
-      ? `Needs Gloomberb ${required}. Update Gloomberb first.`
+      ? `Needs Gloom ${required}. Update Gloom first.`
       : `Install ${plugin.name}`;
 
   const install = () => {

package/src/components/command-bar/routes/root/provider-search.ts

@@ -9,11 +9,34 @@ import {
   QUICK_LOOK_TICKER_SEARCH_OPTIONS,
   tickerSearchFailureItems,
   venueDropdownResults,
+  type RootTickerReading,
 } from "../ticker-search/results";
+import { ProviderBusyError, SEARCH_BUSY_MESSAGE } from "../../../../sources/provider-errors";
+import { OperationTimeoutError, withDeadline } from "../../../../utils/async-deadline";
 import { orderListResults, type ResultItem } from "../../list/model";
 import type { CommandBarCategoryPriorities, CommandBarSectionOrder } from "../../view-model";
 import type { CommandBarRoute } from "../../workflow/types";
 
+/**
+ * How long the root symbol search may take. Gloom Cloud gives up after ten
+ * seconds, so a search still going at this point is stuck on something else (a
+ * broker that never answers) and the bar stops searching for it.
+ */
+const ROOT_SEARCH_TIMEOUT_MS = 15_000;
+
+/**
+ * The search's answer, or a busy service once `timeoutMs` passes: the same row a
+ * rate-limited search gets, since nothing was found and a retry may find it.
+ */
+export async function searchWithinDeadline<T>(search: Promise<T>, timeoutMs = ROOT_SEARCH_TIMEOUT_MS): Promise<T> {
+  try {
+    return await withDeadline(search, timeoutMs, "Symbol search timed out");
+  } catch (error) {
+    if (error instanceof OperationTimeoutError) throw new ProviderBusyError(SEARCH_BUSY_MESSAGE, { cause: error });
+    throw error;
+  }
+}
+
 export function useRootProviderSearch(options: {
   activeCollectionId: string | null;
   buildTickerSearchResultItems: (candidates: TickerSearchCandidate[], query: string) => ResultItem[];
@@ -30,6 +53,8 @@ export function useRootProviderSearch(options: {
   rootPlainTickerSearchArg: string | null;
   /** Set when the query is `NET:` or `NET:N`. Empty string lists every venue. */
   listingSuffix: string | null;
+  /** What a plain query written as a terminal's code or a near miss reads as (`VOD LN`, `700.HK`). */
+  reading?: RootTickerReading | null;
   rootResultItems: ResultItem[];
   rootTickerSearchArg: string | null;
   tickers: AppState["tickers"];
@@ -58,6 +83,7 @@ export function useRootProviderSearch(options: {
     readTickerSearchCache,
     rootPlainTickerSearchArg,
     listingSuffix,
+    reading = null,
     rootResultItems,
     rootTickerSearchArg,
     tickers,
@@ -116,16 +142,18 @@ export function useRootProviderSearch(options: {
 
     const requestId = ++rootSearchRequestIdRef.current;
     rootSearchTimerRef.current = setTimeout(async () => {
+      let published = false;
+      const publish = (candidates: TickerSearchCandidate[]) => {
+        published = true;
+        setRootProviderResults(mergeTickerSearchResultItems(
+          searchQuery,
+          buildTickerSearchResultItems(candidates, searchQuery),
+          localTickerSearchResultItems(searchQuery, { limit: 6 }),
+        ));
+        setRootProviderResultsQuery(searchQuery);
+      };
       try {
-        const publish = (candidates: TickerSearchCandidate[]) => {
-          setRootProviderResults(mergeTickerSearchResultItems(
-            searchQuery,
-            buildTickerSearchResultItems(candidates, searchQuery),
-            localTickerSearchResultItems(searchQuery, { limit: 6 }),
-          ));
-          setRootProviderResultsQuery(searchQuery);
-        };
-        const combined = await searchTickerCandidates({
+        const combined = await searchWithinDeadline(searchTickerCandidates({
           query: searchQuery,
           tickers,
           dataProvider,
@@ -145,7 +173,7 @@ export function useRootProviderSearch(options: {
           // The rows the CLI reads a symbol's listings from, so `CAR:` offers CAR Group on ASX (its eleventh).
           totalLimit: LISTING_SEARCH_LIMIT,
           ...QUICK_LOOK_TICKER_SEARCH_OPTIONS,
-        });
+        }));
         if (requestId !== rootSearchRequestIdRef.current) return;
         writeTickerSearchCache(
           searchQuery,
@@ -156,6 +184,8 @@ export function useRootProviderSearch(options: {
         publish(combined);
       } catch (error) {
         if (requestId !== rootSearchRequestIdRef.current) return;
+        // Rows a faster source already put up are the answer a search that went quiet leaves.
+        if (published && error instanceof ProviderBusyError && error.cause instanceof OperationTimeoutError) return;
         setRootProviderResults(tickerSearchFailureItems(error, searchQuery, localTickerSearchResultItems(searchQuery, { limit: 6 })));
         setRootProviderResultsQuery(searchQuery);
       } finally {
@@ -191,13 +221,14 @@ export function useRootProviderSearch(options: {
     }
     if (rootTickerSearchArg && rootProviderResultsQuery === rootTickerSearchArg && rootProviderResults) {
       if (rootPlainTickerSearchArg) {
-        return mergePlainRootTickerResults(rootPlainTickerSearchArg, rootProviderResults, rootResultItems);
+        return mergePlainRootTickerResults(rootPlainTickerSearchArg, rootProviderResults, rootResultItems, reading);
       }
       return rootProviderResults;
     }
     return rootResultItems;
   }, [
     listingSuffix,
+    reading,
     rootPlainTickerSearchArg,
     rootProviderResults,
     rootProviderResultsQuery,

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

@@ -8,6 +8,7 @@ import type { TickerRecord } from "../../../../types/ticker";
 import { fuzzyFilter } from "../../../../utils/fuzzy-search";
 import {
   buildAssistResultItems,
+  isQuestionLike,
   shouldShowAssistRow,
   type AssistRowHandlers,
 } from "../../assist/model";
@@ -66,6 +67,8 @@ export interface RootResultModelOptions {
     rawInput?: string,
   ) => void | Promise<void>;
   getAvailablePaneShortcutTemplates: (query: string) => PaneTemplateDef[];
+  /** The function code the query starts with, claimed by the parser or not (`leadingShortcutCode`). */
+  leadingShortcutCode?: string | null;
   /** Looks a recorded `pane-template:<id>` entry back up among the templates the bar offers now. */
   getRecentPaneTemplate?: (id: string, arg?: string) => PaneTemplateDef | undefined;
   hasPaneSettings: (paneId: string) => boolean;
@@ -106,15 +109,21 @@ function isAssistSectionVisible(
   query: string,
   resultCount: number,
   hasShortcutIntent: boolean,
+  leadingCode: string | null,
 ): 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 });
+  if (!assist.enabled) {
+    // Text led by a function code (`MOST foo`) is a command gone wrong, not a
+    // question, so the sign-up offer would be the only row and take Enter.
+    if (leadingCode && !isQuestionLike(query)) 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.
+    return shouldShowAssistRow({ query, resultCount });
+  }
   return assist.auto;
 }
 
@@ -235,6 +244,7 @@ export function buildRootResultModel(options: RootResultModelOptions): RootResul
     getAvailablePaneShortcutTemplates,
     getRecentPaneTemplate,
     hasPaneSettings,
+    leadingShortcutCode = null,
     localTickerSearchResultItems,
     nonShortcutPaneTemplateItems,
     openModeRoute,
@@ -268,6 +278,8 @@ export function buildRootResultModel(options: RootResultModelOptions): RootResul
   const items: ResultItem[] = [];
   const match = matchPrefix(rootQuery, availableCommands);
   let initialIdx = 0;
+  // Off when the code's own rows only say why the argument opens nothing (`PF Nope`).
+  let shortcutRuns = true;
   const shortcutItem = buildRootShortcutItem({
     activeCollectionId,
     activeTickerSymbol,
@@ -289,6 +301,7 @@ export function buildRootResultModel(options: RootResultModelOptions): RootResul
         shortcutExecution: true,
       }))
       : [shortcutItem];
+    shortcutRuns = templateItems.some((item) => !item.disabled);
     const relatedTemplateItems = rootQuery.trim().toUpperCase() === rootShortcutIntent.prefix
       ? paneShortcutItems({
         filterQuery: rootQuery,
@@ -393,7 +406,7 @@ export function buildRootResultModel(options: RootResultModelOptions): RootResul
   // Text the parser can run is text a key can run, so the offer sits under the
   // match it would replay. Never the default selection: Enter still runs the
   // command itself.
-  if (bindKey && shortcutClaimedQuery && rootShortcutIntent.kind !== "ambiguous") {
+  if (bindKey && shortcutClaimedQuery && shortcutRuns && rootShortcutIntent.kind !== "ambiguous") {
     items.push(buildBindKeyItem(rootShortcutIntent, bindKey));
   }
   // A resolved prefix means the user is speaking the command language, so
@@ -414,6 +427,7 @@ export function buildRootResultModel(options: RootResultModelOptions): RootResul
       rootQuery,
       matchCount,
       shortcutClaimedQuery,
+      leadingShortcutCode,
     )
     ? buildAssistResultItems({ ...assist, query: rootQuery })
     : [];