feat: 完善统一金融数据网关 - #79
Conversation
统一 provider-neutral 请求类型,保留路由 provenance,并覆盖行情、历史和基本面能力。
helsome
left a comment
There was a problem hiding this comment.
当前实现可以按 #22 的增量 gateway wiring 审,不要求你补完整真实双 provider E2E,但正式测试报告还缺两项:请补 Bun 版本和 OS/平台。现有 typecheck + provider/capability 102 tests 不需要扩大重跑。
另外请把 Closes #22 改成 Refs #22 / Part of #22:你已经明确 Longbridge CLI 未安装、没有真实第二 provider 结果,因此还不足以关闭 #22 的“两种真实 provider + provenance/fallback”整体验收。补这两个小项后即可继续按安全增量审核。
helsome
left a comment
There was a problem hiding this comment.
测试材料已补齐:Bun 1.4.2 / Windows,provider + capability 102 tests 通过、全仓 typecheck 通过;该 PR 也已明确改为 Refs #22,只作为统一金融数据网关的增量实现,不提前声称完成真实双 Provider E2E。按当前 scope 可以合入。
|
代码与测试材料已通过 review;当前唯一阻塞是与最新 main 的 Git 冲突,GitHub merge 返回 |
变更概述
UNSUPPORTED_CAPABILITY结构化错误行为。验证结果
bun run typecheck:全部通过。bun test packages/shared/src/providers:110 项通过。bun run test:provider-e2e:guard:2 项通过。git diff --check:通过。共享包全量测试中有 1 个既有 Windows 路径断言失败:
PiRuntimeAdapter > isolates sessions by switching the runtime session file。该失败与本次 Provider/Massive 改动无关,表现为测试期望
/tmp/pi/...,Windows 实际返回\\tmp\\pi\\...。Refs #22