[bug] acp.json 的 repetitionGuard 键被静默丢弃（user-config KNOWN 白名单缺失）— 已修复于 #352

[ranxianglei]

来源: #351 #351 分析时发现（为新增 degenerationGuard 走 acp.json 配置路径时顺带核查）

问题

src/user-config.ts 的 KNOWN 白名单（pickKnown 的过滤集合）没有包含 repetitionGuard。CONFIGURATION.md 明确文档化了该键可经 acp.json 配置（#308 引入），但用户写入后会被 loadUserConfig 静默丢弃——acp.json 里写 "repetitionGuard": false 毫无效果，且无任何日志提示（unknown key 本应有 warn，但白名单过滤发生在 warn 之前）。

影响

依赖 acp.json 关闭/调参工具调用熔断的用户实际仍跑默认值（warn=3/abort=5）。对贪心小模型场景（#308 的目标场景）意味着误配的 abort 阈值不生效。

修复

已在 PR #352 中补入 KNOWN + UserAcpConfig 类型（与 degenerationGuard 同批）。该 PR 合并后本 issue 可关闭。

教训 / 建议

KNOWN 白名单与 CONFIGURATION.md 的文档键表缺乏一致性测试——建议加一个测试：解析 CONFIGURATION.md 主键表中的所有顶层键，断言均在 KNOWN 中（文档漂移即红）。

[ranxianglei]

🤖 Powered by ework · qwen3.8-27b

已按 bug 流程完成审核与修复验证，结论如下。

一、验证：问题真实存在（master @ 6c10dab）

• src/user-config.ts:57-63：KNOWN 白名单确实缺 repetitionGuard（同批的 degenerationGuard 也缺）。
• pickKnown（同文件 :66-72）对白名单外的键静默丢弃；loadUserConfig（:27-46）没有任何 unknown-key 告警分支——catch 块只对文件缺失之外的加载错误打 load-failed。所以 acp.json 里写 "repetitionGuard": false 确实被静默丢弃、零日志痕迹，用户实际跑默认值（warn=3/abort=5）。影响评估成立。
• CONFIGURATION.md 对照：repetitionGuard 在 ## Parameter Reference → ### Summary 的 "Top-level keys" 主键表中有行，且有独立的 ## Tool Call Repetition Guard 章节（通用工具重复熔断缺失:acp_status 30× 字节级相同调用死循环(贪心模型,任何工具可成环;compress 熔断器覆盖不到) #308 引入）——文档侧完备，纯代码侧白名单漏项。

二、修复核验（PR #352）

逐行核了 origin/pr-352 的 diff：src/user-config.ts 确实在 KNOWN 补入 "repetitionGuard" 与 "degenerationGuard"，并同步补齐 UserAcpConfig 类型（repetitionGuard?: boolean | RepetitionGuardConfig、degenerationGuard?: boolean | DegenerationGuardConfig）。修复完整、正确。

但 PR #352 目前仍是 open 未合并（base 已是最新 master 6c10dab）。按仓库规则合并仅人工操作，我不代合。#352 合并后本 issue 即可关闭。

三、一致性测试建议：已评估，直接实现

评估结论：合理（防同类回归）、低耦合（只读仓库内文档 + 复用现有测试隔离模式）、无设计冲突，故直接实现而非再讨论。设计要点：

• 解析 CONFIGURATION.md ## Parameter Reference 章节中所有表头为 | Key | … 的参数表（自动跳过 status legend 与环境变量表），取每行键首个 . 前的前缀作为顶层键；
• 不导出内部 KNOWN，改用行为断言：把解析出的全部顶层键写入临时 acp.json（值统一 true，pickKnown 不做类型校验），断言全部存活于 loadUserConfig 返回值——测的是"无静默丢弃"这一用户可见契约，对内部实现重构免疫；
• 非空守卫：断言解析结果包含 enabled/debug/delegate/compress/prompts 等核心稳定键且总数 ≥9，防止解析逻辑损坏时测试空转通过；
• 本次只覆盖英文主表（issue 所指范围）；zh-CN 可后续对称扩展。

测试 PR 稍后补链接（引用本 issue）。⚠️ 注意：该测试在 #352 合并前 CI 会是红的——当前 master 上 repetitionGuard 仍被丢弃，正是测试应当捕获的现象。建议顺序：先合 #352，再合该测试 PR。

---

一句话总结：确认 acp.json 的 repetitionGuard 静默丢弃问题真实存在（白名单漏项且无 unknown-key 告警路径），修复已在 PR #352 中完成并核验正确（待人工合并后关单）；另按建议新增 CONFIGURATION.md ↔ loadUserConfig 一致性回归测试 PR，#352 合并前其 CI 预期为红。

[ranxianglei]

🤖 Powered by ework · qwen3.8-27b

一致性测试 PR 已就绪：**https://github.com/ranxianglei/billion-context-pi/pull/355**（body 含 Fixes #354）

• 测试文件：tests/user-config-doc-consistency.test.ts（新增 79 行，零 src 改动）。
• 实现：解析 CONFIGURATION.md ## Parameter Reference 章节中所有表头为 | Key | … 的参数表（status legend 与环境变量表自动排除），取每行键首个 . 前前缀为顶层键；把全部顶层键写入临时 .pi/acp.json（值统一 true）断言全部存活于 loadUserConfig 返回值——行为断言，不导出内部 KNOWN，对内部重构免疫；非空守卫（≥9 键 + enabled/debug/autoUpdate/delegate/compress/prompts 必须出现），防止解析逻辑损坏时空转通过。
• 双向验证：当前 master（6c10dab）上恰好失败 1 例，错误信息点名 documented top-level key 'repetitionGuard' was silently dropped by loadUserConfig——精确复现本 issue；在 origin/pr-352（54b9fbb）worktree 中运行全绿（含其新增的 degenerationGuard 文档行）。全量套件 657 例：仅该测试失败、653 通过、3 个既有 skip；typecheck/build 干净。
• ⚠️ fix: degeneration guard — collapse single-codepoint runs in assistant text/thinking + one-shot recovery notice #352 合并前该 PR 的 CI 保持红——正是测试应捕获的现象。建议顺序：先合 fix: degeneration guard — collapse single-codepoint runs in assistant text/thinking + one-shot recovery notice #352 → 该 PR 立即转绿可合 → 合并后本 issue 经 Fixes #354 自动关闭。

---

一句话总结：已按 issue 建议落地 CONFIGURATION.md ↔ loadUserConfig 一致性回归测试（PR #355），master 上红（精确复现 repetitionGuard 静默丢弃）、#352 合并后绿，建议先合 #352 再合 #355。

[ranxianglei]

标题即结论:已修复于 #352(repetitionGuard 已入 KNOWN 白名单,master src/user-config.ts:65 可验证)。守卫测试 PR #355 已 rebase 全绿待合并。