dsh-turn-recovery 0.2.3 → 0.2.4

package/CHANGELOG.md

@@ -7,6 +7,55 @@
 > `@nlflora/dsh-empty-turn-guard`）冲突，自 0.2.0 起更名为 `dsh-turn-recovery`。0.1.x 及更早的条目保留旧名原文，
 > 其提到的 `plugin:empty-turn-guard`、`empty-turn-guard.log` 等标识在 0.2.0 起分别改为 `plugin:turn-recovery`、`turn-recovery.log`。
 
+## [0.2.4] - 2026-11-09
+
+**运行期复核版**。0.2.3 之后按 8 天生产日志（645 个 `turn/end`）回头复查代码，
+修掉一处「装了但可能静默不工作」的状态索引隐患，并清掉每回合都要付的同步与
+日志开销；顺带把「守卫究竟为什么没触发」变成一行可读的聚合统计。
+
+### 修复
+
+- **会话状态只按 session 对象身份索引**（`P1`）：`state` 是 WeakMap，而宿主在
+  不同代码路径上可能给出不同的 session 实例（同文件的 `agentOf` 早就为此加了
+  `agentsById`）。一旦身份漂移，`turn/start` 落在对象 A、`assistant/message` 与
+  `turn/end` 落在对象 B，B 的状态里 `sawTurnStart` 恒为 false，判定链整体早退，
+  只留一行 `turn/start not observed` —— 与 0.2.1 修过的「orchestrator-* 被判成
+  子代理、19 次静默跳过」是同一类故障。现在加有界的 `stateById`（500 条）双索引，
+  `agent/disposed` 时两个索引同时清理。
+- **布尔配置拼错不会被发现**：`enabled: "flase"` 被 `enabledOr` 当成 `true` 且
+  无人提示。布尔键现在只承认 `true/false/1/0/on/off/yes/no`，其余记一条
+  `config issue: "enabled" looks like a typo`。
+- **`foregroundKinds` 里的非字符串元素被静默丢弃**：现在记一条 config issue。
+- **预算文件压缩失败会留下 `.tmp-<pid>`**：`rename` 失败（Windows 上 EPERM/EBUSY）
+  时先 unlink 再抛，不再往 state 目录里积垃圾。
+
+### 性能 / 日志
+
+- **每行日志都同步 `mkdirSync`**：目录改为只建一次（写失败复位、下次自愈），
+  少掉宿主事件循环上每行一次的同步系统调用。
+- **正常回合的 `turn/end` 行占日志一半**（实测 645/1286 行）：新增
+  `logTurns: 'empty' | 'all'`，默认 `empty` 只在空回合/跳过判定时写行，排查期切
+  `all` 恢复逐回合记录。
+- **每条 assistant 消息两遍遍历 + 一次 `Array.from()`**：合并成一次遍历的
+  `inspectMessage()`，字符数用 `for...of` 累加（仍给精确总数，日志的
+  `visibleChars` 不退化成「至少 1」）。
+- **`GUARD_IDS_MAX` 与字面量 `8` 脱钩**：投递时的截断改用常量。
+- **一小时预算谓词写了 4 遍**：抽成 `withinWindow(at, now)` 与
+  `budgetExhausted(s, now)`，窗口与时钟容忍度以后只改一处。
+
+### 可观测性
+
+- **新增 `stats:` 聚合行**（每 50 个回合 + 卸载时 `(final)`）：
+  `turns / emptyCompleted / emptyOther / delivered / claimed`。生产经验是
+  `completed` 空回合 8 天为 0、空回合全是 `steps=0` 的 `error`/`aborted`；有了
+  这行就能区分「守卫没触发是因为目标场景没发生」和「守卫坏了」。
+
+### 文档
+
+- 更正 README「已知限制」里的实测数据：空回合不是「`aborted` / `blocked`」，
+  而是 `error` 162 / `aborted` 20、`completed` 0；并补上 `logTurns`、
+  `stats` 与状态双索引的说明。
+
 ## [0.2.3] - 2026-11-05
 
 **三方复核（架构 / 安全 / 验收）后的加固版**。0.2.2 的判定链没有 P0 缺陷，但复核在

package/README.md

@@ -67,10 +67,11 @@
     budgetFile: ''           # 空 = <DSH_HOME>/state/turn-recovery/nudges.log（跨重启限流）
                              # 同目录还会写 guard-ids.log（跨重启认出晚到认领的补写 id）
     logFile: ''              # 空 = <DSH_HOME>/logs/turn-recovery.log
