opencandle 0.11.1 → 0.12.0

package/src/tools/portfolio/alerts.ts

@@ -1,457 +0,0 @@
-import type { AgentTool } from "@earendil-works/pi-agent-core";
-import { Type } from "@sinclair/typebox";
-import {
-  ALERT_CONDITION_VERSION,
-  percentMove,
-  priceCrossesAbove,
-  priceCrossesBelow,
-  priceCrossesSma,
-  rsiThreshold,
-  smaCross,
-  volumeSpike,
-} from "../../market-state/alert-conditions.js";
-import { defaultAlertRunnerProviders, runAlertChecks } from "../../market-state/alert-runner.js";
-import { deliverPendingNotifications } from "../../market-state/notification-delivery.js";
-import { resolveInstrumentForMutation } from "../../market-state/resolve-for-mutation.js";
-import { type AlertRuleRecord, MarketStateService } from "../../market-state/service.js";
-import { initDefaultDatabase } from "../../memory/sqlite.js";
-
-const ACTION_DESCRIPTION = [
-  "One of: create_price_above, create_price_below, create_price_above_sma,",
-  "create_price_below_sma, create_rsi_above, create_rsi_below,",
-  "create_volume_spike, create_percent_move_up, create_percent_move_down,",
-  "create_sma_cross_above, create_sma_cross_below, set_enabled, list, status, check.",
-  "Use create_price_above/create_price_below for price alerts,",
-  "create_price_above_sma/create_price_below_sma for SMA crossing alerts,",
-  "create_percent_move_up/create_percent_move_down for one-day percent move alerts,",
-  "create_sma_cross_above/create_sma_cross_below for fast/slow SMA crossing alerts,",
-  "create_rsi_above/create_rsi_below for RSI alerts,",
-  "create_volume_spike for volume alerts, set_enabled to enable or disable an alert,",
-  "and status to inspect local runner/check history.",
-  "When the user asks to create an alert and check it now in the same request,",
-  "set check_after_create to true on the create action.",
-].join(" ");
-
-const params = Type.Object({
-  action: Type.Union(
-    [
-      Type.Literal("create_price_above"),
-      Type.Literal("create_price_below"),
-      Type.Literal("create_price_above_sma"),
-      Type.Literal("create_price_below_sma"),
-      Type.Literal("create_rsi_above"),
-      Type.Literal("create_rsi_below"),
-      Type.Literal("create_volume_spike"),
-      Type.Literal("create_percent_move_up"),
-      Type.Literal("create_percent_move_down"),
-      Type.Literal("create_sma_cross_above"),
-      Type.Literal("create_sma_cross_below"),
-      Type.Literal("set_enabled"),
-      Type.Literal("list"),
-      Type.Literal("status"),
-      Type.Literal("check"),
-    ],
-    { description: ACTION_DESCRIPTION },
-  ),
-  symbol: Type.Optional(Type.String({ description: "Ticker symbol for create actions" })),
-  threshold: Type.Optional(
-    Type.Number({ description: "Price or indicator threshold for create actions" }),
-  ),
-  period: Type.Optional(
-    Type.Integer({
-      minimum: 1,
-      description:
-        "Indicator lookback period for SMA/RSI alerts. Max: 200 for price-SMA, 100 for RSI/volume alerts",
-    }),
-  ),
-  fast_period: Type.Optional(
-    Type.Integer({
-      minimum: 1,
-      description: "Fast SMA lookback period for SMA-cross alerts. Default: 50",
-    }),
-  ),
-  slow_period: Type.Optional(
-    Type.Integer({
-      minimum: 1,
-      maximum: 400,
-      description: "Slow SMA lookback period for SMA-cross alerts. Default: 200, max: 400",
-    }),
-  ),
-  cooldown_seconds: Type.Optional(
-    Type.Integer({
-      minimum: 0,
-      description: "Cooldown between repeated trigger events. Default: 3600",
-    }),
-  ),
-  id: Type.Optional(Type.Number({ description: "Alert rule id for update actions" })),
-  enabled: Type.Optional(Type.Boolean({ description: "Whether an alert rule is enabled" })),
-  check_after_create: Type.Optional(
-    Type.Boolean({
-      description:
-        "Set true only when the user asks to check/run alerts immediately after creating this alert in the same request.",
-    }),
-  ),
-});
-
-export const alertsTool: AgentTool<typeof params> = {
-  name: "manage_alerts",
-  label: "Alerts",
-  description:
-    "Create, pause/resume, list, check, and inspect status for durable local alerts. Actions include create_price_above, create_price_below, create_price_above_sma, create_price_below_sma, create_rsi_above, create_rsi_below, create_volume_spike, create_percent_move_up, create_percent_move_down, create_sma_cross_above, create_sma_cross_below, set_enabled, list, status, and check. Local background monitoring runs only while an OpenCandle writer/monitor process is active; manual checks are always available. If the user asks to create an alert and check it now, use the create action with check_after_create=true.",
-  parameters: params,
-  async execute(_toolCallId, args) {
-    const db = initDefaultDatabase();
-    const service = new MarketStateService(db);
-
-    try {
-      const cooldownSeconds = validateCooldownSeconds(args.cooldown_seconds);
-
-      if (args.action === "create_price_above" || args.action === "create_price_below") {
-        if (!args.symbol || args.threshold == null) {
-          throw new Error("symbol and threshold are required for create alert actions.");
-        }
-        const resolution = await resolveInstrumentForMutation(args.symbol);
-        if (resolution.status === "needs_selection") return candidateResult(resolution, "alert");
-        const instrument = service.upsertInstrumentRecord(resolution.instrument);
-        const isAbove = args.action === "create_price_above";
-        const rule = service.createAlertRule({
-          scopeType: "instrument",
-          instrumentId: instrument.id,
-          conditionType: isAbove ? "price_crosses_above" : "price_crosses_below",
-          conditionVersion: ALERT_CONDITION_VERSION,
-          condition: isAbove
-            ? priceCrossesAbove(args.threshold)
-            : priceCrossesBelow(args.threshold),
-          timeframe: "quote",
-          cooldownSeconds,
-        });
-        return await createResultMaybeChecked(
-          service,
-          rule,
-          `Created local alert ${rule.conditionType} for ${instrument.symbol} at $${args.threshold}.`,
-          args.check_after_create,
-        );
-      }
-
-      if (args.action === "create_price_above_sma" || args.action === "create_price_below_sma") {
-        if (!args.symbol) {
-          throw new Error("symbol is required for SMA alert actions.");
-        }
-        // Runner evaluates price_crosses_sma against 1y of daily bars (~252).
-        const period = validateLookbackPeriod(args.period ?? 50, 200);
-        const resolution = await resolveInstrumentForMutation(args.symbol);
-        if (resolution.status === "needs_selection") return candidateResult(resolution, "alert");
-        const instrument = service.upsertInstrumentRecord(resolution.instrument);
-        const direction = args.action === "create_price_above_sma" ? "above" : "below";
-        const rule = service.createAlertRule({
-          scopeType: "instrument",
-          instrumentId: instrument.id,
-          conditionType: "price_crosses_sma",
-          conditionVersion: ALERT_CONDITION_VERSION,
-          condition: priceCrossesSma(period, direction),
-          timeframe: "1d",
-          cooldownSeconds,
-        });
-        return await createResultMaybeChecked(
-          service,
-          rule,
-          `Created local alert price_crosses_sma for ${instrument.symbol} using SMA(${period}).`,
-          args.check_after_create,
-        );
-      }
-
-      if (args.action === "create_rsi_above" || args.action === "create_rsi_below") {
-        if (!args.symbol || args.threshold == null) {
-          throw new Error("symbol and threshold are required for RSI alert actions.");
-        }
-        // Runner evaluates rsi_threshold against 6mo of daily bars (~126).
-        const period = validateLookbackPeriod(args.period ?? 14, 100);
-        const resolution = await resolveInstrumentForMutation(args.symbol);
-        if (resolution.status === "needs_selection") return candidateResult(resolution, "alert");
-        const instrument = service.upsertInstrumentRecord(resolution.instrument);
-        const direction = args.action === "create_rsi_above" ? "above" : "below";
-        const rule = service.createAlertRule({
-          scopeType: "instrument",
-          instrumentId: instrument.id,
-          conditionType: "rsi_threshold",
-          conditionVersion: ALERT_CONDITION_VERSION,
-          condition: rsiThreshold(period, args.threshold, direction),
-          timeframe: "1d",
-          cooldownSeconds,
-        });
-        return await createResultMaybeChecked(
-          service,
-          rule,
-          `Created local alert rsi_threshold for ${instrument.symbol}: RSI(${period}) ${direction} ${args.threshold}.`,
-          args.check_after_create,
-        );
-      }
-
-      if (args.action === "create_volume_spike") {
-        if (!args.symbol) {
-          throw new Error("symbol is required for volume-spike alert actions.");
-        }
-        // Runner evaluates volume_spike against 6mo of daily bars (~126).
-        const period = validateLookbackPeriod(args.period ?? 20, 100);
-        const multiplier = args.threshold ?? 2;
-        const resolution = await resolveInstrumentForMutation(args.symbol);
-        if (resolution.status === "needs_selection") return candidateResult(resolution, "alert");
-        const instrument = service.upsertInstrumentRecord(resolution.instrument);
-        const rule = service.createAlertRule({
-          scopeType: "instrument",
-          instrumentId: instrument.id,
-          conditionType: "volume_spike",
-          conditionVersion: ALERT_CONDITION_VERSION,
-          condition: volumeSpike(period, multiplier),
-          timeframe: "1d",
-          cooldownSeconds,
-        });
-        return await createResultMaybeChecked(
-          service,
-          rule,
-          `Created local alert volume_spike for ${instrument.symbol}: volume > ${multiplier}x ${period}-bar average.`,
-          args.check_after_create,
-        );
-      }
-
-      if (args.action === "create_percent_move_up" || args.action === "create_percent_move_down") {
-        if (!args.symbol || args.threshold == null) {
-          throw new Error("symbol and threshold are required for percent-move alert actions.");
-        }
-        if (args.threshold <= 0) {
-          throw new Error("threshold must be greater than 0 for percent-move alert actions.");
-        }
-        const resolution = await resolveInstrumentForMutation(args.symbol);
-        if (resolution.status === "needs_selection") return candidateResult(resolution, "alert");
-        const instrument = service.upsertInstrumentRecord(resolution.instrument);
-        const direction = args.action === "create_percent_move_up" ? "up" : "down";
-        const rule = service.createAlertRule({
-          scopeType: "instrument",
-          instrumentId: instrument.id,
-          conditionType: "percent_move",
-          conditionVersion: ALERT_CONDITION_VERSION,
-          condition: percentMove(direction, args.threshold),
-          timeframe: "1d",
-          cooldownSeconds,
-        });
-        return await createResultMaybeChecked(
-          service,
-          rule,
-          `Created local alert percent_move for ${instrument.symbol}: ${direction} ${args.threshold}%.`,
-          args.check_after_create,
-        );
-      }
-
-      if (args.action === "create_sma_cross_above" || args.action === "create_sma_cross_below") {
-        if (!args.symbol) {
-          throw new Error("symbol is required for SMA-cross alert actions.");
-        }
-        const fastPeriod = args.fast_period ?? 50;
-        const slowPeriod = args.slow_period ?? 200;
-        if (!Number.isInteger(fastPeriod) || !Number.isInteger(slowPeriod)) {
-          throw new Error(
-            "fast_period and slow_period must be whole-number lookback periods for SMA-cross alert actions.",
-          );
-        }
-        if (fastPeriod <= 0 || slowPeriod <= 0) {
-          throw new Error(
-            "fast_period and slow_period must be greater than 0 for SMA-cross alert actions.",
-          );
-        }
-        if (fastPeriod >= slowPeriod) {
-          throw new Error("fast_period must be less than slow_period for SMA-cross alert actions.");
-        }
-        // Runner evaluates sma_cross against 2y of daily bars (~504).
-        if (slowPeriod > 400) {
-          throw new Error(
-            "slow_period must be at most 400 so alert checks can evaluate it against the runner's 2y daily history window.",
-          );
-        }
-        const resolution = await resolveInstrumentForMutation(args.symbol);
-        if (resolution.status === "needs_selection") return candidateResult(resolution, "alert");
-        const instrument = service.upsertInstrumentRecord(resolution.instrument);
-        const direction = args.action === "create_sma_cross_above" ? "above" : "below";
-        const rule = service.createAlertRule({
-          scopeType: "instrument",
-          instrumentId: instrument.id,
-          conditionType: "sma_cross",
-          conditionVersion: ALERT_CONDITION_VERSION,
-          condition: smaCross(fastPeriod, slowPeriod, direction),
-          timeframe: "1d",
-          cooldownSeconds,
-        });
-        return await createResultMaybeChecked(
-          service,
-          rule,
-          `Created local alert sma_cross for ${instrument.symbol}: SMA(${fastPeriod}) crosses ${direction} SMA(${slowPeriod}).`,
-          args.check_after_create,
-        );
-      }
-
-      if (args.action === "list") {
-        const rules = service.listAlertRules();
-        if (rules.length === 0) {
-          return { content: [{ type: "text", text: "No alert rules created yet." }], details: [] };
-        }
-        const lines = ["**Alerts** — local runner eligible; manual checks available", ""];
-        for (const rule of rules) {
-          const instrument =
-            rule.instrumentId == null ? null : service.getInstrument(rule.instrumentId);
-          lines.push(
-            `  #${rule.id} ${instrument?.symbol ?? "watchlist"} ${rule.conditionType} (${rule.enabled ? "enabled" : "disabled"})`,
-          );
-        }
-        return { content: [{ type: "text", text: lines.join("\n") }], details: rules };
-      }
-
-      if (args.action === "status") {
-        const runnerLease = service.getAutomationRunnerLease();
-        const recentCheckRuns = service.listAlertCheckRuns().slice(0, 10);
-        const stateLine = runnerLease
-          ? `Running locally — ${runnerLease.ownerKind} ${runnerLease.ownerId}; heartbeat ${runnerLease.heartbeatAt}; expires ${runnerLease.expiresAt}.`
-          : "Manual only — no active local runner lease. Keep the GUI/TUI writer or `opencandle monitor` open for background checks.";
-        const lines = ["**Alert Automation Status**", "", stateLine];
-        if (recentCheckRuns.length === 0) {
-          lines.push("", "No alert check runs yet.");
-        } else {
-          lines.push("", "Recent checks:");
-          for (const run of recentCheckRuns) {
-            lines.push(
-              `  #${run.id} ${run.triggerType} ${run.status} checked=${run.checkedCount} triggered=${run.triggeredCount} unavailable=${run.unavailableCount}`,
-            );
-          }
-        }
-        return {
-          content: [{ type: "text", text: lines.join("\n") }],
-          details: { runnerLease, recentCheckRuns },
-        };
-      }
-
-      if (args.action === "set_enabled") {
-        if (args.id == null || args.enabled == null) {
-          throw new Error("id and enabled are required for set_enabled.");
-        }
-        const rule = service.setAlertRuleEnabled(args.id, args.enabled);
-        if (rule == null) {
-          return {
-            content: [
-              {
-                type: "text",
-                text: `Alert #${args.id} not found. Use the list action to see alert ids.`,
-              },
-            ],
-            details: null,
-          };
-        }
-        return {
-          content: [
-            {
-              type: "text",
-              text: `${rule.enabled ? "Enabled" : "Disabled"} alert #${rule.id}.`,
-            },
-          ],
-          details: rule,
-        };
-      }
-
-      const result = await checkAlerts(service);
-      return {
-        content: [{ type: "text", text: result.lines.join("\n") }],
-        details: result,
-      };
-    } finally {
-      db.close();
-    }
-  },
-};
-
-function validateLookbackPeriod(period: number, maxPeriod: number): number {
-  if (!Number.isInteger(period) || period <= 0) {
-    throw new Error(
-      "period must be a whole-number lookback period greater than 0 for indicator alert actions.",
-    );
-  }
-  if (period > maxPeriod) {
-    throw new Error(
-      `period must be at most ${maxPeriod} so alert checks can evaluate it against the runner's daily history window.`,
-    );
-  }
-  return period;
-}
-
-function validateCooldownSeconds(cooldownSeconds: number | undefined): number {
-  const resolved = cooldownSeconds ?? 3600;
-  if (!Number.isInteger(resolved) || resolved < 0) {
-    throw new Error("cooldown_seconds must be a whole number greater than or equal to 0.");
-  }
-  return resolved;
-}
-
-async function checkAlerts(service: MarketStateService): Promise<{
-  checked: number;
-  triggered: number;
-  lines: string[];
-}> {
-  const enabled = service
-    .listAlertRules()
-    .filter((rule) => rule.enabled && rule.status === "active");
-  if (enabled.length === 0)
-    return { checked: 0, triggered: 0, lines: ["No enabled alert rules to check."] };
-
-  const result = await runAlertChecks(service, {
-    ownerId: "manual-tool",
-    triggerType: "manual",
-    providers: defaultAlertRunnerProviders,
-  });
-  await deliverPendingNotifications(service);
-  return {
-    checked: result.checked,
-    triggered: result.triggered,
-    lines: [
-      `**Manual Alert Check** — ${enabled.length} rule(s)`,
-      "",
-      ...result.lines.map((line) => `  ${line}`),
-    ],
-  };
-}
-
-function candidateResult(
-  resolution: Extract<
-    Awaited<ReturnType<typeof resolveInstrumentForMutation>>,
-    { status: "needs_selection" }
-  >,
-  target: string,
-) {
-  return {
-    content: [
-      {
-        type: "text" as const,
-        text: `Could not verify ${resolution.query}. Choose one of the returned candidates before creating the ${target}.`,
-      },
-    ],
-    details: resolution,
-  };
-}
-
-async function createResultMaybeChecked(
-  service: MarketStateService,
-  rule: AlertRuleRecord,
-  createdText: string,
-  checkAfterCreate?: boolean,
-) {
-  if (!checkAfterCreate) {
-    return {
-      content: [{ type: "text" as const, text: createdText }],
-      details: rule,
-    };
-  }
-
-  const check = await checkAlerts(service);
-  return {
-    content: [{ type: "text" as const, text: `${createdText}\n\n${check.lines.join("\n")}` }],
-    details: {
-      created: rule,
-      check,
-    },
-  };
-}

