@mrrisega/dsh-remote 0.6.4-beta.9 → 0.6.4

package/dsh-setup.mjs

@@ -921,10 +921,26 @@ function writePatchAtomic(patchFile, text) {
 /**
  * 原子地写入插件 patch 激活行；写后**重新解析校验**，不合法立即回滚。
  * 返回 {ok, changed, reason?, basePatch}
+ *
+ * ⚠️ 硬约束：**本函数绝不能在插件已在 dsh.profile.bundles 时写入 patch 行**。
+ * 两个激活点同时存在时，dsh web 启动即 `duplicate loader entry id: dsh-remote-web`
+ * 并导致「plugin tree failed to load」——整个插件树加载失败（实测日志）。
+ * 因此所有 return 之前都要保证：要么只有 patch 行、要么只有 bundles 条目。
  */
 function ensurePatchActivation(patchFile, pluginLocalDir, pkgFile) {
   const original = fs.readFileSync(patchFile, "utf8");
-  // 幂等：已经是唯一激活点就什么都不做（但要确保 bundles 里没有重复激活）
+  const pkgBundles = () => {
+    try {
+      const pkg = JSON.parse(fs.readFileSync(pkgFile, "utf8"));
+      return (pkg.dsh && pkg.dsh.profile && Array.isArray(pkg.dsh.profile.bundles))
+        ? pkg.dsh.profile.bundles.filter((b) => PLUGIN_ALL_IDS.includes(b)) : [];
+    } catch { return []; }
+  };
+  // 基座就是「插件已在 bundles」→ 保持 bundles 形态（不写 patch 行），避免两个激活点并存。
+  // 这一路等价于插件市场安装形态：需要重启 dsh web 才装载。
+  if (pkgBundles().length) {
+    return { ok: false, reason: "插件已由 dsh.profile.bundles 声明（保持单一激活点，不写 patch 行）", changed: false, bundleOnly: true, basePatch: original };
+  }
   const base = validatePatchBase(original);
   if (!base.ok) return { ok: false, reason: base.reason, changed: false, basePatch: original };
   const hasOurRow = base.insertedIds.some((id) => PLUGIN_ALL_IDS.includes(id));
@@ -972,6 +988,12 @@ function ensurePatchActivation(patchFile, pluginLocalDir, pkgFile) {
     return { ok: false, reason: "写入后校验失败，已回滚", changed: false, basePatch: original };
   }
   removeBundleEntry(pkgFile); // 单一激活点：bundles 里不能再有本插件
+  // 双保险：上面那次移除若没生效（并发写 package.json / 解析失败），**回滚本次 patch 行**，
+  // 宁可退回 bundles 形态（需重启）也绝不留"两处激活"→ dsh web 启动会 duplicate id 崩掉。
+  if (pkgBundles().length) {
+    writePatchAtomic(patchFile, original);
+    return { ok: false, reason: "bundles 条目未能移除，已回滚 patch 行以免重复激活", changed: false, bundleOnly: true, basePatch: original };
+  }
   return { ok: true, changed: true, basePatch: original };
 }
 
@@ -1137,10 +1159,14 @@ function convergePluginActivation(profileDir, pkgFile, patchFile, pluginDir, pat
       console.log(`✅ 插件已就绪: ${pluginLocalDir}`);
       return { hotPatch: true, changed: r.changed, basePatch: r.basePatch };
     }
-    // patch 行写不进去(文件形态异常)→ 回退 bundles(需重启),保证功能可用
-    console.warn(`⚠️ 无法写入 patch 激活行（${r.reason}），改为 bundles 形态（需重启 dsh web 生效）`);
-    ensureBundleEntry(pkgFile);
-    console.log(`✅ 插件已就绪: ${pluginLocalDir}（bundles 形态）`);
+    // patch 行不能写(文件形态异常,或插件已由 bundles 声明)→ 回退 bundles(需重启),保证功能可用。
+    // 这条路的输出要如实说明"装完要重启一次",不能让人以为已经热加载了。
+    if (r.bundleOnly) {
+      console.log("ℹ 该插件已由 dsh.profile.bundles 声明（保持单一激活点，不重复写入 patch 行）");
+    } else {
+      console.warn(`⚠️ 无法写入 patch 激活行（${r.reason}），改为 bundles 形态（需重启 dsh web 生效）`);
+      ensureBundleEntry(pkgFile); // 回退激活点：插件仍可用，代价是必须重启一次
+    }
     return { hotPatch: false, changed: true, basePatch: patch };
   }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mrrisega/dsh-remote",
