gloomberb 0.18.0 → 0.18.1

package/src/plugins/builtin/executives/pane.tsx

@@ -27,11 +27,14 @@ import {
 } from "../../../ui";
 import { isPermanentClientError } from "../../../api-client/errors";
 import type { IssuerListingParams } from "../../../api-client/paths";
+import { usePlanAccess } from "../../../api-client/plan-access";
+import { CLOUD_PLAN_KEY, useCloudUpgradeAction } from "../shared/cloud-upgrade";
 import { useFilingYearReader } from "../shared/filing-year-reader";
+import { UpgradeLabel } from "../shared/locked-rows";
 import { useProFeatureWall, type ProFeatureWallCopy, type ReadGuard } from "../shared/pro-feature-wall";
 import { useBoundTicker, useIssuerListing } from "../shared/ticker-request";
 import { SplitBar } from "../../../components/ui/split-bar";
-import { loadProxyStatement, loadProxyStatements } from "./data";
+import { loadProxyStatement, loadProxyStatements, type ProxyTier } from "./data";
 import {
   equityShare,
   formatChange,
@@ -127,9 +130,12 @@ function renderOfficerCell(row: CloudExecutiveRowPayload, column: OfficerColumn)
 function ExecutiveRows({
   rows,
   width,
+  breakdown,
 }: {
   rows: CloudExecutiveRowPayload[];
   width: number;
+  /** False on the free preview, whose rows carry the total alone: no EQ% column. */
+  breakdown: boolean;
 }) {
   const isDesktopWeb = useUiCapabilities().nativePaneChrome === true;
   if (width < 52) {
@@ -146,7 +152,9 @@ function ExecutiveRows({
   }
   // The name column fits the longest name so the title starts after a kit gutter.
   const nameWidth = Math.min(24, Math.max(10, ...rows.map((row) => row.name.length)));
-  const columns = OFFICER_COLUMNS.map((column) => column.id === "name" ? { ...column, width: nameWidth } : column);
+  const columns = OFFICER_COLUMNS
+    .filter((column) => breakdown || column.id !== "equity")
+    .map((column) => column.id === "name" ? { ...column, width: nameWidth } : column);
   const height = rows.length + 1;
   return (
     <Box flexDirection="column" height={isDesktopWeb ? undefined : height} flexShrink={0}>
@@ -251,22 +259,26 @@ export function ExecutivesPane({
   /** Inside Ticker Research, whose own tab strip keeps h/l and the arrows. */
   nested?: boolean;
 }) {
-  const { wall, guard } = useProFeatureWall(EXEC_WALL, { width, height });
+  // Every plan reads EXEC: without Pro the server serves each executive's
+  // total pay, so only its refusal (an older server, an expired session) walls it.
+  const { wall, guard } = useProFeatureWall(EXEC_WALL, { width, height }, { preview: true });
+  const tier: ProxyTier = usePlanAccess().hasProAccess ? "full" : "preview";
   const { symbol, ticker: boundTicker } = useBoundTicker();
   const ticker = symbol ? symbol.toUpperCase() : null;
   const listing = useIssuerListing(boundTicker);
   if (wall) return wall;
   if (!ticker) return <EmptyState title="Select a ticker." />;
-  return <ExecutiveResearch key={ticker} ticker={ticker} listing={listing} focused={focused} width={width} nested={nested} guard={guard} />;
+  return <ExecutiveResearch key={ticker} ticker={ticker} listing={listing} tier={tier} focused={focused} width={width} nested={nested} guard={guard} />;
 }
 
 export function ExecutivesResearchTab(props: { focused: boolean; width: number; height: number }) {
   return <ExecutivesPane {...props} nested />;
 }
 
-function ExecutiveResearch({ ticker, listing, focused, width, nested, guard }: {
+function ExecutiveResearch({ ticker, listing, tier, focused, width, nested, guard }: {
   ticker: string;
   listing: IssuerListingParams;
+  tier: ProxyTier;
   focused: boolean;
   width: number;
   nested: boolean;
@@ -274,15 +286,20 @@ function ExecutiveResearch({ ticker, listing, focused, width, nested, guard }: {
 }) {
   const nativePaneChrome = useUiCapabilities().nativePaneChrome === true;
   const rendererHost = useRendererHost();
+  const openUpgrade = useCloudUpgradeAction("exec");
   const [selectedYear, setYear] = usePaneStateValue<number | null>("proxyYear", null);
-  const loadYears = useCallback((force: boolean) => guard(loadProxyStatements(ticker, { force, listing })), [guard, listing, ticker]);
+  const loadYears = useCallback((force: boolean) => guard(loadProxyStatements(ticker, { force, listing, tier })), [guard, listing, ticker, tier]);
   const list = useAsyncResource(loadYears, { clearOnError: isPermanentClientError });
   const years = useMemo(() => (list.data?.data?.proxies ?? []).map((entry) => entry.proxyYear), [list.data]);
-  const year = selectedYear !== null && years.includes(selectedYear)
+  // Without Pro only the latest proxy is served: an earlier pick from a Pro
+  // session waits for Pro again rather than asking for a year that is refused.
+  const year = tier === "full" && selectedYear !== null && years.includes(selectedYear)
     ? selectedYear : years[0] ?? null;
-  const loadStatement = useCallback((force: boolean) => guard(loadProxyStatement(ticker, year!, { force, listing })), [guard, listing, ticker, year]);
+  const loadStatement = useCallback((force: boolean) => guard(loadProxyStatement(ticker, year!, { force, listing, tier })), [guard, listing, ticker, year, tier]);
   const detail = useAsyncResource(year === null ? null : loadStatement, { clearOnError: isPermanentClientError });
   const statement = detail.data?.data ?? null;
+  // The free preview: each executive's total pay; the breakdown and earlier years are Pro.
+  const preview = statement?.access === "preview" || list.data?.data?.access === "preview";
   const loading = list.loading || detail.loading;
   const listError = list.error ?? list.data?.refreshError;
   const statementError = detail.error ?? detail.data?.refreshError;
@@ -332,8 +349,9 @@ function ExecutiveResearch({ ticker, listing, focused, width, nested, guard }: {
     // `r` refreshes every pane, so it gets no hint here.
     const hints: PaneHint[] = statement ? [{ id: "open", key: "o", label: "pen filing", onPress: openFiling }] : [];
     if (yearHint) hints.push(yearHint);
+    if (preview) hints.push({ id: "upgrade", key: CLOUD_PLAN_KEY, label: "upgrade", title: "Upgrade to Pro", onPress: openUpgrade });
     return { info, hints };
-  }, [loading, statement, openFiling, yearHint]);
+  }, [loading, statement, openFiling, yearHint, preview, openUpgrade]);
 
   const figures = useMemo(
     () => (statement ? figuresOf(statement) : []),
@@ -390,7 +408,7 @@ function ExecutiveResearch({ ticker, listing, focused, width, nested, guard }: {
                 <FigureList figures={figures} width={proseWidth} minValueWidth={4} maxValueWidth={14} />
               </Box>
             )}
-            {statement.ceo && (
+            {statement.ceo && !preview && (
               <Box flexDirection="column">
                 <SectionHeading marginTop={figures.length > 0 ? 1 : 0}
                   title={`How ${statement.ceo.name.trim().split(/\s+/).pop() || "the CEO"} Was Paid`}
@@ -404,9 +422,15 @@ function ExecutiveResearch({ ticker, listing, focused, width, nested, guard }: {
                 <ExecutiveRows
                   rows={statement.namedExecutives}
                   width={proseWidth}
+                  breakdown={!preview}
                 />
               </Box>
             )}
+            {preview && (
+              <Box marginTop={1}>
+                <UpgradeLabel text="Upgrade for the pay breakdown and earlier years" onPress={openUpgrade} role="exec-upgrade" />
+              </Box>
+            )}
             {statement.highlights && (
               <Box flexDirection="column">
                 <SectionHeading marginTop={1} title="What Changed" />

package/src/plugins/builtin/market-movers/client.ts

@@ -110,10 +110,10 @@ export async function loadMarketMoverTab(
   return { ...result, quotes: quotes.slice(0, LIST_ROWS), rankedBy, tab };
 }
 
-/** The server refused the list for the account's plan. */
+/** The server refused the list for the account's plan (gaps, without Pro). */
 export class SessionMoversAccessError extends Error {
   constructor() {
-    super("Pre-market, after-hours and gap lists need Pro");
+    super("Gap lists need Pro");
     this.name = "SessionMoversAccessError";
   }
 }

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

@@ -8,7 +8,6 @@ import type { PluginModule } from "../plugin-module";
 import { TICKER_RESEARCH_PANE_ID } from "../../../types/config";
 import { publicTickerKey } from "../../../utils/exchanges";
 import { usePaneSettingValue } from "../../../state/app/context";
-import { usePlanAccess } from "../../../api-client/plan-access";
 import { useAssetData, usePluginPaneState, usePluginTickerActions } from "../../runtime";
 import { useLiveQuoteEntries } from "../../../state/hooks/quote-streaming";
 import { useAutoRefresh } from "../../../react/auto-refresh";
@@ -87,7 +86,6 @@ function MarketMoversPane({ focused, width, height }: PaneProps) {
   const [savedTab, setSavedTab] = usePluginPaneState<TabId>("activeTab", tabs[0]!.id);
   const [pickedIn, setPickedIn] = usePluginPaneState<string | null>("activeTabSession", null);
   const [requestedTab, setRequestedTab] = usePluginPaneState<TabId | null>("requestedTab", null);
-  const access = usePlanAccess();
   const session = useUsSession();
   const tabIds = useMemo(() => tabs.map((tab) => tab.id), [tabs]);
   const activeTab = resolveActiveTab({
@@ -96,7 +94,6 @@ function MarketMoversPane({ focused, width, height }: PaneProps) {
     pickedIn,
     session,
     requested: requestedTab,
-    sessionListsOpen: access.signedIn && access.emailVerified && access.hasProAccess,
   });
 
   // The index summary is a quote board like any other, so it runs on the shared

package/src/plugins/builtin/market-movers/session-body.tsx

@@ -24,6 +24,7 @@ import { summaryFooterSegments } from "./footer";
 import type { MarketSummaryQuote } from "./screener";
 import {
   earlierSessionLabel,
+  listAsOfLabel,
   overlaySessionMovers,
   resolveSide,
   sessionListIsLive,
@@ -46,7 +47,13 @@ interface LoadedList {
   payload: CloudSessionMoversPayload;
 }
 
-/** Pre-market, after-hours and gap lists: Pro only, like the ranked lists they sit beside. */
+/**
+ * Pre-market, after-hours and gap lists. Pre-market and after hours open on
+ * every plan: without Pro the server answers with the list as it was built a
+ * delay before (`access: "delayed"`), as old as the account's quotes, and the
+ * footer gives that time. Gaps are Pro only, like the ranked lists they sit
+ * beside.
+ */
 export function SessionMoversBody(props: {
   view: CloudSessionMoversCategory;
   session: UsSession;
@@ -57,29 +64,31 @@ export function SessionMoversBody(props: {
   liveStreaming: boolean;
 }) {
   const access = usePlanAccess();
+  const proOnly = props.view === "gaps";
   const [denied, setDenied] = useState(false);
   const deny = useCallback(() => setDenied(true), []);
   // A refusal is about the plan the account had then; a new plan asks again.
   useEffect(() => setDenied(false), [access.signedIn, access.emailVerified, access.hasProAccess]);
-  const entitled = access.emailVerified && access.hasProAccess && !denied;
+  const entitled = !proOnly || (access.emailVerified && access.hasProAccess && !denied);
   // Behind a wall the footer still carries the index summary; the list registers its own.
   usePaneFooter("market-movers-wall", () => (
     entitled ? null : { info: summaryFooterSegments(props.summaryQuotes) }
   ), [entitled, props.summaryQuotes]);
 
-  if (!access.signedIn) return <SignInWall placement="most-signin" width={props.width} height={props.height} action="see pre-market, after-hours and gap movers" />;
-  if (!access.emailVerified) return <SignInWall placement="most-signin" width={props.width} height={props.height} action="see pre-market, after-hours and gap movers" needsVerification />;
+  if (proOnly && !access.signedIn) return <SignInWall placement="most-signin" width={props.width} height={props.height} action="see gap movers" />;
+  if (proOnly && !access.emailVerified) return <SignInWall placement="most-signin" width={props.width} height={props.height} action="see gap movers" needsVerification />;
   if (!entitled) {
     return (
       <ProWall
         placement="most-wall"
         width={props.width} height={props.height}
-        title="Pre-market, after-hours and gap movers are part of Gloom Cloud Pro."
-        message="The whole listed US market from 04:00 to 20:00, with relative volume, VWAP, float and catalysts."
+        title="Gap movers are part of Gloom Cloud Pro."
+        message="Every listed US stock that opened away from its prior close, with relative volume, VWAP, float and catalysts."
       />
     );
   }
-  return <SessionMoversTable {...props} onDenied={deny} />;
+  // A new plan reads the list again: live with Pro, a delay before without.
+  return <SessionMoversTable key={access.hasProAccess ? "pro" : "free"} {...props} onDenied={proOnly ? deny : undefined} />;
 }
 
 function SessionMoversTable({ view, session, focused, width, summaryQuotes, liveStreaming, onDenied }: {
@@ -89,7 +98,8 @@ function SessionMoversTable({ view, session, focused, width, summaryQuotes, live
   width: number;
   summaryQuotes: MarketSummaryQuote[];
   liveStreaming: boolean;
-  onDenied: () => void;
+  /** Gaps: the server refused the list for the account's plan. */
+  onDenied?: () => void;
 }) {
   const { pinTicker } = usePluginTickerActions();
   const [savedSide, setSavedSide] = usePluginPaneState<CloudSessionMoversSide>("sessionSide", "up");
@@ -106,7 +116,10 @@ function SessionMoversTable({ view, session, focused, width, summaryQuotes, live
 
   const payload = loaded?.key === listKey ? loaded.payload : null;
   const listRows = payload?.items ?? NO_ROWS;
-  const live = !!payload && sessionListIsLive(view, payload.session, session);
+  // A free account's list is the one built a delay before: no live prices on its rows.
+  const delayed = payload?.access === "delayed";
+  const sessionTrading = !!payload && sessionListIsLive(view, payload.session, session);
+  const live = sessionTrading && !delayed;
   // While the list's session trades it is rebuilt every minute, and a failed or
   // stale answer is asked for again as often; a finished list does not change,
   // so it follows the refresh interval the user set.
@@ -126,7 +139,7 @@ function SessionMoversTable({ view, session, focused, width, summaryQuotes, live
       setLastLoadedAt(Date.now());
     } catch (error) {
       if (fetchGenRef.current !== gen) return;
-      if (error instanceof SessionMoversAccessError) {
+      if (error instanceof SessionMoversAccessError && onDenied) {
         onDenied();
         return;
       }
@@ -172,11 +185,14 @@ function SessionMoversTable({ view, session, focused, width, summaryQuotes, live
   }, [rows, selectedId]);
 
   const sessionLabel = payload ? earlierSessionLabel(payload.session, session) : null;
+  // While its session trades, a free list says when it was built; a finished one is final and its date says enough.
+  const asOfLabel = payload && delayed && sessionTrading ? listAsOfLabel(payload.asOf, payload.session, session) : null;
   usePaneFooter("market-movers", () => ({
     info: [
       ...(loadError ? [{ id: "load-error", parts: [{ text: loadError, tone: "warning" as const }] }] : []),
       ...summaryFooterSegments(summaryQuotes),
       ...(sessionLabel ? [{ id: "session", parts: [{ text: sessionLabel, tone: "muted" as const }] }] : []),
+      ...(asOfLabel ? [{ id: "as-of", asOf: true, parts: [{ text: asOfLabel, tone: "muted" as const }] }] : []),
       ...(loading ? [{ id: "loading", parts: [{ text: "loading", tone: "muted" as const }] }] : []),
       ...(feedStatus ? [{
         id: "feed",
@@ -184,7 +200,7 @@ function SessionMoversTable({ view, session, focused, width, summaryQuotes, live
       }] : []),
       ...(payload?.stale ? [{ id: "stale", parts: [{ text: "stale", tone: "muted" as const }] }] : []),
     ],
-  }), [feedStatus, loadError, loading, payload?.stale, sessionLabel, summaryQuotes]);
+  }), [asOfLabel, feedStatus, loadError, loading, payload?.stale, sessionLabel, summaryQuotes]);
 
   const openRow = useCallback((row: SessionMoverRow) => {
     pinTicker(rowKey(row), { floating: true, paneType: TICKER_RESEARCH_PANE_ID, instrument: null });

package/src/plugins/builtin/market-movers/session.ts

@@ -11,7 +11,7 @@ import { buildQuoteKey, resolveEntryData } from "../../../market-data/selectors"
 import type { QueryEntry } from "../../../market-data/result-types";
 import type { Quote } from "../../../types/financials";
 import { compareSortValues, type SortDirection } from "../../../utils/sort-values";
-import { zonedDateKey, zonedWallClockToUtcMs } from "../../../utils/zoned-date-time";
+import { zonedDateKey, zonedDateTimeParts, zonedWallClockToUtcMs } from "../../../utils/zoned-date-time";
 import type { SessionTabId } from "./model";
 
 const NEW_YORK = "America/New_York";
@@ -67,9 +67,8 @@ export function usSessionAt(now: number): UsSession {
  * stays while the session it was picked in lasts; otherwise the pane follows
  * the session that is trading: Pre-market before 09:30, the day's lists during
  * the regular session, After hours after the close. Overnight and on closed
- * days the last pick stands. An account that cannot open the session lists
- * (signed out, unverified, free) keeps the day's lists instead of being moved
- * onto a wall.
+ * days the last pick stands. Every plan opens the two lists it follows, a
+ * free one as of a delay before (session-body.tsx).
  */
 export function resolveActiveTab<T extends string>(args: {
   tabs: readonly T[];
@@ -77,14 +76,13 @@ export function resolveActiveTab<T extends string>(args: {
   pickedIn: string | null | undefined;
   session: UsSession;
   requested?: string | null;
-  sessionListsOpen: boolean;
 }): T {
   const { tabs, session } = args;
   const requested = tabs.find((tab) => tab === args.requested);
   if (requested) return requested;
   const saved = tabs.find((tab) => tab === args.saved) ?? tabs[0]!;
   if (session.phase === "closed" || args.pickedIn === session.key) return saved;
-  if (session.phase === "regular" || !args.sessionListsOpen) {
+  if (session.phase === "regular") {
     return REGULAR_TABS.has(saved) ? saved : tabs.find((tab) => REGULAR_TABS.has(tab)) ?? saved;
   }
   const sessionTab = session.phase === "pre" ? "premarket" : "afterhours";
@@ -223,11 +221,28 @@ export function sortSessionRows(rows: SessionMoverRow[], preference: SessionMove
   return [...rows].sort((left, right) => compareSortValues(sortValue(columnId, left), sortValue(columnId, right), preference.direction));
 }
 
+function shortDay(date: string): string | null {
+  const day = new Date(`${date}T12:00:00Z`);
+  return Number.isFinite(day.getTime())
+    ? day.toLocaleDateString("en-US", { month: "short", day: "numeric", timeZone: "UTC" })
+    : null;
+}
+
 /** The session date when it is not today's, e.g. last evening's after-hours list. */
 export function earlierSessionLabel(listSession: string | null, session: UsSession): string | null {
   if (!listSession || listSession === session.date) return null;
-  const date = new Date(`${listSession}T12:00:00Z`);
-  return Number.isFinite(date.getTime())
-    ? date.toLocaleDateString("en-US", { month: "short", day: "numeric", timeZone: "UTC" })
-    : null;
+  return shortDay(listSession);
+}
+
+/**
+ * When a free account's list was built, New York time: "as of 07:42 ET", and
+ * "as of Oct 19 19:58 ET" when that is not on the list's own session day.
+ */
+export function listAsOfLabel(asOf: string, listSession: string | null, session: UsSession): string | null {
+  const at = Date.parse(asOf);
+  if (!Number.isFinite(at)) return null;
+  const { hour, minute } = zonedDateTimeParts(at, NEW_YORK);
+  const clock = `${String(hour).padStart(2, "0")}:${String(minute).padStart(2, "0")} ET`;
+  const day = zonedDateKey(at, NEW_YORK);
+  return day === (listSession ?? session.date) ? `as of ${clock}` : `as of ${shortDay(day) ?? day} ${clock}`;
 }

package/src/plugins/builtin/news/wire/feed-config.ts

@@ -1,4 +1,5 @@
 import type { PluginConfigState } from "../../../../types/plugin";
+import { logFailure } from "../../../../utils/log-failure";
 import { DEFAULT_FEEDS } from "./default-feeds";
 import { hashString } from "./hash";
 import type { RssFeedConfig } from "./rss/parser";
@@ -30,7 +31,8 @@ function parseJsonArray(value: unknown): unknown[] {
   try {
     const parsed = JSON.parse(value);
     return Array.isArray(parsed) ? parsed : [];
-  } catch {
+  } catch (error) {
+    logFailure("news.feed-config", "Read stored news feed settings failed")(error);
     return [];
   }
 }
@@ -93,6 +95,7 @@ function normalizeUserFeed(value: unknown): RssFeedConfig | null {
       enabled: record.enabled !== false,
     });
   } catch {
+    // Intentional ignore: invalid stored feed entries are omitted during normalization.
     return null;
   }
 }

package/src/plugins/builtin/news/wire/letters/client.ts

@@ -31,6 +31,7 @@ const isWebUrl = (value: unknown): value is string => {
     const url = new URL(value);
     return url.protocol === "https:" || url.protocol === "http:";
   } catch {
+    // Intentional ignore: malformed links fail letter validation like other invalid fields.
     return false;
   }
 };

package/src/plugins/builtin/news/wire/news/command-bar-search.ts

@@ -8,6 +8,7 @@ import type {
   GloomPluginContext,
 } from "../../../../../types/plugin";
 import { formatFeedTime } from "../../../../../utils/datetime-format";
+import { logFailure } from "../../../../../utils/log-failure";
 import { openNewsStoryPane } from "./pop-out";
 import { NEWS_QUERY_PRESETS } from "./query-presets";
 
@@ -75,9 +76,11 @@ let warming: Promise<unknown> | null = null;
  */
 async function warmLatestStories(service: NewsService): Promise<void> {
   if (service.getQueryState(NEWS_QUERY_PRESETS.feed).articles.length > 0) return;
-  warming ??= service.load(NEWS_QUERY_PRESETS.feed).catch(() => null).finally(() => {
-    warming = null;
-  });
+  warming ??= service.load(NEWS_QUERY_PRESETS.feed)
+    .catch(logFailure("news.command-bar-search", "Load news for command bar search failed", null))
+    .finally(() => {
+      warming = null;
+    });
   await warming;
 }
 

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

@@ -14,6 +14,7 @@ import { useInlineTickers } from "../../../../../state/hooks/inline-tickers";
 import { truncateToDisplayWidth } from "../../../../../utils/format";
 import { wrapTextLines } from "../../../../../utils/text-wrap";
 import { formatDetailDate } from "../../../../../utils/datetime-format";
+import { logFailure } from "../../../../../utils/log-failure";
 import { mergeNewsArticle } from "../../../../../news/news-model";
 import { usePluginPaneState } from "../../../../runtime";
 import { formatNewsCategory } from "../categories";
@@ -277,7 +278,7 @@ export function NewsDetailView({ item, focused, width, showTitle = true }: {
     }).then((choice) => {
       const url = choice ? coverage[Number(choice)]?.url : undefined;
       if (url) void rendererHost.openExternal(url);
-    }).catch(() => {});
+    }).catch(logFailure("news.detail-view", "Choose news coverage failed"));
   }, [coverage, dialog, rendererHost]);
   usePaneFooter("news-detail:story", () => ({
     order: -1,

package/src/plugins/builtin/news/wire/news/table.tsx

@@ -15,6 +15,7 @@ import { colors } from "../../../../../theme/colors";
 import { collectNewsDisplayTickers } from "../../../../../news/ticker-symbols";
 import { useLoadNewsStory } from "../../../../../news/hooks";
 import { formatFeedTime } from "../../../../../utils/datetime-format";
+import { logFailure } from "../../../../../utils/log-failure";
 import { nextHeaderSort } from "../../../../../utils/sort-values";
 import { formatNewsCategory } from "../categories";
 import { useOpenTickerChoice } from "../../../shared/ticker-choice";
@@ -270,7 +271,7 @@ export function NewsArticleStackView({
   const loadNewsStory = useLoadNewsStory();
   const prefetchStory = useCallback((article: MarketNewsItem) => {
     if ((article.items?.length ?? 0) > 0) return;
-    void loadNewsStory(article.id).catch(() => {});
+    void loadNewsStory(article.id).catch(logFailure("news.table", "Prefetch news story failed"));
   }, [loadNewsStory]);
 
   // A crosshair pinned in the tab picks the newest story out by then, and

package/src/plugins/builtin/news/wire/rss/feed-xml.ts

@@ -14,7 +14,12 @@ export interface FeedElement {
 
 export function resolveFeedUrl(value: string, base: string): string {
   if (!value.trim()) return "";
-  try { return new URL(value.trim(), base).href; } catch { return ""; }
+  try {
+    return new URL(value.trim(), base).href;
+  } catch {
+    // Intentional ignore: malformed optional feed links are treated as absent.
+    return "";
+  }
 }
 
 // Read the structure of a syndication document without executing declarations or

package/src/plugins/builtin/news/wire/rss/managed-feed.ts

@@ -14,6 +14,7 @@ export function isManagedNewsFeedUrl(value: string): boolean {
     const url = new URL(value);
     return url.origin === "https://api.gloom.sh" && url.pathname.replace(/\/$/, "") === "/news/rss";
   } catch {
+    // Intentional ignore: malformed URLs cannot identify the managed news feed.
     return false;
   }
 }

package/src/plugins/builtin/notes/cli.ts

@@ -1,5 +1,6 @@
 import { apiClient } from "../../../api-client";
 import { withCliServices, withConfigData } from "../../../cli/scoped-context";
+import type { CliServicesContext } from "../../../cli/types";
 import { CLI_COMMAND_GROUPS } from "../../../cli/help";
 import { dryRunNote } from "../../../cli/helpers";
 import { requireArg, takeOption } from "../../../cli/commands/command-utils";
@@ -7,9 +8,10 @@ import { EXCHANGE_OPTION, listingReadNotes, requireListingArg } from "../../../c
 import { createPluginPersistence } from "../../plugin-persistence";
 import type { CliCommandDef } from "../../../types/plugin";
 import { cliStyles } from "../../../utils/cli-output";
+import { errorMessage } from "../../../utils/errors";
 import { logFailure } from "../../../utils/log-failure";
 import { readChatSessionState } from "../chat/controller/persistence";
-import { exportNotesToDirectory, type NotesExportSource } from "./export";
+import { describeExportSkip, exportNotesToDirectory, type NotesExportSource } from "./export";
 import { NotesFiles } from "./files";
 import { CloudNotesStore } from "./store";
 
@@ -42,29 +44,52 @@ export const notesCliCommand: CliCommandDef = {
         // (personal and every team), one folder each.
         const dir = args[1] ?? `${config.dataDir}/notes-export-${new Date().toISOString().slice(0, 10)}`;
         const sources: NotesExportSource[] = [{ label: "local", store: notes }];
-        await withCliServices(ctx, async (services) => {
+        // A Cloud source left out or unreadable is a warning, as `quote` reports
+        // a symbol it found nothing for: the local files are always written.
+        const warnings: string[] = [];
+        const leftOut = (what: string, failure: string) => (error: unknown) => {
+          logFailure("notes.cli", failure)(error);
+          warnings.push(`${what} (${errorMessage(error)}).`);
+          return null;
+        };
+        const addCloudSources = async (services: CliServicesContext) => {
           const cloudPersistence = createPluginPersistence(services.persistence.pluginState, services.persistence.resources, "plugin:gloomberb-cloud", "gloomberb-cloud");
           const session = readChatSessionState(cloudPersistence, null);
           if (!session?.sessionToken) return;
           apiClient.setSessionToken(session.sessionToken);
           const user = await apiClient.ensureVerifiedSession()
-            .catch(logFailure("notes.cli", "Verify the saved session for the notes export failed", null));
+            .catch(leftOut("Skipped Gloom Cloud notes: the saved session could not be checked", "Verify the saved session for the notes export failed"));
           if (!user) return;
           const notesPersistence = createPluginPersistence(services.persistence.pluginState, services.persistence.resources, "plugin:notes", "notes");
           sources.push({ label: "mine", store: new CloudNotesStore({ kind: "user" }, notesPersistence) });
-          const teams = await apiClient.listTeams().catch(logFailure("notes.cli", "List teams for the notes export failed", []));
+          const teams = await apiClient.listTeams()
+            .catch(leftOut("Skipped team notes: the team list could not be read", "List teams for the notes export failed")) ?? [];
           for (const team of teams) {
             sources.push({ label: team.name, store: new CloudNotesStore({ kind: "team", teamId: team.id }, notesPersistence) });
           }
+        };
+        // The Cloud stores read and cache through these services, so the
+        // export runs before they close.
+        const result = await withCliServices(ctx, async (services) => {
+          await addCloudSources(services);
+          return ctx.cliOptions.dryRun ? null : await exportNotesToDirectory(dir, sources);
         });
-        if (ctx.cliOptions.dryRun) {
-          ctx.printResult({ data: { dryRun: true, dir, sources: sources.map((source) => source.label) } }, {
+        if (!result) {
+          ctx.printResult({ data: { dryRun: true, dir, sources: sources.map((source) => source.label) }, ...(warnings.length > 0 ? { warnings } : {}) }, {
             text: (data) => `Would export ${data.sources.join(", ")} notes to ${data.dir}.${dryRunNote(true)}`,
           });
           return;
         }
-        const result = await exportNotesToDirectory(dir, sources);
-        ctx.printResult({ data: { dir: result.dir, files: result.files, sources: sources.map((source) => source.label) } }, {
+        warnings.push(...result.skipped.map(describeExportSkip));
+        ctx.printResult({
+          data: {
+            dir: result.dir,
+            files: result.files,
+            sources: result.exported,
+            ...(result.skipped.length > 0 ? { skipped: result.skipped } : {}),
+          },
+          ...(warnings.length > 0 ? { warnings } : {}),
+        }, {
           text: (data) => `Exported ${data.files} notes from ${data.sources.join(", ")} to ${data.dir}.`,
         });
         return;

package/src/plugins/builtin/notes/export.ts

@@ -1,3 +1,4 @@
+import { errorMessage } from "../../../utils/errors";
 import type { NoteFileEntry, QuickNoteEntry } from "./files";
 import type { NotesStore } from "./store";
 import { noteOwnerKey, splitNoteKey } from "./store";
@@ -7,9 +8,23 @@ export interface NotesExportSource {
   store: NotesStore;
 }
 
+/**
+ * What an export left out, and why: a whole source whose notes could not be
+ * listed, or with `note`, one note of that source that could not be read.
+ */
+export interface NotesExportSkip {
+  label: string;
+  /** The note's symbol, or a quick note's title. */
+  note?: string;
+  reason: string;
+}
+
 export interface NotesExportResult {
   dir: string;
   files: number;
+  /** The sources whose notes were written, in the order given. */
+  exported: string[];
+  skipped: NotesExportSkip[];
 }
 
 function safeFileName(value: string): string {
@@ -29,16 +44,27 @@ export async function exportNotesToDirectory(
   const fsModulePath = "fs/promises";
   const { mkdir, writeFile } = await import(fsModulePath) as typeof import("fs/promises");
   let files = 0;
+  const exported: string[] = [];
+  const skipped: NotesExportResult["skipped"] = [];
   for (const source of sources) {
     const folder = sources.length > 1 ? `${dir}/${safeFileName(source.label)}` : dir;
     let entries: NoteFileEntry[];
     let quick: QuickNoteEntry[];
+    const unreadable: Array<{ key: string; error: unknown }> = [];
     try {
-      [entries, quick] = await Promise.all([source.store.list(), source.store.loadQuickNotesIndex()]);
-    } catch {
+      [entries, quick] = await Promise.all([
+        source.store.list({ strict: true, skipped: (key, error) => unreadable.push({ key, error }) }),
+        source.store.loadQuickNotesIndex(),
+      ]);
+    } catch (error) {
+      skipped.push({ label: source.label, reason: errorMessage(error) });
       continue;
     }
     const titles = new Map(quick.map((entry) => [entry.id, entry.title]));
+    for (const { key: noteKey, error } of unreadable) {
+      const { kind, key } = splitNoteKey(noteKey);
+      skipped.push({ label: source.label, note: kind === "quick" ? (titles.get(key) ?? key) : key, reason: errorMessage(error) });
+    }
     for (const entry of entries) {
       const { kind, key } = splitNoteKey(entry.key);
       const path = kind === "quick"
@@ -48,8 +74,28 @@ export async function exportNotesToDirectory(
       await writeFile(path, entry.text, "utf-8");
       files += 1;
     }
+    exported.push(source.label);
   }
-  return { dir, files };
+  return { dir, files, exported, skipped };
+}
+
+/** One line per skip, as `notes export` warns and the Export Notes command reports. */
+export function describeExportSkip(skip: NotesExportSkip): string {
+  return skip.note === undefined
+    ? `Skipped ${skip.label}: its notes could not be read (${skip.reason}).`
+    : `Skipped the ${skip.label} note ${skip.note}: it could not be read (${skip.reason}).`;
+}
+
+/** The Export Notes command's notice: the count, then what was skipped. */
+export function exportNotice(result: NotesExportResult): { body: string; type: "success" | "error" } {
+  const exported = `Exported ${result.files} ${result.files === 1 ? "note" : "notes"} to ${result.dir}`;
+  if (result.skipped.length === 0) return { body: exported, type: "success" };
+  const shown = result.skipped.slice(0, 3).map(describeExportSkip);
+  const more = result.skipped.length - shown.length;
+  return {
+    body: [`${exported}.`, ...shown, ...(more > 0 ? [`${more} more could not be read.`] : [])].join(" "),
+    type: "error",
+  };
 }
 
 export function exportSourceLabel(store: NotesStore, teamName?: string): string {