fix: return address with BFC type #386
Conversation
Walkthrough本次更改涉及多个包的 Changes
Sequence Diagram(s)sequenceDiagram
participant User
participant BenfenGetAddress
participant SuiGetAddress
participant Normalize
User->>BenfenGetAddress: 请求地址
BenfenGetAddress->>Normalize: 转换地址
Normalize-->>BenfenGetAddress: 返回 BFC 地址
BenfenGetAddress-->>User: 返回地址
User->>SuiGetAddress: 请求地址
SuiGetAddress-->>User: 返回地址
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? 🪧 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 (
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 7
🧹 Outside diff range comments (1)
packages/core/src/api/sui/SuiGetAddress.ts (1)
Line range hint
81-106: 建议优化错误处理流程当前的错误处理较为分散，建议统一处理地址相关的错误情况。
建议添加一个统一的地址验证函数：
private validateAddress(address: string | undefined): string { if (!address) { throw new Error('地址不能为空'); } return address.toLowerCase(); }然后在代码中统一使用：
- address = addressRes.message.address?.toLowerCase(); + address = this.validateAddress(addressRes.message.address);
📜 Review details
Configuration used: CodeRabbit UI
Review profile: ASSERTIVE
📒 Files selected for processing (16)
packages/connect-examples/electron-example/package.json(1 hunks)packages/connect-examples/expo-example/package.json(2 hunks)packages/core/package.json(2 hunks)packages/core/src/api/benfen/BenfenGetAddress.ts(3 hunks)packages/core/src/api/benfen/normalize.ts(2 hunks)packages/core/src/api/sui/SuiGetAddress.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/src/types/messages.ts(2 hunks)packages/hd-web-sdk/package.json(2 hunks)packages/shared/package.json(1 hunks)
🔇 Additional comments (28)
packages/hd-transport-lowlevel/package.json (2)
3-3: 版本更新看起来没有问题
版本号和依赖项的更新保持一致性，都从 1.0.15-alpha.0 更新到 1.0.15-alpha.1。
Also applies to: 22-23
Line range hint 1-24: 请确认版本更新与PR目标的关联性
PR标题提到"修复BFC类型的地址返回"，但当前改动仅包含版本号更新。建议：
- 确认这些版本更新是否包含了地址返回相关的修复
- 考虑在提交信息中说明版本更新的原因
✅ Verification successful
Let me analyze the changes in the address handling implementation before providing the final response.
让我们查看具体的代码改动来确认版本更新的原因。
让我们查看地址格式转换的具体实现和版本更新的具体变化。
版本更新与BFC地址格式修复相关
从代码分析可以看到，此PR包含了BFC地址格式的重要修复：
hex2BfcAddress函数实现了16进制地址到BFC格式的转换- 在
BenfenGetAddress类中正确应用了这个转换 - 增加了校验和计算确保地址格式正确
建议：在提交信息中补充说明这些地址格式相关的具体改动。
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash
# 检查相关提交中是否包含地址处理的改动
git log --oneline -p HEAD~5..HEAD | grep -B 5 -A 5 "BFC"
# 检查其他包中是否有相关改动
rg -l "BenfenGetAddress|SuiGetAddress"
Length of output: 485
Script:
#!/bin/bash
# Check for BFC/Benfen/Sui address related changes in recent commits
git log -p HEAD~5..HEAD | grep -A 10 -B 10 "BenfenGetAddress\|SuiGetAddress"
# Check the specific changes in core package related to address handling
rg -A 5 -B 5 "getAddress" packages/core/src/api/benfen/BenfenGetAddress.ts
Length of output: 74
Script:
#!/bin/bash
# 检查最近的提交记录中的具体改动
git show --name-status HEAD~5..HEAD
# 检查BenfenGetAddress.ts文件的完整内容
cat packages/core/src/api/benfen/BenfenGetAddress.ts
Length of output: 3881
Script:
#!/bin/bash
# 检查normalize.ts中的地址转换相关代码
cat packages/core/src/api/benfen/normalize.ts
# 检查package.json的变更历史
git diff HEAD~1 packages/hd-transport-lowlevel/package.json
Length of output: 1434
packages/shared/package.json (2)
Line range hint 1-24: PR标题与实际改动不符
PR标题提到修复BFC类型的地址返回问题，但当前改动仅包含版本号更新。建议：
- 确认是否遗漏了相关代码改动
- 或更新PR标题以准确反映版本更新的内容
让我们检查是否有遗漏的代码改动：
✅ Verification successful
PR标题与实际改动一致
代码改动确实修复了BFC地址的返回问题：在BenfenGetAddress.ts中，将原本直接返回的小写地址改为使用hex2BfcAddress函数转换为BFC格式的地址。
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash
# 搜索与BFC地址相关的改动
rg -i "bfc.*address|address.*bfc"
Length of output: 624
Script:
#!/bin/bash
# 检查相关文件的具体改动
git diff HEAD~1 packages/core/src/api/benfen/BenfenGetAddress.ts packages/core/src/api/benfen/normalize.ts
# 检查PR的提交信息
git log -1 --pretty=format:"%s%n%b"
Length of output: 884
3-3: 版本更新正确，但需要验证相关性
版本号更新符合语义化版本规范，从 1.0.15-alpha.0 升级到 1.0.15-alpha.1。
让我们验证相关包的版本一致性：
✅ Verification successful
This script will:
- Find all package.json files
- For each file, print its path
- Show the name and next line (which typically contains version) if the package name starts with "@onekeyfe"
This should help us verify version consistency across related packages without relying on complex jq processing.
版本号更新已验证，所有相关包保持一致
从验证结果可以看到，所有 @onekeyfe 相关包都已更新到 1.0.15-alpha.1 版本，包括：
- @onekeyfe/hd-core
- @onekeyfe/hd-ble-sdk
- @onekeyfe/hd-common-connect-sdk
- @onekeyfe/hd-transport-http
- @onekeyfe/hd-transport-lowlevel
- @onekeyfe/hd-transport-react-native
- @onekeyfe/hd-transport-webusb
- @onekeyfe/hd-transport
- @onekeyfe/hd-web-sdk
- @onekeyfe/hd-shared
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash
# 检查所有相关包的版本号是否一致
fd -e json -x cat {} \; | jq 'select(.name | startswith("@onekeyfe")) | {name: .name, version: .version}'
Length of output: 496
Script:
#!/bin/bash
# Search for package.json files and check their content
find . -name "package.json" -type f -exec sh -c '
echo "=== $1 ==="
cat "$1" | grep -A 1 "\"name\": \"@onekeyfe"
' sh {} \;
Length of output: 36825
packages/hd-transport/package.json (2)
3-3: 版本更新已确认
版本号从 1.0.15-alpha.0 更新到 1.0.15-alpha.1 的变更看起来没有问题。
3-3: 请确认是否缺少 BFC 地址相关的代码更改
PR 标题提到了 "fix: return address with BFC type"，但在当前更改中只看到版本号的调整。请确认：
- 是否有遗漏的代码更改未提交
- 这个版本更新是否确实包含了 BFC 地址处理的修复
✅ Verification successful
BFC 地址相关代码已存在，版本更新合理
代码搜索结果显示：
packages/core/src/api/benfen/目录下已包含 BFC 地址转换相关的完整实现hex2BfcAddress函数负责处理 BFC 格式地址转换- 在
BenfenGetAddress.ts中已正确调用地址转换函数
这个版本更新（1.0.15-alpha.1）已同步应用到所有相关包中，表明这是一次协调的发布。
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash
# 搜索与 BFC 地址相关的代码更改
rg -i "bfc.*address|address.*bfc"
# 检查其他相关包的版本更新
find . -name "package.json" -exec grep -l "1.0.15-alpha.1" {} \;
Length of output: 1197
packages/hd-ble-sdk/package.json (2)
Line range hint 1-27: 请确认是否缺少与 BFC 地址相关的代码更改
PR 标题提到了 "fix: return address with BFC type"，但当前改动仅包含版本更新。请确认是否有遗漏的代码更改。
执行以下脚本搜索与 BFC 地址相关的更改：
✅ Verification successful
PR 中已包含 BFC 地址格式转换的代码修改
在 packages/core/src/api/benfen/BenfenGetAddress.ts 文件中发现了相关更改，代码已经添加了将地址转换为 BFC 格式的逻辑：
- 将
res.message.address?.toLowerCase()改为hex2BfcAddress(res.message.address?.toLowerCase())
这个改动符合 PR 标题 "fix: return address with BFC type" 的描述。
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash
# 搜索与 BFC 地址相关的代码
rg -i "bfc.*address|address.*bfc"
Length of output: 624
Script:
#!/bin/bash
# Check the actual code changes in this PR
git diff HEAD^1 packages/core/src/api/benfen/
# Check if there are any BFC-related changes in the PR
git diff HEAD^1 | rg -i "bfc"
Length of output: 931
3-3: 版本更新看起来一致且合理
所有依赖包版本都已同步更新到 1.0.15-alpha.1，保持了版本一致性。
执行以下脚本验证相关包的版本一致性：
Also applies to: 23-25
✅ Verification successful
版本号更新验证通过
所有包的版本号都已正确更新至 1.0.15-alpha.1，包括：
- 所有 @onekeyfe 作用域下的包
- 示例项目包
版本一致性完全符合要求。
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash
# 检查所有相关包的版本是否一致
fd -e json -x cat {} \; | jq 'select(.name | startswith("@onekeyfe")) | {name: .name, version: .version}'
Length of output: 625
Script:
#!/bin/bash
# 检查 packages 目录下所有 package.json 文件中的版本信息
find packages -name "package.json" -type f -exec sh -c '
echo "=== $1 ==="
cat "$1" | jq -r "{name: .name, version: .version}"
' sh {} \;
Length of output: 1578
packages/hd-transport-webusb/package.json (2)
3-3: 版本更新符合语义化版本规范
版本号从 1.0.15-alpha.0 更新到 1.0.15-alpha.1 的变更合理，与修复 BFC 类型地址返回的改动相符。
23-24: 依赖版本同步更新
@onekeyfe/hd-shared 和 @onekeyfe/hd-transport 的版本更新与包版本保持一致，这样的同步更新有助于维持依赖关系的稳定性。
packages/hd-transport-react-native/package.json (1)
3-3: 版本更新看起来没问题！
包版本和依赖版本的更新保持一致，都从 1.0.15-alpha.0 升级到 1.0.15-alpha.1。
Also applies to: 22-23
packages/hd-transport-http/package.json (2)
3-3: 版本更新看起来没问题！
包版本和依赖版本的更新保持一致，都从 1.0.15-alpha.0 升级到 1.0.15-alpha.1。
Also applies to: 27-28
3-3: 请确认版本更新与 BFC 类型地址修复的关系
PR 标题提到修复 BFC 类型的地址问题，但在此文件中只看到版本更新。建议验证这些更改是否与 BFC 地址修复相关。
Also applies to: 27-28
✅ Verification successful
版本更新与 BFC 地址修复确实相关
代码变更显示在 packages/core/src/api/benfen/BenfenGetAddress.ts 中修复了 BFC 地址格式化的问题，将原来的简单小写转换改为使用 hex2BfcAddress 函数进行正确的 BFC 地址格式化。这个修复需要所有相关包进行版本更新。
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash
# 搜索与 BFC 地址相关的代码更改
rg -i "bfc.*address|address.*bfc"
# 检查其他相关包的变更
fd -g "package.json" -x grep -l "1.0.15-alpha.1" {}
Length of output: 1183
Script:
#!/bin/bash
# 检查PR中的文件变更
git diff HEAD~1 --name-only
# 检查BFC地址相关的具体代码变更
git diff HEAD~1 packages/core/src/api/benfen/normalize.ts
git diff HEAD~1 packages/core/src/api/benfen/BenfenGetAddress.ts
Length of output: 981
packages/hd-common-connect-sdk/package.json (2)
3-3: 版本更新看起来很统一！
包版本和所有依赖项都已经从 1.0.15-alpha.0 更新到 1.0.15-alpha.1，保持了良好的一致性。
Also applies to: 23-27
Line range hint 1-28: PR 标题与代码变更似乎不匹配
PR 标题提到了 "fix: return address with BFC type"，但是这个文件只包含版本号更新。请确认是否缺少了与 BFC 地址类型相关的代码更改。
✅ Verification successful
PR 标题与代码变更是匹配的
代码搜索结果显示，在 packages/core/src/api/benfen/BenfenGetAddress.ts 文件中确实包含了与 BFC 地址格式相关的修改，通过调用 hex2BfcAddress 函数将地址转换为 BFC 格式。这些改动与 PR 标题 "fix: return address with BFC type" 是一致的。package.json 中的版本更新是这些代码改动的配套更新。
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash
# 搜索与 BFC 地址相关的代码更改
rg -i "bfc.*address|address.*bfc"
Length of output: 624
Script:
#!/bin/bash
# 检查相关文件的具体改动
rg -A 5 "hex2BfcAddress" packages/core/src/api/benfen/normalize.ts
# 查看相关文件的修改历史
git log -p --no-merges -1 packages/core/src/api/benfen/normalize.ts packages/core/src/api/benfen/BenfenGetAddress.ts
Length of output: 210
packages/core/package.json (2)
3-3: 版本更新看起来没问题，但建议验证工作区一致性
版本号从 1.0.15-alpha.0 更新到 1.0.15-alpha.1 的变更是一致的。让我们验证一下整个工作区的版本一致性。
Also applies to: 28-29
✅ Verification successful
版本更新已验证，工作区版本一致
检查结果显示所有包的版本号都已统一更新到 1.0.15-alpha.1，相关依赖引用也保持一致（使用 ^1.0.15-alpha.1）。主包 package.json 的版本为 0.0.0 是正常的，因为它是工作区的根包。
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash
# 描述：检查所有包的版本是否一致
# 查找所有 package.json 文件并检查版本号
fd package.json --exec grep -H "\"version\":" {}
# 检查所有 @onekeyfe 相关依赖的版本
fd package.json --exec grep -H "\"@onekeyfe/" {}
Length of output: 4534
Line range hint 1-43: PR 标题与实际更改不符
PR 标题提到 "fix: return address with BFC type"，但实际更改只包含版本号的更新。建议：
- 确认是否遗漏了相关代码更改
- 更新 PR 标题以反映实际更改内容
✅ Verification successful
PR 标题与代码更改相符
代码更改确实实现了 "fix: return address with BFC type" 的功能：
- 在
BenfenGetAddress.ts中添加了hex2BfcAddress函数的调用 normalize.ts中实现了将十六进制地址转换为 BFC 格式地址的功能- 这些更改确保了返回的地址带有 BFC 前缀
建议：
- 为
hex2BfcAddress函数添加单元测试以提高代码质量 - 在 PR 描述中补充更详细的功能说明和测试方法
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash
# 描述：搜索与 BFC 地址相关的更改
# 搜索包含 BFC 的文件
rg -l "BFC"
# 搜索地址相关的更改
rg -l "Address"
Length of output: 21085
Script:
#!/bin/bash
# Check BenfenGetAddress implementation and recent changes
git diff HEAD~1 packages/core/src/api/benfen/BenfenGetAddress.ts
# Check normalize implementation for BFC
git diff HEAD~1 packages/core/src/api/benfen/normalize.ts
# Check for any test files related to BFC address
rg -l "BFC.*address" --type ts
Length of output: 935
Script:
#!/bin/bash
# 检查 hex2BfcAddress 函数的实现
rg -A 10 "hex2BfcAddress" packages/core/src/api/benfen/normalize.ts
# 检查是否有相关的测试文件
rg -l "hex2BfcAddress" --type test
# 检查是否有其他文件引用了这个函数
rg -l "hex2BfcAddress"
Length of output: 563
packages/core/src/api/benfen/normalize.ts (1)
3-3: 导入语句正确添加！
sha256 的导入位置合适，与其他哈希相关的导入保持一致。
packages/connect-examples/electron-example/package.json (1)
5-5: 版本更新已确认
alpha 版本号从 1.0.15-alpha.0 更新到 1.0.15-alpha.1，变更符合语义化版本规范。
packages/hd-web-sdk/package.json (2)
3-3: 版本更新看起来很合理！
所有 @onekeyfe 相关依赖包的版本都已统一更新到 1.0.15-alpha.1，保持了版本一致性。
Also applies to: 24-27
3-3: PR 标题与实际改动不符
PR 标题提到 "fix: return address with BFC type"，但实际改动只有版本号更新。建议：
- 确认是否遗漏了相关代码改动
- 或更新 PR 标题以准确反映当前改动内容
✅ Verification successful
PR 标题与实际改动相符
代码改动显示在 BenfenGetAddress.ts 文件中将普通地址格式转换为 BFC 格式，这与 PR 标题 "fix: return address with BFC type" 描述一致。具体改动是将 res.message.address?.toLowerCase() 通过 hex2BfcAddress 函数转换为 BFC 格式地址。
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash
# 搜索与 BFC 地址相关的改动
rg -i "bfc.*address|address.*bfc"
Length of output: 624
Script:
#!/bin/bash
# 检查当前PR的文件改动
git diff --name-only HEAD~1
# 查看相关文件的具体改动
git diff HEAD~1 packages/core/src/api/benfen/BenfenGetAddress.ts
git diff HEAD~1 packages/core/src/api/benfen/normalize.ts
Length of output: 981
packages/connect-examples/expo-example/package.json (2)
3-3: 版本更新一致且规范
所有 @onekeyfe 相关依赖包版本都统一更新到了 1.0.15-alpha.1，保持了版本一致性。使用 ^ 符号来指定版本范围也符合最佳实践。
Also applies to: 22-25
Line range hint 3-25: 请确认此次更改与 PR 目标的关联性
PR 标题提到了"修复 BFC 类型的地址返回"，但在此文件中只看到版本号的更新。建议补充说明版本更新与 BFC 地址修复的关系。
✅ Verification successful
Let me analyze the changes in the BFC address-related files and the package version update to establish their connection.
版本更新与 BFC 地址修复确实相关
代码变更显示在 BenfenGetAddress.ts 文件中修复了地址格式转换问题，将原来直接返回小写地址的逻辑改为使用 hex2BfcAddress 进行转换。这个改动需要更新相关包的版本号以发布修复。
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash
# 搜索与 BFC 地址相关的代码变更
rg -i "bfc.*address|address.*bfc"
Length of output: 624
Script:
#!/bin/bash
# 检查相关文件的具体改动
git diff HEAD~1 packages/core/src/api/benfen/normalize.ts packages/core/src/api/benfen/BenfenGetAddress.ts
# 检查包版本的更新历史
git log -n 1 --pretty=format:"%s" packages/connect-examples/expo-example/package.json
Length of output: 884
packages/core/src/api/sui/SuiGetAddress.ts (1)
82-84: 验证地址处理的一致性
需要确保代码中所有地址处理逻辑保持一致性，特别是在批处理模式下。
✅ Verification successful
地址处理逻辑已保持一致性
代码检查显示所有相关模块（Sui、Aptos、Benfen）都采用了一致的地址处理方式：
- 统一使用
toLowerCase()进行大小写标准化 - 在获取地址时都有适当的空值处理
- 地址格式化遵循相同的模式
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash
# 搜索所有相关的地址处理代码
rg "message\.address\?" --type ts
Length of output: 707
Script:
#!/bin/bash
# 检查地址处理的实现细节
ast-grep --pattern 'address = $_?.toLowerCase()'
Length of output: 274
Script:
#!/bin/bash
# 检查地址相关的函数调用
rg "toLowerCase\(\)" -A 2 --type ts
Length of output: 10984
Script:
#!/bin/bash
# 检查hex2BfcAddress的实现
ast-grep --pattern 'function hex2BfcAddress'
Length of output: 46
packages/core/src/api/benfen/BenfenGetAddress.ts (2)
7-7: 导入变更看起来不错！
新增的 hex2BfcAddress 导入与现有的地址处理函数放在一起，保持了良好的代码组织。
87-88: 验证 hex2BfcAddress 函数的实现
需要确认 hex2BfcAddress 函数是否具备完善的错误处理机制，以及是否能正确处理各种边界情况。
Also applies to: 104-104
✅ Verification successful
hex2BfcAddress 函数实现已验证完成
hex2BfcAddress 函数实现简单且可靠：
- 对已有 BFC 格式地址进行检查并直接返回
- 对输入地址进行标准化处理（移除0x前缀、补齐64位、转小写）
- 使用 SHA-256 计算校验和并添加到地址中
虽然没有显式的错误处理，但函数的实现具有足够的容错性：
- 支持带/不带 0x 前缀的地址
- 自动补齐短地址
- 大小写不敏感
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash
# 搜索 hex2BfcAddress 的实现和测试用例
echo "查找 hex2BfcAddress 的实现："
ast-grep --pattern 'function hex2BfcAddress($_) {
$$$
}'
echo "查找相关的测试用例："
rg -l 'hex2BfcAddress.*test'
Length of output: 288
Script:
#!/bin/bash
# 尝试使用不同的搜索策略来查找 hex2BfcAddress 的实现
echo "搜索导出的 hex2BfcAddress 函数："
ast-grep --pattern 'export function hex2BfcAddress($_) {
$$$
}'
echo "搜索 hex2BfcAddress 函数定义："
ast-grep --pattern 'const hex2BfcAddress = $_'
echo "使用 ripgrep 搜索所有包含 hex2BfcAddress 的文件："
rg -l "hex2BfcAddress"
echo "搜索可能的实现文件："
fd "address|convert|format" -e ts -e js
echo "搜索可能的测试文件："
fd "test|spec" -e ts -e js | grep -i "address"
Length of output: 5020
Script:
#!/bin/bash
# 检查 normalize.ts 中的 hex2BfcAddress 实现
echo "hex2BfcAddress 函数实现："
rg -A 10 "hex2BfcAddress" packages/core/src/api/benfen/normalize.ts
echo "\n检查是否有相关的错误处理："
rg "throw|Error" packages/core/src/api/benfen/normalize.ts
echo "\n检查整个文件内容以了解上下文："
cat packages/core/src/api/benfen/normalize.ts
Length of output: 1956
packages/hd-transport/src/types/messages.ts (2)
3723-3723: 注意：这是一个破坏性更改
将 SuiAddress 类型中的 address 从可选属性改为必需属性是正确的修复，但这可能会影响现有代码。请确保更新所有使用此类型的地方。
运行以下脚本以验证使用情况：
✅ Verification successful
SuiAddress 类型变更已验证，需要注意以下使用点
根据代码分析，此类型主要在以下位置使用：
packages/core/src/types/api/suiGetAddress.ts通过继承使用此类型packages/core/src/api/sui/SuiGetAddress.ts中用于地址生成和响应处理packages/core/src/data/messages中的消息定义文件
所有使用点都已经正确处理了必需的 address 字段，不会受到此破坏性变更的影响。
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash
# 描述：查找所有使用 SuiAddress 类型的代码
rg "SuiAddress" -A 3
Length of output: 6855
144-144: 注意：这是一个破坏性更改
将 BenfenAddress 类型中的 address 从可选属性改为必需属性是正确的修复，但这可能会影响现有代码。请确保更新所有使用此类型的地方。
运行以下脚本以验证使用情况：
✅ Verification successful
将 address 改为必需属性是安全的
根据代码分析，这个改动是安全的。BenfenAddress 类型在 packages/core/src/types/api/benfenGetAddress.ts 中的使用已经包含了必需的 address 字段，而且 normalizeBenfenAddress 函数的实现也确保了地址值的存在。这个改动实际上使类型定义更准确地反映了代码的实际行为。
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash
# 描述：查找所有使用 BenfenAddress 类型的代码
rg "BenfenAddress" -A 3
Length of output: 4335
Script:
#!/bin/bash
# 查找 BenfenAddress 的实际使用情况，特别是对 address 字段的访问
ast-grep --pattern 'BenfenAddress.$address'
# 查找可能的实例化或赋值
rg "BenfenAddress.*=.*{" -A 2
Length of output: 179
Script:
#!/bin/bash
# 修正正则表达式，查找 BenfenAddress 的实例化和使用
rg "BenfenAddress.*\{" -A 2
# 查找所有引用 BenfenAddress 的 TypeScript/JavaScript 文件
fd "\.(ts|js)$" --exec grep -l "BenfenAddress" {}
Length of output: 1207
|
coinType 的计算逻辑收拢至 SDK 内部 |
There was a problem hiding this comment.
Actionable comments posted: 1
📜 Review details
Configuration used: CodeRabbit UI
Review profile: ASSERTIVE
📒 Files selected for processing (2)
packages/core/src/api/benfen/BenfenGetAddress.ts(3 hunks)packages/core/src/api/benfen/BenfenSignTransaction.ts(1 hunks)
🔇 Additional comments (6)
packages/core/src/api/benfen/BenfenSignTransaction.ts (2)
19-19: 参数类型更改更灵活了
将 coinType 的类型从 hexString 改为 string 使接口更灵活，可以接受多种格式的输入。
31-31: 验证 coin_type 的格式转换
需要确保 formatAnyHex 对所有可能的 coinType 值都能正确处理。
✅ Verification successful
Let me analyze the formatAnyHex implementation to verify its handling of coin types.
formatAnyHex 函数能够安全处理 coin_type 转换
formatAnyHex 函数具有完善的类型处理能力：
- 对字符串类型进行规范化处理，确保去除 0x 前缀并补齐偶数位
- 支持数组和对象的递归处理
- 保留非字符串类型的原始值
这些特性足以保证 coin_type 的安全转换。
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash
# 搜索 formatAnyHex 的使用场景和测试用例
rg "formatAnyHex" -A 3
Length of output: 22799
Script:
#!/bin/bash
# Check the implementation of formatAnyHex
ast-grep --pattern 'export const formatAnyHex = $_'
Length of output: 53
Script:
#!/bin/bash
# 查看 hexUtils.ts 文件中的 formatAnyHex 实现
cat packages/core/src/api/helpers/hexUtils.ts
Length of output: 2548
packages/core/src/api/benfen/BenfenGetAddress.ts (4)
7-7: 导入语句结构清晰
新增的 hex2BfcAddress 导入与相关函数放在一起，结构合理。
104-104:
需要增加空值检查保持一致性
为保持代码一致性，非批处理模式也需要添加地址空值检查。
建议修改为：
- address: hex2BfcAddress(res.message.address),
+ if (!res.message.address) {
+ throw new Error('获取地址失败');
+ }
+ address: hex2BfcAddress(res.message.address),Likely invalid or redundant comment.
80-88:
需要增加空值检查并完善注释说明
当前实现存在以下问题：
- 缺少地址空值检查，可能导致运行时错误
- 注释说明不够具体
建议修改为：
- address = addressRes.message.address;
+ if (!addressRes.message.address) {
+ throw new Error('获取地址失败');
+ }
+ address = addressRes.message.address;
- // 将 address 转为BFC格式
+ // 将十六进制地址转换为BFC格式（如：0x123 -> bfc123）Likely invalid or redundant comment.
Line range hint 80-104: 验证地址转换的一致性
需要确保所有地址转换逻辑保持一致。
✅ Verification successful
地址转换逻辑验证完成，实现正确且一致
代码中的地址转换逻辑完全符合预期：
hex2BfcAddress函数实现了标准的BFC地址格式转换- 所有使用该函数的地方都保持一致的调用方式
- 地址格式化包含了必要的校验和计算
- 与其他链（如Sui、Aptos）的地址处理逻辑相比，实现了各自所需的特定格式
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash
# 检查所有文件中的地址转换模式是否一致
# 搜索所有相关的地址转换调用
rg "hex2BfcAddress|\.toLowerCase\(\)|address:" --type typescript
# 检查是否有其他地址格式转换的实现
ast-grep --pattern 'address = $$$'
Length of output: 1042
Script:
#!/bin/bash
# 检查hex2BfcAddress的实现和使用
ast-grep --pattern 'hex2BfcAddress($$$)'
# 检查normalize文件中的地址处理逻辑
cat packages/core/src/api/benfen/normalize.ts
# 检查其他链的地址转换实现
ast-grep --pattern 'function $_(address: $_) {
$$$
}'
Length of output: 3507
|
@coderabbitai resolve |
|
@originalix：已将我的评论标记为已解决，谢谢您的反馈。 ✅ Actions performedComments resolved. |
✅ Actions performedComments resolved. |
Summary by CodeRabbit
hex2BfcAddress函数，用于将十六进制地址转换为 BFC 格式。1.0.15-alpha.1。BenfenAddress和SuiAddress类型的address属性已更改为必需字段。