-
Notifications
You must be signed in to change notification settings - Fork 11
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
optimize: allnetwork #369
optimize: allnetwork #369
Conversation
Walkthrough此次更改涉及多个软件包的版本更新,从 Changes
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 (
|
New and removed dependencies detected. Learn more about Socket for GitHub ↗︎
🚮 Removed packages: npm/@onekeyfe/hd-core@1.0.11, npm/@onekeyfe/hd-shared@1.0.11, npm/@onekeyfe/hd-transport-http@1.0.11, npm/@onekeyfe/hd-transport-lowlevel@1.0.11, npm/@onekeyfe/hd-transport-react-native@1.0.11, npm/@onekeyfe/hd-transport-webusb@1.0.11, npm/@onekeyfe/hd-transport@1.0.11 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 3
🧹 Outside diff range comments (1)
packages/connect-examples/expo-example/package.json (1)
Line range hint
3-22
: 建议:全面测试新版本alpha 版本可能不稳定。合并前,请彻底测试所有功能,确保与其他依赖兼容。
📜 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/constants/connect.ts (1 hunks)
- packages/core/package.json (2 hunks)
- packages/core/src/api/allnetwork/AllNetworkGetAddress.ts (1 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 comments (22)
packages/connect-examples/expo-example/src/constants/connect.ts (1)
1-2
: SDK 版本更新和本地开发选项SDK 版本从 1.0.10 升级到了 1.0.12-alpha.0。注意,这是预发布版本,可能不稳定。
保留了本地开发 URL 注释很好,方便切换。考虑用环境变量控制 URL 选择?
✅ Verification successful
修正 SDK 版本搜索脚本
之前的脚本无法识别
typescript
类型。现在,我们将直接搜索.ts
和.tsx
文件。
SDK 版本更新已通过验证
没有在其他 TypeScript 文件中发现旧版本的 SDK URL。
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # 检查其他文件是否也更新了 SDK 版本 rg --type typescript "jssdk\.onekey\.so/1\.0\.(10|11)" -lLength of output: 94
Script:
#!/bin/bash # 检查其他 TypeScript 文件中是否更新了 SDK 版本 rg "jssdk\.onekey\.so/1\.0\.(10|11)" --glob "*.ts" --glob "*.tsx" -lLength of output: 70
packages/hd-transport-lowlevel/package.json (2)
3-3
: 版本更新至预发布状态版本号从 1.0.11 升级到 1.0.12-alpha.0,表明这是一个预发布版本。
22-23
: 依赖项版本同步更新@onekeyfe/hd-shared 和 @onekeyfe/hd-transport 的版本都更新到了 ^1.0.12-alpha.0,与包版本一致。这种同步更新很好。
packages/shared/package.json (1)
3-3
: 版本更新已确认版本号从 1.0.11 升级到了 1.0.12-alpha.0,表明进入了 alpha 阶段。
请确保其他依赖此包的项目已更新。运行以下脚本检查:
✅ Verification successful
版本更新已验证
所有依赖项已更新到 1.0.12-alpha.0 版本。请确保 alpha 版本的兼容性和稳定性。
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # 描述:查找使用 @onekeyfe/hd-shared 的文件 rg --type json '"@onekeyfe/hd-shared":\s*"(.+?)"' -g 'package.json'Length of output: 746
packages/hd-transport/package.json (1)
3-3
: 版本号更新正确。版本号从 1.0.11 升级到 1.0.12-alpha.0,符合语义化版本规范。
别忘了更新 changelog,记录这次预发布版本的变化。
packages/hd-ble-sdk/package.json (2)
3-3
: 版本号更新为预发布版本版本号从 1.0.11 升级到 1.0.12-alpha.0,表明这是一个预发布版本。确保在正式发布前充分测试新功能。
23-25
: 依赖项版本同步更新三个依赖项的版本已更新,与包版本保持一致。这种同步更新有助于维护包之间的兼容性。
packages/hd-transport-webusb/package.json (1)
3-3
: 版本更新检查包版本和依赖项已更新至1.0.12-alpha.0。这表明正在进行预发布测试。
请确保:
- 新功能与alpha版本相符
- 更新日志已更新
- 测试覆盖了新变化
Also applies to: 23-24
✅ Verification successful
版本更新一致性验证通过
所有相关包的版本已统一更新至1.0.12-alpha.0,符合预发布版本标准。
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # 检查其他相关包是否也更新了版本 grep -r '"version": "1.0.12-alpha.0"' packages/*/package.jsonLength of output: 1099
packages/hd-transport-react-native/package.json (2)
3-3
: 版本号更新至预发布版本版本号从 1.0.11 升级到 1.0.12-alpha.0,表明这是一个预发布版本。请确保这符合您的发布计划。
22-23
: 依赖项版本同步更新@onekeyfe/hd-shared 和 @onekeyfe/hd-transport 的版本已更新,与包版本一致。这很好,保持了版本同步。
packages/hd-transport-http/package.json (2)
3-3
: 版本号更新为预发布版本版本号从 1.0.11 升级到 1.0.12-alpha.0,表明这是一个预发布版本。
27-28
: 依赖项版本同步更新@onekeyfe/hd-shared 和 @onekeyfe/hd-transport 的版本已更新至 ^1.0.12-alpha.0,与包版本保持一致。
packages/hd-common-connect-sdk/package.json (1)
23-27
: 依赖项版本同步更新所有依赖项都更新到了 1.0.12-alpha.0 版本,保持了一致性。这种同步更新有助于避免版本冲突。
packages/core/package.json (3)
3-3
: 版本更新正确。版本号从 1.0.11 升级到 1.0.12-alpha.0,符合语义化版本规范。
28-28
: 依赖更新合理。@onekeyfe/hd-shared 依赖版本更新到 ^1.0.12-alpha.0,与包版本一致。
29-29
: 依赖更新合理,建议核实相关包。@onekeyfe/hd-transport 依赖版本更新到 ^1.0.12-alpha.0,与包版本一致。请确认相关包的变更。
运行以下脚本检查相关包的变更:
✅ Verification successful
依赖更新已验证
@onekeyfe/hd-transport 及其相关依赖包版本一致,更新无误。
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # 描述:检查 @onekeyfe/hd-shared 和 @onekeyfe/hd-transport 包的变更。 # 测试:查找这两个包的 package.json 文件并显示其内容。 echo "检查 @onekeyfe/hd-shared 包:" fd -t f "package.json" | rg "hd-shared" | xargs cat echo "检查 @onekeyfe/hd-transport 包:" fd -t f "package.json" | rg "hd-transport" | xargs catLength of output: 4225
packages/connect-examples/electron-example/package.json (1)
5-5
: 版本更新已确认。版本号从 1.0.11 升级到 1.0.12-alpha.0,表明项目进入了新的测试阶段。
请确保相关文档和发布说明已更新,以反映此alpha版本的变化:
packages/hd-web-sdk/package.json (2)
3-3
: 版本更新至预发布状态包版本从 1.0.11 升级到 1.0.12-alpha.0,表明这是新小版本的预发布。
24-27
: 依赖包版本同步更新四个依赖包版本已更新至 1.0.12-alpha.0,与主包版本一致。这确保了包之间的兼容性。
packages/connect-examples/expo-example/package.json (2)
19-22
: @onekeyfe 依赖同步更新@onekeyfe 相关依赖已更新到 1.0.12-alpha.0。保持一致性很好,但要注意可能的兼容性问题。
#!/bin/bash # 检查所有 @onekeyfe 依赖版本是否一致 jq -r '.dependencies | to_entries[] | select(.key | startswith("@onekeyfe")) | .value' packages/connect-examples/expo-example/package.json | sort | uniq -c
3-3
: 版本更新到预发布状态项目版本从 1.0.11 升级到 1.0.12-alpha.0。这是预发布版本,可能包含重大变更。
packages/core/src/types/api/allNetworkGetAddress.ts (1)
92-92
: 函数返回类型修改,需检查调用方
allNetworkGetAddress
函数的返回类型从Response<AllNetworkAddress>
修改为Response<AllNetworkAddress[]>
。请确保所有调用此函数的代码已更新,正确处理返回的数组类型。运行以下脚本查找并检查函数的调用:
#!/bin/bash # 描述:查找所有调用allNetworkGetAddress的地方 # 预期:调用方应处理返回的数组类型 rg --type ts 'allNetworkGetAddress' -A 5 -B 2
Summary by CodeRabbit
1.0.12-alpha.0
,引入了预发布版本,可能包含不稳定或不完整的功能。