sapbuddy-dsh 0.0.7 → 0.0.8

package/README.md

@@ -10,13 +10,13 @@
 在 DSH 的插件管理页添加：
 
 ```text
-sapbuddy-dsh@0.0.7
+sapbuddy-dsh@0.0.8
 ```
 
 也可以通过 DSH 命令行安装：
 
 ```bash
-dsh plugin --profile desktop add sapbuddy-dsh@0.0.7
+dsh plugin --profile desktop add sapbuddy-dsh@0.0.8
 ```
 
 更新已安装插件后，完整退出并重新启动 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 令牌去读写当前系统。
@@ -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
 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/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,6 +1,6 @@
 {
   "name": "sapbuddy-dsh",
-  "version": "0.0.7",
+  "version": "0.0.8",
   "description": "SapBuddy —— SAP ABAP AI 全能助手（DeepSeek Harness 插件）：41 个 SAP 工具 + 8 个 ABAP 技能 + SapBuddy preset + 连接设置页。一个包名装完。",
   "type": "module",
   "main": "dist/index.js",
@@ -77,10 +77,13 @@
     "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",
     "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"
   },
   "publishConfig": {
     "access": "public"