gloomberb 0.15.2 → 0.15.4

package/src/plugins/builtin/research/events-headless.ts

@@ -8,19 +8,8 @@ import type {
   HeadlessPaneDefinition,
   HeadlessPaneLoadArgs,
 } from "../../../types/plugin";
-import { formatEventMetric, buildEventRows, eventSourceNotice, CORPORATE_ACTION_COVERAGE } from "./event-model";
-
-const COLUMNS = [
-  { key: "date", header: "Date" },
-  { key: "status", header: "Event" },
-  { key: "period", header: "Period" },
-  { key: "qEps", header: "Q EPS", align: "right" as const, format: (value: unknown, row: Record<string, unknown>) => formatEventMetric(value == null ? undefined : Number(value), typeof row.epsCurrency === "string" ? row.epsCurrency : undefined, "eps") },
-  { key: "qRevenue", header: "Q revenue", align: "right" as const, format: (value: unknown, row: Record<string, unknown>) => formatEventMetric(value == null ? undefined : Number(value), typeof row.revenueCurrency === "string" ? row.revenueCurrency : undefined, "revenue") },
-  { key: "annualEps", header: "Annual EPS", align: "right" as const, format: (value: unknown, row: Record<string, unknown>) => formatEventMetric(value == null ? undefined : Number(value), typeof row.epsCurrency === "string" ? row.epsCurrency : undefined, "eps") },
-  { key: "annualRevenue", header: "Annual revenue", align: "right" as const, format: (value: unknown, row: Record<string, unknown>) => formatEventMetric(value == null ? undefined : Number(value), typeof row.revenueCurrency === "string" ? row.revenueCurrency : undefined, "revenue") },
-  { key: "value", header: "Value", align: "right" as const },
-  { key: "detail", header: "Detail" },
-];
+import { buildEventRows, eventSourceNotice, CORPORATE_ACTION_COVERAGE } from "./event-model";
+import { EVENT_COLUMNS, loadEventSources } from "./event-sources";
 
 export interface EventHeadlessData {
   actions: CorporateActionsData | null;
@@ -42,25 +31,11 @@ export interface EventsHeadlessDependencies {
 
 const defaultDependencies: EventsHeadlessDependencies = {
   async load(_args, symbol, provider) {
-    const [actions, estimates, financials] = await Promise.allSettled([
-      provider.getCorporateActions ? provider.getCorporateActions(symbol, "") : Promise.reject(new Error("Corporate actions source unavailable")),
-      provider.getAnalystResearch
-        ? provider.getAnalystResearch(symbol, "")
-        : Promise.reject(new Error("Analyst estimates source unavailable")),
-      provider.getTickerFinancials(symbol, ""),
-    ]);
-    const actionsData = actions.status === "fulfilled" ? actions.value : null;
-    const estimatesData = estimates.status === "fulfilled" ? estimates.value : null;
-    const financialsData = financials.status === "fulfilled" ? financials.value : null;
-    const error = (result: PromiseSettledResult<unknown>) => result.status === "rejected" ? String(result.reason instanceof Error ? result.reason.message : result.reason) : null;
+    const sources = await loadEventSources(provider, symbol);
     return {
-      actions: actionsData,
-      estimates: estimatesData,
-      financials: financialsData,
-      actionsError: error(actions),
-      estimatesError: error(estimates),
-      financialsError: error(financials),
-      currency: actionsData?.currency ?? estimatesData?.currency ?? financialsData?.quote?.currency ?? "USD",
+      ...sources,
+      actionsError: provider.getCorporateActions ? sources.actionsError : "Corporate actions source unavailable",
+      estimatesError: provider.getAnalystResearch ? sources.estimatesError : "Analyst estimates source unavailable",
     };
   },
 };
@@ -76,7 +51,7 @@ export function createEventsHeadless(
       description: "Ticker whose corporate actions and estimates should be returned.",
     },
     options: [],
-    columns: COLUMNS,
+    columns: EVENT_COLUMNS,
     describe: (args) => `Corporate Actions | ${String(args.argument)}`,
     async load(args, ctx) {
       const symbol = args.symbols[0]!;

package/src/plugins/builtin/research/headless.ts

@@ -9,19 +9,8 @@ import type {
   HeadlessPaneLoadArgs,
   HeadlessRowsResult,
 } from "../../../types/plugin";
-import { formatEventMetric, buildEventRows, eventSourceNotice, type EventRow } from "./event-model";
-
-const ESTIMATE_COLUMNS = [
-  { key: "date", header: "Date" },
-  { key: "status", header: "Event" },
-  { key: "period", header: "Period" },
-  { key: "qEps", header: "Q EPS", align: "right" as const, format: (value: unknown, row: Record<string, unknown>) => formatEventMetric(value == null ? undefined : Number(value), typeof row.epsCurrency === "string" ? row.epsCurrency : undefined, "eps") },
-  { key: "qRevenue", header: "Q revenue", align: "right" as const, format: (value: unknown, row: Record<string, unknown>) => formatEventMetric(value == null ? undefined : Number(value), typeof row.revenueCurrency === "string" ? row.revenueCurrency : undefined, "revenue") },
-  { key: "annualEps", header: "Annual EPS", align: "right" as const, format: (value: unknown, row: Record<string, unknown>) => formatEventMetric(value == null ? undefined : Number(value), typeof row.epsCurrency === "string" ? row.epsCurrency : undefined, "eps") },
-  { key: "annualRevenue", header: "Annual revenue", align: "right" as const, format: (value: unknown, row: Record<string, unknown>) => formatEventMetric(value == null ? undefined : Number(value), typeof row.revenueCurrency === "string" ? row.revenueCurrency : undefined, "revenue") },
-  { key: "value", header: "Value", align: "right" as const },
-  { key: "detail", header: "Detail" },
-];
+import { buildEventRows, eventSourceNotice, type EventRow } from "./event-model";
+import { EVENT_COLUMNS, loadEventSources } from "./event-sources";
 
 export interface EarningsEstimateSources {
   actions: CorporateActionsData | null;
@@ -38,47 +27,18 @@ export interface EarningsEstimatesHeadlessDependencies {
   ): Promise<EarningsEstimateSources>;
 }
 
-async function settledValue<T>(
-  promise: Promise<T> | null,
-  errors: string[],
-  label: string,
-): Promise<T | null> {
-  if (!promise) return null;
-  try {
-    return await promise;
-  } catch (error) {
-    errors.push(`${label}: ${error instanceof Error ? error.message : String(error)}`);
-    return null;
-  }
-}
-
 const defaultDependencies: EarningsEstimatesHeadlessDependencies = {
   async loadSources(symbol, context) {
-    const errors: string[] = [];
-    const provider = context.marketData;
-    const [actions, estimates, financials] = await Promise.all([
-      settledValue(
-        provider.getCorporateActions?.(symbol, "") ?? null,
-        errors,
-        "corporate actions",
-      ),
-      settledValue(
-        provider.getAnalystResearch?.(symbol, "") ?? null,
-        errors,
-        "analyst estimates",
-      ),
-      settledValue(provider.getTickerFinancials(symbol, ""), errors, "financials"),
-    ]);
+    const { actions, estimates, financials, currency, ...failures } = await loadEventSources(context.marketData, symbol);
+    const errors = [
+      failures.actionsError === null ? null : `corporate actions: ${failures.actionsError}`,
+      failures.estimatesError === null ? null : `analyst estimates: ${failures.estimatesError}`,
+      failures.financialsError === null ? null : `financials: ${failures.financialsError}`,
+    ].filter((error): error is string => error !== null);
     if (!actions && !estimates && !financials && errors.length > 0) {
       throw new Error(errors.join("; "));
     }
-    return {
-      actions,
-      estimates,
-      financials,
-      currency: actions?.currency ?? estimates?.currency ?? financials?.quote?.currency ?? "USD",
-      errors,
-    };
+    return { actions, estimates, financials, currency, errors };
   },
 };
 
@@ -110,7 +70,7 @@ export function projectEarningsEstimatesHeadless(
   if (unmatchedReportedPeriods.length) errors.push(`No statement period identified for ${unmatchedReportedPeriods.length} reported earnings row(s); quarterly revenue is unavailable.`);
 
   return {
-    columns: ESTIMATE_COLUMNS,
+    columns: EVENT_COLUMNS,
     rows,
     errors: errors.length ? errors : undefined,
     metadata: {
@@ -158,7 +118,7 @@ export function createEarningsEstimatesHeadless(
         maximum: 200,
       },
     ],
-    columns: ESTIMATE_COLUMNS,
+    columns: EVENT_COLUMNS,
     describe: (args) => `Earnings Estimates | ${String(args.argument)}`,
     async load(args, context) {
       return projectEarningsEstimatesHeadless(

package/src/plugins/builtin/research/index.tsx

@@ -10,9 +10,6 @@ import { analystResearchHeadless } from "./analyst-headless";
 import { eventsHeadless } from "./events-headless";
 import { earningsEstimatesHeadless } from "./headless";
 
-export { analystResearchHeadless } from "./analyst-headless";
-export { eventsHeadless } from "./events-headless";
-export { earningsEstimatesHeadless } from "./headless";
 
 function EarningsEstimatesPane(props: { focused: boolean; width: number; height: number }) {
   return (

package/src/plugins/builtin/research/relative-valuation-model.ts

@@ -1,7 +1,6 @@
 import { comparablePriceEarnings } from "../../../utils/price-earnings";
 import { selectMarketCapitalization } from "../../../utils/market-capitalization";
 import type { Quote, TickerFinancials } from "../../../types/financials";
-export { convertMarketCapitalization as comparableMarketCap } from "../../../utils/market-capitalization";
 
 export const RELATIVE_VALUATION_STALE_QUOTE_NOTICE = "Quote stale: quote-based values unavailable";
 export const RELATIVE_VALUATION_STALE_FUNDAMENTALS_NOTICE = "Fundamentals stale: retained values may be out of date";

package/src/plugins/builtin/research/relative-valuation-pane.tsx

@@ -1,9 +1,9 @@
 import { formatPriceEarnings } from "../../../utils/price-earnings";
-import { useCallback, useEffect, useMemo, useRef, useState } from "react";
+import { useCallback, useMemo, useState } from "react";
 import { TextAttributes } from "../../../ui";
 import {
   DataTableView,
-  usePaneFooter,
+  usePaneStatusFooter,
   usePaneNoticeFooter,
   type DataTableCell,
   type DataTableColumn,
@@ -19,14 +19,16 @@ import type { QuoteSubscriptionTarget } from "../../../types/data-provider";
 import type { TickerFinancials } from "../../../types/financials";
 import { normalizeSymbol } from "../../../utils/exchanges";
 import { colors, priceColor } from "../../../theme/colors";
-import { compareSortValues, type SortDirection } from "../../../utils/sort-values";
+import { compareSortValues, nextHeaderSort, type SortDirection } from "../../../utils/sort-values";
 import { formatCompact, formatCurrency, formatLevelPercent, formatNumber, formatPercent, formatPercentRaw } from "../../../utils/format";
 import { parseDisplayDate } from "../../../utils/datetime-format";
+import { convertMarketCapitalization } from "../../../utils/market-capitalization";
 import { usePluginTickerActions } from "../../runtime";
-import { handleRefreshKey, loadingErrorFooterInfo, useClampSelectedIndex } from "../shared/table-pane";
+import { handleRefreshKey, useClampSelectedIndex } from "../../../components/data-table/table-pane";
+import { useAsyncResource } from "../../../react/async-resource";
 import { useBoundTicker as useSymbolBinding } from "../shared/ticker-request";
 import { useFxRatesMap } from "../../../market-data/hooks";
-import { comparableMarketCap, RELATIVE_VALUATION_STALE_FUNDAMENTALS_NOTICE, relativeValuationValues, withLiveQuote } from "./relative-valuation-model";
+import { RELATIVE_VALUATION_STALE_FUNDAMENTALS_NOTICE, relativeValuationValues, withLiveQuote } from "./relative-valuation-model";
 
 type RelativeColumnId = "symbol" | "price" | "changePercent" | "marketCap" | "trailingPE" | "forwardPE" | "evSales" | "fcfYield" | "revenueGrowth" | "operatingMargin";
 type RelativeColumn = DataTableColumn & { id: RelativeColumnId };
@@ -107,15 +109,30 @@ function peerQuoteKey(symbol: string): string {
 export function RelativeValuationPane({ focused, width, height }: PaneProps) {
   const pane = usePaneInstance();
   const { symbol } = useSymbolBinding();
-  const symbols = useMemo(
-    () => relativeSymbolsFromPane(symbol, pane?.settings),
-    [pane?.settings, symbol],
-  );
+  // Keyed on the list itself, so a settings write that leaves the peers alone
+  // does not start the snapshot over.
+  const symbolList = relativeSymbolsFromPane(symbol, pane?.settings).join(",");
+  const symbols = useMemo(() => (symbolList ? symbolList.split(",") : []), [symbolList]);
   const { navigateTicker } = usePluginTickerActions();
+  const loadPeers = useCallback(async (forceRefresh: boolean): Promise<PeerSnapshot[]> => {
+    const coordinator = getSharedMarketDataCoordinator();
+    if (!coordinator) throw new Error("Market data unavailable");
+    // One batched snapshot request instead of one request per peer.
+    const entries = await coordinator.loadSnapshotsBatch(symbols.map((peer) => ({ symbol: peer })), { forceRefresh });
+    return symbols.map((peer, index) => {
+      const entry = entries[index];
+      return {
+        symbol: peer,
+        financials: entry?.data ?? entry?.lastGoodData ?? null,
+        error: entry?.error?.message,
+      };
+    });
+  }, [symbols]);
   // Fundamentals come from one batched snapshot; prices stream on top of it.
-  const [snapshots, setSnapshots] = useState<{ version: number; peers: PeerSnapshot[] }>({ version: 0, peers: [] });
-  const [loading, setLoading] = useState(false);
-  const [error, setError] = useState<string | null>(null);
+  // A changed peer list keeps the last snapshot up until its own answer.
+  const peerSnapshot = useAsyncResource(symbols.length > 0 ? loadPeers : null, { keepPreviousData: true });
+  const { data: peers, loading, updatedAt, reload } = peerSnapshot;
+  const error = symbols.length > 0 ? peerSnapshot.error : "No tickers selected";
   const [selectedIdx, setSelectedIdx] = useState(0);
   const [sortPreference, setSortPreference] = useState<RelativeSortPreference>(DEFAULT_RELATIVE_SORT);
   const baseCurrency = useAppSelector((state) => state.config.baseCurrency);
@@ -127,68 +144,19 @@ export function RelativeValuationPane({ focused, width, height }: PaneProps) {
     weight: 60,
   })), [symbols]);
   const { entries: liveQuotes } = useLiveQuoteEntries(quoteTargets);
-  const rows = useMemo<RelativeRow[]>(() => snapshots.peers.map((peer) => {
+  const rows = useMemo<RelativeRow[]>(() => (peers ?? []).map((peer) => {
     const entry = liveQuotes.get(peerQuoteKey(peer.symbol));
     return {
       symbol: peer.symbol,
       ...relativeValuationValues(withLiveQuote(peer.financials, entry ? resolveEntryValue(entry) : null)),
       error: peer.error,
     };
-  }), [liveQuotes, snapshots]);
+  }), [liveQuotes, peers]);
   const fxRates = useFxRatesMap([baseCurrency, ...rows.map((row) => row.marketCapCurrency)]);
   const columns = useMemo(() => buildRelativeColumns(baseCurrency), [baseCurrency]);
-  const fetchGenRef = useRef(0);
-
-  const reload = useCallback((forceRefresh = false) => {
-    fetchGenRef.current += 1;
-    const gen = fetchGenRef.current;
-    if (symbols.length === 0) {
-      setSnapshots((current) => ({ version: current.version + 1, peers: [] }));
-      setLoading(false);
-      setError("No tickers selected");
-      return;
-    }
-    const coordinator = getSharedMarketDataCoordinator();
-    if (!coordinator) {
-      setSnapshots((current) => ({ version: current.version + 1, peers: [] }));
-      setLoading(false);
-      setError("Market data unavailable");
-      return;
-    }
-    setLoading(true);
-    setError(null);
-    // One batched snapshot request instead of one request per peer.
-    coordinator.loadSnapshotsBatch(symbols.map((peer) => ({ symbol: peer })), { forceRefresh })
-      .then((entries) => {
-        if (fetchGenRef.current !== gen) return;
-        setSnapshots((current) => ({
-          version: current.version + 1,
-          peers: symbols.map((peer, index) => {
-            const entry = entries[index];
-            return {
-              symbol: peer,
-              financials: entry?.data ?? entry?.lastGoodData ?? null,
-              error: entry?.error?.message,
-            };
-          }),
-        }));
-      })
-      .catch((err) => {
-        if (fetchGenRef.current !== gen) return;
-        setError(err instanceof Error ? err.message : String(err));
-      })
-      .finally(() => {
-        if (fetchGenRef.current === gen) setLoading(false);
-      });
-  }, [symbols]);
-
-  useEffect(() => {
-    reload(false);
-    return () => { fetchGenRef.current += 1; };
-  }, [reload]);
 
   const comparableRows = useMemo(() => rows.map((row) => ({
-    ...row, marketCap: comparableMarketCap(row.marketCap, row.marketCapCurrency, baseCurrency, fxRates),
+    ...row, marketCap: convertMarketCapitalization(row.marketCap, row.marketCapCurrency, baseCurrency, fxRates),
   })), [rows, baseCurrency, fxRates]);
   const missingFx = rows.some((row, index) => row.marketCap != null && comparableRows[index]?.marketCap == null);
   // Sorting by a live column would reshuffle rows under the cursor on every
@@ -197,7 +165,7 @@ export function RelativeValuationPane({ focused, width, height }: PaneProps) {
   const orderRows = useSampledValue(
     comparableRows,
     RELATIVE_ORDER_SAMPLE_MS,
-    `${snapshots.version}:${sortPreference.columnId}:${sortPreference.direction}`,
+    `${updatedAt}:${sortPreference.columnId}:${sortPreference.direction}`,
   );
   const order = useMemo(() => sortRelativeRowIndices(orderRows, sortPreference), [orderRows, sortPreference]);
   const sortedRows = useMemo(
@@ -219,47 +187,42 @@ export function RelativeValuationPane({ focused, width, height }: PaneProps) {
 
   useClampSelectedIndex(rows.length, selectedIdx, setSelectedIdx);
 
-  const renderCell = useCallback((row: RelativeRow, column: RelativeColumn, _index: number, rowState: { selected: boolean }): DataTableCell => {
-    const selectedColor = rowState.selected ? colors.selectedText : undefined;
+  const renderCell = useCallback((row: RelativeRow, column: RelativeColumn): DataTableCell => {
     switch (column.id) {
       case "symbol":
-        return { text: row.symbol, color: selectedColor ?? (row.error || row.quoteStale || row.fundamentalsProvenance?.stale ? colors.warning : colors.textBright), attributes: TextAttributes.BOLD };
+        return { text: row.symbol, color: row.error || row.quoteStale || row.fundamentalsProvenance?.stale ? colors.warning : colors.textBright, attributes: TextAttributes.BOLD };
       case "price":
-        return { text: row.price != null ? formatCurrency(row.price, row.currency ?? "USD") : "-", color: selectedColor ?? colors.text };
+        return { text: row.price != null ? formatCurrency(row.price, row.currency ?? "USD") : "-", color: colors.text };
       case "changePercent":
-        return { text: row.changePercent != null ? formatPercentRaw(row.changePercent) : "-", color: selectedColor ?? priceColor(row.changePercent ?? 0) };
+        return { text: row.changePercent != null ? formatPercentRaw(row.changePercent) : "-", color: priceColor(row.changePercent ?? 0) };
       case "marketCap":
-        return { text: formatCompact(row.marketCap ?? undefined, { fixedDecimals: true }), color: selectedColor ?? colors.textDim };
+        return { text: formatCompact(row.marketCap ?? undefined, { fixedDecimals: true }), color: colors.textDim };
       case "trailingPE":
-        return { text: formatPriceEarnings(row.reportedMultiples.trailingPE), color: selectedColor ?? colors.text };
+        return { text: formatPriceEarnings(row.reportedMultiples.trailingPE), color: colors.text };
       case "forwardPE":
-        return { text: formatPriceEarnings(row.reportedMultiples.forwardPE), color: selectedColor ?? colors.text };
+        return { text: formatPriceEarnings(row.reportedMultiples.forwardPE), color: colors.text };
       case "evSales":
-        return { text: formatNumber(row.evSales ?? undefined, 1), color: selectedColor ?? colors.text };
+        return { text: formatNumber(row.evSales ?? undefined, 1), color: colors.text };
       case "fcfYield":
-        return { text: formatLevelPercent(row.fcfYield ?? undefined), color: selectedColor ?? priceColor(row.fcfYield ?? 0) };
+        return { text: formatLevelPercent(row.fcfYield ?? undefined), color: priceColor(row.fcfYield ?? 0) };
       case "revenueGrowth":
-        return { text: formatPercent(row.revenueGrowth ?? undefined), color: selectedColor ?? priceColor(row.revenueGrowth ?? 0) };
+        return { text: formatPercent(row.revenueGrowth ?? undefined), color: priceColor(row.revenueGrowth ?? 0) };
       case "operatingMargin":
-        return { text: formatLevelPercent(row.operatingMargin ?? undefined), color: selectedColor ?? colors.text };
+        return { text: formatLevelPercent(row.operatingMargin ?? undefined), color: colors.text };
     }
   }, []);
 
   const handleKeyDown = useCallback((event: DataTableKeyEvent) => {
-    return handleRefreshKey(event, () => reload(true), { stopPropagation: true });
+    return handleRefreshKey(event, () => void reload(), { stopPropagation: true });
   }, [reload]);
 
   const handleHeaderClick = useCallback((columnId: string) => {
-    setSortPreference((current) => (
-      current.columnId === columnId
-        ? { columnId: current.columnId, direction: current.direction === "asc" ? "desc" : "asc" }
-        : { columnId: columnId as RelativeColumnId, direction: columnId === "symbol" ? "asc" : "desc" }
-    ));
+    setSortPreference((current) => nextHeaderSort(current, columnId as RelativeColumnId, {
+      firstDirection: columnId === "symbol" ? "asc" : "desc",
+    }));
   }, []);
 
-  usePaneFooter("relative-valuation", () => ({
-    info: loadingErrorFooterInfo(loading, status),
-  }), [status, loading]);
+  usePaneStatusFooter({ registrationId: "relative-valuation", loading, error: status });
 
   return (
     <DataTableView<RelativeRow, RelativeColumn>
@@ -280,6 +243,7 @@ export function RelativeValuationPane({ focused, width, height }: PaneProps) {
       onHeaderClick={handleHeaderClick}
       getItemKey={(row) => row.symbol}
       renderCell={renderCell}
+      selectedTextOverridesCellColor
       getExportMetadata={() => sortedRows.flatMap((row) => [
         [row.symbol, "Quote as of", parseDisplayDate(row.quoteAsOf)?.toISOString() ?? "unavailable", "Stale", String(row.quoteStale ?? "unknown")],
         [row.symbol, "Fundamentals retrieved", row.fundamentalsProvenance?.retrievedAt ?? "unavailable",

package/src/plugins/builtin/research-search/data.ts

@@ -37,10 +37,6 @@ export function statusOf(error: unknown): number | undefined {
   return error instanceof ApiRequestError ? error.status : undefined;
 }
 
-export function errorMessage(error: unknown): string {
-  return error instanceof Error ? error.message : String(error);
-}
-
 export function isAbortError(error: unknown): boolean {
   return error instanceof Error && error.name === "AbortError";
 }

package/src/plugins/builtin/research-search/document-view.tsx

@@ -1,4 +1,4 @@
-import { useCallback, useEffect, useMemo, useRef } from "react";
+import { useEffect, useMemo, useRef } from "react";
 import type {
   CloudSearchDocType,
   CloudSearchDocument,
@@ -18,6 +18,7 @@ import {
   useUiCapabilities,
   type ScrollBoxRenderable,
 } from "../../../ui";
+import { scrollByLines } from "../../../state/pane-scroll-registry";
 import { isPlainKey } from "../../../utils/keyboard";
 import { wrapTextLines } from "../../../utils/text-wrap";
 import { chunkAttribution, documentBodyWidth } from "./model";
@@ -205,20 +206,13 @@ export function SearchDocumentView({
     scrollBox.scrollTo(Math.max(0, target - 2));
   }, [layout, nativePaneChrome]);
 
-  const scrollBy = useCallback((delta: number) => {
-    const scrollBox = scrollRef.current;
-    if (!scrollBox?.viewport) return;
-    const maxScrollTop = Math.max(0, scrollBox.scrollHeight - scrollBox.viewport.height);
-    scrollBox.scrollTop = Math.max(0, Math.min(maxScrollTop, scrollBox.scrollTop + delta));
-  }, []);
-
   useShortcut((event) => {
     if (event.defaultPrevented || event.targetEditable) return;
     const delta = isPlainKey(event, "j") ? 1 : isPlainKey(event, "k") ? -1 : 0;
     if (!delta) return;
     event.preventDefault();
     event.stopPropagation();
-    scrollBy(delta);
+    if (scrollRef.current) scrollByLines(scrollRef.current, delta);
   }, { enabled: focused && !!layout });
 
   if (loading && !document) {

package/src/plugins/builtin/research-search/index.tsx

@@ -67,6 +67,7 @@ export const researchSearchPlugin: GloomPlugin = {
       ],
       shortcut: {
         prefix: "SRCH",
+        aliases: ["NSE"],
         argPlaceholder: "query",
         argKind: "text",
         argOptional: true,

package/src/plugins/builtin/research-search/model.ts

@@ -8,6 +8,7 @@ import type {
 } from "../../../api-client";
 import type { CloudSearchParams } from "../../../api-client/paths";
 import type { DataTableColumn } from "../../../components";
+import { formatShortDate } from "../../../utils/datetime-format";
 
 export const RESEARCH_SEARCH_PANE_ID = "research-search";
 export const RESEARCH_SEARCH_TEMPLATE_ID = "research-search-pane";
@@ -161,10 +162,7 @@ export function hitTypeLabel(hit: CloudSearchHit): string {
 }
 
 export function formatHitDate(value: string | null | undefined): string {
-  if (!value) return "\u2014";
-  const date = new Date(value);
-  if (Number.isNaN(date.getTime())) return "\u2014";
-  return date.toLocaleDateString("en-US", { month: "short", day: "2-digit", year: "2-digit" });
+  return formatShortDate(value, { day: "2-digit", year: "2-digit", fallback: "\u2014" });
 }
 
 const MS_PER_HOUR = 60 * 60 * 1000;
@@ -184,7 +182,7 @@ export function formatHitDateShort(value: string | null | undefined, now = Date.
   if (age >= 0 && age < MS_PER_HOUR) return `${Math.max(1, Math.floor(age / 60_000))}m`;
   if (age >= 0 && age < 24 * MS_PER_HOUR) return `${Math.floor(age / MS_PER_HOUR)}h`;
   if (date.getFullYear() === new Date(now).getFullYear()) {
-    return date.toLocaleDateString("en-US", { month: "short", day: "numeric" });
+    return formatShortDate(date, { year: false });
   }
   return date.toLocaleDateString("en-US", { month: "short", year: "numeric" });
 }
@@ -261,24 +259,10 @@ export function buildResultColumns(width: number): SearchColumn[] {
  * source document but picks whichever chunk scored best, so the same document
  * can arrive on two pages under two different chunk ids.
  */
-function hitDocumentKey(hit: CloudSearchHit): string {
+export function hitDocumentKey(hit: CloudSearchHit): string {
   return `${hit.docType}:${hit.sourceId}`;
 }
 
-/** Paged responses can overlap when new documents land between requests. */
-export function appendUniqueHits(current: CloudSearchHit[], next: CloudSearchHit[]): CloudSearchHit[] {
-  if (next.length === 0) return current;
-  const seen = new Set(current.map(hitDocumentKey));
-  const merged = [...current];
-  for (const hit of next) {
-    const key = hitDocumentKey(hit);
-    if (seen.has(key)) continue;
-    seen.add(key);
-    merged.push(hit);
-  }
-  return merged;
-}
-
 /** Chunks behind a collapsed row, so hiding the rest stays visible. */
 export function hitMatchCountLabel(hit: CloudSearchHit): string {
   const count = hit.matchCount ?? 0;