Skip to content

总结层篇幅预算治理:prompt 收敛 + max_tokens + 压缩重试 + 分带监控 - #63

Merged
zj1123581321 merged 2 commits into
mainfrom
card/summary-budget
Aug 24, 2026
Merged

总结层篇幅预算治理:prompt 收敛 + max_tokens + 压缩重试 + 分带监控#63
zj1123581321 merged 2 commits into
mainfrom
card/summary-budget

Conversation

@zj1123581321

Copy link
Copy Markdown
Collaborator

背景

生产 8/13 DeepSeek v4 Pro 更新后总结层输出失控膨胀(短视频总结超原文占比 78%→97%,极端 653%;详见任务卡基线数据)。根因:summary prompt 的绝对字数下限(每小节≥500字)+ 无任何输出预算控制,模型更新后更严格执行扩写指令。

产品定位(已拍板):总结层是主阅读面,允许为结构化重述适度超限;超预算是降质不是失败。

改动(四层防线,预算函数为唯一事实源)

  • L0 新增 summary_budget.py:S/M/L 三带预算(目标/硬顶/max_tokens=硬顶×1.5),可配置覆盖
  • L1 prompt 手术:删"框架与心智模型"节;绝对字数下限改为信息量导向措辞;user prompt 顶部注入动态篇幅预算行
  • L2 LLMClient.call(max_tokens=None) 透传至请求 payload(None 时字段完全缺席,其他调用方零行为变化)
  • L3 落盘前超硬顶压缩重试一次;仍超取较短者接受落盘 + summary_over_budget_accepted warning;禁止截断、状态仍 GENERATED
  • L4 新增 GET /api/audit/summary-ratio?days=30 分带比例监控
  • 修复 skipped_short 任务不再显示误导性的"占原文 100.x%"统计

验证

  • uv run python -m pytest tests/llm/ tests/unit/:2824 passed
  • 轴表四格(首答×重试结果)表驱动测试覆盖
  • 红验:新测试拷入 base commit 运行确认失败(测试确实锁本次改动)
  • max_tokens 断言实际请求 payload(跨边界纪律)

Task-Id: VideoTranscriptAPI-20260824-02

建立预算函数为唯一事实源,prompt 注入篇幅目标、LLM call 加 max_tokens、
超硬顶压缩重试一次并 accept 落盘,新增 /api/audit/summary-ratio 监控端点,
修复 skipped_short 误导性统计展示。

Co-authored-by: Cursor <cursoragent@cursor.com>
Agent-Executor: cursor
Agent-Model: composer-2.5
Agent-Effort: unknown
Dispatch-Id: dlg-20260824-085655-4fbb2d
Task-Id: VideoTranscriptAPI-20260824-02
@github-actions

github-actions Bot commented Aug 24, 2026

Copy link
Copy Markdown

Required Gate v2 — 状态面板

当前状态:pass · 可合并

当前裁决:code_pass / primary_pass

Gate 历史(v1;来源为持久化 gate_terminal 制品)

Run Attempt Head 状态 收件人动作
32712072736 1 68e2107 skipped 无需动作(主审未跑,绿≠过审)
32714211440 1 79f5c97 skipped 无需动作(主审未跑,绿≠过审)
32714670098 1 79f5c97 pass 可合并

历史行按 run_id + run_attempt 去重并只增不删;删除本评论后可由 gate_terminal 制品重建。

@github-actions

github-actions Bot commented Aug 24, 2026

Copy link
Copy Markdown

⚙️ Review ledger state(机器状态记录,非评审结论)

这是 review ledger 的机器状态记录,不代表评审结论,通常无需任何操作。

机器状态明细
  • Commit: 79f5c97582f80e2fe185325c9ff8014c28ce0876
  • Round: 3
  • Status / findings: pass / 0
  • Reviewer: codex-sub
  • Comparison: same_head_rerun; stable/missing/appeared = 0/0/0

完整数据保存在 codex-review-ledger-v2 artifact;此 sticky comment 仅保存 v2 epoch 的跨 rerun 连续游标。

Co-authored-by: Cursor <cursoragent@cursor.com>
Agent-Executor: cursor
Agent-Model: composer-2.5
Agent-Effort: unknown
Dispatch-Id: dlg-20260824-094037-24901c
Task-Id: VideoTranscriptAPI-20260824-02
@zj1123581321
zj1123581321 marked this pull request as ready for review August 24, 2026 10:01
@zj1123581321
zj1123581321 merged commit 80c7335 into main Aug 24, 2026
20 checks passed
@zj1123581321
zj1123581321 deleted the card/summary-budget branch August 24, 2026 10:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant