gloomberb 0.11.2 → 0.11.3

package/src/renderers/electrobun/bun/desktop/host-requests.ts

@@ -10,6 +10,7 @@ import type {
 import { safeExternalUrl } from "../../../../utils/external-url";
 import { getContextMenuRequestId, normalizeContextMenuItems } from "../context-menu/normalize";
 import { MAIN_WINDOW_RPC_KEY } from "../window/focus";
+import { saveTextFileToDownloads } from "../../../../utils/save-text-file";
 
 interface DesktopHostRequestOptions<TRpc> {
   clearMainWindow: () => void;
@@ -51,7 +52,7 @@ function normalizeWindowControlAction(action: unknown): DesktopWindowControlActi
   throw new Error("host.windowControl requires a valid action.");
 }
 
-export function handleDesktopHostRequest<TRpc>({
+export async function handleDesktopHostRequest<TRpc>({
   clearMainWindow,
   closeAllDetachedWindows,
   controlWindowForRpcKey,
@@ -63,7 +64,7 @@ export function handleDesktopHostRequest<TRpc>({
   rpc,
   teardownServices,
   trackContextMenuRequest,
-}: DesktopHostRequestOptions<TRpc>): DesktopBackendRequestResponse<DesktopHostRequest["method"]> {
+}: DesktopHostRequestOptions<TRpc>): Promise<DesktopBackendRequestResponse<DesktopHostRequest["method"]>> {
   switch (request.method) {
     case "host.restart":
       restartDesktopApp({
@@ -127,6 +128,12 @@ export function handleDesktopHostRequest<TRpc>({
     }
     case "host.readText":
       return Utils.clipboardReadText() ?? "";
+    case "host.saveTextFile": {
+      if (typeof request.payload.name !== "string" || typeof request.payload.text !== "string") {
+        throw new Error("host.saveTextFile requires a name and text.");
+      }
+      return saveTextFileToDownloads(request.payload.name, request.payload.text);
+    }
     case "host.notify":
       playNotificationSound(normalizeText(request.payload.sound));
       Utils.showNotification({

package/src/renderers/electrobun/bun/index.ts

@@ -476,6 +476,7 @@ async function handleBackendRequest(
     case "host.focusWindow":
     case "host.copyPngImage":
     case "host.readText":
+    case "host.saveTextFile":
     case "host.notify":
     case "host.showContextMenu":
       return handleDesktopHostRequest({

package/src/renderers/electrobun/shared/protocol.ts

@@ -119,6 +119,10 @@ export interface DesktopBackendRequestMap {
   "host.focusWindow": { request: null; response: null };
   "host.copyPngImage": { request: { pngBase64: string }; response: null };
   "host.readText": { request: null; response: string };
+  "host.saveTextFile": {
+    request: { name: string; text: string; mimeType: string };
+    response: string;
+  };
   "host.notify": {
     request: { title?: string; body?: string; subtitle?: string; sound?: string };
     response: null;

package/src/renderers/electrobun/view/ui-host.tsx

@@ -61,6 +61,9 @@ export const webRendererHost: RendererHost = {
       return backendRequest("host.readText");
     }
   },
+  async saveTextFile({ name, text, mimeType }) {
+    return backendRequest("host.saveTextFile", { name, text, mimeType });
+  },
   notify(notification) {
     void backendRequest("host.notify", {
       title: notification.title,

package/src/renderers/opentui/host.tsx

@@ -7,6 +7,7 @@ import type { NativeRendererHost, PixelResolution, RendererHost } from "../../ui
 import { colors } from "../../theme/colors";
 import { safeExternalUrl } from "../../utils/external-url";
 import { createTerminalMediaReaper, terminalMediaStateFile } from "./terminal-media";
+import { saveTextFileToDownloads } from "../../utils/save-text-file";
 
 export { useKeyboard, useTerminalDimensions };
 
@@ -134,6 +135,9 @@ export async function createOpenTuiHost(): Promise<OpenTuiHost> {
       });
       return await new Response(proc.stdout).text();
     },
+    async saveTextFile({ name, text }) {
+      return saveTextFileToDownloads(name, text);
+    },
     notify() {
       // The app-level notifier still owns toast/desktop notification behavior.
     },

package/src/state/pane-table-export-registry.ts

@@ -0,0 +1,31 @@
+export type PaneTableExporter = (filename: string) => Promise<string>;
+
+const exporters = new Map<string, Map<symbol, PaneTableExporter>>();
+
+export function registerPaneTableExporter(
+  paneId: string,
+  exporter: PaneTableExporter,
+): () => void {
+  const token = Symbol(paneId);
+  const paneExporters = exporters.get(paneId) ?? new Map<symbol, PaneTableExporter>();
+  paneExporters.set(token, exporter);
+  exporters.set(paneId, paneExporters);
+
+  return () => {
+    const current = exporters.get(paneId);
+    current?.delete(token);
+    if (current?.size === 0) exporters.delete(paneId);
+  };
+}
+
+export function hasPaneTableExporter(paneId: string): boolean {
+  return exporters.get(paneId)?.size === 1;
+}
+
+export async function exportPaneTable(paneId: string, filename: string): Promise<string> {
+  const paneExporters = exporters.get(paneId);
+  if (paneExporters?.size !== 1) {
+    throw new Error("This pane does not have one active exportable table.");
+  }
+  return [...paneExporters.values()][0]!(filename);
+}

package/src/time-series/range.ts

@@ -7,3 +7,15 @@ export interface ChartDateWindow {
 }
 
 export const TIME_RANGES: TimeRange[] = ["1D", "1W", "1M", "3M", "6M", "1Y", "5Y", "ALL"];
+export const CHART_RESOLUTIONS: ChartResolution[] = [
+  "auto",
+  "1m",
+  "5m",
+  "15m",
+  "30m",
+  "45m",
+  "1h",
+  "1d",
+  "1wk",
+  "1mo",
+];

package/src/time-series/resolution.ts

@@ -1,5 +1,5 @@
 import type { ChartResolution, TimeRange } from "./range";
-import { TIME_RANGES } from "./range";
+import { CHART_RESOLUTIONS, TIME_RANGES } from "./range";
 import { isDateWindowWithinTimeRange } from "./date-window";
 
 export type ManualChartResolution = Exclude<ChartResolution, "auto">;
@@ -9,20 +9,8 @@ export interface ChartResolutionSupport {
   maxRange: TimeRange;
 }
 
-export const TIME_RANGE_ORDER: TimeRange[] = [...TIME_RANGES];
-
-const CHART_RESOLUTION_ORDER: ChartResolution[] = [
-  "auto",
-  "1m",
-  "5m",
-  "15m",
-  "30m",
-  "45m",
-  "1h",
-  "1d",
-  "1wk",
-  "1mo",
-];
+export const TIME_RANGE_ORDER = TIME_RANGES;
+const CHART_RESOLUTION_ORDER = CHART_RESOLUTIONS;
 
 const CHART_RESOLUTION_LABELS: Record<ChartResolution, string> = {
   auto: "AUTO",

package/src/time-series/spec.ts

@@ -1,4 +1,4 @@
-import type { ChartResolution, TimeRange } from "./range";
+import { CHART_RESOLUTIONS, TIME_RANGES, type ChartResolution, type TimeRange } from "./range";
 import { getChartResolutionLabel } from "./resolution";
 import {
   canonicalTimeSeriesFieldId,
@@ -31,8 +31,8 @@ import {
   isValidChartSeriesId,
 } from "../capabilities/chart-series";
 
-const TIME_RANGES = new Set<TimeRange>(["1D", "1W", "1M", "3M", "6M", "1Y", "5Y", "ALL"]);
-const RESOLUTIONS = new Set<ChartResolution>(["auto", "1m", "5m", "15m", "30m", "45m", "1h", "1d", "1wk", "1mo"]);
+const RANGE_SET = new Set<TimeRange>(TIME_RANGES);
+const RESOLUTIONS = new Set<ChartResolution>(CHART_RESOLUTIONS);
 const PERIODS = new Set<SeriesPeriod>(["auto", "daily", "weekly", "monthly", "quarterly", "annual", "ttm"]);
 const STYLES = new Set<SeriesStyle>(["line", "area", "step", "columns", "points", "candles", "ohlc", "hlc"]);
 const ECONOMIC_STYLES = new Set<SeriesStyle>(["line", "area", "step", "columns", "points"]);
@@ -306,7 +306,7 @@ export function normalizeChartSpec(value: unknown, fallback: ChartSpec = DEFAULT
   return {
     version: CHART_SPEC_VERSION,
     viewport: {
-      range: TIME_RANGES.has(viewport?.range as TimeRange)
+      range: RANGE_SET.has(viewport?.range as TimeRange)
         ? viewport!.range as TimeRange
         : fallbackCopy.viewport.range,
       resolution: RESOLUTIONS.has(viewport?.resolution as ChartResolution)
@@ -340,7 +340,7 @@ export function validateChartSpec(spec: ChartSpec): ChartSpecValidationResult {
   if (spec.version !== CHART_SPEC_VERSION) {
     errors.push(issue("version", "unsupported-version", `Unsupported chart spec version ${String(spec.version)}.`));
   }
-  if (!TIME_RANGES.has(spec.viewport.range)) errors.push(issue("viewport.range", "invalid-range", "Invalid date range."));
+  if (!RANGE_SET.has(spec.viewport.range)) errors.push(issue("viewport.range", "invalid-range", "Invalid date range."));
   if (!RESOLUTIONS.has(spec.viewport.resolution)) {
     errors.push(issue("viewport.resolution", "invalid-resolution", "Invalid chart resolution."));
   }

package/src/types/plugin.ts

@@ -67,6 +67,8 @@ export interface PaneDef {
   defaultMode?: "docked" | "floating";
   /** Pane publishes its selected symbol as pane-state `cursorSymbol`, so ticker panes can follow it. */
   tickerSource?: boolean;
+  /** Add an Excel-compatible CSV action for the pane's single active DataTable. */
+  tableExport?: true;
   settings?: PaneSettingsDef | ((context: PaneSettingsContext) => PaneSettingsDef | null);
   /** Portable sharing is public by default; list the few pane-owned fields that must remain local. */
   portableShare?: PanePortableShareDef;
@@ -356,25 +358,6 @@ export interface CliCommandDef {
   execute(args: string[], ctx: CliCommandContext): void | CliDispatchResult | Promise<void | CliDispatchResult>;
 }
 
-export interface PluginCliCommandDescriptor {
-  name: string;
-  aliases?: string[];
-  summary: string;
-  inputShape?: string;
-  outputShape?: string;
-  examples?: string[];
-  sideEffectLevel?: "none" | "local-write" | "network-write" | "external-trade" | "external-side-effect";
-  requirements?: string[];
-  batch?: boolean;
-  formats?: Array<"text" | "json" | "csv" | "ndjson">;
-  safety?: string[];
-  execute?: CliCommandDef["execute"];
-}
-
-export interface PluginCliDescriptor {
-  commands?: PluginCliCommandDescriptor[];
-}
-
 export interface CommandDef {
   id: string;
   label: string;
@@ -602,7 +585,6 @@ export interface GloomPlugin {
   toggleable?: boolean;
   order?: number;
   cliCommands?: CliCommandDef[];
-  cli?: PluginCliDescriptor;
 
   setup?(ctx: GloomPluginContext): void | Promise<void>;
   dispose?(): void;

package/src/ui/host.tsx

@@ -378,6 +378,12 @@ export interface UiHost {
   colorFromHex?(hex: string): unknown;
 }
 
+export interface SaveTextFileRequest {
+  name: string;
+  text: string;
+  mimeType: string;
+}
+
 export interface RendererHost {
   requestExit(): void;
   startWindowDrag?(): Promise<void> | void;
@@ -386,6 +392,7 @@ export interface RendererHost {
   copyText(text: string): Promise<void>;
   copyPngImage?(pngBase64: string): Promise<void>;
   readText(): Promise<string>;
+  saveTextFile?(request: SaveTextFileRequest): Promise<string>;
   supportsNativeDesktopNotifications?: boolean;
   notify(notification: AppNotificationRequest): void;
   showContextMenu?(items: ContextMenuItem[]): Promise<boolean>;

package/src/utils/csv.test.ts

@@ -0,0 +1,13 @@
+import { describe, expect, test } from "bun:test";
+import { EXCEL_CSV_BOM, serializeCsv } from "./csv";
+
+describe("serializeCsv", () => {
+  test("escapes cells and can emit an Excel UTF-8 BOM", () => {
+    const csv = serializeCsv([
+      ["Name", "Note", "Value"],
+      ["ACME, Inc.", "said \"hello\"\nagain", "=1+1"],
+    ], { excelCompatible: true });
+
+    expect(csv).toBe(`${EXCEL_CSV_BOM}Name,Note,Value\n"ACME, Inc.","said ""hello""\nagain",'=1+1`);
+  });
+});

package/src/utils/csv.ts

@@ -0,0 +1,37 @@
+export const EXCEL_CSV_BOM = "\uFEFF";
+
+function normalizeCsvCell(value: unknown): string {
+  if (value == null) return "";
+  if (value instanceof Date) return value.toISOString();
+  if (typeof value === "object") return JSON.stringify(value);
+  return String(value);
+}
+
+function escapeCsvCell(value: unknown, excelCompatible: boolean): string {
+  const normalized = normalizeCsvCell(value);
+  const text = excelCompatible && /^[\t\r ]*[=+\-@]/.test(normalized)
+    ? `'${normalized}`
+    : normalized;
+  if (!/[",\n\r]/.test(text)) return text;
+  return `"${text.replace(/"/g, '""')}"`;
+}
+
+export function serializeCsv(
+  rows: readonly (readonly unknown[])[],
+  options: { excelCompatible?: boolean } = {},
+): string {
+  const excelCompatible = options.excelCompatible === true;
+  const csv = rows.map((row) => row.map((cell) => escapeCsvCell(cell, excelCompatible)).join(",")).join("\n");
+  return excelCompatible ? `${EXCEL_CSV_BOM}${csv}` : csv;
+}
+
+export function createCsvExportFilename(title: string): string {
+  const stem = title.trim()
+    .replace(/[<>:"/\\|?*\u0000-\u001f]/g, "-")
+    .replace(/\s+/g, "-")
+    .replace(/-+/g, "-")
+    .replace(/^[. -]+|[. -]+$/g, "")
+    .slice(0, 80) || "gloomberb-table";
+  const timestamp = new Date().toISOString().replace(/[:.]/g, "-");
+  return `${stem}-${timestamp}.csv`;
+}

package/src/utils/save-text-file.test.ts

@@ -0,0 +1,24 @@
+import { afterEach, expect, test } from "bun:test";
+import { mkdtemp, readFile, rm } from "node:fs/promises";
+import { tmpdir } from "node:os";
+import { join } from "node:path";
+import { saveTextFileToDirectory } from "./save-text-file";
+
+let directory: string | null = null;
+
+afterEach(async () => {
+  if (directory) await rm(directory, { recursive: true, force: true });
+  directory = null;
+});
+
+test("keeps existing exports instead of overwriting them", async () => {
+  directory = await mkdtemp(join(tmpdir(), "gloomberb-export-"));
+
+  const first = await saveTextFileToDirectory(directory, "prices.csv", "first");
+  const second = await saveTextFileToDirectory(directory, "prices.csv", "second");
+
+  expect(first).toBe("prices.csv");
+  expect(second).toBe("prices (2).csv");
+  expect(await readFile(join(directory, first), "utf-8")).toBe("first");
+  expect(await readFile(join(directory, second), "utf-8")).toBe("second");
+});

package/src/utils/save-text-file.ts

@@ -0,0 +1,36 @@
+import { mkdir, writeFile } from "node:fs/promises";
+import { homedir } from "node:os";
+import { basename, extname, join } from "node:path";
+
+function safeFilename(name: string): string {
+  const safe = basename(name.replace(/\\/g, "/"))
+    .replace(/[<>:"|?*\u0000-\u001f]/g, "-")
+    .replace(/[. ]+$/g, "");
+  return !safe || safe === "." || safe === ".." ? "gloomberb-export.txt" : safe;
+}
+
+export async function saveTextFileToDirectory(
+  directory: string,
+  name: string,
+  text: string,
+): Promise<string> {
+  await mkdir(directory, { recursive: true });
+  const safeName = safeFilename(name);
+  const extension = extname(safeName);
+  const stem = safeName.slice(0, safeName.length - extension.length);
+
+  for (let copy = 1; ; copy += 1) {
+    const filename = copy === 1 ? safeName : `${stem} (${copy})${extension}`;
+    try {
+      await writeFile(join(directory, filename), text, { encoding: "utf-8", flag: "wx" });
+      return filename;
+    } catch (error) {
+      if ((error as NodeJS.ErrnoException).code !== "EEXIST") throw error;
+    }
+  }
+}
+
+export async function saveTextFileToDownloads(name: string, text: string): Promise<string> {
+  const filename = await saveTextFileToDirectory(join(homedir(), "Downloads"), name, text);
+  return `~/Downloads/${filename}`;
+}