gloomberb 0.9.8 → 0.9.10

package/src/components/price-selector-dialog.tsx

@@ -87,7 +87,7 @@ export function PriceSelectorDialog({
   const inputRef = useRef<InputRenderable>(null);
 
   useEffect(() => {
-    if (mode === "input") inputRef.current?.focus();
+    if (mode === "input") inputRef.current?.focus?.();
   }, [mode]);
 
   useDialogKeyboard((event) => {
@@ -108,7 +108,7 @@ export function PriceSelectorDialog({
           setMode("input");
           setIndex(CUSTOM_INDEX);
         }
-      } else if (event.name === "return") {
+      } else if (event.name === "enter" || event.name === "return") {
         if (presets[index]) {
           resolve(String(presets[index].value));
         }
@@ -124,7 +124,7 @@ export function PriceSelectorDialog({
       if (event.name === "up" || (event.name === "tab" && presets.length > 0)) {
         setMode("list");
         setIndex(presets.length - 1);
-      } else if (event.name === "return") {
+      } else if (event.name === "enter" || event.name === "return") {
         resolve(customValue.trim());
       }
     }

package/src/components/ticker/badge/text.tsx

@@ -4,7 +4,6 @@ import { TickerBadge } from "./index";
 import { ExternalLinkText, openUrl } from "../../ui";
 import { tokenizeInlineContent } from "../../../utils/inline-content-tokenizer";
 import type { InlineTickerCatalogEntry } from "../../../state/hooks/inline-tickers";
-import { displayWidth } from "../../../utils/format";
 import { splitLongTextSegmentByDisplayWidth } from "../../../utils/text-wrap";
 import { blendHex, colors } from "../../../theme/colors";
 

package/src/components/ui/page-stack-view.tsx

@@ -88,7 +88,7 @@ export function PageStackView({
     >
       <Box height={1} flexDirection="row">
         <Box
-          onMouseDown={(event) => {
+          onMouseDown={(event: { preventDefault?: () => void; stopPropagation?: () => void }) => {
             goBack(event);
           }}
           backgroundColor={colors.selected}

package/src/components/ui/ui.test.tsx

@@ -1,7 +1,6 @@
 import { afterEach, describe, expect, test } from "bun:test";
 import { act, useEffect, useRef, useState } from "react";
 import { TestDialogProvider, testRender } from "../../renderers/opentui/test-utils";
-import { TextAttributes } from "../../ui";
 import type { BoxRenderable, ScrollBoxRenderable } from "@opentui/core";
 import { ChoiceDialog } from "./choice-dialog";
 import { DataTable, type DataTableColumn } from "./data-table";
@@ -171,37 +170,6 @@ function DataTableHorizontalScrollHarness({
   );
 }
 
-function DataTableColumnGapHarness() {
-  const headerScrollRef = useRef<ScrollBoxRenderable>(null);
-  const scrollRef = useRef<ScrollBoxRenderable>(null);
-  const items = [{ id: "row-0", left: "AA", right: "BB" }];
-
-  return (
-    <DataTableView
-      rootWidth={8}
-      rootHeight={4}
-      selection={{ kind: "none" }}
-      columns={[
-        { id: "left", label: "L", width: 2, align: "left" },
-        { id: "right", label: "R", width: 2, align: "left" },
-      ]}
-      items={items}
-      sortColumnId={null}
-      sortDirection="asc"
-      onHeaderClick={() => {}}
-      headerScrollRef={headerScrollRef}
-      scrollRef={scrollRef}
-      getItemKey={(row) => row.id}
-      renderCell={(row, column) => ({ text: column.id === "left" ? row.left : row.right })}
-      emptyStateTitle="No rows."
-      columnGap={0}
-      horizontalPadding={0}
-      fillAvailableWidth={false}
-      showHorizontalScrollbar={false}
-    />
-  );
-}
-
 function DataTableVirtualizationHarness() {
   const headerScrollRef = useRef<ScrollBoxRenderable>(null);
   const scrollRef = useRef<ScrollBoxRenderable>(null);
@@ -392,31 +360,6 @@ describe("shared UI kit", () => {
     expect(lastSelected).toBe("chart");
   });
 
-  test("marks the active bare tab when it has keyboard focus", async () => {
-    testSetup = await testRender(
-      <Tabs
-        tabs={[
-          { label: "One", value: "one" },
-          { label: "Two", value: "two" },
-        ]}
-        activeValue="one"
-        onSelect={() => {}}
-        variant="bare"
-        compact
-        focused
-      />,
-      { width: 24, height: 2 },
-    );
-
-    await act(async () => {
-      await testSetup!.renderOnce();
-    });
-
-    const activeSpan = testSetup.captureSpans().lines[0]?.spans.find((span) => span.text.includes("One"));
-    expect(activeSpan).toBeDefined();
-    expect((activeSpan!.attributes & TextAttributes.UNDERLINE)).toBe(TextAttributes.UNDERLINE);
-  });
-
   test("selects tabs by clicking their text labels", async () => {
     let lastSelected = "overview";
     const selectedValues: string[] = [];
@@ -794,27 +737,6 @@ describe("shared UI kit", () => {
     expect(tableScrollBoxForTest?.horizontalScrollBar.visible).toBe(false);
   });
 
-  test("renders data table columns without spacer cells when column gap is zero", async () => {
-    const state = createInitialState(createDefaultConfig("/tmp/gloomberb-test"));
-    testSetup = await testRender(
-      <AppContext value={{ state, dispatch: () => {} }}>
-        <PaneInstanceProvider paneId="portfolio-list:main">
-          <DataTableColumnGapHarness />
-        </PaneInstanceProvider>
-      </AppContext>,
-      { width: 8, height: 4 },
-    );
-
-    await act(async () => {
-      await testSetup!.renderOnce();
-      await testSetup!.renderOnce();
-    });
-
-    const frame = testSetup.captureCharFrame();
-    expect(frame).toContain("AABB");
-    expect(frame).not.toContain("AA BB");
-  });
-
   test("hides data table horizontal scrolling when content fits", async () => {
     const state = createInitialState(createDefaultConfig("/tmp/gloomberb-test"));
     tableScrollBoxForTest = null;

package/src/core/state/app/layout-reducer.ts

@@ -2,7 +2,6 @@ import { cloneLayout, createBlankLayout, type SavedLayout } from "../../../types
 import {
   clonePaneStateMap,
   cloneSavedLayout,
-  getActiveSavedPaneState,
   historyForIndex,
   removeHistoryIndex,
   setHistoryForIndex,

package/src/data/config/layout.ts

@@ -60,9 +60,9 @@ function sanitizePlacementMemory(value: unknown): PanePlacementMemory | undefine
   const docked = (() => {
     const raw = (value as PanePlacementMemory).docked;
     if (!raw || typeof raw !== "object") return undefined;
-    const path = Array.isArray((raw as { path?: unknown }).path)
-      ? (raw as { path?: unknown }).path
-        ?.filter((segment): segment is 0 | 1 => segment === 0 || segment === 1)
+    const rawPath = (raw as { path?: unknown }).path;
+    const path = Array.isArray(rawPath)
+      ? rawPath.filter((segment): segment is 0 | 1 => segment === 0 || segment === 1)
       : undefined;
     const anchorInstanceId = typeof (raw as { anchorInstanceId?: unknown }).anchorInstanceId === "string"
       ? (raw as { anchorInstanceId: string }).anchorInstanceId

package/src/data/resource-store.ts

@@ -291,6 +291,12 @@ export class ResourceStore {
       }
       records.push(record);
     }
+    if (options.touch !== false) {
+      for (const record of records) {
+        const touchedAt = this.touch(record);
+        if (touchedAt != null) record.lastAccessedAt = touchedAt;
+      }
+    }
     return records;
   }
 

package/src/data/sqlite/cache.test.ts

@@ -35,6 +35,7 @@ describe("AppPersistence", () => {
     }, {
       cachePolicy: { staleMs: 60_000, expireMs: 120_000 },
       schemaVersion: 1,
+      fetchedAt: 1,
     });
 
     expect(persistence.resources.get<{ price: number }>({
@@ -43,6 +44,10 @@ describe("AppPersistence", () => {
       entityKey: "AAPL",
       sourceKey: "provider:yahoo",
     }, { allowExpired: true })?.value.price).toBe(123);
+    const listed = persistence.resources.list<{ price: number }>({
+      namespace: "market", kind: "quote", entityKey: "AAPL",
+    }, { allowExpired: true });
+    expect(listed[0]?.lastAccessedAt).toBeGreaterThan(1);
     persistence.close();
   });
 

package/src/data/ticker-store.ts

@@ -47,11 +47,4 @@ export class TickerStore {
       this.db.query("DELETE FROM tickers WHERE symbol = ?").run(symbol);
     });
   }
-
-  count(): number {
-    const row = withSqliteBusyRetry("count tickers", () => (
-      this.db.query<{ count: number }, []>("SELECT COUNT(*) as count FROM tickers").get()
-    ));
-    return row?.count ?? 0;
-  }
 }

package/src/market-data/coordinator/auxiliary.ts

@@ -1,14 +1,12 @@
 import type { DataProvider, SecFilingDocument, SecFilingItem } from "../../types/data-provider";
-import type { NewsArticle } from "../../news/types";
 import type { OptionsChain } from "../../types/financials";
-import type { NewsRequest, OptionsRequest, SecFilingsRequest } from "../request-types";
+import type { OptionsRequest, SecFilingsRequest } from "../request-types";
 import type { QueryEntry } from "../result-types";
 import { QueryStore } from "../query-store";
 import {
   ARTICLE_SUMMARY_CACHE_TTL_MS,
   EXPECTED_EMPTY,
   FX_CACHE_TTL_MS,
-  NEWS_CACHE_TTL_MS,
   OPTIONS_CACHE_TTL_MS,
   SEC_CONTENT_CACHE_TTL_MS,
   SEC_FILINGS_CACHE_TTL_MS,
@@ -23,7 +21,6 @@ import {
 import {
   buildArticleSummaryKey,
   buildFxKey,
-  buildNewsKey,
   buildOptionsKey,
   buildSecContentKey,
   buildSecDocumentsKey,
@@ -42,46 +39,6 @@ interface AuxiliaryLoaderOptions<T> {
   load: (startedAt: number) => Promise<QueryEntry<T>>;
 }
 
-interface NewsDataProvider extends DataProvider {
-  getNews?: (query: {
-    feed: "ticker";
-    ticker: string;
-    exchange?: string;
-    tickerTier: "primary";
-    limit?: number;
-  }) => Promise<NewsArticle[]>;
-}
-
-export function loadNewsEntry(options: {
-  dataProvider: DataProvider;
-  request: NewsRequest;
-  store: QueryStore<NewsArticle[]>;
-  runSingleFlight: RunSingleFlight;
-}): Promise<QueryEntry<NewsArticle[]>> {
-  const { dataProvider, request, store, runSingleFlight } = options;
-  const key = buildNewsKey(request);
-  return loadAuxiliaryEntry({
-    dataProvider,
-    key,
-    store,
-    ttlMs: NEWS_CACHE_TTL_MS,
-    runSingleFlight,
-    load: async (startedAt) => {
-      const newsProvider = dataProvider as NewsDataProvider;
-      if (!newsProvider.getNews) throw new Error("No news provider available");
-      const data = await newsProvider.getNews({
-        feed: "ticker",
-        ticker: request.instrument.symbol,
-        exchange: request.instrument.exchange ?? "",
-        tickerTier: "primary",
-        limit: request.count ?? 50,
-      });
-      const attempts = [createAttempt(dataProvider.id, startedAt, data.length > 0 ? "success" : "empty", data.length === 0 ? "NO_DATA" : undefined)];
-      return store.update(key, (current) => readyEntry(current, data.length > 0 ? data : null, dataProvider.id, attempts, { keepLastGoodOnEmpty: true }));
-    },
-  });
-}
-
 export function loadOptionsEntry(options: {
   dataProvider: DataProvider;
   request: OptionsRequest;

package/src/market-data/coordinator/entries.ts

@@ -8,7 +8,6 @@ const EMPTY_MESSAGE = "No data available";
 export const EXPECTED_EMPTY = /no data|not found|delisted|unavailable|unsupported/i;
 export const SNAPSHOT_CACHE_TTL_MS = 5 * 60_000;
 export const CHART_CACHE_TTL_MS = 10 * 60_000;
-export const NEWS_CACHE_TTL_MS = 2 * 60_000;
 export const OPTIONS_CACHE_TTL_MS = 10 * 60_000;
 export const SEC_FILINGS_CACHE_TTL_MS = 10 * 60_000;
 export const SEC_CONTENT_CACHE_TTL_MS = 24 * 60 * 60_000;

package/src/market-data/coordinator/index.test.ts

@@ -155,13 +155,8 @@ describe("MarketDataCoordinator", () => {
   });
 
   it("reuses fresh empty tab query results instead of refetching on reopen", async () => {
-    let newsCalls = 0;
     let optionsCalls = 0;
     const provider = createProvider({
-      getNews: async () => {
-        newsCalls += 1;
-        return [];
-      },
       getOptionsChain: async () => {
         optionsCalls += 1;
         return {
@@ -175,16 +170,11 @@ describe("MarketDataCoordinator", () => {
     const coordinator = new MarketDataCoordinator(provider);
     const instrument = { symbol: "AAPL", exchange: "NASDAQ" };
 
-    const firstNews = await coordinator.loadNews({ instrument, count: 20 });
-    const secondNews = await coordinator.loadNews({ instrument, count: 20 });
     const firstOptions = await coordinator.loadOptions({ instrument });
     const secondOptions = await coordinator.loadOptions({ instrument });
 
-    expect(firstNews.error?.reasonCode).toBe("NO_DATA");
-    expect(secondNews.error?.reasonCode).toBe("NO_DATA");
     expect(firstOptions.error?.reasonCode).toBe("NO_DATA");
     expect(secondOptions.error?.reasonCode).toBe("NO_DATA");
-    expect(newsCalls).toBe(1);
     expect(optionsCalls).toBe(1);
   });
 

package/src/market-data/coordinator/index.ts

@@ -1,7 +1,6 @@
 import type { DataProvider, SecFilingDocument, SecFilingItem } from "../../types/data-provider";
-import type { NewsArticle } from "../../news/types";
 import type { OptionsChain, PricePoint, Quote, TickerFinancials } from "../../types/financials";
-import type { ChartRequest, InstrumentRef, NewsRequest, OptionsRequest, SecFilingsRequest } from "../request-types";
+import type { ChartRequest, InstrumentRef, OptionsRequest, SecFilingsRequest } from "../request-types";
 import { QueryStore } from "../query-store";
 import type { QueryEntry } from "../result-types";
 import {
@@ -9,7 +8,6 @@ import {
   buildChartKey,
   buildFxKey,
   buildTickerFinancialsSnapshot,
-  buildNewsKey,
   buildOptionsKey,
   buildQuoteKey,
   buildSecContentKey,
@@ -41,7 +39,6 @@ import {
 import {
   loadArticleSummaryEntry,
   loadFxRateEntry,
-  loadNewsEntry,
   loadOptionsEntry,
   loadSecFilingContentEntry,
   loadSecFilingDocumentsEntry,
@@ -73,7 +70,6 @@ export class MarketDataCoordinator {
   private readonly fundamentalsStore = new QueryStore<TickerFinancials["fundamentals"]>((key) => this.events.bump(key));
   private readonly statementsStore = new QueryStore<Pick<TickerFinancials, "annualStatements" | "quarterlyStatements">>((key) => this.events.bump(key));
   private readonly chartStore = new QueryStore<PricePoint[]>((key) => this.events.bump(key));
-  private readonly newsStore = new QueryStore<NewsArticle[]>((key) => this.events.bump(key));
   private readonly optionsStore = new QueryStore<OptionsChain>((key) => this.events.bump(key));
   private readonly secFilingsStore = new QueryStore<SecFilingItem[]>((key) => this.events.bump(key));
   private readonly secDocumentsStore = new QueryStore<SecFilingDocument[]>((key) => this.events.bump(key));
@@ -124,10 +120,6 @@ export class MarketDataCoordinator {
     return this.chartStore.get(buildChartKey(request));
   }
 
-  getNewsEntry(request: NewsRequest): QueryEntry<NewsArticle[]> {
-    return this.newsStore.get(buildNewsKey(request));
-  }
-
   getOptionsEntry(request: OptionsRequest): QueryEntry<OptionsChain> {
     return this.optionsStore.get(buildOptionsKey(request));
   }
@@ -311,15 +303,6 @@ export class MarketDataCoordinator {
     });
   }
 
-  async loadNews(request: NewsRequest): Promise<QueryEntry<NewsArticle[]>> {
-    return loadNewsEntry({
-      dataProvider: this.dataProvider,
-      request,
-      store: this.newsStore,
-      runSingleFlight: (key, task) => this.runSingleFlight(key, task),
-    });
-  }
-
   async loadOptions(request: OptionsRequest): Promise<QueryEntry<OptionsChain>> {
     return loadOptionsEntry({
       dataProvider: this.dataProvider,

package/src/market-data/coordinator/quotes.ts

@@ -228,7 +228,6 @@ export class QuoteSubscriptionManager {
   private nextQuoteSubscriptionId = 1;
   private quoteSubscriptionDispose: (() => void) | null = null;
   private quoteSubscriptionSignature = "";
-  private pendingQuoteSubscriptionKeys = new Set<string>();
 
   constructor(
     private readonly dataProvider: DataProvider,
@@ -250,7 +249,6 @@ export class QuoteSubscriptionManager {
       if (existing) {
         existing.targets.set(subscriptionId, target);
         existing.target = mergeQuoteSubscriptionTargets(existing.targets.values()) ?? target;
-        this.pendingQuoteSubscriptionKeys.add(key);
         if (existing.removeTimer) {
           clearTimeout(existing.removeTimer);
           existing.removeTimer = null;
@@ -262,7 +260,6 @@ export class QuoteSubscriptionManager {
         targets: new Map([[subscriptionId, target]]),
         removeTimer: null,
       });
-      this.pendingQuoteSubscriptionKeys.add(key);
     }
     this.flush();
 
@@ -274,7 +271,6 @@ export class QuoteSubscriptionManager {
         const mergedTarget = mergeQuoteSubscriptionTargets(existing.targets.values());
         if (mergedTarget) {
           existing.target = mergedTarget;
-          this.pendingQuoteSubscriptionKeys.add(key);
           this.flush();
           continue;
         }
@@ -283,7 +279,6 @@ export class QuoteSubscriptionManager {
           const current = this.quoteSubscriptions.get(key);
           if (!current || current.targets.size > 0) return;
           this.quoteSubscriptions.delete(key);
-          this.pendingQuoteSubscriptionKeys.delete(key);
           this.flush();
         }, QUOTE_SUBSCRIPTION_REMOVE_GRACE_MS);
       }
@@ -292,7 +287,6 @@ export class QuoteSubscriptionManager {
 
   private flush(): void {
     if (!this.dataProvider.subscribeQuotes) return;
-    this.pendingQuoteSubscriptionKeys = new Set();
 
     const activeEntries = [...this.quoteSubscriptions.entries()]
       .filter(([, entry]) => entry.targets.size > 0 || entry.removeTimer)

package/src/market-data/coordinator/subscriptions.test.ts

@@ -40,6 +40,7 @@ function quote(symbol: string, price: number, overrides: Partial<Quote> = {}): Q
 
 async function flushCoordinator(): Promise<void> {
   await Promise.resolve();
+  await new Promise((resolve) => setTimeout(resolve, 0));
 }
 
 describe("MarketDataCoordinator key subscriptions", () => {

package/src/market-data/market/status.ts

@@ -1,5 +1,7 @@
 import type { MarketState, Quote, QuoteFieldProvenance } from "../../types/financials";
-import { colors } from "../../theme/colors";
+import { blendHex, colors, priceColor } from "../../theme/colors";
+
+const CLOSED_CHANGE_MUTING_RATIO = 0.55;
 
 export interface ActiveQuoteDisplay {
   price: number;
@@ -45,6 +47,22 @@ export function marketStateDot(state?: MarketState): string {
   }
 }
 
+export function isClosedMarketState(state?: MarketState): boolean {
+  return state === "CLOSED" || state === "PREPRE" || state === "POSTPOST";
+}
+
+/** Closed prices are final snapshots, so they should not look live or directional. */
+export function marketPriceColor(change: number, state?: MarketState): string {
+  return isClosedMarketState(state) ? colors.textDim : priceColor(change);
+}
+
+/** Preserve a closed session's direction while visually distinguishing it from a live move. */
+export function marketChangeColor(change: number, state?: MarketState): string {
+  const directionalColor = priceColor(change);
+  if (!isClosedMarketState(state) || change === 0) return directionalColor;
+  return blendHex(directionalColor, colors.textDim, CLOSED_CHANGE_MUTING_RATIO);
+}
+
 /** Short exchange display name */
 export function exchangeShortName(exchangeName?: string, fullExchangeName?: string): string {
   if (!exchangeName && !fullExchangeName) return "";

package/src/market-data/query-store.ts

@@ -21,8 +21,4 @@ export class QueryStore<T> {
     this.onChange(key);
     return next;
   }
-
-  values(): IterableIterator<QueryEntry<T>> {
-    return this.entries.values();
-  }
 }

package/src/market-data/quotes/resolution.ts

@@ -147,7 +147,7 @@ function assignField(
   quote: QuoteContribution | undefined,
 ): void {
   if (!quote) return;
-  const value = (quote as Record<string, unknown>)[field];
+  const value = (quote as unknown as Record<string, unknown>)[field];
   if (value === undefined) return;
   (target as Record<string, unknown>)[field] = value;
   provenance.fields ??= {};
@@ -161,7 +161,7 @@ function pickField(
   candidates: QuoteContribution[],
 ): QuoteContribution | undefined {
   for (const candidate of candidates) {
-    if ((candidate as Record<string, unknown>)[field] === undefined) continue;
+    if ((candidate as unknown as Record<string, unknown>)[field] === undefined) continue;
     assignField(target, provenance, field, candidate);
     return candidate;
   }

package/src/market-data/request-types.ts

@@ -28,11 +28,6 @@ export interface ChartRequest {
   barSize?: string;
 }
 
-export interface NewsRequest {
-  instrument: InstrumentRef;
-  count?: number;
-}
-
 export interface OptionsRequest {
   instrument: InstrumentRef;
   expirationDate?: number;

package/src/market-data/result-types.ts

@@ -19,17 +19,6 @@ export interface ProviderAttempt {
   message?: string;
 }
 
-interface ProviderResult<T> {
-  status: ProviderStatus;
-  data: T | null;
-  providerId: string;
-  latencyMs: number;
-  completeness: "full" | "partial";
-  reasonCode?: ProviderReasonCode;
-  asOf?: number;
-  staleAt?: number;
-}
-
 type QueryPhase = "idle" | "loading" | "ready" | "refreshing" | "error";
 
 export interface QueryEntry<T> {

package/src/market-data/selectors.ts

@@ -1,6 +1,6 @@
 import type { MarketDataRequestContext } from "../types/data-provider";
 import type { Quote, TickerFinancials } from "../types/financials";
-import type { InstrumentRef, NewsRequest, OptionsRequest, SecFilingsRequest, ChartRequest } from "./request-types";
+import type { InstrumentRef, OptionsRequest, SecFilingsRequest, ChartRequest } from "./request-types";
 import type { QueryEntry } from "./result-types";
 import { hasLikelyQuoteUnitMismatch } from "../utils/currency-units";
 import { normalizePriceHistory } from "../utils/price-history";
@@ -61,10 +61,6 @@ export function buildChartKey(request: ChartRequest): string {
   ].join(":");
 }
 
-export function buildNewsKey(request: NewsRequest): string {
-  return `news:${buildInstrumentKey(request.instrument)}:${request.count ?? 50}`;
-}
-
 export function buildOptionsKey(request: OptionsRequest): string {
   return `options:${buildInstrumentKey(request.instrument)}:${request.expirationDate ?? "default"}`;
 }

package/src/news/aggregator.ts

@@ -58,7 +58,11 @@ export class NewsService {
     if (this.pollTimer !== null) {
       void this.pollActiveQueries();
     }
-    return () => this.unregister(source.id);
+    return () => {
+      if (this.sources.get(source.id) === source) {
+        this.unregister(source.id);
+      }
+    };
   }
 
   unregister(sourceId: string): void {

package/src/pane-settings.test.ts

@@ -3,24 +3,6 @@ import { cloneLayout, createDefaultConfig, createPaneInstance, DEFAULT_PORTFOLIO
 import { deletePaneSetting, getPaneSettingValue, setPaneSetting } from "./pane-settings";
 
 describe("pane settings helpers", () => {
-  test("seeds built-in default panes with persisted settings", () => {
-    const config = createDefaultConfig("/tmp/gloomberb-pane-settings");
-    const portfolioPane = config.layout.instances.find((instance) => instance.instanceId === "portfolio-list:main");
-    const detailPane = config.layout.instances.find((instance) => instance.instanceId === "ticker-detail:main");
-
-    expect(portfolioPane?.settings).toMatchObject({
-      collectionScope: "all",
-      viewMode: "table",
-    });
-    expect(portfolioPane?.settings?.columnIds).toEqual(DEFAULT_PORTFOLIO_COLUMN_IDS);
-    expect(detailPane?.settings).toEqual({
-      chartRangePreset: "5Y",
-      chartResolution: "auto",
-      hideTabs: false,
-      lockedTabId: "overview",
-    });
-  });
-
   test("cloneLayout deep-clones pane settings", () => {
     const layout = cloneLayout(createDefaultConfig("/tmp/gloomberb-pane-settings").layout);
     const cloned = cloneLayout(layout);