dsh-session-superagent 0.8.13 → 0.8.15

package/CHANGELOG.md

@@ -2,6 +2,63 @@
 
 格式参照 [Keep a Changelog](https://keepachangelog.com/zh-CN/1.1.0/)，版本号遵循 [语义化版本](https://semver.org/lang/zh-CN/)。
 
+## [0.8.15] - 2026-11-02 —— sync-profile 支持修复断链（事故 1 的第二种形态）
+
+### 修什么
+`sync-profile.mjs` 之前只会「补缺失文件」，遇到**已存在但 inode 不同**（断链）只打印一句“注意”。
+断链的来源很日常：任何用「写临时文件再 rename」保存的工具（含某些编辑器的原子写）都会让
+profile 里那份留在旧 inode 上 —— 于是**宿主跑旧代码，仓库里看着是新代码**。
+
+现在三种模式：
+
+```bash
+pnpm run sync:profile           # 补齐缺失（不动断链，只警告）
+pnpm run sync:profile:repair    # 断链重建硬链接 + 清理 profile 里的陈旧文件
+pnpm run sync:profile:check     # 只检查：有缺失/断链/陈旧就 exit 1（CI/发布前用）
+```
+
+实测：profile 里 `README.md` / `CHANGELOG.md` 当时确实是断链，`--repair` 重建后
+`--check` 从 exit 1 变 exit 0。另外故意把 README 换成新 inode，验证 `--check` 会红（exit 1）。
+
+### 守门
+- 新增用例：`sync-profile` 的 `TARGETS` 必须覆盖 `lib/` 下所有 js 文件 ——
+  防止「新增 lib 文件忘了登记」再次导致插件不挂载。
+
+### 测试
+- 合计 **81 例**。
+
+## [0.8.14] - 2026-11-02 —— 第四轮审核：1 条文档 + 3 条代码建议
+
+审核范围 0.8.9→0.8.13（上一轮门禁只覆盖到 0.8.8）。结论：代码侧无阻塞，文档有 1 处必须修。
+
+### 修复
+- **幽灵会话不退避**（真问题）：绑到「已删除会话」的 watcher 每次轮询都打一次 GitHub 且永不休止
+  （0.8.11 的分支直接 `return`，没走 `bumpFailure`）。现在会正常进入 30 分钟退避。
+- **删掉 `/client-hello` 路由**：客户端只在那里发过一次 fire-and-forget fetch，而 `note('apply')`
+  已经覆盖同一用途。路由 7 → 6，并加用例锁死路由集合（`/config`、`/client-hello` 都不该回来）。
+
+### 文档
+- HANDOFF 版本号 0.8.11 → 0.8.14；README / MAINTENANCE / HANDOFF 的测试数 77 → 80。
+
+### 未采纳（附理由）
+- **goal 文本里的 `【主理鱼】` 内联**：已压平换行（实测伪造抬头不会独占一行），
+  且 goal 只能由面板或本地 POST 写入，风险低；强行改写用户文本得不偿失。
+- **`sync-profile --repair`**：断链（inode 不同）目前只警告。已在 HANDOFF 写明「新增文件后必须跑」，
+  留作后续（改起来简单，但此刻没有证据表明它造成过事故）。
+
+### 发布
+- 已发 `dsh-session-superagent@0.8.14`（registry `latest = 0.8.14`）。
+- **从 tarball 真装验证**：`npm i https://registry.npmjs.org/dsh-session-superagent/-/dsh-session-superagent-0.8.14.tgz`
+  → `apply()` 成功、**路由 6 条**（`/client-hello` 确实没了）、`client.js` 语法通过、`lib/` 四个文件齐全。
+  注：刚发布时 `npm i pkg@x.y.z` 可能因客户端元数据缓存报 ETARGET，用 tarball URL 或 `--prefer-online` 可绕过。
+
+### 其他
+- `scripts/smoke-observe.mjs` 加 `gh auth status` 预检：没登录时明确 SKIP 并退出 0，
+  不再把「凭证/网络问题」混报成逻辑失败（第四轮审核的建议）。
+
+### 测试
+- 合计 **80 例**（新增路由集合用例）。
+
 ## [0.8.13] - 2026-11-02 —— 真机跑「只观察」闭环，抓出两个真 bug
 
 拿真仓库（MYCF711/dsh-GOkifu-compact，3 个真 issue）跑 observe 模式验证，抓到：
@@ -21,9 +78,46 @@ history: [{"numbers":[4,2,3,1],"ok":false,"reason":"observe"}]
 filteredLog: []        ← 改成 labels:["ai-fix"] 后仍为空（3 个 issue 都没这个标签）
 ```
 
+### 发布与真机验证
+- 已发 `dsh-session-superagent@0.8.13`。
+- **`watch_ack` 第一次在真仓库上验证**：给 `MYCF711/dsh-GOkifu-compact#2` 打上 `主理鱼:已处理`——
+  GitHub **接受中文 + 冒号**的标签名（此前审核把它列为"未验证"）；重复 ack 幂等不报错；
+  验证完已把 issue 标签与仓库标签定义都撤掉，不留痕迹。
+
 ### 测试
 - 合计 **79 例**。
 
+### 进程内冒烟（补证：这两个修复到底生效没有）
+`pnpm run smoke:observe`（`scripts/smoke-observe.mjs`）—— 独立进程、独立状态目录、
+捕获 `setInterval` 手动驱动，会真的读 GitHub 只读 API：
+
+```
+OK   观模式仍未唤醒            （followups = 0）
+OK   filteredLog 已落盘        （4 条）
+OK   过滤原因可读              （"不在标签白名单: ai-fix"）
+OK   /state 能读到过滤记录
+OK   空轮询清掉了过期的 lastDeferred
+
+== 场景 B：真唤醒 + 隔离 worktree ==
+OK   唤醒了一次                        （followups = 1）
+OK   消息带【主理鱼】抬头
+OK   消息带条目编号                     （#2 #4 #3 #1，真 GitHub 数据）
+OK   消息带本条目主理目标               （「把 widget 的 issue 都处理完」）
+OK   消息带隔离路径
+OK   worktree 真的建了                  （.dsh-zhuli/MYCF711-dsh-GOkifu-compact-smoke-se-…）
+OK   .dsh-zhuli 下正好一个副本
+```
+
+消息实样（真数据）：
+
+```
+【主理鱼】MYCF711/dsh-GOkifu-compact 有 4 条新动态：
+#2 [open] docs/bench：给「351 tokens」和「失败保留 8/8」补一条可复跑的复现命令 — @MYCF711 (issue) …
+#4 [closed] 【联调】v0.1.0 唤醒链路验证 — @MYCF711 (issue) …
+【隔离工作区】…\.dsh-zhuli\MYCF711-dsh-GOkifu-compact-smoke-se-…
+本条目主理目标：把 widget 的 issue 都处理完
+```
+
 ## [0.8.12] - 2026-11-02 —— 体验打磨四条
 
 ### 改了什么

package/README.md

@@ -1,7 +1,7 @@
 # dsh-session-superagent
 
 [![License: MIT](https://img.shields.io/badge/License-MIT-blue.svg)](LICENSE)
-[![Version](https://img.shields.io/badge/version-0.8.13-green.svg)](https://github.com/MYCF711/dsh-session-superagent/releases)
+[![Version](https://img.shields.io/badge/version-0.8.15-green.svg)](https://github.com/MYCF711/dsh-session-superagent/releases)
 [![npm](https://img.shields.io/npm/v/dsh-session-superagent.svg)](https://www.npmjs.com/package/dsh-session-superagent)
 [![dsh](https://img.shields.io/badge/DeepSeek%20Harness-plugin-orange.svg)](https://github.com/topics/dsh-plugin)
 

package/client.js

@@ -14,7 +14,7 @@ window.__ModuleLoader__.load({
     const { useCallback, useEffect, useRef, useState } = React
 
     // 客户端版本探针：每次模块加载写一次，用来确认浏览器里跑的是哪一版
-    const CLIENT_VERSION = '0.8.13'
+    const CLIENT_VERSION = '0.8.15'
     const BASE = '/session-superagent'
     const ENDPOINT = BASE + '/state'
     const POLL_MS = 5000
@@ -974,7 +974,6 @@ window.__ModuleLoader__.load({
       apply(ctx) {
         try { noteOnce('client-version', CLIENT_VERSION) } catch {}
         try { note('apply', 'start') } catch {}
-        try { fetch(BASE + '/client-hello').catch(() => {}) } catch {}
 
         // ① the 主理鱼 tab in the centre-column view ring
         ctx.effect(() => {

package/lib/index.js

@@ -470,8 +470,10 @@ async function pollOnce(ctx, entry, baseMs) {
   // 会话忙就别开隔离区了：白建一个没人用的 worktree
   const liveAgent = ctx.agents && typeof ctx.agents.get === 'function' ? ctx.agents.get(entry.sessionId) : undefined
   if (!liveAgent) {
-    // 会话已经不在了（被删/换机器）：别开隔离区，也别假装在盯
+    // 会话已经不在了（被删/换机器）：别开隔离区，也别假装在盯。
+    // 必须走 bumpFailure —— 否则这条会每 intervalSec 打一次 GitHub，永不退避（第四轮审核指出）。
     entry.lastError = '会话已不在：请删除或重建这条主理鱼'
+    bumpFailure(entry, baseMs)
     return
   }
   const gate = decideWake(liveAgent && liveAgent.status)
@@ -626,11 +628,6 @@ export function apply(ctx) {
           }))
         },
       }))
-      routes.push(httpServer.register({
-        kind: 'exact',
-        path: '/session-superagent/client-hello',
-        handler: (req, res) => { record('hello', String(req.url)); res.statusCode = 204; res.end() },
-      }))
       routes.push(httpServer.register({
         kind: 'exact',
         path: '/session-superagent/client-note',

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dsh-session-superagent",
-  "version": "0.8.13",
+  "version": "0.8.15",
   "description": "Session-level superagent for DeepSeek Harness: watch a GitHub repository and wake a session when something needs doing. Credentials never reach the model.",
   "type": "module",
   "main": "lib/index.js",
@@ -30,7 +30,10 @@
     "test": "node --test",
     "check": "node --test",
     "preview:orbit": "node scripts/preview-orbit.mjs",
-    "sync:profile": "node scripts/sync-profile.mjs"
+    "sync:profile": "node scripts/sync-profile.mjs",
+    "smoke:observe": "node scripts/smoke-observe.mjs",
+    "sync:profile:check": "node scripts/sync-profile.mjs --check",
+    "sync:profile:repair": "node scripts/sync-profile.mjs --repair"
   },
   "keywords": [
     "dsh",

package/scripts/sync-profile.mjs

@@ -1,17 +1,28 @@
 // 把插件源码目录同步到 profile 的安装副本。
 //
 // 为什么需要：profile 里是 `file:` 安装 —— pnpm 为**已存在的文件**建了硬链接，
-// 所以改内容会自动生效；但**新增文件不会**自动出现。
-// 真实事故（2026-11-01 23:59 重启）：新增的 lib/body.js 没进 profile，
-// 宿主 import './body.js' 失败 → 整个插件不挂载 → 工具和 HTTP 路由全消失，
-// 而客户端 bundle 照常加载，表面看起来"插件还在"。
+// 改内容会自动生效；但**新增文件不会**自动出现，而**被原子写入替换的文件会断链**。
 //
-// 用法： node scripts/sync-profile.mjs [profile 安装目录]
-import { existsSync, linkSync, mkdirSync, readFileSync, readdirSync, statSync } from 'node:fs'
+// 真实事故 1（2026-11-01 23:59 重启）：新增的 lib/body.js 没进 profile，
+//   宿主 import './body.js' 失败 → 整个插件不挂载 → 工具和 HTTP 路由全消失，
+//   客户端 bundle 却照常加载，表面看起来"插件还在"。
+// 真实事故 2（潜伏）：某些工具用「写临时文件再 rename」保存，profile 里那份仍是旧 inode，
+//   于是宿主跑的是旧代码，而仓库里看着已经是新代码。
+//
+// 用法：
+//   node scripts/sync-profile.mjs            补齐缺失文件（不动断链，只警告）
+//   node scripts/sync-profile.mjs --repair   断链也重建硬链接，并清理 profile 里的陈旧文件
+//   node scripts/sync-profile.mjs --check    只检查：有缺失/断链/陈旧就 exit 1（给 CI 用）
+//   node scripts/sync-profile.mjs --dir <path>  指定 profile 安装目录
+import { existsSync, linkSync, mkdirSync, readFileSync, readdirSync, statSync, unlinkSync } from 'node:fs'
 import { dirname, join } from 'node:path'
 import { fileURLToPath } from 'node:url'
 
 const ROOT = join(dirname(fileURLToPath(import.meta.url)), '..')
+const args = process.argv.slice(2)
+const REPAIR = args.includes('--repair')
+const CHECK = args.includes('--check')
+const dirFlag = args.indexOf('--dir')
 const DEFAULT_PROFILE = join(
   process.env.DSH_HOME || join(process.env.USERPROFILE || '', '.dsh'),
   'profiles',
@@ -19,7 +30,7 @@ const DEFAULT_PROFILE = join(
   'node_modules',
   'dsh-session-superagent',
 )
-const DEST = process.argv[2] || DEFAULT_PROFILE
+const DEST = dirFlag >= 0 ? args[dirFlag + 1] : DEFAULT_PROFILE
 
 /** 运行时要用的文件（不含 docs / test / scripts 这些开发件）。 */
 const TARGETS = [
@@ -32,45 +43,87 @@ if (!existsSync(DEST)) {
   process.exit(1)
 }
 
+const drift = { missing: [], broken: [], stale: [] }
 let linked = 0
-let skipped = 0
+
 for (const rel of TARGETS) {
   const src = join(ROOT, rel)
   const dst = join(DEST, rel)
   if (!existsSync(src)) continue
-  if (existsSync(dst)) {
-    const same = statSync(src).ino === statSync(dst).ino
-    console.log((same ? 'ok       ' : '注意     ') + rel + (same ? '' : '（inode 不同，内容改动不会同步）'))
-    skipped += 1
+  if (!existsSync(dst)) {
+    drift.missing.push(rel)
+    if (!CHECK) {
+      mkdirSync(dirname(dst), { recursive: true })
+      try { linkSync(src, dst); linked += 1; console.log('linked   ' + rel) } catch (error) {
+        console.log('link 失败：' + rel + ' — ' + (error && error.message))
+      }
+    }
     continue
   }
-  mkdirSync(dirname(dst), { recursive: true })
-  try {
-    linkSync(src, dst)
-    linked += 1
-    console.log('linked   ' + rel)
-  } catch (error) {
-    console.log('link 失败：' + rel + ' — ' + (error && error.message))
+  const same = statSync(src).ino === statSync(dst).ino
+  if (same) continue
+  drift.broken.push(rel)
+  if (REPAIR && !CHECK) {
+    try {
+      unlinkSync(dst)
+      linkSync(src, dst)
+      linked += 1
+      console.log('relinked ' + rel + '（原来断了）')
+    } catch (error) {
+      console.log('重建失败：' + rel + ' — ' + (error && error.message))
+    }
+  } else {
+    console.log('注意     ' + rel + '（inode 不同：内容改动不会同步到 profile，用 --repair 重建）')
+  }
+}
+
+// 陈旧文件：profile 里有、仓库里已经没有（例如删掉的模块）
+for (const name of readdirSync(join(DEST, 'lib')).filter((f) => f.endsWith('.js'))) {
+  if (!existsSync(join(ROOT, 'lib', name))) {
+    drift.stale.push('lib/' + name)
+    if (REPAIR && !CHECK) {
+      try { unlinkSync(join(DEST, 'lib', name)); console.log('pruned   lib/' + name) } catch {}
+    } else {
+      console.log('陈旧     lib/' + name + '（仓库已删除，profile 里还在）')
+    }
   }
 }
 
-// lib/ 里每个相对 import 都必须能解析到真实文件（这次事故的直接原因）
+// lib/ 里每个相对 import 都必须能解析（这正是事故 1 的直接原因）
 const libDir = join(ROOT, 'lib')
-const missing = []
+const missingImports = []
 for (const name of readdirSync(libDir)) {
   if (!name.endsWith('.js')) continue
-  const file = join(libDir, name)
-  const code = readFileSync(file, 'utf8')
-  for (const m of code.matchAll(/from '(\.[^']+)'/g)) {
-    if (!existsSync(join(dirname(file), m[1]))) missing.push(name + ' → ' + m[1])
+  const code = readFileSync(join(libDir, name), 'utf8')
+  for (const line of code.split(String.fromCharCode(10))) {
+    const trimmed = line.trim()
+    if (!trimmed.startsWith('import ')) continue
+    const at = trimmed.lastIndexOf("from '")
+    let spec = null
+    if (at >= 0) {
+      spec = trimmed.slice(at + 6).split("'")[0]
+    } else if (trimmed.slice('import '.length).startsWith("'")) {
+      spec = trimmed.slice('import '.length + 1).split("'")[0]
+    }
+    if (spec && spec.startsWith('.') && !existsSync(join(libDir, spec))) {
+      missingImports.push(name + ' -> ' + spec)
+    }
   }
 }
 
+const drifted = drift.missing.length + drift.broken.length + drift.stale.length
 console.log('---')
-console.log(`新增链接 ${linked} 个，跳过 ${skipped} 个`)
 console.log(`目标：${DEST}`)
-if (missing.length > 0) {
-  console.error('相对 import 解析失败：' + missing.join(', '))
+console.log(`缺失 ${drift.missing.length} / 断链 ${drift.broken.length} / 陈旧 ${drift.stale.length}` +
+  (linked ? `，本次处理 ${linked} 个` : ''))
+
+if (missingImports.length > 0) {
+  console.error('相对 import 解析失败：' + missingImports.join(', '))
   process.exit(2)
 }
 console.log('相对 import 全部可解析 ✓')
+
+if (CHECK && drifted > 0) {
+  console.error('profile 与仓库不一致（跑 --repair 修）')
+  process.exit(1)
+}