dsh-eda-mcp 1.0.0 → 1.0.2

package/CHANGELOG.md

@@ -1,5 +1,67 @@
 # Changelog
 
+## 1.0.2 — 2026-10-20
+
+### 覆铜调查结案（扩展 0.4.7 / 0.4.8 真机）
+
+结论：**这台构建（EDA 3.2.186 / pro-api 0.3.18.0bc12bb0）无法通过 API 创建铜面图元**。不是参数问题，证据链如下。
+
+| 尝试 | 结果 |
+|---|---|
+| `pcb_PrimitivePour.create` 静态入口 × 8 种组合（单/复杂多边形、原始源码数组、空网络、层 1/2/字符串、全参数显式、100×100 与 300×300 mil） | 全部 `无法创建覆铜边框图元` |
+| 实例入口 `new IPCB_PrimitivePour(...).create()`（0.4.7 新增，验证构造器在运行时可取） | 同样报错 |
+| 填充方式 `solid`/`45grid`/`90grid`/`0`/`1`/`2`/`SOLID`/`GRID45`/`45GRID` | 全部报错 |
+| `pcb_PrimitiveFill.create`（文档说明它同样带 `net` 参数）× 4 种（对象/原始数组/复杂源码/显式 fillMode） | 全部 `无法创建填充图元` |
+| **控制组**：同一多边形管线写丝印折线 `pcb_PrimitivePolyline` / `pcb_draw_polyline` | 也写不出来（`getAll` 0 条），而 `pcb_get_polylines` 内部路径能读到既有板框 1 条 |
+
+外部资料也到此为止：`prodocs` 正文是前端渲染（HTML 只有导航），官方配方给的流程就是我们实现的那条（`createPolygon`/`createComplexPolygon` → pour → `rebuildCopperRegion`），搜 "无法创建覆铜边框图元" 无任何社区解法，应用 bundle 内也搜不到该字符串。
+
+**因此撤掉 `pcb_create_copper_fill`**：它在这台构建上必然失败，留着就是给调用方挖坑。白名单里的 `pcb_MathPolygon.createComplexPolygon` 与 `pcb_PrimitiveFill.*` 保留（换构建可能有效），但不再包装成工具。
+
+**正解**：覆铜在界面手工铺（GND → 顶层+底层 → 重新铺铜）；`eda_design_audit` 会持续把缺覆铜记成 `copper-pour` warn，`eda_delivery_checklist` 也会带上它。
+
+
+### 通知堆叠 / 粘连（扩展 0.4.9）
+
+屏幕实测：三条提示叠在画布上（"未连接…点击右侧按钮"、"扩展已加载 v0.4.8"、"已连接；端口 39009"），且"未连接"那条永不消失。
+
+- 根因：`showToastMessage` 的 `timer: 0` 语义是**永不自动隐藏**，而 `SYS_Message` **没有任何隐藏或替换 API**；连接成功时也没有人撤掉它。
+- 修法：未连接提示改成 `showFollowMouseTip(tip, 8000)`，并在 WebSocket `onopen` 里 `removeFollowMouseTip` 显式撤掉；`showToast` 一律传有限时长（默认 4 s）；文案改为"菜单 DSH MCP → 连接 DSH MCP 服务器"（跟随提示没有按钮）。
+- 线协议 **0.4.8 → 0.4.9**（纯 UI 层）。
+
+## 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

@@ -146,7 +146,7 @@
 | eda_launch 带 project 没打开工程 | 命令行路径被应用忽略 | 用 eda_open_project({ name }) 打开 |
 | 关应用后改动丢失 | eda_close 走了强制路径 | 关前先保存；看返回的 forced |
 | run_code 读大 JSON 被截断 | 宿主 read 有单行上限 | 用 node:fs 读盘或分片读 |
