Skip to content

修复 summary-ratio 端点在未配置 storage.audit_db 时 500 - #64

Merged
zj1123581321 merged 1 commit into
mainfrom
card/auditdb-fix
Aug 24, 2026
Merged

修复 summary-ratio 端点在未配置 storage.audit_db 时 500#64
zj1123581321 merged 1 commit into
mainfrom
card/auditdb-fix

Conversation

@zj1123581321

Copy link
Copy Markdown
Collaborator

背景

生产 n305 实测 PR#63 新端点 GET /api/audit/summary-ratio 返回 500:生产 config.jsonc 不写 storage.audit_db(合法形态,context.py 有默认值约定),端点把 None 直接传给 sqlite3.connect("file:None?mode=ro")

修复

复用运行中 app 已解析好的 get_audit_logger().db_path 兜底(与 context.py 同款语义,单一事实源,不新增默认路径逻辑)+ 测试锁定「配置无该键时端点 200」。

验证

  • 本地 tests/unit/ + tests/llm/ 全量 2827 passed
  • 红验:新测试在 base commit 上失败(锁的是本次修复)

Task-Id: VideoTranscriptAPI-20260824-03

Agent-Executor: grok
Agent-Model: grok-4.6
Agent-Effort: unknown
Dispatch-Id: dlg-20260824-102347-4edf77
Task-Id: VideoTranscriptAPI-20260824-03
@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 状态 收件人动作
32718106840 1 a6d7b81 skipped 无需动作(主审未跑,绿≠过审)
32718433164 1 a6d7b81 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: a6d7b8137dfd17fd405acc3cf32c30a03c3c9be6
  • Round: 2
  • 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 连续游标。

@zj1123581321
zj1123581321 marked this pull request as ready for review August 24, 2026 10:45
@zj1123581321
zj1123581321 merged commit 8233e3f into main Aug 24, 2026
20 checks passed
@zj1123581321
zj1123581321 deleted the card/auditdb-fix branch August 24, 2026 10:51
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