Skip to content

fix: enforce confirmed corrections and audit audio quality - #25

Merged
SidUParis merged 4 commits into
mainfrom
fix/volc-protocol-terminal-correction
Sep 4, 2026
Merged

fix: enforce confirmed corrections and audit audio quality#25
SidUParis merged 4 commits into
mainfrom
fix/volc-protocol-terminal-correction

Conversation

@SidUParis

Copy link
Copy Markdown
Owner

摘要

  • 按火山原始 WebSocket 契约将动态上下文放入 request.corpus.context,保留托管词表 selector,并以保守内容预算发送完整 hotword。
  • 在 provider final 之后增加有界、本地、可回放的明确纠错阶段;只执行手工规则与 explicit-active 规则,保护 URL/代码/英文词界,异常或越界时回退原始终稿。
  • 保留原始 provider_final,将纠错与 clean 分成顺序明确的 delivery pipeline,并把新记录升级为 schema v5。
  • 增加不含正文的 result.text / definite 选择诊断。
  • 新增私有、signal-only 音频质量审计工具;原始 utterance 保持二文件不可变,完整 manifest 才表示一次质量快照,标签明确为 non-gold。

验证

  • Voice: 810 tests passed under Xvfb + G_DEBUG=fatal-warnings
  • Engine: 38 tests passed
  • Scripts/installer/packaging: 165 tests passed
  • Audio quality adversarial suite: 25 tests passed
  • Ruff check/format, compileall, diff check, clean single-branch secret scan: passed
  • Independent protocol, correction, quality-publisher, and combined-tree reviews: no remaining P0/P1

边界

  • 未在测试中启动麦克风或调用真实 ASR provider。
  • correct_words 仍作为 provider best-effort context;最终明确纠错不依赖服务端是否采纳。
  • 质量分档是信号启发式,不是人工听感、转录金标或自动训练准入。

Copilot AI lite review requested due to automatic review settings September 4, 2026 06:58

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@SidUParis
SidUParis merged commit b8571a1 into main Sep 4, 2026
4 checks passed
@SidUParis
SidUParis deleted the fix/volc-protocol-terminal-correction branch September 4, 2026 07:17
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.

2 participants