-| 覆铜创建失败（无法创建覆铜边框图元） | 应用侧限制：polygonStage=ok（轮廓被接受）但 pours 被拒，与网络/填充方式/坐标无关 | 工具会重试一次并给出该诊断；手工在 PCB 页框选覆铜（GND，顶层+底层），design_audit 记为 warn |
+| 覆铜/填充一律被拒（无法创建覆铜边框图元 / 无法创建填充图元） | 应用侧限制：静态+实例入口、单/复杂多边形、原始源码数组、6 种填充枚举、全参数组合全部被拒；同一多边形管线在本构建上连丝印折线都写不出来 | 不要在 API 层反复试：覆铜请在界面手工铺（GND、顶层+底层）；design_audit 持续记 copper-pour warn |
 | pcb_draw_mounting_hole 报“无法创建焊盘图元” | pad 形状必填，且槽孔只能用 ELLIPSE | 已内建：圆孔 ELLIPSE(d,d)，槽孔 ELLIPSE(长,宽)+hole[SLOT,宽,长]；OBLONG/RECTANGLE 焊盘会被拒 |
 | pcb_PrimitiveString.getAll / pcb_PrimitivePad.getAll 返回 0 | 这两个 API 按层查询 | 传 layer：丝印 3/4，多层焊盘 12；不传参恒为 0 |
 | 按坐标算的越界没被发现 | 图框/原点假设与真实页面不一致 | 用 eda_get_page_info 的真实值 + eda_capture 导 PNG 看实际版面 |
@@ -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/jlc-extension/api-bridge.js

