research-copilot 0.2.17 → 0.2.21

package/app/out/main/index.mjs

@@ -1,9 +1,10 @@
-import { app, shell, ipcMain, BrowserWindow, dialog, Menu } from "electron";
+import { app, shell, BrowserWindow, ipcMain, dialog, protocol, Menu } from "electron";
 import { setMaxListeners } from "node:events";
 import fs, { existsSync as existsSync$1 } from "node:fs";
+import fsp, { readFile } from "node:fs/promises";
 import { execFile, spawn, execSync } from "node:child_process";
-import path, { resolve, join, sep, isAbsolute, extname, dirname, basename, relative } from "path";
-import { existsSync, statSync, readdirSync as readdirSync$1, readFileSync, writeFileSync, mkdirSync, appendFileSync, rmSync, unlinkSync, renameSync, openSync, constants, writeSync, closeSync, watch } from "fs";
+import path, { resolve, join, sep, isAbsolute, basename, extname, dirname, relative } from "path";
+import { existsSync, statSync, readdirSync, readFileSync, cpSync, writeFileSync, mkdirSync, appendFileSync, rmSync, unlinkSync, renameSync, openSync, constants, writeSync, closeSync, watch } from "fs";
 import os$1, { homedir } from "os";
 import { createHash, randomUUID } from "crypto";
 import { Agent } from "@mariozechner/pi-agent-core";
