optimize: all network #372
Conversation
|
Caution Review failedThe pull request is closed. Walkthrough本次变更主要涉及多个 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. |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 12
📜 Review details
Configuration used: CodeRabbit UI
Review profile: ASSERTIVE
📒 Files selected for processing (15)
- 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/allnetwork.ts (1 hunks)
- packages/core/package.json (2 hunks)
- packages/core/src/api/allnetwork/AllNetworkGetAddress.ts (8 hunks)
- packages/core/src/types/api/allNetworkGetAddress.ts (2 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
🪛 Biome
packages/core/src/api/allnetwork/AllNetworkGetAddress.ts
[error] 380-380: Avoid the use of spread (
...) syntax on accumulators.Spread syntax should be avoided on accumulators (like those in
.reduce) because it causes a time complexity ofO(n^2).
Consider methods such as .splice or .push instead.(lint/performance/noAccumulatingSpread)
🔇 Additional comments (23)
packages/hd-transport-lowlevel/package.json (2)
3-3: 版本号更新正确。版本号从 1.0.12-alpha.0 升级到 1.0.12-alpha.1，符合语义化版本规范。这是个小版本更新。
22-23: 依赖版本同步更新。@onekeyfe/hd-shared 和 @onekeyfe/hd-transport 的版本都更新到了 ^1.0.12-alpha.1。这与包自身版本一致，保持了同步。
packages/shared/package.json (1)
3-3: 版本号更新正确。版本号从 1.0.12-alpha.0 升级到 1.0.12-alpha.1，符合预期。这是个小版本更新，可能用于测试或增量更新。
packages/hd-transport/package.json (1)
3-3: 版本号更新正确。版本号从 1.0.12-alpha.0 升级到 1.0.12-alpha.1，符合语义化版本规范。
别忘了更新 changelog，记录这次变更。
packages/hd-ble-sdk/package.json (1)
3-3: 版本更新看起来不错！版本号和依赖项的更新保持一致，很好。这种同步更新有助于保持包之间的兼容性。
让我们检查一下其他相关包是否也进行了类似的更新：
Also applies to: 23-25
✅ Verification successful
版本更新已验证！
所有相关包的版本号和依赖项均已同步更新，确保了包之间的兼容性。
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # 描述：检查其他包的 package.json 文件是否有类似的版本更新 # 测试：搜索其他包的 package.json 文件中的版本号 fd -e json -x grep -H '"version": "1.0.12-alpha.1"' {} # 测试：检查依赖项版本 fd -e json -x grep -H '"@onekeyfe/.*": "\^1.0.12-alpha.1"' {}Length of output: 3392
packages/hd-transport-webusb/package.json (2)
3-3: 版本号更新版本号从 1.0.12-alpha.0 升至 1.0.12-alpha.1。这是个小改动，符合语义化版本规范。
23-24: 依赖项版本更新@onekeyfe/hd-shared 和 @onekeyfe/hd-transport 的版本都更新了。这与包自身的版本更新一致，保持了依赖关系的同步。
packages/hd-transport-http/package.json (1)
3-3: 版本更新看起来不错！版本号和依赖项的更新保持一致，很好。
请运行以下脚本，检查其他相关包是否也需要类似更新：
Also applies to: 27-28
✅ Verification successful
好的，看起来初始脚本未找到任何使用旧版本的包。请运行以下脚本以进一步确认其他包是否仍在使用版本 "1.0.12-alpha.0"：
版本更新已验证，无其他包使用旧版本。
所有相关包的版本号已正确更新。
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # 描述：检查其他包的版本是否需要更新 # 测试：搜索其他包的 package.json 文件中的版本号 fd -e json -x rg -i '1\.0\.12-alpha\.0' {}Length of output: 46
Script:
#!/bin/bash # 描述：检查所有 package.json 文件中是否存在版本 "1.0.12-alpha.0" fd -e json --exec rg -i '"version":\s*"1\.0\.12-alpha\.0"' {}Length of output: 65
packages/hd-common-connect-sdk/package.json (1)
3-3: 版本更新看起来不错！包版本和依赖版本都从 1.0.12-alpha.0 升级到了 1.0.12-alpha.1。变更保持一致，很好。
运行以下脚本检查版本更新的一致性：
Also applies to: 23-27
packages/core/package.json (4)
3-3: 版本更新正确。版本号从 1.0.12-alpha.0 升级到 1.0.12-alpha.1，符合语义化版本规范。
28-28: 依赖更新正确。@onekeyfe/hd-shared 依赖版本与包版本同步更新，保持了一致性。
29-29: 依赖更新正确。@onekeyfe/hd-transport 依赖版本与包版本同步更新，保持了一致性。
3-3: 总体更改合理。版本号和依赖项的更新保持一致，符合预期。这些小改动有助于保持项目的同步性和稳定性。
Also applies to: 28-29
packages/connect-examples/electron-example/package.json (1)
5-5: 版本号更新正确。版本号从 1.0.12-alpha.0 升级到 1.0.12-alpha.1，符合语义化版本规范。
别忘了更新更改日志，记录这次版本升级的内容。
packages/core/src/types/api/allNetworkGetAddress.ts (2)
36-36: 类型导出变更合理。CommonResponseParams 的导出允许在其他模块中使用此类型。这是个好改动。
请运行以下脚本，检查 CommonResponseParams 的使用情况：
33-34: 新网络类型添加正确。'nostr' 网络类型的添加扩展了支持的网络范围。
请运行以下脚本，确保 'nostr' 网络类型在整个代码库中得到正确处理：
✅ Verification successful
请修正文件类型并重新验证 'nostr' 网络类型的使用情况。
rg命令中指定的文件类型typescript无法识别。请使用正确的文件类型或模式，例如-g "*.ts"。请运行以下脚本，以确保
'nostr'网络类型在整个代码库中得到正确处理：
'nostr' 网络类型已在代码库中正确处理。
'nostr' 网络类型在多个文件中有一致的使用，确认其添加是正确的。
packages/core/src/types/api/allNetworkGetAddress.tspackages/core/src/types/api/index.tspackages/connect-examples/expo-example/src/data/allnetwork.tspackages/connect-examples/expo-example/src/data/nostr.tspackages/core/src/api/nostr/helper/index.ts🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # 描述：验证 'nostr' 网络类型的使用情况 # 测试：搜索 'nostr' 的使用。预期：找到相关的处理逻辑。 rg --type typescript -i '\bnostr\b'Length of output: 72
Script:
#!/bin/bash # 描述：验证 'nostr' 网络类型的使用情况 # 测试：搜索 'nostr' 的使用。预期：找到相关的处理逻辑。 rg -g "*.ts" -i '\bnostr\b'Length of output: 1456
packages/connect-examples/expo-example/package.json (4)
3-3: 版本更新正确。版本号从 "1.0.12-alpha.0" 升级到 "1.0.12-alpha.1"，符合语义化版本规范。
22-22: 依赖版本更新合理。"@onekeyfe/hd-ble-sdk" 的版本更新与项目版本一致，保留了兼容性更新的灵活性。
23-23: 依赖版本更新一致。"@onekeyfe/hd-common-connect-sdk" 的版本更新与项目版本和其他依赖保持一致。
24-25: 依赖版本更新统一，建议验证兼容性。"@onekeyfe/hd-core" 和 "@onekeyfe/hd-web-sdk" 的版本更新与项目版本一致。这些更新看起来很好，但建议检查整个代码库，确保没有引入破坏性变更。
运行以下脚本来验证依赖更新的影响：
✅ Verification successful
运行脚本时，
rg返回“unrecognized file type: typescript”。请使用文件扩展名进行搜索。以下是修正后的脚本：
依赖版本更新已通过验证。
"@onekeyfe/hd-core" 和 "@onekeyfe/hd-web-sdk" 的版本更新在所有相关文件中均未引入破坏性变更。建议进行全面测试以确保兼容性。
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # 描述：检查是否有使用已更新依赖的特定API或功能 # 测试：搜索使用了这些依赖的代码。预期：找到使用这些依赖的地方，确保它们仍然兼容。 rg --type typescript -e '@onekeyfe/hd-ble-sdk' -e '@onekeyfe/hd-common-connect-sdk' -e '@onekeyfe/hd-core' -e '@onekeyfe/hd-web-sdk'Length of output: 161
Script:
#!/bin/bash # 检查已更新依赖的特定API或功能是否被使用 rg -g '*.ts' -g '*.tsx' -e '@onekeyfe/hd-ble-sdk' -e '@onekeyfe/hd-common-connect-sdk' -e '@onekeyfe/hd-core' -e '@onekeyfe/hd-web-sdk'Length of output: 8771
packages/connect-examples/expo-example/src/data/allnetwork.ts (1)
213-217: 新增 Nostr 网络支持添加了 Nostr 网络。这扩展了支持的网络范围。请确保相关文档和测试也已更新。
#!/bin/bash # 检查是否有相关的文档更新 rg --type markdown 'nostr' # 检查是否有相关的测试更新 rg --type typescript 'nostr.*test'packages/core/src/api/allnetwork/AllNetworkGetAddress.ts (2)
230-232: 添加了对 Nostr 网络的支持，确保正确性在第 230-232 行，新增了对 Nostr 网络的支持，请确保该网络的配置与业务需求一致，并在必要时添加相应的测试用例。
293-297: 完善参数结构，确保正确传递第 293-297 行中，
params对象包含了originPayload，请确认这是必要的，并且不会引入不必要的数据冗余。如果
originPayload非必要，建议移除以精简参数。
ByteZhang1024
force-pushed
the
optimize/allnetwork
branch
from
de67b60
to
4c4326d
Compare
There was a problem hiding this comment.
Actionable comments posted: 6
🧹 Outside diff range comments (1)
packages/hd-transport-lowlevel/package.json (1)
Line range hint
1-24: 建议添加 PR 描述。所有更改都很合理，但 PR 描述为空。建议添加简要说明，解释此次版本更新的原因和主要变化。这有助于其他开发者理解更新的目的和影响。
📜 Review details
Configuration used: CodeRabbit UI
Review profile: ASSERTIVE
📒 Files selected for processing (12)
- 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/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 (18)
packages/hd-transport-lowlevel/package.json (3)
3-3: 版本更新正确。版本号从 1.0.12-alpha.0 升级到 1.0.12-alpha.1，符合语义化版本规范。
22-22: 依赖更新合理。@onekeyfe/hd-shared 依赖版本从 ^1.0.12-alpha.0 更新到 ^1.0.12-alpha.1，与包版本更新一致。
23-23: 依赖更新合理。@onekeyfe/hd-transport 依赖版本从 ^1.0.12-alpha.0 更新到 ^1.0.12-alpha.1，与包版本更新一致。
packages/hd-transport-webusb/package.json (2)
3-3: 版本号更新正确。版本号从 1.0.12-alpha.0 升级到 1.0.12-alpha.1，符合语义化版本规范。
23-24: 依赖项版本同步更新。@onekeyfe/hd-shared 和 @onekeyfe/hd-transport 的版本号都更新到了 1.0.12-alpha.1，与包自身版本一致。这种同步更新有助于保持项目各部分的兼容性。
packages/hd-transport-react-native/package.json (3)
22-22: 依赖项更新正确。@onekeyfe/hd-shared 的版本已更新到 ^1.0.12-alpha.1，与包版本一致。
23-23: 依赖项更新正确。@onekeyfe/hd-transport 的版本已更新到 ^1.0.12-alpha.1，与包版本一致。
Line range hint
1-27: 请检查项目中的版本一致性。本文件的更改正确无误。但请注意，之前的审查发现其他文件可能存在版本不一致的问题。建议全面检查项目中所有相关文件的版本号，确保一致性。
运行以下脚本检查所有相关文件的版本：
#!/bin/bash # 检查所有相关文件的版本 fd -e json -x grep -H '"version":' {} | grep -v node_modulespackages/hd-transport-http/package.json (1)
3-3: 版本更新看起来不错！版本号和依赖项的更新保持一致，很好。
请运行以下脚本，确保所有相关包都已更新：
Also applies to: 27-28
packages/hd-common-connect-sdk/package.json (1)
3-3: 版本更新看起来不错！包版本和依赖项版本都已更新到 1.0.12-alpha.1。这很好。
请运行以下脚本，确保所有相关包都已一致更新：
Also applies to: 23-27
✅ Verification successful
版本更新已验证！
所有相关包的版本均已更新到 1.0.12-alpha.1。
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # 描述：检查所有包的版本是否已更新到 1.0.12-alpha.1 # 测试：搜索所有 package.json 文件中的版本号。期望：所有版本都是 1.0.12-alpha.1 fd -e json -x rg -i '"version":\s*"1\.0\.12-alpha\.1"' {}Length of output: 433
packages/core/package.json (2)
3-3: 版本号更新正确。版本号从 1.0.12-alpha.0 升级到 1.0.12-alpha.1，符合语义化版本规范。
28-29: 依赖项版本更新一致。@onekeyfe/hd-shared 和 @onekeyfe/hd-transport 的版本更新与包版本一致，保持了项目的一致性。
packages/hd-web-sdk/package.json (1)
3-3: 版本更新正确。包版本和依赖版本已更新到 1.0.12-alpha.1。变更保持一致，很好。
Also applies to: 24-27
packages/connect-examples/expo-example/package.json (5)
3-3: 版本更新正确。版本号从 1.0.12-alpha.0 升级到 1.0.12-alpha.1，符合语义化版本规范。
22-22: 依赖版本更新正确。@onekeyfe/hd-ble-sdk 依赖版本从 ^1.0.12-alpha.0 升级到 ^1.0.12-alpha.1，与项目版本一致。
23-23: 依赖版本更新正确。@onekeyfe/hd-common-connect-sdk 依赖版本从 ^1.0.12-alpha.0 升级到 ^1.0.12-alpha.1，与项目版本一致。
24-25: 依赖版本更新正确。@onekeyfe/hd-core 和 @onekeyfe/hd-web-sdk 依赖版本从 ^1.0.12-alpha.0 升级到 ^1.0.12-alpha.1，与项目版本一致。
Line range hint
3-25: 版本更新总结本次更新统一将项目版本和相关 @onekeyfe 依赖从 1.0.12-alpha.0 升级到 1.0.12-alpha.1。这种一致性有助于保持项目各组件的同步。请确保这些更改与其他相关模块协调一致。
运行以下脚本验证版本更新的一致性：
✅ Verification successful
版本更新已验证
所有
package.json文件中的项目版本和@onekeyfe依赖都已成功更新至1.0.12-alpha.1，验证通过。🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # 描述：验证所有 package.json 文件中的版本更新是否一致 # 测试：搜索所有 package.json 文件中的版本号。预期：所有版本号都应该是 1.0.12-alpha.1 rg --type json '"version":\s*"1\.0\.12-alpha\.1"' -g 'package.json' # 测试：搜索所有 package.json 文件中的 @onekeyfe 依赖版本。预期：所有版本号都应该是 ^1.0.12-alpha.1 rg --type json '"@onekeyfe/[^"]+": "\^1\.0\.12-alpha\.1"' -g 'package.json'Length of output: 3335
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/allnetwork/AllNetworkGetAddress.ts (8 hunks)
- packages/core/src/types/api/allNetworkGetAddress.ts (2 hunks)
🧰 Additional context used
🔇 Additional comments (5)
packages/core/src/types/api/allNetworkGetAddress.ts (2)
33-34: 网络类型更新正确。'nostr' 网络的添加扩展了支持的网络类型。这个改动很好。
36-36: 类型导出改进了代码复用性。CommonResponseParams 类型现在可以在其他模块中使用了。这是个好改动。
packages/core/src/api/allnetwork/AllNetworkGetAddress.ts (3)
293-297: 检查参数合并的正确性在生成方法名时，合并了
getParams返回的参数和originPayload。请确保合并过程不会覆盖关键属性，导致数据丢失。请验证参数合并是否准确，必要时调整合并顺序或方式。
230-232: 添加了 'nostr' 网络支持新增了 'nostr' 网络配置，使用方法名
nostrGetPublicKey。请确认该方法已在核心 API 中实现，且已妥善测试。运行以下脚本，验证
nostrGetPublicKey方法的存在与实现：#!/bin/bash # 检查核心 API 中是否存在 'nostrGetPublicKey' 方法 rg --type js 'nostrGetPublicKey' -A 5
102-102: 确认 'cosmos' 网络方法名的更新已将 'cosmos' 网络的
methodName更新为cosmosGetPublicKey。请确保所有调用该方法的地方都已同步更新，避免潜在问题。运行以下脚本，检查代码库中对
cosmosGetPublicKey的调用：✅ Verification successful
验证通过：未发现
cosmosGetPublicKey的调用。🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # 查找所有使用 'cosmosGetPublicKey' 的地方 rg --type js 'cosmosGetPublicKey' -A 5Length of output: 37
Script:
#!/bin/bash # 查找所有使用 'cosmosGetPublicKey' 的地方 rg --type js 'cosmosGetPublicKey' -A 5Length of output: 38
Signed-off-by: ByteZhang <[email protected]>
ByteZhang1024
force-pushed
the
optimize/allnetwork
branch
from
7c8e3f3
to
662e5c2
Compare
Summary by CodeRabbit
新特性
nostr网络的支持，包括其对应的派生路径。cosmos网络的methodName，从cosmosGetAddress改为cosmosGetPublicKey。版本更新
1.0.12-alpha.1，确保与最新功能和修复兼容。依赖更新