@@ -212,6 +212,9 @@ function dshEdaMcpStatus() {
     'pcb_PrimitivePad.create',
     'pcb_PrimitivePad.getAll',
     'pcb_PrimitivePad.delete',
+    'pcb_PrimitiveFill.create',
+    'pcb_PrimitiveFill.getAll',
+    'pcb_PrimitiveFill.delete',
     'pcb_PrimitivePour.create',
     'pcb_PrimitivePour.modify',
     'pcb_PrimitivePour.delete',
@@ -459,21 +462,54 @@ function dshEdaMcpStatus() {
         : undefined;
       if (!pourPolygon) pourPolygon = eda.pcb_MathPolygon.createPolygon(pourSource);
       if (!pourPolygon) throw new Error('dsh.internal.pourCreate: invalid polygon source ' + JSON.stringify(pourSource));
-      var pour = await eda.pcb_PrimitivePour.create(
-        optional(pourArgs[0]), optional(pourArgs[1]), pourPolygon, optional(pourArgs[3]), optional(pourArgs[4]),
-        optional(pourArgs[5]), optional(pourArgs[6]), optional(pourArgs[7]), optional(pourArgs[8])
-      );
+      // Route 1: the documented static factory.
+      var pourRoutes = [];
+      var pour = null;
+      var usedRoute = 'static';
+      try {
+        pour = await eda.pcb_PrimitivePour.create(
+          optional(pourArgs[0]), optional(pourArgs[1]), pourPolygon, optional(pourArgs[3]), optional(pourArgs[4]),
+          optional(pourArgs[5]), optional(pourArgs[6]), optional(pourArgs[7]), optional(pourArgs[8])
+        );
+      } catch (staticError) {
+        pourRoutes.push('static: ' + (staticError && staticError.message ? staticError.message : String(staticError)));
+      }
+      // Route 2: the instance entry point (@internal in the typings, but the class
+      // constructor is exported at runtime on some builds).
       if (!pour) {
-        throw new Error('dsh.internal.pourCreate: pcb_PrimitivePour.create returned undefined (net/layer/polygon rejected; copper layers only)');
+        var PourCtor = null;
+        try {
+          if (typeof eda.IPCB_PrimitivePour === 'function') PourCtor = eda.IPCB_PrimitivePour;
+          else if (eda.pcb_PrimitivePour && typeof eda.pcb_PrimitivePour.constructor === 'function' && eda.pcb_PrimitivePour.constructor !== Object) PourCtor = eda.pcb_PrimitivePour.constructor;
+        } catch (ctorProbeError) { PourCtor = null; }
+        if (PourCtor !== null) {
+          usedRoute = 'instance';
+          try {
+            var pourInstance = new PourCtor(
+              optional(pourArgs[0]), optional(pourArgs[1]), pourPolygon, optional(pourArgs[3]), optional(pourArgs[4]),
+              optional(pourArgs[5]), optional(pourArgs[6]), optional(pourArgs[7]), optional(pourArgs[8])
+            );
+            pour = await pourInstance.create();
+          } catch (instanceError) {
+            pourRoutes.push('instance: ' + (instanceError && instanceError.message ? instanceError.message : String(instanceError)));
+            usedRoute = 'none';
+          }
+        } else {
+          pourRoutes.push('instance: no exported constructor');
+          usedRoute = 'none';
+        }
       }
-      // A pour border alone leaves no copper: the region has to be rebuilt for the
-      // fill (and the clearance against other nets) to be computed.
+      if (!pour) {
+        throw new Error('dsh.internal.pourCreate: every route refused (' + pourRoutes.join(' | ') + ')');
+      }
+      // A pour border alone leaves no copper: rebuild so the fill (and the clearance
+      // against other nets) is actually computed.
       var rebuilt = false;
       if (pour && typeof pour.rebuildCopperRegion === 'function') {
         try { await pour.rebuildCopperRegion(); rebuilt = true; } catch (error) { rebuilt = false; }
       }
       var pourJson = toJsonable(pour);
-      if (pourJson && typeof pourJson === 'object') pourJson.rebuilt = rebuilt;
+      if (pourJson && typeof pourJson === 'object') { pourJson.rebuilt = rebuilt; pourJson.route = usedRoute; }
       return pourJson;
     }
     // Internal helper: long-running jobs (auto-routing / auto-layout) are started
@@ -579,6 +615,9 @@ function dshEdaMcpStatus() {
         function () {
           wsReady = true;
           lastPongAt = Date.now();
+          // The 'not connected' tip is sticky by design while we are offline; once the
+          // socket is up it must go away, otherwise it sits on the canvas forever.
+          hideConnectTip();
           sendObject({ type: 'hello', version: VERSION });
         }
       );
@@ -599,10 +638,11 @@ function dshEdaMcpStatus() {
     registerWebSocket();
   }
 
-  function showToast(message, type) {
+  function showToast(message, type, seconds) {
     try {
       if (typeof eda !== 'undefined' && eda.sys_Message && typeof eda.sys_Message.showToastMessage === 'function') {
-        eda.sys_Message.showToastMessage(message, type || 'info');
+        // Always pass a finite timer: omitting it (or 0) leaves the toast on screen forever.
+        eda.sys_Message.showToastMessage(message, type || 'info', seconds || 4);
       } else {
         console.log('[dsh-eda-mcp] ' + message);
       }
@@ -706,20 +746,30 @@ function dshEdaMcpStatus() {
     }
   }
 
+  // One removable tip instead of a stack of toasts: showToastMessage has no hide API
+  // (timer:0 meant "never"), so a stack of them used to pile up over the canvas and the
+  // '未连接' banner survived even after the connection was established (measured).
+  var CONNECT_TIP = 'DSH MCP 未连接：菜单 DSH MCP → 连接 DSH MCP 服务器（ws://127.0.0.1:39009/ws）';
+
+  function hideConnectTip() {
+    try {
+      if (typeof eda !== 'undefined' && eda.sys_Message && typeof eda.sys_Message.removeFollowMouseTip === 'function') {
+        eda.sys_Message.removeFollowMouseTip(CONNECT_TIP);
+      }
+    } catch (_) { /* ignore */ }
+  }
+
   function showConnectToast() {
     if (wsReady) return;
     try {
       if (typeof eda === 'undefined' || !eda.sys_Message) return;
-      eda.sys_Message.showToastMessage(
-        'DSH MCP 未连接。点击右侧按钮连接本地桥接服务（ws://127.0.0.1:39009/ws）。',
-        'warning',
-        0,
-        undefined,
-        '连接 DSH MCP',
-        "eda.sys_MessageBus.publishPublic('dsh-eda-mcp/connect', {})"
-      );
+      if (typeof eda.sys_Message.showFollowMouseTip === 'function') {
+        eda.sys_Message.showFollowMouseTip(CONNECT_TIP, 8000);
+        return;
+      }
+      eda.sys_Message.showToastMessage(CONNECT_TIP, 'warning', 8);
     } catch (error) {
-      console.error('[dsh-eda-mcp] connect toast failed:', error);
+      console.error('[dsh-eda-mcp] connect tip failed:', error);
     }
   }
 

package/jlc-extension/extension.json

@@ -2,7 +2,7 @@
   "uuid": "d5e0a1b2c3f4456789abcdef01234567",
   "name": "dsh-eda-bridge",
   "displayName": "DSH EDA MCP Bridge",
-  "version": "0.4.6",
+  "version": "0.4.9",
   "description": "Local WebSocket bridge used by the dsh-eda-mcp plugin to let DeepSeek Harness draw schematics and drive PCB operations in JLCEDA Pro through the official extension API.",
   "entry": "./api-bridge.js",
   "publisher": "dsh-eda-mcp",

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";
@@ -807,7 +807,7 @@ var WORKFLOW_GUIDE = WORKFLOW_SUMMARY + "\n\n" + [
 ].join("\n");
 
 // src/bridge/index.ts
-var VERSION = "0.4.6";
+var VERSION = "0.4.9";
 var WS_PORT = Number(process.env.DSH_EDA_MCP_WS_PORT ?? 39009);
 var WS_HOST = "127.0.0.1";
 var ALLOW_RAW_API = (process.env.DSH_EDA_MCP_ALLOW_RAW_API ?? "1") !== "0";
@@ -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

@@ -179,7 +179,7 @@ var SKILL_PLAYBOOK_LINES = [
   "| eda_launch \u5E26 project \u6CA1\u6253\u5F00\u5DE5\u7A0B | \u547D\u4EE4\u884C\u8DEF\u5F84\u88AB\u5E94\u7528\u5FFD\u7565 | \u7528 eda_open_project({ name }) \u6253\u5F00 |",
   "| \u5173\u5E94\u7528\u540E\u6539\u52A8\u4E22\u5931 | eda_close \u8D70\u4E86\u5F3A\u5236\u8DEF\u5F84 | \u5173\u524D\u5148\u4FDD\u5B58\uFF1B\u770B\u8FD4\u56DE\u7684 forced |",
   "| run_code \u8BFB\u5927 JSON \u88AB\u622A\u65AD | \u5BBF\u4E3B read \u6709\u5355\u884C\u4E0A\u9650 | \u7528 node:fs \u8BFB\u76D8\u6216\u5206\u7247\u8BFB |",
-  "| \u8986\u94DC\u521B\u5EFA\u5931\u8D25\uFF08\u65E0\u6CD5\u521B\u5EFA\u8986\u94DC\u8FB9\u6846\u56FE\u5143\uFF09 | \u5E94\u7528\u4FA7\u9650\u5236\uFF1ApolygonStage=ok\uFF08\u8F6E\u5ED3\u88AB\u63A5\u53D7\uFF09\u4F46 pours \u88AB\u62D2\uFF0C\u4E0E\u7F51\u7EDC/\u586B\u5145\u65B9\u5F0F/\u5750\u6807\u65E0\u5173 | \u5DE5\u5177\u4F1A\u91CD\u8BD5\u4E00\u6B21\u5E76\u7ED9\u51FA\u8BE5\u8BCA\u65AD\uFF1B\u624B\u5DE5\u5728 PCB \u9875\u6846\u9009\u8986\u94DC\uFF08GND\uFF0C\u9876\u5C42+\u5E95\u5C42\uFF09\uFF0Cdesign_audit \u8BB0\u4E3A warn |",
+  "| \u8986\u94DC/\u586B\u5145\u4E00\u5F8B\u88AB\u62D2\uFF08\u65E0\u6CD5\u521B\u5EFA\u8986\u94DC\u8FB9\u6846\u56FE\u5143 / \u65E0\u6CD5\u521B\u5EFA\u586B\u5145\u56FE\u5143\uFF09 | \u5E94\u7528\u4FA7\u9650\u5236\uFF1A\u9759\u6001+\u5B9E\u4F8B\u5165\u53E3\u3001\u5355/\u590D\u6742\u591A\u8FB9\u5F62\u3001\u539F\u59CB\u6E90\u7801\u6570\u7EC4\u30016 \u79CD\u586B\u5145\u679A\u4E3E\u3001\u5168\u53C2\u6570\u7EC4\u5408\u5168\u90E8\u88AB\u62D2\uFF1B\u540C\u4E00\u591A\u8FB9\u5F62\u7BA1\u7EBF\u5728\u672C\u6784\u5EFA\u4E0A\u8FDE\u4E1D\u5370\u6298\u7EBF\u90FD\u5199\u4E0D\u51FA\u6765 | \u4E0D\u8981\u5728 API \u5C42\u53CD\u590D\u8BD5\uFF1A\u8986\u94DC\u8BF7\u5728\u754C\u9762\u624B\u5DE5\u94FA\uFF08GND\u3001\u9876\u5C42+\u5E95\u5C42\uFF09\uFF1Bdesign_audit \u6301\u7EED\u8BB0 copper-pour warn |",
   "| pcb_draw_mounting_hole \u62A5\u201C\u65E0\u6CD5\u521B\u5EFA\u710A\u76D8\u56FE\u5143\u201D | pad \u5F62\u72B6\u5FC5\u586B\uFF0C\u4E14\u69FD\u5B54\u53EA\u80FD\u7528 ELLIPSE | \u5DF2\u5185\u5EFA\uFF1A\u5706\u5B54 ELLIPSE(d,d)\uFF0C\u69FD\u5B54 ELLIPSE(\u957F,\u5BBD)+hole[SLOT,\u5BBD,\u957F]\uFF1BOBLONG/RECTANGLE \u710A\u76D8\u4F1A\u88AB\u62D2 |",
   "| pcb_PrimitiveString.getAll / pcb_PrimitivePad.getAll \u8FD4\u56DE 0 | \u8FD9\u4E24\u4E2A API \u6309\u5C42\u67E5\u8BE2 | \u4F20 layer\uFF1A\u4E1D\u5370 3/4\uFF0C\u591A\u5C42\u710A\u76D8 12\uFF1B\u4E0D\u4F20\u53C2\u6052\u4E3A 0 |",
   "| \u6309\u5750\u6807\u7B97\u7684\u8D8A\u754C\u6CA1\u88AB\u53D1\u73B0 | \u56FE\u6846/\u539F\u70B9\u5047\u8BBE\u4E0E\u771F\u5B9E\u9875\u9762\u4E0D\u4E00\u81F4 | \u7528 eda_get_page_info \u7684\u771F\u5B9E\u503C + eda_capture \u5BFC PNG \u770B\u5B9E\u9645\u7248\u9762 |",
@@ -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.2",
   "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'
@@ -21,7 +21,7 @@ import {
 } from './design-rules.ts'
 import { WORKFLOW_GUIDE, WORKFLOW_SUMMARY } from './workflow-guide.ts'
 
-const VERSION = '0.4.6'
+const VERSION = '0.4.9'
 const WS_PORT = Number(process.env.DSH_EDA_MCP_WS_PORT ?? 39009)
 const WS_HOST = '127.0.0.1'
 const ALLOW_RAW_API = (process.env.DSH_EDA_MCP_ALLOW_RAW_API ?? '1') !== '0'
@@ -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.'
@@ -3026,6 +3136,7 @@ async function executeTool(name, rawArgs, timeoutMs) {
     case 'eda_pcb_delete_via':
       return await callExtension('pcb_PrimitiveVia.delete', [String(args.primitiveId)], timeoutMs)
 
+
     case 'eda_pcb_create_pour': {
       const layer = resolveRegionLayer(args.layer, 'eda_pcb_create_pour')
       const points = requireFlatPoints(args.points, 'eda_pcb_create_pour')
@@ -3065,7 +3176,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

@@ -153,7 +153,7 @@ export const SKILL_PLAYBOOK_LINES: readonly string[] = [
   "| eda_launch 带 project 没打开工程 | 命令行路径被应用忽略 | 用 eda_open_project({ name }) 打开 |",
   "| 关应用后改动丢失 | eda_close 走了强制路径 | 关前先保存；看返回的 forced |",
   "| run_code 读大 JSON 被截断 | 宿主 read 有单行上限 | 用 node:fs 读盘或分片读 |",
-  "| 覆铜创建失败（无法创建覆铜边框图元） | 应用侧限制：polygonStage=ok（轮廓被接受）但 pours 被拒，与网络/填充方式/坐标无关 | 工具会重试一次并给出该诊断；手工在 PCB 页框选覆铜（GND，顶层+底层），design_audit 记为 warn |",
+  "| 覆铜/填充一律被拒（无法创建覆铜边框图元 / 无法创建填充图元） | 应用侧限制：静态+实例入口、单/复杂多边形、原始源码数组、6 种填充枚举、全参数组合全部被拒；同一多边形管线在本构建上连丝印折线都写不出来 | 不要在 API 层反复试：覆铜请在界面手工铺（GND、顶层+底层）；design_audit 持续记 copper-pour warn |",
   "| pcb_draw_mounting_hole 报“无法创建焊盘图元” | pad 形状必填，且槽孔只能用 ELLIPSE | 已内建：圆孔 ELLIPSE(d,d)，槽孔 ELLIPSE(长,宽)+hole[SLOT,宽,长]；OBLONG/RECTANGLE 焊盘会被拒 |",
   "| pcb_PrimitiveString.getAll / pcb_PrimitivePad.getAll 返回 0 | 这两个 API 按层查询 | 传 layer：丝印 3/4，多层焊盘 12；不传参恒为 0 |",
   "| 按坐标算的越界没被发现 | 图框/原点假设与真实页面不一致 | 用 eda_get_page_info 的真实值 + eda_capture 导 PNG 看实际版面 |",
@@ -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` 清空。",
   "",