sapbuddy-dsh 0.0.7 → 0.0.9

package/README.md

@@ -2,7 +2,7 @@
 
 [![npm](https://img.shields.io/npm/v/sapbuddy-dsh)](https://www.npmjs.com/package/sapbuddy-dsh) [GitHub](https://github.com/gxx950224/sapbuddy-dsh)
 
-[SapBuddy](https://github.com/gxx950224/sapbuddy) 的 DeepSeek Harness 插件，提供 41 个 SAP ABAP 工具、8 个技能、SapBuddy agent preset 和 SAP 连接设置页。
+[SapBuddy](https://github.com/gxx950224/sapbuddy) 的 DeepSeek Harness 插件，提供 41 个 SAP ABAP 工具、10 个技能、SapBuddy agent preset 和 SAP 连接设置页。
 
 ## 安装
 
@@ -10,13 +10,13 @@
 在 DSH 的插件管理页添加：
 
 ```text
-sapbuddy-dsh@0.0.7
+sapbuddy-dsh@0.0.9
 ```
 
 也可以通过 DSH 命令行安装：
 
 ```bash
-dsh plugin --profile desktop add sapbuddy-dsh@0.0.7
+dsh plugin --profile desktop add sapbuddy-dsh@0.0.9
 ```
 
 更新已安装插件后，完整退出并重新启动 DSH，再新建会话选择 **SapBuddy**。
@@ -32,6 +32,19 @@ dsh plugin --profile desktop add sapbuddy-dsh@0.0.7
 
 源码不一致、相关对象仍未激活或核对查询失败时，继续保留失败。对象元数据中的 `active` 字段不能单独证明本次修改已生效。非源码对象沿用其激活接口结果。
 
+## 连接配置与排查
+
+地址必须写全协议，协议要和端口一致：
+
+| 端口 | 协议 | 示例 |
+|---|---|---|
+| ICM HTTP（常见 8000） | `http://` | `http://192.168.111.85:8000` |
+| ICM HTTPS（常见 44300） | `https://` | `https://10.8.202.35:44300` |
+
+把 HTTP 端口写成 `https://`（或反过来）会在 TLS 阶段直接失败。客户端号就是登录用的 `sap-client`，必须与要登录的客户端一致——系统是 300 却填 200 会直接返回 401 认证失败；登录语言不确定时填 `EN`。内部系统若是自签名证书，编辑连接时保持「允许自签名证书」勾选。
+
+设置页的 **测试** 按钮与真实 SAP 工具走同一套协议/客户端号/语言（早期版本把探活写死成 https 且只发 `X-sap-login-client` 头，会出现"账号密码都对却报 401"或"http 端口永远探测失败"的假阴性）。测试返回 401 时按提示核对客户端号、账号状态与登录语言；403/404 多为 ADT 服务未激活或用户无权限。
+
 ## 连接切换与会话自愈
 
 切换或保存 SAP 连接后，插件会立刻作废进程内池化的 ADT 客户端，下一次调用按新配置重新登录，不会带着上一个系统的 cookie 与 CSRF 令牌去读写当前系统。
@@ -44,7 +57,7 @@ dsh plugin --profile desktop add sapbuddy-dsh@0.0.7
 
 工具涵盖对象搜索与读码、代码导航、语法诊断与激活、ATC 与单元测试、SQL 查询、DDIC、文本池、屏幕文本、消息类、函数模块接口和传输请求。
 
-技能包括 `abap-code-review`、`abap-explain`、`abap-templates`、`abap-wiki`、`clean-abap`、`dev-write`、`sap-abap-skills`、`translate-workflow`。
+技能包括 `abap-code-review`、`abap-explain`、`abap-templates`、`abap-wiki`、`clean-abap`、`dev-write`、`sap-abap-skills`、`translate-workflow`、`sap-fs`（四类 FS：MD + Claude 风 HTML）、`grilling`（经 sap-fs 加载的追问澄清）。
 
 ## 配置和写入保护
 
@@ -61,11 +74,11 @@ dsh plugin --profile desktop add sapbuddy-dsh@0.0.7
 从 GitHub 克隆仓库后，在 `packages/sapbuddy-dsh` 下运行激活回归测试（需要 Node.js 24+，无需连接 SAP）：
 
 ```bash
-npm test                      # = test:activation + test:session + test:store
+npm test                      # = test:activation + test:session + test:store + test:probe + test:route + test:bundle + test:sap-fs
 npm pack --dry-run --ignore-scripts
 ```
 
-`test:activation` 覆盖激活结果核对的误报恢复、失败计数、审计记录和核对失败边界；`test:session` 覆盖写前会话刷新、CSRF 失效自动重建，以及"已落库则绝不重试"的边界；`test:store` 用真实模块验证设置页保存/切换/删除连接确实会作废连接池。测试都使用模拟 SAP 响应，不能替代真实 SAP 联机验收。
+`test:activation` 覆盖激活结果核对的误报恢复、失败计数、审计记录和核对失败边界；`test:session` 覆盖写前会话刷新、CSRF 失效自动重建，以及"已落库则绝不重试"的边界；`test:store` 用真实模块验证设置页保存/切换/删除连接确实会作废连接池；`test:probe` 覆盖连接探测的协议/客户端号/语言与 401 提示；`test:route` 起一个本地 http 服务端到端验证「测试」按钮（含 sap-client 传递与 401 提示）；`test:bundle` 对设置页 bundle 做渲染冒烟。测试都使用模拟 SAP 响应，不能替代真实 SAP 联机验收。
 
 ## 许可证
 

package/defaults/prompts/SYSTEM.md

@@ -32,7 +32,8 @@ SapBuddy — SAP ABAP AI 全能助手，面向**开发顾问与业务顾问**。
    - **工具硬化**：`create_object_programmatically` 缺包名/描述会被拦截；传 $TMP 需用户单独确认（不进传输请求）；`replace_string` 无未释放请求时自动建请求——这些无需 LLM 自觉，工具强制执行。
 6c. **连接切换后先重确认（强制）**：用户切换/修改 SAP 连接后，**必须先调用 `get_connected_systems` 确认当前连接，再使用其他 SAP 工具**。工具层强制拦截：未确认前其他 SAP 工具一律失败（返回"连接配置已变更，先调用 get_connected_systems"）。看到该拦截错误时，立即调用 `get_connected_systems`，再重试被拦的操作。
 7. **写操作谨慎**：写工具（创建/编辑/激活）需要服务器 `security.readOnly=false`；生产系统只读。
-8. **按技能加载（强制）**：写代码/写 SAP 对象（函数模块/表结构/函数组 include/文本元素等）先遵循 `dev-write` 技能（编码规范、写入规范）；批量翻译（Excel 清单→SAP）先遵循 `translate-workflow` 技能（先 list 后 set、匹配不上报告、**禁止猜 key 写入**）。技能按话题自动加载，做对应任务时**必须遵守**。
+8. **按技能加载（强制）**：写代码/写 SAP 对象（函数模块/表结构/函数组 include/文本元素等）先遵循 `dev-write` 技能（编码规范、写入规范）；批量翻译（Excel 清单→SAP）先遵循 `translate-workflow` 技能（先 list 后 set、匹配不上报告、**禁止猜 key 写入**）。技能按话题自动加载，做对应任务时**必须遵守**；写 FS / 功能设计书时遵循 `sap-fs` 技能（意图不明先问，见铁律 9）。
+9. **FS 意图门禁（强制）**：仅当用户**明确**要写/改 FS、功能设计书、接口/报表/增强**规格文档**（或明确说按 sap-fs 出可交付文档）时，才加载并执行 `sap-fs`。日常问答、配置怎么配、表/事务码解释、程序解释/审查、写 ABAP 代码 → **禁止**触发 sap-fs（写码→`dev-write`；审查→`abap-code-review`；解释逻辑→`abap-explain` / `abap-wiki`）。用户只抛业务描述或方案讨论、**未明确**要出 FS → **先问一句**是否要按 sap-fs 产出 `FS.md` + `FS.html`（接口另附 JSON）；未确认前只澄清/回答，**不**套 FS 模板、不落盘 `output/fs/`。用户确认后才 Skill 加载 `sap-fs`；追问业务决策时再按 sap-fs 流程加载 `grilling`。
 
 ## 防死循环与止损（软约束，靠自觉；能代码强制的已由工具层兜底）
 

package/dist/client.js

@@ -66,6 +66,7 @@ var S = {
   badge: { border: "1px solid var(--dsw-alias-border-l3)", color: "var(--dsw-alias-label-secondary)", borderRadius: 4, padding: "1px 6px", fontSize: 11, lineHeight: "16px", flex: "none" },
   badgeActive: { background: "var(--dsw-alias-state-success-primary)", borderColor: "var(--dsw-alias-state-success-primary)", color: "var(--dsw-alias-label-primary-foreground)" },
   meta: { margin: 0, fontSize: 12, lineHeight: "18px", color: "var(--dsw-alias-label-tertiary)", overflow: "hidden", textOverflow: "ellipsis", whiteSpace: "nowrap" },
+  testMsg: { margin: 0, fontSize: 12, lineHeight: "18px", color: "var(--dsw-alias-label-secondary)", whiteSpace: "pre-wrap" },
   actions: { display: "flex", gap: 6, marginLeft: "auto", flex: "none" },
   btn: { height: 28, padding: "0 10px", borderRadius: 14, fontSize: 12, lineHeight: "18px", cursor: "pointer", border: "1px solid var(--dsw-alias-border-l2)", background: "transparent", color: "var(--dsw-alias-label-primary)", font: "inherit" },
   btnPrimary: { background: "var(--dsw-alias-button-primary-fill)", color: "var(--dsw-alias-label-primary-foreground)", border: "none" },
@@ -178,24 +179,38 @@ function Section() {
       { key: "m", style: S.meta },
       c.url + (c.client ? " \xB7 Client " + c.client : "") + " \xB7 " + c.username + (c.hasPassword ? " \xB7 \u5BC6\u7801\u5DF2\u8BBE\u7F6E" : " \xB7 \u672A\u8BBE\u7F6E\u5BC6\u7801")
     ),
-    testResult[c.id] ? React.createElement("p", { key: "tr", style: S.meta }, testResult[c.id]) : null
+    testResult[c.id] ? React.createElement("p", { key: "tr", style: S.testMsg }, testResult[c.id]) : null
   ]);
   const editor = React.createElement("div", { key: "editor", style: S.editor }, [
     React.createElement("div", { key: "g1", style: S.grid }, [
       input("id", "\u8FDE\u63A5 ID\uFF08\u552F\u4E00\uFF0C\u5982 dev/prod\uFF09", "dev"),
       input("name", "\u540D\u79F0", "\u5F00\u53D1\u7CFB\u7EDF"),
-      input("url", "\u5730\u5740\uFF08https://host:port\uFF09", "https://your-sap-host:44300"),
-      input("client", "\u5BA2\u6237\u7AEF\u53F7", "100")
+      input("url", "\u5730\u5740\uFF08http(s)://host:port\uFF09", "http://10.8.202.35:8000 \u6216 https://10.8.202.35:44300"),
+      input("client", "\u5BA2\u6237\u7AEF\u53F7\uFF083 \u4F4D\uFF0C\u5FC5\u987B\u4E0E\u7CFB\u7EDF\u4E00\u81F4\uFF09", "100")
     ]),
     React.createElement("div", { key: "g2", style: S.grid }, [
       input("username", "\u7528\u6237\u540D", "YOUR_USER"),
       input("password", "\u5BC6\u7801\uFF08\u7F16\u8F91\u65F6\u7559\u7A7A\u4FDD\u6301\u539F\u5BC6\u7801\uFF09", "\u2022\u2022\u2022\u2022\u2022\u2022"),
-      input("language", "\u8BED\u8A00", "ZH"),
+      input("language", "\u767B\u5F55\u8BED\u8A00\uFF08\u4E0D\u786E\u5B9A\u586B EN\uFF09", "ZH"),
       input("authMethod", "\u8BA4\u8BC1\u65B9\u5F0F", "basic")
     ]),
     React.createElement("div", { key: "g3", style: S.grid }, [
       input("description", "\u63CF\u8FF0", "\u5F00\u53D1\u7CFB\u7EDF")
     ]),
+    React.createElement("label", { key: "ssl", style: { ...S.field, flexDirection: "row", alignItems: "center", gap: 8 } }, [
+      React.createElement("input", {
+        key: "i",
+        type: "checkbox",
+        checked: !draft.ssl || draft.ssl.allowSelfSigned !== false,
+        onChange: (evt) => setDraft((d) => ({ ...d, ssl: { ...(d.ssl || {}), allowSelfSigned: evt.target.checked } }))
+      }),
+      React.createElement("span", { key: "l", style: S.label }, "\u5141\u8BB8\u81EA\u7B7E\u540D\u8BC1\u4E66\uFF08\u5185\u90E8\u7CFB\u7EDF https \u7528\uFF09")
+    ]),
+    React.createElement(
+      "p",
+      { key: "hint", style: S.hint },
+      "\u7AEF\u53E3\u5BF9\u5E94\u534F\u8BAE\uFF1ASAP \u7684 ICM HTTP \u7AEF\u53E3\uFF08\u5E38\u89C1 8000\uFF09\u7528 http://\uFF0CHTTPS \u7AEF\u53E3\uFF08\u5E38\u89C1 44300\uFF09\u7528 https://\u3002\u5BA2\u6237\u7AEF\u53F7\u5C31\u662F\u767B\u5F55\u5BA2\u6237\u7AEF\uFF08sap-client\uFF09\uFF0C\u586B\u9519\u4F1A\u8FD4\u56DE 401\uFF1B\u767B\u5F55\u8BED\u8A00\u4E0D\u786E\u5B9A\u65F6\u586B EN\u3002"
+    ),
     React.createElement("div", { key: "acts", style: { display: "flex", gap: 8, justifyContent: "flex-end" } }, [
       React.createElement("button", { key: "c", style: S.btn, onClick: () => {
         setMode(null);
@@ -209,7 +224,7 @@ function Section() {
     React.createElement(
       "p",
       { key: "i", style: S.intro },
-      "\u914D\u7F6E\u4E00\u4E2A\u6216\u591A\u4E2A SAP \u7CFB\u7EDF\u8FDE\u63A5\uFF08ADT\uFF09\u3002\u5207\u6362\u4E3A\u542F\u7528\u540E\uFF0CSapBuddy \u4F1A\u8BDD\u7684 SAP \u5DE5\u5177\u9ED8\u8BA4\u4F7F\u7528\u8BE5\u8FDE\u63A5\uFF1B\u5207\u6362\u540E\u9700\u5728\u4F1A\u8BDD\u4E2D\u5148\u8C03\u7528 get_connected_systems \u786E\u8BA4\u3002\u5BC6\u7801\u4EC5\u4FDD\u5B58\u5728\u672C\u673A\u914D\u7F6E\u6587\u4EF6\u3002"
+      "\u914D\u7F6E\u4E00\u4E2A\u6216\u591A\u4E2A SAP \u7CFB\u7EDF\u8FDE\u63A5\uFF08ADT\uFF09\u3002\u5207\u6362\u4E3A\u542F\u7528\u540E\uFF0CSapBuddy \u4F1A\u8BDD\u7684 SAP \u5DE5\u5177\u9ED8\u8BA4\u4F7F\u7528\u8BE5\u8FDE\u63A5\uFF1B\u5207\u6362\u540E\u9700\u5728\u4F1A\u8BDD\u4E2D\u5148\u8C03\u7528 get_connected_systems \u786E\u8BA4\u3002\u5BC6\u7801\u4EC5\u4FDD\u5B58\u5728\u672C\u673A\u914D\u7F6E\u6587\u4EF6\u3002\u5730\u5740\u8981\u5199\u5168\u534F\u8BAE\uFF1Ahttp:// \u7528\u4E8E ICM HTTP \u7AEF\u53E3\uFF08\u5E38\u89C1 8000\uFF09\uFF0Chttps:// \u7528\u4E8E HTTPS \u7AEF\u53E3\uFF08\u5E38\u89C1 44300\uFF09\u3002"
     ),
     msg ? React.createElement("p", { key: "m", style: S.msg }, msg) : null,
     err ? React.createElement("p", { key: "e", style: S.err }, err) : null,

package/dist/index.js

@@ -1,3 +1,4 @@
+import http from "node:http";
 import https from "node:https";
 import { listConnections, saveConnection, deleteConnection, activateConnection, getConnectionSecret, } from "./sap-tools/connections-store.js";
 const ROUTES = [
@@ -33,22 +34,95 @@ function readBody(req) {
         req.on("error", reject);
     });
 }
+/**
+ * 把连接配置规范成一次 ADT 探活请求。
+ *
+ * 注意两件事（曾经踩过的坑）：
+ * 1. 协议必须跟随配置里的 URL：SAP 的 ICM HTTP 端口（常见 8000）是 **http**，
+ *    HTTPS 端口（常见 44300）才是 https。写死 https 会让 http 连接永远探测失败。
+ * 2. 客户端号/语言要与真实工具调用一致——通过查询参数 `sap-client` / `sap-language` 传
+ *    （abap-adt-api 登录就是这么做的），同时保留 `X-sap-login-client` 头兼容只认该头的系统。
+ *    只发头不发参数时，部分系统会忽略头、用默认客户端登录，于是返回 401。
+ *
+ * @returns {{ error: string } | { scheme: "http" | "https", url: URL, headers: Record<string, string> }}
+ */
+export function buildProbeRequest(conf) {
+    const raw = String(conf?.url ?? "").trim().replace(/\/+$/, "");
+    if (!/^https?:\/\//i.test(raw))
+        return { error: `连接地址要以 http:// 或 https:// 开头（当前是 "${raw || "(空)"}"）。` +
+                `例如 http://10.8.202.35:8000（ICM HTTP 端口）或 https://10.8.202.35:44300（HTTPS 端口）` };
+    let url;
+    try {
+        url = new URL(`${raw}/sap/bc/adt/compatibility/graph`);
+    }
+    catch {
+        return { error: `连接地址不合法: ${raw}` };
+    }
+    const scheme = url.protocol === "https:" ? "https" : "http";
+    const qs = new URLSearchParams();
+    if (conf.client)
+        qs.set("sap-client", String(conf.client));
+    if (conf.language)
+        qs.set("sap-language", String(conf.language));
+    if ([...qs.keys()].length > 0)
+        url.search = qs.toString();
+    const headers = {
+        Authorization: "Basic " + Buffer.from(`${conf.username}:${conf.password ?? ""}`).toString("base64"),
+        Accept: "*/*",
+    };
+    if (conf.client)
+        headers["X-sap-login-client"] = String(conf.client);
+    return { scheme, url, headers };
+}
+/** 把探活拿到的 HTTP 状态码翻译成能直接照做的排查建议；null 表示成功 */
+export function describeProbeStatus(status, conf) {
+    if (status >= 200 && status < 300)
+        return null;
+    if (status === 401)
+        return "认证失败（HTTP 401）：服务器拒绝了账号/密码。按顺序排查：\n" +
+            `① 客户端号是否与系统一致（当前填的是 ${conf.client || "空"}，登录时作为 sap-client 发送）；\n` +
+            `② 用户名/密码是否正确、账号是否被锁或过期（当前用户 ${conf.username || "空"}）；\n` +
+            `③ 语言 ${conf.language || "空"} 是否为该账号可用语言（不确定先填 EN）；\n` +
+            "④ 该客户端是否允许 ADT 登录（SICF 里 /sap/bc/adt 的登录过程）。";
+    if (status === 403)
+        return "HTTP 403：账号认证通过但无权访问 ADT，或 /sap/bc/adt 服务未激活/被限制。请让 BASIS 检查 SICF 与用户 ADT 权限。";
+    if (status === 404)
+        return "HTTP 404：这个地址上没有 ADT 服务。确认端口是 SAP 的 HTTP/HTTPS 端口（8000/44300），且 /sap/bc/adt 已在 SICF 激活。";
+    if (status >= 500)
+        return `HTTP ${status}：SAP 服务端错误（ICM/ADT 异常），请稍后重试或联系 BASIS。`;
+    return `HTTP ${status}：未预期的响应，请检查地址与端口。`;
+}
+/** 网络层失败时给出"协议/端口写反"这类最常见的提示 */
+function describeProbeFailure(message, scheme) {
+    if (/wrong version number|SSL routines|EPROTO|packet length too long|socket hang up/i.test(message))
+        return `连接失败: ${message}\n` +
+            `协议和端口对不上：这个端口很可能是 ${scheme === "https" ? "纯 HTTP（用 http://）" : "HTTPS（用 https://）"}。` +
+            "SAP 的 ICM HTTP 端口（常见 8000）要用 http://，HTTPS 端口（常见 44300）要用 https://。";
+    if (/certificate|self-signed|SELF_SIGNED|CERT_|unable to verify/i.test(message))
+        return `连接失败: ${message}\n证书不被信任：内部/自签名证书请在连接里开启「允许自签名证书」。`;
+    if (/ENOTFOUND|EAI_AGAIN/i.test(message))
+        return `连接失败: 主机名解析不了（${message}）。检查地址拼写，或改用 IP。`;
+    if (/ECONNREFUSED/i.test(message))
+        return `连接失败: 端口没有监听（${message}）。检查端口号与 SAP 实例状态。`;
+    if (/timeout|ETIMEDOUT|超时/i.test(message))
+        return `连接失败: 连接超时（${message}）。检查网络/防火墙是否放通该端口。`;
+    return `连接失败: ${message}`;
+}
 /** 测试连接：用连接明文凭据做一次 ADT 请求（结果只回状态码，不回敏感信息）。
- * 用 node:https 直连以支持 ssl.allowSelfSigned（Node fetch 不支持该选项）。 */
+ * 用 node:http/https 直连以支持 ssl.allowSelfSigned（Node fetch 不支持该选项）。 */
 async function testConnection(id) {
     const c = getConnectionSecret(id);
     if (!c)
         return { ok: false, status: 0, message: `连接 ${id} 不存在` };
-    const url = new URL(`${String(c.url).replace(/\/+$/, "")}/sap/bc/adt/compatibility/graph`);
-    const auth = "Basic " + Buffer.from(`${c.username}:${c.password ?? ""}`).toString("base64");
-    const headers = { Authorization: auth, Accept: "*/*" };
-    if (c.client)
-        headers["X-sap-login-client"] = c.client;
+    const probe = buildProbeRequest(c);
+    if (probe.error)
+        return { ok: false, status: 0, message: probe.error };
+    const transport = probe.scheme === "https" ? https : http;
     try {
         const status = await new Promise((resolve, reject) => {
-            const req = https.request(url, {
+            const req = transport.request(probe.url, {
                 method: "GET",
-                headers,
+                headers: probe.headers,
                 rejectUnauthorized: !(c.ssl?.allowSelfSigned ?? false),
                 timeout: 15_000,
             }, (res) => { res.resume(); resolve(res.statusCode ?? 0); });
@@ -56,11 +130,11 @@ async function testConnection(id) {
             req.on("error", reject);
             req.end();
         });
-        const ok = status >= 200 && status < 300;
-        return { ok, status, message: ok ? "连接成功" : `认证失败（HTTP ${status}）：请检查账号/密码/客户端号，或 ADT 服务是否激活` };
+        const detail = describeProbeStatus(status, c);
+        return detail ? { ok: false, status, message: detail } : { ok: true, status, message: "连接成功" };
     }
     catch (e) {
-        return { ok: false, status: 0, message: `连接失败: ${e instanceof Error ? e.message : String(e)}` };
+        return { ok: false, status: 0, message: describeProbeFailure(e instanceof Error ? e.message : String(e), probe.scheme) };
     }
 }
 /**

package/dist/sap-tools/connections-store.js

@@ -55,10 +55,19 @@ function validate(input) {
         return "连接 ID 不能为空";
     if (!/^[A-Za-z0-9_-]+$/.test(id))
         return "连接 ID 只能包含字母/数字/下划线/连字符";
-    if (!String(input.url ?? "").trim())
+    const url = String(input.url ?? "").trim();
+    if (!url)
         return "连接地址（url）不能为空";
+    // 协议必须显式写：SAP 的 ICM HTTP 端口（常见 8000）是 http，HTTPS 端口（常见 44300）才是 https。
+    // 不写协议时真实工具与探测都会拿不到明确结论，干脆在保存时就拦下。
+    if (!/^https?:\/\//i.test(url))
+        return "连接地址要以 http:// 或 https:// 开头（ICM HTTP 端口常见 8000 用 http://，HTTPS 端口常见 44300 用 https://）";
     if (!String(input.username ?? "").trim())
         return "用户名不能为空";
+    // 客户端号就是登录时的 sap-client，填错（比如系统是 300 却填 200）会直接 401
+    const client = String(input.client ?? "").trim();
+    if (client && !/^\d{1,3}$/.test(client))
+        return "客户端号应为 1-3 位数字（如 100）";
     return undefined;
 }
 /** 新建或更新连接；更新时密码为空/'***' 保持原密码。返回 {ok} 或 {ok:false,error} */
@@ -72,12 +81,13 @@ export function saveConnection(input) {
     const password = input.password === undefined || input.password === "" || input.password === PASSWORD_MASK
         ? existing?.password ?? ""
         : input.password;
+    const rawClient = String(input.client ?? existing?.client ?? "").trim();
     const next = {
         ...existing,
         ...input,
         id,
         password,
-        client: String(input.client ?? existing?.client ?? "").trim(),
+        client: rawClient ? rawClient.padStart(3, "0") : rawClient,
         language: input.language ?? existing?.language ?? "ZH",
         authMethod: input.authMethod ?? existing?.authMethod ?? "basic",
     };

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "sapbuddy-dsh",
-  "version": "0.0.7",
-  "description": "SapBuddy —— SAP ABAP AI 全能助手（DeepSeek Harness 插件）：41 个 SAP 工具 + 8 个 ABAP 技能 + SapBuddy preset + 连接设置页。一个包名装完。",
+  "version": "0.0.9",
+  "description": "SapBuddy —— SAP ABAP AI 全能助手（DeepSeek Harness 插件）：41 个 SAP 工具 + 10 个 ABAP 技能 + SapBuddy preset + 连接设置页。一个包名装完。",
   "type": "module",
   "main": "dist/index.js",
   "exports": {
@@ -77,10 +77,14 @@
     "typescript": "^5.6.0"
   },
   "scripts": {
-    "test": "npm run test:activation && npm run test:session && npm run test:store",
+    "test": "npm run test:activation && npm run test:session && npm run test:store && npm run test:probe && npm run test:route && npm run test:bundle && npm run test:sap-fs",
     "test:activation": "node --experimental-vm-modules --test --test-isolation=none test/activation.test.mjs",
     "test:session": "node --experimental-vm-modules --test --test-isolation=none test/session-recovery.test.mjs",
-    "test:store": "node --test --test-isolation=none test/connections-store.test.mjs"
+    "test:store": "node --test --test-isolation=none test/connections-store.test.mjs",
+    "test:probe": "node --test --test-isolation=none test/connection-probe.test.mjs",
+    "test:route": "node --test --test-isolation=none test/connection-test-route.test.mjs",
+    "test:bundle": "node --test --test-isolation=none test/client-bundle.test.mjs",
+    "test:sap-fs": "node --test --test-isolation=none test/sap-fs/loading.test.mjs"
   },
   "publishConfig": {
     "access": "public"

package/skills/grilling/SKILL.md

@@ -0,0 +1,43 @@
+---
+name: grilling
+description: >-
+  Grill the user relentlessly about a plan, decision, or idea. Use when the user
+  wants to stress-test their thinking, or uses any 'grill' trigger phrases
+  (including "grill me"). Also used by sap-fs when clarifying FS business decisions;
+  not the entry skill for FS intent (load via sap-fs after user confirms FS deliverable).
+---
+
+<!--
+Third-party: mattpocock/skills (MIT, Copyright 2026 Matt Pocock)
+Upstream: https://github.com/mattpocock/skills
+  skills/productivity/grilling/SKILL.md
+Pinned blob: df69d9936880cd3314c28858c6c52f48db23b856 (fetched 2026-10-28)
+Adaptation: bundled under sapbuddy-dsh/skills for DSH discovery; description notes sap-fs;
+  "grill me" triggers are covered here directly (no separate grill-me alias).
+-->
+
+Interview the user relentlessly until you reach a shared understanding. Map this as a **design tree**: every decision branches into the decisions that hang off it.
+
+Work the tree in **rounds**. The **frontier** is every decision whose prerequisites are already settled: the questions you can ask _now_ without guessing at answers you haven't heard yet. Ask the whole frontier in one round: number each question and give your recommended answer. Then wait for the user's answers before the next round.
+
+Format a round like so:
+
+```
+❓ **Q1** - **<question title>**: <question body, might be multiple paragraphs, including multiple choices>
+
+➡️ <your recommended answer>
+
+---
+
+❓ **Q2** - **<question title>**: <question body, might be multiple paragraphs, including multiple choices>
+
+➡️ <your recommended answer>
+```
+
+Word each question so "yes" accepts your recommended answer.
+
+Each round the user answers reshapes the tree: settled decisions push the frontier outward and unblock questions that depended on them. Recompute the frontier and ask the next round. A question whose answer depends on another question still open in this round belongs to a _later_ round, not this one.
+
+Finding _facts_ is your job, never the user's. When a frontier question needs a fact from the environment (filesystem, tools, etc.), dispatch a sub-agent to find it; don't ask the user for anything you could look up yourself. Don't block on it: a running exploration is an unsettled prerequisite, so only the questions downstream of it wait for the sub-agent to report; ask the rest of the frontier now. The _decisions_ are the user's: put each to them and wait.
+
+The session is done when the frontier is empty: every branch of the design tree visited, nothing left silently assumed. Do not act on it until the user confirms you have reached a shared understanding.

package/skills/sap-fs/SKILL.md

@@ -0,0 +1,76 @@
+---
+name: sap-fs
+description: >-
+  编写 SAP 功能规格书（FS / 功能设计书）。仅在用户明确要求写/改 FS、功能设计书、
+  接口设计、报表设计、增强设计，或明确说按 sap-fs 出可交付规格文档时触发。
+  日常问答、配置咨询、解释程序、代码审查、写 ABAP 不触发；意图不明时先问是否出 FS。
+  确认后：识别需求类型 → 系统取证 → 用 grilling 追问业务决策 → 按模板生成 FS。
+  触发关键词：写FS、功能规格、功能设计书、接口设计、报表设计、增强设计、出一份FS。
+agent_created: true
+disable: false
+---
+
+# sap-fs — 写 SAP 功能规格书
+
+> 草稿结构（Codex 第一版目录）。模板在 `assets/templates/`，过程规范在 `references/`。
+> 追问逻辑不内嵌：加载 `grilling`。上游许可与钉选说明见 `grilling/SKILL.md` 顶部注释。
+
+## 何时使用
+
+- 用户**明确**要写 / 改 FS、功能设计书、接口 / 报表 / 增强**规格文档**，或明确说按 sap-fs 出可交付文档。
+- 用户确认「要出 FS」之后（见 SYSTEM 铁律 FS 意图门禁）。
+
+不触发：日常问答、配置怎么配、表/事务码解释、程序解释（`abap-explain` / `abap-wiki`）、代码审查（`abap-code-review`）、写 ABAP（`dev-write` / `abap-templates`）。
+意图不明（只抛业务描述 / 方案讨论、未说要出 FS）：**先问一句**是否要按 sap-fs 产出 `FS.md` + `FS.html`（接口另附 JSON）；未确认前不套模板、不落盘 `output/fs/`。
+
+## 资源地图（按需加载，勿一次全读）
+
+| 时机 | 读什么 |
+| --- | --- |
+| 触发后立刻 | 本文件 + `references/fs-contract.md` |
+| 需要连系统 | `references/system-verification.md` |
+| 需要追问 | `references/grilling-integration.md`，并 **Skill 加载 `grilling`**（含 “grill me” 类触发） |
+| 已判定类型 | `references/types/<type>.md` + `assets/templates/fs-base.md` + `assets/templates/<type>.md` |
+| 对齐详略 | `references/example-enhancement.md`（仅增强类样例） |
+| 交付前 | `references/review-checklist.md` |
+| 出归档 HTML / 接口 JSON | `references/archive-html.md` + `assets/templates/fs-archive.html` |
+
+类型文件名：`interface` / `query-report` / `functional-report` / `enhancement`。
+
+## 工作流程
+
+1. **判类型**（混合取主导；ALV 按钮会改业务状态 → 功能报表）。读对应 `types/` 与模板。
+2. **整理已知**：用户已说内容分到需求 / 现状 / 规则，先不提问。
+3. **取证**：按 `system-verification.md` 只查能查的。
+4. **追问**：按 `grilling-integration.md` 加载 `grilling`，只问业务决策。
+5. **生成**：拼装 `fs-base` + 类型专项模板；规则 / 验收 / 待确认 / 附录按 `fs-contract.md`。
+6. **自检**：`review-checklist.md` 通过后再交付。
+
+## 拼装
+
+```
+assets/templates/fs-base.md     # 第1、2、4、5、6、7节 + 附录A
+  + assets/templates/<type>.md  # 第3节
+  + 头部状态行
+= 一份完整 FS
+```
+
+默认落盘目录（用户未指定时）：`output/fs/<需求名>/`（允许的 `output` 子树）；详见下方交付物。
+
+## 交付物（Markdown + HTML，接口另附 JSON）
+
+同一需求目录默认：`output/fs/<需求名>/`
+
+1. `FS.md` — 协作 / AI 主规格。
+2. `FS.html` — 企业归档：套 `assets/templates/fs-archive.html`，规则见 `references/archive-html.md`。
+3. **禁止 logo**（徽标、水印、品牌图都不放）。
+4. **接口类**另出：正文 3.6 样例 JSON + 同目录 `io-request.json` / `io-response.json`（及必要的回调文件）。
+
+先定稿 Markdown，再渲染 HTML / 写出 JSON；自检通过后一并交付路径。
+
+## 硬边界
+
+- 本技能流程**只读 SAP**；FS 确认 ≠ 写入许可。
+- 正文只写事实与决定；训诫与生成纪律留在技能文件，不进 FS。
+- 不虚构表名 / 字段 / 增强点 / 签署与测试结果。
+- HTML 归档不使用任何 logo。

package/skills/sap-fs/assets/templates/enhancement.md

@@ -0,0 +1,52 @@
+<!-- 本片段插入骨架第3节。增强只描述"在哪个时点、改变哪段行为、其余保持原样"。
+     第2节写整体现状；这里只写本次涉及的动作。 -->
+
+## 3. 增强设计
+
+### 3.1 改变的行为
+
+| 动作 | 现状 | 目标 | 保持不变 |
+| --- | --- | --- | --- |
+| <保存 / 过账 / 审批> | <现在怎么处理> | <改变后怎么处理> | <仍须沿用的标准行为> |
+
+**实施方式**：<标准配置能否满足；选增强的理由。未验证的判断标 Q。>
+
+### 3.2 触发点
+
+| 入口 | 时点 | 触发前状态 | 取得的数据 |
+| --- | --- | --- | --- |
+| <事务码 / 接口 / 后台> | <校验阶段 / 保存前 / 提交后> | <订单已保存 / 未保存> | <当前输入值 / 数据库旧值> |
+
+- **覆盖入口**：<前台、后台、接口分别是否覆盖；未确认的标 Q。>
+- **新旧值**：<规则用的是用户当前输入，还是已落库的值；修改后是否重新校验。>
+- **未保存数据**：<字段尚未落库时以什么为准。>
+- **派生值**：<标准逻辑或其它增强是否也会改同一字段；先后关系。>
+
+### 3.3 适用与排除
+
+| 项目 | 定义 |
+| --- | --- |
+| 命中条件 | <组织、单据类型、业务状态、角色> |
+| 排除条件 | <必须绕过增强的场景> |
+| 与其它规则的优先级 | <同时命中时先后；先报哪条> |
+| 不命中时 | <维持原有行为> |
+| 既有校验先失败时 | <是否继续执行本增强；用户最终看到什么> |
+
+### 3.4 处理逻辑
+
+<按顺序写：取数 → 判断 → 结果。引用 R-01 等规则编号，不在这里重复条件。>
+
+### 3.5 重复触发与事务
+
+| 场景 | 期望 |
+| --- | --- |
+| 同一操作多次触发 | <哪些可重复，哪些副作用只能发生一次> |
+| 保存后再次修改 | <哪些字段变化需要重新判断> |
+| 标准动作最终失败 | <本次已产生的变化是否必须一同撤销> |
+| 已调用外部系统 | <外部成功但本地失败的核对与补偿> |
+
+### 3.6 停用与回归
+
+- **停用后**：<恢复原有行为；已保存数据不回滚。>
+- **在途业务**：<处理中的单据、任务、外部请求如何处理。>
+- **回归范围**：<既有增强、标准校验、相关模块（凭证流、库存、财务）需要复测的部分。>