Repository navigation
工具字段完善 - #447
工具字段完善#447
Conversation
|
Start review |
52997b3 to
b5e4401
Compare
|
Start review |
|
Start review |
cuericlee
left a comment
cuericlee
left a comment
There was a problem hiding this comment.
已修复并逐项实测通过
query_task 轮询报错已解决：_do_query 改回同步函数，轮询逻辑也同步重写。我实测了四种情况：追加查询上限为 2 时正好做 3 次查询（首次加 2 次）后返回运行中状态且不报错；第 2 次查询到终态时正常返回结果；默认参数只查一次；持续轮询加上总时长限制会按时停止。
环境变量已改回 STATLESS_HTTP，代码和中英文文档一致，不会再出现配置被静默忽略的问题。
未接线的新中间件文件已删除，仓库里没有残留引用。
仍然存在但不阻塞合入的项
这个包至今没有仓库内测试，本次改动的字段关系校验、配置解析和轮询逻辑全靠手工验证。你附的测试文档我依然没有访问权限，无法确认覆盖范围，建议后续把这几类单测补进仓库。
文档仍把 MEDIAKIT_ENDPOINT 描述为「可选的服务地址覆盖」，但代码现在只接受官方域名，且非标准端口（如带 :8443）也能通过校验。建议补一句域名限制说明，或显式限定端口。
查询结果改为嵌套在 result 字段下、同步接口缺少 result 时改为报错，属于调用方可见的行为变化，建议在发布说明里明示。
separate_voice 默认输出改为 aac、部分旧参数被移除，与服务端新版本的对齐情况我无法从代码侧确认，依赖你的线上验证。
本轮验证范围：80 个工具全部成功注册，工具名与路由一一对应，21 条字段大小关系约束引用的字段全部存在；非法区域参数会在发出请求前被拦截，合法参数正常放行；全部文件编译通过。未对真实 MediaKit 服务发请求，意图对照来源（PR 说明和测试文档）仍不完整，这部分置信度有限。
工具字段完善