+    logTurns: empty          # empty = 只记空回合/被跳过判定的回合；all = 连正常回合也记
     dryRun: false            # true = 判定/限流/日志全部照走，但不真的投递（验证用）
 ```
 
-配置做严格解析：`maxNudgesPerHour: 0` 是**关闭补写**，不会被默认值吞成 3；`delayMs: "abc"` 这类非法值回落到默认值；`enabled: "false"`（字符串）与 `false` 等价。
+配置做严格解析：`maxNudgesPerHour: 0` 是**关闭补写**，不会被默认值吞成 3；`delayMs: "abc"` 这类非法值回落到默认值；`enabled: "false"`（字符串）与 `false` 等价。布尔键只承认 `true/false/1/0/on/off/yes/no`，`enabled: "flase"` 这类拼错会被记成 config issue（行为仍按真值走，所以要留意日志）。
 
 宿主**不校验**插件配置（本包不导出 `Config` schema，原因见「实现约束」），所以键名写错不会被谁拦下来。插件改为在加载期自查：未知键、类型不符、`sessionScope` 取值不对等，逐条写进日志（`config issue: "minNudgesPerHour" is unknown -> ignored`），插件照常启用。改完配置先看这几行，比猜「为什么没生效」划算。
 
@@ -102,7 +103,19 @@
 [2026-10-25T02:03:45.004Z] guard message claimed turn=5 matched=true [5f884825]
 ```
 
-正常回合只需看 `empty=false`；出现 `empty=true` 就应紧跟 `nudge outcome turn=N: scheduled` 与 `nudge DELIVERED`，再往后出现 `guard message claimed ... matched=true` 才说明补写回合真的起来了。`[...]` 是会话 id（剥掉 `session-` / `orchestrator-` / `worker-` 前缀）前 8 位，`wasGuard` 表示该回合是不是补写回合，`sawStart` 表示插件是否看到过该回合的 `turn/start`（中途热加载时为 false，此时不判定）。
+正常回合默认**不写** `turn/end` 行（`logTurns: empty`）：实测这类行占日志的一半，而它只在「某个回合为什么没被判空」时才有人看。要看逐回合记录就把 `logTurns` 切成 `all`，此时格式同上。空回合的 `empty=true` 行在任何粒度下都会留下。
+
+每 50 个回合会多一行聚合统计，卸载/重载时再打一行 `(final)`：
+
+```
+[…Z] stats: turns=50 emptyCompleted=0 emptyOther=37 delivered=0 claimed=0
+```
+
+`emptyCompleted` 是守卫真正要救的那一类（`completed` 却没有正文）；`emptyOther` 是 `error` / `aborted` 等按设计不补写的空回合。长期出现 `emptyCompleted=0` 而 `emptyOther>0`，说明「守卫没触发」是因为目标场景本身没发生，而不是插件坏了；只有 `delivered` 与 `claimed` 的差值才说明投递链路是否真的接通。
+
+出现 `empty=true` 时应紧跟 `nudge outcome turn=N: scheduled` 与 `nudge DELIVERED`，再往后出现 `guard message claimed ... matched=true` 才说明补写回合真的起来了。
+
+`[...]` 是会话 id（剥掉 `session-` / `orchestrator-` / `worker-` 前缀）前 8 位，`wasGuard` 表示该回合是不是补写回合，`sawStart` 表示插件是否看到过该回合的 `turn/start`（中途热加载时为 false，此时不判定）。
 
 只想验证判定链、不想真的多跑一轮模型时，把 `dryRun` 打开，日志会写 `nudge DRY-RUN: would deliver turn=N` 并停在那里（既不投递，也不占小时预算）。
 
@@ -125,6 +138,7 @@
 
 ## 已知限制
 
