Skip to content

修复本地上传 processing_options 透传 - #203

Merged
zj1123581321 merged 7 commits into
mainfrom
card/vta-upload-B-options-261009
Oct 8, 2026
Merged

zj1123581321 merged 7 commits into
mainfrom
card/vta-upload-B-options-261009

Conversation

@zj1123581321

@zj1123581321 zj1123581321 commented Oct 8, 2026 •

Copy link
Copy Markdown
Collaborator

摘要

本 draft PR 承接原 Task16 的 B 卡实现,并在继承 H1 顺序/文件所有权修复的基础上,补齐公开 processing_options 从上传 producer 到实际 worker consumer 的透传。原 PR #202 待本继任 PR 建立后由 lead 关闭;本 PR supersedes #202。

本次变更

  • 在 CacheManager.accept_local_upload() 的同一 SQL 临界 queue payload 中发布已持久化的规范化 processing_options。
  • 使用现有真实 HTTP→SQLite→runtime queue→dispatcher→process_transcription fixture,覆盖全关、混合、缺省三类合法输入,并断言 header/SQL/queue/consumer 一致。
  • 更新 B-progress 映射与低级诊断意见接受不修说明。

验证

  • 预修复红验:三组真实边界测试以 AssertionError 暴露 queue payload 缺少 options。
  • 窄测:uv run --frozen pytest -q tests/unit/test_upload_routes.py tests/unit/test_upload_dispatch.py tests/integration/test_upload_intake.py tests/unit/test_api_routes.py,exit 0。
  • 完整 make test,pytest 到 100%,3 个既有测试 skip,exit 0;原始日志为 /tmp/vta-upload-B-options-make-test.log。

不 ready、不 merge;交由 lead 做 H1→H2 独立复审、正式 CI 与合并。

当前验收合同与接受不修事项

本PR承接 #202 的完整实现,当前固定head为6a497263e67789b4ce8fef39d407e6e8f77faff3。SQL root/accepted/单次同步queueput在同无await临界,commit后才正式202;SQL失败留下的已发queue项由dispatcher持久admission拒绝,零PROCESSING/submit/ASR,不采用事后撤回补偿。receiving清理先事务确认并退休仍过窗/未移交记录,再删除其暂存文件,已accepted文件不删。

合法processing_options按与SQL相同options_json生成实际queuepayload,传给真实dispatcher/consumer,全关/混合/缺省三类别不能静默默认化;源metadata/独立SQLite行/真实queuepayload/consumer参数都有回归断言。旧URL选项行为不变,未添配置/状态/重试或第二纠错事实源。主脑独立复跑当前三选项场景及四发布/清理顺序场景均通过,完整make由执行器报告退出0;独立复验与正式ready主审尚待,不能把draft绿作正式通过。

当前个人使用范围接受两项不阻断的诊断欠缺:暂存目录创建故障已明确HTTP500且释放预算/inflight,但重复查询的receiving error_code可能为null;SQL commit故障已明确500、回滚且stale队列不处理,但错误码为receive_incomplete。本轮不承诺每个前置故障都有细分准确标签;不以它们为理由增加广泛catch/重试/新状态,不把500说成成功,UI也不可将receiving/null task_id当正式受理。两项保留后续诊断改进,未改变公开分享或既有错误HTTP语义。

配置四个限额生产实测未知、示例null且环境flag默认off;临时合成媒体/测试正向限额不是生产容量或恢复域保证。本PR不部署、不运行生产ASR/LLM/通知,C/D/E/F完整功能/证明仍后续卡。

zj1123581321 and others added 7 commits October 8, 2026 21:15
Agent-Executor: codex-pi
Agent-Model: gpt-6-luna
Agent-Effort: xhigh
Dispatch-Id: dlg-20261008-130923-6b2088
Task-Id: VideoTranscriptAPI-20261008-16
Agent-Executor: codex-pi
Agent-Model: gpt-6-luna
Agent-Effort: xhigh
Dispatch-Id: dlg-20261008-130923-6b2088
Task-Id: VideoTranscriptAPI-20261008-16
Agent-Executor: codex-pi
Agent-Model: gpt-6-luna
Agent-Effort: xhigh
Dispatch-Id: dlg-20261008-130923-6b2088
Task-Id: VideoTranscriptAPI-20261008-16
Agent-Executor: codex-pi
Agent-Model: gpt-6-luna
Agent-Effort: xhigh
Dispatch-Id: dlg-20261008-151543-50422d
Task-Id: VideoTranscriptAPI-20261008-16
Agent-Executor: codex-pi
Agent-Model: gpt-6-luna
Agent-Effort: xhigh
Dispatch-Id: dlg-20261008-151543-50422d
Task-Id: VideoTranscriptAPI-20261008-16
Agent-Executor: codex-pi
Agent-Model: gpt-6-luna
Agent-Effort: xhigh
Dispatch-Id: dlg-20261008-151543-50422d
Task-Id: VideoTranscriptAPI-20261008-16
Co-authored-by: Cursor <cursoragent@cursor.com>
Agent-Executor: cursor
Agent-Model: gpt-5.6-luna-xhigh
Agent-Effort: unknown
Dispatch-Id: dlg-20261008-165700-787f04
Task-Id: VideoTranscriptAPI-20261009-02
@zj1123581321
zj1123581321 marked this pull request as ready for review October 8, 2026 17:44
@github-actions

github-actions Bot commented Oct 8, 2026

Copy link
Copy Markdown

Required Gate v2 — 状态面板

当前状态:pass · 可合并

当前裁决:code_pass / primary_pass

PR 累计主审轮(影子):1/5 · collecting

历史可能不完整:1 个制品处理失败:URLError;1 个历史重建预算耗尽

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

Run Attempt Head 状态 PR 累计主审轮(影子) 收件人动作
37813770975 1 6a49726 skipped — 无需动作(主审未跑,绿≠过审)
37818928677 1 6a49726 pass 1/5 · collecting 可合并

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

@zj1123581321
zj1123581321 merged commit c7b3856 into main Oct 8, 2026
26 checks passed
@zj1123581321
zj1123581321 deleted the card/vta-upload-B-options-261009 branch October 8, 2026 18:09
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