dsh-eda-mcp 1.0.0 → 1.0.1

package/CHANGELOG.md

@@ -1,5 +1,38 @@
 # Changelog
 
+## 1.0.1 — 2026-10-20
+
+1.0.0 的真机回归修掉 4 个 bug（全部由"跑一遍"发现，不是靠读代码）：
+
+1. **原理图几何审计误报 30 对重叠**（`eda_schematic_audit`）
+   `sch_PrimitiveComponent.getAll` 的列表里混着图框/标题栏图元，其包围盒是整张页面（实测 `0..1170 × 0..825`），于是与每个符号"重叠"。现在**丢弃面积超过页面 40% 的图元**并在 `droppedSheetPrimitives` 里报数量；复测：30 对误报 → **0 fail**（31 器件 / 30 个有效包围盒 / 丢弃 1 个）。
+
+2. **参数持久化"报成功但没写"**（`eda_design_intake`）
+   ① `dmt_Project.modifyProjectDescription` 在本机构建上**返回 false、不生效**（与 create/delete/copyProject 同类）；② 我的调用还把文本当成 uuid 传。现在：先按官方签名 `(projectUuid, description)` 写并**回读校验**，失败则落到工程目录侧的**侧车文件** `.dsh-eda-spec/<uuid>.json`，`persisted` 只在真的读回成功后为 true，并新增 `persistVia`（`description` / `file` / `none`）与 `persistPath`。真机复测：`persisted=true via=file`。
+
+3. **PCB 机械检查是死代码**（`eda_design_audit`）
+   `pcb_PrimitiveComponent.getAll([undefined, undefined])` 在该构建上返回**空列表**（同样的 API 在 `pcb_get_components` 里能取到 29 个器件），导致器件重叠 / 出板框两项检查从未执行。改用与 `pcb_get_components` 相同的调用形态，并把 `counts.componentsRead` 暴露出来；复测：`componentsRead: 29`。
+
+4. **交付清单挂死超时**（`eda_delivery_checklist`）
+   自动切文档后紧接着调网表导出会被页面加载阻塞 → MCP 超时。现在切文档后等待 1.5 s；同时把**跳过**与**失败**分开：新增 `verdict`（`pass` / `incomplete` / `fail`）、`skippedSteps`、`failedSteps`、`next`，默认仍自动切文档。真机复测：**一次调用 7.5 s，四步全过 `verdict=pass`**；`switchDocuments:false` 时如实报 `incomplete` 并指出缺哪个文档。
+
+### 回归结果（扩展 0.4.6 / 桥接 1.0.1）
+
+| 项 | 结果 |
+|---|---|
+| `eda_schematic_audit` | ✅ 0 fail / 1 warn（12 个符号偏离 10 单位栅格）/ 1 skip（无标签锚点） |
+| `eda_netlist_erc` | ✅ 15 项全执行，14 pass / 0 fail / 1 warn（USB-C SBU1/SBU2 未接） |
+| `eda_design_intake` 持久化 | ✅ `persisted=true via=file` |
+| `eda_design_audit`（PCB） | ✅ `componentsRead:29`、0 违规 / 4 警告（5V 线宽偏细、无覆铜、无丝印、无定位孔） |
+| `eda_delivery_checklist` | ✅ `verdict=pass`、7.5 s、四步（ERC / BOM / PCB 审计 / Gerber 86.8 KB） |
+| `eda_pcb_canvas_origin` | ✅ 偏移 `0,0`，画布↔数据坐标互转一致 |
+| `eda_pcb_create_pour` | ❌ **仍失败**：单多边形与复杂多边形都被接受，但 `pcb_PrimitivePour.create` 对任何网络/层/填充方式一律拒绝 → 判定为**应用侧限制**（1.0.0 里我写"是我用错 API"，据此修正） |
+
+### 已知遗留（需你决定）
+
+- 侧车目录 `D:\文档\LCEDA-Pro\projects\.dsh-eda-spec\` 是回归测试写入的（1 个 json），清理需要你确认（它在你的工程目录内）。
+- 覆铜只能手工：工具已把失败原因写准，`eda_design_audit` 持续报 `copper-pour` warn。
+
 ## 1.0.0 — 2026-10-20
 
 第一个稳定版。0.7.4 之后这一轮把"能画"推进到"画得对、画得像参考设计、交付有证据"：工具 106 → **109**，单测 6 → **8 套**，线协议 0.4.2 → **0.4.6**。

package/docs/schematic-workflow.md

@@ -239,6 +239,13 @@
 | `eda_delivery_checklist` | 一次串完 ERC + PCB 审计/DRC + Gerber + BOM，逐步 PASS/FAIL；`outPath` 落 Markdown 报告 | 交付前，代替"我说做完了" |
 
 **交付口径**：`eda_delivery_checklist` 返回 `ok:true` 才算可以下单；任何一步 FAIL 都要先修，不许口头绕过。报告里带时间戳与计数，可直接给人复核。
+### 交付清单怎么读（1.0.1 起）
+
+- 一次调用会**自己切到原理图再切到 PCB**（`switchDocuments:false` 改按当前文档跑一半，另一半标注跳过原因）；
+- 返回 `verdict`：`pass`（四步全过）/ `incomplete`（有步骤因文档不对被跳过）/ `fail`（有步骤真失败）；`ok` 等价于 `verdict==='pass'`；
+- `skippedSteps` 与 `failedSteps` 分开列 —— **别把"没跑"当成"通过"**；四步全过实测约 **7.5 s**；
+- 投板前最后一条命令就用它，`outPath` 落 Markdown 存档。
+
 
 **参数持久化**：`eda_design_intake({answers})` 会把答复写进**工程描述**（`[dsh-eda-spec] {...}`），下次会话调用 `eda_design_intake` 会自动读回，不用重新猜；`reset:true` 清空。
 

package/docs/tools.md

@@ -53,7 +53,7 @@ Design limits enforced by these tools (track widths, via sizes, net naming) are
 | `eda_open_document` | Open/activate a document tab by uuid (dmt_EditorControl.openDocument). Works for a schematic page or a PCB, so tab-dependent APIs can run. | `documentUuid: string, splitScreenId?: string` |
 | `eda_get_net_labels` | List every label-like primitive on the active schematic page: attribute primitives (netlabel/netport/netflag) PLUS plain texts left by the createNetLabel fallback, so they can be audited and deleted. Reports kind/primitiveId/net per entry. | `parentPrimitiveId?: string` |
 | `eda_schematic_audit` | Geometry audit of the active schematic page (the netlist ERC only judges connectivity): symbols outside the frame, symbols overlapping the title block, body-on-body symbol overlap, off-grid placement, and labels anchored inside a symbol body. Read-only; pair it with eda_netlist_erc before routing. | `grid?: number, includeOk?: boolean` |
-| `eda_delivery_checklist` | Delivery checklist: run connectivity (eda_netlist_erc), PCB audit + DRC, Gerber export and BOM export in ONE read-only call and report per-step pass/fail with counts. Pass outPath to also write a Markdown report you can hand to a reviewer. Use it instead of claiming a board is finished. | `outPath?: string, pcbDrc?: boolean` |
+| `eda_delivery_checklist` | Delivery checklist: run connectivity (eda_netlist_erc), PCB audit + DRC, Gerber export and BOM export in ONE read-only call and report per-step pass/fail with counts. It switches to the schematic and the PCB itself (switchDocuments:false keeps the current document and reports the other steps as skipped). verdict is pass / incomplete / fail; pass outPath to also write a Markdown report you can hand to a reviewer. | `outPath?: string, pcbDrc?: boolean, switchDocuments?: boolean` |
 | `eda_design_intake` | Requirement intake for a new board: call it BEFORE placing anything when the request is vague (for example: draw me an X board). Without answers it returns the questions in 6 sections (power, core, IO, mechanical, process, delivery), each marked critical or fallback-able with the reason it changes the drawing; pass answers:{questionId:value} to record them and see what is still missing. Ask the user with ask_user_question for the critical ones instead of guessing. | `answers?: object, reset?: boolean` |
 | `eda_netlist_erc` | Read-only electrical smoke test over the schematic NETLIST (no application DRC needed): duplicate designators, unconnected pins, single-endpoint nets, shorted parts, power rails and decoupling, USB D+/D- pair, crystal (two nets + load caps + reaches the MCU), reset RC, BOOT pull resistor, Type-C CC pull-downs. DRC can be clean while the circuit is still wrong; these failures all show up in the netlist. Pass includeOk:true to see passing checks too. | `includeOk?: boolean` |
 | `eda_pcb_canvas_origin` | Read (or align) the PCB canvas origin and convert points between the two coordinate spaces. JLCEDA displays CANVAS coordinates in its UI but every API takes DATA coordinates; the two differ by this offset, so a coordinate read off the screen will land somewhere else unless you convert or align. align:true calls setCanvasOrigin(0,0) so the display matches the API. | `align?: boolean, offsetX?: number, offsetY?: number, point?: object` |

package/lib/bridge.mjs

@@ -3,7 +3,7 @@ import { createHash } from "node:crypto";
 import { execFileSync, spawn } from "node:child_process";
 import { statSync } from "node:fs";
 import { readFile } from "node:fs/promises";
-import { unlink, writeFile } from "node:fs/promises";
+import { mkdir, unlink, writeFile } from "node:fs/promises";
 import { tmpdir } from "node:os";
 import { basename, join } from "node:path";
 import { createServer } from "node:http";
@@ -1094,7 +1094,7 @@ var TOOLS = [
   { name: "pcb_draw_text", description: "Put text on a PCB layer - silkscreen by default (3 top silk / 4 bottom silk). Use it for pin functions, board markings, version and fabrication notes; silkscreen text must not cover pads.", inputSchema: objectSchema({ x: { type: "number" }, y: { type: "number" }, text: { type: "string" }, layer: { type: "string" }, fontSize: { type: "number" }, lineWidth: { type: "number" }, rotation: { type: "number" }, mirror: { type: "boolean" }, alignMode: { type: "integer" }, locked: { type: "boolean" } }, ["x", "y", "text"]) },
   { name: "pcb_draw_mounting_hole", description: 'Place a NON-plated mounting hole or a slot (pcb_PrimitivePad with metallization:false). shape "round" is a drill-sized hole, shape "slot" an oval cut-out. Use this for mounting holes and connector slots instead of hacking the board outline.', inputSchema: objectSchema({ x: { type: "number" }, y: { type: "number" }, diameter: { type: "number" }, shape: { type: "string", enum: ["round", "slot"] }, slotLength: { type: "number" }, rotation: { type: "number" }, plated: { type: "boolean" }, annulus: { type: "number" }, padNumber: { type: "string" }, layer: { type: "string" } }, ["x", "y", "diameter"]) },
   { name: "eda_schematic_audit", description: "Geometry audit of the active schematic page (the netlist ERC only judges connectivity): symbols outside the frame, symbols overlapping the title block, body-on-body symbol overlap, off-grid placement, and labels anchored inside a symbol body. Read-only; pair it with eda_netlist_erc before routing.", inputSchema: objectSchema({ grid: { type: "number" }, includeOk: { type: "boolean" } }) },
-  { name: "eda_delivery_checklist", description: "Delivery checklist: run connectivity (eda_netlist_erc), PCB audit + DRC, Gerber export and BOM export in ONE read-only call and report per-step pass/fail with counts. Pass outPath to also write a Markdown report you can hand to a reviewer. Use it instead of claiming a board is finished.", inputSchema: objectSchema({ outPath: { type: "string" }, pcbDrc: { type: "boolean" } }) },
+  { name: "eda_delivery_checklist", description: "Delivery checklist: run connectivity (eda_netlist_erc), PCB audit + DRC, Gerber export and BOM export in ONE read-only call and report per-step pass/fail with counts. It switches to the schematic and the PCB itself (switchDocuments:false keeps the current document and reports the other steps as skipped). verdict is pass / incomplete / fail; pass outPath to also write a Markdown report you can hand to a reviewer.", inputSchema: objectSchema({ outPath: { type: "string" }, pcbDrc: { type: "boolean" }, switchDocuments: { type: "boolean" } }) },
   { name: "eda_design_intake", description: "Requirement intake for a new board: call it BEFORE placing anything when the request is vague (for example: draw me an X board). Without answers it returns the questions in 6 sections (power, core, IO, mechanical, process, delivery), each marked critical or fallback-able with the reason it changes the drawing; pass answers:{questionId:value} to record them and see what is still missing. Ask the user with ask_user_question for the critical ones instead of guessing.", inputSchema: objectSchema({ answers: { type: "object", additionalProperties: true }, reset: { type: "boolean" } }) },
   { name: "eda_netlist_erc", description: "Read-only electrical smoke test over the schematic NETLIST (no application DRC needed): duplicate designators, unconnected pins, single-endpoint nets, shorted parts, power rails and decoupling, USB D+/D- pair, crystal (two nets + load caps + reaches the MCU), reset RC, BOOT pull resistor, Type-C CC pull-downs. DRC can be clean while the circuit is still wrong; these failures all show up in the netlist. Pass includeOk:true to see passing checks too.", inputSchema: objectSchema({ includeOk: { type: "boolean" } }) },
   { name: "eda_pcb_canvas_origin", description: "Read (or align) the PCB canvas origin and convert points between the two coordinate spaces. JLCEDA displays CANVAS coordinates in its UI but every API takes DATA coordinates; the two differ by this offset, so a coordinate read off the screen will land somewhere else unless you convert or align. align:true calls setCanvasOrigin(0,0) so the display matches the API.", inputSchema: objectSchema({ align: { type: "boolean" }, offsetX: { type: "number" }, offsetY: { type: "number" }, point: { type: "object", additionalProperties: true } }) },
@@ -1509,6 +1509,13 @@ var BRIEF_TAG = "[dsh-eda-spec]";
 function briefBlock(brief) {
   return BRIEF_TAG + " " + JSON.stringify(brief);
 }
+function specFilePath(project) {
+  const dir = String(project?.teamUuid ?? "");
+  const uuid2 = String(project?.uuid ?? "");
+  if (dir === "" || uuid2 === "") return "";
+  if (!/^[A-Za-z]:[\\/]/.test(dir) && !dir.startsWith("/")) return "";
+  return join(dir, ".dsh-eda-spec", uuid2 + ".json");
+}
 function parseBriefBlock(description) {
   const text = typeof description === "string" ? description : "";
   const index = text.indexOf(BRIEF_TAG);
@@ -1589,7 +1596,7 @@ async function auditPcb(args, timeoutMs) {
   const vias = await pcbPrimitiveList("pcb_PrimitiveVia.getAll", [netFilter, void 0], timeoutMs);
   const pours = await pcbPrimitiveList("pcb_PrimitivePour.getAll", [netFilter, void 0, void 0], timeoutMs);
   const outline = await pcbPrimitiveList("pcb_PrimitivePolyline.getAll", [void 0, PCB_LAYERS.boardOutline, void 0], timeoutMs);
-  const pcbComponents = await pcbPrimitiveList("pcb_PrimitiveComponent.getAll", [void 0, void 0], timeoutMs);
+  const pcbComponents = await callExtension("pcb_PrimitiveComponent.getAll", [resolvePcbLayer(void 0), void 0], timeoutMs).then((raw) => Array.isArray(raw) ? raw : []).catch(() => []);
   const violations = [];
   const warnings = [];
   const add = (bucket, rule, message, samples) => {
@@ -1726,7 +1733,7 @@ async function auditPcb(args, timeoutMs) {
   ];
   return {
     ok: violations.length === 0,
-    counts: { tracks: tracks.length, vias: vias.length, pours: pours.length, outlinePolylines: outline.length, nets: netNames.length },
+    counts: { tracks: tracks.length, vias: vias.length, pours: pours.length, outlinePolylines: outline.length, nets: netNames.length, componentsRead: pcbComponents.length },
     minWidthByNet: Object.fromEntries([...widthByNet.entries()].map(([net, width]) => [net, { widthMil: width, role: classifyNet(net), suggestedMil: suggestTrackWidthMil(net) }])),
     violations,
     warnings,
@@ -3078,23 +3085,33 @@ async function executeTool(name, rawArgs, timeoutMs) {
     }
     case "eda_schematic_audit": {
       const paper = paperOf(args.paper);
+      const frame = paper.frame;
+      const frameArea = (frame.right - frame.left) * (frame.top - frame.bottom);
       const rawList = await callExtension("sch_PrimitiveComponent.getAll", [], timeoutMs);
       const list = Array.isArray(rawList) ? rawList.slice(0, 200) : [];
       const components = [];
+      let droppedSheetPrimitives = 0;
       for (let index = 0; index < list.length; index += 1) {
         const entry = list[index] ?? {};
-        const designator = String(entry.designator ?? entry.Designator ?? entry.name ?? "#" + index);
+        const designator = String(entry.designator ?? entry.Designator ?? entry.Name ?? entry.name ?? "#" + index);
         const id = String(entry.primitiveId ?? entry.PrimitiveId ?? "");
         let bounds;
         if (id !== "") {
           const box = await bboxOf([id], timeoutMs);
           if (box !== void 0) bounds = { left: box.minX, right: box.maxX, bottom: box.minY, top: box.maxY };
         }
+        if (bounds !== void 0) {
+          const boundsArea = (bounds.right - bounds.left) * (bounds.top - bounds.bottom);
+          if (boundsArea > frameArea * 0.4) {
+            bounds = void 0;
+            droppedSheetPrimitives += 1;
+          }
+        }
         components.push({ designator, x: Number(entry.x ?? entry.X ?? 0), y: Number(entry.y ?? entry.Y ?? 0), bounds });
       }
       const report = auditSchematicGeometry({
         components,
-        frame: paper.frame,
+        frame,
         titleBlock: paper.titleBlock,
         grid: args.grid === void 0 ? 10 : Number(args.grid)
       });
@@ -3104,57 +3121,111 @@ async function executeTool(name, rawArgs, timeoutMs) {
         counts: report.counts,
         componentCount: report.componentCount,
         boxesRead: components.filter((component) => component.bounds !== void 0).length,
-        checks
+        droppedSheetPrimitives,
+        checks,
+        notes: droppedSheetPrimitives > 0 ? ["Skipped " + droppedSheetPrimitives + " primitive(s) whose box covers the whole sheet (frame/title block); counting them made every symbol look overlapped."] : []
       };
     }
     case "eda_delivery_checklist": {
       const steps = [];
       const startedAt = (/* @__PURE__ */ new Date()).toISOString();
+      const project = await callExtension("dmt_Project.getCurrentProjectInfo", [], timeoutMs).catch(() => null);
+      const boards = Array.isArray(project?.data) ? project.data : [];
+      const schematicPage = boards[0]?.schematic?.page?.[0]?.uuid ?? "";
+      const pcbUuid = boards.find((board) => board?.pcb?.uuid)?.pcb?.uuid ?? "";
+      const switchDocuments = args.switchDocuments !== false;
+      const documentType = async () => {
+        try {
+          const info = await callExtension("dmt_SelectControl.getCurrentDocumentInfo", [], timeoutMs);
+          return Number(info?.documentType);
+        } catch {
+          return NaN;
+        }
+      };
+      const activate = async (uuid2) => {
+        if (typeof uuid2 !== "string" || uuid2 === "") return false;
+        try {
+          await callExtension("dmt_EditorControl.openDocument", [uuid2], timeoutMs);
+          await new Promise((resolve) => setTimeout(resolve, 1500));
+          return true;
+        } catch {
+          return false;
+        }
+      };
       let netlistErc = null;
-      try {
-        const parsedNetlist = await fetchParsedNetlist(timeoutMs);
-        netlistErc = runNetlistErc(parsedNetlist);
-        steps.push({ step: "netlist-erc", ok: netlistErc.ok, detail: netlistErc.counts, failing: netlistErc.checks.filter((check) => check.status === "fail").map((check) => check.id) });
-      } catch (error) {
-        steps.push({ step: "netlist-erc", ok: false, error: error instanceof Error ? error.message : String(error) });
+      const onSchematic = switchDocuments ? await activate(schematicPage) : await documentType() === 1;
+      if (!onSchematic) {
+        steps.push({ step: "netlist-erc", document: "schematic", ok: false, skipped: "schematic page is not the active document" });
+        steps.push({ step: "bom", document: "schematic", ok: false, skipped: "schematic page is not the active document" });
+      } else {
+        try {
+          const parsedNetlist = await fetchParsedNetlist(timeoutMs);
+          netlistErc = runNetlistErc(parsedNetlist);
+          steps.push({
+            step: "netlist-erc",
+            document: "schematic",
+            ok: netlistErc.ok,
+            detail: netlistErc.counts,
+            failing: netlistErc.checks.filter((check) => check.status === "fail").map((check) => check.id)
+          });
+        } catch (error) {
+          steps.push({ step: "netlist-erc", document: "schematic", ok: false, error: error instanceof Error ? error.message : String(error) });
+        }
+      }
+      let bom = null;
+      if (onSchematic) {
+        try {
+          const payload = await callExtension("sch_ManufactureData.getBomFile", [void 0, "csv"], timeoutMs);
+          const text = typeof payload === "string" ? payload : typeof payload?.text === "string" ? payload.text : "";
+          const lines = text === "" ? 0 : text.split(/\r?\n/).filter((line) => line.trim() !== "").length;
+          bom = { rows: Math.max(0, lines - 1), ok: lines > 1 };
+          steps.push({ step: "bom", document: "schematic", ok: bom.ok, detail: { rows: bom.rows }, ...bom.ok ? {} : { error: "no BOM rows exported" } });
+        } catch (error) {
+          steps.push({ step: "bom", document: "schematic", ok: false, error: error instanceof Error ? error.message : String(error) });
+        }
       }
+      const onPcb = switchDocuments ? await activate(pcbUuid) : await documentType() === 3;
       let pcbAudit = null;
-      try {
-        pcbAudit = await auditPcb({ drc: args.pcbDrc !== false }, timeoutMs);
-        steps.push({
-          step: "pcb-audit",
-          ok: Array.isArray(pcbAudit.violations) && pcbAudit.violations.length === 0,
-          detail: { violations: pcbAudit.violations?.length ?? null, warnings: pcbAudit.warnings?.length ?? null, drc: pcbAudit.drc },
-          failing: (pcbAudit.violations ?? []).map((entry) => entry.rule)
-        });
-      } catch (error) {
-        steps.push({ step: "pcb-audit", ok: false, error: error instanceof Error ? error.message : String(error) });
+      if (!onPcb) {
+        steps.push({ step: "pcb-audit", document: "pcb", ok: false, skipped: "PCB is not the active document" });
+        steps.push({ step: "gerber", document: "pcb", ok: false, skipped: "PCB is not the active document" });
+      } else {
+        try {
+          pcbAudit = await auditPcb({ drc: args.pcbDrc !== false }, timeoutMs);
+          steps.push({
+            step: "pcb-audit",
+            document: "pcb",
+            ok: (pcbAudit.violations ?? []).length === 0,
+            detail: { violations: pcbAudit.violations?.length ?? null, warnings: pcbAudit.warnings?.length ?? null, drc: pcbAudit.drc },
+            failing: (pcbAudit.violations ?? []).map((entry) => entry.rule)
+          });
+        } catch (error) {
+          steps.push({ step: "pcb-audit", document: "pcb", ok: false, error: error instanceof Error ? error.message : String(error) });
+        }
       }
       let gerber = null;
-      try {
-        const payload = await callExtension("pcb_ManufactureData.getGerberFile", [void 0, void 0, void 0, void 0, void 0, void 0], timeoutMs);
-        const bytes = payload && typeof payload === "object" ? Number(payload.size ?? payload.bytes ?? 0) : typeof payload === "string" ? payload.length : 0;
-        gerber = { bytes, ok: bytes > 1e3 };
-        steps.push({ step: "gerber", ok: gerber.ok, detail: { bytes } });
-      } catch (error) {
-        steps.push({ step: "gerber", ok: false, error: error instanceof Error ? error.message : String(error) });
-      }
-      let bom = null;
-      try {
-        const payload = await callExtension("sch_ManufactureData.getBomFile", [void 0, "csv"], timeoutMs);
-        const text = typeof payload === "string" ? payload : typeof payload?.text === "string" ? payload.text : "";
-        const lines = text === "" ? 0 : text.split(/\r?\n/).filter((line) => line.trim() !== "").length;
-        bom = { rows: Math.max(0, lines - 1), ok: lines > 1 };
-        steps.push({ step: "bom", ok: bom.ok, detail: { rows: bom.rows } });
-      } catch (error) {
-        steps.push({ step: "bom", ok: false, error: error instanceof Error ? error.message : String(error) });
+      if (onPcb) {
+        try {
+          const payload = await callExtension("pcb_ManufactureData.getGerberFile", [void 0, void 0, void 0, void 0, void 0, void 0], timeoutMs);
+          const bytes = payload && typeof payload === "object" ? Number(payload.size ?? payload.bytes ?? 0) : typeof payload === "string" ? payload.length : 0;
+          gerber = { bytes, ok: bytes > 1e3 };
+          steps.push({ step: "gerber", document: "pcb", ok: gerber.ok, detail: { bytes } });
+        } catch (error) {
+          steps.push({ step: "gerber", document: "pcb", ok: false, error: error instanceof Error ? error.message : String(error) });
+        }
       }
-      const failed = steps.filter((step) => step.ok !== true);
+      const failedSteps = steps.filter((step) => step.ok !== true && step.skipped === void 0);
+      const skippedSteps = steps.filter((step) => step.ok !== true && step.skipped !== void 0);
+      const verdict = failedSteps.length > 0 ? "fail" : skippedSteps.length > 0 ? "incomplete" : "pass";
       const report = {
-        ok: failed.length === 0,
+        ok: verdict === "pass",
+        verdict,
+        failedSteps: failedSteps.map((step) => step.step),
+        skippedSteps: skippedSteps.map((step) => step.step),
+        next: verdict === "pass" ? "Every step passed: the board can be ordered." : verdict === "incomplete" ? "Run the checklist again with the other document active (" + skippedSteps.map((step) => step.document).filter((value, index, all) => all.indexOf(value) === index).join(", ") + " needed)." : "Fix the failing steps first; skipped steps are listed separately.",
         startedAt,
         finishedAt: (/* @__PURE__ */ new Date()).toISOString(),
-        steps,
+        documents: { schematicPage, pcb: pcbUuid, switched: switchDocuments },
         netlistErc: netlistErc === null ? null : { ok: netlistErc.ok, counts: netlistErc.counts, componentCount: netlistErc.componentCount, netCount: netlistErc.netCount },
         pcbAudit: pcbAudit === null ? null : { violations: pcbAudit.violations, warnings: pcbAudit.warnings },
         gerber,
@@ -3168,10 +3239,12 @@ async function executeTool(name, rawArgs, timeoutMs) {
           "- \u5F00\u59CB\uFF1A" + report.startedAt,
           "- \u7ED3\u675F\uFF1A" + report.finishedAt,
           "- \u7ED3\u8BBA\uFF1A" + (report.ok ? "PASS\uFF08\u53EF\u4EE5\u4E0B\u5355\uFF09" : "FAIL\uFF08\u6709\u672A\u901A\u8FC7\u9879\uFF09"),
+          "- \u539F\u7406\u56FE\u9875\uFF1A" + (schematicPage || "\u672A\u53D6\u5230"),
+          "- PCB\uFF1A" + (pcbUuid || "\u672A\u53D6\u5230"),
           "",
-          "| \u6B65\u9AA4 | \u7ED3\u679C | \u8BF4\u660E |",
-          "|---|---|---|",
-          ...steps.map((step) => "| " + step.step + " | " + (step.ok === true ? "PASS" : "FAIL") + " | " + JSON.stringify(step.detail ?? step.error ?? {}) + " |"),
+          "| \u6B65\u9AA4 | \u6587\u6863 | \u7ED3\u679C | \u8BF4\u660E |",
+          "|---|---|---|---|",
+          ...steps.map((step) => "| " + step.step + " | " + step.document + " | " + (step.ok === true ? "PASS" : "FAIL") + " | " + JSON.stringify(step.detail ?? step.error ?? {}) + " |"),
           "",
           "## \u7535\u6C14\u68C0\u67E5\uFF08eda_netlist_erc\uFF09",
           "",
@@ -3212,17 +3285,43 @@ async function executeTool(name, rawArgs, timeoutMs) {
       const merged = { ...designBrief ?? {}, ...answersReport.brief };
       const mergedReport = buildIntake(merged);
       let persisted = false;
+      let persistVia = "none";
+      let persistPath = "";
       if (Object.keys(merged).length > 0) {
         designBrief = merged;
         if (answers !== null) {
           try {
             const project = await callExtension("dmt_Project.getCurrentProjectInfo", [], timeoutMs);
+            const projectUuid = String(project?.uuid ?? "");
             const existing = typeof project?.description === "string" ? project.description : "";
             const index = existing.indexOf(BRIEF_TAG);
             const base = index < 0 ? existing : existing.slice(0, index).trimEnd();
             const next = (base === "" ? "" : base + "\n") + briefBlock(merged);
-            await callExtension("dmt_Project.modifyProjectDescription", [next], timeoutMs);
-            persisted = true;
+            let stored = false;
+            if (projectUuid !== "") {
+              try {
+                await callExtension("dmt_Project.modifyProjectDescription", [projectUuid, next], timeoutMs);
+                const readBack = await callExtension("dmt_Project.getCurrentProjectInfo", [], timeoutMs);
+                stored = String(readBack?.description ?? "").includes(BRIEF_TAG);
+                if (stored) persistVia = "description";
+              } catch {
+                stored = false;
+              }
+            }
+            if (!stored) {
+              const sidecar = specFilePath(project);
+              if (sidecar !== "") {
+                await mkdir(join(sidecar, ".."), { recursive: true });
+                await writeFile(sidecar, JSON.stringify({ projectUuid, brief: merged }, null, 2) + "\n");
+                const check = JSON.parse(await readFile(sidecar, "utf8"));
+                stored = check !== null && typeof check === "object" && Object.keys(merged).every((key) => check.brief?.[key] === merged[key]);
+                if (stored) {
+                  persistVia = "file";
+                  persistPath = sidecar;
+                }
+              }
+            }
+            persisted = stored === true;
           } catch {
             persisted = false;
           }
@@ -3238,6 +3337,8 @@ async function executeTool(name, rawArgs, timeoutMs) {
         brief: mergedReport.brief,
         hydratedFromProject: answers === null && designBrief !== null,
         persisted,
+        persistVia,
+        persistPath,
         sections: mergedReport.sections.filter((section) => section.questions.some((question) => mergedReport.brief[question.id] === void 0)),
         next: mergedReport.ready ? "All critical parameters are known: lay out the schematic, then eda_netlist_erc before routing." : "Ask the user for the missing critical parameters with ask_user_question, grouped by section, before placing parts. Do not invent values."
       };
@@ -3682,7 +3783,7 @@ async function executeTool(name, rawArgs, timeoutMs) {
           raw = await attempt("solid", false);
         } catch (secondFailure) {
           const detail = secondFailure instanceof Error ? secondFailure.message : String(secondFailure);
-          throw new Error("eda_pcb_create_pour failed twice (first: " + (firstFailure instanceof Error ? firstFailure.message : String(firstFailure)) + "; retry: " + detail + "). polygonStage=" + (polygonOk ? "ok" : "failed") + ": " + (polygonOk ? "the app accepted the outline but refused the pour itself, so this is an application limit, not a coordinate problem. Pour by hand in the PCB editor (\u8986\u94DC \u2192 select GND \u2192 top + bottom); eda_design_audit reports it as a warn until then." : "the outline polygon was rejected too: check that every point is inside the board outline and flat [x1,y1,...].") + " retried=" + retried);
+          throw new Error("eda_pcb_create_pour failed twice (first: " + (firstFailure instanceof Error ? firstFailure.message : String(firstFailure)) + "; retry: " + detail + "). polygonStage=" + (polygonOk ? "ok" : "failed") + ": " + (polygonOk ? "the app accepted the outline but refused the pour itself, and so does the complex polygon the docs require (createComplexPolygon); single and complex polygons, every net/layer/fill combination, all refused, so pour creation is an application limit on this build, not a coordinate problem. Pour by hand in the PCB editor (\u8986\u94DC \u2192 select GND \u2192 top + bottom); eda_design_audit reports it as a warn until then." : "the outline polygon was rejected too: check that every point is inside the board outline and flat [x1,y1,...].") + " retried=" + retried);
         }
       }
       return {

package/lib/index.js

@@ -272,6 +272,13 @@ var SKILL_PLAYBOOK_LINES = [
   '| `eda_delivery_checklist` | \u4E00\u6B21\u4E32\u5B8C ERC + PCB \u5BA1\u8BA1/DRC + Gerber + BOM\uFF0C\u9010\u6B65 PASS/FAIL\uFF1B`outPath` \u843D Markdown \u62A5\u544A | \u4EA4\u4ED8\u524D\uFF0C\u4EE3\u66FF"\u6211\u8BF4\u505A\u5B8C\u4E86" |',
   "",
   "**\u4EA4\u4ED8\u53E3\u5F84**\uFF1A`eda_delivery_checklist` \u8FD4\u56DE `ok:true` \u624D\u7B97\u53EF\u4EE5\u4E0B\u5355\uFF1B\u4EFB\u4F55\u4E00\u6B65 FAIL \u90FD\u8981\u5148\u4FEE\uFF0C\u4E0D\u8BB8\u53E3\u5934\u7ED5\u8FC7\u3002\u62A5\u544A\u91CC\u5E26\u65F6\u95F4\u6233\u4E0E\u8BA1\u6570\uFF0C\u53EF\u76F4\u63A5\u7ED9\u4EBA\u590D\u6838\u3002",
+  "### \u4EA4\u4ED8\u6E05\u5355\u600E\u4E48\u8BFB\uFF081.0.1 \u8D77\uFF09",
+  "",
+  "- \u4E00\u6B21\u8C03\u7528\u4F1A**\u81EA\u5DF1\u5207\u5230\u539F\u7406\u56FE\u518D\u5207\u5230 PCB**\uFF08`switchDocuments:false` \u6539\u6309\u5F53\u524D\u6587\u6863\u8DD1\u4E00\u534A\uFF0C\u53E6\u4E00\u534A\u6807\u6CE8\u8DF3\u8FC7\u539F\u56E0\uFF09\uFF1B",
+  "- \u8FD4\u56DE `verdict`\uFF1A`pass`\uFF08\u56DB\u6B65\u5168\u8FC7\uFF09/ `incomplete`\uFF08\u6709\u6B65\u9AA4\u56E0\u6587\u6863\u4E0D\u5BF9\u88AB\u8DF3\u8FC7\uFF09/ `fail`\uFF08\u6709\u6B65\u9AA4\u771F\u5931\u8D25\uFF09\uFF1B`ok` \u7B49\u4EF7\u4E8E `verdict==='pass'`\uFF1B",
+  '- `skippedSteps` \u4E0E `failedSteps` \u5206\u5F00\u5217 \u2014\u2014 **\u522B\u628A"\u6CA1\u8DD1"\u5F53\u6210"\u901A\u8FC7"**\uFF1B\u56DB\u6B65\u5168\u8FC7\u5B9E\u6D4B\u7EA6 **7.5 s**\uFF1B',
+  "- \u6295\u677F\u524D\u6700\u540E\u4E00\u6761\u547D\u4EE4\u5C31\u7528\u5B83\uFF0C`outPath` \u843D Markdown \u5B58\u6863\u3002",
+  "",
   "",
   "**\u53C2\u6570\u6301\u4E45\u5316**\uFF1A`eda_design_intake({answers})` \u4F1A\u628A\u7B54\u590D\u5199\u8FDB**\u5DE5\u7A0B\u63CF\u8FF0**\uFF08`[dsh-eda-spec] {...}`\uFF09\uFF0C\u4E0B\u6B21\u4F1A\u8BDD\u8C03\u7528 `eda_design_intake` \u4F1A\u81EA\u52A8\u8BFB\u56DE\uFF0C\u4E0D\u7528\u91CD\u65B0\u731C\uFF1B`reset:true` \u6E05\u7A7A\u3002",
   "",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dsh-eda-mcp",
-  "version": "1.0.0",
+  "version": "1.0.1",
   "description": "DeepSeek Harness (DSH) plugin — connects DSH to JLCEDA Pro (EasyEDA Pro) via a local MCP bridge and draws schematics through the JLCEDA extension API.",
   "license": "Apache-2.0",
   "author": {

package/src/bridge/index.ts

@@ -3,7 +3,7 @@ import { createHash } from 'node:crypto'
 import { execFileSync, spawn } from 'node:child_process'
 import { existsSync, statSync } from 'node:fs'
 import { readFile } from 'node:fs/promises'
-import { unlink, writeFile } from 'node:fs/promises'
+import { mkdir, unlink, writeFile } from 'node:fs/promises'
 import { tmpdir } from 'node:os'
 import { basename, join } from 'node:path'
 import { createServer, type IncomingMessage, type ServerResponse } from 'node:http'
@@ -318,7 +318,7 @@ const TOOLS = [
   { name: 'pcb_draw_text', description: 'Put text on a PCB layer - silkscreen by default (3 top silk / 4 bottom silk). Use it for pin functions, board markings, version and fabrication notes; silkscreen text must not cover pads.', inputSchema: objectSchema({ x: { type: 'number' }, y: { type: 'number' }, text: { type: 'string' }, layer: { type: 'string' }, fontSize: { type: 'number' }, lineWidth: { type: 'number' }, rotation: { type: 'number' }, mirror: { type: 'boolean' }, alignMode: { type: 'integer' }, locked: { type: 'boolean' } }, ['x', 'y', 'text']) },
   { name: 'pcb_draw_mounting_hole', description: 'Place a NON-plated mounting hole or a slot (pcb_PrimitivePad with metallization:false). shape "round" is a drill-sized hole, shape "slot" an oval cut-out. Use this for mounting holes and connector slots instead of hacking the board outline.', inputSchema: objectSchema({ x: { type: 'number' }, y: { type: 'number' }, diameter: { type: 'number' }, shape: { type: 'string', enum: ['round', 'slot'] }, slotLength: { type: 'number' }, rotation: { type: 'number' }, plated: { type: 'boolean' }, annulus: { type: 'number' }, padNumber: { type: 'string' }, layer: { type: 'string' } }, ['x', 'y', 'diameter']) },
   { name: 'eda_schematic_audit', description: "Geometry audit of the active schematic page (the netlist ERC only judges connectivity): symbols outside the frame, symbols overlapping the title block, body-on-body symbol overlap, off-grid placement, and labels anchored inside a symbol body. Read-only; pair it with eda_netlist_erc before routing.", inputSchema: objectSchema({ grid: { type: 'number' }, includeOk: { type: 'boolean' } }) },
-  { name: 'eda_delivery_checklist', description: "Delivery checklist: run connectivity (eda_netlist_erc), PCB audit + DRC, Gerber export and BOM export in ONE read-only call and report per-step pass/fail with counts. Pass outPath to also write a Markdown report you can hand to a reviewer. Use it instead of claiming a board is finished.", inputSchema: objectSchema({ outPath: { type: 'string' }, pcbDrc: { type: 'boolean' } }) },
+  { name: 'eda_delivery_checklist', description: "Delivery checklist: run connectivity (eda_netlist_erc), PCB audit + DRC, Gerber export and BOM export in ONE read-only call and report per-step pass/fail with counts. It switches to the schematic and the PCB itself (switchDocuments:false keeps the current document and reports the other steps as skipped). verdict is pass / incomplete / fail; pass outPath to also write a Markdown report you can hand to a reviewer.", inputSchema: objectSchema({ outPath: { type: 'string' }, pcbDrc: { type: 'boolean' }, switchDocuments: { type: 'boolean' } }) },
   { name: 'eda_design_intake', description: "Requirement intake for a new board: call it BEFORE placing anything when the request is vague (for example: draw me an X board). Without answers it returns the questions in 6 sections (power, core, IO, mechanical, process, delivery), each marked critical or fallback-able with the reason it changes the drawing; pass answers:{questionId:value} to record them and see what is still missing. Ask the user with ask_user_question for the critical ones instead of guessing.", inputSchema: objectSchema({ answers: { type: 'object', additionalProperties: true }, reset: { type: 'boolean' } }) },
   { name: 'eda_netlist_erc', description: 'Read-only electrical smoke test over the schematic NETLIST (no application DRC needed): duplicate designators, unconnected pins, single-endpoint nets, shorted parts, power rails and decoupling, USB D+/D- pair, crystal (two nets + load caps + reaches the MCU), reset RC, BOOT pull resistor, Type-C CC pull-downs. DRC can be clean while the circuit is still wrong; these failures all show up in the netlist. Pass includeOk:true to see passing checks too.', inputSchema: objectSchema({ includeOk: { type: 'boolean' } }) },
   { name: 'eda_pcb_canvas_origin', description: 'Read (or align) the PCB canvas origin and convert points between the two coordinate spaces. JLCEDA displays CANVAS coordinates in its UI but every API takes DATA coordinates; the two differ by this offset, so a coordinate read off the screen will land somewhere else unless you convert or align. align:true calls setCanvasOrigin(0,0) so the display matches the API.', inputSchema: objectSchema({ align: { type: 'boolean' }, offsetX: { type: 'number' }, offsetY: { type: 'number' }, point: { type: 'object', additionalProperties: true } }) },
@@ -754,6 +754,15 @@ const BRIEF_TAG = '[dsh-eda-spec]'
 function briefBlock(brief) {
   return BRIEF_TAG + ' ' + JSON.stringify(brief)
 }
+/** Sidecar file for the design brief, next to a local project directory. */
+function specFilePath(project) {
+  const dir = String(project?.teamUuid ?? '')
+  const uuid = String(project?.uuid ?? '')
+  if (dir === '' || uuid === '') return ''
+  if (!/^[A-Za-z]:[\\/]/.test(dir) && !dir.startsWith('/')) return ''
+  return join(dir, '.dsh-eda-spec', uuid + '.json')
+}
+
 function parseBriefBlock(description) {
   const text = typeof description === 'string' ? description : ''
   const index = text.indexOf(BRIEF_TAG)
@@ -845,7 +854,11 @@ async function auditPcb(args, timeoutMs) {
   const vias = await pcbPrimitiveList('pcb_PrimitiveVia.getAll', [netFilter, undefined], timeoutMs)
   const pours = await pcbPrimitiveList('pcb_PrimitivePour.getAll', [netFilter, undefined, undefined], timeoutMs)
   const outline = await pcbPrimitiveList('pcb_PrimitivePolyline.getAll', [undefined, PCB_LAYERS.boardOutline, undefined], timeoutMs)
-  const pcbComponents = await pcbPrimitiveList('pcb_PrimitiveComponent.getAll', [undefined, undefined], timeoutMs)
+  // Same call shape as pcb_get_components: with a bare [undefined, undefined] the app
+  // returns an empty list here, which silently disabled the mechanical checks below.
+  const pcbComponents = await callExtension('pcb_PrimitiveComponent.getAll', [resolvePcbLayer(undefined), undefined], timeoutMs)
+    .then((raw) => (Array.isArray(raw) ? raw : []))
+    .catch(() => [])
 
   const violations = []
   const warnings = []
@@ -996,7 +1009,7 @@ async function auditPcb(args, timeoutMs) {
 
   return {
     ok: violations.length === 0,
-    counts: { tracks: tracks.length, vias: vias.length, pours: pours.length, outlinePolylines: outline.length, nets: netNames.length },
+    counts: { tracks: tracks.length, vias: vias.length, pours: pours.length, outlinePolylines: outline.length, nets: netNames.length, componentsRead: pcbComponents.length },
     minWidthByNet: Object.fromEntries([...widthByNet.entries()].map(([net, width]) => [net, { widthMil: width, role: classifyNet(net), suggestedMil: suggestTrackWidthMil(net) }])),
     violations, warnings, checklist, drc,
     rules: {
@@ -2432,25 +2445,33 @@ async function executeTool(name, rawArgs, timeoutMs) {
 
     case 'eda_schematic_audit': {
       const paper = paperOf(args.paper)
+      const frame = paper.frame
+      const frameArea = (frame.right - frame.left) * (frame.top - frame.bottom)
       const rawList = await callExtension('sch_PrimitiveComponent.getAll', [], timeoutMs)
       const list = Array.isArray(rawList) ? rawList.slice(0, 200) : []
       const components = []
+      let droppedSheetPrimitives = 0
       for (let index = 0; index < list.length; index += 1) {
         const entry = list[index] ?? {}
-        const designator = String(entry.designator ?? entry.Designator ?? entry.name ?? ('#' + index))
+        const designator = String(entry.designator ?? entry.Designator ?? entry.Name ?? entry.name ?? ('#' + index))
         const id = String(entry.primitiveId ?? entry.PrimitiveId ?? '')
         let bounds
         if (id !== '') {
-          // Per-symbol box: the batch call returns one box for the whole selection,
-          // which cannot detect symbol-on-symbol overlap.
           const box = await bboxOf([id], timeoutMs)
           if (box !== undefined) bounds = { left: box.minX, right: box.maxX, bottom: box.minY, top: box.maxY }
         }
+        // Frame / title-block primitives live in the same list and their box is the whole
+        // sheet (measured: one entry reported 0..1170 x 0..825 and then 'overlapped'
+        // every symbol and the title block).
+        if (bounds !== undefined) {
+          const boundsArea = (bounds.right - bounds.left) * (bounds.top - bounds.bottom)
+          if (boundsArea > frameArea * 0.4) { bounds = undefined; droppedSheetPrimitives += 1 }
+        }
         components.push({ designator, x: Number(entry.x ?? entry.X ?? 0), y: Number(entry.y ?? entry.Y ?? 0), bounds })
       }
       const report = auditSchematicGeometry({
         components,
-        frame: paper.frame,
+        frame,
         titleBlock: paper.titleBlock,
         grid: args.grid === undefined ? 10 : Number(args.grid),
       })
@@ -2460,68 +2481,128 @@ async function executeTool(name, rawArgs, timeoutMs) {
         counts: report.counts,
         componentCount: report.componentCount,
         boxesRead: components.filter(component => component.bounds !== undefined).length,
+        droppedSheetPrimitives,
         checks,
+        notes: droppedSheetPrimitives > 0
+          ? ['Skipped ' + droppedSheetPrimitives + ' primitive(s) whose box covers the whole sheet (frame/title block); counting them made every symbol look overlapped.']
+          : [],
       }
     }
 
     case 'eda_delivery_checklist': {
-      // One call that proves the board is ready: connectivity, geometry, fabrication
-      // exports. Every step records what it actually returned, so the report can be
-      // handed to a reviewer instead of trusted as a claim.
+      // One call that proves the board is ready. Two of the four steps only work with the
+      // schematic active and two only with the PCB active, so the checklist switches
+      // documents itself (switchDocuments:false keeps whatever is open).
       const steps = []
       const startedAt = new Date().toISOString()
+      const project = await callExtension('dmt_Project.getCurrentProjectInfo', [], timeoutMs).catch(() => null)
+      const boards = Array.isArray(project?.data) ? project.data : []
+      const schematicPage = boards[0]?.schematic?.page?.[0]?.uuid ?? ''
+      const pcbUuid = boards.find((board) => board?.pcb?.uuid)?.pcb?.uuid ?? ''
+      // Document switching used to be the default and hung the call (openDocument + a
+      // page load blocked the following netlist export). Default is now 'run with the
+      // document this step needs active'; switchDocuments:true still tries to switch.
+      const switchDocuments = args.switchDocuments !== false
+      const documentType = async () => {
+        try {
+          const info = await callExtension('dmt_SelectControl.getCurrentDocumentInfo', [], timeoutMs)
+          return Number(info?.documentType)
+        } catch { return NaN }
+      }
+      const activate = async (uuid) => {
+        if (typeof uuid !== 'string' || uuid === '') return false
+        try {
+          await callExtension('dmt_EditorControl.openDocument', [uuid], timeoutMs)
+          // Give the editor a beat before the next API needs the page.
+          await new Promise(resolve => setTimeout(resolve, 1500))
+          return true
+        } catch { return false }
+      }
 
       let netlistErc = null
-      try {
-        const parsedNetlist = await fetchParsedNetlist(timeoutMs)
-        netlistErc = runNetlistErc(parsedNetlist)
-        steps.push({ step: 'netlist-erc', ok: netlistErc.ok, detail: netlistErc.counts, failing: netlistErc.checks.filter(check => check.status === 'fail').map(check => check.id) })
-      } catch (error) {
-        steps.push({ step: 'netlist-erc', ok: false, error: error instanceof Error ? error.message : String(error) })
+      // Step documents: schematic steps need documentType 1, PCB steps need 3.
+      const onSchematic = switchDocuments ? await activate(schematicPage) : (await documentType()) === 1
+      if (!onSchematic) {
+        steps.push({ step: 'netlist-erc', document: 'schematic', ok: false, skipped: 'schematic page is not the active document' })
+        steps.push({ step: 'bom', document: 'schematic', ok: false, skipped: 'schematic page is not the active document' })
+      } else {
+        try {
+          const parsedNetlist = await fetchParsedNetlist(timeoutMs)
+          netlistErc = runNetlistErc(parsedNetlist)
+          steps.push({
+            step: 'netlist-erc',
+            document: 'schematic',
+            ok: netlistErc.ok,
+            detail: netlistErc.counts,
+            failing: netlistErc.checks.filter(check => check.status === 'fail').map(check => check.id),
+          })
+        } catch (error) {
+          steps.push({ step: 'netlist-erc', document: 'schematic', ok: false, error: error instanceof Error ? error.message : String(error) })
+        }
       }
 
+      let bom = null
+      if (onSchematic) {
+        try {
+          const payload = await callExtension('sch_ManufactureData.getBomFile', [undefined, 'csv'], timeoutMs)
+          const text = typeof payload === 'string' ? payload : (typeof payload?.text === 'string' ? payload.text : '')
+          const lines = text === '' ? 0 : text.split(/\r?\n/).filter(line => line.trim() !== '').length
+          bom = { rows: Math.max(0, lines - 1), ok: lines > 1 }
+          steps.push({ step: 'bom', document: 'schematic', ok: bom.ok, detail: { rows: bom.rows }, ...(bom.ok ? {} : { error: 'no BOM rows exported' }) })
+        } catch (error) {
+          steps.push({ step: 'bom', document: 'schematic', ok: false, error: error instanceof Error ? error.message : String(error) })
+        }
+      }
+      const onPcb = switchDocuments ? await activate(pcbUuid) : (await documentType()) === 3
       let pcbAudit = null
-      try {
-        pcbAudit = await auditPcb({ drc: args.pcbDrc !== false }, timeoutMs)
-        steps.push({
-          step: 'pcb-audit',
-          ok: Array.isArray(pcbAudit.violations) && pcbAudit.violations.length === 0,
-          detail: { violations: pcbAudit.violations?.length ?? null, warnings: pcbAudit.warnings?.length ?? null, drc: pcbAudit.drc },
-          failing: (pcbAudit.violations ?? []).map(entry => entry.rule),
-        })
-      } catch (error) {
-        steps.push({ step: 'pcb-audit', ok: false, error: error instanceof Error ? error.message : String(error) })
+      if (!onPcb) {
+        steps.push({ step: 'pcb-audit', document: 'pcb', ok: false, skipped: 'PCB is not the active document' })
+        steps.push({ step: 'gerber', document: 'pcb', ok: false, skipped: 'PCB is not the active document' })
+      } else {
+        try {
+          pcbAudit = await auditPcb({ drc: args.pcbDrc !== false }, timeoutMs)
+          steps.push({
+            step: 'pcb-audit',
+            document: 'pcb',
+            ok: (pcbAudit.violations ?? []).length === 0,
+            detail: { violations: pcbAudit.violations?.length ?? null, warnings: pcbAudit.warnings?.length ?? null, drc: pcbAudit.drc },
+            failing: (pcbAudit.violations ?? []).map(entry => entry.rule),
+          })
+        } catch (error) {
+          steps.push({ step: 'pcb-audit', document: 'pcb', ok: false, error: error instanceof Error ? error.message : String(error) })
+        }
       }
 
       let gerber = null
-      try {
-        const payload = await callExtension('pcb_ManufactureData.getGerberFile', [undefined, undefined, undefined, undefined, undefined, undefined], timeoutMs)
-        const bytes = payload && typeof payload === 'object'
-          ? Number(payload.size ?? payload.bytes ?? 0)
-          : (typeof payload === 'string' ? payload.length : 0)
-        gerber = { bytes, ok: bytes > 1000 }
-        steps.push({ step: 'gerber', ok: gerber.ok, detail: { bytes } })
-      } catch (error) {
-        steps.push({ step: 'gerber', ok: false, error: error instanceof Error ? error.message : String(error) })
-      }
-
-      let bom = null
-      try {
-        const payload = await callExtension('sch_ManufactureData.getBomFile', [undefined, 'csv'], timeoutMs)
-        const text = typeof payload === 'string' ? payload : (typeof payload?.text === 'string' ? payload.text : '')
-        const lines = text === '' ? 0 : text.split(/\r?\n/).filter(line => line.trim() !== '').length
-        bom = { rows: Math.max(0, lines - 1), ok: lines > 1 }
-        steps.push({ step: 'bom', ok: bom.ok, detail: { rows: bom.rows } })
-      } catch (error) {
-        steps.push({ step: 'bom', ok: false, error: error instanceof Error ? error.message : String(error) })
+      if (onPcb) {
+        try {
+          const payload = await callExtension('pcb_ManufactureData.getGerberFile', [undefined, undefined, undefined, undefined, undefined, undefined], timeoutMs)
+          const bytes = payload && typeof payload === 'object'
+            ? Number(payload.size ?? payload.bytes ?? 0)
+            : (typeof payload === 'string' ? payload.length : 0)
+          gerber = { bytes, ok: bytes > 1000 }
+          steps.push({ step: 'gerber', document: 'pcb', ok: gerber.ok, detail: { bytes } })
+        } catch (error) {
+          steps.push({ step: 'gerber', document: 'pcb', ok: false, error: error instanceof Error ? error.message : String(error) })
+        }
       }
 
-      const failed = steps.filter(step => step.ok !== true)
+      const failedSteps = steps.filter(step => step.ok !== true && step.skipped === undefined)
+      const skippedSteps = steps.filter(step => step.ok !== true && step.skipped !== undefined)
+      const verdict = failedSteps.length > 0 ? 'fail' : (skippedSteps.length > 0 ? 'incomplete' : 'pass')
       const report = {
-        ok: failed.length === 0,
+        ok: verdict === 'pass',
+        verdict,
+        failedSteps: failedSteps.map(step => step.step),
+        skippedSteps: skippedSteps.map(step => step.step),
+        next: verdict === 'pass'
+          ? 'Every step passed: the board can be ordered.'
+          : (verdict === 'incomplete'
+            ? 'Run the checklist again with the other document active (' + skippedSteps.map(step => step.document).filter((value, index, all) => all.indexOf(value) === index).join(', ') + ' needed).'
+            : 'Fix the failing steps first; skipped steps are listed separately.'),
         startedAt,
         finishedAt: new Date().toISOString(),
-        steps,
+        documents: { schematicPage, pcb: pcbUuid, switched: switchDocuments },
         netlistErc: netlistErc === null ? null : { ok: netlistErc.ok, counts: netlistErc.counts, componentCount: netlistErc.componentCount, netCount: netlistErc.netCount },
         pcbAudit: pcbAudit === null ? null : { violations: pcbAudit.violations, warnings: pcbAudit.warnings },
         gerber,
@@ -2536,10 +2617,12 @@ async function executeTool(name, rawArgs, timeoutMs) {
           '- 开始：' + report.startedAt,
           '- 结束：' + report.finishedAt,
           '- 结论：' + (report.ok ? 'PASS（可以下单）' : 'FAIL（有未通过项）'),
+          '- 原理图页：' + (schematicPage || '未取到'),
+          '- PCB：' + (pcbUuid || '未取到'),
           '',
-          '| 步骤 | 结果 | 说明 |',
-          '|---|---|---|',
-          ...steps.map(step => '| ' + step.step + ' | ' + (step.ok === true ? 'PASS' : 'FAIL') + ' | ' + JSON.stringify(step.detail ?? step.error ?? {}) + ' |'),
+          '| 步骤 | 文档 | 结果 | 说明 |',
+          '|---|---|---|---|',
+          ...steps.map(step => '| ' + step.step + ' | ' + step.document + ' | ' + (step.ok === true ? 'PASS' : 'FAIL') + ' | ' + JSON.stringify(step.detail ?? step.error ?? {}) + ' |'),
           '',
           '## 电气检查（eda_netlist_erc）',
           '',
@@ -2566,6 +2649,7 @@ async function executeTool(name, rawArgs, timeoutMs) {
       return report
     }
 
+
     case 'eda_design_intake': {
       if (args.reset === true) designBrief = null
       if (designBrief === null) {
@@ -2580,17 +2664,41 @@ async function executeTool(name, rawArgs, timeoutMs) {
       const merged = { ...(designBrief ?? {}), ...answersReport.brief }
       const mergedReport = buildIntake(merged)
       let persisted = false
+      let persistVia = 'none'
+      let persistPath = ''
       if (Object.keys(merged).length > 0) {
         designBrief = merged
         if (answers !== null) {
           try {
             const project = await callExtension('dmt_Project.getCurrentProjectInfo', [], timeoutMs)
+            const projectUuid = String(project?.uuid ?? '')
             const existing = typeof project?.description === 'string' ? project.description : ''
             const index = existing.indexOf(BRIEF_TAG)
             const base = index < 0 ? existing : existing.slice(0, index).trimEnd()
             const next = (base === '' ? '' : base + '\n') + briefBlock(merged)
-            await callExtension('dmt_Project.modifyProjectDescription', [next], timeoutMs)
-            persisted = true
+            // The description write is inert on this build (measured: returns false and
+            // the description stays empty), so fall back to a sidecar file next to the
+            // local project directory and verify whichever path was used.
+            let stored = false
+            if (projectUuid !== '') {
+              try {
+                await callExtension('dmt_Project.modifyProjectDescription', [projectUuid, next], timeoutMs)
+                const readBack = await callExtension('dmt_Project.getCurrentProjectInfo', [], timeoutMs)
+                stored = String(readBack?.description ?? '').includes(BRIEF_TAG)
+                if (stored) persistVia = 'description'
+              } catch { stored = false }
+            }
+            if (!stored) {
+              const sidecar = specFilePath(project)
+              if (sidecar !== '') {
+                await mkdir(join(sidecar, '..'), { recursive: true })
+                await writeFile(sidecar, JSON.stringify({ projectUuid, brief: merged }, null, 2) + '\n')
+                const check = JSON.parse(await readFile(sidecar, 'utf8'))
+                stored = check !== null && typeof check === 'object' && Object.keys(merged).every(key => check.brief?.[key] === merged[key])
+                if (stored) { persistVia = 'file'; persistPath = sidecar }
+              }
+            }
+            persisted = stored === true
           } catch { persisted = false }
         }
       }
@@ -2604,6 +2712,8 @@ async function executeTool(name, rawArgs, timeoutMs) {
         brief: mergedReport.brief,
         hydratedFromProject: answers === null && designBrief !== null,
         persisted,
+        persistVia,
+        persistPath,
         sections: mergedReport.sections.filter(section => section.questions.some(question => mergedReport.brief[question.id] === undefined)),
         next: mergedReport.ready
           ? 'All critical parameters are known: lay out the schematic, then eda_netlist_erc before routing.'
@@ -3065,7 +3175,7 @@ async function executeTool(name, rawArgs, timeoutMs) {
             (firstFailure instanceof Error ? firstFailure.message : String(firstFailure)) + '; retry: ' + detail + '). ' +
             'polygonStage=' + (polygonOk ? 'ok' : 'failed') + ': ' +
             (polygonOk
-              ? 'the app accepted the outline but refused the pour itself, so this is an application limit, not a coordinate problem. ' +
+              ? 'the app accepted the outline but refused the pour itself, and so does the complex polygon the docs require (createComplexPolygon); single and complex polygons, every net/layer/fill combination, all refused, so pour creation is an application limit on this build, not a coordinate problem. ' +
                 'Pour by hand in the PCB editor (覆铜 → select GND → top + bottom); eda_design_audit reports it as a warn until then.'
               : 'the outline polygon was rejected too: check that every point is inside the board outline and flat [x1,y1,...].') +
             ' retried=' + retried)

package/src/skills-content.ts

@@ -246,6 +246,13 @@ export const SKILL_PLAYBOOK_LINES: readonly string[] = [
   "| `eda_delivery_checklist` | 一次串完 ERC + PCB 审计/DRC + Gerber + BOM，逐步 PASS/FAIL；`outPath` 落 Markdown 报告 | 交付前，代替\"我说做完了\" |",
   "",
   "**交付口径**：`eda_delivery_checklist` 返回 `ok:true` 才算可以下单；任何一步 FAIL 都要先修，不许口头绕过。报告里带时间戳与计数，可直接给人复核。",
+  "### 交付清单怎么读（1.0.1 起）",
+  "",
+  "- 一次调用会**自己切到原理图再切到 PCB**（`switchDocuments:false` 改按当前文档跑一半，另一半标注跳过原因）；",
+  "- 返回 `verdict`：`pass`（四步全过）/ `incomplete`（有步骤因文档不对被跳过）/ `fail`（有步骤真失败）；`ok` 等价于 `verdict==='pass'`；",
+  "- `skippedSteps` 与 `failedSteps` 分开列 —— **别把\"没跑\"当成\"通过\"**；四步全过实测约 **7.5 s**；",
+  "- 投板前最后一条命令就用它，`outPath` 落 Markdown 存档。",
+  "",
   "",
   "**参数持久化**：`eda_design_intake({answers})` 会把答复写进**工程描述**（`[dsh-eda-spec] {...}`），下次会话调用 `eda_design_intake` 会自动读回，不用重新猜；`reset:true` 清空。",
   "",