@mrrisega/dsh-remote 0.6.18 → 0.7.0-beta.2

package/packages/dsh-remote-web/test/diag-report.test.mjs

@@ -20,6 +20,12 @@ import { apply } from "../lib/index.js";
 // ⚠️ 必须用 fileURLToPath：Windows 上 `new URL(...).pathname` 得到的是 "/D:/a/..."（带前导斜杠），
 // path.join 之后会拼成 "D:\\D:\\a\\..." → ENOENT。CI 的 Windows 专项就死在这条上。
 const HERE = path.dirname(fileURLToPath(import.meta.url));
+/**
+ * 版本号**从真实 manifest 派生**，不要写死。
+ * 写死会在每次发版时把这条无关用例弄红（2026-10-07 发 0.7.0 时实测踩到：断言里钉着 `/0\.6\./`），
+ * 而"按版本分桶"要的恰恰是**跟着真实版本走**。
+ */
+const PLUGIN_VERSION = JSON.parse(readFileSync(path.join(HERE, "..", "package.json"), "utf8")).version;
 const readOrEmpty = async (p) => { try { return await readFile(p, "utf8"); } catch { return ""; } };
 
 /** 假反馈服务：只收 /api/feedback 与 /api/feedback/:id/replies（与企业端契约同构）。 */
