feat: 完善统一金融数据网关 - #79
Open
yyq66 wants to merge 3 commits into
Open
Conversation
统一 provider-neutral 请求类型,保留路由 provenance,并覆盖行情、历史和基本面能力。
helsome
previously requested changes
Sep 11, 2026
helsome
left a comment
Owner
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
approved these changes
Sep 11, 2026
helsome
left a comment
Owner
There was a problem hiding this comment.
测试材料已补齐:Bun 1.4.2 / Windows,provider + capability 102 tests 通过、全仓 typecheck 通过;该 PR 也已明确改为 Refs #22,只作为统一金融数据网关的增量实现,不提前声称完成真实双 Provider E2E。按当前 scope 可以合入。
Owner
|
代码与测试材料已通过 review;当前唯一阻塞是与最新 main 的 Git 冲突,GitHub merge 返回 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
变更概述
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