fix: ton token transfer #379
Conversation
Walkthrough本次变更主要涉及多个软件包的版本更新，从“1.0.12-alpha.2”更新为“1.0.12”。多个依赖项的版本也进行了相应更新，以确保所有相关模块使用相同的稳定版本。此外，部分文件中添加了新方法和字段，更新了参数验证和类型签名，以支持新的功能和数据结构。 Changes
🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
|
👍 Dependency issues cleared. Learn more about Socket for GitHub ↗︎ This PR previously contained dependency changes with security issues that have been resolved, removed, or ignored. |
ByteZhang1024
requested review from
wabicai and
originalix
and removed request for
wabicai
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 8
🧹 Outside diff range comments (4)
packages/hd-transport/src/types/messages.ts (3)
Line range hint
3889-3892: 修正拼写错误并重新考虑字段可选性发现以下问题：
- "signning_message"拼写错误，应为"signing_message"。
- 所有字段都是可选的，这对签名消息结果来说不太常见。
建议：
- 更正拼写错误。
- 至少将
signature字段设为必填。应用此更改：
export type TonSignedMessage = { - signature?: string; - signning_message?: string; + signature: string; + signing_message?: string; };
Line range hint
3894-3905: 结构合理，可进一步提高一致性
TonSignProof类型定义结构清晰，字段选择恰当。建议：
- 与
TonSignMessage保持一致，考虑添加chain_id字段。- 对可选字段添加简短注释，说明其用途。
Line range hint
3907-3909: 签名字段应为必填对于签名证明结果，当前结构可能不够严谨。
建议：
- 将
signature字段改为必填，确保始终有签名结果。应用此更改：
export type TonSignedProof = { - signature?: string; + signature: string; };packages/core/src/data/messages/messages.json (1)
Line range hint
10635-10638:signning_message字段有拼写错误新增的
signning_message字段名拼错了。应该改为signing_message。建议修改如下：
- "signning_message": { + "signing_message": { "type": "bytes", "id": 2 }
📜 Review details
Configuration used: CodeRabbit UI
Review profile: ASSERTIVE
📒 Files selected for processing (21)
- packages/connect-examples/electron-example/package.json (1 hunks)
- packages/connect-examples/expo-example/package.json (2 hunks)
- packages/connect-examples/expo-example/src/data/ton.ts (1 hunks)
- packages/core/package.json (2 hunks)
- packages/core/src/api/ton/TonSignMessage.ts (3 hunks)
- packages/core/src/api/ton/TonSignProof.ts (1 hunks)
- packages/core/src/data/messages/messages.json (1 hunks)
- packages/core/src/types/api/tonSignMessage.ts (1 hunks)
- packages/core/src/types/api/tonSignProof.ts (1 hunks)
- packages/hd-ble-sdk/package.json (2 hunks)
- packages/hd-common-connect-sdk/package.json (2 hunks)
- packages/hd-transport-http/package.json (2 hunks)
- packages/hd-transport-lowlevel/package.json (2 hunks)
- packages/hd-transport-react-native/package.json (2 hunks)
- packages/hd-transport-webusb/package.json (2 hunks)
- packages/hd-transport/package.json (1 hunks)
- packages/hd-transport/scripts/protobuf-patches/index.js (1 hunks)
- packages/hd-transport/src/types/messages.ts (2 hunks)
- packages/hd-web-sdk/package.json (2 hunks)
- packages/shared/package.json (1 hunks)
- submodules/firmware (1 hunks)
🧰 Additional context used
🔇 Additional comments (27)
packages/core/src/types/api/tonSignProof.ts (2)
1-6: 导入语句更新得当！新增的
UintType导入提高了类型安全性。这个改动与后续的使用一致，值得称赞。
13-13: 类型更新有助于提高安全性把
expireAt的类型从number改为UintType是个好主意。这样可以更精确地表示无符号整数。请运行以下脚本，检查是否有代码需要更新：
✅ Verification successful
请运行以下脚本，检查是否有代码需要更新：
请运行以下脚本，进一步检查是否有代码需要更新：
类型更新已验证且无影响
通过执行脚本，未发现任何使用旧
TonSignProofParams类型expireAt: number的代码。这表明类型更新不会影响现有代码。🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # 描述：查找使用旧 TonSignProofParams 类型的代码 rg --type typescript 'TonSignProofParams.*expireAt.*number'Length of output: 96
Script:
#!/bin/bash # 描述：查找使用旧 TonSignProofParams 类型的代码 rg --glob '*.ts' 'TonSignProofParams.*expireAt.*number'Length of output: 57
Script:
#!/bin/bash # 描述：查找使用旧 TonSignProofParams 类型的代码，包括 .ts 和 .tsx 文件 rg --glob '*.{ts,tsx}' 'TonSignProofParams\s*:\s*number'Length of output: 58
packages/hd-transport-lowlevel/package.json (4)
3-3: 版本更新正确。从alpha版本升级到稳定版本，移除了预发布标识符。这符合语义化版本控制的最佳实践。
22-22: 依赖版本更新合理。@onekeyfe/hd-shared 的版本更新与包自身版本一致，保持了相关包之间的版本同步。这是个好做法。
23-23: 依赖版本更新合理。@onekeyfe/hd-transport 的版本更新与包自身版本一致，保持了相关包之间的版本同步。这是个好做法。
3-3: 整体更新看起来不错。这次更新将包版本从alpha升级到稳定版，并同步更新了相关依赖。这表明项目正在向更稳定的版本迈进。请确保在发布前进行充分测试，以验证新版本的稳定性。
Also applies to: 22-23
packages/shared/package.json (1)
3-3: 版本更新获批！版本号从 1.0.12-alpha.2 升至 1.0.12，表明包已达到稳定状态。好样的！
别忘了更新更新日志哦，让用户知道这个版本有啥新鲜玩意儿。
packages/hd-ble-sdk/package.json (2)
3-3: 版本更新获批！从alpha版本升级到稳定版本是个好举措。这表明软件包已经准备好进行更广泛的使用。
23-25: 依赖项版本同步更新，很好！所有依赖项都更新到了与主包相同的版本。这种一致性有助于避免版本冲突，提高系统稳定性。
packages/hd-transport/package.json (1)
3-3: 版本更新已确认版本号从 1.0.12-alpha.2 升级到 1.0.12，表明包已进入稳定版本。这是个好消息！
请运行以下脚本，检查相关包是否也需要更新：
✅ Verification successful
版本更新已验证
未发现其他包使用旧版本 "1.0.12-alpha.2"。当前版本更新仅影响
packages/hd-transport。🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # 描述：检查其他包的版本是否需要更新 # 测试：搜索其他包中的版本号 rg --type json '"version": "1\.0\.12-alpha\.2"' packages/Length of output: 59
packages/hd-transport-webusb/package.json (1)
3-3: 版本更新到稳定版，依赖同步更新。包版本从1.0.12-alpha.2升级到1.0.12，标志着从alpha版本转为稳定版。依赖项@onekeyfe/hd-shared和@onekeyfe/hd-transport也相应更新，保持一致性。这是个好变更。
Also applies to: 23-24
packages/hd-transport-react-native/package.json (2)
3-3: 版本更新到稳定版包版本从 1.0.12-alpha.2 更新到 1.0.12。这是个好变化，表明软件已经准备好发布稳定版本了。
22-23: 依赖项版本同步更新@onekeyfe/hd-shared 和 @onekeyfe/hd-transport 的版本都更新到了 ^1.0.12。这保持了包之间的一致性，是个好做法。
packages/hd-transport-http/package.json (3)
3-3: 版本更新到稳定版从alpha版升级到稳定版。好的进展！
27-27: 依赖项同步更新hd-shared依赖版本与主包保持一致。很好！
28-28: 依赖项同步更新，建议验证hd-transport依赖版本更新正确。建议检查其他相关包是否也需要更新。
运行以下脚本检查其他包的版本：
packages/hd-common-connect-sdk/package.json (1)
3-3: 版本更新看起来不错！包版本从 alpha 升级到稳定版。依赖项也同步更新了。干得好！
运行以下脚本，确保相关包都已更新：
Also applies to: 23-27
✅ Verification successful
版本更新已经确认！
所有相关包版本均已更新至稳定版 1.0.12，没有发现残留的 alpha 版本。
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # 描述：检查相关包是否都更新到了 1.0.12 版本 # 测试：搜索所有 package.json 文件中的版本号 fd -e json -x rg -i '"version":\s*"1\.0\.12' # 测试：检查是否有遗漏的 alpha 版本 fd -e json -x rg -i 'alpha'Length of output: 566
packages/core/src/types/api/tonSignMessage.ts (2)
1-6: 导入语句更新得当。新增的
UintType导入与接口中的使用一致，提高了类型安全性。
14-16: 类型更新提高了精确度。将
number改为UintType增强了类型安全性。建议检查使用这些属性的代码，确保兼容性。运行以下脚本验证更改影响：
Also applies to: 21-21, 28-28
✅ Verification successful
类型更新已验证，无发现问题。
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # 描述：查找使用 TonSignMessageParams 的地方 rg --type typescript "TonSignMessageParams" -C 5Length of output: 83
Script:
#!/bin/bash # 描述：查找使用 TonSignMessageParams 的地方 rg "TonSignMessageParams" --glob "*.ts" --glob "*.tsx" -C 5Length of output: 4290
packages/core/package.json (1)
3-3: 版本更新到稳定版包版本从alpha升级到稳定版。依赖项也相应更新。这是个好变化。
Also applies to: 28-29
packages/connect-examples/electron-example/package.json (1)
5-5: 版本更新已确认。版本从 alpha 升级到稳定版，这是个好消息。
请运行以下脚本，确保项目中的版本号一致：
packages/hd-web-sdk/package.json (1)
3-3: 版本更新已完成包版本和依赖项已更新到稳定版1.0.12。这是个好变化。
- 主包：1.0.12-alpha.2 → 1.0.12
- 依赖项：^1.0.12-alpha.2 → ^1.0.12
版本一致性提高了。干得好！
Also applies to: 24-27
packages/connect-examples/expo-example/package.json (1)
3-3: 版本更新看起来不错！版本从 alpha 升级到稳定版本，这是个好消息。
请运行以下脚本，确保所有相关包都使用相同的版本：
另外，建议全面测试项目，确保与新版本兼容。
Also applies to: 22-25
✅ Verification successful
版本更新一致，验证通过！
所有相关包已成功更新到
^1.0.12。🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # 描述：检查所有 @onekeyfe 包是否使用相同的版本 # 测试：搜索所有 package.json 文件中的 @onekeyfe 包版本 fd -e json -x grep -H '"@onekeyfe/' {} \;Length of output: 3355
packages/core/src/data/messages/messages.json (1)
10631-10634: 新字段jetton_amount_bytes添加得很好这个新字段可以更精确地处理 Jetton 金额。用字节类型存储有助于保持数据的完整性。
packages/core/src/api/ton/TonSignMessage.ts (2)
73-76: 新增对 classic1s 设备的版本支持在
getVersionRange方法中，增加了对classic1s设备的支持。这是很好的改进。
79-85: 添加了 getSupportJettonAmountBytesVersionRange 方法新增了
getSupportJettonAmountBytesVersionRange方法，返回支持jetton_amount_bytes的设备固件版本范围。packages/hd-transport/scripts/protobuf-patches/index.js (1)
204-209: 新增的TON字段类型定义正确添加的
TonSignMessage和TonSignProof的字段类型均为UINT_TYPE，符合预期。
ByteZhang1024
force-pushed
the
fix/tonTransfer
branch
from
d8dbe7e
to
ce60972
Compare
ByteZhang1024
force-pushed
the
fix/tonTransfer
branch
from
ce60972
to
01b81c9
Compare
|
Report too large to display inline |
There was a problem hiding this comment.
Actionable comments posted: 9
📜 Review details
Configuration used: CodeRabbit UI
Review profile: ASSERTIVE
📒 Files selected for processing (14)
- packages/connect-examples/electron-example/package.json (1 hunks)
- packages/connect-examples/expo-example/package.json (2 hunks)
- packages/connect-examples/expo-example/src/data/ton.ts (1 hunks)
- packages/core/package.json (2 hunks)
- packages/core/src/api/ton/TonSignMessage.ts (3 hunks)
- packages/hd-ble-sdk/package.json (2 hunks)
- packages/hd-common-connect-sdk/package.json (2 hunks)
- packages/hd-transport-http/package.json (2 hunks)
- packages/hd-transport-lowlevel/package.json (2 hunks)
- packages/hd-transport-react-native/package.json (2 hunks)
- packages/hd-transport-webusb/package.json (2 hunks)
- packages/hd-transport/package.json (1 hunks)
- packages/hd-web-sdk/package.json (2 hunks)
- packages/shared/package.json (1 hunks)
🧰 Additional context used
🔇 Additional comments (12)
packages/hd-transport-lowlevel/package.json (1)
3-3: 版本更新看起来不错！从alpha版本升级到稳定版本是个好主意。这表明软件包已经准备好更广泛的使用了。
Also applies to: 22-23
packages/hd-ble-sdk/package.json (2)
3-3: 版本更新到稳定版包版本从alpha升级到稳定版。好事！确保测试充分。
23-25: 依赖项版本同步更新核心依赖全部更新到1.0.12。保持一致性，减少潜在冲突。
packages/hd-transport-webusb/package.json (2)
3-3: 版本更新到稳定版从alpha版本升级到稳定版本。这是个重要的里程碑。
23-24: 依赖项版本同步更新依赖项版本与主版本一致更新。保持一致性很重要。
packages/hd-transport-react-native/package.json (1)
3-3: 版本更新看起来不错！包版本和依赖版本从alpha升级到稳定版，保持一致。好的做法。
运行以下脚本检查其他相关包是否也已更新：
Also applies to: 22-23
packages/hd-transport-http/package.json (3)
3-3: 版本升级到稳定版从alpha版本升级到稳定版。这是个重要变更，可能影响用户和依赖包。
27-27: 依赖包版本同步更新hd-shared依赖包版本更新，与主包版本一致。保持项目版本同步很重要。
28-28: 依赖包版本同步更新hd-transport依赖包版本更新，与主包版本一致。这确保了整个项目的版本一致性。
packages/hd-common-connect-sdk/package.json (1)
3-3: 版本更新看起来不错！从alpha版本升级到稳定版本是个好主意。依赖项也同步更新了，保持一致性。干得漂亮！
让我们确认一下其他相关包是否也已更新：
Also applies to: 23-27
packages/core/package.json (1)
3-3: 版本和依赖更新包版本从alpha升级到稳定版。依赖项也相应更新。这些变更保持了包之间的一致性。
Also applies to: 28-29
packages/connect-examples/expo-example/package.json (1)
3-3: 版本更新看起来不错！项目和依赖包从 alpha 版本升级到稳定版本，这是个好消息。
请运行以下脚本，确保所有相关包都已更新：
Also applies to: 22-25
✅ Verification successful
版本更新已验证！
所有相关的 @onekeyfe 包已成功更新到 1.0.12 版本。
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # 描述：检查所有 @onekeyfe 包是否已更新到 1.0.12 版本 # 测试：搜索所有 package.json 文件中的 @onekeyfe 包版本 fd -e json -x rg --json '"@onekeyfe/.*": "\^1\.0\.12.*"' {} | jq -r '.data.lines.text'Length of output: 1477
Summary by CodeRabbit
新特性
Bug 修复
文档