-  "version": "0.6.4-beta.9",
+  "version": "0.6.4",
   "description": "手机远程控制 DeepSeek Harness · Remote control DeepSeek Harness (dsh web) from any phone browser — 100% 全功能 App 级体验：发消息、看工具执行、审批权限、改设置、管凭据，含特权操作，免内网穿透。一条命令安装 npx @mrrisega/dsh-remote。Mobile remote control for DSH, self-host or SaaS, no server needed on LAN.",
   "keywords": [
     "deepseek-harness",

package/packages/dsh-remote-web/lib/index.js

@@ -887,7 +887,15 @@ function pluginInstalledAfterBoot() {
     const pkgFile = join(dirname(dirname(here)), "package.json");
     const mtimeMs = statSync(pkgFile).mtimeMs;
     const bootMs = Date.now() - process.uptime() * 1000;
-    return mtimeMs > bootMs + 2000;
+    if (!(mtimeMs > bootMs + 2000)) return false;
+    // ⚠️ 只看时间戳会**误报**:安装器补装/在线更新会把同样的文件重写一遍(哪怕是同一个版本),
+    // 时间戳变新 → 被当成"有新插件要装载" → 弹出"需要重启/刷新"横幅(用户实测踩到:
+    // 自愈补装把 profile 副本重写了一次,运行中的旧代码就写下了"需要重启")。
+    // 真正该提示用户的只有一件事:**磁盘上的插件版本与本进程装载的版本不同**。
+    // 版本相同 → 运行中的代码就是目标版本,什么都不用做。
+    const diskVersion = String(JSON.parse(readFileSync(pkgFile, "utf8")).version || "");
+    const runningVersion = String(PLUGIN_VERSION || "");
+    return Boolean(diskVersion && runningVersion && diskVersion !== runningVersion);
   } catch { return false; }
 }
 
@@ -1745,7 +1753,7 @@ const PLUGIN_ID = "dsh-remote-web";
 const PLUGIN_LEGACY_IDS = ["dsh-remote-ui"];
 const PLUGIN_ALL_IDS = [PLUGIN_ID, ...PLUGIN_LEGACY_IDS];
 /** 插件自身发布版本（与 dsh-remote 根包同步递增）。 */
-const PLUGIN_VERSION = "0.6.4-beta.9";
+const PLUGIN_VERSION = "0.6.4";
 const UPDATE_LOG = ".dsh-update.log";
 const UPDATE_MARKER = ".dsh-update-running";
 
@@ -1770,6 +1778,44 @@ async function npmLatestVersion() {
   return "";
 }
 
