dsh-remote-agent 0.9.1 → 0.9.3

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）──
@@ -1098,6 +1129,45 @@ check('65', 'eslint 严格档：lint script + 配置文件 + CI job 三件套齐
     'lib/client.js（dir: shareDir + nodeRoots UI）+ bin/dsh-remote-setup.js（envChecks）')
 }
 
+/* ── 86~87. 独立监听端口（2026-10-18，架构级修复）──
+ *
+ * 副机端点原来注册在 `ctx.webServer` 上 → 副机必须访问 DSH 的 webserver 端口。
+ * 而那个端口：
+ *   · **DSH 每次启动重写 profile patch**，把写进去的 `webserver` 段删掉
+ *     （实测：3962B/15 条 → 661B/3 条）→ 绑回 127.0.0.1 → 副机永远连不上
+ *   · `webServer` 激活即 listen，没有 rebind 方法
+ *   · 新版硬拒绝 0.0.0.0（会让 DSH 启动崩溃，踩过 10 份 crash 日志）
+ *
+ * 所以插件自己开一个 node:http server 服务节点端点，完全绕开 DSH 的 webserver。
+ * 这两条断言钉住"这条通道存在且不依赖 webServer"。
+ */
+{
+  const hubSrc2 = read('lib/hub/index.js')
+  const listenSrc = read('lib/listen-server.js')
+
+  check('86', '独立监听：插件自己开端口服务节点端点（不依赖 webServer 配置）',
+    /startListenServer/.test(hubSrc2)
+    && /resolveListenHost/.test(hubSrc2)
+    // 默认端口 19387（与历史配置/配对码/文档一致）
+    && /DEFAULT_LISTEN_PORT/.test(listenSrc)
+    && /19387/.test(listenSrc)
+    // 必须真的用 node:http 起 server（不是只 import 不用）
+    && /createServer/.test(listenSrc)
+    && /node:http/.test(listenSrc)
+    // 路径精确匹配（只承载节点端点，不做通用 web server）
+    && /pathname !== upgradePath/.test(listenSrc),
+    'lib/hub/index.js + lib/listen-server.js（node:http + acceptUpgrade + 精确路径）')
+
+  check('87', '独立监听：拒绝 0.0.0.0 + Tailscale 优先 + 测试环境不占端口',
+    // 拒绝通配地址（与新版 DSH 同规则；0.0.0.0 会让 DSH 崩溃，我们踩过）
+    /h !== '0\.0\.0\.0'/.test(listenSrc)
+    // 自动选地址时 Tailscale 优先（本插件的目标场景就是跨网）
+    && /isTailscaleIp/.test(listenSrc)
+    // 测试环境下不真的监听（否则 npm test 会挂住 —— 实测过）
+    && /NODE_TEST_CONTEXT/.test(hubSrc2),
+    'lib/listen-server.js（拒 0.0.0.0 + Tailscale 优先）+ hub/index.js（测试环境跳过）')
+}
+
 /* ── 输出 ── */
 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 ''