[Bug]: 压缩质量检测失败时返回的错误信息过长，导致额外的上下文消耗 #396

Activity

1. WantexZard

   added

   bugSomething isn't workingSomething isn't working

   on Sep 15, 2026
2. ranxianglei commented on Sep 15, 2026

   

   ranxianglei

   on Sep 15, 2026

   Owner

   More actions

   🤖 Powered by ework · qwen3.8-27b

   Triage 结论（Bug 类，三步审核通过）

   1. 复现/查证：确认存在。pre-commit quality gate 拒绝压缩时，buildQualityRejectionError（lib/compress/quality-gate/rejection.ts）把完整的 HOW_TO_COMPRESS_RULES（4,936 字符 ≈ ~1,234 tokens）外加一段多行 CRITICAL 警告嵌入返回给模型的错误信息。而且同一份规则已经存在于 ACP 系统提示中（lib/prompts/system.ts:58，每次请求都发送）——拒绝时注入的这部分是 100% 冗余内容，你截图里的 KEEP VERBATIM / DROP / PRIORITY / KEEP MARKERS 全段都是重复注入。
   2. 层次判断：你描述的就是根因本身，而非表面现象——根因不是"报错文案太长"，而是"系统提示中已有的内容被重复塞进 tool error payload"。修复方向是去掉重复注入，而不是单纯裁剪措辞。
   3. 方案评估：你的 Expected Behavior（失败原因 + 关键指标 + 简短重试指引，不重复注入完整规则）打到了核心，实现完全按此方向进行。

   分析中发现并一并修复的两个附加问题：

   • result.reason（如 "Summary too short: …" / "Content coverage too low: …"）此前根本没有出现在消息里——这恰是你要的"失败原因"——已补上；
   • 原重试文本声称 "Without acknowledgeRisk: true, the compression will be rejected again"，与实际行为不符（compress acknowledgeRisk failed #301/fix(compress): make preemptive acknowledgeRisk a no-op instead of an error [stable] #303 语义：重写后摘要若通过 gate，不带 flag 也会被接受；flag 仅在已有一次拒绝、qualityGateRetryPending 置位后才生效）。已修正措辞。

   效果：单次拒绝的注入量从 ~5.4K 字符（~1.4K tokens）降到 ~600 字符（~150 tokens），减少约 85%。gate 本身的拦截行为不变；E2E fake-LLM 依赖的标记字符串（COMPRESSION REJECTED / QUALITY GATE FAILURE）保留在头部。全量测试 1,133 pass / 0 fail，typecheck/build 通过。

   重复判定：与 #339 相关但不重复——#339 讨论的是参数语义与交互设计（dangerous 残留、acknowledgeRisk 交互、advisory 模式），本 issue 是拒绝 payload 的上下文消耗问题，两者独立处理；advisory 模式等设计议题仍留在 #339 由 owner 拍板。

   PR: #397 （双 agent 评审进行中，完成后更新）
3. ranxianglei commented on Sep 15, 2026

   

   ranxianglei

   on Sep 15, 2026

   Owner

   More actions

   🤖 Powered by ework · qwen3.8-27b

   进展更新：评审与 CI 全部完成，PR 可以合并（合并操作按规范由人工执行）

   • 双 agent 独立评审（AGENTS.md §5.3/§5.6）：代码评审 APPROVE、测试评审 APPROVE。测试评审做了变异验证——把 master 的旧 rejection.ts 换回去跑新测试，3 个新/改测试如预期失败（缺 reason / 规则被重新内嵌 / 5811 字符超预算），恢复后 15/15 全过，证明新测试确实能抓住本 bug。两条 NIT 已直接修复并推送（补 Retry: 断言；acknowledgeRisk 生效窗口措辞精确化为"下一次 compress 调用"——qualityGateRetryPending 在每次 compress 调用开始时清除）。
   • CI 6/6 绿：pr-validation、build、build-artifact、test (Node 22)、test (Node 24)、e2e。其中 e2e 通过 fake-LLM 服务端到端验证了拒绝→重试链路不受影响（检测用的标记字符串保留在消息头部）。
   • 最终拒绝消息样例（620 字符，修复前 ~5800）：

   ⚠️ COMPRESSION REJECTED — QUALITY GATE FAILURE
   
   Range: m00001–m00005
   Reason: Summary too short
   Original: ~1000 tokens
   Summary: 18 chars
   Ratio: 222.2:1
   Retention: 0.45%
   Gate layer: L1-length
   rougeF1: ?
   top20Recall: ?
   
   Retry: rewrite a more complete summary that preserves critical details
   (file paths, decisions, exact values, errors) and call compress again on
   the same range — the gate re-evaluates automatically. Full compression
   rules are already in your system prompt. If you are confident the summary
   is correct despite the metrics, add "acknowledgeRisk": true to bypass the
   quality gate on your next compress call.
   

   PR: #397 — 请 owner 确认后自行合并（Agent 无合并权限）。
4. ranxianglei

   added a commit that references this issue on Sep 15, 2026

   Merge pull request #397 from ranxianglei/2026-09-15_concise-quality-r…

   ...

   e3536b3
5. ranxianglei

   closed this as completedin #397on Sep 15, 2026