@@ -12,9 +13,9 @@ import { createCodingTools, createGrepTool, createFindTool, createLsTool, DEFAUL
 import { Type } from "@sinclair/typebox";
 import { mkdir, writeFile } from "fs/promises";
 import path$1 from "node:path";
-import fsp from "node:fs/promises";
 import crypto$1, { createHash as createHash$1 } from "node:crypto";
 import { promisify } from "node:util";
+import { Blob } from "node:buffer";
 import os from "node:os";
 import { fileURLToPath } from "node:url";
 import { execFile as execFile$1, execSync as execSync$1 } from "child_process";
@@ -90,7 +91,7 @@ function isIgnored(relativePath, isDirectory, rules, showIgnored) {
 }
 function hasVisibleChildren(dirPath, relativePath, rules, showIgnored) {
   try {
-    const entries = readdirSync$1(dirPath, { withFileTypes: true });
+    const entries = readdirSync(dirPath, { withFileTypes: true });
     for (const entry of entries) {
       const childRelative = relativePath ? `${relativePath}/${entry.name}` : entry.name;
       if (!isIgnored(childRelative, entry.isDirectory(), rules, showIgnored)) {
@@ -107,7 +108,7 @@ function listTreeChildren(rootPath, relativePath = "", showIgnored = false, limi
   if (!isWithinRoot(rootPath, basePath)) return [];
   if (!existsSync(basePath) || !statSync(basePath).isDirectory()) return [];
   const rules = readGitIgnoreRules(rootPath);
-  const entries = readdirSync$1(basePath, { withFileTypes: true });
+  const entries = readdirSync(basePath, { withFileTypes: true });
   const out = [];
   entries.sort((a, b) => {
     if (a.isDirectory() !== b.isDirectory()) return a.isDirectory() ? -1 : 1;
@@ -145,7 +146,7 @@ function searchTree(rootPath, query, showIgnored = false, maxResults = 200) {
     const node = stack.pop();
     let entries = [];
     try {
-      entries = readdirSync$1(node.absPath, { withFileTypes: true });
+      entries = readdirSync(node.absPath, { withFileTypes: true });
     } catch {
       continue;
     }
@@ -180,7 +181,8 @@ function searchTree(rootPath, query, showIgnored = false, maxResults = 200) {
 const DEFAULT_SETTINGS = {
   research: { researchIntensity: "medium", webSearchDepth: "standard", autoSaveSensitivity: "balanced" },
   dataAnalysis: { executionTimeLimit: "standard" },
-  wikiAgent: { model: "none", speed: "medium" }
+  wikiAgent: { model: "none", speed: "medium" },
+  diagram: { reviewProvider: "auto" }
 };
 const CONFIG_DIR = join(homedir(), ".research-copilot");
 const CONFIG_FILE = join(CONFIG_DIR, "config.json");
@@ -247,7 +249,8 @@ function loadSettingsFromConfig() {
   return {
     research: { ...DEFAULT_SETTINGS.research, ...config.settings.research },
     dataAnalysis: { ...DEFAULT_SETTINGS.dataAnalysis, ...config.settings.dataAnalysis },
-    wikiAgent: { ...DEFAULT_SETTINGS.wikiAgent, ...config.settings.wikiAgent }
+    wikiAgent: { ...DEFAULT_SETTINGS.wikiAgent, ...config.settings.wikiAgent },
+    diagram: { ...DEFAULT_SETTINGS.diagram, ...config.settings.diagram }
   };
 }
 function hasLlmAuth() {
@@ -421,7 +424,7 @@ function registerFileHandlers(handle, getCtx) {
     const { projectPath } = getCtx();
     if (!projectPath) return [];
     try {
-      const entries = readdirSync$1(projectPath);
+      const entries = readdirSync(projectPath);
       const files = [];
       for (const entry of entries) {
         if (entry.startsWith(".")) continue;
@@ -496,6 +499,38 @@ function registerFileHandlers(handle, getCtx) {
       return { success: false, error: err.message };
     }
   });
+  handle("file:reveal", (filePath) => {
+    const { projectPath } = getCtx();
+    const absPath = isAbsolute(filePath) ? filePath : resolve(projectPath, filePath);
+    if (!existsSync(absPath)) return { success: false, error: "Path not found." };
+    shell.showItemInFolder(absPath);
+    return { success: true };
+  });
+  handle("file:copy-item", (srcRelPath, destDirRelPath) => {
+    const { projectPath } = getCtx();
+    if (!projectPath) return { success: false, error: "No project folder selected." };
+    const srcAbs = isAbsolute(srcRelPath) ? srcRelPath : resolve(projectPath, srcRelPath);
+    const destDirAbs = isAbsolute(destDirRelPath) ? destDirRelPath : resolve(projectPath, destDirRelPath);
+    if (!isWithinRoot(projectPath, srcAbs)) return { success: false, error: "Source is outside workspace." };
+    if (!isWithinRoot(projectPath, destDirAbs)) return { success: false, error: "Destination is outside workspace." };
+    if (!existsSync(srcAbs)) return { success: false, error: "Source not found." };
+    const name = basename(srcAbs);
+    const ext = extname(name);
+    const stem = ext ? name.slice(0, -ext.length) : name;
+    let destAbs = join(destDirAbs, name);
+    let counter = 1;
+    while (existsSync(destAbs)) {
+      const suffix = counter === 1 ? " copy" : ` copy ${counter}`;
+      destAbs = join(destDirAbs, ext ? `${stem}${suffix}${ext}` : `${stem}${suffix}`);
+      counter++;
+    }
+    try {
+      cpSync(srcAbs, destAbs, { recursive: true });
+      return { success: true, destPath: destAbs };
+    } catch (err) {
+      return { success: false, error: err.message };
+    }
+  });
   handle("file:drop-to-dir", (fileName, base64Content, targetDirRelPath) => {
     const { projectPath } = getCtx();
     if (!projectPath) return { success: false, error: "No project folder selected." };
@@ -781,7 +816,7 @@ function registerAuthHandlers(handleRaw) {
     if (!creds) return { success: false, error: "Not logged in" };
     try {
       const { refreshOpenAICodexToken } = await import("@mariozechner/pi-ai/oauth");
-      const newCreds = await refreshOpenAICodexToken(creds);
+      const newCreds = await refreshOpenAICodexToken(creds.refresh);
       saveCodexCredentials(newCreds);
       return { success: true };
     } catch (err) {
@@ -939,7 +974,7 @@ const PATHS = {
 const _warnedReaddirPaths = /* @__PURE__ */ new Set();
 function safeReaddir(dir) {
   try {
-    return readdirSync$1(dir);
+    return readdirSync(dir);
   } catch (err) {
     const code = err?.code;
     if (code === "EPERM" || code === "ENOENT" || code === "ENOTDIR" || code === "EACCES") {
@@ -1688,7 +1723,7 @@ function listMemoryFiles(projectPath) {
   const dir = memoryDir(projectPath);
   if (!existsSync(dir)) return [];
   try {
-    const files = readdirSync$1(dir).filter((f) => f.endsWith(".md")).sort();
+    const files = readdirSync(dir).filter((f) => f.endsWith(".md")).sort();
     const entries = [];
     for (const filename of files) {
       const entry = readMemoryFile(projectPath, filename);
@@ -2463,6 +2498,7 @@ Hard rules:
 - General web facts → brave_web_search or fetch.
 - If a required paper PDF/full text cannot be retrieved (paywall/auth/access blocked), do NOT infer missing content. Ask user to provide/upload the file and continue only after file is available.
 - Any data analysis / visualization / statistics → data-analyze (do not analyze raw data with read/grep).
+- Any scientific diagram / schematic / flowchart / architecture / pathway / circuit → generate_diagram (it runs a provider-backed generate → review → edit loop; do not draft SVG or ASCII diagrams by hand). If the scientific-schematics skill is loaded, use its guidance to shape the prompt before calling the tool.
 - For reusable methodology, writing scaffolding, or plot/style templates, check if a relevant skill summary is already pre-loaded below. If so, follow it; call load_skill(name) for full procedures when needed. You can also browse the Skills Catalog and load any skill on demand.
 - For repository/text inspection, use this order by default:
   1) glob/grep to locate relevant files/sections;
@@ -4394,160 +4430,2555 @@ const DataAnalyzeSchema = Type.Object({
 });
 function createDataAnalyzeTool(ctx) {
   return {
-    name: "data_analyze",
-    label: "Data Analysis",
-    description: "Analyze a dataset using Python. Supports statistics, visualization (matplotlib/seaborn), data transformation, and modeling. Generated outputs (figures, tables) are saved to disk.\nUsage guidelines: (1) Use this tool for ANY analysis, visualization, statistics, or modeling — do not compute from raw data with read/grep. (2) Generate only the outputs the user requested; no extras.",
-    parameters: DataAnalyzeSchema,
+    name: "data_analyze",
+    label: "Data Analysis",
+    description: "Analyze a dataset using Python. Supports statistics, visualization (matplotlib/seaborn), data transformation, and modeling. Generated outputs (figures, tables) are saved to disk.\nUsage guidelines: (1) Use this tool for ANY analysis, visualization, statistics, or modeling — do not compute from raw data with read/grep. (2) Generate only the outputs the user requested; no extras.",
+    parameters: DataAnalyzeSchema,
+    execute: async (_toolCallId, rawParams) => {
+      const params = rawParams;
+      const filePath = typeof params.file_path === "string" ? params.file_path.trim() : "";
+      const instructions = typeof params.instructions === "string" ? params.instructions.trim() : "";
+      const taskType = typeof params.task_type === "string" ? params.task_type.trim().toLowerCase() : "analyze";
+      if (!filePath) {
+        return toAgentResult("data_analyze", toolError("MISSING_PARAMETER", "Missing file_path.", {
+          suggestions: ["Provide a relative path to the data file (CSV, JSON, TSV, or XLSX)."]
+        }));
+      }
+      if (!instructions) {
+        return toAgentResult("data_analyze", toolError("MISSING_PARAMETER", "Missing instructions.", {
+          suggestions: ["Describe what analysis to perform on the data."]
+        }));
+      }
+      if (!["analyze", "visualize", "transform", "model"].includes(taskType)) {
+        return toAgentResult("data_analyze", toolError("INVALID_PARAMETER", `Invalid task_type: ${taskType}. Use: analyze | visualize | transform | model.`, {
+          suggestions: ["Valid task types: analyze, visualize, transform, model."]
+        }));
+      }
+      const absDataFile = path$1.resolve(ctx.workspacePath, filePath);
+      if (!fs.existsSync(absDataFile)) {
+        return toAgentResult("data_analyze", toolError("FILE_NOT_FOUND", `File not found: ${filePath}`, {
+          suggestions: [
+            "Verify the file path is relative to the workspace root.",
+            "Use the find or glob tool to locate the data file."
+          ],
+          context: { workspacePath: ctx.workspacePath, resolvedPath: absDataFile }
+        }));
+      }
+      const runId = Date.now().toString(36);
+      const outputBase = path$1.join(ctx.workspacePath, ".research-pilot", "data-runs", runId);
+      const figuresDir = path$1.join(outputBase, "figures");
+      const tablesDir = path$1.join(outputBase, "tables");
+      const dataDir = path$1.join(outputBase, "data");
+      fs.mkdirSync(figuresDir, { recursive: true });
+      fs.mkdirSync(tablesDir, { recursive: true });
+      fs.mkdirSync(dataDir, { recursive: true });
+      const resultsFile = path$1.join(outputBase, "results.json");
+      const rawPreview = fs.readFileSync(absDataFile, "utf-8").slice(0, 2e3);
+      const ext = path$1.extname(absDataFile).toLowerCase();
+      const formatHint = ext === ".csv" ? "CSV" : ext === ".tsv" ? "TSV" : ext === ".json" ? "JSON" : ext === ".xlsx" ? "XLSX" : "unknown";
+      if (!ctx.callLlm) {
+        return toAgentResult("data_analyze", toolError("LLM_UNAVAILABLE", "LLM not available for code generation.", {
+          suggestions: ["Ensure the agent runtime has an LLM provider configured (callLlm in ResearchToolContext)."]
+        }));
+      }
+      const taskSection = DATA_ANALYSIS_TASKS.split(/^## /m).find((s) => s.startsWith(taskType));
+      const taskDesc = taskSection ? `## ${taskSection}` : "";
+      const userPrompt = [
+        `Task type: ${taskType} (${TASK_DESCRIPTIONS[taskType] ?? "Analysis"})`,
+        "",
+        taskDesc,
+        "",
+        `Data file format: ${formatHint}`,
+        `Data file preview (first 2000 chars):`,
+        "```",
+        rawPreview,
+        "```",
+        "",
+        `Instructions: ${instructions}`,
+        "",
+        "IMPORTANT: Use the pre-defined path variables (DATA_FILE, FIGURES_DIR, TABLES_DIR, DATA_DIR, RESULTS_FILE).",
+        "Call write_results() at the end with your outputs list and summary dict.",
+        "Output ONLY the Python code in a ```python code block."
+      ].join("\n");
+      let generatedCode;
+      try {
+        const llmResponse = await ctx.callLlm(DATA_ANALYSIS_SYSTEM, userPrompt);
+        const codeMatch = llmResponse.match(/```python\n([\s\S]*?)```/) || llmResponse.match(/```\n([\s\S]*?)```/);
+        generatedCode = codeMatch ? codeMatch[1] : llmResponse;
+      } catch (err) {
+        return toAgentResult("data_analyze", toolError("EXECUTION_FAILED", `Code generation failed: ${err.message}`, {
+          retryable: true,
+          suggestions: ["Retry — the LLM may produce valid code on a subsequent attempt.", "Try simplifying the instructions."]
+        }));
+      }
+      const pathDefinitions = [
+        "",
+        "# Pre-defined paths (set by the tool runtime)",
+        `DATA_FILE = ${JSON.stringify(absDataFile)}`,
+        `FIGURES_DIR = ${JSON.stringify(figuresDir)}`,
+        `TABLES_DIR = ${JSON.stringify(tablesDir)}`,
+        `DATA_DIR = ${JSON.stringify(dataDir)}`,
+        `RESULTS_FILE = ${JSON.stringify(resultsFile)}`,
+        ""
+      ].join("\n");
+      const fullScript = DATA_CODE_TEMPLATE + pathDefinitions + "\n" + generatedCode;
+      const scriptPath = path$1.join(outputBase, "script.py");
+      fs.writeFileSync(scriptPath, fullScript, "utf-8");
+      try {
+        const { stdout, stderr } = await execFileAsync$2("python3", [scriptPath], {
+          cwd: ctx.workspacePath,
+          timeout: ctx.settings?.dataAnalysis?.timeoutMs ?? 12e4,
+          maxBuffer: 10 * 1024 * 1024
+        });
+        let manifest = null;
+        if (fs.existsSync(resultsFile)) {
+          try {
+            manifest = JSON.parse(fs.readFileSync(resultsFile, "utf-8"));
+          } catch {
+          }
+        }
+        const outputs = [];
+        for (const [dir, type] of [
+          [figuresDir, "figure"],
+          [tablesDir, "table"],
+          [dataDir, "data"]
+        ]) {
+          if (fs.existsSync(dir)) {
+            for (const f of fs.readdirSync(dir)) {
+              outputs.push({
+                name: f,
+                type,
+                path: path$1.relative(ctx.workspacePath, path$1.join(dir, f))
+              });
+            }
+          }
+        }
+        const payload = {
+          stdout: stdout.slice(0, 4e3),
+          stderr: stderr ? stderr.slice(0, 1e3) : void 0,
+          outputs,
+          manifest: manifest ? {
+            summary: manifest.summary,
+            warnings: manifest.warnings
+          } : void 0,
+          scriptPath: path$1.relative(ctx.workspacePath, scriptPath),
+          runId
+        };
+        return toAgentResult("data_analyze", { success: true, data: payload });
+      } catch (err) {
+        const errorDetail = err.stderr?.slice(0, 2e3) || err.message;
+        return toAgentResult("data_analyze", toolError("EXECUTION_FAILED", `Python execution failed: ${errorDetail}`, {
+          retryable: true,
+          suggestions: [
+            `Review the generated script at ${path$1.relative(ctx.workspacePath, scriptPath)} for errors.`,
+            "Check that required Python packages (pandas, matplotlib, seaborn, etc.) are installed.",
+            "Try simplifying the analysis instructions."
+          ],
+          context: {
+            scriptPath: path$1.relative(ctx.workspacePath, scriptPath),
+            runId
+          },
+          data: {
+            scriptPath: path$1.relative(ctx.workspacePath, scriptPath),
+            runId
+          }
+        }));
+      }
+    }
+  };
+}
+const DEFAULT_HOUSE_PROFILE = {
+  id: "editorial-institutional-v1",
+  themeNarrative: [
+    "Editorial scientific figure with a fixed institutional identity.",
+    "Off-white or warm-light background, graphite text and strokes, restrained contrast, disciplined whitespace.",
+    'No generic "AI" aesthetic (no glow, no particles, no gradient backgrounds).',
+    "No startup-whitepaper look (no flat-design pastels, no rounded-pill overloads).",
+    "Every figure in this system should look like it belongs to the same visual family across papers, slides, and reports."
+  ].join(" "),
+  typography: {
+    hierarchy: {
+      sectionLabel: "700 13px uppercase tracked",
+      nodeLabel: "600 12px",
+      edgeAnnotation: "500 10px",
+      footLabel: "400 10px italic"
+    },
+    fontStack: ["Inter", "SF Pro Text", "Helvetica Neue", "Arial", "sans-serif"],
+    voice: "Compact, disciplined, editorial. Hierarchy is fixed. No oversized headings, no micro-labels under 10px."
+  },
+  palette: [
+    { role: "text", value: "#1F2937", description: "Graphite text and primary strokes." },
+    { role: "stroke", value: "#1F2937", description: "Same graphite for structural outlines." },
+    { role: "primaryStructure", value: "#2C5282", description: "Primary structural entities — main nodes, canonical path." },
+    { role: "secondaryStructure", value: "#718096", description: "Secondary structural entities — supporting nodes, context." },
+    { role: "contextFill", value: "#F7FAFC", description: "Off-white background fill for grouping containers." },
+    { role: "resultAccent", value: "#2B6CB0", description: "Deep editorial blue, used sparingly on result/output elements." },
+    { role: "warningAccent", value: "#C53030", description: "Deep editorial red, used only for warnings or peak-state emphasis." },
+    { role: "grid", value: "#E2E8F0", description: "Soft graphite grid or axis line." }
+  ],
+  geometry: {
+    strokeWidth: { primary: "1.5px", secondary: "1px" },
+    cornerRadius: "8px",
+    arrowheadStyle: "Filled triangle, width 8px, extending 9px along the stroke — never hollow arrows, never emoji.",
+    groupContainerStyle: "Dashed 1px secondary-structure border with contextFill background; no solid coloured group panels.",
+    outerMargin: "5% of the viewBox on every side",
+    minGutter: "32px between adjacent boxes",
+    boxHeightSteps: ["48px", "72px", "96px"]
+  },
+  motifs: [
+    "Labels sit above or to the left of the element they describe; never inside node boxes unless the box is a labelled region.",
+    "Arrow stroke matches its source node colour.",
+    "Consistent gutter rhythm across panels in multi-panel figures.",
+    "Section labels (panel headers, region titles) always use the same section-label typography token.",
+    "No more than two accent colours visible in a single figure."
+  ],
+  avoid: [
+    "Gradients, glow, blur, or filter effects of any kind",
+    "Rounded-pill buttons, soft-shadow cards, or other app-UI tropes",
+    "Neon or highly saturated colours",
+    "Oversized emoji, cartoon icons, or mascot-style illustrations",
+    "Display fonts, script fonts, or hand-lettering styles"
+  ]
+};
+function renderPalette(tokens) {
+  const lines = [
+    "Semantic role → colour mapping (house palette). Use these tokens as defaults; a user prompt may override any specific role."
+  ];
+  for (const t of tokens) {
+    lines.push(`- ${t.role}: ${t.value} — ${t.description}`);
+  }
+  lines.push("All category distinctions must remain readable in grayscale AND under colour-vision deficiency; fall back to an accessible substitute only if a house palette choice fails that test.");
+  return lines.join("\n");
+}
+function renderGeometry(g) {
+  return [
+    "Geometry language (house tokens — reuse across every diagram in the system):",
+    `- Stroke widths: primary ${g.strokeWidth.primary}, secondary ${g.strokeWidth.secondary}.`,
+    `- Corner radius: ${g.cornerRadius} for all rounded rectangles.`,
+    `- Arrowheads: ${g.arrowheadStyle}`,
+    `- Group container: ${g.groupContainerStyle}`,
+    `- Outer margin: ${g.outerMargin}. Min gutter between elements: ${g.minGutter}.`,
+    `- Node box heights: choose one of ${g.boxHeightSteps.join(" / ")} and keep it consistent within a panel.`
+  ].join("\n");
+}
+function renderTypography(t) {
+  return [
+    `Typographic voice: ${t.voice}`,
+    `Font stack (first installed wins): ${t.fontStack.join(", ")}.`,
+    `Hierarchy tokens (use these exact sizes/weights):`,
+    `- Section label:      ${t.hierarchy.sectionLabel}`,
+    `- Node label:         ${t.hierarchy.nodeLabel}`,
+    `- Edge annotation:    ${t.hierarchy.edgeAnnotation}`,
+    `- Foot label:         ${t.hierarchy.footLabel}`
+  ].join("\n");
+}
+function renderMotifs(motifs) {
+  if (motifs.length === 0) return "";
+  return [
+    'Recurring motifs (these are the "signature" details reviewers should see every time):',
+    ...motifs.map((m) => `- ${m}`)
+  ].join("\n");
+}
+function renderProfile(profile = DEFAULT_HOUSE_PROFILE) {
+  return {
+    theme: profile.themeNarrative,
+    typography: renderTypography(profile.typography),
+    palette: renderPalette(profile.palette),
+    geometry: renderGeometry(profile.geometry),
+    motifs: renderMotifs(profile.motifs),
+    avoid: profile.avoid,
+    summaryForReviewer: [
+      `House profile: ${profile.id}.`,
+      `Theme: ${profile.themeNarrative}`,
+      `Palette roles: ${profile.palette.map((p) => `${p.role}=${p.value}`).join(", ")}.`,
+      `Geometry: stroke ${profile.geometry.strokeWidth.primary}/${profile.geometry.strokeWidth.secondary}, corner ${profile.geometry.cornerRadius}, gutter ${profile.geometry.minGutter}.`,
+      `Typography: ${profile.typography.voice} Stack: ${profile.typography.fontStack[0]}.`
+    ].join(" ")
+  };
+}
+const DESIGN_PRINCIPLES = `【DESIGN PRINCIPLES】 Apply all nine. These are non-negotiable:
+
+  1. SINGLE PRIMARY AXIS — Pick one dominant spatial axis (left-to-right
+     flow, top-to-bottom flow, or a time axis) and make every other
+     element serve it. Never mix two competing primary axes.
+
+  2. NO NESTED SOLID-BORDERED BOXES — Group elements using:
+       • a light background tint (fill only, no stroke)
+       • a dashed border (for "optional / future / sketch" categories)
+       • whitespace and alignment
+       • a small header label positioned just above the group
+     NEVER place a solid-bordered rect inside another solid-bordered rect.
+     Small backfilled pills (fill only, no stroke) INSIDE a bordered rect
+     are fine — one level of visual nesting, not two.
+
+  3. TYPOGRAPHY BY CATEGORY — Match font size, weight, and style to the
+     element's CATEGORY, not to its perceived "importance". Consistent
+     scales:
+       • section header:    14 pt bold, Title Case ("Monitoring Pipeline")
+                            NEVER ALL CAPS ("MONITORING PIPELINE") — ALL
+                            CAPS is reserved for the red "!" error prefix
+                            only. Section headers in caps look like
+                            PowerPoint slides, not publication figures.
+       • node title:        12-13 pt semibold or bold
+       • sub-op label:      10 pt regular
+       • caption:           9-10 pt italic
+       • footnote:          9 pt regular, muted grey
+     Never vary font size within the same category.
+
+  4. SEMANTIC COLOUR + COMPACT LEGEND — Every colour must mean something.
+     If you introduce a colour, explain it in a small (≤ 15% of figure
+     area) legend placed in a quiet corner. Typical semantic roles:
+       • navy blue            = current / in-scope / active
+       • forest green         = output / result
+       • dashed muted amber   = planned / future / optional
+       • saturated red        = error / problem / rejection ONLY
+
+  5. REAL CONTENT, NOT PLACEHOLDERS — Boxes must contain real labels,
+     real parameters, real units. "Filter" alone is weak; "Filter —
+     threshold 5°, deadband, clipping" tells the reader what the box
+     actually does. Prefer specifics (100 Hz, int16, 2.4 MB/s) over
+     generic descriptors ("fast", "large", "Module A").
+
+  6. ICONS REPLACE TEXT, NEVER DECORATE — Only draw an icon if it
+     replaces a short phrase more clearly than text would. Wavy line =
+     signal. Monitor glyph = display. Alert bell = notification.
+     Never add icons for decoration.
+
+  7. ARROWS ARE ANCHORED, LABELS RIDE THE LINE, MULTI-BEND = ONE POLYLINE
+     • Every arrow must start and end precisely on a box edge (use
+       rect.x, rect.y, rect.x+width, or rect.y+height).
+     • Edge labels sit ON the arrow, not floating near it. Use a small
+       opaque backfill rect behind the label so the line is masked.
+     • L-shape, Z-shape, or any multi-bend arrow MUST be ONE
+       <polyline points="x1,y1 x2,y2 x3,y3 ..."> element with all
+       vertices in a single declaration. NEVER stitch together
+       multiple <line> elements to approximate an L-shape — they
+       render as visually disconnected segments.
+
+  8. RED IS RESERVED FOR PROBLEMS — Saturated red (≈#C53030) appears
+     only on error / loss / rejection callouts. Muted reds (rust,
+     terracotta) may be used for categorical "sketch / proposal"
+     borders, but high-contrast red must stay precious.
+
+  9. COMPACT LAYOUT — Let content density drive spacing, not the
+     viewBox. Typical gaps:
+       • adjacent elements in the same group:  10–30 px
+       • distinct sections (e.g. "Planned" vs current work):  40–60 px
+       • group backdrop padding (inside tint):  15–25 px
+     NEVER exceed ~80 px of uninterrupted empty space unless the gap
+     is semantically meaningful. If the user-provided viewBox is
+     larger than your content needs, SHRINK THE VIEWBOX — do NOT
+     stretch or spread content to fill the canvas. "Below X" means a
+     20–30 px gap below X, NOT "somewhere at the bottom of the
+     canvas". A figure with visible gutters of empty space looks
+     unfinished regardless of how clean the content itself is.`;
+const POSITIVE_EXAMPLE_SVG = `<svg viewBox="0 0 600 440" xmlns="http://www.w3.org/2000/svg">
+  <defs>
+    <marker id="arrow" viewBox="0 0 10 10" refX="9" refY="5" markerWidth="9" markerHeight="9" orient="auto">
+      <path d="M0,0 L10,5 L0,10 Z" fill="#1F2937"/>
+    </marker>
+    <marker id="arrowBlue" viewBox="0 0 10 10" refX="9" refY="5" markerWidth="9" markerHeight="9" orient="auto">
+      <path d="M0,0 L10,5 L0,10 Z" fill="#2C5282"/>
+    </marker>
+    <marker id="arrowAmber" viewBox="0 0 10 10" refX="9" refY="5" markerWidth="9" markerHeight="9" orient="auto">
+      <path d="M0,0 L10,5 L0,10 Z" fill="#9C5C00"/>
+    </marker>
+    <marker id="arrowRed" viewBox="0 0 10 10" refX="9" refY="5" markerWidth="9" markerHeight="9" orient="auto">
+      <path d="M0,0 L10,5 L0,10 Z" fill="#C53030"/>
+    </marker>
+  </defs>
+
+  <rect x="0" y="0" width="600" height="440" fill="#F7FAFC"/>
+
+  <!-- Legend: compact, top-right. Four categories. -->
+  <rect x="480" y="12" width="110" height="85" rx="4" ry="4" fill="#FFFFFF" stroke="#CBD5E0" stroke-width="1"/>
+  <rect x="492" y="24" width="16" height="10" rx="2" ry="2" fill="#FFFFFF" stroke="#2C5282" stroke-width="1.5"/>
+  <text x="516" y="33" font-family="Inter, sans-serif" font-size="10" fill="#1F2937">current stage</text>
+  <rect x="492" y="42" width="16" height="10" rx="2" ry="2" fill="#FFFFFF" stroke="#2F855A" stroke-width="1.5"/>
+  <text x="516" y="51" font-family="Inter, sans-serif" font-size="10" fill="#1F2937">output</text>
+  <rect x="492" y="60" width="16" height="10" rx="2" ry="2" fill="none" stroke="#9C5C00" stroke-width="1.2" stroke-dasharray="3 2"/>
+  <text x="516" y="69" font-family="Inter, sans-serif" font-size="10" fill="#1F2937">planned</text>
+  <text x="494" y="84" font-family="Inter, sans-serif" font-size="11" font-weight="700" fill="#C53030">!</text>
+  <text x="516" y="84" font-family="Inter, sans-serif" font-size="10" fill="#1F2937">error / loss</text>
+
+  <!-- Section header: Title Case, bold. Above the group. -->
+  <text x="15" y="35" font-family="Inter, sans-serif" font-size="14" font-weight="700" fill="#1F2937">Monitoring Pipeline</text>
+
+  <!-- Pipeline backdrop: light tint, no border — groups Parse/Filter/Aggregate. -->
+  <rect x="155" y="40" width="325" height="170" rx="10" ry="10" fill="#EDF2F7" stroke="none"/>
+
+  <!-- Input: Sensor Stream. Box widened to 130 so all four text lines fit. -->
+  <rect x="15" y="55" width="130" height="140" rx="6" ry="6" fill="#FFFFFF" stroke="#4A5568" stroke-width="1.5"/>
+  <path d="M22,85 q3,-5 6,0 t6,0 t6,0" fill="none" stroke="#2C5282" stroke-width="1.7" stroke-linecap="round"/>
+  <text x="45" y="89" font-family="Inter, sans-serif" font-size="11" font-weight="600" fill="#1F2937">Sensor Stream</text>
+  <text x="25" y="112" font-family="Inter, sans-serif" font-size="9" font-style="italic" fill="#4A5568">raw telemetry • 100 Hz</text>
+  <text x="25" y="132" font-family="Inter, sans-serif" font-size="10" fill="#1F2937">12 channels • int16</text>
+  <text x="25" y="152" font-family="Inter, sans-serif" font-size="9" fill="#4A5568">throughput: 2.4 MB/s</text>
+
+  <line x1="145" y1="125" x2="160" y2="125" stroke="#1F2937" stroke-width="1.5" marker-end="url(#arrow)"/>
+
+  <!-- Parse: title + divider + three sub-op pills (backfill only, no stroke). -->
+  <rect x="160" y="55" width="95" height="140" rx="6" ry="6" fill="#FFFFFF" stroke="#2C5282" stroke-width="1.5"/>
+  <text x="207" y="78" font-family="Inter, sans-serif" font-size="12" font-weight="700" fill="#1F2937" text-anchor="middle">Parse</text>
+  <line x1="170" y1="85" x2="245" y2="85" stroke="#2C5282" stroke-width="0.5" opacity="0.5"/>
+  <rect x="170" y="93" width="75" height="18" rx="3" ry="3" fill="#E6F0FA"/>
+  <text x="175" y="105" font-family="Inter, sans-serif" font-size="10" fill="#1F2937">schema v2</text>
+  <rect x="170" y="117" width="75" height="18" rx="3" ry="3" fill="#E6F0FA"/>
+  <text x="175" y="129" font-family="Inter, sans-serif" font-size="10" fill="#1F2937">decode int16</text>
+  <rect x="170" y="141" width="75" height="18" rx="3" ry="3" fill="#E6F0FA"/>
+  <text x="175" y="153" font-family="Inter, sans-serif" font-size="10" fill="#1F2937">timestamp</text>
+
+  <line x1="255" y1="125" x2="270" y2="125" stroke="#2C5282" stroke-width="1.5" marker-end="url(#arrowBlue)"/>
+
+  <!-- Filter -->
+  <rect x="270" y="55" width="95" height="140" rx="6" ry="6" fill="#FFFFFF" stroke="#2C5282" stroke-width="1.5"/>
+  <text x="317" y="78" font-family="Inter, sans-serif" font-size="12" font-weight="700" fill="#1F2937" text-anchor="middle">Filter</text>
+  <line x1="280" y1="85" x2="355" y2="85" stroke="#2C5282" stroke-width="0.5" opacity="0.5"/>
+  <rect x="280" y="93" width="75" height="18" rx="3" ry="3" fill="#E6F0FA"/>
+  <text x="285" y="105" font-family="Inter, sans-serif" font-size="10" fill="#1F2937">threshold 5°</text>
+  <rect x="280" y="117" width="75" height="18" rx="3" ry="3" fill="#E6F0FA"/>
+  <text x="285" y="129" font-family="Inter, sans-serif" font-size="10" fill="#1F2937">deadband</text>
+  <rect x="280" y="141" width="75" height="18" rx="3" ry="3" fill="#E6F0FA"/>
+  <text x="285" y="153" font-family="Inter, sans-serif" font-size="10" fill="#1F2937">clipping</text>
+
+  <!-- RED error callout — saturated red reserved for problems only. -->
+  <line x1="317" y1="195" x2="317" y2="213" stroke="#C53030" stroke-width="1.2" stroke-dasharray="3 2" marker-end="url(#arrowRed)"/>
+  <rect x="275" y="215" width="85" height="22" rx="3" ry="3" fill="#FED7D7" stroke="#C53030" stroke-width="1"/>
+  <text x="283" y="231" font-family="Inter, sans-serif" font-size="11" font-weight="700" fill="#C53030">!</text>
+  <text x="295" y="231" font-family="Inter, sans-serif" font-size="10" font-weight="500" fill="#C53030">invalid: 3%</text>
+
+  <line x1="365" y1="125" x2="380" y2="125" stroke="#2C5282" stroke-width="1.5" marker-end="url(#arrowBlue)"/>
+
+  <!-- Aggregate -->
+  <rect x="380" y="55" width="95" height="140" rx="6" ry="6" fill="#FFFFFF" stroke="#2C5282" stroke-width="1.5"/>
+  <text x="427" y="78" font-family="Inter, sans-serif" font-size="12" font-weight="700" fill="#1F2937" text-anchor="middle">Aggregate</text>
+  <line x1="390" y1="85" x2="465" y2="85" stroke="#2C5282" stroke-width="0.5" opacity="0.5"/>
+  <rect x="390" y="93" width="75" height="18" rx="3" ry="3" fill="#E6F0FA"/>
+  <text x="395" y="105" font-family="Inter, sans-serif" font-size="10" fill="#1F2937">1-min window</text>
+  <rect x="390" y="117" width="75" height="18" rx="3" ry="3" fill="#E6F0FA"/>
+  <text x="395" y="129" font-family="Inter, sans-serif" font-size="10" fill="#1F2937">mean/max/p99</text>
+  <rect x="390" y="141" width="75" height="18" rx="3" ry="3" fill="#E6F0FA"/>
+  <text x="395" y="153" font-family="Inter, sans-serif" font-size="10" fill="#1F2937">change rate</text>
+
+  <!-- Two arrows diverge from Aggregate bottom, offset horizontally so
+       they don't leave the same pixel. Amber L-shape drops BELOW the
+       red error callout (y=245, callout ends at y=237) to avoid overlap. -->
+  <line x1="447" y1="195" x2="447" y2="252" stroke="#1F2937" stroke-width="1.5" marker-end="url(#arrow)"/>
+  <polyline points="415,195 415,245 160,245 160,252" fill="none" stroke="#9C5C00" stroke-width="1.2" stroke-dasharray="4 3" marker-end="url(#arrowAmber)"/>
+
+  <!-- Planned extension: Title Case header above the dashed-amber box. -->
+  <text x="20" y="250" font-family="Inter, sans-serif" font-size="12" font-weight="700" fill="#9C5C00">Planned</text>
+  <rect x="15" y="255" width="290" height="100" rx="6" ry="6" fill="none" stroke="#9C5C00" stroke-width="1.5" stroke-dasharray="6 5"/>
+  <text x="160" y="277" font-family="Inter, sans-serif" font-size="12" font-weight="600" fill="#1F2937" text-anchor="middle">Anomaly Detection</text>
+  <rect x="30" y="290" width="125" height="20" rx="3" ry="3" fill="#FEF3E6"/>
+  <text x="37" y="304" font-family="Inter, sans-serif" font-size="10" fill="#1F2937">Autoencoder baseline</text>
+  <rect x="165" y="290" width="125" height="20" rx="3" ry="3" fill="#FEF3E6"/>
+  <text x="172" y="304" font-family="Inter, sans-serif" font-size="10" fill="#1F2937">Adaptive threshold</text>
+  <text x="30" y="335" font-family="Inter, sans-serif" font-size="9" font-style="italic" fill="#4A5568">Q3 rollout • replaces static threshold filter</text>
+
+  <!-- Dashboard: different colour family (green) signals OUTPUT category. -->
+  <rect x="310" y="255" width="275" height="120" rx="6" ry="6" fill="#FFFFFF" stroke="#2F855A" stroke-width="1.5"/>
+  <rect x="325" y="272" width="22" height="15" rx="2" ry="2" fill="none" stroke="#2F855A" stroke-width="1.3"/>
+  <line x1="331" y1="291" x2="341" y2="291" stroke="#2F855A" stroke-width="1.3"/>
+  <text x="355" y="285" font-family="Inter, sans-serif" font-size="13" font-weight="700" fill="#1F2937">Dashboard</text>
+  <line x1="325" y1="299" x2="570" y2="299" stroke="#2F855A" stroke-width="0.5" opacity="0.5"/>
+  <text x="325" y="316" font-family="Inter, sans-serif" font-size="10" fill="#1F2937">• live charts (5-sec refresh)</text>
+  <text x="325" y="335" font-family="Inter, sans-serif" font-size="10" fill="#1F2937">• threshold alerts → email / SMS</text>
+  <text x="325" y="354" font-family="Inter, sans-serif" font-size="10" fill="#1F2937">• historical query via Grafana</text>
+</svg>`;
+const POSITIVE_EXAMPLE_BLOCK = `【POSITIVE EXAMPLE】 Study and emulate the design language of this
+reference SVG. It is a yardstick for "what a publication-grade systems
+diagram looks like" — NOT a template to copy literally.
+
+${POSITIVE_EXAMPLE_SVG}
+
+NOTES ON THE REFERENCE (each illustrates a specific principle):
+  • Viewbox 600:440 ≈ 1.36:1 — single-column-friendly, not strongly
+    landscape. Figures for a two-column paper template should usually
+    stay near 1:1, 10:8, or 12:10. (Principle 1)
+  • The three pipeline stages are grouped by a tinted backdrop
+    (fill="#EDF2F7", stroke="none"), NOT by nesting another bordered
+    rect. (Principle 2)
+  • Each stage has title + a subtle divider line + three sub-op pills
+    (fill only, no stroke). Pills INSIDE a bordered rect are fine
+    because pills are backfills, not nested rects. (Principle 2)
+  • Section header "Monitoring Pipeline" is Title Case, not ALL CAPS.
+    The only ALL CAPS in the figure is the red "!" error prefix,
+    where the caps-lock is deliberately emphatic. (Principles 3, 8)
+  • Dashboard uses a DIFFERENT colour family (green, #2F855A) from
+    pipeline stages (navy, #2C5282). The legend explains the
+    categorical distinction. (Principle 4)
+  • Each box contains real, specific content: "threshold 5°",
+    "decode int16", "1-min window", "change rate", "2.4 MB/s". No
+    generic "Module A / Stage 1" placeholders. (Principle 5)
+  • The wavy-signal icon (~~) replaces the phrase "signal" for Sensor
+    Stream; the monitor glyph replaces "display" for Dashboard.
+    Nothing else is iconified. (Principle 6)
+  • Every arrow terminates on a box edge. The "100 Hz" style edge
+    labels (not present here, but the technique is in GEOMETRIC
+    DISCIPLINE) sit ON the arrow with a small opaque backfill rect.
+    (Principle 7)
+  • Saturated red (#C53030) appears ONLY in the "! invalid: 3%"
+    error callout. Muted amber is used for the planned category.
+    (Principle 8)
+  • Two arrows diverge from Aggregate's bottom edge at different x
+    (447 and 415) so they do not leave the same pixel point. The
+    amber L-shape polyline routes BELOW (y=245) the red error callout
+    (which ends at y=237) rather than through it — no arrows cross
+    unrelated boxes. (Principle 7 + GEOMETRIC DISCIPLINE)`;
+const GEOMETRIC_DISCIPLINE = `【GEOMETRIC DISCIPLINE】 Apply these pixel-level rules for all SVG:
+
+  TEXT CENTERING INSIDE A RECT
+    • Vertical center: text.y = rect.y + rect.height/2 + font_size*0.35
+    • Horizontal center: set text-anchor="middle" and
+      text.x = rect.x + rect.width/2
+    • Two-line text inside a rect:
+        line 1 baseline y = rect.cy - 4
+        line 2 baseline y = rect.cy + font_size + 2
+
+  TEXT POSITIONED ABOVE A RECT (as a title)
+    • text.y must be ≤ rect.y - 4 (glyphs extend upward from baseline)
+    • NEVER use text.y = rect.y + 3 style positioning — the title will
+      cross through the rect's top stroke.
+
+  TEXT WIDTH ESTIMATION (Inter font family, approximate):
+    • regular:        char_count × font_size × 0.55
+    • semibold/bold:  char_count × font_size × 0.58
+    • italic:         char_count × font_size × 0.52
+    A rect that contains text must satisfy:
+      rect.width ≥ text_width_estimate + 16   (8 px padding each side)
+    If the text overflows: SHORTEN the text OR SHRINK the font —
+    NEVER let glyphs bleed past the rect's right stroke.
+
+  ARROWS — SOURCE AND DESTINATION ANCHORING
+    • Arrow endpoint (x2, y2) must fall precisely on the destination
+      rect's edge (use rect.x, rect.y, rect.x+width, or rect.y+height).
+    • marker-end with orient="auto" and refX near the marker's tip
+      places the arrow-head tip AT the line endpoint.
+
+  ARROWS CROSSING OTHER BOXES — three priorities, first wins:
+    1. ROUTE AROUND. Shift the arrow's y or x so it never overlaps an
+       unrelated box. This is always preferred.
+    2. USE AN L-SHAPED POLYLINE. A right-angle bend avoids the
+       obstructing box cleanly. Diagonal arrows that clip through
+       other boxes are forbidden.
+    3. Z-ORDER. If an arrow MUST cross a box, declare the arrow's
+       <line> or <polyline> BEFORE the box in SVG source, so the
+       box paints over the arrow (arrow renders behind the box).
+       This is the last-resort fix.
+
+  MULTI-SEGMENT ARROWS (L-shape, Z-shape, any multi-bend)
+    An arrow with one or more bends MUST be a SINGLE <polyline>
+    element with ALL vertices in one declaration:
+        CORRECT:
+          <polyline points="100,50 100,80 250,80"
+                    fill="none" stroke="#1F2937" stroke-width="1.5"
+                    marker-end="url(#arrow)"/>
+        WRONG (produces visually disconnected segments):
+          <line x1="100" y1="50" x2="100" y2="80" .../>
+          <line x1="100" y1="80" x2="250" y2="80" .../>
+    Even when the endpoints line up arithmetically, SVG renders
+    consecutive <line> elements as independent strokes with no
+    shared corner — the result is a broken-looking path. Always
+    use <polyline points="..."> for multi-bend arrows.
+
+  MULTIPLE ARROWS FROM ONE BOX
+    When a single box emits more than one outgoing arrow, offset the
+    starting x (or y) by 20–30 px so the arrows diverge at the box's
+    edge. Two arrows leaving the exact same pixel and then splaying
+    outward looks chaotic.
+
+  EDGE LABELS ON ARROWS
+    Draw a small opaque rect (fill = page background color) BEFORE the
+    text, covering the arrow line at the label's y-midpoint. Then draw
+    the text on top. This masks the line behind the label so the
+    label is readable without floating off the arrow.`;
+const COMMON_MISTAKES = `【COMMON MISTAKES — DO NOT】
+
+  ✗ text.y = rect.y + 3 or text.y = rect.y + font_size*0.2 inside a
+    rect → glyphs cross the rect's top stroke, creating a
+    strikethrough effect. Use the centering formula from GEOMETRIC
+    DISCIPLINE.
+
+  ✗ rect.width narrower than the text's estimated width → glyphs
+    spill past the rect's right stroke. Either widen the rect or
+    shorten / shrink the text.
+
+  ✗ solid-bordered rect inside another solid-bordered rect → visual
+    noise, the "box in box in box" pathology. Use tinted backdrops or
+    dashed borders for grouping instead.
+
+  ✗ a label floating 10–20 px away from its arrow → the reader cannot
+    tell which arrow the label annotates. Place the label ON the
+    arrow with a small opaque bg rect behind the text.
+
+  ✗ diagonal arrows that clip through unrelated boxes → use
+    L-shaped polylines (right-angle bends) to route around instead.
+
+  ✗ two arrows leaving the same pixel of a box and then diverging →
+    offset the starting x or y by 20–30 px so the arrows separate
+    cleanly at the box edge.
+
+  ✗ saturated red used for a non-error element (a section header in
+    red, a decorative border in red) → dilutes the semantic weight
+    of red. Reserve saturated red for errors, losses, rejections.
+
+  ✗ ALL CAPS for regular section headers → dilutes emphasis. Title
+    Case for normal headers; ALL CAPS only where semantic emphasis
+    is intended (red error tags).
+
+  ✗ generic placeholder text: "stage 1", "Component A", "Module" →
+    makes the figure feel empty. Use real domain content ("Filter —
+    threshold 5°", "100 Hz samples", "int16 decode").
+
+  ✗ figures that are strongly landscape (ratio > 1.6:1) for a paper
+    where the template is two-column → the figure will either span
+    both columns (wasteful) or be shrunk into unreadability. Prefer
+    near-square or gently landscape (1:1 to 12:8) for single-column
+    placement.
+
+  ✗ large uninterrupted empty regions (> 80 px in any direction) in
+    the viewBox. Whether you shrink the viewBox or tighten the
+    spacing, a figure with visible gutters looks unfinished. If the
+    request says "Planned section below Layer 1", that's a 20–30 px
+    gap below Layer 1, NOT an empty band in the lower half of the
+    canvas.
+
+  ✗ multi-bend arrow drawn as several <line> elements chained end-
+    to-end → renders as visually disconnected segments. Use a single
+    <polyline points="x1,y1 x2,y2 x3,y3 ..."> element instead.
+
+  ✗ edge label placed on an arrow whose path crosses through a
+    tinted group backdrop → the backdrop, drawn last, paints over
+    the label. Either route the label to lie entirely outside any
+    backdrop, or draw the label AFTER the backdrop in SVG source.`;
+const DIAGRAM_TYPE_DETAILS = {
+  flowchart: `Standard shapes: rectangle = process, diamond = decision, rounded rectangle = start/end. Arrows must point unambiguously from source to target. Every numeric count (n=…) and every branch label (yes/no, conditions) is MANDATORY when mentioned in the subject. Prefer vertical top-to-bottom flow unless specified.`,
+  architecture: `Layered or blocked composition; each component is a labelled box. Show data flow direction with arrows; annotate protocols or interfaces on edges. Group related components (shared background tint or dashed boundary). Consistent box sizes within a layer.`,
+  pathway: `Every molecule/gene/protein labelled with its exact symbol. Arrows distinguish activation (→) from inhibition (⊣ or ⊥). Preserve the exact order and directionality specified. Oval = molecule, rectangle = complex, rounded = process. Include cellular compartments (membrane, nucleus) when relevant.`,
+  circuit: `Standard electronic symbols (IEEE/IEC): resistor zigzag, capacitor parallel lines, ground triangle, op-amp triangle with + and − inputs labelled. Component values carry units ("1kΩ", "10µF", "5V"). Wires: dot at crossing = connection, no dot = jump-over. Ground and supply rails explicitly marked.`,
+  network: `Nodes are labelled entities; edges are relationships. Consistent node sizes unless hierarchy is intentional. For neural networks, label layer types and dimensions ("Dense 128", "Conv 3×3"). For hierarchies, root at top.`,
+  // Previously this said "light graphical flourishes (icons, gradients)
+  // are acceptable if they serve comprehension". That conflicted with
+  // the global no-gradients rule. Conceptual diagrams now follow the
+  // same geometry/colour discipline as the rest — identity is more
+  // valuable than flourish.
+  conceptual: `Focus on clarity. Group related ideas; use the house palette roles to communicate categories. Geometric shapes only — no decorative icons, no photographic inserts, no illustrations.`
+};
+const COMPOSITION_HINTS = {
+  flowchart: "Top-to-bottom vertical flow unless the subject specifies otherwise. Generous gutters between stages.",
+  architecture: "Left-to-right or layered horizontal composition. Align boxes to a common grid.",
+  pathway: "Directional cascade — usually top-to-bottom for signalling, left-to-right for metabolic. Group molecules by compartment.",
+  circuit: "Standard schematic layout: input on the left, output on the right, ground at the bottom.",
+  network: "For neural nets, encoder-left/decoder-right or input-at-top. For trees, root-up.",
+  conceptual: "Choose a composition (flow / radial / Venn / concentric) that makes the relationship obvious, then keep it regular."
+};
+const VERBATIM_TEXT_RULE = `Render every quoted string EXACTLY as given — verbatim, character for character. Do not paraphrase, auto-correct, translate, or substitute. This includes counts like "n=350", identifiers like "EGFR", units like "1kΩ", colour hex codes like "#2B6CB0", and any symbols. If the request contains unfamiliar words or abbreviations, keep them as written.`;
+const UNIVERSAL_AVOID = [
+  'Figure numbers or titles ("Figure 1:", "Fig 1.", etc.) anywhere in the image',
+  "Captions or header bars that repeat information the slide/paper will add around the figure",
+  "Photorealistic photographs, hand-drawn sketches, 3D shading, bevel/emboss/drop-shadow effects",
+  'Watermarks, logos, signatures, or stylistic "AI" ornaments'
+];
+function extractLiterals(prompt) {
+  const found = /* @__PURE__ */ new Set();
+  for (const m of prompt.matchAll(/"([^"]+)"|'([^']+)'/g)) {
+    const value = (m[1] ?? m[2] ?? "").trim();
+    if (value) found.add(`"${value}"`);
+  }
+  for (const m of prompt.matchAll(/\bn\s*=\s*([\d,]+)/gi)) {
+    found.add(`n=${m[1].replace(/,/g, "")}`);
+  }
+  for (const m of prompt.matchAll(/\b\d+(?:\.\d+)?\s*(?:kΩ|Ω|µF|uF|mF|nF|pF|kHz|MHz|GHz|Hz|mV|V|mA|A|pt|px|nm|µm|mm|cm|m|ns|µs|ms|s|nodes|layers)\b/gi)) {
+    found.add(m[0].trim());
+  }
+  for (const m of prompt.matchAll(/#[0-9a-fA-F]{3}(?:[0-9a-fA-F]{3}(?:[0-9a-fA-F]{2})?)?\b/g)) {
+    found.add(m[0]);
+  }
+  for (const m of prompt.matchAll(/\b(?:rgba?|hsla?)\([^)]+\)/gi)) {
+    found.add(m[0]);
+  }
+  return Array.from(found);
+}
+function resolveDiagramType(type) {
+  return type === "auto" ? "conceptual" : type;
+}
+function buildBrief(s) {
+  const lines = [];
+  const svgPath = s.format === "svg";
+  const includeGuide = !s.skipDesignGuide;
+  lines.push(`【SCENE / USE】 ${s.sceneUse}`);
+  lines.push(`【SUBJECT】 ${s.subject}`);
+  if (includeGuide) {
+    lines.push("");
+    lines.push(DESIGN_PRINCIPLES);
+    if (svgPath) {
+      lines.push("");
+      lines.push(POSITIVE_EXAMPLE_BLOCK);
+    }
+  }
+  lines.push("");
+  lines.push(`【COMPOSITION】 ${s.composition}`);
+  lines.push(`【KEY DETAILS】`);
+  lines.push(s.keyDetails);
+  if (includeGuide && svgPath) {
+    lines.push("");
+    lines.push(GEOMETRIC_DISCIPLINE);
+  }
+  if (s.motifs) {
+    lines.push(`【MOTIFS】`);
+    lines.push(s.motifs);
+  }
+  lines.push(`【TEXT】 ${s.textRules}`);
+  if (s.mustKeep.length > 0) {
+    lines.push(`【MUST KEEP】`);
+    for (const item of s.mustKeep) lines.push(`- ${item}`);
+  }
+  if (includeGuide) {
+    lines.push("");
+    lines.push(COMMON_MISTAKES);
+  }
+  lines.push(`【AVOID】`);
+  for (const item of s.avoid) lines.push(`- ${item}`);
+  return lines.join("\n");
+}
+function buildKeyDetails(diagramTypeRules, rendered) {
+  return [
+    diagramTypeRules,
+    "",
+    "Typography",
+    rendered.typography,
+    "",
+    "Colour palette",
+    rendered.palette,
+    "",
+    "Geometry",
+    rendered.geometry
+  ].join("\n");
+}
+function buildAvoid(profile) {
+  return [...UNIVERSAL_AVOID, ...profile.avoid];
+}
+function composeGenerationPrompt(userPrompt, diagramType, format, profile = DEFAULT_HOUSE_PROFILE) {
+  const dt = resolveDiagramType(diagramType);
+  const rendered = renderProfile(profile);
+  const literals = extractLiterals(userPrompt);
+  return buildBrief({
+    sceneUse: `A publication-grade scientific ${dt} diagram in the house visual system. ${rendered.theme}`,
+    subject: userPrompt,
+    composition: COMPOSITION_HINTS[dt],
+    keyDetails: buildKeyDetails(DIAGRAM_TYPE_DETAILS[dt], rendered),
+    motifs: rendered.motifs,
+    textRules: VERBATIM_TEXT_RULE,
+    mustKeep: literals,
+    avoid: buildAvoid(profile),
+    format
+  });
+}
+function composeEditPrompt(userPrompt, diagramType, issues, preservedFixes = [], profile = DEFAULT_HOUSE_PROFILE) {
+  const dt = resolveDiagramType(diagramType);
+  const rendered = renderProfile(profile);
+  const literals = extractLiterals(userPrompt);
+  const fixes = issues.length > 0 ? issues.map((i, idx) => `${idx + 1}. [${i.kind}] ${i.fix}`).join("\n") : "Tighten the details that needed tightening; keep the rest unchanged.";
+  const keyDetailLines = [
+    `TARGETED FIXES (in priority order):`,
+    fixes,
+    `Apply ONLY these fixes. Every other element — including existing labels, arrows, boxes, whitespace, colours, and typography — must be preserved exactly as in the attached image.`
+  ];
+  if (preservedFixes.length > 0) {
+    const preservedLines = preservedFixes.map((p, idx) => `${idx + 1}. [${p.kind}] ${p.description} — already resolved; keep it that way`).join("\n");
+    keyDetailLines.push(
+      `ALREADY-RESOLVED ITEMS (DO NOT regress these — they were broken in a previous draft and fixed):`,
+      preservedLines
+    );
+  }
+  keyDetailLines.push(
+    `Type-specific rules still apply: ${DIAGRAM_TYPE_DETAILS[dt]}`,
+    "",
+    `House-style tokens (unchanged from the original brief — do not drift from these):`,
+    rendered.typography,
+    "",
+    rendered.palette,
+    "",
+    rendered.geometry
+  );
+  return buildBrief({
+    sceneUse: `Surgical revision of an existing ${dt} diagram in the house visual system.`,
+    subject: userPrompt,
+    composition: `Keep the existing layout, sizing, colour palette, and element positions UNCHANGED unless a specific fix below requires otherwise. Do not redraw from scratch.`,
+    keyDetails: keyDetailLines.join("\n"),
+    motifs: rendered.motifs,
+    textRules: VERBATIM_TEXT_RULE,
+    mustKeep: literals,
+    avoid: [
+      ...buildAvoid(profile),
+      "Any change to the overall composition, aspect ratio, or background",
+      "Any change to elements that were already correct",
+      "Regressing any item listed in ALREADY-RESOLVED ITEMS",
+      "Drifting away from the house-style tokens (typography, palette, geometry) listed above"
+    ],
+    // Edit is surgical — the caller has an already-composed draft. Injecting
+    // global design principles here would invite the model to "improve"
+    // elements that were intentionally left alone. Suppress the guide.
+    skipDesignGuide: true
+  });
+}
+function composeRegenPrompt(userPrompt, diagramType, issues, format, profile = DEFAULT_HOUSE_PROFILE) {
+  const base = composeGenerationPrompt(userPrompt, diagramType, format, profile);
+  if (issues.length === 0) return base;
+  const negatives = issues.map((i) => `- ${i.description} (fix: ${i.fix})`).join("\n");
+  return `${base}
+
+【PREVIOUS ATTEMPT HAD THESE PROBLEMS — DO NOT REPEAT】
+${negatives}`;
+}
+function composeStyleOnlyPrompt(userPrompt, diagramType, format, profile = DEFAULT_HOUSE_PROFILE) {
+  const base = composeGenerationPrompt(userPrompt, diagramType, format, profile);
+  return [
+    "The attached image is provided as a STYLE REFERENCE ONLY.",
+    "Match its visual idiom — colour palette, line weights, typography, corner radii, arrow style, spacing rhythm.",
+    "Do NOT copy its layout, content, panel structure, or composition.",
+    "Design a new diagram from scratch for the request below, expressed in that style.",
+    "",
+    base
+  ].join("\n");
+}
+function composeSurgicalRevisionPrompt(userPrompt, diagramType) {
+  const dt = resolveDiagramType(diagramType);
+  const literals = extractLiterals(userPrompt);
+  const mustKeep = literals.length > 0 ? `
+
+【MUST KEEP — verbatim from the request】
+${literals.map((l) => `- ${l}`).join("\n")}` : "";
+  return [
+    "【MODE】 SURGICAL REVISION of an existing SVG.",
+    "",
+    `The SVG source is provided separately. It is a ${dt} diagram already in the house visual system.`,
+    "Apply ONLY the changes described in the REQUEST below. Preserve",
+    "every other element exactly — layout, sizing, colours, typography,",
+    "arrow routing, group boundaries, viewBox, background. Do NOT redraw",
+    'from scratch. Do NOT "improve" anything the user did not mention.',
+    "Do NOT introduce new design principles, legends, or colours that",
+    "the existing SVG does not already use.",
+    "",
+    "If a requested change appears to conflict with something already in",
+    "the SVG, honour the REQUEST — the user knows what they asked for.",
+    "If a change is ambiguous, make the smallest edit that satisfies it.",
+    "",
+    "【REQUEST — apply only these changes】",
+    userPrompt,
+    mustKeep
+  ].join("\n");
+}
+function detectDiagramType(prompt) {
+  const p = prompt.toLowerCase();
+  const match = (keywords) => keywords.some((k) => p.includes(k));
+  if (match(["consort", "prisma", "flowchart", "flow chart", "flow diagram", "swimlane", "decision tree"])) return "flowchart";
+  if (match(["architecture", "system diagram", "microservice", "pipeline", "data flow", "block diagram"])) return "architecture";
+  if (match(["pathway", "signaling", "signalling", "cascade", "mapk", "egfr", "phosphoryl", "receptor", "kinase"])) return "pathway";
+  if (match(["circuit", "resistor", "capacitor", "op-amp", "opamp", "transistor", "voltage", "schematic circuit"])) return "circuit";
+  if (match(["neural network", "transformer", "cnn", "rnn", "lstm", "attention", "encoder-decoder", "graph", "tree", "hierarchy"])) return "network";
+  return "conceptual";
+}
+const GENERATIONS_URL = "https://api.openai.com/v1/images/generations";
+const EDITS_URL = "https://api.openai.com/v1/images/edits";
+const DEFAULT_MODEL$2 = "gpt-image-2";
+const DEFAULT_SIZE = "auto";
+const REQUEST_TIMEOUT_MS$2 = 3e5;
+function extractBytes(response) {
+  const choice = response.data?.[0];
+  if (!choice) throw new Error("OpenAI image response had no choices");
+  if (choice.b64_json) {
+    return Buffer.from(choice.b64_json, "base64");
+  }
+  if (choice.url) {
+    throw new Error(
+      "OpenAI returned an image URL instead of base64 bytes. The selected model may be dall-e-*; this backend targets gpt-image-2 which returns b64_json by default."
+    );
+  }
+  throw new Error("OpenAI image response did not contain image data");
+}
+async function postJson(url, apiKey, body) {
+  const ctl = new AbortController();
+  const timer = setTimeout(() => ctl.abort(), REQUEST_TIMEOUT_MS$2);
+  try {
+    const res = await fetch(url, {
+      method: "POST",
+      headers: {
+        "Authorization": `Bearer ${apiKey}`,
+        "Content-Type": "application/json"
+      },
+      body: JSON.stringify(body),
+      signal: ctl.signal
+    });
+    const json = await res.json();
+    if (!res.ok) {
+      const msg = json.error?.message || `HTTP ${res.status}`;
+      throw new Error(`OpenAI image API error: ${msg}`);
+    }
+    return json;
+  } finally {
+    clearTimeout(timer);
+  }
+}
+async function postMultipart(url, apiKey, fields, files) {
+  const form = new FormData();
+  for (const [k, v] of Object.entries(fields)) {
+    form.append(k, v);
+  }
+  for (const [k, f] of Object.entries(files)) {
+    const view = new Uint8Array(f.data.buffer, f.data.byteOffset, f.data.byteLength);
+    form.append(k, new Blob([view], { type: f.contentType }), f.filename);
+  }
+  const ctl = new AbortController();
+  const timer = setTimeout(() => ctl.abort(), REQUEST_TIMEOUT_MS$2);
+  try {
+    const res = await fetch(url, {
+      method: "POST",
+      headers: { "Authorization": `Bearer ${apiKey}` },
+      body: form,
+      signal: ctl.signal
+    });
+    const json = await res.json();
+    if (!res.ok) {
+      const msg = json.error?.message || `HTTP ${res.status}`;
+      throw new Error(`OpenAI image edit API error: ${msg}`);
+    }
+    return json;
+  } finally {
+    clearTimeout(timer);
+  }
+}
+function createOpenAIImageProvider(opts = {}) {
+  const apiKey = opts.apiKey ?? process.env.OPENAI_API_KEY?.trim();
+  if (!apiKey) {
+    throw new Error(
+      "OPENAI_API_KEY is required for diagram generation. Add it under Settings → API Keys, or set OPENAI_API_KEY in your shell."
+    );
+  }
+  const model = opts.model || DEFAULT_MODEL$2;
+  const size = opts.size || DEFAULT_SIZE;
+  const defaultQuality = opts.quality || "auto";
+  const capabilities = /* @__PURE__ */ new Set(["text_to_image", "image_to_image"]);
+  const effectiveQuality = (callOpts) => callOpts?.quality ?? defaultQuality;
+  return {
+    id: `openai:${model}`,
+    label: `OpenAI ${model}`,
+    capabilities,
+    async textToImage(prompt, options) {
+      const body = {
+        model,
+        prompt,
+        size,
+        n: 1,
+        quality: effectiveQuality(options)
+      };
+      const response = await postJson(GENERATIONS_URL, apiKey, body);
+      return extractBytes(response);
+    },
+    async imageToImage(prompt, image, options) {
+      const response = await postMultipart(
+        EDITS_URL,
+        apiKey,
+        { model, prompt, size, n: "1", quality: effectiveQuality(options) },
+        { image: { data: image, filename: "image.png", contentType: "image/png" } }
+      );
+      return extractBytes(response);
+    }
+  };
+}
+const CHAT_URL = "https://api.openai.com/v1/chat/completions";
+const DEFAULT_MODEL$1 = "gpt-4o";
+const REQUEST_TIMEOUT_MS$1 = 18e4;
+const OPENAI_THRESHOLDS = {
+  journal: 8.3,
+  conference: 7.8,
+  thesis: 7.8,
+  grant: 7.8,
+  preprint: 7.3,
+  report: 7.3,
+  poster: 6.8,
+  presentation: 6.3,
+  default: 7.3
+};
+const REVIEW_SYSTEM$1 = `You are a rigorous reviewer for scientific publication-grade diagrams.
+Rate on a strict scale — 9+ is reserved for camera-ready figures. Return only valid JSON matching the schema. Be specific: every blocking_issue must describe exactly what is wrong AND a concrete fix another tool can act on.`;
+function buildUserPrompt(req, threshold) {
+  const houseBlock = req.houseProfileSummary ? `HOUSE STYLE (the figure must belong to this visual system):
+${req.houseProfileSummary}
+
+` : "";
+  const fifthDimension = req.houseProfileSummary ? "5. House-style adherence & consistency — matches the supplied palette, typography voice, geometry tokens, and motifs; feels like a sibling of other figures in the same system" : "5. Professional appearance — publication-ready polish";
+  return `Evaluate this diagram for "${req.docType}" publication (acceptance threshold: ${threshold}/10).
+
+DIAGRAM TYPE: ${req.diagramType}
+ORIGINAL REQUEST: ${req.prompt}
+ITERATION: ${req.iteration}/${req.maxIterations}
+${houseBlock}
+Score five dimensions independently (0-2 each, total 0-10):
+  1. Scientific accuracy — concepts, relationships, notation correct
+  2. Clarity & readability — hierarchy, unambiguous at a glance
+  3. Label quality — complete, legible, consistent
+  4. Layout & composition — balanced, no overlap, logical flow
+  ${fifthDimension}
+
+For every issue that blocks acceptance, emit an entry in blocking_issues with:
+  - kind: wrong_content | illegible_text | layout_collision | missing_element | style_mismatch
+  - description: what is wrong (call out house-style deviations under style_mismatch — wrong palette role, wrong corner radius, wrong typography voice, broken motif, etc.)
+  - fix: precise instruction to correct it (will be fed to an image editor)
+
+Choose verdict:
+  - "acceptable" if score >= ${threshold} and no blocking_issues
+  - "needs_edit" if problems are localised/cosmetic (labels, overlaps, styling) — image-to-image editing can fix them
+  - "needs_regen" if content is wrong or structure is broken — the image must be redrawn`;
+}
+function coerceVerdict$2(raw) {
+  if (raw === "acceptable" || raw === "needs_edit" || raw === "needs_regen") return raw;
+  return "needs_edit";
+}
+function clampScore$2(n) {
+  const v = typeof n === "number" ? n : Number(n);
+  if (Number.isNaN(v)) return 0;
+  return Math.min(10, Math.max(0, v));
+}
+const RESPONSE_SCHEMA = {
+  type: "object",
+  additionalProperties: false,
+  required: ["score", "requestAlignment", "legibility", "blockingIssues", "summary", "verdict"],
+  properties: {
+    score: { type: "number" },
+    requestAlignment: { type: "number" },
+    legibility: { type: "number" },
+    blockingIssues: {
+      type: "array",
+      items: {
+        type: "object",
+        additionalProperties: false,
+        required: ["kind", "description", "fix"],
+        properties: {
+          kind: {
+            type: "string",
+            enum: ["wrong_content", "illegible_text", "layout_collision", "missing_element", "style_mismatch"]
+          },
+          description: { type: "string" },
+          fix: { type: "string" }
+        }
+      }
+    },
+    summary: { type: "string" },
+    verdict: {
+      type: "string",
+      enum: ["acceptable", "needs_edit", "needs_regen"]
+    }
+  }
+};
+function createOpenAIReviewProvider(opts = {}) {
+  const apiKey = opts.apiKey ?? process.env.OPENAI_API_KEY?.trim();
+  if (!apiKey) {
+    throw new Error("OPENAI_API_KEY is required for OpenAI-based review.");
+  }
+  const model = opts.model || DEFAULT_MODEL$1;
+  const thresholds = opts.thresholds ?? OPENAI_THRESHOLDS;
+  async function review(req) {
+    const threshold = thresholds[req.docType] ?? thresholds.default;
+    const b64 = req.image.toString("base64");
+    const dataUri = `data:image/png;base64,${b64}`;
+    const body = {
+      model,
+      messages: [
+        { role: "system", content: REVIEW_SYSTEM$1 },
+        {
+          role: "user",
+          content: [
+            { type: "text", text: buildUserPrompt(req, threshold) },
+            { type: "image_url", image_url: { url: dataUri } }
+          ]
+        }
+      ],
+      response_format: {
+        type: "json_schema",
+        json_schema: { name: "DiagramReview", strict: true, schema: RESPONSE_SCHEMA }
+      },
+      temperature: 0
+    };
+    const ctl = new AbortController();
+    const timer = setTimeout(() => ctl.abort(), REQUEST_TIMEOUT_MS$1);
+    try {
+      const res = await fetch(CHAT_URL, {
+        method: "POST",
+        headers: {
+          "Authorization": `Bearer ${apiKey}`,
+          "Content-Type": "application/json"
+        },
+        body: JSON.stringify(body),
+        signal: ctl.signal
+      });
+      const json = await res.json();
+      if (!res.ok) {
+        throw new Error(`OpenAI review API error: ${json.error?.message || `HTTP ${res.status}`}`);
+      }
+      const content = json.choices?.[0]?.message?.content ?? "";
+      if (!content) throw new Error("OpenAI review returned empty content");
+      const parsed = JSON.parse(content);
+      const result = {
+        score: clampScore$2(parsed.score),
+        requestAlignment: clampScore$2(parsed.requestAlignment),
+        legibility: clampScore$2(parsed.legibility),
+        blockingIssues: Array.isArray(parsed.blockingIssues) ? parsed.blockingIssues : [],
+        summary: typeof parsed.summary === "string" ? parsed.summary : "",
+        verdict: coerceVerdict$2(parsed.verdict)
+      };
+      return result;
+    } finally {
+      clearTimeout(timer);
+    }
+  }
+  return {
+    id: `openai:${model}`,
+    label: `OpenAI ${model}`,
+    thresholds,
+    review
+  };
+}
+const MESSAGES_URL = "https://api.anthropic.com/v1/messages";
+const DEFAULT_MODEL = "claude-opus-4-5";
+const ANTHROPIC_VERSION = "2023-06-01";
+const REQUEST_TIMEOUT_MS = 18e4;
+const MAX_TOKENS = 2048;
+const CLAUDE_CODE_IDENTITY = "You are Claude Code, Anthropic's official CLI for Claude.";
+const BETA_API_KEY = "fine-grained-tool-streaming-2025-05-14";
+const BETA_OAUTH = `claude-code-20250219,oauth-2025-04-20,${BETA_API_KEY}`;
+const ANTHROPIC_THRESHOLDS = {
+  journal: 8.5,
+  conference: 8,
+  thesis: 8,
+  grant: 8,
+  preprint: 7.5,
+  report: 7.5,
+  poster: 7,
+  presentation: 6.5,
+  default: 7.5
+};
+const REVIEW_SYSTEM = `You are a rigorous reviewer for scientific publication-grade diagrams.
+Rate on a strict scale — 9+ is reserved for camera-ready figures. Always emit your answer by calling the emit_review tool. Every blocking_issue must describe what is wrong AND a concrete fix another tool can act on.`;
+function buildUserText(req, threshold) {
+  const houseBlock = req.houseProfileSummary ? `
+HOUSE STYLE (figure must belong to this visual system):
+${req.houseProfileSummary}
+` : "";
+  const fifthDimension = req.houseProfileSummary ? "house-style adherence & consistency" : "professional appearance";
+  return `Evaluate this diagram for "${req.docType}" publication (acceptance threshold: ${threshold}/10).
+
+DIAGRAM TYPE: ${req.diagramType}
+ORIGINAL REQUEST: ${req.prompt}
+ITERATION: ${req.iteration}/${req.maxIterations}
+${houseBlock}
+Score five dimensions (0-2 each, total 0-10): scientific accuracy, clarity & readability, label quality, layout & composition, ${fifthDimension}.
+
+When scoring dimension 5, compare against the HOUSE STYLE block above — wrong palette roles, wrong corner radius, wrong typography voice, or broken motifs are all style_mismatch blocking issues.
+
+Choose verdict:
+  - "acceptable"  if score >= ${threshold} and no blocking_issues
+  - "needs_edit"  if problems are localised (labels, overlaps, styling) — image-to-image can fix them
+  - "needs_regen" if content is wrong or structure is broken — must redraw
+
+Call emit_review once with your structured verdict.`;
+}
+const EMIT_REVIEW_TOOL = {
+  name: "emit_review",
+  description: "Emit the structured review verdict for the diagram.",
+  input_schema: {
+    type: "object",
+    additionalProperties: false,
+    required: ["score", "requestAlignment", "legibility", "blockingIssues", "summary", "verdict"],
+    properties: {
+      score: { type: "number" },
+      requestAlignment: { type: "number" },
+      legibility: { type: "number" },
+      blockingIssues: {
+        type: "array",
+        items: {
+          type: "object",
+          additionalProperties: false,
+          required: ["kind", "description", "fix"],
+          properties: {
+            kind: {
+              type: "string",
+              enum: ["wrong_content", "illegible_text", "layout_collision", "missing_element", "style_mismatch"]
+            },
+            description: { type: "string" },
+            fix: { type: "string" }
+          }
+        }
+      },
+      summary: { type: "string" },
+      verdict: {
+        type: "string",
+        enum: ["acceptable", "needs_edit", "needs_regen"]
+      }
+    }
+  }
+};
+function coerceVerdict$1(raw) {
+  if (raw === "acceptable" || raw === "needs_edit" || raw === "needs_regen") return raw;
+  return "needs_edit";
+}
+function clampScore$1(n) {
+  const v = typeof n === "number" ? n : Number(n);
+  if (Number.isNaN(v)) return 0;
+  return Math.min(10, Math.max(0, v));
+}
+function isOAuthAccessToken(token) {
+  return token.startsWith("sk-ant-oat");
+}
+function buildHeaders(token, isOAuth) {
+  const common = {
+    "anthropic-version": ANTHROPIC_VERSION,
+    "Content-Type": "application/json"
+  };
+  if (isOAuth) {
+    return {
+      ...common,
+      "Authorization": `Bearer ${token}`,
+      "anthropic-beta": BETA_OAUTH,
+      "anthropic-dangerous-direct-browser-access": "true",
+      "user-agent": "claude-cli/research-copilot",
+      "x-app": "cli"
+    };
+  }
+  return {
+    ...common,
+    "x-api-key": token,
+    "anthropic-beta": BETA_API_KEY
+  };
+}
+function buildRequestBody(req, threshold, isOAuth) {
+  const b64 = req.image.toString("base64");
+  const systemBlocks = [];
+  if (isOAuth) {
+    systemBlocks.push({ type: "text", text: CLAUDE_CODE_IDENTITY });
+  }
+  systemBlocks.push({ type: "text", text: REVIEW_SYSTEM });
+  return {
+    model: DEFAULT_MODEL,
+    max_tokens: MAX_TOKENS,
+    system: systemBlocks,
+    messages: [
+      {
+        role: "user",
+        content: [
+          {
+            type: "image",
+            source: { type: "base64", media_type: "image/png", data: b64 }
+          },
+          { type: "text", text: buildUserText(req, threshold) }
+        ]
+      }
+    ],
+    tools: [EMIT_REVIEW_TOOL],
+    tool_choice: { type: "tool", name: "emit_review" }
+  };
+}
+async function doRequest(token, isOAuth, body) {
+  const ctl = new AbortController();
+  const timer = setTimeout(() => ctl.abort(), REQUEST_TIMEOUT_MS);
+  try {
+    const res = await fetch(MESSAGES_URL, {
+      method: "POST",
+      headers: buildHeaders(token, isOAuth),
+      body: JSON.stringify(body),
+      signal: ctl.signal
+    });
+    const json = await res.json();
+    return { status: res.status, json };
+  } finally {
+    clearTimeout(timer);
+  }
+}
+function createAnthropicReviewProvider(opts = {}) {
+  const envKey = process.env.ANTHROPIC_API_KEY?.trim();
+  const initialTokenRaw = opts.token ?? envKey;
+  if (!initialTokenRaw) {
+    throw new Error("ANTHROPIC_API_KEY or an anthropic-sub OAuth token is required for Claude-based review.");
+  }
+  let currentToken = initialTokenRaw;
+  const isOAuth = opts.isOAuth ?? isOAuthAccessToken(currentToken);
+  const refreshToken = opts.refreshToken;
+  const model = opts.model || DEFAULT_MODEL;
+  const thresholds = opts.thresholds ?? ANTHROPIC_THRESHOLDS;
+  async function review(req) {
+    const threshold = thresholds[req.docType] ?? thresholds.default;
+    const body = buildRequestBody(req, threshold, isOAuth);
+    if (model !== DEFAULT_MODEL) {
+      body.model = model;
+    }
+    let { status, json } = await doRequest(currentToken, isOAuth, body);
+    if (status === 401 && isOAuth && refreshToken) {
+      try {
+        currentToken = await refreshToken();
+        ({ status, json } = await doRequest(currentToken, isOAuth, body));
+      } catch (err) {
+        throw new Error(`Anthropic OAuth token refresh failed: ${err.message}`);
+      }
+    }
+    if (status < 200 || status >= 300) {
+      throw new Error(`Anthropic review API error (HTTP ${status}): ${json.error?.message || "unknown"}`);
+    }
+    const toolUse = json.content?.find(
+      (c) => c.type === "tool_use" && c.name === "emit_review"
+    );
+    if (!toolUse) {
+      throw new Error("Claude review did not emit the expected tool_use block");
+    }
+    const input = toolUse.input;
+    const result = {
+      score: clampScore$1(input.score),
+      requestAlignment: clampScore$1(input.requestAlignment),
+      legibility: clampScore$1(input.legibility),
+      blockingIssues: Array.isArray(input.blockingIssues) ? input.blockingIssues : [],
+      summary: typeof input.summary === "string" ? input.summary : "",
+      verdict: coerceVerdict$1(input.verdict)
+    };
+    return result;
+  }
+  return {
+    id: `anthropic:${model}${isOAuth ? ":oauth" : ":api-key"}`,
+    label: `Anthropic ${model}${isOAuth ? " (subscription)" : ""}`,
+    thresholds,
+    review
+  };
+}
+const ASPECT_VIEWBOX = {
+  auto: "0 0 1200 900",
+  // 4:3 lands moderate info density
+  square: "0 0 900 900",
+  landscape: "0 0 1400 900",
+  portrait: "0 0 900 1400"
+};
+const SVG_SYSTEM_PROMPT = `You are a diagram artist producing publication-quality SVG in the house visual system.
+Return ONLY a single valid SVG document wrapped in \`\`\`svg fences — no prose, no explanation.
+
+Allowed SVG 1.1 elements:
+  svg, defs, marker, g, rect, circle, ellipse, line, path, polyline, polygon, text, tspan, title, desc, clipPath, pattern
+
+Hard requirements:
+- Root element has an explicit viewBox and no width/height attributes (so it scales).
+- No <script>, no <foreignObject>, no external href references, no embedded base64.
+- Font-family: use the house font stack supplied in the REQUEST; fall back to sans-serif when no listed font is available. Minimum font size 10px.
+- Every labelled element gets a <text> child with readable contents (no lorem ipsum, no placeholder text).
+- Arrows use <marker> definitions inside <defs>, referenced from <line>/<path> via marker-end — no emoji arrows, no unicode replacement characters.
+- Use the colour tokens supplied in the REQUEST — do not invent a new palette.
+- No gradients, filters, blur, or glow anywhere. No drop-shadow, bevel, or emboss effects.
+- Do NOT include figure numbers, titles, or captions inside the SVG ("Figure 1: …"). Those are added in-document.
+- Do NOT include <?xml?> declarations; start directly with <svg ...>.`;
+const SVG_REPAIR_SYSTEM_PROMPT = `You are repairing a previous SVG response that failed to parse.
+Return ONLY a single valid SVG document wrapped in \`\`\`svg fences — no prose, no commentary, no markdown headings.
+Hard requirements:
+- Start with <svg ...> and end with </svg>. No <?xml?> declaration.
+- Root element MUST have a viewBox attribute.
+- Every <marker id="X"> referenced via marker-end="url(#X)" must be defined inside <defs>.
+- All tags must be properly nested and closed.
+- No <script>, no <foreignObject>, no external href.`;
+function buildGenerationUser(prompt, aspect) {
+  const viewBox = ASPECT_VIEWBOX[aspect];
+  return `Produce an SVG diagram for the request below.
+
+VIEWBOX HINT: ${viewBox}
+  - Use this as a starting hint. If your content fits compactly in a
+    smaller area, SHRINK either dimension (by up to 30%) to eliminate
+    empty gutters. A figure with visible empty regions at the bottom
+    or side looks unfinished.
+  - Keep the overall aspect (landscape / portrait / square) close to
+    the hint. Do NOT flip orientation.
+  - Do NOT grow either dimension beyond the hint — that pushes the
+    figure past canonical paper-column widths.
+
+REQUEST:
+${prompt}
+
+Emit the SVG now.`;
+}
+function buildEditUser(prompt, previousSvg) {
+  return `Revise the SVG below by applying the changes in REQUEST. Preserve everything that is already correct — do not redraw the whole diagram from scratch.
+
+VIEWBOX: preserve the viewBox attribute from the PREVIOUS SVG exactly. Do NOT change its dimensions, origin, or aspect.
+
+REQUEST:
+${prompt}
+
+PREVIOUS SVG:
+\`\`\`svg
+${previousSvg}
+\`\`\`
+
+Emit the revised SVG now, as a single complete <svg>…</svg> document.`;
+}
+function buildRepairUser$1(originalUserPrompt, brokenRaw, failureReason) {
+  const HEAD = 1200;
+  const TAIL = 400;
+  const head = brokenRaw.slice(0, HEAD);
+  const tail = brokenRaw.length > HEAD + TAIL ? brokenRaw.slice(-TAIL) : "";
+  const excerpt = tail ? `${head}
+
+[... ${brokenRaw.length - HEAD - TAIL} chars elided ...]
+
+${tail}` : head;
+  return `Your previous response could not be parsed as a valid SVG document.
+
+PARSE FAILURE: ${failureReason}
+
+ORIGINAL REQUEST (unchanged):
+${originalUserPrompt}
+
+YOUR BROKEN RESPONSE (head + tail, ${brokenRaw.length} chars total):
+${excerpt}
+
+Emit a corrected SVG now. Output ONLY the SVG inside \`\`\`svg fences — no apology, no explanation.`;
+}
+function extractSvg(raw) {
+  const fenceMatch = raw.match(/```(?:svg|xml|html)?\s*([\s\S]*?)```/i);
+  const candidate = fenceMatch ? fenceMatch[1] : raw;
+  const svgMatch = candidate.match(/<svg\b[\s\S]*?<\/svg>/i);
+  if (!svgMatch) {
+    const hasOpening = /<svg\b/i.test(candidate);
+    const reason = hasOpening ? "Found <svg> opening but no matching </svg> close — response likely truncated." : "No <svg>…</svg> block found in the response.";
+    return { ok: false, reason };
+  }
+  return { ok: true, svg: svgMatch[0].trim() };
+}
+function validateSvg(svg) {
+  const rootMatch = svg.match(/<svg\b([^>]*)>/i);
+  if (!rootMatch) {
+    return { ok: false, reason: "No <svg> root element found after extraction (should be impossible)." };
+  }
+  if (!/\bviewBox\s*=/i.test(rootMatch[1])) {
+    return { ok: false, reason: "Root <svg> element is missing a viewBox attribute." };
+  }
+  const refIds = /* @__PURE__ */ new Set();
+  const refRe = /marker-(?:start|mid|end)\s*=\s*"url\(#([^)"]+)\)"/gi;
+  let m;
+  while ((m = refRe.exec(svg)) !== null) refIds.add(m[1]);
+  if (refIds.size > 0) {
+    const defIds = /* @__PURE__ */ new Set();
+    const defRe = /<marker\b[^>]*\bid\s*=\s*"([^"]+)"/gi;
+    while ((m = defRe.exec(svg)) !== null) defIds.add(m[1]);
+    const missing = [];
+    for (const id of refIds) if (!defIds.has(id)) missing.push(id);
+    if (missing.length > 0) {
+      return {
+        ok: false,
+        reason: `Marker reference(s) without matching <marker id="…">: ${missing.join(", ")}.`
+      };
+    }
+  }
+  if (!/<text\b/i.test(svg)) {
+    return { ok: false, reason: "No <text> elements present — diagram has no labels." };
+  }
+  const tagRe = /<\s*(\/?)([a-zA-Z][a-zA-Z0-9-]*)\b[^>]*?(\/?)>/g;
+  const stack = [];
+  while ((m = tagRe.exec(svg)) !== null) {
+    const isClose = m[1] === "/";
+    const isSelfClosing = m[3] === "/";
+    const name = m[2].toLowerCase();
+    if (isClose) {
+      const top = stack.pop();
+      if (top !== name) {
+        return {
+          ok: false,
+          reason: `Tag imbalance: </${name}> at offset ${m.index} does not match the open stack (top=${top ?? "empty"}).`
+        };
+      }
+    } else if (!isSelfClosing) {
+      stack.push(name);
+    }
+  }
+  if (stack.length > 0) {
+    return { ok: false, reason: `Unclosed tag(s) at end of document: ${stack.join(" > ")}.` };
+  }
+  return { ok: true, svg };
+}
+function summariseRawForError(raw) {
+  const HEAD = 600;
+  const TAIL = 200;
+  const head = raw.slice(0, HEAD).replace(/\s+/g, " ").trim();
+  if (raw.length <= HEAD + TAIL) return `${raw.length} chars: ${head}`;
+  const tail = raw.slice(-TAIL).replace(/\s+/g, " ").trim();
+  return `${raw.length} chars; head: ${head} … tail: ${tail}`;
+}
+function tryParseAndValidate(raw) {
+  const extracted = extractSvg(raw);
+  if (!extracted.ok) return extracted;
+  return validateSvg(extracted.svg);
+}
+async function generateWithRepair(callLlm, systemPrompt, userPrompt) {
+  const firstRaw = await callLlm(systemPrompt, userPrompt);
+  const first = tryParseAndValidate(firstRaw);
+  if (first.ok) return first.svg;
+  const repairUser = buildRepairUser$1(userPrompt, firstRaw, first.reason);
+  let secondRaw;
+  try {
+    secondRaw = await callLlm(SVG_REPAIR_SYSTEM_PROMPT, repairUser);
+  } catch (err) {
+    throw new Error(
+      `SVG fallback: parse failed and repair call errored. First failure: ${first.reason}. First response: ${summariseRawForError(firstRaw)}. Repair error: ${err.message}`
+    );
+  }
+  const second = tryParseAndValidate(secondRaw);
+  if (second.ok) return second.svg;
+  throw new Error(
+    `SVG fallback: parse failed twice (initial + repair). First failure: ${first.reason}. Initial response: ${summariseRawForError(firstRaw)}. Repair failure: ${second.reason}. Repair response: ${summariseRawForError(secondRaw)}`
+  );
+}
+function createSvgFallbackImageProvider(opts) {
+  const { callLlm } = opts;
+  const aspect = opts.aspect ?? "auto";
+  const modelLabel = opts.modelLabel || "chat-model";
+  const capabilities = /* @__PURE__ */ new Set(["text_to_image", "image_to_image"]);
+  return {
+    id: `svg-fallback:${modelLabel}`,
+    label: `SVG fallback (${modelLabel})`,
+    capabilities,
+    async textToImage(prompt) {
+      const userPrompt = buildGenerationUser(prompt, aspect);
+      const svg = await generateWithRepair(callLlm, SVG_SYSTEM_PROMPT, userPrompt);
+      return Buffer.from(svg, "utf-8");
+    },
+    async imageToImage(prompt, image) {
+      const previousSvg = image.toString("utf-8");
+      const userPrompt = buildEditUser(prompt, previousSvg);
+      const svg = await generateWithRepair(callLlm, SVG_SYSTEM_PROMPT, userPrompt);
+      return Buffer.from(svg, "utf-8");
+    }
+  };
+}
+const FALLBACK_THRESHOLDS = {
+  journal: 8.7,
+  conference: 8.2,
+  thesis: 8.2,
+  grant: 8.2,
+  preprint: 7.7,
+  report: 7.7,
+  poster: 7.2,
+  presentation: 6.7,
+  default: 7.7
+};
+const REVIEW_SYSTEM_PROMPT = `You are a rigorous reviewer of SVG scientific diagrams. The SVG source is provided; read it as text to evaluate structure, labels, and layout. Use a strict scale — 9+ is reserved for camera-ready figures.
+
+Always respond with exactly one JSON object wrapped in \`\`\`json fences. No prose before or after. Schema:
+
+{
+  "score": number,             // 0-10 total
+  "requestAlignment": number,  // 0-10, matches user intent?
+  "legibility": number,        // 0-10, labels readable at intended size?
+  "blockingIssues": [
+    {
+      "kind": "wrong_content" | "illegible_text" | "layout_collision" | "missing_element" | "style_mismatch",
+      "description": string,
+      "fix": string             // concrete enough to feed back to the generator
+    }
+  ],
+  "summary": string,
+  "verdict": "acceptable" | "needs_edit" | "needs_regen"
+}
+
+Verdict rules:
+- "acceptable"  if score >= threshold AND blockingIssues is empty
+- "needs_edit"  for localised problems (labels, overlaps, styling) fixable by revising the SVG in place
+- "needs_regen" for structural or content errors — the diagram must be redrawn`;
+function buildUser(req, threshold, svgSource) {
+  const houseBlock = req.houseProfileSummary ? `
+HOUSE STYLE (figure must belong to this visual system):
+${req.houseProfileSummary}
+
+Under blockingIssues of kind style_mismatch, call out any deviation from this house style — wrong palette tokens, wrong stroke widths, wrong corner radii, wrong typography voice, broken motifs.
+` : "";
+  return `Evaluate this diagram for "${req.docType}" publication (acceptance threshold: ${threshold}/10).
+
+DIAGRAM TYPE: ${req.diagramType}
+ORIGINAL REQUEST: ${req.prompt}
+ITERATION: ${req.iteration}/${req.maxIterations}
+${houseBlock}
+SVG SOURCE:
+\`\`\`svg
+${svgSource}
+\`\`\`
+
+Respond with the JSON review object only.`;
+}
+function coerceVerdict(raw) {
+  if (raw === "acceptable" || raw === "needs_edit" || raw === "needs_regen") return raw;
+  return "needs_edit";
+}
+function clampScore(n) {
+  const v = typeof n === "number" ? n : Number(n);
+  if (Number.isNaN(v)) return 0;
+  return Math.min(10, Math.max(0, v));
+}
+function extractJsonObject(text) {
+  const fenceMatch = text.match(/```(?:json)?\s*(\{[\s\S]*?\})\s*```/i);
+  if (fenceMatch) {
+    try {
+      return JSON.parse(fenceMatch[1]);
+    } catch {
+    }
+  }
+  const firstBrace = text.indexOf("{");
+  if (firstBrace === -1) return null;
+  let depth = 0;
+  for (let i = firstBrace; i < text.length; i++) {
+    const ch = text[i];
+    if (ch === "{") depth++;
+    else if (ch === "}") {
+      depth--;
+      if (depth === 0) {
+        try {
+          return JSON.parse(text.slice(firstBrace, i + 1));
+        } catch {
+          return null;
+        }
+      }
+    }
+  }
+  return null;
+}
+function createSvgFallbackReviewProvider(opts) {
+  const { callLlm } = opts;
+  const modelLabel = opts.modelLabel || "chat-model";
+  const thresholds = opts.thresholds ?? FALLBACK_THRESHOLDS;
+  async function review(req) {
+    const threshold = thresholds[req.docType] ?? thresholds.default;
+    const svgSource = req.image.toString("utf-8");
+    const text = await callLlm(REVIEW_SYSTEM_PROMPT, buildUser(req, threshold, svgSource));
+    const parsed = extractJsonObject(text);
+    if (!parsed) {
+      return {
+        score: Math.max(0, threshold - 0.5),
+        requestAlignment: Math.max(0, threshold - 0.5),
+        legibility: Math.max(0, threshold - 0.5),
+        blockingIssues: [{
+          kind: "style_mismatch",
+          description: "Reviewer did not return a parseable structured response.",
+          fix: "Regenerate with clearer composition; ensure all labels are explicit and arrows have markers."
+        }],
+        summary: "Review parse failed — falling back to conservative needs_edit verdict.",
+        verdict: "needs_edit"
+      };
+    }
+    return {
+      score: clampScore(parsed.score),
+      requestAlignment: clampScore(parsed.requestAlignment),
+      legibility: clampScore(parsed.legibility),
+      blockingIssues: Array.isArray(parsed.blockingIssues) ? parsed.blockingIssues : [],
+      summary: typeof parsed.summary === "string" ? parsed.summary : "",
+      verdict: coerceVerdict(parsed.verdict)
+    };
+  }
+  return {
+    id: `svg-fallback:${modelLabel}`,
+    label: `SVG fallback review (${modelLabel})`,
+    thresholds,
+    review
+  };
+}
+function createRasterizeThenReviewProvider(opts) {
+  const { rasterizer, inner, dimensions } = opts;
+  async function review(req) {
+    const png = await rasterizer(req.image, dimensions);
+    return inner.review({ ...req, image: png });
+  }
+  return {
+    id: `rasterize+${inner.id}`,
+    label: `SVG rasterised → ${inner.label}`,
+    thresholds: inner.thresholds,
+    review
+  };
+}
+const TRANSCRIPTION_SYSTEM_PROMPT = `You transcribe a finalized scientific diagram (provided as a PNG image) into clean, editable SVG markup.
+
+The PNG is the ground truth. Your job is to produce SVG that, when rendered, is a faithful structural copy — NOT to redesign, improve, or "polish" anything.
+
+Output rules (strict):
+- Output ONLY a single SVG document wrapped in \`\`\`svg fences. No prose, no apology, no commentary.
+- Start directly with <svg viewBox="…" xmlns="http://www.w3.org/2000/svg">. No <?xml?> declaration. No width/height attributes on root.
+- Use only these elements: svg, defs, marker, g, rect, circle, ellipse, line, path, polyline, polygon, text, tspan, title, desc.
+- No <script>, no <foreignObject>, no external href, no embedded base64 images.
+- No filters, no gradients, no blur, no drop-shadow.
+
+Transcription requirements:
+- For every box you see → emit one <rect> at approximately the same position and size (drift up to ~20px is acceptable).
+- For every label you see → emit one <text> with the label VERBATIM. Never paraphrase, never abbreviate, never translate.
+- For every arrow you see → emit one <line>, <polyline>, or <path> with marker-end referencing a <marker> in <defs>.
+- Sample dominant colors from the PNG and use them directly as fill/stroke values (hex codes).
+- Match the rough rounded-corner radius you observe (sharp = 0, slightly rounded = 6-8, very rounded = 12+).
+- Preserve grouping: dashed-bordered containers in the PNG → <rect> with stroke-dasharray.
+
+Hard prohibitions:
+- Do NOT add labels, captions, legends, watermarks, or any element not visible in the PNG.
+- Do NOT redesign the layout, change the topology, or "fix" perceived issues.
+- Do NOT include figure numbers or titles ("Figure 1: …").
+- Do NOT use placeholder text like "Lorem ipsum" or "[Label]".
+
+Position drift up to ~20px is acceptable. Missing labels, paraphrased text, or invented elements are NOT.`;
+const REPAIR_SYSTEM_PROMPT = `You are repairing a previous SVG transcription that failed to parse.
+Return ONLY a single valid SVG document wrapped in \`\`\`svg fences — no prose.
+Hard requirements:
+- Start with <svg ...> and end with </svg>. No <?xml?> declaration.
+- Root MUST have a viewBox attribute.
+- Every <marker id="X"> referenced via marker-end="url(#X)" must be defined inside <defs>.
+- All tags properly nested and closed.
+- Preserve every <text> element verbatim from the original PNG content — do not drop labels.`;
+function buildTranscriptionUser(req) {
+  const lines = [];
+  lines.push("Transcribe the attached PNG diagram into editable SVG.");
+  lines.push("");
+  if (req.viewBoxHint) {
+    lines.push(`VIEWBOX HINT: ${req.viewBoxHint}`);
+    lines.push("  - Match the PNG aspect ratio within ±10%.");
+    lines.push("  - Do NOT flip orientation.");
+    lines.push("");
+  }
+  if (req.originalPromptHint) {
+    lines.push("ORIGINAL DIAGRAM REQUEST (for disambiguating any unclear labels):");
+    lines.push(req.originalPromptHint);
+    lines.push("");
+  }
+  lines.push("Emit the SVG now. Output ONLY the SVG inside ```svg fences.");
+  return lines.join("\n");
+}
+function buildRepairUser(originalUserPrompt, brokenRaw, failureReason) {
+  const HEAD = 1200;
+  const TAIL = 400;
+  const head = brokenRaw.slice(0, HEAD);
+  const tail = brokenRaw.length > HEAD + TAIL ? brokenRaw.slice(-TAIL) : "";
+  const excerpt = tail ? `${head}
+
+[... ${brokenRaw.length - HEAD - TAIL} chars elided ...]
+
+${tail}` : head;
+  return `Your previous SVG transcription could not be parsed.
+
+PARSE FAILURE: ${failureReason}
+
+ORIGINAL REQUEST (unchanged):
+${originalUserPrompt}
+
+YOUR BROKEN RESPONSE (head + tail, ${brokenRaw.length} chars total):
+${excerpt}
+
+Re-transcribe the attached PNG into a corrected SVG. The PNG is unchanged — produce a faithful structural copy. Output ONLY the SVG inside \`\`\`svg fences.`;
+}
+async function transcribePngToSvg(opts, req) {
+  const image = {
+    base64: req.png.toString("base64"),
+    mimeType: "image/png"
+  };
+  const userPrompt = buildTranscriptionUser(req);
+  const firstRaw = await opts.callLlmVision(
+    TRANSCRIPTION_SYSTEM_PROMPT,
+    userPrompt,
+    [image]
+  );
+  const first = tryParseAndValidate(firstRaw);
+  if (first.ok) {
+    return { svg: first.svg, repaired: false };
+  }
+  const repairUser = buildRepairUser(userPrompt, firstRaw, first.reason);
+  let secondRaw;
+  try {
+    secondRaw = await opts.callLlmVision(
+      REPAIR_SYSTEM_PROMPT,
+      repairUser,
+      [image]
+    );
+  } catch (err) {
+    throw new Error(
+      `PNG-to-SVG transcription: parse failed and repair call errored. First failure: ${first.reason}. First response: ${summariseRawForError(firstRaw)}. Repair error: ${err.message}`
+    );
+  }
+  const second = tryParseAndValidate(secondRaw);
+  if (second.ok) {
+    return { svg: second.svg, repaired: true, firstAttemptFailure: first.reason };
+  }
+  throw new Error(
+    `PNG-to-SVG transcription: parse failed twice (initial + repair). First failure: ${first.reason}. Initial response: ${summariseRawForError(firstRaw)}. Repair failure: ${second.reason}. Repair response: ${summariseRawForError(secondRaw)}`
+  );
+}
+function resolveAuth(auth) {
+  if (auth) {
+    return {
+      openaiKey: auth.openaiKey ?? null,
+      anthropic: auth.anthropic ?? null
+    };
+  }
+  const envOpenAI = process.env.OPENAI_API_KEY?.trim() || null;
+  const envAnthropic = process.env.ANTHROPIC_API_KEY?.trim();
+  return {
+    openaiKey: envOpenAI,
+    anthropic: envAnthropic ? { token: envAnthropic, isOAuth: false } : null
+  };
+}
+const ASPECT_TO_VIEWBOX = {
+  auto: "0 0 1200 900",
+  square: "0 0 900 900",
+  landscape: "0 0 1400 900",
+  portrait: "0 0 900 1400"
+};
+function buildPngToSvgTranscriber(callLlmVision, visionModelLabel, viewBoxHint) {
+  return async (png, originalPromptHint) => {
+    const result = await transcribePngToSvg(
+      { callLlmVision, modelLabel: visionModelLabel },
+      { png, viewBoxHint, originalPromptHint }
+    );
+    return {
+      svg: result.svg,
+      repaired: result.repaired,
+      firstAttemptFailure: result.firstAttemptFailure,
+      visionModelLabel
+    };
+  };
+}
+function pickImageProvider(prefs, auth, fallback) {
+  const choice = prefs.generation ?? "openai";
+  if (prefs.forceSvg) {
+    if (auth.openaiKey) {
+      if (fallback?.callLlmVision && fallback.visionCapable !== false) {
+        const provider = createOpenAIImageProvider({
+          apiKey: auth.openaiKey,
+          model: prefs.imageModel,
+          size: prefs.imageSize,
+          quality: prefs.imageQuality
+        });
+        const transcriber = buildPngToSvgTranscriber(
+          fallback.callLlmVision,
+          fallback.modelLabel || "chat-model",
+          ASPECT_TO_VIEWBOX[prefs.svgAspect ?? "auto"]
+        );
+        return { provider, pngToSvgTranscriber: transcriber, svgPath: "png_anchored" };
+      }
+      throw new Error(
+        "SVG_REQUIRES_VISION_MODEL: Editable SVG output requires a vision-capable chat model (e.g. GPT-4o, Claude Opus, Gemini 2.5) to transcribe the rendered PNG into SVG markup. Either switch to a vision-capable model in Settings, or request a raster output (.png)."
+      );
+    }
+    if (fallback) {
+      const provider = createSvgFallbackImageProvider({
+        callLlm: fallback.callLlm,
+        modelLabel: fallback.modelLabel,
+        aspect: prefs.svgAspect
+      });
+      return { provider, svgPath: "chat_model_only" };
+    }
+    throw new Error(
+      "SVG output requested but neither an OpenAI API key (for the PNG-anchored path) nor a chat model (for the safety-net path) is configured."
+    );
+  }
+  if (choice === "openai") {
+    if (auth.openaiKey) {
+      return {
+        provider: createOpenAIImageProvider({
+          apiKey: auth.openaiKey,
+          model: prefs.imageModel,
+          size: prefs.imageSize,
+          quality: prefs.imageQuality
+        }),
+        svgPath: null
+      };
+    }
+    if (fallback) {
+      return {
+        provider: createSvgFallbackImageProvider({
+          callLlm: fallback.callLlm,
+          modelLabel: fallback.modelLabel,
+          aspect: prefs.svgAspect
+        }),
+        svgPath: "chat_model_only"
+      };
+    }
+    throw new Error(
+      "Diagram generation requires either OPENAI_API_KEY or a callLlm fallback (which the coordinator supplies from the current chat model). Neither is available here."
+    );
+  }
+  throw new Error(`Unknown generation provider: ${choice}`);
+}
+function buildRealReviewProvider(choice, auth, reviewModel) {
+  if (choice === "openai") {
+    if (!auth.openaiKey) return null;
+    return createOpenAIReviewProvider({ apiKey: auth.openaiKey, model: reviewModel });
+  }
+  if (choice === "anthropic") {
+    if (!auth.anthropic) return null;
+    return createAnthropicReviewProvider({
+      token: auth.anthropic.token,
+      isOAuth: auth.anthropic.isOAuth,
+      refreshToken: auth.anthropic.refresh,
+      model: reviewModel
+    });
+  }
+  if (auth.anthropic) {
+    return createAnthropicReviewProvider({
+      token: auth.anthropic.token,
+      isOAuth: auth.anthropic.isOAuth,
+      refreshToken: auth.anthropic.refresh,
+      model: reviewModel
+    });
+  }
+  if (auth.openaiKey) {
+    return createOpenAIReviewProvider({ apiKey: auth.openaiKey, model: reviewModel });
+  }
+  return null;
+}
+function pickReviewProvider(prefs, auth, fallback, usingSvgGen = false) {
+  const choice = prefs.review ?? "auto";
+  if (usingSvgGen) {
+    if (!fallback) {
+      throw new Error("SVG review fallback requires a callLlm fallback.");
+    }
+    if (fallback.rasterizeSvg) {
+      const visionReviewer = buildRealReviewProvider(choice, auth, prefs.reviewModel);
+      if (visionReviewer) {
+        return createRasterizeThenReviewProvider({
+          rasterizer: fallback.rasterizeSvg,
+          inner: visionReviewer
+        });
+      }
+    }
+    return createSvgFallbackReviewProvider({
+      callLlm: fallback.callLlm,
+      modelLabel: fallback.modelLabel
+    });
+  }
+  if (choice === "openai") {
+    if (!auth.openaiKey) {
+      throw new Error("Review provider set to OpenAI but no OpenAI API key is configured.");
+    }
+    return createOpenAIReviewProvider({ apiKey: auth.openaiKey, model: prefs.reviewModel });
+  }
+  if (choice === "anthropic") {
+    if (!auth.anthropic) {
+      throw new Error(
+        "Review provider set to Anthropic but no Claude credentials are available. Either set ANTHROPIC_API_KEY under Settings → API Keys, or sign in via Claude subscription login."
+      );
+    }
+    return createAnthropicReviewProvider({
+      token: auth.anthropic.token,
+      isOAuth: auth.anthropic.isOAuth,
+      refreshToken: auth.anthropic.refresh,
+      model: prefs.reviewModel
+    });
+  }
+  if (auth.anthropic) {
+    return createAnthropicReviewProvider({
+      token: auth.anthropic.token,
+      isOAuth: auth.anthropic.isOAuth,
+      refreshToken: auth.anthropic.refresh,
+      model: prefs.reviewModel
+    });
+  }
+  if (auth.openaiKey) {
+    return createOpenAIReviewProvider({ apiKey: auth.openaiKey, model: prefs.reviewModel });
+  }
+  throw new Error("No review provider is configured. Add OPENAI_API_KEY or sign in to Claude.");
+}
+function resolveProviders(prefs = {}, auth, fallback) {
+  const resolved = resolveAuth(auth);
+  const picked = pickImageProvider(prefs, resolved, fallback);
+  const usingSvgGen = picked.svgPath === "chat_model_only";
+  const review = pickReviewProvider(prefs, resolved, fallback, usingSvgGen);
+  return {
+    image: picked.provider,
+    review,
+    svgFallback: usingSvgGen,
+    pngToSvgTranscriber: picked.pngToSvgTranscriber,
+    svgPath: picked.svgPath ?? null
+  };
+}
+const KIND_BUCKET = {
+  layout_collision: "visual",
+  style_mismatch: "visual",
+  illegible_text: "visual",
+  missing_element: "content",
+  wrong_content: "content"
+};
+const PREFIX_LEN = 20;
+const MIN_SHARED_TOKENS = 5;
+const MIN_JACCARD = 0.2;
+function normalise(s) {
+  return s.toLowerCase().replace(/\s+/g, " ").trim();
+}
+function panelNumbers(s) {
+  const out = /* @__PURE__ */ new Set();
+  for (const m of normalise(s).matchAll(/panel\s+(\d+)/g)) out.add(m[1]);
+  return out;
+}
+function distinctiveTokens(s) {
+  return new Set(
+    normalise(s).replace(/[^a-z0-9\s]/g, " ").split(/\s+/).filter((w) => w.length >= 4)
+  );
+}
+function issuesMatch(a, b) {
+  if (KIND_BUCKET[a.kind] !== KIND_BUCKET[b.kind]) return false;
+  const panelsA = panelNumbers(a.description);
+  const panelsB = panelNumbers(b.description);
+  if (panelsA.size > 0 && panelsB.size > 0) {
+    let hasShared = false;
+    for (const p of panelsA) {
+      if (panelsB.has(p)) {
+        hasShared = true;
+        break;
+      }
+    }
+    if (!hasShared) return false;
+  }
+  const aDesc = normalise(a.description);
+  const bDesc = normalise(b.description);
+  if (!aDesc || !bDesc) return false;
+  const prefixLen = Math.min(PREFIX_LEN, aDesc.length, bDesc.length);
+  if (prefixLen >= 10 && aDesc.slice(0, prefixLen) === bDesc.slice(0, prefixLen)) {
+    return true;
+  }
+  const aTokens = distinctiveTokens(a.description);
+  const bTokens = distinctiveTokens(b.description);
+  let shared = 0;
+  for (const t of aTokens) if (bTokens.has(t)) shared++;
+  if (shared < MIN_SHARED_TOKENS) return false;
+  const union = aTokens.size + bTokens.size - shared;
+  if (union === 0) return false;
+  return shared / union >= MIN_JACCARD;
+}
+function findMatch(target, pool) {
+  return pool.find((candidate) => issuesMatch(target, candidate));
+}
+function fixedBetween(previous, current) {
+  return previous.filter((p) => !findMatch(p, current));
+}
+function regressionsAgainst(fixedHistory, current) {
+  return current.filter((c) => findMatch(c, fixedHistory));
+}
+const VALID_DOC_TYPES = [
+  "journal",
+  "conference",
+  "thesis",
+  "grant",
+  "preprint",
+  "report",
+  "poster",
+  "presentation",
+  "default"
+];
+const VALID_DIAGRAM_TYPES = [
+  "flowchart",
+  "architecture",
+  "pathway",
+  "circuit",
+  "network",
+  "conceptual",
+  "auto"
+];
+const VALID_REFERENCE_MODES = [
+  "revise_layout",
+  "style_only",
+  "local_edit"
+];
+const SUPPORTED_REFERENCE_MODES = ["revise_layout", "style_only"];
+const VALID_ASPECTS = ["auto", "square", "landscape", "portrait"];
+function aspectToSize(aspect) {
+  switch (aspect) {
+    case "square":
+      return "1024x1024";
+    case "landscape":
+      return "1536x1024";
+    case "portrait":
+      return "1024x1536";
+    case "auto":
+      return "auto";
+  }
+}
+function sanitizeAspect(value) {
+  const v = typeof value === "string" ? value.trim().toLowerCase() : "auto";
+  return VALID_ASPECTS.includes(v) ? v : "auto";
+}
+const VALID_FORMATS = ["auto", "png", "svg"];
+function sanitizeFormat(value) {
+  const v = typeof value === "string" ? value.trim().toLowerCase() : "auto";
+  return VALID_FORMATS.includes(v) ? v : "auto";
+}
+function resolveEffectiveFormat(format, extension) {
+  if (format === "svg") return "svg";
+  if (format === "png") return "png";
+  return extension.toLowerCase() === ".svg" ? "svg" : "png";
+}
+const VALID_QUALITIES = ["low", "medium", "high", "auto"];
+const QUALITY_ORDER = ["low", "medium", "high"];
+function defaultQualityForDocType(docType) {
+  switch (docType) {
+    case "journal":
+    case "conference":
+    case "thesis":
+    case "grant":
+      return "high";
+    case "preprint":
+    case "report":
+    case "poster":
+      return "medium";
+    case "presentation":
+      return "low";
+    default:
+      return "medium";
+  }
+}
+function sanitizeQuality(value) {
+  if (value === void 0 || value === null) return void 0;
+  const v = typeof value === "string" ? value.trim().toLowerCase() : "";
+  return VALID_QUALITIES.includes(v) ? v : void 0;
+}
+function bumpedQuality(current) {
+  const anchor = current === "auto" ? "medium" : current;
+  const idx = QUALITY_ORDER.indexOf(anchor);
+  if (idx < 0 || idx >= QUALITY_ORDER.length - 1) return current;
+  return QUALITY_ORDER[idx + 1];
+}
+const GenerateDiagramSchema = Type.Object({
+  prompt: Type.String({
+    description: "Natural-language description of the diagram to generate. Be specific about components, labels, layout, and quantities."
+  }),
+  output: Type.String({
+    description: 'Workspace-relative output path (e.g. "figures/consort.png"). Parent directories are created if missing.'
+  }),
+  doc_type: Type.Optional(Type.String({
+    description: `Target publication venue. One of: ${VALID_DOC_TYPES.join(" | ")}. Controls quality threshold.`
+  })),
+  diagram_type: Type.Optional(Type.String({
+    description: `Diagram category. One of: ${VALID_DIAGRAM_TYPES.join(" | ")}. Defaults to auto (keyword-detected).`
+  })),
+  iterations: Type.Optional(Type.Number({
+    description: "Maximum refinement iterations (1-3). Each costs one generation + one review. Default: 2."
+  })),
+  aspect: Type.Optional(Type.String({
+    description: 'Output aspect ratio. One of: auto | square | landscape | portrait. Default: auto (model picks from the prompt). Use "landscape" for wide architecture / flow diagrams with >3 horizontal panels, "portrait" for top-to-bottom CONSORT / PRISMA flows, "square" for single-concept schematics.'
+  })),
+  format: Type.Optional(Type.String({
+    description: 'Desired output format: auto | png | svg. Default: auto (inferred from the output extension). Pass "svg" when the user says "SVG / 矢量图 / vector / 向量图", pass "png" when they say "PNG / 图片 / raster". Explicit format beats the extension: if format="svg" but output="foo.png", the file is renamed to foo.svg and extensionChanged is reported.'
+  })),
+  quality: Type.Optional(Type.String({
+    description: "gpt-image-2 rendering quality: low | medium | high | auto. When omitted, defaults from doc_type — high for journal/conference/thesis/grant, medium for preprint/report/poster, low for presentation. Start at low for cheap exploration; the tool automatically bumps one tier on a needs_edit verdict in later iterations."
+  })),
+  reference_path: Type.Optional(Type.String({
+    description: "Optional workspace-relative path to a reference image the first iteration edits instead of drawing from scratch. Requires reference_mode: revise_layout (other modes are not yet implemented)."
+  })),
+  reference_mode: Type.Optional(Type.String({
+    description: `How to use the reference image. Only "revise_layout" is implemented in this version; "style_only" and "local_edit" are reserved and currently rejected. Default: revise_layout.`
+  }))
+});
+function sanitizeDocType(value) {
+  const v = typeof value === "string" ? value.trim().toLowerCase() : "";
+  return VALID_DOC_TYPES.includes(v) ? v : "default";
+}
+function sanitizeDiagramType(value, prompt) {
+  const v = typeof value === "string" ? value.trim().toLowerCase() : "auto";
+  if (VALID_DIAGRAM_TYPES.includes(v)) {
+    return v === "auto" ? detectDiagramType(prompt) : v;
+  }
+  return detectDiagramType(prompt);
+}
+function sanitizeIterations(value) {
+  const n = typeof value === "number" ? value : Number(value);
+  if (!Number.isFinite(n)) return 2;
+  return Math.max(1, Math.min(3, Math.round(n)));
+}
+function sanitizeReferenceMode(value) {
+  if (value === void 0 || value === null || typeof value === "string" && !value.trim()) {
+    return { ok: true, mode: "revise_layout" };
+  }
+  const v = typeof value === "string" ? value.trim().toLowerCase() : "";
+  if (!VALID_REFERENCE_MODES.includes(v)) {
+    return {
+      ok: false,
+      value: typeof value === "string" ? value : String(value),
+      reason: `Unknown reference_mode. Must be one of: ${VALID_REFERENCE_MODES.join(" | ")}.`
+    };
+  }
+  if (!SUPPORTED_REFERENCE_MODES.includes(v)) {
+    return {
+      ok: false,
+      value: v,
+      reason: `reference_mode "${v}" is reserved but not yet implemented. Only ${SUPPORTED_REFERENCE_MODES.join(", ")} is available in this version.`
+    };
+  }
+  return { ok: true, mode: v };
+}
+const CRITICAL_ISSUE_KINDS = [
+  "wrong_content",
+  "missing_element",
+  "illegible_text"
+];
+function hasCriticalIssue(issues) {
+  return issues.some((i) => CRITICAL_ISSUE_KINDS.includes(i.kind));
+}
+function reconcileVerdict(review, threshold) {
+  if (review.verdict === "acceptable") {
+    if (review.score < threshold) {
+      return {
+        final: { ...review, verdict: "needs_edit" },
+        override: `reviewer said acceptable but score ${review.score} < threshold ${threshold}`
+      };
+    }
+    if (review.blockingIssues.length > 0) {
+      const hasStructural = hasCriticalIssue(review.blockingIssues);
+      return {
+        final: { ...review, verdict: hasStructural ? "needs_regen" : "needs_edit" },
+        override: `reviewer said acceptable but listed ${review.blockingIssues.length} blocking issues`
+      };
+    }
+    return { final: review };
+  }
+  if (review.verdict === "needs_edit" && review.score >= threshold + 1 && !hasCriticalIssue(review.blockingIssues)) {
+    return {
+      final: { ...review, verdict: "acceptable" },
+      override: `score ${review.score} ≥ threshold ${threshold} + 1.0 with no critical issues — accepting despite ${review.blockingIssues.length} minor note(s)`
+    };
+  }
+  return { final: review };
+}
+function ensureInsideWorkspace(workspace, target) {
+  const abs = path$1.resolve(workspace, target);
+  const rel = path$1.relative(workspace, abs);
+  if (rel.startsWith("..") || path$1.isAbsolute(rel)) {
+    throw new Error(`Output path must live inside the workspace: ${target}`);
+  }
+  return abs;
+}
+function createGenerateDiagramTool(ctx) {
+  return {
+    name: "generate_diagram",
+    label: "Generate Diagram",
+    description: "Generate a publication-quality scientific diagram from a natural-language description. The tool runs a verdict-driven generate → review → (optional) edit loop using the configured image provider (OpenAI) and review provider (OpenAI or Anthropic). Supports flowcharts, architecture, pathways, circuits, networks, and conceptual frameworks. Prompt guidance: be specific about components, labels, exact counts/values, and layout direction. Decomposition: if the diagram has more than ~6 top-level components or crosses more than 2 logical regions/phases (e.g. offline+online, frontend+backend+storage, multi-stage pipelines with nested loops), prefer producing 2–3 linked diagrams (one per region/phase, plus an optional overview) over a single dense figure — gpt-image-2 reliability drops sharply on crowded geometry, and reviewer scores plateau around 7/10 for monolithic architectures.",
+    parameters: GenerateDiagramSchema,
     execute: async (_toolCallId, rawParams) => {
       const params = rawParams;
-      const filePath = typeof params.file_path === "string" ? params.file_path.trim() : "";
-      const instructions = typeof params.instructions === "string" ? params.instructions.trim() : "";
-      const taskType = typeof params.task_type === "string" ? params.task_type.trim().toLowerCase() : "analyze";
-      if (!filePath) {
-        return toAgentResult("data_analyze", toolError("MISSING_PARAMETER", "Missing file_path.", {
-          suggestions: ["Provide a relative path to the data file (CSV, JSON, TSV, or XLSX)."]
-        }));
-      }
-      if (!instructions) {
-        return toAgentResult("data_analyze", toolError("MISSING_PARAMETER", "Missing instructions.", {
-          suggestions: ["Describe what analysis to perform on the data."]
+      const userPrompt = typeof params.prompt === "string" ? params.prompt.trim() : "";
+      const outputArg = typeof params.output === "string" ? params.output.trim() : "";
+      if (!userPrompt) {
+        return toAgentResult("generate_diagram", toolError("MISSING_PARAMETER", "Missing prompt.", {
+          suggestions: ["Describe the diagram: type, components, labels, layout."]
         }));
       }
-      if (!["analyze", "visualize", "transform", "model"].includes(taskType)) {
-        return toAgentResult("data_analyze", toolError("INVALID_PARAMETER", `Invalid task_type: ${taskType}. Use: analyze | visualize | transform | model.`, {
-          suggestions: ["Valid task types: analyze, visualize, transform, model."]
+      if (!outputArg) {
+        return toAgentResult("generate_diagram", toolError("MISSING_PARAMETER", "Missing output.", {
+          suggestions: ["Provide a workspace-relative output path, e.g. figures/diagram.png."]
         }));
       }
-      const absDataFile = path$1.resolve(ctx.workspacePath, filePath);
-      if (!fs.existsSync(absDataFile)) {
-        return toAgentResult("data_analyze", toolError("FILE_NOT_FOUND", `File not found: ${filePath}`, {
-          suggestions: [
-            "Verify the file path is relative to the workspace root.",
-            "Use the find or glob tool to locate the data file."
-          ],
-          context: { workspacePath: ctx.workspacePath, resolvedPath: absDataFile }
-        }));
+      const docType = sanitizeDocType(params.doc_type);
+      const diagramType = sanitizeDiagramType(params.diagram_type, userPrompt);
+      const iterations = sanitizeIterations(params.iterations);
+      const aspect = sanitizeAspect(params.aspect);
+      const explicitQuality = sanitizeQuality(params.quality);
+      const defaultQuality = defaultQualityForDocType(docType);
+      const initialQuality = explicitQuality ?? defaultQuality;
+      const hasReference = typeof params.reference_path === "string" && !!params.reference_path.trim();
+      let referenceMode = "revise_layout";
+      if (hasReference || params.reference_mode !== void 0) {
+        const parsed = sanitizeReferenceMode(params.reference_mode);
+        if (!parsed.ok) {
+          return toAgentResult("generate_diagram", toolError("INVALID_PARAMETER", parsed.reason, {
+            suggestions: [
+              `Use reference_mode: ${SUPPORTED_REFERENCE_MODES.join(" or ")}.`,
+              "Omit reference_path to draw without a reference image."
+            ],
+            context: { provided: parsed.value }
+          }));
+        }
+        referenceMode = parsed.mode;
       }
-      const runId = Date.now().toString(36);
-      const outputBase = path$1.join(ctx.workspacePath, ".research-pilot", "data-runs", runId);
-      const figuresDir = path$1.join(outputBase, "figures");
-      const tablesDir = path$1.join(outputBase, "tables");
-      const dataDir = path$1.join(outputBase, "data");
-      fs.mkdirSync(figuresDir, { recursive: true });
-      fs.mkdirSync(tablesDir, { recursive: true });
-      fs.mkdirSync(dataDir, { recursive: true });
-      const resultsFile = path$1.join(outputBase, "results.json");
-      const rawPreview = fs.readFileSync(absDataFile, "utf-8").slice(0, 2e3);
-      const ext = path$1.extname(absDataFile).toLowerCase();
-      const formatHint = ext === ".csv" ? "CSV" : ext === ".tsv" ? "TSV" : ext === ".json" ? "JSON" : ext === ".xlsx" ? "XLSX" : "unknown";
-      if (!ctx.callLlm) {
-        return toAgentResult("data_analyze", toolError("LLM_UNAVAILABLE", "LLM not available for code generation.", {
-          suggestions: ["Ensure the agent runtime has an LLM provider configured (callLlm in ResearchToolContext)."]
+      let absOutput;
+      try {
+        absOutput = ensureInsideWorkspace(ctx.workspacePath, outputArg);
+      } catch (err) {
+        return toAgentResult("generate_diagram", toolError("PATH_OUTSIDE_WORKSPACE", err.message, {
+          suggestions: ["Use a path relative to the workspace root."]
         }));
       }
-      const taskSection = DATA_ANALYSIS_TASKS.split(/^## /m).find((s) => s.startsWith(taskType));
-      const taskDesc = taskSection ? `## ${taskSection}` : "";
-      const userPrompt = [
-        `Task type: ${taskType} (${TASK_DESCRIPTIONS[taskType] ?? "Analysis"})`,
-        "",
-        taskDesc,
-        "",
-        `Data file format: ${formatHint}`,
-        `Data file preview (first 2000 chars):`,
-        "```",
-        rawPreview,
-        "```",
-        "",
-        `Instructions: ${instructions}`,
-        "",
-        "IMPORTANT: Use the pre-defined path variables (DATA_FILE, FIGURES_DIR, TABLES_DIR, DATA_DIR, RESULTS_FILE).",
-        "Call write_results() at the end with your outputs list and summary dict.",
-        "Output ONLY the Python code in a ```python code block."
-      ].join("\n");
-      let generatedCode;
+      const outDir = path$1.dirname(absOutput);
+      const baseName = path$1.basename(absOutput, path$1.extname(absOutput));
+      const originalExtension = path$1.extname(absOutput) || ".png";
+      fs.mkdirSync(outDir, { recursive: true });
+      const formatPref = sanitizeFormat(params.format);
+      const effectiveFormat = resolveEffectiveFormat(formatPref, originalExtension);
+      const userRequestedSvg = effectiveFormat === "svg";
+      const liveSettings = ctx.getSettings?.() ?? ctx.settings;
+      const reviewPref = liveSettings?.diagram?.reviewProvider ?? "auto";
+      const prefs = {
+        generation: "openai",
+        review: reviewPref,
+        imageSize: aspectToSize(aspect),
+        imageQuality: initialQuality,
+        svgAspect: aspect,
+        forceSvg: userRequestedSvg
+      };
+      const auth = ctx.getDiagramAuth?.();
+      const fallback = ctx.callLlm ? {
+        callLlm: ctx.callLlm,
+        rasterizeSvg: ctx.rasterizeSvg,
+        // Vision channel powers Path A (PNG-anchored SVG transcription).
+        // When absent, registry chooses Path B (hard error if user has
+        // OpenAI key) or Path C (chat-model-only safety net).
+        callLlmVision: ctx.callLlmVision,
+        visionCapable: ctx.visionCapable
+      } : void 0;
+      let providers;
       try {
-        const llmResponse = await ctx.callLlm(DATA_ANALYSIS_SYSTEM, userPrompt);
-        const codeMatch = llmResponse.match(/```python\n([\s\S]*?)```/) || llmResponse.match(/```\n([\s\S]*?)```/);
-        generatedCode = codeMatch ? codeMatch[1] : llmResponse;
+        providers = resolveProviders(prefs, auth, fallback);
       } catch (err) {
-        return toAgentResult("data_analyze", toolError("EXECUTION_FAILED", `Code generation failed: ${err.message}`, {
-          retryable: true,
-          suggestions: ["Retry — the LLM may produce valid code on a subsequent attempt.", "Try simplifying the instructions."]
+        const msg = err.message;
+        if (msg.startsWith("SVG_REQUIRES_VISION_MODEL:")) {
+          return toAgentResult("generate_diagram", toolError("SVG_REQUIRES_VISION_MODEL", msg.slice("SVG_REQUIRES_VISION_MODEL:".length).trim(), {
+            suggestions: [
+              "Switch the chat model to a vision-capable one (GPT-4o, Claude Opus 4.5, Gemini 2.5) under Settings → Model.",
+              'Or, request a raster output by using a .png extension instead of .svg, or by setting format: "png".'
+            ]
+          }));
+        }
+        return toAgentResult("generate_diagram", toolError("LLM_UNAVAILABLE", msg, {
+          suggestions: [
+            "Ask the user to add OPENAI_API_KEY under Settings → API Keys for native image generation (gpt-image-2). ChatGPT / Codex subscription tokens are scoped to the Codex endpoint and cannot call the Images API.",
+            'Alternatively, leave a "figure TBD" caption with a textual description so the user can regenerate later without re-explaining the intent.'
+          ]
         }));
       }
-      const pathDefinitions = [
-        "",
-        "# Pre-defined paths (set by the tool runtime)",
-        `DATA_FILE = ${JSON.stringify(absDataFile)}`,
-        `FIGURES_DIR = ${JSON.stringify(figuresDir)}`,
-        `TABLES_DIR = ${JSON.stringify(tablesDir)}`,
-        `DATA_DIR = ${JSON.stringify(dataDir)}`,
-        `RESULTS_FILE = ${JSON.stringify(resultsFile)}`,
-        ""
-      ].join("\n");
-      const fullScript = DATA_CODE_TEMPLATE + pathDefinitions + "\n" + generatedCode;
-      const scriptPath = path$1.join(outputBase, "script.py");
-      fs.writeFileSync(scriptPath, fullScript, "utf-8");
-      try {
-        const { stdout, stderr } = await execFileAsync$2("python3", [scriptPath], {
-          cwd: ctx.workspacePath,
-          timeout: ctx.settings?.dataAnalysis?.timeoutMs ?? 12e4,
-          maxBuffer: 10 * 1024 * 1024
-        });
-        let manifest = null;
-        if (fs.existsSync(resultsFile)) {
-          try {
-            manifest = JSON.parse(fs.readFileSync(resultsFile, "utf-8"));
-          } catch {
+      const pathAActive = providers.svgPath === "png_anchored" && !!providers.pngToSvgTranscriber;
+      const providerFormat = providers.svgFallback ? "svg" : "png";
+      const extension = providerFormat === "svg" ? ".svg" : ".png";
+      const finalExtension = pathAActive ? ".svg" : extension;
+      const finalAbsOutput = finalExtension !== originalExtension ? path$1.join(outDir, `${baseName}${finalExtension}`) : absOutput;
+      const promptFormat = providerFormat === "svg" ? "svg" : "raster";
+      let referenceBytes = null;
+      if (hasReference) {
+        try {
+          const refAbs = ensureInsideWorkspace(ctx.workspacePath, params.reference_path.trim());
+          if (!fs.existsSync(refAbs)) {
+            return toAgentResult("generate_diagram", toolError("FILE_NOT_FOUND", `Reference image not found: ${params.reference_path}`, {
+              suggestions: ["Check the reference_path is relative to the workspace root."]
+            }));
           }
+          referenceBytes = fs.readFileSync(refAbs);
+        } catch (err) {
+          return toAgentResult("generate_diagram", toolError("PATH_OUTSIDE_WORKSPACE", err.message));
         }
-        const outputs = [];
-        for (const [dir, type] of [
-          [figuresDir, "figure"],
-          [tablesDir, "table"],
-          [dataDir, "data"]
-        ]) {
-          if (fs.existsSync(dir)) {
-            for (const f of fs.readdirSync(dir)) {
-              outputs.push({
-                name: f,
-                type,
-                path: path$1.relative(ctx.workspacePath, path$1.join(dir, f))
-              });
-            }
+      }
+      const threshold = providers.review.thresholds[docType] ?? providers.review.thresholds.default;
+      const houseProfileSummary = renderProfile(DEFAULT_HOUSE_PROFILE).summaryForReviewer;
+      const history = [];
+      let prevImage = referenceBytes && referenceMode !== "style_only" ? referenceBytes : null;
+      let stoppedEarly = false;
+      let stoppedReason;
+      const fixedSoFar = [];
+      let currentQuality = initialQuality;
+      for (let i = 1; i <= iterations; i++) {
+        const canEdit = !!providers.image.imageToImage && providers.image.capabilities.has("image_to_image");
+        const lastReview = history[history.length - 1]?.review;
+        let image;
+        let usedEdit = false;
+        let promptForThisIter;
+        if (i === 1) {
+          if (referenceBytes && referenceMode === "style_only" && canEdit) {
+            promptForThisIter = composeStyleOnlyPrompt(userPrompt, diagramType, promptFormat);
+            image = await providers.image.imageToImage(
+              promptForThisIter,
+              referenceBytes,
+              { quality: currentQuality }
+            );
+            usedEdit = true;
+          } else if (referenceBytes && referenceMode === "revise_layout" && canEdit) {
+            promptForThisIter = composeSurgicalRevisionPrompt(userPrompt, diagramType);
+            image = await providers.image.imageToImage(
+              promptForThisIter,
+              referenceBytes,
+              { quality: currentQuality }
+            );
+            usedEdit = true;
+          } else {
+            promptForThisIter = composeGenerationPrompt(userPrompt, diagramType, promptFormat);
+            image = await providers.image.textToImage(promptForThisIter, { quality: currentQuality });
           }
+        } else if (lastReview?.verdict === "needs_edit" && prevImage && canEdit) {
+          currentQuality = bumpedQuality(currentQuality);
+          promptForThisIter = composeEditPrompt(
+            userPrompt,
+            diagramType,
+            lastReview.blockingIssues,
+            fixedSoFar
+          );
+          image = await providers.image.imageToImage(promptForThisIter, prevImage, { quality: currentQuality });
+          usedEdit = true;
+        } else {
+          promptForThisIter = composeRegenPrompt(userPrompt, diagramType, lastReview?.blockingIssues ?? [], promptFormat);
+          image = await providers.image.textToImage(promptForThisIter, { quality: currentQuality });
         }
-        const payload = {
-          stdout: stdout.slice(0, 4e3),
-          stderr: stderr ? stderr.slice(0, 1e3) : void 0,
-          outputs,
-          manifest: manifest ? {
-            summary: manifest.summary,
-            warnings: manifest.warnings
-          } : void 0,
-          scriptPath: path$1.relative(ctx.workspacePath, scriptPath),
-          runId
-        };
-        return toAgentResult("data_analyze", { success: true, data: payload });
-      } catch (err) {
-        const errorDetail = err.stderr?.slice(0, 2e3) || err.message;
-        return toAgentResult("data_analyze", toolError("EXECUTION_FAILED", `Python execution failed: ${errorDetail}`, {
-          retryable: true,
-          suggestions: [
-            `Review the generated script at ${path$1.relative(ctx.workspacePath, scriptPath)} for errors.`,
-            "Check that required Python packages (pandas, matplotlib, seaborn, etc.) are installed.",
-            "Try simplifying the analysis instructions."
-          ],
-          context: {
-            scriptPath: path$1.relative(ctx.workspacePath, scriptPath),
-            runId
-          },
-          data: {
-            scriptPath: path$1.relative(ctx.workspacePath, scriptPath),
-            runId
-          }
+        const iterPath = path$1.join(outDir, `${baseName}_v${i}${extension}`);
+        fs.writeFileSync(iterPath, image);
+        let rawReview;
+        try {
+          rawReview = await providers.review.review({
+            image,
+            prompt: userPrompt,
+            docType,
+            diagramType,
+            iteration: i,
+            maxIterations: iterations,
+            houseProfileSummary
+          });
+        } catch (err) {
+          return toAgentResult("generate_diagram", toolError("API_ERROR", `Review failed: ${err.message}`, {
+            retryable: true,
+            suggestions: ["Retry the tool call.", "Switch the review provider in Settings."],
+            context: { imagePath: path$1.relative(ctx.workspacePath, iterPath) },
+            data: { partialPath: path$1.relative(ctx.workspacePath, iterPath) }
+          }));
+        }
+        const reconciled = reconcileVerdict(rawReview, threshold);
+        const review = reconciled.final;
+        const regressed = regressionsAgainst(fixedSoFar, review.blockingIssues);
+        history.push({
+          iteration: i,
+          imagePath: path$1.relative(ctx.workspacePath, iterPath),
+          usedEdit,
+          quality: currentQuality,
+          review,
+          rawVerdict: rawReview.verdict,
+          verdictOverrideReason: reconciled.override,
+          regressedIssues: regressed.length > 0 ? regressed : void 0,
+          promptSnippet: promptForThisIter.slice(0, 400)
+        });
+        prevImage = image;
+        if (lastReview) {
+          const newlyFixed = fixedBetween(lastReview.blockingIssues, review.blockingIssues);
+          for (const f of newlyFixed) fixedSoFar.push(f);
+        }
+        if (review.verdict === "acceptable") {
+          stoppedEarly = i < iterations;
+          stoppedReason = "acceptable";
+          break;
+        }
+        if (regressed.length > 0 && i < iterations) {
+          stoppedEarly = true;
+          stoppedReason = `regression_detected: ${regressed.length} previously-fixed issue(s) returned`;
+          break;
+        }
+      }
+      const last = history[history.length - 1];
+      if (!last) {
+        return toAgentResult("generate_diagram", toolError("EXECUTION_FAILED", "No iterations completed.", {
+          retryable: true
         }));
       }
+      const finalAbsIter = path$1.resolve(ctx.workspacePath, last.imagePath);
+      let transcriptionRecord;
+      if (pathAActive && providers.pngToSvgTranscriber) {
+        const pngBytes = fs.readFileSync(finalAbsIter);
+        let transcribed;
+        try {
+          transcribed = await providers.pngToSvgTranscriber(pngBytes, userPrompt);
+        } catch (err) {
+          return toAgentResult("generate_diagram", toolError("SVG_TRANSCRIPTION_FAILED", err.message, {
+            retryable: true,
+            suggestions: [
+              'Retry the tool call with format: "png" — the PNG anchor was already generated successfully.',
+              "Or, switch to a different vision-capable chat model and retry."
+            ],
+            context: { pngAnchorPath: path$1.relative(ctx.workspacePath, finalAbsIter) }
+          }));
+        }
+        const anchorAbs = path$1.join(outDir, `${baseName}.png`);
+        if (history.length === 1 && finalAbsIter !== anchorAbs) {
+          fs.renameSync(finalAbsIter, anchorAbs);
+          last.imagePath = path$1.relative(ctx.workspacePath, anchorAbs);
+        } else if (finalAbsIter !== anchorAbs) {
+          fs.copyFileSync(finalAbsIter, anchorAbs);
+        }
+        fs.writeFileSync(finalAbsOutput, transcribed.svg, "utf-8");
+        transcriptionRecord = {
+          repaired: transcribed.repaired,
+          firstAttemptFailure: transcribed.firstAttemptFailure,
+          visionModelLabel: transcribed.visionModelLabel,
+          anchorPngPath: path$1.relative(ctx.workspacePath, anchorAbs),
+          svgBytes: Buffer.byteLength(transcribed.svg, "utf-8")
+        };
+      } else if (finalAbsIter !== finalAbsOutput) {
+        if (history.length === 1) {
+          fs.renameSync(finalAbsIter, finalAbsOutput);
+          last.imagePath = path$1.relative(ctx.workspacePath, finalAbsOutput);
+        } else {
+          fs.copyFileSync(finalAbsIter, finalAbsOutput);
+        }
+      }
+      const reviewLogPath = path$1.join(outDir, `${baseName}_review_log.json`);
+      const logPayload = {
+        prompt: userPrompt,
+        docType,
+        diagramType,
+        aspect,
+        format: {
+          requested: formatPref,
+          effectiveFromParams: effectiveFormat,
+          actual: providerFormat,
+          originalExtension,
+          finalExtension
+        },
+        threshold,
+        // mode classifications:
+        //   'image'           — PNG path, no SVG output requested
+        //   'svg_fallback'    — Path C, chat-model SVG synthesis (no OpenAI key)
+        //   'png_anchored_svg' — Path A, PNG-anchored vision transcription
+        mode: providers.svgFallback ? "svg_fallback" : pathAActive ? "png_anchored_svg" : "image",
+        svgPath: providers.svgPath,
+        transcription: transcriptionRecord ?? void 0,
+        quality: {
+          initial: initialQuality,
+          defaultForDocType: defaultQuality,
+          explicit: explicitQuality ?? null
+        },
+        houseProfile: DEFAULT_HOUSE_PROFILE.id,
+        provider: {
+          image: providers.image.id,
+          review: providers.review.id
+        },
+        iterations: history.map((h) => ({
+          iteration: h.iteration,
+          imagePath: h.imagePath,
+          usedEdit: h.usedEdit,
+          quality: h.quality,
+          review: h.review,
+          rawVerdict: h.rawVerdict,
+          verdictOverrideReason: h.verdictOverrideReason,
+          regressedIssues: h.regressedIssues
+        })),
+        stoppedEarly,
+        stoppedReason: stoppedReason ?? (stoppedEarly ? void 0 : "max_iterations"),
+        fixedAcrossIterations: fixedSoFar
+      };
+      fs.writeFileSync(reviewLogPath, JSON.stringify(logPayload, null, 2), "utf-8");
+      const payload = {
+        outputPath: path$1.relative(ctx.workspacePath, finalAbsOutput),
+        absoluteOutputPath: finalAbsOutput,
+        iterations: history,
+        finalScore: last.review.score,
+        finalVerdict: last.review.verdict,
+        threshold,
+        mode: providers.svgFallback ? "svg_fallback" : pathAActive ? "png_anchored_svg" : "image",
+        provider: {
+          image: providers.image.id,
+          review: providers.review.id
+        },
+        reviewLogPath: path$1.relative(ctx.workspacePath, reviewLogPath),
+        stoppedEarly,
+        stoppedReason: stoppedReason ?? (stoppedEarly ? void 0 : "max_iterations"),
+        extensionChanged: originalExtension !== finalExtension ? { requested: originalExtension, actual: finalExtension } : void 0,
+        transcription: transcriptionRecord ? {
+          anchorPngPath: transcriptionRecord.anchorPngPath,
+          repaired: transcriptionRecord.repaired,
+          visionModelLabel: transcriptionRecord.visionModelLabel,
+          svgBytes: transcriptionRecord.svgBytes
+        } : void 0
+      };
+      return toAgentResult("generate_diagram", { success: true, data: payload });
     }
   };
 }
@@ -6815,7 +9246,7 @@ function rebuildIndex() {
   const lines = ["# Paper Wiki Index\n"];
   lines.push("## Papers\n");
   if (existsSync(papersDir)) {
-    const files = readdirSync$1(papersDir).filter((f) => f.endsWith(".md")).sort();
+    const files = readdirSync(papersDir).filter((f) => f.endsWith(".md")).sort();
     for (const file of files) {
       const content = safeReadFile(join(papersDir, file));
       if (!content) continue;
@@ -6827,7 +9258,7 @@ function rebuildIndex() {
   if (lines[lines.length - 1] === "## Papers\n") lines.push("_(empty)_");
   lines.push("\n## Concepts\n");
   if (existsSync(conceptsDir)) {
-    const files = readdirSync$1(conceptsDir).filter((f) => f.endsWith(".md")).sort();
+    const files = readdirSync(conceptsDir).filter((f) => f.endsWith(".md")).sort();
     for (const file of files) {
       const content = safeReadFile(join(conceptsDir, file));
       if (!content) continue;
@@ -6865,19 +9296,19 @@ function readRecentLog(n = 20) {
 function countPaperPages() {
   const dir = join(getWikiRoot(), "papers");
   if (!existsSync(dir)) return 0;
-  return readdirSync$1(dir).filter((f) => f.endsWith(".md")).length;
+  return readdirSync(dir).filter((f) => f.endsWith(".md")).length;
 }
 function countConceptPages() {
   const dir = join(getWikiRoot(), "concepts");
   if (!existsSync(dir)) return 0;
-  return readdirSync$1(dir).filter((f) => f.endsWith(".md")).length;
+  return readdirSync(dir).filter((f) => f.endsWith(".md")).length;
 }
 function listWikiPages() {
   const root = getWikiRoot();
   const readEntries = (subdir, kind) => {
     const dir = join(root, subdir);
     if (!existsSync(dir)) return [];
-    return readdirSync$1(dir).filter((f) => f.endsWith(".md")).map((f) => {
+    return readdirSync(dir).filter((f) => f.endsWith(".md")).map((f) => {
       const slug = f.replace(".md", "");
       const content = safeReadFile(join(dir, f));
       const titleMatch = content?.match(/^#\s+(.+)$/m);
@@ -7399,7 +9830,7 @@ function rebuildMemoryIndex() {
   if (!existsSync(papersDir)) {
     return { numPapers: 0, numTokens: 0, numAliases: 0, numEdges: 0 };
   }
-  const files = readdirSync$1(papersDir).filter((f) => f.endsWith(".md"));
+  const files = readdirSync(papersDir).filter((f) => f.endsWith(".md"));
   const papers = [];
   for (const file of files) {
     const slug = file.replace(/\.md$/, "");
@@ -7874,7 +10305,7 @@ function listAllPaperSlugs() {
   const dir = join(getWikiRoot(), "papers");
   if (!existsSync(dir)) return [];
   try {
-    return readdirSync$1(dir).filter((f) => f.endsWith(".md")).map((f) => f.replace(/\.md$/, ""));
+    return readdirSync(dir).filter((f) => f.endsWith(".md")).map((f) => f.replace(/\.md$/, ""));
   } catch {
     return [];
   }
@@ -8316,6 +10747,7 @@ function createResearchTools(ctx) {
   tools.push(createLiteratureSearchTool(ctx));
   tools.push(createConvertDocumentTool(ctx));
   tools.push(createDataAnalyzeTool(ctx));
+  tools.push(createGenerateDiagramTool(ctx));
   const artifactTools = createResearchMemoryTools({
     sessionId: ctx.sessionId,
     projectPath: ctx.projectPath
@@ -8375,7 +10807,7 @@ function simplifyMessages(messages, maxMessages) {
             const text = block.text;
             content += text.length > 500 ? text.slice(0, 500) + "...[truncated]" : text;
             content += "\n";
-          } else if (block.type === "tool_use" && "name" in block) {
+          } else if (block.type === "toolCall" && "name" in block) {
             content += `[Called ${block.name}]
 `;
           }
@@ -8399,7 +10831,7 @@ function agentCalledSaveMemoryThisTurn(messages) {
     if (msg.role === "user") break;
     if (msg.role === "assistant" && Array.isArray(msg.content)) {
       for (const block of msg.content) {
-        if (block && typeof block === "object" && "type" in block && block.type === "tool_use") {
+        if (block && typeof block === "object" && "type" in block && block.type === "toolCall") {
           if (block.name === "save-memory") return true;
         }
       }
@@ -8424,6 +10856,8 @@ async function maybeExtractMemories(config, messages, turnCount, extractEveryN =
     });
     const result = await completeSimple(config.model, {
       systemPrompt: config.systemPrompt,
+      // pi-ai's Message union requires extra fields on assistant turns, but
+      // for this context-only replay the providers read role/content/timestamp.
       messages: simplified
     }, {
       maxTokens: 1024,
@@ -8797,11 +11231,11 @@ function detectIntentsByRules(message) {
   return intents;
 }
 const MAX_SKILL_PRELOAD = 5;
-async function matchSkillsWithLLM(model, apiKey, message, skills) {
+async function matchSkillsWithLLM(model, apiKey, message, skills, priorTurns = []) {
   if (!model || skills.length === 0) return [];
   const skillList = skills.map((s) => `- ${s.name}: ${s.description}`).join("\n");
   const systemPrompt = [
-    "You are a skill router for a research assistant. Given a user message, select which skills should be activated.",
+    "You are a skill router for a research assistant. Given a user message (and recent conversation context, if any), select which skills should be activated.",
     "Return ONLY a JSON array of skill names. Return [] if none are relevant.",
     "",
     "Rules:",
@@ -8809,14 +11243,22 @@ async function matchSkillsWithLLM(model, apiKey, message, skills) {
     "- Do not select skills speculatively",
     `- Maximum ${MAX_SKILL_PRELOAD} skills`,
     "- Consider both English and Chinese messages",
+    '- If the current message is a short follow-up or confirmation (e.g. "yes", "do that", "go ahead", "好的", "继续"), infer intent from the recent context',
     "",
     "Available skills:",
     skillList
   ].join("\n");
+  const contextBlock = priorTurns.length > 0 ? priorTurns.map((t) => `User: ${t.userMessage}
+Assistant: ${t.response}`).join("\n\n") : "";
+  const userContent = contextBlock ? `Recent conversation:
+${contextBlock}
+
+Current user message:
+${message}` : message;
   try {
     const result = await completeSimple(model, {
       systemPrompt,
-      messages: [{ role: "user", content: message, timestamp: Date.now() }]
+      messages: [{ role: "user", content: userContent, timestamp: Date.now() }]
     }, {
       maxTokens: 100,
       apiKey
@@ -9003,9 +11445,41 @@ async function createCoordinator(config) {
       const textContent = result.content.find((c) => c.type === "text");
       return textContent?.text ?? "";
     },
+    // Vision-capable sibling of callLlm. Mirrors the stateless completeSimple
+    // shape above plus the ImageContent transformation used by chat() at the
+    // user-attached-images path. Reused by the diagram tool's PNG-anchored
+    // SVG transcription path, where a final PNG is fed back to the model to
+    // be re-emitted as editable SVG markup.
+    callLlmVision: async (systemPrompt, userContent, images) => {
+      if (!piModel) throw new Error("No model available for sub-call");
+      if (!piModel.input.includes("image")) {
+        throw new Error(
+          `Selected model "${piModel.id}" does not accept image input. Switch to a vision-capable model (e.g. GPT-4o, Claude Opus, Gemini 2.5).`
+        );
+      }
+      const currentKey = await resolveApiKey();
+      const content = [
+        { type: "text", text: userContent },
+        ...images.map((img) => ({
+          type: "image",
+          data: img.base64,
+          mimeType: img.mimeType
+        }))
+      ];
+      const result = await completeSimple(piModel, {
+        systemPrompt,
+        messages: [{ role: "user", content, timestamp: Date.now() }]
+      }, { maxTokens: 8192, apiKey: currentKey });
+      const textContent = result.content.find((c) => c.type === "text");
+      return textContent?.text ?? "";
+    },
+    visionCapable: !!piModel?.input.includes("image"),
     onToolCall,
     onToolResult: wrappedOnToolResult,
-    settings: config.resolvedSettings
+    settings: config.resolvedSettings,
+    getSettings: config.getResolvedSettings,
+    getDiagramAuth: config.getDiagramAuth,
+    rasterizeSvg: config.rasterizeSvg
   };
   const { tools: researchAgentTools, destroy: destroyResearchTools } = createResearchTools(toolCtx);
   const codingTools = createCodingTools(projectPath);
@@ -9086,6 +11560,7 @@ async function createCoordinator(config) {
           piModel,
           settings.reserveTokens,
           currentKey,
+          void 0,
           signal,
           void 0,
           compactionSummary
@@ -9224,8 +11699,12 @@ ${historyText}`,
     async chat(message, mentions, images) {
       try {
         const intents = detectIntentsByRules(message);
+        const priorTurns = turnHistory.slice(-2).map((t) => ({
+          userMessage: t.userMessage,
+          response: t.response
+        }));
         const currentKey = await resolveApiKey();
-        const matchedSkillNames = await matchSkillsWithLLM(intentRouterModel, currentKey, message, skills);
+        const matchedSkillNames = await matchSkillsWithLLM(intentRouterModel, currentKey, message, skills, priorTurns);
         const matchedSkills = matchedSkillNames.map((name) => skills.find((s) => s.name === name)).filter((s) => s !== void 0);
         for (const s of matchedSkills) {
           onSkillLoaded?.(s.name);
@@ -9399,7 +11878,7 @@ ${message}` : message;
 function readLatestExplainTurn(projectPath) {
   const dir = join(projectPath, PATHS.explainDir);
   if (!existsSync(dir)) return null;
-  const files = readdirSync$1(dir).filter((name) => name.endsWith(".turn.json")).sort();
+  const files = readdirSync(dir).filter((name) => name.endsWith(".turn.json")).sort();
   if (files.length === 0) return null;
   const latest = files[files.length - 1];
   return JSON.parse(readFileSync(join(dir, latest), "utf-8"));
@@ -9862,7 +12341,9 @@ function toPaperInput(paper) {
   };
 }
 function hasCoreMetadataDelta(before, after) {
-  return after.venue && after.venue !== before.venue || after.doi && after.doi !== before.doi || after.citationCount != null && after.citationCount !== before.citationCount || after.url && after.url !== before.url || after.abstract && after.abstract !== before.abstract;
+  return Boolean(
+    after.venue && after.venue !== before.venue || after.doi && after.doi !== before.doi || after.citationCount != null && after.citationCount !== before.citationCount || after.url && after.url !== before.url || after.abstract && after.abstract !== before.abstract
+  );
 }
 async function enrichPaperArtifacts(options) {
   const { projectPath, sessionId, debug, paperIds, onProgress } = options;
@@ -10121,7 +12602,7 @@ function resolveEntity(ref, dir, entityType, projectPath) {
   if (!existsSync(dir)) {
     return { ref, label: ref.raw, content: "", error: `No ${entityType} directory found` };
   }
-  const files = readdirSync$1(dir).filter((f) => f.endsWith(".json"));
+  const files = readdirSync(dir).filter((f) => f.endsWith(".json"));
   const key = ref.key.toLowerCase();
   for (const file of files) {
     try {
@@ -10335,7 +12816,7 @@ function walk(root, rel, depth, out) {
   const dir = rel ? join(root, rel) : root;
   let entries;
   try {
-    entries = readdirSync$1(dir);
+    entries = readdirSync(dir);
   } catch {
     return;
   }
@@ -11077,7 +13558,8 @@ function resolveSettings(settings) {
     researchIntensity: resolveResearchIntensity(settings.research.researchIntensity),
     webSearch: resolveWebSearchDepth(settings.research.webSearchDepth),
     dataAnalysis: { timeoutMs: resolveDataAnalysisTimeout(settings.dataAnalysis.executionTimeLimit) },
-    autoSaveThreshold: resolveAutoSaveThreshold(settings.research.autoSaveSensitivity)
+    autoSaveThreshold: resolveAutoSaveThreshold(settings.research.autoSaveSensitivity),
+    diagram: { reviewProvider: settings.diagram?.reviewProvider ?? "auto" }
   };
 }
 let _wikiLock = Promise.resolve();
@@ -11278,7 +13760,7 @@ function applyIdentityMigration(change) {
   if (oldSlug !== newSlug) {
     const conceptsDir = join(root, "concepts");
     if (existsSync(conceptsDir)) {
-      for (const f of readdirSync$1(conceptsDir)) {
+      for (const f of readdirSync(conceptsDir)) {
         if (!f.endsWith(".md")) continue;
         const path2 = join(conceptsDir, f);
         const content = safeReadFile(path2);
@@ -11346,6 +13828,46 @@ function scanForNewContent(projectPaths) {
       byCanonicalKey.set(identity.canonicalKey, existing);
     }
   }
+  {
+    const byArxivId = /* @__PURE__ */ new Map();
+    for (const [key, entries] of byCanonicalKey) {
+      for (const { artifact } of entries) {
+        if (artifact.arxivId && isValidArxivId(artifact.arxivId)) {
+          const bareId = artifact.arxivId.replace(/^https?:\/\/arxiv\.org\/abs\//, "").replace(/v\d+$/, "");
+          const keys = byArxivId.get(bareId);
+          if (keys) {
+            if (!keys.includes(key)) keys.push(key);
+          } else {
+            byArxivId.set(bareId, [key]);
+          }
+        }
+      }
+    }
+    for (const [, keys] of byArxivId) {
+      if (keys.length < 2) continue;
+      let winnerKey = null;
+      const keyPriority = (k) => k.startsWith("doi:") ? 3 : k.startsWith("arxiv:") ? 2 : 1;
+      for (const k of keys) {
+        for (const entry of byCanonicalKey.get(k) || []) {
+          const ident = computeCanonicalKey(entry.artifact);
+          if (!winnerKey || keyPriority(ident.canonicalKey) > keyPriority(winnerKey)) {
+            winnerKey = ident.canonicalKey;
+          }
+        }
+      }
+      if (!winnerKey || !keys.includes(winnerKey)) {
+        winnerKey = keys.reduce((a, b) => keyPriority(a) > keyPriority(b) ? a : b);
+      }
+      for (const loserKey of keys) {
+        if (loserKey === winnerKey) continue;
+        const loserEntries = byCanonicalKey.get(loserKey);
+        if (!loserEntries) continue;
+        const winnerEntries = byCanonicalKey.get(winnerKey);
+        winnerEntries.push(...loserEntries);
+        byCanonicalKey.delete(loserKey);
+      }
+    }
+  }
   for (const [canonicalKey, entries] of byCanonicalKey) {
     const best = entries.reduce((a, b) => {
       if (a.artifact.fulltextPath && !b.artifact.fulltextPath) return a;
@@ -11646,7 +14168,7 @@ function listExistingConceptSlugs() {
   const dir = join(getWikiRoot(), "concepts");
   if (!existsSync(dir)) return [];
   try {
-    return readdirSync$1(dir).filter((f) => f.endsWith(".md")).map((f) => f.replace(".md", ""));
+    return readdirSync(dir).filter((f) => f.endsWith(".md")).map((f) => f.replace(".md", ""));
   } catch {
     return [];
   }
@@ -11827,7 +14349,7 @@ function createWikiAgent(config) {
   }
   function emitStatus(pending = 0) {
     if (!config.onStatus) return;
-    const totalInWiki = existsSync(join(getWikiRoot(), "papers")) ? readdirSync$1(join(getWikiRoot(), "papers")).filter((f) => f.endsWith(".md")).length : 0;
+    const totalInWiki = existsSync(join(getWikiRoot(), "papers")) ? readdirSync(join(getWikiRoot(), "papers")).filter((f) => f.endsWith(".md")).length : 0;
     config.onStatus({
       state: state === "processing" ? "processing" : state === "paused" ? "paused" : "idle",
       processed: processedThisSession,
@@ -12284,7 +14806,7 @@ function listWikiPaperMeta() {
   if (!existsSync(papersDir)) return [];
   const seen = /* @__PURE__ */ new Set();
   const results = [];
-  for (const file of readdirSync$1(papersDir)) {
+  for (const file of readdirSync(papersDir)) {
     if (!file.endsWith(".md")) continue;
     const slug = file.slice(0, -3);
     const filePath = join(papersDir, file);
@@ -12321,9 +14843,9 @@ function classifyKey(canonicalKey) {
   return "title";
 }
 const TIER_RANK = {
-  arxiv: 3,
-  // prefer real arxiv
-  doi: 2,
+  doi: 3,
+  // DOI > arXiv > title — matches computeCanonicalKey priority
+  arxiv: 2,
   title: 1,
   bogus: 0
 };
@@ -12351,7 +14873,7 @@ function findDriftGroups() {
     }
   }
   const byTitle = /* @__PURE__ */ new Map();
-  for (const f of readdirSync$1(papersDir)) {
+  for (const f of readdirSync(papersDir)) {
     if (!f.endsWith(".md")) continue;
     const slug = f.slice(0, -3);
     const content = safeReadFile(join(papersDir, f));
@@ -12449,7 +14971,7 @@ async function reconcileIdentityDrift(opts = { dryRun: true }) {
       const papersDir = join(getWikiRoot(), "papers");
       const liveSlugs = /* @__PURE__ */ new Set();
       if (existsSync(papersDir)) {
-        for (const f of readdirSync$1(papersDir)) {
+        for (const f of readdirSync(papersDir)) {
           if (f.endsWith(".md")) liveSlugs.add(f.slice(0, -3));
         }
       }
@@ -12467,6 +14989,61 @@ async function reconcileIdentityDrift(opts = { dryRun: true }) {
     };
   });
 }
+const DEFAULT_WIDTH = 1200;
+const DEFAULT_HEIGHT = 900;
+const SETTLE_MS = 200;
+function viewBoxDimensions(svg) {
+  const match = svg.match(/viewBox\s*=\s*"([^"]+)"/i);
+  if (!match) return null;
+  const parts = match[1].trim().split(/[\s,]+/).map(Number);
+  if (parts.length !== 4 || parts.some((n) => !Number.isFinite(n))) return null;
+  const [, , w, h] = parts;
+  if (w <= 0 || h <= 0) return null;
+  return { width: Math.round(w), height: Math.round(h) };
+}
+function wrapHtml(svg, width, height) {
+  return [
+    '<!doctype html><html><head><meta charset="utf-8"><style>',
+    `html, body { margin: 0; padding: 0; background: #ffffff; width: ${width}px; height: ${height}px; overflow: hidden; }`,
+    "svg { display: block; width: 100%; height: 100%; }",
+    "</style></head><body>",
+    svg,
+    "</body></html>"
+  ].join("");
+}
+async function rasterizeSvg(svg, options = {}) {
+  const svgStr = svg.toString("utf-8");
+  const vb = viewBoxDimensions(svgStr);
+  const width = options.width ?? vb?.width ?? DEFAULT_WIDTH;
+  const height = options.height ?? vb?.height ?? DEFAULT_HEIGHT;
+  const win = new BrowserWindow({
+    width,
+    height,
+    show: false,
+    frame: false,
+    transparent: false,
+    webPreferences: {
+      offscreen: true,
+      backgroundThrottling: false,
+      nodeIntegration: false,
+      contextIsolation: true,
+      sandbox: true,
+      // Sanity: SVG content is produced by our own prompt; nothing is user-
+      // supplied from a network. Still, disable remote resources to be safe.
+      webSecurity: true
+    }
+  });
+  try {
+    const html = wrapHtml(svgStr, width, height);
+    const dataUrl = "data:text/html;charset=utf-8," + encodeURIComponent(html);
+    await win.loadURL(dataUrl);
+    await new Promise((resolve2) => setTimeout(resolve2, SETTLE_MS));
+    const image = await win.webContents.capturePage();
+    return image.toPNG();
+  } finally {
+    if (!win.isDestroyed()) win.destroy();
+  }
+}
 const EMPTY = {
   version: 1,
   updatedAt: (/* @__PURE__ */ new Date(0)).toISOString(),
@@ -12775,7 +15352,7 @@ async function ensureCoordinator(state, win, model, options) {
         if (creds.expires < Date.now() + 6e4) {
           try {
             const { refreshOpenAICodexToken } = await import("@mariozechner/pi-ai/oauth");
-            const newCreds = await refreshOpenAICodexToken(creds);
+            const newCreds = await refreshOpenAICodexToken(creds.refresh);
             saveCodexCredentials(newCreds);
             return newCreds.access;
           } catch {
@@ -12788,12 +15365,46 @@ async function ensureCoordinator(state, win, model, options) {
     const initEvent = { type: "system", summary: "Initializing agent (first run may take 1-2 minutes for document processing setup)..." };
     state.realtimeBuffer.pushActivity(initEvent);
     safeSend(win, "agent:activity", initEvent);
+    const getDiagramAuth = () => {
+      const openaiKey = (process.env.OPENAI_API_KEY || "").trim() || null;
+      const envAnthropic = (process.env.ANTHROPIC_API_KEY || "").trim();
+      if (envAnthropic) {
+        return { openaiKey, anthropic: { token: envAnthropic, isOAuth: false } };
+      }
+      const creds = loadAnthropicSubCredentials();
+      if (creds?.access) {
+        const refreshTokenSymbol = creds.refresh;
+        return {
+          openaiKey,
+          anthropic: {
+            token: creds.access,
+            isOAuth: true,
+            refresh: async () => {
+              const { refreshAnthropicToken } = await import("@mariozechner/pi-ai/oauth");
+              const fresh = await refreshAnthropicToken(refreshTokenSymbol);
+              saveAnthropicSubCredentials(fresh);
+              return fresh.access;
+            }
+          }
+        };
+      }
+      return { openaiKey, anthropic: null };
+    };
     state.coordinator = await createCoordinator({
       apiKey,
       getApiKeyOverride,
       model: state.currentModel,
       reasoningEffort: state.currentReasoningEffort,
       resolvedSettings: resolveSettings(loadSettingsFromConfig()),
+      // Live settings reader: re-reads ~/.research-copilot/config.json per
+      // tool call so diagram review-provider choice (and similar
+      // presentation-layer settings) take effect without restart.
+      getResolvedSettings: () => resolveSettings(loadSettingsFromConfig()),
+      getDiagramAuth,
+      // Only wired in Electron's main process; pure-Node contexts (tests)
+      // will leave this undefined and the tool will degrade to source-
+      // level SVG review.
+      rasterizeSvg,
       projectPath: state.projectPath,
       sessionId: state.sessionId,
       debug: !!process.env.RESEARCH_COPILOT_DEBUG,
@@ -12851,16 +15462,14 @@ async function ensureCoordinator(state, win, model, options) {
           const r2 = result;
           if (r2.success) {
             invalidateEntityCache(runProjectPath);
-            if (tool === "artifact-create") {
-              safeSend(win, "agent:entity-created", {
-                type: r2.data?.type || "artifact",
-                id: r2.data?.id,
-                title: r2.data?.title
-              });
-              if (r2.data?.filePath) {
-                const absPath = isAbsolute(r2.data.filePath) ? r2.data.filePath : resolve(runProjectPath, r2.data.filePath);
-                safeSend(win, "agent:file-created", absPath);
-              }
+            safeSend(win, "agent:entity-created", {
+              type: r2.data?.type || "artifact",
+              id: r2.data?.id,
+              title: r2.data?.title
+            });
+            if (tool === "artifact-create" && r2.data?.filePath) {
+              const absPath = isAbsolute(r2.data.filePath) ? r2.data.filePath : resolve(runProjectPath, r2.data.filePath);
+              safeSend(win, "agent:file-created", absPath);
             }
           }
         }
@@ -13123,7 +15732,7 @@ function registerIpcHandlers() {
             if (creds.expires < Date.now() + 6e4) {
               try {
                 const { refreshOpenAICodexToken } = await import("@mariozechner/pi-ai/oauth");
-                const newCreds = await refreshOpenAICodexToken(creds);
+                const newCreds = await refreshOpenAICodexToken(creds.refresh);
                 saveCodexCredentials(newCreds);
                 return newCreds.access;
               } catch {
@@ -13957,6 +16566,35 @@ function destroyAllTerminals() {
     terminals.delete(id);
   }
 }
+protocol.registerSchemesAsPrivileged([
+  {
+    scheme: "workspace-asset",
+    privileges: {
+      standard: true,
+      secure: true,
+      supportFetchAPI: true,
+      stream: true,
+      corsEnabled: true
+    }
+  }
+]);
+function mimeForExtension(p) {
+  const ext = (p.split(".").pop() || "").toLowerCase();
+  const table = {
+    png: "image/png",
+    jpg: "image/jpeg",
+    jpeg: "image/jpeg",
+    gif: "image/gif",
+    webp: "image/webp",
+    svg: "image/svg+xml",
+    bmp: "image/bmp",
+    avif: "image/avif",
+    ico: "image/x-icon",
+    heic: "image/heic",
+    heif: "image/heif"
+  };
+  return table[ext] || "application/octet-stream";
+}
 setMaxListeners(20);
 loadApiKeysFromConfig();
 if (!process.env.PI_CACHE_RETENTION) {
@@ -14026,6 +16664,22 @@ app.whenReady().then(() => {
   if (iconPath && process.platform === "darwin") {
     app.dock?.setIcon(iconPath);
   }
+  protocol.handle("workspace-asset", async (request) => {
+    try {
+      const url = new URL(request.url);
+      const absPath = decodeURIComponent(url.pathname);
+      const data = await readFile(absPath);
+      return new Response(data, {
+        status: 200,
+        headers: {
+          "Content-Type": mimeForExtension(absPath),
+          "Cache-Control": "no-cache"
+        }
+      });
+    } catch {
+      return new Response("", { status: 404 });
+    }
+  });
   registerIpcHandlers();
   registerTerminalHandlers();
   registerWindow(createWindow());