@@ -124,7 +130,7 @@ test("预览：把「将要发送的每一个字」摊开，且不含手机号
     const b = await getJson(`${base}/dsh-remote/diag/bundle`);
     assert.equal(b.ok, true);
     assert.match(b.title, /\[诊断\]/);
-    assert.match(b.title, /0\.6\./, "标题要带插件版本，便于后台按版本分桶");
+    assert.ok(b.title.includes(PLUGIN_VERSION), `标题要带插件版本（${PLUGIN_VERSION}），便于后台按版本分桶`);
     const all = b.summary + "\n" + b.parts.join("\n");
     for (const need of ["【阶段】", "【bridge】", "【上游】", "【配置目录】", env.relayDir]) {
       assert.ok(all.includes(need), `预览必须包含 ${need}（这些正是 2026-09-23 那次反馈里缺失的证据）`);

package/packages/dsh-remote-web/test/patch-activation.test.mjs

@@ -178,7 +178,12 @@ test("回归护栏：plugin 子命令必须把激活结果 return 给 setup（
   const call = src.match(/return convergePluginActivation\(profileDir[^\n]*/);
   assert.ok(call, "pluginCmd 必须 return convergePluginActivation(...) 的结果");
   // setup() 必须真的消费这个结果
-  assert.match(src, /pluginResult = await pluginCmd\(\[\]\)/, "setup 应保存 pluginCmd 的返回值");
+  // ⚠️ 2026-10-07 起参数从 `[]` 改为 `profileArgvOf(argv)`：桌面端要支持
+  //    `--profile desktop`，而 setup() 原本**不把 --profile 透传给 pluginCmd**，
+  //    导致"指定 desktop profile"被静默忽略、插件装进 web profile，
+  //    桌面端用户装完看不到面板且极难自查。这里是那次修复的护栏。
+  assert.match(src, /pluginResult = await pluginCmd\(profileArgvOf\(argv\)\)/, "setup 应保存 pluginCmd 的返回值");
+  assert.match(src, /function profileArgvOf\(/, "必须有 profileArgvOf 把 --profile 挑出来");
   assert.match(src, /pluginResult\.hotPatch/, "应据 hotPatch 判断是否走热挂载形态");
 });
 

package/packages/dsh-remote-web/test/uninstall-safety.test.mjs

@@ -407,7 +407,7 @@ test("清理助手：护栏命中时绝不删 profile（relayDir 误配成 profi
 test("安全：三处机密写入都要显式收紧 ACL（Windows 上 mode 0o600 无效）", () => {
   for (const [label, src, callSites] of [
     ["安装器", SETUP_SRC, [/hardenFile\(CONFIG_PATH\)/]],
-    ["插件半", INDEX_SRC, [/hardenFile\(configPathOf\(relayDir\)\)/, /hardenFile\(join\(relayDir, HARNESS_COOKIE_FILE\)\)/]],
+    ["插件半", INDEX_SRC, [/hardenFile\(configPathOf\(relayDir\)\)/]],
     ["bridge", fs.readFileSync(path.join(HERE, "..", "..", "..", "clients", "dsh-remote", "dsh-bridge.mjs"), "utf8"), [/hardenFile\(CONFIG_PATH\)/]],
   ]) {
     assert.ok(/function hardenFile\(file\)/.test(src), `${label} 应有 hardenFile`);
@@ -417,6 +417,18 @@ test("安全：三处机密写入都要显式收紧 ACL（Windows 上 mode 0o600
   }
 });
 
+test("安全：Cookie 的每个写入点都必须紧跟着 hardenFile（按 authority 分文件后不得漏收）", () => {
+  // 0.7.0-beta.2 把 Cookie 从「一份共享文件」改成「本宿主专属 + 旧版镜像」两份（见 P0-2 事故根治）。
+  // 写点从 1 个变 2 个 —— 逐个按变量名断言，漏收的那一份在 Windows 上就是 ACL 敞开状态。
+  const mint = INDEX_SRC.slice(INDEX_SRC.indexOf("async function mintHarnessCookie"), INDEX_SRC.indexOf("function readHarnessCookie"));
+  assert.ok(mint.length > 0, "应能定位到 mintHarnessCookie（找不到说明函数被改名，请同步本用例）");
+  const written = [...mint.matchAll(/writeFileSync\((\w+),/g)].map((m) => m[1]);
+  assert.ok(written.length >= 2, `mintHarnessCookie 应写两份 Cookie（本宿主专属 + 旧版镜像），实际 ${written.length}`);
+  for (const v of written) {
+    assert.ok(new RegExp(`hardenFile\\(${v}\\)`).test(mint), `Cookie 写入点 ${v} 之后必须调用 hardenFile 收紧 ACL`);
+  }
+});
+
 test("安全：hardenFile 在非 Windows 上只 chmod（不引入任何子进程，零副作用）", () => {
   const src = SETUP_SRC.slice(SETUP_SRC.indexOf("function hardenFile(file)"), SETUP_SRC.indexOf("// ---------- 配置读写 ----------"));
   const calls = [];

package/packages/dsh-remote-web/test/windows-compat.test.mjs

@@ -529,9 +529,10 @@ test("win32：登录任务经 wscript+VBS 隐藏启动（旧版会弹黑色空
       SETUP_SRC.indexOf("function winHiddenLauncherPath()"),
       SETUP_SRC.indexOf("function writeWindowsTask()"),
     );
-    const make = new Function("CONFIG_DIR", "fs", "path",
+    // SERVICE_NODE_ENV 是模块级常量：切片只含函数体，必须显式注入（真实 node 时为空对象）。
+    const make = new Function("CONFIG_DIR", "fs", "path", "SERVICE_NODE_ENV",
       `${src}\nreturn { winHiddenLauncherPath, writeWinHiddenLauncher };`);
-    const { winHiddenLauncherPath, writeWinHiddenLauncher } = make(dir, nodeFs, path);
+    const { winHiddenLauncherPath, writeWinHiddenLauncher } = make(dir, nodeFs, path, {});
 
     // 用「空格 + 中文 + 括号」的路径：Windows 上最常见的翻车形状
     const node = "C:\\Program Files\\nodejs\\node.exe";
@@ -554,6 +555,36 @@ test("win32：登录任务经 wscript+VBS 隐藏启动（旧版会弹黑色空
     assert.equal(value, `"${node}" "${setup}" run`, "VBS 字面量转义必须可往返（内嵌引号写成两个）");
     assert.ok(value.includes("My App (x64)"), "含空格/括号的路径必须完整保留");
     assert.ok(value.includes("张三"), "中文路径必须完整保留");
+    assert.ok(!/ELECTRON_RUN_AS_NODE/.test(vbs), "真实 node 时不得注入 Electron 变量（避免污染环境）");
+  } finally {
+    await rm(dir, { recursive: true, force: true });
+  }
+});
+
+test("win32：桌面端（Electron 解释器）自启动必须先设 ELECTRON_RUN_AS_NODE=1", async () => {
+  // 现场价值：DeepSeek Harness 桌面端宿主里 process.execPath 是 Electron 应用二进制。
+  // 不设这个变量，自启动拉起的不是 bridge 守护，而是**又开一个桌面端应用** ——
+  // 表现为「服务登记成功、进程秒退、bridge 永远起不来」，与 0.6.15 端口事故一样难排查。
+  const dir = await mkdtemp(path.join(os.tmpdir(), "dsh-win-electron-"));
+  try {
+    const src = SETUP_SRC.slice(
+      SETUP_SRC.indexOf("function winHiddenLauncherPath()"),
+      SETUP_SRC.indexOf("function writeWindowsTask()"),
+    );
+    const make = new Function("CONFIG_DIR", "fs", "path", "SERVICE_NODE_ENV",
+      `${src}\nreturn { writeWinHiddenLauncher };`);
+    const { writeWinHiddenLauncher } = make(dir, nodeFs, path, { ELECTRON_RUN_AS_NODE: "1" });
+
+    const node = "C:\\Program Files\\DeepSeek Harness\\DeepSeek Harness.exe";
+    const setup = path.join(dir, "dsh-setup.mjs");
+    const vbs = readFileSync(writeWinHiddenLauncher(node, setup), "utf8");
+    assert.match(vbs, /Environment\("PROCESS"\)\("ELECTRON_RUN_AS_NODE"\)\s*=\s*"1"/,
+      "必须先往进程环境写入 ELECTRON_RUN_AS_NODE=1，再 Run");
+    assert.ok(vbs.indexOf("ELECTRON_RUN_AS_NODE") < vbs.indexOf(".Run "),
+      "设置环境变量必须发生在 Run **之前**（否则白设）");
+    // 注入变量不得破坏原有的隐藏启动语义
+    const line = vbs.replace(/\r\n/g, "\n").split("\n").find((l) => l.includes(".Run "));
+    assert.ok(/,\s*0,\s*False\s*$/.test(line), `Run 仍必须是 (cmd, 0, False)，实际：${line}`);
   } finally {
     await rm(dir, { recursive: true, force: true });
   }