gloomberb 0.7.2 → 0.7.4

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

@@ -11,9 +11,11 @@ import {
 } from "../../../components";
 import { useShortcut } from "../../../react/input";
 import { blendHex, colors, priceColor } from "../../../theme/colors";
+import { higherContrast } from "../../../theme/color-utils";
 import type { HolderData, HolderRecord } from "../../../types/financials";
 import type { GloomPlugin } from "../../../types/plugin";
 import { formatCompact, formatPercent, formatPercentRaw, padTo } from "../../../utils/format";
+import { isPlainKey } from "../../../utils/keyboard";
 import { useAssetData, usePluginPaneState } from "../../plugin-runtime";
 import { createTickerSurfacePaneTemplate } from "../ticker-surface";
 
@@ -445,6 +447,14 @@ function desktopTileColor(row: HolderRow): string {
   return blendHex(colors.panel, row.changePercent > 0 ? colors.positive : colors.negative, intensity);
 }
 
+function tileTextColor(backgroundColor: string): string {
+  return higherContrast(
+    higherContrast(colors.text, colors.textBright, backgroundColor),
+    colors.selectedText,
+    backgroundColor,
+  );
+}
+
 function Tile({ tile, selected, currency, marketCap, onSelect }: {
   tile: TileLayout;
   selected: boolean;
@@ -455,7 +465,8 @@ function Tile({ tile, selected, currency, marketCap, onSelect }: {
   const renderWidth = Math.max(1, tile.width - (tile.width > 2 ? 1 : 0));
   const renderHeight = Math.max(1, tile.height - (tile.height > 2 ? 1 : 0));
   const innerWidth = Math.max(1, renderWidth - 1);
-  const textColor = "#ffffff";
+  const backgroundColor = selected ? colors.selected : tileColor(tile.row);
+  const textColor = tileTextColor(backgroundColor);
   const attributes = selected ? TextAttributes.BOLD : TextAttributes.NONE;
   const label = tile.row.name;
   const amount = tile.row.value != null
@@ -471,7 +482,7 @@ function Tile({ tile, selected, currency, marketCap, onSelect }: {
       top={tile.y}
       width={renderWidth}
       height={renderHeight}
-      backgroundColor={selected ? colors.selected : tileColor(tile.row)}
+      backgroundColor={backgroundColor}
       onMouseDown={(event: PreventableMouseEvent) => {
         event.preventDefault();
         onSelect();
@@ -516,6 +527,7 @@ function DesktopTile({ tile, chartWidth, chartHeight, selected, hovered, currenc
   const canShowLabel = tile.width >= 4 && tile.height >= 1.4;
   const isTiny = tile.width < 5 || tile.height < 2;
   const backgroundColor = desktopTileColor(tile.row);
+  const textColor = tileTextColor(backgroundColor);
   const style: CSSProperties = {
     position: "absolute",
     left: `calc(${pct(tile.x, chartWidth)} + 1px)`,
@@ -531,14 +543,14 @@ function DesktopTile({ tile, chartWidth, chartHeight, selected, hovered, currenc
       ? `1px solid ${colors.textBright}`
       : hovered
         ? `1px solid ${blendHex(colors.textBright, backgroundColor, 0.55)}`
-        : "1px solid rgba(255,255,255,0.11)",
+        : `1px solid ${blendHex(backgroundColor, colors.textBright, 0.11)}`,
     boxShadow: selected
-      ? "inset 0 0 0 1px rgba(255,255,255,0.55)"
+      ? `inset 0 0 0 1px ${blendHex(backgroundColor, colors.textBright, 0.55)}`
       : hovered
-        ? "inset 0 0 0 1px rgba(255,255,255,0.18)"
+        ? `inset 0 0 0 1px ${blendHex(backgroundColor, colors.textBright, 0.18)}`
         : "none",
     backgroundColor,
-    color: "#ffffff",
+    color: textColor,
     display: "flex",
     flexDirection: "column",
     alignItems: "stretch",
@@ -557,7 +569,7 @@ function DesktopTile({ tile, chartWidth, chartHeight, selected, hovered, currenc
     whiteSpace: "nowrap",
     lineHeight: 1.1,
     letterSpacing: 0,
-    textShadow: "0 1px 1px rgba(0,0,0,0.35)",
+    textShadow: `0 1px 1px ${blendHex(backgroundColor, colors.bg, 0.35)}`,
   };
 
   return (
@@ -573,7 +585,7 @@ function DesktopTile({ tile, chartWidth, chartHeight, selected, hovered, currenc
     >
       {canShowLabel && (
         <Text
-          fg="#ffffff"
+          fg={textColor}
           attributes={selected ? TextAttributes.BOLD : TextAttributes.NONE}
           style={{
             ...textStyle,
@@ -586,10 +598,10 @@ function DesktopTile({ tile, chartWidth, chartHeight, selected, hovered, currenc
       )}
       {canShowDetails && (
         <>
-          <Text fg="#ffffff" style={{ ...textStyle, fontSize: 12, fontWeight: 600 }}>{amount}</Text>
-          <Text fg="#ffffff" style={{ ...textStyle, fontSize: 12, fontWeight: 600 }}>{ownership ?? change}</Text>
+          <Text fg={textColor} style={{ ...textStyle, fontSize: 12, fontWeight: 600 }}>{amount}</Text>
+          <Text fg={textColor} style={{ ...textStyle, fontSize: 12, fontWeight: 600 }}>{ownership ?? change}</Text>
           {ownership && canShowChange && (
-            <Text fg="#ffffff" style={{ ...textStyle, fontSize: 12, fontWeight: 600 }}>{change}</Text>
+            <Text fg={textColor} style={{ ...textStyle, fontSize: 12, fontWeight: 600 }}>{change}</Text>
           )}
         </>
       )}
@@ -810,7 +822,7 @@ function HoldersView({ focused, width, height }: { focused: boolean; width: numb
 
   useShortcut((event) => {
     if (!focused || viewMode !== "chart") return;
-    if (event.name === "j" || event.name === "down") {
+    if (isPlainKey(event, "j", "down")) {
       event.preventDefault?.();
       event.stopPropagation?.();
       const nextIdx = Math.min((selectedIdx >= 0 ? selectedIdx : 0) + 1, sortedRows.length - 1);
@@ -818,7 +830,7 @@ function HoldersView({ focused, width, height }: { focused: boolean; width: numb
       if (nextRow) setSelectedId(nextRow.id);
       return;
     }
-    if (event.name === "k" || event.name === "up") {
+    if (isPlainKey(event, "k", "up")) {
       event.preventDefault?.();
       event.stopPropagation?.();
       const nextIdx = Math.max((selectedIdx >= 0 ? selectedIdx : 0) - 1, 0);

package/src/plugins/builtin/layout-manager.tsx

@@ -1,4 +1,3 @@
-import { saveConfig } from "../../data/config-store";
 import { findPaneInstance, type LayoutConfig } from "../../types/config";
 import type { AppNotificationRequest, GloomPlugin, GloomPluginContext } from "../../types/plugin";
 import type { AppAction } from "../../state/app-context";
@@ -30,15 +29,10 @@ export function clearLayoutManagerDispatch() {
   getStateRef = null;
 }
 
-function persistLayout(ctx: Pick<GloomPluginContext, "getConfig">, layout: LayoutConfig) {
+function persistLayout(_ctx: Pick<GloomPluginContext, "getConfig">, layout: LayoutConfig) {
   if (!dispatchRef) return;
   dispatchRef({ type: "PUSH_LAYOUT_HISTORY" });
   dispatchRef({ type: "UPDATE_LAYOUT", layout });
-  const config = ctx.getConfig();
-  const layouts = config.layouts.map((savedLayout, index) => (
-    index === config.activeLayoutIndex ? { ...savedLayout, layout } : savedLayout
-  ));
-  saveConfig({ ...config, layout, layouts }).catch(() => {});
 }
 
 function getFocusedPane(layout: LayoutConfig, focusedPaneId: string | null) {

package/src/plugins/builtin/market-movers/index.tsx

@@ -158,6 +158,39 @@ function createRows(quotes: ScreenerQuote[]): MarketMoverRow[] {
   }));
 }
 
+function summaryQuoteFromQuote(
+  symbol: string,
+  quote: { name?: string; price: number; change: number; changePercent: number },
+): MarketSummaryQuote {
+  return {
+    symbol,
+    name: quote.name ?? symbol,
+    price: quote.price,
+    change: quote.change,
+    changePercent: quote.changePercent,
+  };
+}
+
+function screenerQuoteFromQuote(symbol: string, quote: { name?: string; price?: number; change?: number; changePercent?: number; volume?: number; currency?: string }): ScreenerQuote {
+  return {
+    symbol,
+    name: quote.name ?? symbol,
+    price: quote.price ?? 0,
+    change: quote.change ?? 0,
+    changePercent: quote.changePercent ?? 0,
+    volume: quote.volume ?? 0,
+    avgVolume: 0,
+    volumeRatio: 0,
+    marketCap: undefined,
+    currency: quote.currency ?? "USD",
+    fiftyTwoWeekHigh: undefined,
+    fiftyTwoWeekLow: undefined,
+    dayHigh: undefined,
+    dayLow: undefined,
+    exchange: "",
+  };
+}
+
 function buildColumns(width: number): MarketMoverColumn[] {
   const rankWidth = 3;
   const tickerWidth = 8;
@@ -227,28 +260,34 @@ function MarketMoversPane({ focused, width, height }: PaneProps) {
   useEffect(() => {
     if (!dataProvider) return;
     const loadSummary = async () => {
-      const results: MarketSummaryQuote[] = [];
-      await Promise.allSettled(
-        MARKET_SUMMARY_SYMBOLS.map(async (symbol) => {
-          try {
-            const q = await dataProvider.getQuote(symbol, "");
-            if (q) {
-              results.push({
-                symbol,
-                name: q.name ?? symbol,
-                price: q.price,
-                change: q.change,
-                changePercent: q.changePercent,
-              });
-            }
-          } catch { /* skip */ }
-        }),
-      );
-      // Preserve the original symbol order
+      if (dataProvider.getQuotesBatch) {
+        const batchResults = await dataProvider.getQuotesBatch(
+          MARKET_SUMMARY_SYMBOLS.map((symbol) => ({ symbol, exchange: "" })),
+        ).catch(() => []);
+        const bySymbol = new Map(batchResults.map((result) => [result.target.symbol, result]));
+        setSummaryQuotes(
+          MARKET_SUMMARY_SYMBOLS
+            .map((symbol) => {
+              const result = bySymbol.get(symbol);
+              return result?.quote ? summaryQuoteFromQuote(symbol, result.quote) : null;
+            })
+            .filter((quote): quote is MarketSummaryQuote => !!quote),
+        );
+        return;
+      }
+
+      const results = await Promise.all(MARKET_SUMMARY_SYMBOLS.map(async (symbol) => {
+        try {
+          const quote = await dataProvider.getQuote(symbol, "");
+          return quote ? summaryQuoteFromQuote(symbol, quote) : null;
+        } catch {
+          return null;
+        }
+      }));
       setSummaryQuotes(
         MARKET_SUMMARY_SYMBOLS
           .map((s) => results.find((r) => r.symbol === s))
-          .filter((r): r is MarketSummaryQuote => r !== undefined),
+          .filter((r): r is MarketSummaryQuote => !!r),
       );
     };
     loadSummary();
@@ -280,33 +319,26 @@ function MarketMoversPane({ focused, width, height }: PaneProps) {
         const resolved: ScreenerQuote[] = [];
 
         if (dataProvider) {
-          await Promise.allSettled(
-            trending.slice(0, 25).map(async ({ symbol }) => {
+          const targets = trending.slice(0, 25).map(({ symbol }) => ({ symbol, exchange: "" }));
+          if (dataProvider.getQuotesBatch) {
+            const batchResults = await dataProvider.getQuotesBatch(targets).catch(() => []);
+            for (const result of batchResults) {
+              if (fetchGenRef.current !== gen) return;
+              if (result.quote) {
+                resolved.push(screenerQuoteFromQuote(result.target.symbol, result.quote));
+              }
+            }
+          } else {
+            await Promise.allSettled(targets.map(async ({ symbol }) => {
               try {
                 const q = await dataProvider.getQuote(symbol, "");
                 if (fetchGenRef.current !== gen) return;
                 if (q) {
-                  resolved.push({
-                    symbol,
-                    name: q.name ?? symbol,
-                    price: q.price ?? 0,
-                    change: q.change ?? 0,
-                    changePercent: q.changePercent ?? 0,
-                    volume: q.volume ?? 0,
-                    avgVolume: 0,
-                    volumeRatio: 0,
-                    marketCap: undefined,
-                    currency: q.currency ?? "USD",
-                    fiftyTwoWeekHigh: undefined,
-                    fiftyTwoWeekLow: undefined,
-                    dayHigh: undefined,
-                    dayLow: undefined,
-                    exchange: "",
-                  });
+                  resolved.push(screenerQuoteFromQuote(symbol, q));
                 }
               } catch { /* skip */ }
-            }),
-          );
+            }));
+          }
         }
 
         if (fetchGenRef.current !== gen) return;

package/src/plugins/builtin/news-wire/breaking-pane.test.tsx

@@ -0,0 +1,156 @@
+import { afterEach, describe, expect, test } from "bun:test";
+import { act } from "react";
+import { PaneFooterBar, PaneFooterProvider } from "../../../components/layout/pane-footer";
+import type { NewsService } from "../../../news/aggregator";
+import { setSharedNewsService } from "../../../news/hooks";
+import type { NewsArticle, NewsQueryState } from "../../../news/types";
+import { testRender } from "../../../renderers/opentui/test-utils";
+import { AppContext, PaneInstanceProvider, createInitialState } from "../../../state/app-context";
+import { createStatefulTestPluginRuntime } from "../../../test-support/plugin-runtime";
+import { createDefaultConfig } from "../../../types/config";
+import { Box } from "../../../ui";
+import { PluginRenderProvider } from "../../plugin-runtime";
+import { __setDetectedProvidersForTests } from "../ai/providers";
+import { setAiRunHost } from "../ai/runner";
+import { BreakingPane } from "./breaking-pane";
+
+const PANE_ID = "news-breaking:test";
+
+let testSetup: Awaited<ReturnType<typeof testRender>> | undefined;
+
+function makeArticle(): NewsArticle {
+  return {
+    id: "story-1",
+    title: "Chip stocks rally on new AI demand",
+    url: "https://example.com/story",
+    source: "example",
+    publishedAt: new Date("2026-05-13T12:00:00.000Z"),
+    summary: "Semiconductor names moved higher after hyperscaler capex commentary.",
+    topic: "earnings",
+    topics: ["earnings"],
+    sectors: ["information_technology"],
+    categories: ["earnings", "information_technology"],
+    tickers: ["AMD", "NVDA"],
+    sentiment: "positive",
+    scores: {
+      importance: 85,
+      urgency: 80,
+      marketImpact: 75,
+      novelty: 60,
+      confidence: 90,
+    },
+    isBreaking: true,
+    isDeveloping: false,
+    importance: 85,
+  };
+}
+
+function createReadyNewsService(articles: NewsArticle[]): { service: NewsService; getQueryStateCalls: () => number } {
+  const state: NewsQueryState = {
+    phase: "ready",
+    articles,
+    error: null,
+    updatedAt: Date.now(),
+    sourceIds: ["test"],
+  };
+  const listeners = new Set<() => void>();
+  let queryStateCalls = 0;
+  const service = {
+    subscribe(listener: () => void) {
+      listeners.add(listener);
+      return () => {
+        listeners.delete(listener);
+      };
+    },
+    getVersion() {
+      return 1;
+    },
+    getQueryState() {
+      queryStateCalls += 1;
+      return state;
+    },
+    async load() {
+      return state;
+    },
+    async loadStory() {
+      return null;
+    },
+  } as unknown as NewsService;
+  return { service, getQueryStateCalls: () => queryStateCalls };
+}
+
+function createHarness() {
+  const config = createDefaultConfig("/tmp/gloomberb-breaking-news");
+  config.layout.instances.push({
+    instanceId: PANE_ID,
+    paneId: "news-breaking",
+    title: "Breaking News",
+  });
+  const state = createInitialState(config);
+  state.focusedPaneId = PANE_ID;
+
+  return (
+    <AppContext value={{ state, dispatch: () => {} }}>
+      <PaneInstanceProvider paneId={PANE_ID}>
+        <PluginRenderProvider pluginId="news" runtime={createStatefulTestPluginRuntime()}>
+          <PaneFooterProvider>
+            {(footer) => (
+              <Box flexDirection="column" width={90} height={18}>
+                <BreakingPane focused width={90} height={17} />
+                <PaneFooterBar footer={footer} focused width={90} />
+              </Box>
+            )}
+          </PaneFooterProvider>
+        </PluginRenderProvider>
+      </PaneInstanceProvider>
+    </AppContext>
+  );
+}
+
+afterEach(async () => {
+  setSharedNewsService(null);
+  setAiRunHost(null);
+  __setDetectedProvidersForTests(null);
+  if (testSetup) {
+    await act(async () => {
+      testSetup!.renderer.destroy();
+    });
+    testSetup = undefined;
+  }
+});
+
+describe("BreakingPane", () => {
+  test("does not run local AI providers when breaking news is mounted", async () => {
+    let runCalls = 0;
+    const newsService = createReadyNewsService([makeArticle()]);
+    setSharedNewsService(newsService.service);
+    __setDetectedProvidersForTests([
+      {
+        id: "claude",
+        name: "Claude",
+        command: "claude",
+        available: true,
+        buildArgs: () => [],
+      },
+    ]);
+    setAiRunHost({
+      run() {
+        runCalls += 1;
+        return {
+          done: Promise.resolve("digest"),
+          cancel: () => {},
+        };
+      },
+    });
+
+    await act(async () => {
+      testSetup = await testRender(createHarness(), { width: 90, height: 18 });
+      await Bun.sleep(20);
+      await testSetup.renderOnce();
+      await testSetup.renderOnce();
+    });
+
+    expect(runCalls).toBe(0);
+    expect(newsService.getQueryStateCalls()).toBeGreaterThan(0);
+  });
+});

package/src/plugins/builtin/news-wire/breaking-pane.tsx

@@ -1,13 +1,8 @@
 import { Box } from "../../../ui";
-import { useCallback, useEffect, useMemo, useRef, useState } from "react";
 import type { PaneProps } from "../../../types/plugin";
 import { useLoadNewsStory, useNewsArticles } from "../../../news/hooks";
 import { useDebouncedPluginPaneState, usePluginPaneState } from "../../plugin-runtime";
 import { Spinner } from "../../../components";
-import type { MarketNewsItem } from "../../../types/news-source";
-import { detectProviders, getAiProvider, resolveDefaultAiProviderId } from "../ai/providers";
-import { runAiPrompt } from "../ai/runner";
-import { getDigest, setDigest, isDigestInFlight, markDigestInFlight, clearDigestInFlight } from "./digest-store";
 import { NewsDetailView, useNewsArticleDetail } from "./news-detail-view";
 import { NewsArticleStackView, type NewsSortPreference } from "./news-table";
 import { useNewsArticleFooter } from "./news-footer";
@@ -15,19 +10,7 @@ import { NEWS_QUERY_PRESETS } from "./news-query-presets";
 import { usePersistedNewsArticles } from "./persisted-articles";
 import { useNewsReadState } from "./read-state";
 
-const DIGEST_PROMPT = `You are a financial news wire editor. Condense this headline and summary into a single concise actionable bullet point for a professional trader. Include why it matters and potential market impact. Keep it under 120 characters. Respond with ONLY the bullet text, nothing else.
-
-Headline: {title}
-Summary: {summary}`;
-
 const DEFAULT_SORT: NewsSortPreference = { columnId: "importance", direction: "desc" };
-const BRAILLE_FRAMES = ["⠋", "⠙", "⠹", "⠸", "⠼", "⠴", "⠦", "⠧", "⠇", "⠏"];
-
-function buildDigestPrompt(item: MarketNewsItem): string {
-  return DIGEST_PROMPT
-    .replace("{title}", item.title)
-    .replace("{summary}", item.summary ?? item.title);
-}
 
 export function BreakingPane({ focused, width, height }: PaneProps) {
   const breakingState = useNewsArticles(NEWS_QUERY_PRESETS.breaking);
@@ -35,86 +18,14 @@ export function BreakingPane({ focused, width, height }: PaneProps) {
   const loading = breakingState.phase === "loading" || (breakingState.phase === "refreshing" && articles.length === 0);
   const [selectedArticleId, setSelectedArticleId] = useDebouncedPluginPaneState<string | null>("breaking:selectedArticleId", null);
   const [sortPreference, setSortPreference] = usePluginPaneState<NewsSortPreference>("breaking:sort", DEFAULT_SORT);
-  const [digestVersion, setDigestVersion] = useState(0);
-  const [aiAvailable, setAiAvailable] = useState(false);
-  const [aiError, setAiError] = useState<string | null>(null);
-  const [aiRunning, setAiRunning] = useState(false);
-  const [spinFrame, setSpinFrame] = useState(0);
-  const processingRef = useRef(false);
   const loadNewsStory = useLoadNewsStory();
   const { detailArticle, openArticle, closeDetail } = useNewsArticleDetail(articles, loadNewsStory);
   const { readArticleIds, markArticleRead } = useNewsReadState();
 
-  useEffect(() => {
-    const providers = detectProviders();
-    setAiAvailable(providers.some((provider) => provider.available));
-  }, []);
-
-  useEffect(() => {
-    if (!aiRunning) return;
-    const id = setInterval(() => setSpinFrame((frame) => (frame + 1) % BRAILLE_FRAMES.length), 80);
-    return () => clearInterval(id);
-  }, [aiRunning]);
-
-  useEffect(() => {
-    if (!aiAvailable || articles.length === 0) return;
-    if (processingRef.current) return;
-
-    const providerId = resolveDefaultAiProviderId();
-    const provider = getAiProvider(providerId);
-    if (!provider?.available) return;
-
-    const toDigest = articles.filter((article) => !getDigest(article.id) && !isDigestInFlight(article.id));
-    if (toDigest.length === 0) return;
-
-    processingRef.current = true;
-    setAiRunning(true);
-
-    (async () => {
-      for (const article of toDigest) {
-        if (getDigest(article.id) || isDigestInFlight(article.id)) continue;
-        markDigestInFlight(article.id);
-
-        try {
-          const result = await runAiPrompt({
-            provider,
-            prompt: buildDigestPrompt(article),
-          }).done;
-
-          const digest = result.trim().slice(0, 150);
-          if (digest) {
-            setDigest(article.id, digest);
-            setDigestVersion((version) => version + 1);
-          }
-        } catch (error) {
-          const message = error instanceof Error ? error.message : String(error);
-          if (message.includes("Credit") || message.includes("quota") || message.includes("rate limit")) {
-            setAiError(message);
-            break;
-          }
-        } finally {
-          clearDigestInFlight(article.id);
-        }
-      }
-      setAiRunning(false);
-      processingRef.current = false;
-    })();
-  }, [aiAvailable, articles]);
-
-  const titleForArticle = useCallback((article: MarketNewsItem) => (
-    getDigest(article.id) ?? article.title
-  ), [digestVersion]);
-
-  const footerInfo = useMemo(() => [
-    ...(aiRunning ? [{ id: "running", parts: [{ text: BRAILLE_FRAMES[spinFrame] ?? "running", tone: "positive" as const }] }] : []),
-    ...(aiError ? [{ id: "paused", parts: [{ text: "AI paused", tone: "warning" as const }] }] : []),
-  ], [aiError, aiRunning, spinFrame]);
-
   useNewsArticleFooter({
     registrationId: "news-wire:breaking",
     focused,
     article: detailArticle,
-    info: footerInfo,
   });
 
   const detailContent = detailArticle ? (
@@ -152,7 +63,6 @@ export function BreakingPane({ focused, width, height }: PaneProps) {
       columns={["time", "title", "tickers", "importance"]}
       emptyStateTitle="No breaking news"
       emptyStateHint="Breaking stories appear when high-priority headlines arrive."
-      titleForArticle={titleForArticle}
     />
   );
 }

package/src/plugins/builtin/news-wire/index.tsx

@@ -8,7 +8,6 @@ import {
   BREAKING_NEWS_NOTIFICATIONS_ENABLED_KEY,
   setupBreakingNewsNotifications,
 } from "./breaking-notifications";
-import { setDigestPersistence } from "./digest-store";
 import {
   addUserNewsFeed,
   getEnabledNewsFeeds,
@@ -39,7 +38,7 @@ const TopPane = createNewsPresetPane({
   paneKey: "top",
   title: "Top News",
   query: NEWS_QUERY_PRESETS.top,
-  columns: ["rank", "time", "source", "title", "tickers", "importance"],
+  columns: ["time", "title", "tickers", "importance"],
   defaultSort: { columnId: "importance", direction: "desc" },
   emptyStateTitle: "No top stories yet",
   emptyStateHint: "Try refreshing later as new headlines are ranked.",
@@ -56,7 +55,6 @@ const FeedPane = createNewsPresetPane({
 });
 
 export function registerNewsWireFeatures(ctx: GloomPluginContext): () => void {
-  setDigestPersistence(ctx.persistence);
   ctx.registerPane({ id: "news-top", name: "Top News", icon: "T", component: TopPane, defaultPosition: "right", defaultMode: "floating", defaultFloatingSize: { width: 90, height: 30 } });
   ctx.registerPane({ id: "news-feed", name: "News Feed", icon: "N", component: FeedPane, defaultPosition: "right", defaultMode: "floating", defaultFloatingSize: { width: 100, height: 35 } });
   ctx.registerPane({ id: "news-industry", name: "Sector News", icon: "S", component: IndustryPane, defaultPosition: "right", defaultMode: "floating", defaultFloatingSize: { width: 100, height: 35 } });

package/src/plugins/builtin/news-wire/news-detail-view.tsx

@@ -8,6 +8,7 @@ import { TickerBadge } from "../../../components/ticker-badge";
 import { ExternalLink, ExternalLinkText } from "../../../components/ui";
 import { collectNewsDisplayTickers } from "../../../news/ticker-symbols";
 import { useInlineTickers } from "../../../state/use-inline-tickers";
+import { isPlainKey } from "../../../utils/keyboard";
 import { wrapTextLines } from "../../../utils/text-wrap";
 
 function hasStoryItems(article: MarketNewsItem | null): boolean {
@@ -196,13 +197,13 @@ export function NewsDetailView({ item, focused, width, showTitle = true }: {
 
   useShortcut((event) => {
     if (!focused) return;
-    if (event.name === "j" || event.name === "down") {
+    if (isPlainKey(event, "j", "down")) {
       event.stopPropagation?.();
       event.preventDefault?.();
       scrollBy(1);
       return;
     }
-    if (event.name === "k" || event.name === "up") {
+    if (isPlainKey(event, "k", "up")) {
       event.stopPropagation?.();
       event.preventDefault?.();
       scrollBy(-1);

package/src/plugins/builtin/news-wire/news-preset-pane.tsx

@@ -45,6 +45,9 @@ export function NewsPresetPane({
     `${paneKey}:sort`,
     defaultSort,
   );
+  const effectiveSortPreference = columns.includes(sortPreference.columnId)
+    ? sortPreference
+    : defaultSort;
   const loadNewsStory = useLoadNewsStory();
   const { detailArticle, openArticle, closeDetail } = useNewsArticleDetail(articles, loadNewsStory);
   const { readArticleIds, markArticleRead } = useNewsReadState();
@@ -79,7 +82,7 @@ export function NewsPresetPane({
       readArticleIds={readArticleIds}
       selectedArticleId={selectedArticleId}
       setSelectedArticleId={setSelectedArticleId}
-      sortPreference={sortPreference}
+      sortPreference={effectiveSortPreference}
       setSortPreference={setSortPreference}
       onOpenArticle={openArticle}
       onArticleRead={markArticleRead}

package/src/plugins/builtin/news-wire/persisted-articles.ts

@@ -65,10 +65,16 @@ function samePersistedArticles(left: PersistedNewsArticle[], right: PersistedNew
   if (left.length !== right.length) return false;
   return left.every((article, index) => {
     const other = right[index];
-    return other?.id === article.id && other.publishedAt === article.publishedAt;
+    return other?.id === article.id &&
+      other.publishedAt === article.publishedAt &&
+      storyItemsSignature(other.items) === storyItemsSignature(article.items);
   });
 }
 
+function storyItemsSignature(items: PersistedNewsStoryItem[] | undefined): string {
+  return items?.map((item) => `${item.id}:${item.publishedAt}`).join("|") ?? "";
+}
+
 export function usePersistedNewsArticles(key: string, articles: MarketNewsItem[]): MarketNewsItem[] {
   const [persistedArticles, setPersistedArticles] = usePluginPaneState<PersistedNewsArticle[]>(
     key,