dsh-remote-agent 0.9.0 → 0.9.2

package/scripts/check-doc-drift.mjs

@@ -929,37 +929,55 @@ check('65', 'eslint 严格档：lint script + 配置文件 + CI job 三件套齐
     'lib/env-check.js（可达性/inject/roots/role + fix + 不代下载边界）')
 }
 
-/* ── 79~80. 开箱即用（2026-10-17）──
+/* ── 79~80. 开箱即用（2026-10-17，2026-10-18 修订）──
  *
  * 产品标准：用户装完就该能用，不该读文档、改 YAML、猜字段。
  *
- * 此前三道门槛：
- *   1. `role` 默认 '' → 两台机器都要操作一次
- *   2. `webserver` 没绑 0.0.0.0 → 副机永远连不上（且完全静默）
- *   3. 副机表单 4 个字段 → 小白不知道节点 ID / 工作区填什么
- *
  * 核心设计：`role` 默认 **auto**，按 `hubUrl` 推断。
  *   · 默认 hub → 副机装完以 hub 身份启动，注册无鉴权端点（**攻击面**）
  *   · 默认 ''  → 两台都要操作，不是开箱即用
  *   · 默认 auto → 主机零操作、副机贴完配对码零操作，且不给副机开错端点
+ *
+ * ⚠ **2026-10-18 事故修订**：79 号断言原来**要求**包内 patch 绑
+ * `host: '0.0.0.0'` —— 那正是导致 **DSH 启动崩溃**的写法。新版
+ * `dsh-host-webserver` 在 schema 校验阶段就硬拒绝通配地址：
+ *
+ *     host "0.0.0.0" is an unspecified (wildcard) address, which is not
+ *     supported: binding every interface would expose remote code execution
+ *     to the network; bind one concrete IPv4 or IPv6 address ...
+ *
+ * 后果：webserver（required）激活失败 → webServer 服务缺失 → 至少 6 个插件
+ * 排队等它 → host 启动失败（实测 10 份 crash 日志）。
+ *
+ * 所以断言方向**反过来**：现在要求包内 patch **不碰** webserver 的任何字段，
+ * 且所有提示文案都不得建议 0.0.0.0。
  */
 {
   const indexSrc = read('lib/index.js')
   const patchSrc = read('cordis.patch.yml')
   const clientSrc3 = read('lib/client.js')
   const setupSrc = read('lib/setup-routes.js')
-
-  check('79', '开箱即用：role 默认 auto + 包内 patch 绑 0.0.0.0:19387',
+  const envSrc = read('lib/env-check.js')
+  const hubSrc = read('lib/hub/index.js')
+
+  // ⚠ 只扫**非注释行**：注释里必须保留那次事故的原文（含 `0.0.0.0` 与
+  // `- id: webserver` 示例），那是给后来者的警示 —— 不该被断言判违规。
+  // 这一点和 L3 断言踩过的坑同源（扫全文会命中"正确留档的注释"）。
+  const patchCode = patchSrc
+    .split('\n')
+    .filter((l) => !/^\s*#/.test(l))
+    .join('\n')
+
+  check('79', '开箱即用：role 默认 auto；且包内 patch **不得**绑 0.0.0.0（会让 DSH 崩溃）',
     /\.default\('auto'\)/.test(indexSrc)
     && /ROLE\.AUTO/.test(indexSrc)
     && /config\.hubUrl/.test(indexSrc)
-    // 包内 patch 必须 override webserver（否则副机连不上）
-    && /^- id: webserver$/m.test(patchSrc)
-    && /host:\s*'0\.0\.0\.0'/.test(patchSrc)
-    && /port:\s*19387/.test(patchSrc)
-    // 必须披露安全影响
-    && /暴露/.test(patchSrc),
-    'lib/index.js（auto 推断）+ cordis.patch.yml（webserver 0.0.0.0:19387 + 安全披露）')
+    // ⚠ 关键：包内 patch 的**生效行**不得 override webserver 的 host
+    && !/^- id: webserver$/m.test(patchCode)
+    && !/host:\s*'0\.0\.0\.0'/.test(patchCode)
+    // 必须把这个坑写进注释（防止后来者又"帮用户改好"）
+    && /启动崩溃/.test(patchSrc),
+    'lib/index.js（auto 推断）+ cordis.patch.yml（生效行不碰 webserver + 事故留档）')
 
   check('80', '开箱即用：副机表单简化 + 工作区自动创建 + 首次自动体检',
     // 工作区预填
@@ -972,6 +990,19 @@ check('65', 'eslint 严格档：lint script + 配置文件 + CI job 三件套齐
     && /mkdirSync\(r, \{ recursive: true \}\)/.test(setupSrc)
     && /warnings: applyWarnings/.test(setupSrc),
     'lib/client.js（预填/折叠/自动体检）+ lib/setup-routes.js（自动建目录）')
+
+  // ── 79b：**所有面向用户的修法文案**都不得建议 0.0.0.0 ──────────────
+  //
+  // 这是本次事故最有价值的一条守卫：代码自己不能犯那个错，提示文案也不能。
+  // 检查的是字符串字面量（`host: '0.0.0.0'` 这种形态），不是注释里的说明。
+  check('79b', '所有修法文案都不得建议 host 0.0.0.0（会让 DSH 启动崩溃）',
+    !/host:\s*'0\.0\.0\.0'/.test(clientSrc3)
+    && !/host:\s*'0\.0\.0\.0'/.test(envSrc)
+    && !/加 host:\s*'0\.0\.0\.0'/.test(hubSrc)
+    // 但要**明确警告不能用**它（否则用户可能自己想到这个"标准做法"）
+    && /不能用 0\.0\.0\.0/.test(envSrc)
+    && /不能用 0\.0\.0\.0/.test(clientSrc3),
+    'lib/client.js + lib/env-check.js + lib/hub/index.js（修法不含 0.0.0.0，且有明确警告）')
 }
 
 /* ── 81~82. 工作区 SMB 共享（2026-10-17）──
@@ -1052,6 +1083,52 @@ check('65', 'eslint 严格档：lint script + 配置文件 + CI job 三件套齐
     + ` vs README.md（声明 ${declared === null ? '未找到' : declared} 个）`)
 }
 
+/* ── 84~85. v0.9.0 审查的两个 High：**接线断链**（2026-10-17）──
+ *
+ * 这两个 bug 的共同点：**模块本身完全正确，装配进系统时接线断了**。
+ *
+ *   H1  env-check.js 的 24 条单测全绿，但 /env-check 路由传的是**原始
+ *       config.role**（'auto'）→ 五项检查全跳过、两项误报 → 对 100% 默认
+ *       安装用户失效，且首次自动体检让它在每个新用户面前弹出。
+ *   H2  share-service.js 的 17 条单测全绿，但面板恒发 dir: '' → 副机按设计
+ *       拒绝 → SMB 共享的唯一主动作 100% 失败。
+ *
+ * 单测证明不了"模块被装配进系统时的形状"。这两条断言锚在接线处。
+ */
+{
+  const uiSrc4 = read('lib/hub/ui-routes.js')
+  const clientSrc5 = read('lib/client.js')
+  const setupBin = read('bin/dsh-remote-setup.js')
+
+  check('84', 'env-check 路由传解析后角色（v0.9.0 审查 H1：否则 auto 判成非 hub）',
+    // 反例：把原始 config.role 直接传给 envChecks
+    (() => {
+      const i = uiSrc4.indexOf('/env-check`')
+      if (i < 0) return false
+      const seg = uiSrc4.slice(i, i + 900)
+      // 不得出现 `role: typeof config.role === 'string' ? config.role`
+      if (/role:\s*typeof config\.role === 'string'\s*\?\s*config\.role/.test(seg)) return false
+      // 必须用 resolveRole（或字面量 'hub'）
+      return /resolveRole/.test(seg)
+    })(),
+    'lib/hub/ui-routes.js（role: resolveRole(config) 而非原始 config.role）')
+
+  check('85', 'SMB 面板发真实 dir + CLI doctor 有 loopback 检查（H2 / M1）',
+    // H2：不得写死 dir: ''
+    (() => {
+      const i = clientSrc5.indexOf('function doPlan()')
+      if (i < 0) return false
+      const seg = clientSrc5.slice(i, i + 900)
+      return !/dir:\s*''/.test(seg) && /dir:\s*shareDir/.test(seg)
+    })()
+    // H2：必须有目录选择 UI（数据源 node.roots）
+    && /nodeRoots/.test(clientSrc5)
+    // M1：CLI 必须调 envChecks（否则没有 loopback 诊断）
+    && /envChecks/.test(setupBin)
+    && /isLoopbackOrigin/.test(setupBin),
+    'lib/client.js（dir: shareDir + nodeRoots UI）+ bin/dsh-remote-setup.js（envChecks）')
+}
+
 /* ── 输出 ── */
 const failed = results.filter((r) => !r.ok)
 if (process.argv.includes('--json')) {

package/scripts/diag-worker.ps1

@@ -0,0 +1,148 @@
+# dsh-remote-agent worker diagnostic
+#
+# Read-only: changes nothing. Paste into PowerShell on the WORKER machine:
+#     powershell -ExecutionPolicy Bypass -File .\diag-worker.ps1
+
+$ErrorActionPreference = 'SilentlyContinue'
+$profiles = Join-Path $env:USERPROFILE '.dsh\profiles'
+
+function Line { param($t) Write-Host $t }
+
+Write-Host ''
+Write-Host '=========================================='
+Write-Host ' dsh-remote-agent WORKER DIAGNOSTIC'
+Write-Host '=========================================='
+Write-Host ''
+
+# ---- 1. profile dirs ----
+Write-Host '[1] DSH profile directories'
+if (Test-Path $profiles) {
+    Get-ChildItem $profiles -Directory | ForEach-Object {
+        $patch = Join-Path $_.FullName 'cordis.patch.yml'
+        if (Test-Path $patch) {
+            $f = Get-Item $patch
+            Line ("    {0}  patch={1}B  mtime={2}" -f $_.Name, $f.Length, $f.LastWriteTime.ToString('MM-dd HH:mm:ss'))
+        } else {
+            Line ("    {0}  (no cordis.patch.yml)" -f $_.Name)
+        }
+    }
+    if ($env:DSH_PROFILE) {
+        Line ("    env DSH_PROFILE = " + $env:DSH_PROFILE)
+    } else {
+        Line '    env DSH_PROFILE = (not set -> code defaults to "desktop")'
+    }
+} else {
+    Line '    ERROR: no profile directory'
+}
+Write-Host ''
+
+# ---- 2. remote-agent config per profile ----
+Write-Host '[2] remote-agent config in each profile'
+if (Test-Path $profiles) {
+    Get-ChildItem $profiles -Directory | ForEach-Object {
+        $patch = Join-Path $_.FullName 'cordis.patch.yml'
+        if (-not (Test-Path $patch)) { return }
+        $c = Get-Content $patch -Raw
+        if ($c -notmatch 'remote-agent') { return }
+
+        $roleM   = [regex]::Match($c, '(?m)^\s*role:\s*(\S+)')
+        $injectM = [regex]::Match($c, '(?m)^\s*inject:\s*(.+)$')
+        $hubM    = [regex]::Match($c, '(?m)^\s*hubUrl:\s*(\S+)')
+        $wsM     = [regex]::Match($c, '(?ms)- id:\s*webserver.*?host:\s*(\S+)')
+        $tokM    = [regex]::Match($c, '(?m)^\s*nodeToken:\s*(\S+)')
+
+        Line ('  --- ' + $_.Name + ' ---')
+        if ($roleM.Success)   { Line ('      role      = ' + $roleM.Groups[1].Value) }   else { Line '      role      = (NOT SET -> package default)' }
+        if ($injectM.Success) { Line ('      inject    = ' + $injectM.Groups[1].Value.Trim()) } else { Line '      inject    = (not set)' }
+        if ($hubM.Success)    { Line ('      hubUrl    = ' + $hubM.Groups[1].Value) }    else { Line '      hubUrl    = (not set)' }
+        if ($wsM.Success)     { Line ('      webserver = host=' + $wsM.Groups[1].Value) } else { Line '      webserver = (no webserver section)' }
+        if ($tokM.Success)    { Line '      nodeToken = (set)' } else { Line '      nodeToken = (not set)' }
+    }
+} else {
+    Line '    (skipped)'
+}
+Write-Host ''
+
+# ---- 3. installed plugin version ----
+Write-Host '[3] installed dsh-remote-agent version'
+$found = $false
+if (Test-Path $profiles) {
+    Get-ChildItem $profiles -Directory | ForEach-Object {
+        $nm = Join-Path $_.FullName 'node_modules\dsh-remote-agent'
+        $pj = Join-Path $nm 'package.json'
+        if (Test-Path $pj) {
+            $script:found = $true
+            $v = (Get-Content $pj -Raw | ConvertFrom-Json).version
+            $hasEnvCheck = Test-Path (Join-Path $nm 'lib\env-check.js')
+            $hasShare    = Test-Path (Join-Path $nm 'lib\node\share-service.js')
+            $pkgPatch    = Join-Path $nm 'cordis.patch.yml'
+            $pkgRole = ''
+            $hasWs = $false
+            if (Test-Path $pkgPatch) {
+                $pc = Get-Content $pkgPatch -Raw
+                $m = [regex]::Match($pc, '(?m)^\s*role:\s*(\S+)')
+                if ($m.Success) { $pkgRole = $m.Groups[1].Value }
+                $hasWs = $pc -match '(?m)^- id:\s*webserver'
+            }
+            Line ('  ' + $_.Name + ': version ' + $v)
+            Line ('    has lib/env-check.js (0.9+)     : ' + $hasEnvCheck)
+            Line ('    has lib/node/share-service.js   : ' + $hasShare)
+            if ($pkgRole -ne '') { Line ('    package default role            : ' + $pkgRole) } else { Line '    package default role            : (not found)' }
+            Line ('    package has webserver binding   : ' + $hasWs)
+            if (-not $hasEnvCheck) {
+                Line '    WARNING: this is pre-0.9, no "works out of the box"'
+            }
+        }
+    }
+}
+if (-not $script:found) {
+    Line '  ERROR: dsh-remote-agent not found under any profile'
+}
+Write-Host ''
+
+# ---- 4. panel endpoints ----
+Write-Host '[4] panel endpoint probe (is the plugin loaded at all?)'
+$ports = @()
+$dshPids = (Get-Process 'DeepSeek Harness').Id
+if ($dshPids) {
+    $ports += (Get-NetTCPConnection -State Listen | Where-Object { $dshPids -contains $_.OwningProcess } | ForEach-Object { $_.LocalPort })
+}
+$ports += 19387
+$ports = $ports | Sort-Object -Unique
+foreach ($port in $ports) {
+    foreach ($path in @('/api/remote-agent/setup/state', '/api/remote-agent/status')) {
+        $url = 'http://127.0.0.1:' + $port + $path
+        try {
+            $r = Invoke-WebRequest $url -UseBasicParsing -TimeoutSec 3
+            $body = $r.Content
+            if ($body.Length -gt 240) { $body = $body.Substring(0, 240) + '...' }
+            Line ('    [' + $port + '] ' + $path + ' -> HTTP ' + $r.StatusCode)
+            Line ('           ' + $body)
+        } catch {
+            $code = $_.Exception.Response.StatusCode.value__
+            if ($code) {
+                Line ('    [' + $port + '] ' + $path + ' -> HTTP ' + $code)
+            } else {
+                Line ('    [' + $port + '] ' + $path + ' -> no listener')
+            }
+        }
+    }
+}
+Write-Host ''
+
+# ---- 5. recent backups ----
+Write-Host '[5] recent config backups (was the file rewritten?)'
+if (Test-Path $profiles) {
+    Get-ChildItem $profiles -Recurse -File -Filter 'cordis.patch.yml*' |
+        Sort-Object LastWriteTime -Descending |
+        Select-Object -First 8 |
+        ForEach-Object {
+            Line ('    ' + $_.LastWriteTime.ToString('MM-dd HH:mm:ss') + '  ' + $_.Length + 'B  ' + $_.Name)
+        }
+}
+Write-Host ''
+
+Write-Host '=========================================='
+Write-Host ' END. Paste the whole output back.'
+Write-Host '=========================================='
+Write-Host ''