@gloom-sh/terminal 0.18.5 → 0.19.1

package/src/architecture/pane-tabs-allowlist.json

@@ -53,6 +53,9 @@
   "src/plugins/builtin/research-search/pane.tsx": {
     "signInRequired || verificationRequired": "Signed out or unverified, every mode is the same sign-in wall."
   },
+  "src/plugins/builtin/scanner/flow-pane.tsx": {
+    "!feed.denied": "Signed out or without Pro, every tab is the same sign-in or plan wall."
+  },
   "src/plugins/builtin/shared/filing-year-reader.tsx": {
     "years.length > 1": "The tabs are the filings on record, known once their list answers; a single filing has no other year to pick."
   },

package/src/architecture/ttl-caches-allowlist.json

@@ -0,0 +1,19 @@
+{
+  "src/plugins/builtin/alerts/metric-model.ts": {
+    "deadline: now < cached.retryAt": "A byte-identical copy of Gloom Cloud's delivery model (metric-parity-fixture.json pins its hash), so it cannot import the helper. Its failed-read backoff is the rejectionMs rule; move it when both copies move together.",
+    "map: new Map<string, { day: string; retryAt: number; result: Promise<MetricReading>; }>()": "A byte-identical copy of Gloom Cloud's delivery model (metric-parity-fixture.json pins its hash), so it cannot import the helper. Its failed-read backoff is the rejectionMs rule; move it when both copies move together."
+  },
+  "src/plugins/builtin/cloud-tweets/table.tsx": {
+    "age: Date.now() - cached.fetchedAt < TWEET_CACHE_TTL_MS": "Feed results kept across tab switches and shown stale while a refetch runs; the entry's fetchedAt also drives the pane's auto-refresh and moves with every appended page, so the stamp is state the view reads.",
+    "map: new Map<string, { data: CloudTweetSearchResponse; fetchedAt: number; hasMore: boolean; }>()": "Feed results kept across tab switches and shown stale while a refetch runs; the entry's fetchedAt also drives the pane's auto-refresh and moves with every appended page, so the stamp is state the view reads."
+  },
+  "src/plugins/builtin/correlation/index.tsx": {
+    "map: new Map<string, { values: DailyClose[]; fetchedAt: number; }>()": "Not a TTL cache: the last good series per range, kept so a failed refresh still draws it. Nothing expires, and fetchedAt is shown with the data."
+  },
+  "src/plugins/builtin/portfolio-list/since-last-seen.ts": {
+    "map: new Map<string, Departure>()": "Not loaded data: what a pane showed when it went out of sight, read once on return against the reader's own away threshold, then used up."
+  },
+  "src/plugins/builtin/sec/command-bar-search.ts": {
+    "map: new Map<string, FocusRequest>()": "Not loaded data: a filing to focus handed from the command bar to the SEC pane, taken on first read and dropped after a few seconds unread."
+  }
+}

package/src/brokers/sync-broker-instance.ts

@@ -32,7 +32,6 @@ export interface SyncBrokerInstanceArgs {
   tickerRepository: AppTickerRepositoryPort;
   existingTickers?: Map<string, TickerRecord>;
   resources?: AppResourceStorePort;
-  persistResolvedBrokerConfig?: boolean;
   signal?: AbortSignal;
   deferPersistence?: boolean;
 }
