Repository navigation
Add a bounded, nullable-aware wallet balance consumer - #9
Merged
Merged
Conversation
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.
钱包资产接入需要正确处理 Data API 的响应封装、可空元数据和多页结果。本变更新增可选的只读 TypeScript 消费模块：保留余额整数字符串及未知 decimals/symbol，逐页收集并保留原 meta；后页错误、重复 token、循环游标、超过页数或已知快照时间变化时整体拒绝。任一页快照时间未知，汇总 refreshedAt 保持 null。原 npm start 开户/RPC 流程不变，英文与中文 README 增加服务端调用示例。
最终源码 b8991f1 经独立源码复审 APPROVE。Leader Node 24.21.0 实跑 npm test：85通过/0失败/0跳过，npm run typecheck 通过；本 PR 原生 Node24 CI同为85通过/0跳过、类型检查通过。16项新增离线测试覆盖封装、精度、null、游标、限额和失败传播。初版混合未知时间汇总问题经独审阻断，同一反例已实跑修复前失败→修复后通过，并保留原结果。未调用真实 API、创建账号、交易或验证任何链的实际余额数据覆盖。
验收项: OFFLINE_WALLET_BALANCE_CONSUMER