+/**
+ * 语义化版本比较（只处理本仓库用到的形态：`X.Y.Z` 与 `X.Y.Z-<pre>.<n>`）。
+ * 为什么需要它：面板判断"有没有新版本"不能只用 `latest !== current` ——
+ * 预设版之间（beta.10 → beta.11）无法比较大小，而且预设版在 semver 里**低于**同号正式版
+ * （0.6.4 < 0.6.4），朴素不等判断会把"已经装了预设版"误判成落后。
+ * @returns {number} a>b → 1；a<b → -1；相等 → 0
+ */
+function compareVersions(a, b) {
+  const parse = (v) => {
+    const m = /^(\d+)\.(\d+)\.(\d+)(?:-([0-9A-Za-z.-]+))?/.exec(String(v || "").trim());
+    if (!m) return null;
+    return { nums: [Number(m[1]), Number(m[2]), Number(m[3])], pre: m[4] ? m[4].split(".") : null };
+  };
+  const x = parse(a);
+  const y = parse(b);
+  if (!x || !y) return null; // 无法解析 → 交给调用方保守处理
+  for (let i = 0; i < 3; i += 1) {
+    if (x.nums[i] !== y.nums[i]) return x.nums[i] > y.nums[i] ? 1 : -1;
+  }
+  if (!x.pre && !y.pre) return 0;
+  if (!x.pre) return 1;   // 有预设版 < 无预设版（semver §11）
+  if (!y.pre) return -1;
+  const len = Math.max(x.pre.length, y.pre.length);
+  for (let i = 0; i < len; i += 1) {
+    const p1 = x.pre[i];
+    const p2 = y.pre[i];
+    if (p1 === undefined) return -1;
+    if (p2 === undefined) return 1;
+    const n1 = /^\d+$/.test(p1) ? Number(p1) : null;
+    const n2 = /^\d+$/.test(p2) ? Number(p2) : null;
+    if (n1 !== null && n2 !== null) { if (n1 !== n2) return n1 > n2 ? 1 : -1; continue; }
+    if (n1 !== null) return -1; // 数字标识符 < 字母标识符
+    if (n2 !== null) return 1;
+    if (p1 !== p2) return p1 > p2 ? 1 : -1;
+  }
+  return 0;
+}
+
 /** 以 detached 子进程执行 `npx --yes <UPDATE_SPEC>`（env 可覆盖 npm 源/更新通道）。 */
 function spawnUpdater(relayDir, extraEnv) {
   const log = join(relayDir, UPDATE_LOG);
@@ -2848,7 +2894,12 @@ function registerRoutes(ctx, relayDir) {
       handler: async (_req, res) => {
         const latest = await npmLatestVersion();
         const current = PLUGIN_VERSION;
-        sendJson(res, 200, { ok: true, current, latest, outdated: !!latest && latest !== current });
+        // 正规比较：注册表版本**严格大于**本地版本才算有更新。
+        // 这样 0.6.4 < 0.6.4（正式版发布后预设版会正确提示升级），
+        // 而 0.6.4-beta.10 < 0.6.4 也能正确判断；解析不了时退回朴素不等（保守提示）。
+        const cmp = compareVersions(latest, current);
+        const outdated = cmp === null ? Boolean(latest && latest !== current) : cmp > 0;
+        sendJson(res, 200, { ok: true, current, latest, outdated });
       },
     },
     {

package/packages/dsh-remote-web/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dsh-remote-web",
-  "version": "0.6.4-beta.9",
+  "version": "0.6.4",
   "description": "公网远程控制 DeepSeek Harness（dsh web）：安装即得专属加密地址，人在外面也能用手机访问电脑上的 dsh——无需同一局域网/WiFi、无需公网 IP 与内网穿透，全程加密；手机端 100% 还原电脑体验（对话/工具/审批/设置）。技术用户可选自建服务，流量走自己的服务器。Remote control DeepSeek Harness (dsh web) from anywhere over the public internet — install, get an encrypted URL, use it from your phone on any network (dual-half cordis plugin: Settings panel + same-origin /dsh-remote routes). (2026-09 由 dsh-remote-ui 更名 / renamed from dsh-remote-ui; dsh-remote 的 dsh web 插件半，不是纯 UI/皮肤插件)",
   "type": "module",
   "main": "lib/index.js",

package/packages/dsh-remote-web/test/harness-restart.test.mjs

@@ -140,6 +140,17 @@ test("首次安装（运行环境在本进程内补齐）→ **不再**提示重
   } finally { await env.restore(); }
 });
 
