dsh-remote-agent 0.9.3 → 0.9.5

package/lib/setup.js

@@ -15,6 +15,10 @@ import fs from 'node:fs'
 import net from 'node:net'
 import { randomBytes } from 'node:crypto'
 import { expandPairConfig } from './pairing.js'
+// 地址枚举**必须只有一份实现**（2026-10-20 收口）：
+// 此前 detectLanIps（私网优先）与 resolveListenHost（Tailscale 优先）各写一套，
+// 排序方向相反 → 面板生成的配对码第一条必定不是实际绑定地址（C2）。
+import { listCandidateAddresses } from './listen-server.js'
 
 /** 与 hub 侧 newNodeToken 完全同构（nt_ 前缀 + 24 字节 base64url）。 */
 export function newNodeToken() {
@@ -26,23 +30,38 @@ export const NODE_ID_RE = /^[A-Za-z0-9][A-Za-z0-9._-]{0,63}$/
 
 /**
  * 枚举本机局域网 IPv4（排除 loopback / 链路本地 / 虚拟接口）。
- * 返回 [{ name, address, private }]，private = RFC1918。
+ * 返回 [{ name, address, private }]，private = RFC1918 或 Tailscale（可信内网）。
+ *
+ * ═══════════════════════════════════════════════════════════════════════════
+ * 2026-10-20 改为复用 `listCandidateAddresses()`（v0.9.3 审查 C2）
+ * ═══════════════════════════════════════════════════════════════════════════
+ *
+ * 原来这里自己实现了"枚举 + 私网优先排序"，而 `lib/listen-server.js` 的
+ * `resolveListenHost()` 实现了"Tailscale 优先"。两套判据方向**相反**，于是：
+ *
+ *   独立监听实际绑   → 100.86.48.54（Tailscale）
+ *   面板配对码第一条 → ws://10.239.22.41:56594/...（WLAN，私网优先排第一）
+ *
+ * 用户复制第一条 → 连不上，而两端都不报错。
+ *
+ * 现在**一份实现**：排序 / 私网判定 / APIPA 过滤全部交给 listen-server，
+ * 这里只做字段名适配（它返回 iface，历史调用方要 name）。
+ *
+ * @returns {Array<{name:string, address:string, private:boolean, score:number}>}
  */
 export function detectLanIps() {
-  const out = []
-  const ifaces = os.networkInterfaces()
-  for (const [name, addrs] of Object.entries(ifaces)) {
-    for (const a of addrs ?? []) {
-      if (a.family !== 'IPv4' || a.internal) continue
-      if (a.address.startsWith('169.254.')) continue // 链路本地：没拿到 DHCP 的假地址
-      const [o1, o2] = a.address.split('.').map(Number)
-      const priv = o1 === 10 || (o1 === 172 && o2 >= 16 && o2 <= 31) || (o1 === 192 && o2 === 168)
-      out.push({ name, address: a.address, private: priv })
-    }
+  try {
+    return listCandidateAddresses().map((c) => ({
+      name: c.iface,
+      address: c.address,
+      // Tailscale（100.64/10）在语义上属于"可信内网"，与 RFC1918 同等看待：
+      // 面板用它区分"要不要提示公网风险"，把它算公网会给出错误警告。
+      private: c.score <= 1,
+      score: c.score,
+    }))
+  } catch {
+    return []
   }
-  // 私网优先，稳定的地址排前面
-  out.sort((x, y) => (x.private === y.private ? 0 : x.private ? -1 : 1))
-  return out
 }
 
 /**
@@ -455,6 +474,145 @@ export function hasUnmanagedRemoteAgent(fileText) {
   return RA_ENTRY_RE.test(src.slice(0, i) + src.slice(j + MANAGED_END.length))
 }
 
+/**
+ * 面板 setup 表单**管辖**的顶层 config 键。
+ *
+ * 只有这些字段允许被"整块重写"覆盖；其余（`nodeToken` / `uiPullRoot` /
+ * `listenHost` / `listenPort` / `relay` / `pair` / `token` …）一律原样保留。
+ *
+ * 放在这里而不是 setup-routes.js：它是「哪些字段归表单管」这个**领域事实**，
+ * CLI 向导与测试也要按同一份清单判断（两处各写一份就会分叉 —— 本版审查里
+ * C2/L2 就是这么来的）。
+ */
+export const FORM_OWNED_KEYS = Object.freeze([
+  'role',
+  'nodeId',
+  'transportMode',
+  'hubUrl',
+  'relayUrl',
+  'roots',
+])
+
+/**
+ * 从**现有托管块**里抽出"本次表单不管辖"的 config 字段，原样返回（含缩进）。
+ *
+ * ═══════════════════════════════════════════════════════════════════════════
+ * 为什么需要（2026-10-20，v0.9.3 审查 H2）
+ * ═══════════════════════════════════════════════════════════════════════════
+ *
+ * `upsertManagedBlock` 是**块级整体替换**：新块进来，旧块整段消失。
+ * 而面板的 setup 表单只提交 role/nodeId/transportMode/hubUrl/relayUrl/roots。
+ *
+ * 于是用户在面板点一次「保存」，托管块里的 `nodeToken`、`uiPullRoot`、
+ * `listenHost`、`listenPort`、`relay` **全部被静默删掉**（已实证）。
+ * 而 instance.json 那边也没存这些 —— 等于"点一次保存，凭证和设置就没了"。
+ *
+ * 修法不是让表单管更多字段（字段会一直加），而是**只重写自己管辖的那几个**，
+ * 其余原样搬回来。
+ *
+ * 实现刻意走**文本级切分**而不是 YAML 解析：
+ *   · 不需要引入解析器（本包零运行时依赖是硬约束）
+ *   · 保留用户的原始格式（注释、引号风格、键顺序）—— 解析再序列化会全部丢失
+ *
+ * @param {string} blockText 托管块正文（`MANAGED_BEGIN/END` 之间的内容）
+ * @param {Set<string>|string[]} ownedKeys 本次由表单管辖的顶层 config 键
+ * @returns {string[]} 需要原样保留的行
+ */
+export function extractPreservedConfig(blockText, ownedKeys) {
+  const src = typeof blockText === 'string' ? blockText : ''
+  const owned = ownedKeys instanceof Set ? ownedKeys : new Set(Array.isArray(ownedKeys) ? ownedKeys : [])
+  const lines = src.split('\n')
+
+  // 定位 `  config:`（两个空格缩进 —— loader 条目的标准形态）
+  const start = lines.findIndex((l) => /^\s{2}config:\s*$/.test(l))
+  if (start === -1) return []
+
+  const groups = []
+  let cur = null
+  for (let i = start + 1; i < lines.length; i += 1) {
+    const line = lines[i]
+    if (line.trim() === '') {
+      // 空行跟着当前组（保留段落感），组还没开始就忽略
+      if (cur) cur.lines.push(line)
+      continue
+    }
+    // 顶层字段：4 个空格 + key:
+    const m = /^ {4}([A-Za-z0-9_]+):/.exec(line)
+    if (m) {
+      cur = { key: m[1], lines: [line] }
+      groups.push(cur)
+      continue
+    }
+    // 更深缩进（嵌套对象 / 列表项）→ 属于当前组
+    if (cur && /^ {5,}\S/.test(line)) {
+      cur.lines.push(line)
+      continue
+    }
+    // 缩进 ≤3 或新条目 —— config 段结束
+    break
+  }
+
+  const kept = []
+  for (const g of groups) {
+    if (owned.has(g.key)) continue
+    // 去掉尾部多余空行，避免拼回去时积累空行
+    const ls = g.lines.slice()
+    while (ls.length > 0 && ls[ls.length - 1].trim() === '') ls.pop()
+    kept.push(...ls)
+  }
+  return kept
+}
+
+/**
+ * 读取文件里现有托管块的正文（不含标记行）。没有托管块时返回 ''。
+ * @param {string} fileText
+ * @returns {string}
+ */
+export function readManagedBlockBody(fileText) {
+  const src = typeof fileText === 'string' ? fileText : ''
+  const i = src.indexOf(MANAGED_BEGIN)
+  if (i === -1) return ''
+  const j = src.indexOf(MANAGED_END, i)
+  if (j === -1) return ''
+  return src.slice(i + MANAGED_BEGIN.length, j)
+}
+
+/**
+ * 在托管块正文里设置 `nodeToken`（有则替换，无则插入），返回**新的块正文**。
+ *
+ * 为什么不是"重新生成整个块"：那会把用户手写的其它字段一起冲掉（H2）。
+ * 这里只动 token 这一行，其余字节保持原样。
+ *
+ * @param {string} blockText 托管块正文（不含标记）
+ * @param {string} token 新口令
+ * @returns {string}
+ */
+export function upsertTokenInBlock(blockText, token) {
+  const body = typeof blockText === 'string' ? blockText : ''
+  const t = typeof token === 'string' ? token.trim() : ''
+  if (t === '') return body
+  // 单引号包裹 + 转义：token 是 nt_ + base64url，理论上不含引号，但不赌
+  const line = `    nodeToken: '${t.replace(/'/g, "''")}'`
+  const lines = body.split('\n')
+
+  const i = lines.findIndex((l) => /^ {4}nodeToken:/.test(l))
+  if (i !== -1) {
+    lines[i] = line
+    return lines.join('\n')
+  }
+
+  // 没有 → 插到 `  config:` 之后（保持它作为 config 的第一个子键，可读性好）
+  const ci = lines.findIndex((l) => /^\s{2}config:\s*$/.test(l))
+  if (ci !== -1) {
+    lines.splice(ci + 1, 0, line)
+    return lines.join('\n')
+  }
+
+  // 连 config 行都没有（极早期手写块）→ 追加一个 config 段
+  const trimmed = body.replace(/\s+$/, '')
+  return `${trimmed}${trimmed === '' ? '' : '\n'}  config:\n${line}\n`
+}
+
 /**
  * 写盘主流程：备份 → 合并 → 写入 → 校验，失败自动回滚。
  *

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dsh-remote-agent",
-  "version": "0.9.3",
+  "version": "0.9.5",
   "type": "module",
   "description": "把另一台电脑变成你 DSH 的远程 Agent 同事：副机本地持久跑任务、独立 Agent 循环，只回传 handoff 摘要与产物清单，成品按需拉取。",
   "license": "MIT",

package/scripts/check-doc-drift.mjs

@@ -1113,7 +1113,7 @@ check('65', 'eslint 严格档：lint script + 配置文件 + CI job 三件套齐
     })(),
     'lib/hub/ui-routes.js（role: resolveRole(config) 而非原始 config.role）')
 
-  check('85', 'SMB 面板发真实 dir + CLI doctor 有 loopback 检查（H2 / M1）',
+  check('85', 'SMB 面板发真实 dir + CLI doctor 走独立监听判可达性（H2 / M1 / H5）',
     // H2：不得写死 dir: ''
     (() => {
       const i = clientSrc5.indexOf('function doPlan()')
@@ -1125,8 +1125,17 @@ check('65', 'eslint 严格档：lint script + 配置文件 + CI job 三件套齐
     && /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）')
+    // H5：回环判据必须来自 listen-server（一份实现），不是 CLI 自己拼 origin
+    && /isLoopbackHost/.test(setupBin)
+    // H5 负向守卫：CLI **不得**再从 patch 里找 webserver 段判可达性。
+    // 0.9.3 起端点由插件自己的独立监听承载，webserver 的 host 与副机接入
+    // 毫无关系 —— 读了会把人引向"改 webserver"这个改了也会被冲掉的死路。
+    // ⚠ 扫**剥注释后**的代码：注释里必须保留事故说明（含 `- id: webserver`
+    // 字样），那是给后来者的警示，不该被这条断言判违规。
+    && !/- id:\s*webserver/.test(stripComments(setupBin))
+    // 生效配置以 instance.json 为准（否则体检报告的是已失效的 patch 值）
+    && /readInstanceConfig/.test(setupBin),
+    'lib/client.js（dir: shareDir + nodeRoots UI）+ bin/dsh-remote-setup.js（envChecks + listen-server 判据）')
 }
 
 /* ── 86~87. 独立监听端口（2026-10-18，架构级修复）──