+- **会话状态是双索引**：`state` 是 WeakMap（按 session 对象身份，快路径），另有有界（500 条）的 `stateById`（按 `session.id`，兜底）。宿主在某些代码路径上会给出不同的 session 实例；只按对象身份索引时，`turn/start` 与 `turn/end` 会落在两份状态上，`sawTurnStart` 恒为 false、判定链整体静默早退（只留一行 `turn/start not observed`）。现在两个索引同时写，`agent/disposed` 时同时清。
 - **补写消息以 `role: 'user'` 落盘**（`source.kind = 'plugin:turn-recovery'`）：宿主 session 层对 `source.kind` 不做枚举校验（只要求非空字符串），插件侧也改不了 role。桌面前端按 `source.kind !== 'user'` 渲染成上下文卡片（不是用户气泡），mnemon 只收录 `source.kind === 'user'`，所以界面与记忆都不受污染；但**会话转录文件里它确实是一条 user 消息**，审计时要看 `source.kind` 区分。另外 `plugin:*` 这个前缀不在宿主 `MessageSourceMap` 的类型联合里 —— 运行时不受影响（前端对未知 kind 走 `default` 分支，显示为「请求」卡片），但下游按类型联合写的消费者会漏掉它。
 - **用户点停止的窗口**：宿主 `cancel()` 只清 inbox + abort，不 append 任何 session 事件。插件订阅了宿主 dispatch 的 `agent/inbox/discarded` 来撤掉自己那条补写，但该事件是否向插件广播取决于宿主的 dispatch 链路；收不到时，「用户已点停止、插件仍在 `delayMs` 后补写」这个窗口依然存在（默认 900ms，可通过 `delayMs` 调小）。真正发出去的那一回合仍受宿主 abort 语义约束。
 - **`sessionScope: 'root'` 的判定依据是宿主的 `Session.header`**：`origin === 'subagent'` 或 `delegationDepth > 0` 才算子代理会话，`orchestrator-*` 这类顶层会话会被正常处理（0.2.0 用 id 前缀判断，把它们全部判成子代理，实测日志里有 19 次静默跳过）。宿主没有 header 的场合（老宿主、测试桩）退回 id 形态：`session-*` / `orchestrator-*` 算根，纯 UUID / `social-*` 不算。被过滤的会话每个 id 只记一条 `skip scope` 日志，不会静默消失。
@@ -134,7 +148,7 @@
 - **多进程（桌面宿主 + 独立的 web/headless 宿主）**：各自一套内存态，`agent.inbox.nextTurn` 只对本进程可见。预算文件与补写 id 文件是共享的，因此限流大致成立，但「同一会话同时被两个宿主打开」仍可能各补一笔。同一会话请只用一个宿主。
 - **`dryRun` 不写预算文件**：判定链、日志、限流检查照走，但不投递、不记账，所以它只能证明「判定会发生」，不能用来验证限流是否生效（日志里的 `budgetUsed=n/max` 是当时的已用额度）。
 - **多宿主共用同一日志文件**：两个进程同时写同一路径时轮转可能失败，此时回退 `truncate(0)` 并记一条 `log rotate fell back to truncate`。
-- **生产环境仍未观测到真实补写**：截至 2026-11-04，本机生产日志里 `nudge DELIVERED` 仍为 0（空回合全部是 `aborted` / `blocked`），所有结论来自单测与 mock 复现。要闭合这一项，请在真机上临时设置 `dryRun: true`，等一次 `completed` 空回合，确认日志出现 `nudge DRY-RUN: would deliver`（判定链在生产成立），再关掉 `dryRun` 等真实投递 —— 这比直接开补写更安全：干跑不会多跑一轮模型。
+- **生产环境仍未观测到真实补写（截至 2026-11-09 复核）**：本机 8 天日志里共 645 个 `turn/end`（`completed` 455 / `error` 165 / `aborted` 25），空回合 182 个 —— 其中 `error` 162、`aborted` 20、`completed` **0**，`nudge SCHEDULED` / `DELIVERED` 也仍是 0。也就是说：唯一要救的那一类（`completed` 却只剩零宽占位符）在本机一次都没发生，出现的全是 `steps=0` 的调用失败（162 次集中在 11-06 / 11-07 两天，属于上游故障；此时补写只会反复撞同一堵墙，所以按设计不补）。所有结论仍来自单测与 mock 复现。要闭合这一项，请在真机上临时设置 `dryRun: true`，等一次 `completed` 空回合，确认日志出现 `nudge DRY-RUN: would deliver`（判定链在生产成立），再关掉 `dryRun` 等真实投递 —— 这比直接开补写更安全：干跑不会多跑一轮模型。看 `stats:` 聚合行可以一眼判断当前处在哪种状态。
 
 ## License
 

package/lib/index.js

@@ -99,6 +99,16 @@ const NUDGE_CLAIM_TIMEOUT_MS = 120000;
  * 记录一律丢弃（不报错、不中断，只丢那一条）。
  */
 const BUDGET_FUTURE_SKEW_MS = 5 * 60 * 1000;
+/**
+ * 一条投递时刻是否落在「最近一小时」的预算窗口内。
+ * 未来时间戳（> now + 容忍窗口）必须一起丢掉：`now - t` 对它是负数，
+ * `< HOUR_MS` 恒成立，只按上半句过滤就会把它永久留在预算里、让该会话永久
+ * 判成「本小时已用尽」。这个谓词此前在 4 处各写一遍（会话级修剪、全局修剪、
+ * 预算文件解析、两处上限比较），改窗口或容忍度时很容易漏改其中一处。
+ */
+function withinWindow(at, now) {
+  return now - at < HOUR_MS && at <= now + BUDGET_FUTURE_SKEW_MS;
+}
 /** 已投递补写 id 的持久化保留时长：覆盖宿主重启/插件热重载后的晚到认领。 */
 const GUARD_ID_TTL_MS = 24 * 60 * 60 * 1000;
 /** 每个会话保留的补写 id 上限（与内存态 guardIds 一致）。 */
