@bushizhanghan/dsh-publish-gate 0.9.0 → 0.10.0

package/CHANGELOG.md

@@ -1,5 +1,11 @@
 # Changelog
 
+## 0.10.0
+
+- `client-double-load` (warn): the client probe now evaluates the bundle a second time against the same
+  `window`, with a loader that throws on a repeated registration. A bundle inserted by two patches (its own,
+  plus any sidebar plugin that injects `dsh-client-modules`) aborted web boot this way. A window-level guard
+  makes it a no-op, which is what the recommended bundle shape now shows.
 ## 0.9.0
 
 - `checkChangelog()`: `changelog-missing-version` (warn), `changelog-no-versions` (warn), `no-changelog` (info).

package/README.md

@@ -124,6 +124,7 @@ dsh plugin --profile desktop add @bushizhanghan/dsh-publish-gate
 | `client-apply-throws` | `apply(ctx)` 抛 |
 | `client-no-registration` | `apply` 跑完一次服务都没调用——面板不会出现 |
 | `client-inject-missing-<service>` | `apply` 用了 `ctx.sidebarRightTabs` 但 `inject` 数组里没写它。cordis 不会把没声明的服务给你，于是 `ctx.sidebarRightTabs` 是 `undefined`，`apply` 一碰就抛（或静默什么都没注册） |
+| `client-double-load` | 同一个 bundle 被注入了两次（它自己的 patch 一次，注入 `dsh-client-modules` 的侧边栏插件再一次）。loader 按 id 建工厂，第二次注册直接抛错，整个 Web 启动被中断——不是这个面板没了，是所有面板都没了。用 `if (window.__xLoaded) return; window.__xLoaded = true` 挡一下 |
 | `client-parse-failed` | 文件有语法错误。**单独判**：`SyntaxError` 说明 bundle 本身是坏的，别的报错说明是沙箱不够真，两者别混为一谈 |
 | `client-unavailable` | 子进程没跑起来（warn）——这一层没跑，别把它当干净 |
 
@@ -216,7 +217,7 @@ dsh plugin --profile desktop add @bushizhanghan/dsh-publish-gate
 ## 开发
 
 ```sh
-node --test                      # 97 个引擎单测
+node --test                      # 99 个引擎单测
 node test/integration.test.mjs   # 27 条集成断言（9 个真实插件作 fixture）
 ```
 

package/lib/engine.js

@@ -1786,6 +1786,24 @@ if (!records.parseError) {
     else records.evalError = text
   }
 }
+// The same bundle can reach the page twice: its own patch inserts it, and any
+// sidebar-style plugin that injects dsh-client-modules inserts it again. The
+// loader keys factories by id and throws on the second registration, which
+// aborts web boot. Evaluate a second time against the same window and see
+// whether load() is called again; a window-level guard makes it a no-op.
+records.doubleLoad = null
+if (!records.parseError && records.loaded > 0) {
+  let second = 0
+  const first = globalThis.window.__ModuleLoader__.load
+  globalThis.window.__ModuleLoader__.load = (spec) => {
+    second += 1
+    throw new Error('duplicate factory registration: ' + String(spec && spec.id))
+  }
+  try { new Function(source)() } catch (e) {
+    records.doubleLoad = second > 0 ? String(e && e.message || e) : null
+  }
+  globalThis.window.__ModuleLoader__.load = first
+}
 process.stdout.write('\\n@@PG@@' + JSON.stringify(records))
 process.exit(0)
 `
@@ -1847,6 +1865,17 @@ export async function checkClientRuntime(dir, opts = {}) {
   if (r.applyError) {
     out.push(F('client-apply-throws', 'error', `apply() in ${target} throws: ${String(r.applyError).slice(0, 160)}`, 'Registration happens inside apply(); a throw there leaves the panel unregistered while the host half keeps working.', 'Wrap the failing registration, or fix the cause.'))
   }
+  if (r.doubleLoad) {
+    out.push(
+      F(
+        'client-double-load',
+        'warn',
+        `${target} registers a factory again when evaluated a second time (${String(r.doubleLoad).slice(0, 120)})`,
+        'A client bundle can reach the page twice: its own patch inserts it, and any sidebar-style plugin that injects dsh-client-modules inserts it again. The loader keys factories by id and throws on the second registration, which aborts web boot — the whole panel disappears, not just this one.',
+        'Guard the load call with a window-level flag: if (window.__myPluginLoaded) return; window.__myPluginLoaded = true.',
+      ),
+    )
+  }
   if (r.hasApply && !r.applyError && r.registrations.length === 0) {
     out.push(
       F(
@@ -2234,7 +2263,7 @@ const RULE_ORDER = [
   'missing-field', 'not-esm', 'files-omits-lib', 'files-omits-patch', 'no-exports', 'client-without-export', 'client-without-platform',
   'entry-path-missing', 'bundle-patch-not-declared', 'no-publish-config-access', 'private-field-set', 'publish-config-registry',
   'client-file-missing', 'client-no-module-loader', 'client-parse-failed', 'client-load-not-called',
-  'client-factory-throws', 'client-no-apply', 'client-apply-throws', 'client-no-registration',
+  'client-factory-throws', 'client-no-apply', 'client-apply-throws', 'client-no-registration', 'client-double-load',
   'client-inject-missing-slots', 'client-inject-missing-sidebarRightTabs', 'client-unavailable',
   'no-patch-file', 'no-insert-row', 'insert-missing-name', 'insert-name-mismatch',
   'patch-name-unresolvable', 'no-name-export', 'no-apply-export', 'no-inject', 'require-in-esm',

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bushizhanghan/dsh-publish-gate",
-  "version": "0.9.0",
+  "version": "0.10.0",
   "description": "Pre-publish gate for DSH plugins: manifest, dependencies, artifact, runtime, browser half, traceability and market readiness — plus an input fuzzer that drives every declared parameter with the wrong type. 发布前的七层门禁 + 入参模糊测试。",
   "type": "module",
   "main": "lib/index.js",