@mrrisega/dsh-remote 0.6.4-beta.10 → 0.6.4-beta.11

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.10",
+  "version": "0.6.4-beta.11",
   "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

@@ -1753,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.10";
+const PLUGIN_VERSION = "0.6.4-beta.11";
 const UPDATE_LOG = ".dsh-update.log";
 const UPDATE_MARKER = ".dsh-update-running";
 

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

@@ -1,6 +1,6 @@
 {
   "name": "dsh-remote-web",
-  "version": "0.6.4-beta.10",
+  "version": "0.6.4-beta.11",
   "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/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); }
+});