@@ -69,7 +68,6 @@ export interface SyncBrokerInstancesArgs {
   tickerRepository: AppTickerRepositoryPort;
   existingTickers: Map<string, TickerRecord>;
   resources?: AppResourceStorePort;
-  persistResolvedBrokerConfig?: boolean;
   /** Leave each result's ticker writes to `onResult`, which calls `result.commit()`. */
   deferPersistence?: boolean;
   onResult?: (result: SyncBrokerInstanceResult, instance: BrokerInstanceConfig, previousConfig: AppConfig) => void | Promise<void>;
@@ -168,9 +166,8 @@ async function maybePersistResolvedBrokerConfig(
   config: AppConfig,
   instance: BrokerInstanceConfig,
   broker: BrokerAdapter,
-  persistResolvedBrokerConfig: boolean,
 ): Promise<AppConfig> {
-  if (!persistResolvedBrokerConfig || !broker.getPersistedConfigUpdate) {
+  if (!broker.getPersistedConfigUpdate) {
     return config;
   }
 
@@ -203,7 +200,6 @@ export async function syncBrokerInstance({
   tickerRepository,
   existingTickers,
   resources,
-  persistResolvedBrokerConfig = false,
   signal,
   deferPersistence = false,
 }: SyncBrokerInstanceArgs): Promise<SyncBrokerInstanceResult> {
@@ -336,7 +332,7 @@ export async function syncBrokerInstance({
     : []);
 
   nextConfig = removeStaleBrokerPortfolios(nextConfig, instance.id, currentPortfolioIds);
-  nextConfig = await maybePersistResolvedBrokerConfig(nextConfig, instance, broker, persistResolvedBrokerConfig);
+  nextConfig = await maybePersistResolvedBrokerConfig(nextConfig, instance, broker);
   throwIfBrokerImportCancelled(signal);
   nextConfig = markBrokerInstanceSynced(nextConfig, instance.id, syncedAt);
   const syncedConfig = nextConfig;
@@ -436,7 +432,6 @@ export async function syncBrokerInstances({
   tickerRepository,
   existingTickers,
   resources,
-  persistResolvedBrokerConfig = false,
   deferPersistence = false,
   onResult,
 }: SyncBrokerInstancesArgs): Promise<SyncBrokerInstancesResult> {
@@ -457,7 +452,6 @@ export async function syncBrokerInstances({
         tickerRepository,
         existingTickers: nextTickers,
         resources,
-        persistResolvedBrokerConfig,
         deferPersistence,
       });
 

package/src/capabilities/chart-series.ts

@@ -28,7 +28,7 @@ const SERIES_ID_PATTERN = /^[A-Za-z0-9][A-Za-z0-9._~%:/-]{0,239}$/;
 const chartSeriesLog = debugLog.createLogger("chart-series");
 const ranges = new Set<string>(TIME_RANGES);
 const periods = new Set(["auto", "daily", "weekly", "monthly", "quarterly", "annual", "ttm"]);
-const styles = new Set(["line", "area", "step", "columns", "points", "candles", "ohlc", "hlc"]);
+const styles = new Set(["line", "area", "step", "columns", "points", "candles", "hollow", "heikin-ashi", "ohlc", "hlc"]);
 const transforms = new Set(["raw", "percent", "index100", "yoy", "qoq", "log"]);
 const shapes = new Set(["scalar", "ohlcv", "event", "band"]);
 const axes = new Set(["left", "right"]);

package/src/cli/commands/crypto-hints.ts

@@ -1,7 +1,7 @@
 import { isCryptoInstrumentType } from "../../tickers/search/ranking";
 import { isCryptoVenuePerpetual } from "../../tickers/instrument-kind";
 import type { Quote } from "../../types/financials";
-import { isCryptoPairSymbol, MAJOR_COIN_NAMES } from "../../utils/crypto-pair";
+import { isCryptoPairSymbol, TOP_COIN_NAMES } from "../../utils/crypto-pair";
 import { isNotATickerMessage } from "../not-a-ticker";
 import { groupByCompany } from "../../tickers/company-name";
 import { DATA_SERVICE_UNREACHABLE_MESSAGE } from "../../utils/network-failure";
@@ -10,27 +10,49 @@ import type { TickerRecord } from "../../types/ticker";
 /** Where a crypto pair that will not load can still be read. `fn CRYP` takes no argument. */
 export const CRYPTO_BOARD_HINT = "Crypto prices may be briefly unavailable; the CRYP function (gloom fn CRYP) shows the crypto board.";
 
-// How the note names the coin, and what a listing's name has to say before the note applies.
-const MAJOR_COINS = MAJOR_COIN_NAMES;
+// How the note names the coin, and which tickers it speaks for.
+const MAJOR_COINS = TOP_COIN_NAMES;
+
+/** What a coin note reads of a listing: its quote's, or what a saved listing keeps. */
+interface CoinListing {
+  name?: string | null;
+  instrumentType?: string | null;
+  /** The feed says the price is old. */
+  stale?: boolean;
+  /** The exchange removed the listing, or its last trade is all that is known. */
+  delisting?: unknown;
+}
 
 /**
- * A bare coin ticker usually resolves to a listed fund, not the coin. The note applies only when
- * the listing is not crypto and its own name says the coin (a "Bitcoin" trust), so a company that
- * happens to share the letters (Interlink Electronics on LINK) is left alone. Every command that
- * reads a bare symbol as a listing says it: quote, ticker, and the portfolio and watchlist adds.
+ * A bare coin ticker usually resolves to a listing that is not the coin. The note applies to a
+ * top coin's symbol whenever the listing is not a crypto product, and says how sure it is:
+ *
+ * - "SOL is Emeren Group ..., not solana. For solana use SOL-USD." when the listing cannot be
+ *   what was meant: its name says the coin (a "Bitcoin" trust), or it is stale or delisted
+ *   (SOL's Emeren Group last traded months ago).
+ * - "LINK is Interlink Electronics, Inc. Also a coin: LINK-USD." for a live company that
+ *   happens to share the letters, which may well be the one meant.
+ *
+ * Every command that reads a bare symbol as a listing says it: quote, ticker, history, options,
+ * news, charts, alerts, and the portfolio and watchlist adds.
  */
-export function notTheCoinNote(symbol: string, quote: Pick<Quote, "name" | "instrumentType"> | null | undefined): string | null {
+export function notTheCoinNote(symbol: string, listing: CoinListing | null | undefined): string | null {
   const coin = MAJOR_COINS.get(symbol);
-  const type = quote?.instrumentType?.trim();
-  const name = quote?.name?.trim();
+  const type = listing?.instrumentType?.trim();
+  const name = listing?.name?.trim();
   if (!coin || !name || !type || isCryptoInstrumentType(type)) return null;
-  if (!new RegExp(`\\b${coin}\\b`, "i").test(name)) return null;
-  return `${symbol} is ${name}, not ${coin}. For ${coin} use ${symbol}-USD.`;
+  const cannotBeIt = listing?.stale === true || !!listing?.delisting || new RegExp(`\\b${coin}\\b`, "i").test(name);
+  return cannotBeIt
+    ? `${symbol} is ${name}, not ${coin}. For ${coin} use ${symbol}-USD.`
+    : `${symbol} is ${name}${/[.!?]$/.test(name) ? "" : "."} Also a coin: ${symbol}-USD.`;
 }
 
-/** `notTheCoinNote` for a saved listing, by the name and asset category it was saved with. */
-export function savedListingCoinNote(symbol: string, ticker: Pick<TickerRecord, "metadata">): string | null {
-  return notTheCoinNote(symbol, { name: ticker.metadata.name, instrumentType: ticker.metadata.assetCategory });
+/**
+ * `notTheCoinNote` for a saved listing, by the name and asset category it was saved with.
+ * `ended` says the listing stopped trading, which the saved record does not keep.
+ */
+export function savedListingCoinNote(symbol: string, ticker: Pick<TickerRecord, "metadata">, { ended = false }: { ended?: boolean } = {}): string | null {
+  return notTheCoinNote(symbol, { name: ticker.metadata.name, instrumentType: ticker.metadata.assetCategory, stale: ended });
 }
 
 /**
@@ -70,10 +92,10 @@ function failureReason(message: string, symbol: string): string {
 
 /**
  * Text-mode notes under a quote table: one line per distinct failure reason (with the crypto board
- * pointer when a failed symbol is a crypto pair, unless nothing could be reached), then a line for each bare coin ticker that
- * resolved to something else. `--exchange` names the listing on purpose, so it gets no coin note.
+ * pointer when a failed symbol is a crypto pair, unless nothing could be reached), then the perpetual futures line.
+ * A bare coin ticker read as another listing is said with the listing (`listingChoiceNotes`).
  */
-export function quoteNotes(results: QuoteNoteResult[], options: { exchange?: string } = {}): string[] {
+export function quoteNotes(results: QuoteNoteResult[]): string[] {
   const failures = new Map<string, string[]>();
   for (const result of results) {
     if (!result.error || result.quote) continue;
@@ -87,12 +109,6 @@ export function quoteNotes(results: QuoteNoteResult[], options: { exchange?: str
       ? reason
       : `${symbols.join(", ")}: ${reason}${symbols.some(isCryptoPairSymbol) && reason !== DATA_SERVICE_UNREACHABLE_MESSAGE ? `${/[.!?]$/.test(reason) ? "" : "."} ${CRYPTO_BOARD_HINT}` : ""}`
   ));
-  if (!options.exchange) {
-    for (const result of results) {
-      const note = notTheCoinNote(result.target.symbol, result.quote);
-      if (note) notes.push(note);
-    }
-  }
   const perpetuals = results
     .filter((result) => result.quote && isCryptoVenuePerpetual(result.quote.instrumentType, result.quote.name))
     .map((result) => result.target.symbol);