@oqtopus-team/pi-qdash 0.1.1 → 0.1.3

package/extensions/qdash.ts

@@ -5,9 +5,20 @@ import { homedir } from "node:os";
 
 import type { ExtensionAPI, Theme } from "@earendil-works/pi-coding-agent";
 import { QDashClient, defaultConfigPath, type DownloadedFile, type TaskResultFigureOptions } from "@oqtopus-team/qdash-client";
-import { Image, Text, truncateToWidth, visibleWidth } from "@earendil-works/pi-tui";
+import { Text, truncateToWidth } from "@earendil-works/pi-tui";
 import { Type } from "typebox";
 
+import { analyzePlotlyFigure, figureAnalysisText } from "./lib/figure-analysis.js";
+import { fetchFigureDetails, figureComponent, figureResultText, mediaTypeForPath, type FigureDetails } from "./lib/figures.js";
+import { forumDetailComponent, forumDetailLines, forumListLines, forumPostTitle, forumPostsFromPayload } from "./lib/forum-render.js";
+import { qdashObjectLinks, qdashWebBaseUrl, qdashWebUrl, safeConfig, withQDashLinks } from "./lib/links.js";
+import { arrayFromPayload, compactItems, firstNumber, firstString, formatItem, payloadTotal, statusIcon } from "./lib/payload.js";
+import { toTextToolResult, toToolResult } from "./lib/results.js";
+import { ansi, boxLinesToWidth, boxed, formatNumber, textComponent } from "./lib/render.js";
+import { plotSeriesLines, timeseriesPlotComponent, timeseriesPoints } from "./lib/timeseries-plot.js";
+import { installQDashWriteGate } from "./lib/write-gate.js";
+import { analyzeWiringMarkdown, wiringInsightLines, type WiringInsights } from "./lib/wiring-analysis.js";
+
 type QDashQueryParams = {
   profile?: string;
   configPath?: string;
@@ -111,18 +122,6 @@ type ConfirmableParams = {
   confirmWrite?: boolean;
 };
 
-type FigureDetails = {
-  tool: string;
-  path: string;
-  mediaType: string;
-  sizeBytes: number;
-  base64?: string;
-  text?: string;
-  taskId?: string;
-  figurePaths?: string[];
-  jsonFigurePaths?: string[];
-};
-
 type QDashContextState = {
   profile?: string;
   chipId?: string;
@@ -136,17 +135,6 @@ type QDashContextState = {
 
 const CONTEXT_ENTRY_TYPE = "qdash-context";
 const GLOBAL_CONTEXT_PATH = join(homedir(), ".pi", "agent", "qdash-context.json");
-const WRITE_TOOL_NAMES = new Set([
-  "qdash_create_agent_session",
-  "qdash_submit_agent_action",
-  "qdash_commit_agent_candidate",
-  "qdash_execute_agent_action",
-  "qdash_commit_agent_campaign_candidates",
-  "qdash_apply_agent_candidate_commit",
-  "qdash_create_forum_post",
-  "qdash_update_forum_post",
-  "qdash_create_forum_evidence_reply",
-]);
 let currentContext: QDashContextState = {};
 
 function applyQDashContext<T extends { profile?: string; chipId?: string; sessionId?: string }>(params: T): T {
@@ -448,23 +436,6 @@ type WiringInsightParams = CooldownWiringParams & {
   color?: boolean;
 };
 
-type AttenuationEntry = {
-  mux: string;
-  line: "control" | "readout_send";
-  totalDb: number;
-  components: number[];
-  panel?: string;
-  hardwarePort?: string;
-  qid?: string;
-};
-
-type WiringInsights = {
-  entries: AttenuationEntry[];
-  control: { commonTotalDb?: number; anomalies: AttenuationEntry[]; totals: Record<string, number> };
-  readoutSend: { commonTotalDb?: number; anomalies: AttenuationEntry[]; totals: Record<string, number> };
-  suggestions: string[];
-};
-
 function newestCooldown(cooldowns: CooldownRecord[]): CooldownRecord | undefined {
   return [...cooldowns].sort((a, b) => {
     const activeDifference = Number(a.ended_at != null) - Number(b.ended_at != null);
@@ -473,145 +444,6 @@ function newestCooldown(cooldowns: CooldownRecord[]): CooldownRecord | undefined
   })[0];
 }
 
-function markdownTableRows(markdown: string): string[][] {
-  return markdown.split("\n")
-    .map((line) => line.trim())
-    .filter((line) => line.startsWith("|") && line.endsWith("|"))
-    .map((line) => line.slice(1, -1).split("|").map((cell) => cell.trim()))
-    .filter((cells) => cells.length >= 7 && !cells.every((cell) => /^-+$/.test(cell.replace(/\s/g, ""))));
-}
-
-function expandPanelSpec(value: string): string[] {
-  return value.split(/[、,]/).flatMap((part) => {
-    const token = part.trim();
-    if (!token) return [];
-    const range = token.match(/^([A-Za-z]+)(\d+)\s*-\s*(\d+)$/);
-    if (range) {
-      const [, prefix, startText, endText] = range;
-      const start = Number(startText);
-      const end = Number(endText);
-      const step = start <= end ? 1 : -1;
-      const panels: string[] = [];
-      for (let value = start; step > 0 ? value <= end : value >= end; value += step) panels.push(`${prefix}${value}`);
-      return panels;
-    }
-    return [token];
-  });
-}
-
-function splitPortsAndPanels(cell: string): { ports: string[]; panels: string[] } {
-  const [portsText = cell, panelsText = ""] = cell.split(/\s+\/\s+/, 2);
-  return {
-    ports: portsText.split(/[、,]/).map((part) => part.trim()).filter(Boolean),
-    panels: expandPanelSpec(panelsText),
-  };
-}
-
-function attenuationNumbers(text: string): number[] {
-  return [...text.matchAll(/(?:追加\s*)?(\d+(?:\.\d+)?)\s*dB/gi)].map((match) => Number(match[1])).filter(Number.isFinite);
-}
-
-function panelAttenuationTotals(cell: string): Array<{ panels?: string[]; components: number[]; totalDb: number }> {
-  const parenthesized = [...cell.matchAll(/([^（）]+)（([^）]+)）/g)].map((match) => {
-    const components = attenuationNumbers(match[1]);
-    return { panels: expandPanelSpec(match[2]), components, totalDb: components.reduce((sum, value) => sum + value, 0) };
-  }).filter((segment) => segment.components.length > 0);
-  if (parenthesized.length > 0) return parenthesized;
-  const components = attenuationNumbers(cell);
-  return components.length > 0 ? [{ components, totalDb: components.reduce((sum, value) => sum + value, 0) }] : [];
-}
-
-function mostCommonTotal(entries: AttenuationEntry[]): number | undefined {
-  const counts = new Map<number, number>();
-  for (const entry of entries) counts.set(entry.totalDb, (counts.get(entry.totalDb) ?? 0) + 1);
-  return [...counts.entries()].sort((a, b) => b[1] - a[1] || a[0] - b[0])[0]?.[0];
-}
-
-function totalHistogram(entries: AttenuationEntry[]): Record<string, number> {
-  const histogram: Record<string, number> = {};
-  for (const entry of entries) histogram[String(entry.totalDb)] = (histogram[String(entry.totalDb)] ?? 0) + 1;
-  return histogram;
-}
-
-function analyzeWiringMarkdown(markdown: string | undefined): WiringInsights | undefined {
-  if (!markdown) return undefined;
-  const rows = markdownTableRows(markdown);
-  const dataRows = rows.filter((cells) => /^\d+$/.test(cells[0]));
-  const entries: AttenuationEntry[] = [];
-  for (const cells of dataRows) {
-    const mux = cells[0].padStart(2, "0");
-    const muxNumber = Number(cells[0]);
-    const control = splitPortsAndPanels(cells[1] ?? "");
-    const controlSegments = panelAttenuationTotals(cells[5] ?? "");
-    for (let index = 0; index < control.panels.length; index++) {
-      const panel = control.panels[index];
-      const segment = controlSegments.find((item) => item.panels?.includes(panel)) ?? (controlSegments.length === 1 ? controlSegments[0] : undefined);
-      if (!segment) continue;
-      entries.push({
-        mux,
-        line: "control",
-        panel,
-        hardwarePort: control.ports[index],
-        qid: Number.isFinite(muxNumber) ? String(muxNumber * 4 + index) : undefined,
-        totalDb: segment.totalDb,
-        components: segment.components,
-      });
-    }
-    const readoutSegments = panelAttenuationTotals(cells[6] ?? "");
-    const readout = splitPortsAndPanels(cells[2] ?? "");
-    if (readoutSegments[0]) {
-      entries.push({
-        mux,
-        line: "readout_send",
-        panel: readout.panels[0],
-        hardwarePort: readout.ports[0],
-        totalDb: readoutSegments[0].totalDb,
-        components: readoutSegments[0].components,
-      });
-    }
-  }
-  const controlEntries = entries.filter((entry) => entry.line === "control");
-  const readoutEntries = entries.filter((entry) => entry.line === "readout_send");
-  const controlCommon = mostCommonTotal(controlEntries);
-  const readoutCommon = mostCommonTotal(readoutEntries);
-  const controlAnomalies = controlCommon === undefined ? [] : controlEntries.filter((entry) => entry.totalDb !== controlCommon);
-  const readoutAnomalies = readoutCommon === undefined ? [] : readoutEntries.filter((entry) => entry.totalDb !== readoutCommon);
-  const suggestions = [
-    controlCommon !== undefined ? `Control の最頻総減衰量は ${controlCommon} dB です。` : undefined,
-    controlAnomalies.length > 0
-      ? `Control で最頻値と異なるポートが ${controlAnomalies.length} 件あります: ${controlAnomalies.map((entry) => entry.qid ? `q${entry.qid}` : `${entry.mux}/${entry.panel}`).join(", ")}`
-      : controlEntries.length > 0 ? "Control の総減衰量は全ポートで揃っています。" : undefined,
-    readoutCommon !== undefined ? `Readout send の最頻総減衰量は ${readoutCommon} dB です。` : undefined,
-    readoutAnomalies.length > 0
-      ? `Readout send で最頻値と異なる MUX が ${readoutAnomalies.length} 件あります: ${readoutAnomalies.map((entry) => `MUX${entry.mux}`).join(", ")}`
-      : readoutEntries.length > 0 ? "Readout send の総減衰量は全 MUX で揃っています。" : undefined,
-  ].filter((line): line is string => Boolean(line));
-  return {
-    entries,
-    control: { commonTotalDb: controlCommon, anomalies: controlAnomalies, totals: totalHistogram(controlEntries) },
-    readoutSend: { commonTotalDb: readoutCommon, anomalies: readoutAnomalies, totals: totalHistogram(readoutEntries) },
-    suggestions,
-  };
-}
-
-function wiringInsightLines(insights: WiringInsights, color = false): string[] {
-  const accent = (text: string) => color ? ansi("1;36", text) : text;
-  const muted = (text: string) => color ? ansi("90", text) : text;
-  const warning = (text: string) => color ? ansi("33", text) : text;
-  const formatEntry = (entry: AttenuationEntry) => `${entry.qid ? `q${entry.qid}` : `MUX${entry.mux}`} ${muted(`MUX${entry.mux}`)} ${entry.panel ?? ""}${entry.hardwarePort ? `/${entry.hardwarePort}` : ""}: ${warning(`${entry.totalDb} dB`)} (${entry.components.join("+")})`;
-  return boxed("QDash Wiring Insights", [
-    ...insights.suggestions.map((line) => `• ${line}`),
-    "",
-    accent("Control totals"),
-    `  ${Object.entries(insights.control.totals).map(([total, count]) => `${total} dB × ${count}`).join(", ") || muted("none")}`,
-    ...(insights.control.anomalies.length > 0 ? ["", accent("Control anomalies"), ...insights.control.anomalies.map((entry) => `  ${formatEntry(entry)}`)] : []),
-    "",
-    accent("Readout send totals"),
-    `  ${Object.entries(insights.readoutSend.totals).map(([total, count]) => `${total} dB × ${count}`).join(", ") || muted("none")}`,
-    ...(insights.readoutSend.anomalies.length > 0 ? ["", accent("Readout send anomalies"), ...insights.readoutSend.anomalies.map((entry) => `  ${formatEntry(entry)}`)] : []),
-  ], color);
-}
-
 async function resolveCooldownWiring(params: CooldownWiringParams) {
   params = applyQDashContext(params);
   const client = await makeClient(params);
@@ -682,84 +514,6 @@ async function resolveCooldownWiring(params: CooldownWiringParams) {
   };
 }
 
-function qdashWebBaseUrl(client: QDashClient): string {
-  return client.config.baseUrl.replace(/\/api\/?$/, "").replace(/\/$/, "");
-}
-
-function qdashWebUrl(client: QDashClient, path: string): string {
-  return `${qdashWebBaseUrl(client)}${path.startsWith("/") ? path : `/${path}`}`;
-}
-
-function qdashObjectLinks(client: QDashClient, object: Record<string, unknown>): Record<string, string> {
-  const links: Record<string, string> = {};
-  const taskId = firstString(object, ["task_id", "taskId"]);
-  const executionId = firstString(object, ["execution_id", "executionId"]);
-  const postId = firstString(object, ["post_id", "forum_post_id", "id"]);
-  const issueId = firstString(object, ["issue_id"]);
-  const sessionId = firstString(object, ["session_id", "sessionId"]);
-  if (taskId) links.task_result = qdashWebUrl(client, `/task-results/${encodeURIComponent(taskId)}`);
-  if (executionId) links.execution = qdashWebUrl(client, `/executions/${encodeURIComponent(executionId)}`);
-  if (postId) links.forum_post = qdashWebUrl(client, `/forum/posts/${encodeURIComponent(postId)}`);
-  if (issueId) links.issue = qdashWebUrl(client, `/issues/${encodeURIComponent(issueId)}`);
-  if (sessionId) links.agent_session = qdashWebUrl(client, `/agent-sessions/${encodeURIComponent(sessionId)}`);
-  return links;
-}
-
-function withQDashLinks(client: QDashClient, data: unknown): unknown {
-  if (!data || typeof data !== "object" || Array.isArray(data)) return data;
-  const object = data as Record<string, unknown>;
-  const links = qdashObjectLinks(client, object);
-  if (Object.keys(links).length === 0) return data;
-  return { ...object, _links: links };
-}
-
-function safeConfig(client: QDashClient, source: string) {
-  const config = client.config;
-  return {
-    source,
-    baseUrl: config.baseUrl,
-    projectId: config.projectId ?? null,
-    timeoutSeconds: config.timeoutSeconds,
-    verifyTls: config.verifyTls,
-    proxyConfigured: Boolean(config.proxy),
-    apiTokenConfigured: Boolean(config.apiToken),
-    cloudflareAccessConfigured: Boolean(config.cfAccessClientId || config.cfAccessClientSecret),
-    userAgent: config.userAgent,
-    retry: config.retry,
-  };
-}
-
-function redact(value: unknown): unknown {
-  if (Array.isArray(value)) return value.map(redact);
-  if (value && typeof value === "object") {
-    return Object.fromEntries(
-      Object.entries(value as Record<string, unknown>).map(([key, item]) => {
-        if (/token|password|secret|authorization|api[_-]?key/i.test(key)) return [key, "[redacted]"];
-        return [key, redact(item)];
-      }),
-    );
-  }
-  return value;
-}
-
-function toToolResult(data: unknown, details: Record<string, unknown> = {}) {
-  const safeData = redact(data);
-  let text = JSON.stringify(safeData, null, 2);
-  if (text.length > 20_000) text = `${text.slice(0, 20_000)}\n... [truncated]`;
-  return {
-    content: [{ type: "text" as const, text }],
-    details: { ...details, data: safeData },
-  };
-}
-
-function toTextToolResult(text: string, data: unknown, details: Record<string, unknown> = {}) {
-  const safeData = redact(data);
-  return {
-    content: [{ type: "text" as const, text }],
-    details: { ...details, data: safeData },
-  };
-}
-
 function configProfiles(configPath = defaultConfigPath()): { path: string; exists: boolean; profiles: string[] } {
   if (!existsSync(configPath)) return { path: configPath, exists: false, profiles: [] };
   const contents = readFileSync(configPath, "utf8");
@@ -767,64 +521,6 @@ function configProfiles(configPath = defaultConfigPath()): { path: string; exist
   return { path: configPath, exists: true, profiles };
 }
 
-function arrayFromPayload(payload: unknown): unknown[] {
-  if (Array.isArray(payload)) return payload;
-  if (payload && typeof payload === "object") {
-    const object = payload as Record<string, unknown>;
-    for (const key of ["items", "results", "data", "issues", "executions", "chips", "tasks", "insights"]) {
-      if (Array.isArray(object[key])) return object[key];
-    }
-  }
-  return [];
-}
-
-function compactItems(payload: unknown, keys: string[], limit: number): Record<string, unknown>[] {
-  return arrayFromPayload(payload).slice(0, limit).map((item) => {
-    if (!item || typeof item !== "object") return { value: item };
-    const object = item as Record<string, unknown>;
-    return Object.fromEntries(keys.filter((key) => key in object).map((key) => [key, object[key]]));
-  });
-}
-
-function payloadTotal(payload: unknown): number {
-  const fallback = arrayFromPayload(payload).length;
-  if (!payload || typeof payload !== "object") return fallback;
-  const object = payload as Record<string, unknown>;
-  for (const key of ["total", "total_count", "count", "total_items"]) {
-    const value = object[key];
-    if (typeof value === "number" && Number.isFinite(value)) return value;
-  }
-  return fallback;
-}
-
-function firstString(object: Record<string, unknown>, keys: string[]): string | undefined {
-  for (const key of keys) {
-    const value = object[key];
-    if (typeof value === "string" && value.length > 0) return value;
-    if (typeof value === "number" && Number.isFinite(value)) return String(value);
-  }
-  return undefined;
-}
-
-function statusIcon(status: string | undefined): string {
-  const normalized = status?.toLowerCase() ?? "";
-  if (["completed", "success", "succeeded", "active", "applied"].includes(normalized)) return "✓";
-  if (["failed", "error", "crashed"].includes(normalized)) return "✗";
-  if (["running", "pending", "queued", "in_progress"].includes(normalized)) return "…";
-  if (["cancelled", "canceled"].includes(normalized)) return "-";
-  return "•";
-}
-
-function formatItem(item: Record<string, unknown>, fallbackId: string): string {
-  const id = firstString(item, ["issue_id", "execution_id", "flow_run_id", "task_id", "id"]) ?? fallbackId;
-  const title = firstString(item, ["title", "task_name", "flow_name", "name"]);
-  const status = firstString(item, ["status", "execution_status", "activity_status"]);
-  const target = firstString(item, ["qid", "coupling_id"]);
-  return [statusIcon(status), shortId(id, 14), title, target ? `(${target})` : undefined, status ? `[${status}]` : undefined]
-    .filter(Boolean)
-    .join(" ");
-}
-
 async function buildDashboard(params: { profile?: string; configPath?: string; useEnv?: boolean; chipId?: string; limit?: number }) {
   params = applyQDashContext(params);
   const limit = params.limit ?? 5;
@@ -866,58 +562,6 @@ async function buildDashboard(params: { profile?: string; configPath?: string; u
   };
 }
 
-function ansi(code: string, text: string): string {
-  return `\u001b[${code}m${text}\u001b[0m`;
-}
-
-function stripAnsi(text: string): string {
-  return text.replace(/\u001b\[[0-9;]*m/g, "");
-}
-
-function displayWidth(text: string): number {
-  return visibleWidth(stripAnsi(text));
-}
-
-function padAnsi(text: string, width: number): string {
-  return text + " ".repeat(Math.max(0, width - displayWidth(text)));
-}
-
-function truncateDisplay(text: string, width: number): string {
-  if (displayWidth(text) <= width) return text;
-  const ellipsis = "…";
-  const target = Math.max(0, width - visibleWidth(ellipsis));
-  let out = "";
-  let used = 0;
-  for (const char of Array.from(text)) {
-    const charWidth = visibleWidth(char);
-    if (used + charWidth > target) break;
-    out += char;
-    used += charWidth;
-  }
-  return out + ellipsis;
-}
-
-function boxed(title: string, body: string[], color = false): string[] {
-  const plainTitle = ` ${title} `;
-  const contentWidth = Math.min(
-    92,
-    Math.max(36, ...body.map((line) => displayWidth(line)), visibleWidth(plainTitle)),
-  );
-  const innerWidth = contentWidth + 2;
-  const borderColor = (text: string) => color ? ansi("90", text) : text;
-  const titleText = color ? ansi("1;36", plainTitle) : plainTitle;
-  const top = borderColor("╭") + titleText + borderColor("─".repeat(Math.max(0, innerWidth - plainTitle.length))) + borderColor("╮");
-  const bottom = borderColor("╰" + "─".repeat(innerWidth) + "╯");
-  return [
-    top,
-    ...body.map((line) => {
-      const clipped = color ? truncateToWidth(line, contentWidth) : truncateDisplay(line, contentWidth);
-      return `${borderColor("│")} ${padAnsi(clipped, contentWidth)} ${borderColor("│")}`;
-    }),
-    bottom,
-  ];
-}
-
 function dashboardLines(dashboard: Awaited<ReturnType<typeof buildDashboard>>, color = false): string[] {
   const chips = arrayFromPayload(dashboard.chips);
   const profile = dashboard.context.profile ?? "env/default";
@@ -1713,355 +1357,6 @@ async function buildForumEvidenceReply(client: QDashClient, params: ForumEvidenc
   return { request, preview: content, parent, task, figures };
 }
 
-function forumPostsFromPayload(payload: unknown): Record<string, unknown>[] {
-  if (Array.isArray(payload)) return payload.filter((item): item is Record<string, unknown> => Boolean(item) && typeof item === "object");
-  if (payload && typeof payload === "object") {
-    const object = payload as Record<string, unknown>;
-    for (const key of ["posts", "items", "results", "data"]) {
-      if (Array.isArray(object[key])) return object[key].filter((item): item is Record<string, unknown> => Boolean(item) && typeof item === "object");
-    }
-  }
-  return [];
-}
-
-function forumPostId(post: Record<string, unknown>): string {
-  return firstString(post, ["post_id", "id", "forum_post_id"]) ?? "unknown";
-}
-
-function forumPostTitle(post: Record<string, unknown>): string {
-  return firstString(post, ["title", "subject", "summary"]) ?? firstString(post, ["content", "body", "text"])?.slice(0, 60) ?? "(untitled)";
-}
-
-function firstNumber(object: Record<string, unknown>, keys: string[]): number | undefined {
-  for (const key of keys) {
-    const value = object[key];
-    if (typeof value === "number" && Number.isFinite(value)) return value;
-  }
-  return undefined;
-}
-
-function stringList(value: unknown): string[] {
-  return Array.isArray(value) ? value.filter((item): item is string => typeof item === "string" && item.length > 0) : [];
-}
-
-function forumPostLine(post: Record<string, unknown>): string {
-  const number = firstNumber(post, ["number"]);
-  const id = shortId(forumPostId(post), 10);
-  const title = forumPostTitle(post);
-  const status = firstString(post, ["status"]);
-  const category = firstString(post, ["category"]);
-  const targetType = firstString(post, ["target_type"]);
-  const target = firstString(post, ["target_id", "chip_id", "task_id", "qid"]);
-  const replies = firstNumber(post, ["reply_count"]);
-  const assignee = firstString(post, ["assignee_username", "username"]);
-  return [
-    "•",
-    number ? `#${number}` : id,
-    title,
-    category ? `[${category}]` : undefined,
-    status ? `[${status}]` : undefined,
-    target ? `(${targetType ? `${targetType}:` : ""}${target})` : undefined,
-    replies && replies > 0 ? `↩${replies}` : undefined,
-    assignee ? `@${assignee}` : undefined,
-  ].filter(Boolean).join(" ");
-}
-
-function inlineContentText(value: unknown): string {
-  if (typeof value === "string") return value;
-  if (!Array.isArray(value)) return "";
-  return value.map((item) => {
-    if (typeof item === "string") return item;
-    if (!item || typeof item !== "object") return "";
-    const object = item as Record<string, unknown>;
-    if (typeof object.text === "string") return object.text;
-    if (object.type === "link") return inlineContentText(object.content);
-    return "";
-  }).join("");
-}
-
-function forumBlockLines(blocks: unknown): string[] {
-  if (!Array.isArray(blocks)) return [];
-  const lines: string[] = [];
-  for (const item of blocks) {
-    if (!item || typeof item !== "object") continue;
-    const block = item as Record<string, unknown>;
-    const type = firstString(block, ["type"]) ?? "block";
-    const props = block.props && typeof block.props === "object" ? block.props as Record<string, unknown> : {};
-    const text = inlineContentText(block.content).trimEnd();
-    if (type === "heading") {
-      const level = firstNumber(props, ["level"]) ?? 2;
-      lines.push(`${"#".repeat(Math.max(1, Math.min(6, level)))} ${text}`.trimEnd());
-    } else if (type === "image") {
-      const name = firstString(props, ["name", "caption"]) ?? "image";
-      const url = firstString(props, ["url"]);
-      lines.push(`🖼 ${name}${url ? `  ${url}` : ""}`);
-    } else if (type === "bulletListItem") {
-      lines.push(`- ${text}`);
-    } else if (type === "numberedListItem") {
-      lines.push(`1. ${text}`);
-    } else if (text) {
-      lines.push(...text.split("\n"));
-    }
-    const childLines = forumBlockLines(block.children);
-    if (childLines.length > 0) lines.push(...childLines.map((line) => `  ${line}`));
-  }
-  return lines;
-}
-
-function forumContentLines(object: Record<string, unknown>): string[] {
-  const blockLines = forumBlockLines(object.content_blocks);
-  if (blockLines.length > 0) return blockLines;
-  const content = firstString(object, ["content", "body", "text", "message", "description"]);
-  return content ? content.split("\n") : ["(no content)"];
-}
-
-function forumListLines(payload: unknown, title = "QDash Forum", color = false): string[] {
-  const posts = forumPostsFromPayload(payload);
-  const total = payloadTotal(payload);
-  const accent = (text: string) => color ? ansi("1;36", text) : text;
-  const dim = (text: string) => color ? ansi("2", text) : text;
-  return boxed(title, [
-    `posts ${accent(`${posts.length}/${total}`)}`,
-    "",
-    ...(posts.length > 0 ? posts.map((post) => `  ${forumPostLine(post)}`) : [`  ${dim("none")}`]),
-  ], color);
-}
-
-function forumDetailBodyLines(post: unknown, color = false): string[] {
-  const object = post && typeof post === "object" ? post as Record<string, unknown> : {};
-  const accent = (text: string) => color ? ansi("1;36", text) : text;
-  const muted = (text: string) => color ? ansi("90", text) : text;
-  const labels = stringList(object.labels);
-  const replyCount = firstNumber(object, ["reply_count"]);
-  const meta = [
-    firstString(object, ["category"]) ? `${muted("category")} ${firstString(object, ["category"])}` : undefined,
-    firstString(object, ["status"]) ? `${muted("status")} ${firstString(object, ["status"])}` : undefined,
-    firstString(object, ["target_type"]) || firstString(object, ["target_id"]) ? `${muted("target")} ${[firstString(object, ["target_type"]), firstString(object, ["target_id"])].filter(Boolean).join(":")}` : undefined,
-    firstString(object, ["chip_id"]) ? `${muted("chip")} ${firstString(object, ["chip_id"])}` : undefined,
-    replyCount !== undefined ? `${muted("replies")} ${replyCount}` : undefined,
-  ].filter((line): line is string => typeof line === "string");
-  return [
-    `${muted("id")} ${accent(forumPostId(object))}${firstNumber(object, ["number"]) ? `  ${muted("#")} ${firstNumber(object, ["number"])}` : ""}`,
-    `${muted("title")} ${forumPostTitle(object)}`,
-    `${muted("author")} ${firstString(object, ["username", "user_id"]) ?? "unknown"}${firstString(object, ["assignee_username"]) ? `  ${muted("assignee")} ${firstString(object, ["assignee_username"])}` : ""}`,
-    meta.join("  "),
-    labels.length > 0 ? `${muted("labels")} ${labels.map((label) => `#${label}`).join(" ")}` : undefined,
-    firstString(object, ["created_at"]) ? `${muted("created")} ${firstString(object, ["created_at"])}${firstString(object, ["updated_at"]) ? `  ${muted("updated")} ${firstString(object, ["updated_at"])}` : ""}` : undefined,
-    "",
-    ...forumContentLines(object).map((line) => `  ${line}`),
-  ].filter((line): line is string => typeof line === "string");
-}
-
-function forumDetailLines(post: unknown, title = "QDash Forum Post", color = false): string[] {
-  return boxed(title, forumDetailBodyLines(post, color), color);
-}
-
-function wrapPlainLine(line: string, width: number): string[] {
-  if (width <= 0 || visibleWidth(line) <= width) return [line];
-  const output: string[] = [];
-  let rest = line;
-  while (visibleWidth(rest) > width) {
-    let slice = "";
-    for (const char of rest) {
-      if (visibleWidth(slice + char) > width) break;
-      slice += char;
-    }
-    output.push(slice);
-    rest = rest.slice(slice.length);
-  }
-  if (rest.length > 0) output.push(rest);
-  return output;
-}
-
-function boxLinesToWidth(title: string, body: string[], width: number, theme?: Theme): string[] {
-  const contentWidth = Math.max(24, Math.min(100, width - 4));
-  const border = (text: string) => theme ? theme.fg("borderMuted", text) : text;
-  const titleText = ` ${title} `;
-  const top = `${border("╭")}${theme ? theme.fg("accent", theme.bold(titleText)) : titleText}${border("─".repeat(Math.max(0, contentWidth + 2 - visibleWidth(titleText))))}${border("╮")}`;
-  const bottom = border(`╰${"─".repeat(contentWidth + 2)}╯`);
-  const rows = body.flatMap((line) => wrapPlainLine(line, contentWidth));
-  return [
-    top,
-    ...rows.map((line) => `${border("│")} ${padAnsi(truncateDisplay(line, contentWidth), contentWidth)} ${border("│")}`),
-    bottom,
-  ];
-}
-
-function textComponent(lines: string[], _theme: Theme, wrap = false) {
-  return {
-    render(width: number) {
-      if (!wrap) return lines.map((line) => truncateToWidth(line, width));
-      return lines.flatMap((line) => wrapPlainLine(line, width));
-    },
-    invalidate() {},
-  };
-}
-
-function forumDetailComponent(post: unknown, theme: Theme) {
-  return {
-    render(width: number) {
-      return boxLinesToWidth("QDash Forum Post", forumDetailBodyLines(post), width, theme);
-    },
-    invalidate() {},
-  };
-}
-
-type TimeseriesPoint = {
-  series: string;
-  value: number;
-  at?: string;
-  unit?: string;
-  taskId?: string;
-  executionId?: string;
-};
-
-function timeseriesPoints(payload: unknown): TimeseriesPoint[] {
-  const points: TimeseriesPoint[] = [];
-  const addPoint = (series: string, item: unknown) => {
-    if (!item || typeof item !== "object") return;
-    const object = item as Record<string, unknown>;
-    const value = object.value;
-    if (typeof value !== "number" || !Number.isFinite(value)) return;
-    points.push({
-      series,
-      value,
-      at: firstString(object, ["calibrated_at", "timestamp", "created_at", "start_at"]),
-      unit: firstString(object, ["unit"]),
-      taskId: firstString(object, ["task_id"]),
-      executionId: firstString(object, ["execution_id"]),
-    });
-  };
-  const visit = (value: unknown, series = "series") => {
-    if (Array.isArray(value)) {
-      for (const item of value) addPoint(series, item);
-      return;
-    }
-    if (!value || typeof value !== "object") return;
-    const object = value as Record<string, unknown>;
-    if ("value" in object) {
-      addPoint(series, object);
-      return;
-    }
-    for (const [key, nested] of Object.entries(object)) visit(nested, key);
-  };
-  if (payload && typeof payload === "object" && "data" in payload) visit((payload as Record<string, unknown>).data);
-  else visit(payload);
-  return points.sort((a, b) => (a.at ?? "").localeCompare(b.at ?? ""));
-}
-
-function formatNumber(value: number): string {
-  const abs = Math.abs(value);
-  if (abs >= 1000 || (abs > 0 && abs < 0.001)) return value.toExponential(2);
-  if (abs >= 100) return value.toFixed(1);
-  if (abs >= 10) return value.toFixed(3);
-  return value.toFixed(5).replace(/0+$/, "").replace(/\.$/, "");
-}
-
-function compactDate(value: string | undefined): string {
-  if (!value) return "";
-  const date = new Date(value.endsWith("Z") ? value : `${value}Z`);
-  if (Number.isNaN(date.getTime())) return value.slice(0, 10);
-  return `${String(date.getUTCMonth() + 1).padStart(2, "0")}/${String(date.getUTCDate()).padStart(2, "0")} ${String(date.getUTCHours()).padStart(2, "0")}:${String(date.getUTCMinutes()).padStart(2, "0")}`;
-}
-
-function samplePoints(points: TimeseriesPoint[], width: number): TimeseriesPoint[] {
-  if (points.length <= width) return points;
-  const sampled: TimeseriesPoint[] = [];
-  for (let i = 0; i < width; i++) sampled.push(points[Math.round(i * (points.length - 1) / (width - 1))]);
-  return sampled;
-}
-
-function plotSeriesLines(points: TimeseriesPoint[], options: { title: string; height?: number; width?: number; color?: boolean }): string[] {
-  const height = Math.max(3, Math.min(20, options.height ?? 8));
-  const plotWidth = Math.max(8, Math.min(100, options.width ?? 60));
-  const accent = (text: string) => options.color ? ansi("1;36", text) : text;
-  const muted = (text: string) => options.color ? ansi("90", text) : text;
-  if (points.length === 0) return boxed(options.title, [muted("no numeric data")], options.color);
-
-  const sampled = samplePoints(points, plotWidth);
-  const values = sampled.map((point) => point.value);
-  const min = Math.min(...values);
-  const max = Math.max(...values);
-  const span = max - min || 1;
-  const grid = Array.from({ length: height }, () => Array.from({ length: sampled.length }, () => " "));
-  sampled.forEach((point, col) => {
-    const row = height - 1 - Math.round((point.value - min) / span * (height - 1));
-    grid[Math.max(0, Math.min(height - 1, row))][col] = "●";
-  });
-  const yLabelWidth = Math.max(formatNumber(max).length, formatNumber(min).length, 6);
-  const rows = grid.map((row, index) => {
-    const value = max - (span * index / (height - 1));
-    return `${formatNumber(value).padStart(yLabelWidth)} ┤${row.join("")}`;
-  });
-  const unit = points.find((point) => point.unit)?.unit;
-  const first = points[0];
-  const last = points[points.length - 1];
-  const body = [
-    `${muted("series")} ${accent([...new Set(points.map((point) => point.series))].join(", "))}${unit ? `  ${muted("unit")} ${unit}` : ""}`,
-    `${muted("count")} ${points.length}  ${muted("min")} ${formatNumber(Math.min(...points.map((point) => point.value)))}  ${muted("max")} ${formatNumber(Math.max(...points.map((point) => point.value)))}  ${muted("last")} ${formatNumber(last.value)}`,
-    `${muted("range")} ${compactDate(first.at)} → ${compactDate(last.at)}`,
-    "",
-    ...rows,
-    `${" ".repeat(yLabelWidth)} └${"─".repeat(sampled.length)}`,
-  ];
-  return boxed(options.title, body, options.color);
-}
-
-function timeseriesPlotComponent(data: unknown, title: string, theme: Theme) {
-  return {
-    render(width: number) {
-      const points = timeseriesPoints(data);
-      return plotSeriesLines(points, { title, width: Math.max(8, width - 16) }).map((line) => truncateToWidth(line, width));
-    },
-    invalidate() {},
-  };
-}
-
-function mediaTypeForPath(path: string): string {
-  const lower = path.toLowerCase();
-  if (lower.endsWith(".png")) return "image/png";
-  if (lower.endsWith(".jpg") || lower.endsWith(".jpeg")) return "image/jpeg";
-  if (lower.endsWith(".gif")) return "image/gif";
-  if (lower.endsWith(".webp")) return "image/webp";
-  if (lower.endsWith(".json")) return "application/json";
-  return "application/octet-stream";
-}
-
-async function fetchFigureDetails(client: QDashClient, path: string): Promise<FigureDetails> {
-  const file = await client.getExecutionFigure(path);
-  const bytes = Buffer.from(file.data);
-  const mediaType = file.mediaType || mediaTypeForPath(path);
-  const details: FigureDetails = {
-    tool: "qdash_get_figure",
-    path,
-    mediaType,
-    sizeBytes: bytes.byteLength,
-  };
-  if (mediaType.startsWith("image/")) details.base64 = bytes.toString("base64");
-  else details.text = bytes.toString("utf8");
-  return details;
-}
-
-function figureResultText(details: FigureDetails): string {
-  const lines = boxed("QDash Figure", [
-    `path ${details.path}`,
-    `type ${details.mediaType}`,
-    `size ${details.sizeBytes} bytes`,
-    ...(details.taskId ? [`task ${details.taskId}`] : []),
-    ...(details.figurePaths?.length ? [`figures ${details.figurePaths.length}`] : []),
-    ...(details.jsonFigurePaths?.length ? [`json figures ${details.jsonFigurePaths.length}`] : []),
-    ...(details.text ? ["", ...details.text.split("\n").slice(0, 12).map((line) => `  ${line}`)] : []),
-  ]);
-  return lines.join("\n");
-}
-
-function figureComponent(details: FigureDetails, theme: Theme) {
-  if (details.base64 && details.mediaType.startsWith("image/")) {
-    return new Image(details.base64, details.mediaType, { fallbackColor: (text: string) => theme.fg("dim", text) }, { maxWidthCells: 90, maxHeightCells: 30 });
-  }
-  return textComponent(figureResultText(details).split("\n"), theme);
-}
-
 async function executeQuery(params: QDashQueryParams) {
   params = applyQDashContext(params);
   const client = await makeClient(params);
@@ -2119,25 +1414,7 @@ async function executeQuery(params: QDashQueryParams) {
 }
 
 export default function qdashExtension(pi: ExtensionAPI) {
-  pi.on("tool_call", async (event, ctx) => {
-    if (!WRITE_TOOL_NAMES.has(event.toolName)) return;
-    const input = event.input as { confirmWrite?: boolean };
-    if (input.confirmWrite === true) return;
-
-    if (!ctx.hasUI) {
-      return {
-        block: true,
-        reason: `${event.toolName} is a QDash write operation and requires confirmWrite: true in non-interactive mode.`,
-      };
-    }
-
-    const ok = await ctx.ui.confirm(
-      "Approve QDash write operation?",
-      `${event.toolName} will create or modify data in QDash. Continue?`,
-    );
-    if (!ok) return { block: true, reason: "QDash write operation rejected by user" };
-    input.confirmWrite = true;
-  });
+  installQDashWriteGate(pi);
 
   pi.registerTool({
     name: "qdash_config_info",
@@ -3150,6 +2427,41 @@ export default function qdashExtension(pi: ExtensionAPI) {
     },
   });
 
+  pi.registerTool({
+    name: "qdash_analyze_figure_json",
+    label: "QDash Analyze Figure JSON",
+    description: "Read-only helper that fetches a Plotly JSON calibration figure and summarizes numeric traces for agentic diagnosis.",
+    promptSnippet: "Analyze QDash calibration figure JSON traces without changing parameters",
+    promptGuidelines: [
+      "Use qdash_analyze_figure_json when a calibration figure has json_figure_path and you need numeric evidence from scatter or heatmap traces.",
+      "Treat the output as evidence for a skill/runbook decision; do not commit/apply parameters from this analysis alone.",
+    ],
+    parameters: Type.Object({
+      ...connectionParams,
+      path: Type.Optional(Type.String({ description: "Absolute JSON figure path. If omitted, taskId is required." })),
+      taskId: Type.Optional(Type.String({ description: "Task result ID used to select json_figure_path." })),
+      index: Type.Optional(Type.Number({ description: "JSON figure index for taskId selection. Defaults to 0." })),
+      includeData: Type.Optional(Type.Boolean({ description: "Include parsed Plotly JSON in structured data. Defaults to false." })),
+    }),
+    async execute(_toolCallId, params: { profile?: string; configPath?: string; useEnv?: boolean; path?: string; taskId?: string; index?: number; includeData?: boolean }) {
+      const client = await makeClient(params);
+      let path = params.path;
+      let text = "";
+      if (path) {
+        const details = await fetchFigureDetails(client, path);
+        text = details.text ?? "";
+      } else {
+        const taskId = requireValue(params.taskId, "taskId");
+        const file: TaskResultFigureFile = await client.getTaskResultFigure(taskId, { index: params.index, preferJson: true });
+        path = file.path;
+        text = Buffer.from(file.data).toString("utf8");
+      }
+      const figure = JSON.parse(text) as unknown;
+      const analysis = analyzePlotlyFigure(path ?? "", figure, params.taskId);
+      return toTextToolResult(figureAnalysisText(analysis), { analysis, figure: params.includeData ? figure : undefined }, { tool: "qdash_analyze_figure_json", path, data: analysis });
+    },
+  });
+
   registerQueryTool({
     name: "qdash_get_provenance_stats",
     label: "QDash Provenance Stats",