@@ -180,7 +190,7 @@ function enabledOr(value, fallback) {
 /** 配置项清单：只用于在加载期挑出「写错的键」，不作为校验依据。 */
 const CONFIG_KEYS = new Set([
   'enabled', 'injectInstruction', 'dryRun', 'instructionText',
-  'minVisibleChars', 'delayMs', 'maxNudgesPerHour', 'logFile',
+  'minVisibleChars', 'delayMs', 'maxNudgesPerHour', 'logFile', 'logTurns',
   'sessionScope', 'foregroundKinds', 'budgetFile',
 ]);
 const NUMERIC_KEYS = ['minVisibleChars', 'delayMs', 'maxNudgesPerHour'];
@@ -221,7 +231,18 @@ function describeConfigIssues(raw) {
   for (const key of BOOLEAN_KEYS) {
     const v = raw[key];
     if (v === undefined || typeof v === 'boolean') continue;
-    if (typeof v === 'string' || typeof v === 'number') continue;
+    if (typeof v === 'number') continue;
+    if (typeof v === 'string') {
+      // 旧版对布尔键接受任意字符串，于是 enabled: "flase" 这类拼错会被
+      // enabledOr 静默当成 true，用户以为关掉了、其实一直开着。这里只承认
+      // 明确的真/假写法，其余按拼错报出来（行为仍是 enabledOr 的结果）。
+      const word = v.trim().toLowerCase();
+      if (word === '' || word === 'true' || word === 'false' || word === '1' || word === '0'
+        || word === 'on' || word === 'off' || word === 'yes' || word === 'no') continue;
+      issues.push('"' + key + '" looks like a typo (' + brief(v) + ') -> treated as '
+        + (enabledOr(v, false) ? 'true' : 'false'));
+      continue;
+    }
     issues.push('"' + key + '" expects a boolean, got ' + brief(v) + ' -> ' + (enabledOr(v, false) ? 'true' : 'false'));
   }
   for (const key of STRING_KEYS) {
@@ -234,23 +255,15 @@ function describeConfigIssues(raw) {
   }
   if (raw.foregroundKinds !== undefined && !Array.isArray(raw.foregroundKinds)) {
     issues.push('"foregroundKinds" expects an array, got ' + brief(raw.foregroundKinds) + ' -> ignored');
+  } else if (Array.isArray(raw.foregroundKinds)) {
+    // 非字符串元素会在构建 cfg 时被静默过滤掉，配置看着生效、实际没加进去。
+    const bad = raw.foregroundKinds.filter((k) => typeof k !== 'string' || k.trim() === '').length;
+    if (bad > 0) issues.push(bad + ' non-string/empty entry in "foregroundKinds" -> dropped');
   }
-  return issues;
-}
-
-/** 一条消息里所有 text 块拼起来，剥掉不可见字符和首尾空白后的可见文本。 */
-function visibleText(message) {
-  const content = message && Array.isArray(message.content) ? message.content : [];
-  let acc = '';
-  for (const part of content) {
-    if (part && part.type === 'text' && typeof part.text === 'string') acc += part.text;
+  if (raw.logTurns !== undefined && raw.logTurns !== 'empty' && raw.logTurns !== 'all') {
+    issues.push('"logTurns" expects "empty" | "all", got ' + brief(raw.logTurns) + ' -> "empty"');
   }
-  return acc.replace(ZERO_WIDTH, '').trim();
-}
-
-/** 按码位计数，避免代理对被算成 2 个字符。 */
-function codepointLength(text) {
-  return Array.from(text).length;
+  return issues;
 }
 
 /**
@@ -260,12 +273,32 @@ function codepointLength(text) {
  */
 const VISIBLE_NON_TEXT = new Set(['image', 'file', 'attachment', 'audio', 'video']);
 
-function hasVisibleNonText(message) {
+/**
+ * 一条 assistant 消息的可见性画像：可见字符数（按码位）+ 是否有可见非文本块。
+ *
+ * 0.2.4 之前这里是「两遍遍历 + 一次 Array.from()」：visibleText 先拼字符串，
+ * hasVisibleNonText 再扫一遍，最后 codepointLength 用 Array.from 建出一整个
+ * 码位数组。assistant 消息可能很长，而一个回合有几十步，这笔分配纯粹是浪费。
+ * 现在一次遍历同时得出两个结论，字符数用 for...of 累加（代理对算 1 个）。
+ * 仍然给精确总数而不是「到阈值就停」：日志里的 visibleChars 是有用的诊断值，
+ * 为了省几次迭代把它退化成「至少 1」不划算。
+ */
+function inspectMessage(message) {
   const content = message && Array.isArray(message.content) ? message.content : [];
+  let text = '';
+  let hasNonText = false;
   for (const part of content) {
-    if (part && typeof part.type === 'string' && VISIBLE_NON_TEXT.has(part.type)) return true;
+    if (!part || typeof part.type !== 'string') continue;
+    if (part.type === 'text') {
+      if (typeof part.text === 'string') text += part.text;
+    } else if (VISIBLE_NON_TEXT.has(part.type)) {
+      hasNonText = true;
+    }
   }
-  return false;
+  const visible = text.replace(ZERO_WIDTH, '').trim();
+  let chars = 0;
+  for (const _ignored of visible) chars += 1;
+  return { chars, hasNonText };
 }
 
 export function apply(ctx, config) {
@@ -283,6 +316,11 @@ export function apply(ctx, config) {
     // 0 表示关闭补写（而不是被 || 吞成 3）。
     maxNudgesPerHour: intOr(raw.maxNudgesPerHour, 3, 0, 1000),
     logFile: typeof raw.logFile === 'string' ? raw.logFile.trim() : '',
+    // 回合日志粒度：'empty'（默认）只在「空回合 / 判定被跳过」时写一行，
+    // 'all' 连正常回合的 turn/end 也写。实测 1286 行日志里 645 行是正常回合
+    // 的 turn/end —— 它们只在「某回合为什么没被判空」时才有人看，默认不值得
+    // 每回合付一次同步写。排查期把这里切 'all'。
+    logTurns: raw.logTurns === 'all' ? 'all' : 'empty',
     // 'root'（默认）只处理根会话（header.delegationDepth 为 0 或缺省）；'all' 连
     // 子代理会话一起处理。默认不碰子会话：子代理「只调工具、不回正文」是正常收尾，
     // 补写会浪费 token 甚至改变委派行为。
@@ -351,6 +389,11 @@ export function apply(ctx, config) {
 
   // 主日志写在 $DSH_HOME/logs；万一主路径不可写（受限环境/目录被锁），
   // 再退到系统临时目录，避免「插件其实活着但一行日志都没有」的误判。
+  //
+  // 目录只建一次：旧版每写一行都跑一次同步 mkdirSync，而宿主事件循环上的
+  // 每一次同步系统调用都是白给的延迟（turn/end、补写、判空都会写日志）。
+  // 写失败时把标志复位，下一次重新 mkdir —— 目录被删/权限变化仍能自愈。
+  let logDirReady = false;
   const log = (line) => {
     // 日志行内不允许出现换行：会话 id、错误文本都可能带 \n，拼进去就能在日志里
     // 伪造出一行看似插件自己写的记录（伪造 turn/end 或 guard armed 会误导排障）。
@@ -358,10 +401,14 @@ export function apply(ctx, config) {
     const record = '[' + new Date().toISOString() + '] ' + safe + '\n';
     maybeRotate();
     try {
-      fs.mkdirSync(path.dirname(logPath), { recursive: true });
+      if (!logDirReady) {
+        fs.mkdirSync(path.dirname(logPath), { recursive: true });
+        logDirReady = true;
+      }
       fs.appendFileSync(logPath, record);
       return;
     } catch {
+      logDirReady = false;
       /* 落到下面的兜底 */
     }
     try {
@@ -425,7 +472,7 @@ export function apply(ctx, config) {
    * 0.2.2 及更早写的是 "id at" 纯文本，两者都认。
    * 时间戳晚于当前时钟 + 容忍窗口的丢弃。
    */
-  const parseBudgetLine = (line, now, cutoff) => {
+  const parseBudgetLine = (line, now) => {
     const text = line.trim();
     if (text === '') return null;
     let id = '';
@@ -443,8 +490,7 @@ export function apply(ctx, config) {
       at = Number(text.slice(sp + 1).trim());
     }
     if (id === '' || !Number.isFinite(at)) return null;
-    if (at > now + BUDGET_FUTURE_SKEW_MS) return null;
-    if (at < cutoff) return null;
+    if (!withinWindow(at, now)) return null;
     return { id, at };
   };
 
@@ -454,9 +500,8 @@ export function apply(ctx, config) {
     try {
       const rawText = readTail(budgetPath, BUDGET_MAX_BYTES);
       const now = Date.now();
-      const cutoff = now - HOUR_MS;
       for (const line of rawText.split('\n')) {
-        const rec = parseBudgetLine(line, now, cutoff);
+        const rec = parseBudgetLine(line, now);
         if (!rec) continue;
         const list = budgetHistory.get(rec.id);
         if (list) list.push(rec.at);
@@ -533,9 +578,8 @@ export function apply(ctx, config) {
   let recordTicks = 0;
 
   const pruneBudgetHistory = (now) => {
-    const floor = now - HOUR_MS;
     for (const [sid, times] of budgetHistory) {
-      const kept = times.filter((t) => t >= floor && t <= now + BUDGET_FUTURE_SKEW_MS);
+      const kept = times.filter((t) => withinWindow(t, now));
       if (kept.length === 0) budgetHistory.delete(sid);
       else if (kept.length !== times.length) budgetHistory.set(sid, kept);
     }
@@ -549,9 +593,8 @@ export function apply(ctx, config) {
   const compactBudget = (now) => {
     try {
       const rawText = readTail(budgetPath, BUDGET_MAX_BYTES);
-      const cutoff = now - HOUR_MS;
       for (const line of rawText.split('\n')) {
-        const rec = parseBudgetLine(line, now, cutoff);
+        const rec = parseBudgetLine(line, now);
         if (!rec) continue;
         const list = budgetHistory.get(rec.id);
         if (!list) budgetHistory.set(rec.id, [rec.at]);
@@ -565,7 +608,14 @@ export function apply(ctx, config) {
     }
     const tmp = budgetPath + '.tmp-' + process.pid;
     fs.writeFileSync(tmp, lines.length > 0 ? lines.join('\n') + '\n' : '');
-    fs.renameSync(tmp, budgetPath);
+    try {
+      fs.renameSync(tmp, budgetPath);
+    } catch (err) {
+      // 替换失败（Windows 上目标被另一进程占用时是 EPERM/EBUSY）不能把
+      // .tmp-<pid> 留在 state 目录里：长期跑会积一堆同名的垃圾文件。
+      try { fs.unlinkSync(tmp); } catch { /* ignore */ }
+      throw err;
+    }
   };
 
   const recordNudge = (session, at) => {
@@ -709,11 +759,48 @@ export function apply(ctx, config) {
   let armed = false;
   const agents = new Map(); // session -> agent
   const agentsById = new Map(); // session.id -> agent（session 对象身份不稳定时兜底）
+  /**
+   * 会话状态双索引：WeakMap 按对象身份（快路径），Map 按 session.id（兜底）。
+   *
+   * 只按对象身份索引是原实现的一个静默失效点：宿主在不同代码路径上可能给出
+   * 不同的 session 实例（agentOf 早就为此加了 agentsById，注释也写明了这个
+   * 风险）。一旦身份漂移，turn/start 落在对象 A、assistant/message 与 turn/end
+   * 落在对象 B，B 的状态里 sawTurnStart 恒为 false，判定链整体早退 ——
+   * 插件不报错、只留下 "turn/start not observed" 一行，表现为「装了但没用」。
+   * 这与 0.2.1 修过的「orchestrator-* 被判成子代理、19 次静默跳过」是同一类故障。
+   */
   const state = new WeakMap(); // session -> per-session guard state
+  const stateById = new Map(); // session.id -> per-session guard state（有界）
+  const STATE_BY_ID_MAX = 500;
   const timers = new Set(); // 所有在跑的定时器，便于 dispose 时一次性清掉
 
+  /**
+   * 运行期聚合统计。
+   * 生产日志实测（2026-11-01 → 11-09，645 个回合）：completed 空回合为 0，
+   * 182 个空回合全是 steps=0 的 error / aborted。逐行日志回答不了
+   * 「守卫没触发是目标场景没发生，还是它坏了」—— 每 STATS_EVERY_TURNS 个回合
+   * 打一行汇总，两个问题一眼可辨；卸载时再打一行 final。
+   */
+  const stats = { turns: 0, emptyCompleted: 0, emptyOther: 0, delivered: 0, claimed: 0 };
+  const STATS_EVERY_TURNS = 50;
+  const statsLine = (suffix) => 'stats: turns=' + stats.turns
+    + ' emptyCompleted=' + stats.emptyCompleted
+    + ' emptyOther=' + stats.emptyOther
+    + ' delivered=' + stats.delivered
+    + ' claimed=' + stats.claimed
+    + (suffix ? ' (' + suffix + ')' : '');
+
+  /** 只读查找（不创建状态）：先对象身份、再 id。丢弃事件等旁路用它。 */
+  const stateLookup = (session) => {
+    if (!session || typeof session !== 'object') return undefined;
+    const direct = state.get(session);
+    if (direct) return direct;
+    const id = typeof session.id === 'string' ? session.id : '';
+    return id !== '' ? stateById.get(id) : undefined;
+  };
+
   const stateOf = (session) => {
-    let s = state.get(session);
+    let s = stateLookup(session);
     if (!s) {
       s = {
         turn: null,
@@ -763,7 +850,19 @@ export function apply(ctx, config) {
         nudgeTimes: historyOf(session),
         timer: null,
       };
-      state.set(session, s);
+    }
+    // 两个索引都写：对象身份是最快路径，id 是宿主换 session 实例时的兜底。
+    if (session && typeof session === 'object') state.set(session, s);
+    const sid = session && typeof session.id === 'string' ? session.id : '';
+    if (sid !== '' && stateById.get(sid) !== s) {
+      if (stateById.size >= STATE_BY_ID_MAX && !stateById.has(sid)) {
+        // Map 保序，第一个就是最早插入的一条。淘汰是安全的：该会话的下一次
+        // 事件会拿到一个新状态（字段齐全，只是当回合累计值从零开始），
+        // 而状态本身是缓存，淘汰不等同于丢判定能力。
+        const oldest = stateById.keys().next();
+        if (!oldest.done) stateById.delete(oldest.value);
+      }
+      stateById.set(sid, s);
     }
     return s;
   };
@@ -880,7 +979,7 @@ export function apply(ctx, config) {
         const agent = payload && payload.agent ? payload.agent : payload;
         const session = agent && agent.session;
         if (!session) return;
-        const s = state.get(session);
+        const s = stateLookup(session);
         if (s) {
           clearTimer(s, null);
           clearNudgeInFlight(s, null);
@@ -889,7 +988,12 @@ export function apply(ctx, config) {
           s.guardIds = [];
         }
         agents.delete(session);
-        if (typeof session.id === 'string' && session.id !== '') agentsById.delete(session.id);
+        if (typeof session.id === 'string' && session.id !== '') {
+          agentsById.delete(session.id);
+          // 两个状态索引都要清：否则 id 兜底会一直把已销毁会话的状态留在表里。
+          stateById.delete(session.id);
+          state.delete(session);
+        }
       } catch (err) {
         log('agent/disposed handler error: ' + errText(err));
       }
@@ -911,7 +1015,7 @@ export function apply(ctx, config) {
       const agent = (payload && payload.agent) || payload;
       const session = agent && agent.session;
       if (!session) return;
-      const s = state.get(session);
+      const s = stateLookup(session);
       if (!s) return;
       const message = payload && payload.message;
       const id = message && typeof message.id === 'string' ? message.id : '';
@@ -1013,6 +1117,7 @@ export function apply(ctx, config) {
       const fn = attempts[i][1];
       try {
         fn();
+        stats.delivered += 1;
         log('nudge DELIVERED via ' + label + ' [' + tag(session) + ']');
         return message;
       } catch (err) {
@@ -1024,16 +1129,23 @@ export function apply(ctx, config) {
   };
 
   /**
-   * 修剪本会话最近一小时的投递时刻。
-   * 未来时间戳（> now + 容忍窗口）必须一起丢掉：`now - t` 对它是负数，
-   * `< HOUR_MS` 恒成立，只按上半句过滤就会把它永久留在预算里。
+   * 修剪本会话最近一小时的投递时刻，返回窗口内还剩几次。
+   * 未来时间戳的丢弃规则统一走 withinWindow（见该函数的注释）。
    */
   const pruneBudget = (s, now) => {
-    s.nudgeTimes = s.nudgeTimes.filter(
-      (t) => now - t < HOUR_MS && t <= now + BUDGET_FUTURE_SKEW_MS);
+    s.nudgeTimes = s.nudgeTimes.filter((t) => withinWindow(t, now));
     return s.nudgeTimes.length;
   };
 
+  /**
+   * 本会话窗口内的补写次数是否已达上限。
+   * 「定时器到点时」与「turn/end 判定时」各查一次（两者相隔 delayMs），
+   * 两处必须用同一套修剪+比较；写成分散表达式时，一处改了窗口、另一处没改
+   * 就会从某条路径漏限流。
+   */
+  const budgetExhausted = (s, now = Date.now()) =>
+    pruneBudget(s, now) >= cfg.maxNudgesPerHour;
+
   /**
    * 排一笔补写。返回结果字符串，供调用侧决定是否把本回合标记为「已处理」：
    *   'scheduled' | 'dry-run' | 'skipped:<原因>' | 'error'
@@ -1086,7 +1198,7 @@ export function apply(ctx, config) {
             return;
           }
           // 预算按「真正投递成功」计，取消/失败不占额度。
-          if (pruneBudget(s, Date.now()) >= cfg.maxNudgesPerHour) {
+          if (budgetExhausted(s)) {
             log('hourly nudge budget exhausted (' + s.nudgeTimes.length + ') -> no nudge');
             return;
           }
@@ -1109,7 +1221,7 @@ export function apply(ctx, config) {
             s.nudgeTimes.push(Date.now());
             if (typeof message.id === 'string' && message.id !== '') {
               s.guardIds.push(message.id);
-              while (s.guardIds.length > 8) s.guardIds.shift();
+              while (s.guardIds.length > GUARD_IDS_MAX) s.guardIds.shift();
             }
             // 跨重启限流：把投递时刻落盘，重启后的新进程回读最近一小时。
             recordNudge(session, s.nudgeDeliveredAt);
@@ -1179,6 +1291,7 @@ export function apply(ctx, config) {
               return;
             }
             s.awaitingGuardTurn = true;
+            stats.claimed += 1;
             log('guard message claimed' + (late ? ' (LATE, from guardIds)' : '')
               + ' turn=' + (typeof data.turn === 'number' ? data.turn : '?')
               + ' matched=true [' + tag(session) + ']');
@@ -1204,9 +1317,10 @@ export function apply(ctx, config) {
           const message = data.message;
           if (!message || message.role !== 'assistant') return;
           if (typeof data.turn === 'number' && s.turn !== data.turn) resetTurnCounters(s, data.turn);
+          const seen = inspectMessage(message);
           s.assistantMessages += 1;
-          s.visibleChars += codepointLength(visibleText(message));
-          if (hasVisibleNonText(message)) s.visibleNonText = true;
+          s.visibleChars += seen.chars;
+          if (seen.hasNonText) s.visibleNonText = true;
           return;
         }
 
@@ -1236,9 +1350,19 @@ export function apply(ctx, config) {
 
         const sawStart = s.sawTurnStart === true;
         const empty = !s.visibleNonText && s.visibleChars < cfg.minVisibleChars;
-        log('turn/end turn=' + turn + ' kind=' + kind + ' steps=' + s.assistantMessages
-          + ' visibleChars=' + s.visibleChars + ' empty=' + empty + ' wasGuard=' + wasGuard
-          + ' sawStart=' + sawStart + ' [' + tag(session) + ']');
+        stats.turns += 1;
+        if (empty) {
+          if (kind === 'completed') stats.emptyCompleted += 1;
+          else stats.emptyOther += 1;
+        }
+        // 正常回合默认不写行：实测这类行占日志的一半，而它只在「某回合为什么
+        // 没被判空」时才有人看。排查期把 logTurns 切成 'all' 即可恢复逐回合记录。
+        if (empty || cfg.logTurns === 'all') {
+          log('turn/end turn=' + turn + ' kind=' + kind + ' steps=' + s.assistantMessages
+            + ' visibleChars=' + s.visibleChars + ' empty=' + empty + ' wasGuard=' + wasGuard
+            + ' sawStart=' + sawStart + ' [' + tag(session) + ']');
+        }
+        if (stats.turns % STATS_EVERY_TURNS === 0) log(statsLine(null));
 
         if (!sawStart) {
           // 插件在回合中途才加载/热重载：看不到 turn/start，也就漏掉了该回合早前的
@@ -1264,7 +1388,7 @@ export function apply(ctx, config) {
           return;
         }
         const now = Date.now();
-        if (pruneBudget(s, now) >= cfg.maxNudgesPerHour) {
+        if (budgetExhausted(s, now)) {
           log('hourly nudge budget exhausted (' + s.nudgeTimes.length + ') -> no nudge');
           return;
         }
@@ -1291,6 +1415,9 @@ export function apply(ctx, config) {
         try { clearTimeout(timer); } catch { /* ignore */ }
       }
       timers.clear();
+      // 卸载时打一行 final 统计：短命进程（频繁重启的宿主）也留得下
+      // 「这段时间到底判了多少回合、补了几笔」的证据，不必等满 50 个回合。
+      try { log(statsLine('final')); } catch { /* 日志失败不影响卸载 */ }
     });
   } catch { /* effect 不可用时忽略 */ }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dsh-turn-recovery",
-  "version": "0.2.3",
+  "version": "0.2.4",
   "description": "DSH plugin: detect assistant turns that end with no visible text (zero-width placeholder) and re-prompt the model to write the answer; also injects a visible-closing-answer system prompt rule.",
   "license": "MIT",
   "author": "HERO476",