+test("补装把**同版本**插件重写一遍 → 不得提示（用户实测的误报）", async () => {
+  // 事故复盘:安装器/自愈补装会把 profile 副本整目录重写,即使是**同一个版本**。
+  // 旧实现只看"插件文件 mtime 晚于进程启动"→ 误判成"有新插件要装载"→ 弹出需要重启/刷新的横幅,
+  // 而用户刷新后横幅还在(因为第二次补装又写了一遍)、最后靠重启才消失。
+  // 现在判的是「磁盘版本 != 运行版本」,版本相同就什么都不提示。
+  const src = readFileSync(new URL("../lib/index.js", import.meta.url), "utf8");
+  const fn = src.slice(src.indexOf("function pluginInstalledAfterBoot("), src.indexOf("/** 本进程的启动命令"));
+  assert.match(fn, /diskVersion !== runningVersion/, "必须比较版本,而不是只看时间戳");
+  assert.match(fn, /PLUGIN_VERSION/, "必须用本进程装载的版本号做比较");
+});
+
 test("插件文件晚于本进程启动 → 提示「刷新页面」，且**不**自动重启", async () => {
   const env = await setup();
   try {

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

@@ -181,3 +181,30 @@ test("回归护栏：plugin 子命令必须把激活结果 return 给 setup（
   assert.match(src, /pluginResult = await pluginCmd\(\[\]\)/, "setup 应保存 pluginCmd 的返回值");
   assert.match(src, /pluginResult\.hotPatch/, "应据 hotPatch 判断是否走热挂载形态");
 });
+
+test("硬约束：插件已在 bundles 时**绝不**写 patch 行（两个激活点 = duplicate id 崩溃）", () => {
+  // 事故复盘：patch 行与插件自带 bundle patch 同时生效时，dsh web 启动即
+  //   TypeError: duplicate loader entry id: dsh-remote-web
+  //   Error: dsh: plugin tree failed to load: failed to apply loader entry include (cordis:include)
+  // 整个插件树加载失败（实测生产日志）。所以这两条激活路径必须互斥。
+  const profile = makeProfile({ bundles: [...PLUGIN_BUNDLES, "dsh-remote-web"] });
+  const relay = join(profile, "relay");
+  try {
+    const out = installPlugin(profile, relay);
+    assert.match(out, /已由 dsh\.profile\.bundles 声明/, "应识别并保持 bundles 形态");
+    assert.equal((readPatch(profile).match(/- id: dsh-remote-web/g) || []).length, 0,
+      "bundles 已声明时不得再写 patch 行（否则重复 ID）");
+    assert.ok(readBundles(profile).includes("dsh-remote-web"), "bundles 形态应保持");
+  } finally { cleanup(profile); }
+});
+
+test("硬约束：写入 patch 行后 bundles 必须被清空（单一激活点双向收口）", () => {
+  const profile = makeProfile({ bundles: [...PLUGIN_BUNDLES, "dsh-remote-web"] });
+  const relay = join(profile, "relay");
+  try {
+    installPlugin(profile, relay);
+    const patchRows = (readPatch(profile).match(/- id: dsh-remote-web/g) || []).length;
+    const inBundles = readBundles(profile).filter((b) => b === "dsh-remote-web").length;
+    assert.equal(patchRows + inBundles, 1, `激活点必须恰好一个：patch=${patchRows} bundles=${inBundles}`);
+  } finally { cleanup(profile); }
+});

package/packages/dsh-remote-web/test/setup-output.test.mjs

@@ -109,3 +109,28 @@ test("安装输出:服务状态字符串仍是插件面板可解析的稳定契
   assert.ok(/\? `✅ 运行中/.test(src), "running 口径保留(带 pid)");
   assert.ok(/`未运行 \(\$\{st\.detail \|\| st\.status\}\)`/.test(src), "未运行口径保留(带原因)");
 });
+
+test("版本比较：预设版 < 同号正式版；beta.10 < beta.11（更新提示不能误判）", () => {
+  // 事故背景：面板原本用 `latest !== current` 判断"有没有新版本"，于是
+  // ① 0.6.4-beta.11 与 0.6.4-beta.10 无法比大小；② 正式版 0.6.4 发布后，
+  // 装了 0.6.4-beta.11 的人反而被当成"落后"（其实预设版 semver 低于正式版）。
+  const src = readFileSync(join(HERE, "..", "lib", "index.js"), "utf8");
+  const fnSrc = src.slice(src.indexOf("function compareVersions(a, b)"), src.indexOf("/** 以 detached 子进程执行"));
+  const compare = new Function(`${fnSrc}; return compareVersions;`)();
+  // semver §11：预设版低于同号正式版
+  assert.equal(compare("0.6.4-beta.11", "0.6.4"), -1, "预设版应低于同号正式版");
+  assert.equal(compare("0.6.4", "0.6.4-beta.11"), 1, "正式版应高于同号预设版");
+  // 预设版之间按数字标识符比较（不是字符串比较）
+  assert.equal(compare("0.6.4-beta.11", "0.6.4-beta.10"), 1, "beta.11 > beta.10");
+  assert.equal(compare("0.6.4-beta.10", "0.6.4-beta.9"), 1, "beta.10 > beta.9");
+  assert.equal(compare("0.6.4-beta.10", "0.6.4-beta.10"), 0);
+  // 正式版之间
+  assert.equal(compare("0.6.5", "0.6.4"), 1);
+  assert.equal(compare("0.7.0", "0.6.9"), 1);
+  assert.equal(compare("0.6.4", "0.6.4"), 0);
+  // 解析不了 → null（调用方退回保守判断）
+  assert.equal(compare("weird", "0.6.4"), null);
+  // 路由必须用严格大于判断 outdated
+  assert.match(src, /const cmp = compareVersions\(latest, current\)/, "update-check 应用 semver 比较");
+  assert.match(src, /cmp === null \? Boolean\(latest && latest !== current\) : cmp > 0/);
+});