feat(core): reconcile conflicting financial facts - #78
Open
13pgpg wants to merge 2 commits into
Open
Conversation
Author
|
Validation update: installed Bun 1.4.2. Reconciliation tests pass (3/3), and workspace typecheck passes for core, i18n, and shared. |
helsome
requested changes
Sep 11, 2026
helsome
left a comment
Owner
There was a problem hiding this comment.
当前先卡测试证据,不扩大实现 scope。金融事实 reconciliation 属数据正确性路径,PR 不能以“环境没有 Bun”作为正式验收结果。请补一份可复现测试报告:Bun 版本 + OS、实际执行命令、pass/fail 数量;至少覆盖 provider 冲突、语义不可比、容差内一致、显式 provider priority 且不做平均这几条核心行为。若本机没有 Bun,可用项目固定的 Bun 1.4.2 或 npx --yes bun 执行。无需接真实 provider,这层确定性 reconciliation 用 focused test 即可证明。
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.
Summary
Refs #26
Validation