package/src/tools/portfolio/correlation.ts

@@ -1,189 +0,0 @@
-import type { AgentTool } from "@earendil-works/pi-agent-core";
-import { Type } from "@sinclair/typebox";
-import { wrapProvider } from "../../providers/wrap-provider.js";
-import { getHistory } from "../../providers/yahoo-finance.js";
-import type { OHLCV } from "../../types/market.js";
-import { computeDailyReturns } from "./risk-analysis.js";
-
-export function computeCorrelation(returnsA: number[], returnsB: number[]): number {
-  const n = Math.min(returnsA.length, returnsB.length);
-  if (n === 0) return 0;
-
-  let sumA = 0,
-    sumB = 0;
-  for (let i = 0; i < n; i++) {
-    sumA += returnsA[i];
-    sumB += returnsB[i];
-  }
-  const meanA = sumA / n;
-  const meanB = sumB / n;
-
-  let cov = 0,
-    varA = 0,
-    varB = 0;
-  for (let i = 0; i < n; i++) {
-    const dA = returnsA[i] - meanA;
-    const dB = returnsB[i] - meanB;
-    cov += dA * dB;
-    varA += dA * dA;
-    varB += dB * dB;
-  }
-
-  if (varA === 0 || varB === 0) return 0;
-  return cov / Math.sqrt(varA * varB);
-}
-
-const DEFAULT_MIN_OVERLAP = 20;
-const CORRELATION_PERIODS = ["6mo", "1y", "2y"] as const;
-
-export function alignReturnsByDate(
-  historiesBySymbol: Map<string, OHLCV[]>,
-  minOverlap: number = DEFAULT_MIN_OVERLAP,
-): Map<string, number[]> {
-  // Build date → close price maps for each symbol
-  const priceByDate = new Map<string, Map<string, number>>();
-  for (const [symbol, bars] of historiesBySymbol) {
-    const dateMap = new Map<string, number>();
-    for (const bar of bars) {
-      dateMap.set(bar.date, bar.close);
-    }
-    priceByDate.set(symbol, dateMap);
-  }
-
-  // Find common dates across all symbols
-  const symbols = [...historiesBySymbol.keys()];
-  const firstDates = priceByDate.get(symbols[0])!;
-  const commonDates = [...firstDates.keys()]
-    .filter((date) => symbols.every((s) => priceByDate.get(s)!.has(date)))
-    .sort();
-
-  if (commonDates.length < minOverlap) {
-    throw new Error(
-      `Insufficient date overlap for correlation: ${commonDates.length} common dates (need ${minOverlap}+). Symbols may trade on different exchanges or have sparse history.`,
-    );
-  }
-
-  // Extract aligned close prices, then compute returns
-  const result = new Map<string, number[]>();
-  for (const symbol of symbols) {
-    const dateMap = priceByDate.get(symbol)!;
-    const alignedCloses = commonDates.map((d) => dateMap.get(d)!);
-    result.set(symbol, computeDailyReturns(alignedCloses));
-  }
-
-  return result;
-}
-
-const params = Type.Object({
-  symbols: Type.Array(Type.String(), {
-    description:
-      "Array of 2+ ticker symbols to compute correlation matrix (e.g. ['AAPL','MSFT','GOOGL'])",
-    minItems: 2,
-  }),
-  period: Type.Optional(
-    Type.Union(
-      CORRELATION_PERIODS.map((period) => Type.Literal(period)),
-      {
-        description: "Historical period: 6mo, 1y, 2y. Default: 1y",
-      },
-    ),
-  ),
-});
-
-export const correlationTool: AgentTool<typeof params> = {
-  name: "analyze_correlation",
-  label: "Correlation Matrix",
-  description:
-    "Compute pairwise return correlations between 2+ stocks. If one symbol cannot be fetched, computes over the remaining 2+ symbols and lists dropped symbols. Identifies highly correlated positions (|r| > 0.7) as concentration risk. Useful for portfolio diversification analysis.",
-  parameters: params,
-  async execute(_toolCallId, args) {
-    const symbols = args.symbols.map((s) => s.toUpperCase());
-    const period = args.period ?? "1y";
-
-    if (symbols.length < 2) {
-      throw new Error("Need at least 2 symbols for correlation analysis.");
-    }
-
-    // Fetch history for all symbols in parallel
-    const results = await Promise.all(
-      symbols.map(async (s) => ({
-        symbol: s,
-        result: await wrapProvider("yahoo", () => getHistory(s, period, "1d")),
-      })),
-    );
-
-    const dropped = results.flatMap((r) =>
-      r.result.status === "unavailable" ? [{ symbol: r.symbol, reason: r.result.reason }] : [],
-    );
-    const succeeded = results.filter((r) => r.result.status === "ok");
-    if (succeeded.length < 2) {
-      const droppedText =
-        dropped.length > 0
-          ? `\n\nSymbols dropped:\n${dropped.map((d) => `  - ${d.symbol}: ${d.reason}`).join("\n")}`
-          : "";
-      return {
-        content: [
-          {
-            type: "text",
-            text: `⚠ Correlation analysis unavailable — need at least 2 symbols with usable history.${droppedText}`,
-          },
-        ],
-        details: null as any,
-      };
-    }
-
-    const historiesBySymbol = new Map<string, OHLCV[]>();
-    for (const { symbol: sym, result: r } of results) {
-      if (r.status === "ok") historiesBySymbol.set(sym, r.data);
-    }
-
-    const returnsBySymbol = alignReturnsByDate(historiesBySymbol);
-    const survivorSymbols = [...historiesBySymbol.keys()];
-
-    // Build correlation matrix
-    const matrix: Record<string, Record<string, number>> = {};
-    const warnings: string[] = [];
-
-    for (const a of survivorSymbols) {
-      matrix[a] = {};
-      for (const b of survivorSymbols) {
-        if (a === b) {
-          matrix[a][b] = 1.0;
-        } else if (matrix[b]?.[a] != null) {
-          matrix[a][b] = matrix[b][a];
-        } else {
-          const r = computeCorrelation(returnsBySymbol.get(a)!, returnsBySymbol.get(b)!);
-          matrix[a][b] = r;
-          if (Math.abs(r) > 0.7 && a < b) {
-            warnings.push(`${a}/${b}: r=${r.toFixed(2)} — high correlation, concentration risk`);
-          }
-        }
-      }
-    }
-
-    // Format output
-    const header = `**Correlation Matrix** (${period} daily returns)`;
-    const colHeader = `${"".padEnd(8)} ${survivorSymbols.map((s) => s.padStart(8)).join("")}`;
-    const rows = survivorSymbols.map((a) => {
-      const cells = survivorSymbols.map((b) => matrix[a][b].toFixed(2).padStart(8));
-      return `${a.padEnd(8)} ${cells.join("")}`;
-    });
-
-    const lines = [header, "", colHeader, ...rows];
-    if (dropped.length > 0) {
-      lines.push("", "Symbols dropped:");
-      for (const drop of dropped) lines.push(`  - ${drop.symbol}: ${drop.reason}`);
-    }
-    if (warnings.length > 0) {
-      lines.push("", "**Concentration Warnings:**");
-      for (const w of warnings) lines.push(`  - ${w}`);
-    } else {
-      lines.push("", "No high-correlation pairs detected. Portfolio appears diversified.");
-    }
-
-    return {
-      content: [{ type: "text", text: lines.join("\n") }],
-      details: { matrix, warnings, dropped },
-    };
-  },
-};