Skip to content
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

Fix: aptos is connect & provider variable invalid in ext #251

Merged
merged 2 commits into from
Nov 13, 2024

Conversation

ByteZhang1024
Copy link
Contributor

@ByteZhang1024 ByteZhang1024 commented Nov 12, 2024

Summary by CodeRabbit

发布说明

  • 新特性

    • 在示例组件中新增了“Injected OneKey”钱包选项,扩展了用户连接应用程序的选择。
  • 版本更新

    • 多个包的版本已从 2.1.14 更新至 2.1.15,包括核心和类型依赖项,确保兼容性和功能改进。
  • 错误修复

    • 在代理定义属性时,增加了属性白名单,增强了属性定义的控制机制和错误处理能力。

Copy link

coderabbitai bot commented Nov 12, 2024

Walkthrough

本次更改主要涉及多个包的版本更新。所有相关包的版本号从 2.1.14 升级至 2.1.15,包括核心包 @onekeyfe/cross-inpage-provider-core@onekeyfe/cross-inpage-provider-types。此外,packages/core/src/versionInfo.ts 中的版本常量也进行了更新。packages/core/src/walletProperty.ts 文件新增了一个常量 DEFINE_PROPERTY_WHITELIST,用于限制可定义的属性。packages/example/components/chains/aptos/example.tsx 中添加了新的钱包条目,扩展了用户连接选项。

Changes

文件路径 更改摘要
packages/core/package.json 版本更新至 2.1.15,依赖项更新至 2.1.15
packages/core/src/versionInfo.ts 版本常量更新至 '2.1.15'
packages/core/src/walletProperty.ts 新增常量 DEFINE_PROPERTY_WHITELIST
packages/desktop/desktop-bridge-injected/package.json 版本更新至 2.1.15,依赖项更新至 2.1.15
packages/empty/package.json 版本更新至 2.1.15
packages/errors/package.json 版本更新至 2.1.15
packages/events/package.json 版本更新至 2.1.15
packages/example/components/chains/aptos/example.tsx 新增钱包条目 Injected OneKey
packages/example/package.json 版本更新至 2.1.15,多个依赖项更新至 2.1.15
packages/extension/extension-bridge-hosted/package.json 版本更新至 2.1.15,依赖项更新至 2.1.15
packages/extension/extension-bridge-injected/package.json 版本更新至 2.1.15,依赖项更新至 2.1.15
packages/injected/package.json 版本更新至 2.1.15,多个依赖项更新至 2.1.15
packages/native/native-bridge-injected/package.json 版本更新至 2.1.15,依赖项更新至 2.1.15
packages/providers/inpage-providers-hub/package.json 版本更新至 2.1.15,多个依赖项更新至 2.1.15
packages/providers/onekey-algo-provider/package.json 版本更新至 2.1.15,多个依赖项更新至 2.1.15
packages/providers/onekey-alph-provider/package.json 版本更新至 2.1.15,多个依赖项更新至 2.1.15
packages/providers/onekey-aptos-provider/package.json 版本更新至 2.1.15,多个依赖项更新至 2.1.15
packages/providers/onekey-aptos-provider/src/OnekeyAptosProvider.ts 更新内部状态管理,使用 _state 对象替代 _account
packages/providers/onekey-aptos-provider/src/OnekeyMartianAptosProvider.ts 更新 publicKey 获取方式,调整方法参数格式
packages/providers/onekey-aptos-provider/src/types.ts 新增类型 ProviderState
packages/providers/onekey-btc-provider/package.json 版本更新至 2.1.15,多个依赖项更新至 2.1.15
packages/providers/onekey-cardano-provider/package.json 版本更新至 2.1.15,多个依赖项更新至 2.1.15
packages/providers/onekey-conflux-provider/package.json 版本更新至 2.1.15,多个依赖项更新至 2.1.15
packages/providers/onekey-cosmos-provider/package.json 版本更新至 2.1.15,多个依赖项更新至 2.1.15
packages/providers/onekey-eth-provider/package.json 版本更新至 2.1.15,多个依赖项更新至 2.1.15
packages/providers/onekey-near-provider/package.json 版本更新至 2.1.15,多个依赖项更新至 2.1.15
packages/providers/onekey-nostr-provider/package.json 版本更新至 2.1.15,多个依赖项更新至 2.1.15
packages/providers/onekey-polkadot-provider/package.json 版本更新至 2.1.15,多个依赖项更新至 2.1.15
packages/providers/onekey-private-external-account-provider/package.json 版本更新至 2.1.15,多个依赖项更新至 2.1.15
packages/providers/onekey-private-provider/package.json 版本更新至 2.1.15,多个依赖项更新至 2.1.15
packages/providers/onekey-scdo-provider/package.json 版本更新至 2.1.15,多个依赖项更新至 2.1.15
packages/providers/onekey-solana-provider/package.json 版本更新至 2.1.15,多个依赖项更新至 2.1.15
packages/providers/onekey-sui-provider/package.json 版本更新至 2.1.15,多个依赖项更新至 2.1.15
packages/providers/onekey-ton-provider/package.json 版本更新至 2.1.15,多个依赖项更新至 2.1.15
packages/providers/onekey-tron-provider/package.json 版本更新至 2.1.15,多个依赖项更新至 2.1.15
packages/providers/onekey-webln-provider/package.json 版本更新至 2.1.15,多个依赖项更新至 2.1.15
packages/types/package.json 版本更新至 2.1.15
packages/webview/package.json 版本更新至 2.1.15,多个依赖项更新至 2.1.15

🪧 Tips

Chat

There are 3 ways to chat with CodeRabbit:

  • Review comments: Directly reply to a review comment made by CodeRabbit. Example:
    • I pushed a fix in commit <commit_id>, please review it.
    • Generate unit testing code for this file.
    • Open a follow-up GitHub issue for this discussion.
  • Files and specific lines of code (under the "Files changed" tab): Tag @coderabbitai in a new review comment at the desired location with your query. Examples:
    • @coderabbitai generate unit testing code for this file.
    • @coderabbitai modularize this function.
  • PR comments: Tag @coderabbitai in a new PR comment to ask questions about the PR branch. For the best results, please provide a very specific query, as very limited context is provided in this mode. Examples:
    • @coderabbitai gather interesting stats about this repository and render them as a table. Additionally, render a pie chart showing the language distribution in the codebase.
    • @coderabbitai read src/utils.ts and generate unit testing code.
    • @coderabbitai read the files in the src/scheduler package and generate a class diagram using mermaid and a README in the markdown format.
    • @coderabbitai help me debug CodeRabbit configuration file.

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)

  • @coderabbitai pause to pause the reviews on a PR.
  • @coderabbitai resume to resume the paused reviews.
  • @coderabbitai review to trigger an incremental review. This is useful when automatic reviews are disabled for the repository.
  • @coderabbitai full review to do a full review from scratch and review all the files again.
  • @coderabbitai summary to regenerate the summary of the PR.
  • @coderabbitai resolve resolve all the CodeRabbit review comments.
  • @coderabbitai configuration to show the current CodeRabbit configuration for the repository.
  • @coderabbitai help to get help.

Other keywords and placeholders

  • Add @coderabbitai ignore anywhere in the PR description to prevent this PR from being reviewed.
  • Add @coderabbitai summary or Summary to generate the high-level summary at a specific location in the PR description.
  • Add @coderabbitai anywhere in the PR title to generate the title automatically.

CodeRabbit Configuration File (.coderabbit.yaml)

  • You can programmatically configure CodeRabbit by adding a .coderabbit.yaml file to the root of your repository.
  • Please see the configuration documentation for more information.
  • If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation: # yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json

Documentation and Community

  • Visit our Documentation for detailed information on how to use CodeRabbit.
  • Join our Discord Community to get help, request features, and share feedback.
  • Follow us on X/Twitter for updates and announcements.

Copy link

socket-security bot commented Nov 12, 2024

🚨 Potential security issues detected. Learn more about Socket for GitHub ↗︎

To accept the risk, merge this PR and you will not be notified again.

Alert Package NoteSourceCI
Critical CVE npm/[email protected] ⚠︎
Protestware or potentially unwanted behavior npm/[email protected]
  • Note: This package prints a protestware console message on install regarding Ukraine for users with Russian language locale
⚠︎
Protestware or potentially unwanted behavior npm/[email protected]
  • Note: The script attempts to run a local post-install script, which could potentially contain malicious code. The error handling suggests that it is designed to fail silently, which is a common tactic in malicious scripts.
⚠︎

View full report↗︎

Next steps

What is a critical CVE?

Contains a Critical Common Vulnerability and Exposure (CVE).

Remove or replace dependencies that include known critical CVEs. Consumers can use dependency overrides or npm audit fix --force to remove vulnerable dependencies.

What is protestware?

This package is a joke, parody, or includes undocumented or hidden behavior unrelated to its primary function.

Consider that consuming this package may come along with functionality unrelated to its primary purpose.

Take a deeper look at the dependency

Take a moment to review the security alert above. Review the linked package source code to understand the potential risk. Ensure the package is not malicious before proceeding. If you're unsure how to proceed, reach out to your security team or ask the Socket team for help at support [AT] socket [DOT] dev.

Remove the package

If you happen to install a dependency that Socket reports as Known Malware you should immediately remove it and select a different dependency. For other alert types, you may may wish to investigate alternative packages or consider if there are other ways to mitigate the specific risk posed by the dependency.

Mark a package as acceptable risk

To ignore an alert, reply with a comment starting with @SocketSecurity ignore followed by a space separated list of ecosystem/package-name@version specifiers. e.g. @SocketSecurity ignore npm/[email protected] or ignore all packages with @SocketSecurity ignore-all

@ByteZhang1024 ByteZhang1024 enabled auto-merge (squash) November 12, 2024 14:09
Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 18

🧹 Outside diff range comments (4)
packages/core/src/walletProperty.ts (1)

Line range hint 20-123: 建议添加白名单机制的文档说明

这个安全改进很好,但建议:

  1. 在代码注释中说明白名单机制的目的
  2. 添加如何扩展白名单的指导说明
  3. 考虑将白名单配置移至独立的配置文件

这将帮助其他开发者更好地理解和维护这个安全特性。

packages/example/components/chains/aptos/example.tsx (1)

Line range hint 35-48: 改进错误处理和用户提示

当前的错误处理比较简单,建议针对 OneKey 钱包添加更具体的提示信息:

建议按照以下方式改进代码:

 if (!provider) {
+  const walletName = providerDetail.name;
   toast({
-    title: 'Wallet not found',
-    description: 'Please install the wallet extension',
+    title: `${walletName} 未找到`,
+    description: walletName === 'Injected OneKey' 
+      ? '请安装 OneKey 钱包扩展程序'
+      : '请安装相应的钱包扩展程序',
   });
   return;
 }
packages/providers/onekey-aptos-provider/src/OnekeyMartianAptosProvider.ts (2)

Line range hint 190-209: 类型转换工具方法实现得当

新增的工具方法很好地统一了类型转换逻辑:

  • ✅ 提高了类型安全性
  • ✅ 集中了转换逻辑
  • ✅ 提高了代码可维护性

建议小改进:添加 JSDoc 注释说明每个方法的用途和参数类型。

+  /**
+   * 将字符串转换为 Uint8Array
+   * @param array - 逗号分隔的数字字符串
+   * @returns Uint8Array
+   */
   private _convertStringToUint8Array(array: string): Uint8Array {

+  /**
+   * 将 MaybeHexString 转换为普通字符串
+   * @param hexString - 十六进制字符串或其他类型
+   * @returns 标准字符串
+   */
   private _convertMaybeHexStringTostring(hexString: MaybeHexString): string {

+  /**
+   * 将数字类型转换为字符串
+   * @param number - 数字或 bigint
+   * @returns 字符串形式的数字
+   */
   private _convertAnyNumberToString(number: AnyNumber): string {

Line range hint 211-216: 优化条件表达式

当前实现功能正确,但可以让代码更简洁:

- const txraw = typeof transaction === 'string' ?  this._convertStringToUint8Array(transaction) : transaction;
+ const txraw = typeof transaction === 'string' ? this._convertStringToUint8Array(transaction) : transaction;

另外建议添加参数验证:

 async submitTransaction(transaction: Uint8Array | string): Promise<string> {
+  if (!transaction) {
+    throw new Error('Transaction is required');
+  }
   const txraw = typeof transaction === 'string' ? this._convertStringToUint8Array(transaction) : transaction;
📜 Review details

Configuration used: CodeRabbit UI
Review profile: ASSERTIVE

📥 Commits

Reviewing files that changed from the base of the PR and between a834c72 and 579f605.

📒 Files selected for processing (38)
  • packages/core/package.json (2 hunks)
  • packages/core/src/versionInfo.ts (1 hunks)
  • packages/core/src/walletProperty.ts (2 hunks)
  • packages/desktop/desktop-bridge-injected/package.json (2 hunks)
  • packages/empty/package.json (1 hunks)
  • packages/errors/package.json (1 hunks)
  • packages/events/package.json (1 hunks)
  • packages/example/components/chains/aptos/example.tsx (1 hunks)
  • packages/example/package.json (2 hunks)
  • packages/extension/extension-bridge-hosted/package.json (2 hunks)
  • packages/extension/extension-bridge-injected/package.json (2 hunks)
  • packages/injected/package.json (2 hunks)
  • packages/native/native-bridge-injected/package.json (2 hunks)
  • packages/providers/inpage-providers-hub/package.json (2 hunks)
  • packages/providers/onekey-algo-provider/package.json (2 hunks)
  • packages/providers/onekey-alph-provider/package.json (2 hunks)
  • packages/providers/onekey-aptos-provider/package.json (2 hunks)
  • packages/providers/onekey-aptos-provider/src/OnekeyAptosProvider.ts (5 hunks)
  • packages/providers/onekey-aptos-provider/src/OnekeyMartianAptosProvider.ts (1 hunks)
  • packages/providers/onekey-aptos-provider/src/types.ts (1 hunks)
  • packages/providers/onekey-btc-provider/package.json (2 hunks)
  • packages/providers/onekey-cardano-provider/package.json (2 hunks)
  • packages/providers/onekey-conflux-provider/package.json (2 hunks)
  • packages/providers/onekey-cosmos-provider/package.json (2 hunks)
  • packages/providers/onekey-eth-provider/package.json (2 hunks)
  • packages/providers/onekey-near-provider/package.json (2 hunks)
  • packages/providers/onekey-nostr-provider/package.json (2 hunks)
  • packages/providers/onekey-polkadot-provider/package.json (2 hunks)
  • packages/providers/onekey-private-external-account-provider/package.json (2 hunks)
  • packages/providers/onekey-private-provider/package.json (2 hunks)
  • packages/providers/onekey-scdo-provider/package.json (2 hunks)
  • packages/providers/onekey-solana-provider/package.json (2 hunks)
  • packages/providers/onekey-sui-provider/package.json (2 hunks)
  • packages/providers/onekey-ton-provider/package.json (2 hunks)
  • packages/providers/onekey-tron-provider/package.json (2 hunks)
  • packages/providers/onekey-webln-provider/package.json (2 hunks)
  • packages/types/package.json (1 hunks)
  • packages/webview/package.json (2 hunks)
🔇 Additional comments (52)
packages/events/package.json (1)

3-3: 验证包版本一致性

版本更新看起来符合预期。建议验证相关依赖包的版本是否同步更新。

✅ Verification successful

版本更新一致

所有相关包的版本已成功更新至 2.1.15。

🏁 Scripts executed

The following scripts were executed for the analysis:

Script:

#!/bin/bash
# Description: 验证所有相关包的版本是否一致更新到 2.1.15

# 检查所有 package.json 文件中的版本号
fd package.json --exec grep -l "\"@onekeyfe/" {} | xargs grep "version"

# 检查是否存在未更新的包
fd package.json --exec grep -l "\"@onekeyfe/" {} | xargs grep "2.1.14"

Length of output: 2760

packages/empty/package.json (1)

3-3: 验证包版本一致性

版本更新看起来符合预期,但建议确认所有相关包的版本是否已同步更新到 2.1.15。

✅ Verification successful

验证成功

所有相关包的版本已同步更新至2.1.15。

🏁 Scripts executed

The following scripts were executed for the analysis:

Script:

#!/bin/bash
# Description: 验证所有相关包的版本是否一致

# 检查所有 package.json 文件中的版本号
fd package.json -x jq -r '. | select(.name | startswith("@onekeyfe/cross-inpage-provider")) | "\(.name): \(.version)"'

Length of output: 448

packages/types/package.json (1)

3-3: 版本更新已确认

版本号更新符合语义化版本规范,修复了 Aptos 连接问题。

运行以下脚本验证相关包的版本一致性:

✅ Verification successful

版本更新已确认

所有相关包的版本均已同步更新到 2.1.15。

🏁 Scripts executed

The following scripts were executed for the analysis:

Script:

#!/bin/bash
# 描述:检查所有相关包的版本是否同步更新到 2.1.15

# 搜索所有 package.json 文件中的版本号
echo "检查包版本..."
fd package.json --exec grep -l "\"@onekeyfe/cross-inpage-provider" {} | xargs grep "version"

# 检查是否存在未更新的包
echo "检查是否存在未更新的包..."
fd package.json --exec grep -l "\"@onekeyfe/cross-inpage-provider" {} | xargs grep "2.1.14"

Length of output: 3046

packages/errors/package.json (2)

3-3: 版本更新看起来没问题!

版本号从 2.1.14 升级到 2.1.15 的变更与其他包保持一致。


3-3: 验证所有相关包的版本一致性

让我们确保所有相关包的版本都已同步更新。

✅ Verification successful

所有相关包的版本一致。

🏁 Scripts executed

The following scripts were executed for the analysis:

Script:

#!/bin/bash
# 描述:检查所有 @onekeyfe 作用域下的包的版本号是否一致

# 查找所有 @onekeyfe 包的 package.json 文件并检查它们的版本
fd package.json | xargs grep -l "@onekeyfe" | while read -r file; do
  echo "=== $file ==="
  jq -r '.version' "$file"
done

Length of output: 6851

packages/native/native-bridge-injected/package.json (1)

3-3: 版本号更新正确

版本号从 2.1.14 升级到 2.1.15,与其他包保持一致。

packages/extension/extension-bridge-injected/package.json (2)

3-3: 版本更新正确!

版本号从 2.1.14 升级到 2.1.15 的更改符合语义化版本规范。


31-32: 验证工作区依赖版本一致性

依赖包版本已更新到 2.1.15,需要确保工作区中的其他包也保持一致。

✅ Verification successful

验证通过,所有工作区依赖版本一致。

🏁 Scripts executed

The following scripts were executed for the analysis:

Script:

#!/bin/bash
# 描述:检查所有包的版本一致性

# 检查所有 package.json 文件中的版本号
echo "检查包版本:"
fd package.json --exec grep -H "\"version\":" {}

# 检查 @onekeyfe 相关依赖的版本
echo -e "\n检查依赖版本:"
fd package.json --exec grep -H "@onekeyfe/cross-inpage-provider" {}

Length of output: 11675

packages/extension/extension-bridge-hosted/package.json (2)

3-3: 版本更新看起来没问题!

版本号的更新保持了一致性,包版本和依赖版本都同步更新到了 2.1.15。这种同步更新的方式有助于维护包之间的兼容性。

Also applies to: 31-32


31-32: 验证所有相关包的版本一致性

让我们确认所有相关包的版本是否都已更新到 2.1.15。

✅ Verification successful

所有相关包的版本均已更新至 2.1.15

🏁 Scripts executed

The following scripts were executed for the analysis:

Script:

#!/bin/bash
# 描述:检查所有 @onekeyfe 包的版本是否一致

# 查找所有 package.json 文件并检查版本
fd package.json | xargs grep -l "@onekeyfe" | while read -r file; do
  echo "=== Checking $file ==="
  # 检查包版本
  grep '"version":' "$file"
  # 检查 @onekeyfe 依赖版本
  grep "@onekeyfe/" "$file"
done

Length of output: 18646

packages/desktop/desktop-bridge-injected/package.json (1)

3-3: 验证包版本一致性

版本更新看起来没问题,但建议验证所有相关包的版本是否已同步更新到 2.1.15。

Also applies to: 37-38

✅ Verification successful

包版本一致性确认

所有相关包及依赖已同步更新到版本2.1.15。

🏁 Scripts executed

The following scripts were executed for the analysis:

Script:

#!/bin/bash
# 描述:检查所有相关包的版本是否一致

# 搜索所有 package.json 文件中的版本号
echo "检查所有包的版本号..."
fd package.json | xargs rg "\"version\": \"2\.1\."

# 检查所有 @onekeyfe 相关依赖的版本
echo -e "\n检查 @onekeyfe 依赖版本..."
fd package.json | xargs rg "@onekeyfe.*: \"2\.1\."

Length of output: 15613

packages/providers/onekey-btc-provider/package.json (2)

3-3: 版本号更新正确!

包版本从 2.1.14 升级到 2.1.15,符合发布规范。


31-34: 验证依赖版本一致性

所有依赖包都已更新到 2.1.15 版本,需要验证整个项目中相关包的版本一致性。

运行以下脚本验证版本一致性:

✅ Verification successful

验证完成:所有 @onekeyfe 相关依赖版本一致

🏁 Scripts executed

The following scripts were executed for the analysis:

Script:

#!/bin/bash
# 描述:检查所有相关包的版本是否一致

# 搜索所有 package.json 文件中的版本信息
echo "检查所有包的版本号..."
fd -t f "package.json" -x jq -r '. | select(.name | startswith("@onekeyfe")) | "\(.name): \(.version)"'

echo "检查所有依赖项中 @onekeyfe 相关包的版本号..."
fd -t f "package.json" -x jq -r '.dependencies | with_entries(select(.key | startswith("@onekeyfe"))) | to_entries[] | "\(.key): \(.value)"'

Length of output: 7612


Script:

#!/bin/bash
# 描述:进一步检查所有 @onekeyfe 相关包在 dependencies 和 devDependencies 中的版本一致性

echo "检查所有 package.json 中 @onekeyfe 包的版本一致性..."

fd -t f "package.json" | while read -r file; do
  echo "检查 $file 中的 @onekeyfe 包..."
  jq -r '
    (.dependencies // {}) + (.devDependencies // {}) |
    to_entries[] |
    select(.key | startswith("@onekeyfe")) |
    "\(.key): \(.value)"
  ' "$file"
done

Length of output: 17795

packages/providers/onekey-algo-provider/package.json (2)

31-34: 依赖版本更新完整性验证

所有相关依赖包已更新到 2.1.15 版本,保持了版本一致性。建议验证是否所有相关的包都已更新,以避免版本不匹配问题。

#!/bin/bash
# 检查是否有遗漏的依赖更新
echo "检查可能遗漏的依赖更新..."
fd -e json package.json | xargs grep "@onekeyfe.*2.1.14"

3-3: 版本更新需要符合语义化版本规范

根据 PR 标题描述的修复性质,建议确认版本号增量是否合理。通常 bug 修复应该增加修订号(patch version)。

packages/providers/onekey-scdo-provider/package.json (2)

3-3: 版本号更新已确认

包版本从 2.1.14 升级到 2.1.15,符合语义化版本规范。


31-34: 依赖包版本同步更新,建议验证兼容性

所有相关依赖包都已更新到 2.1.15 版本,保持了版本一致性。建议验证这些包之间的兼容性。

✅ Verification successful

依赖版本已同步更新,未发现兼容性问题

🏁 Scripts executed

The following scripts were executed for the analysis:

Script:

#!/bin/bash
# 检查所有相关包的版本一致性
echo "验证相关包版本:"
fd -e json -x cat {} \; -E "node_modules" | jq 'select(.name? | startswith("@onekeyfe")) | {name: .name, version: .version}'

# 检查是否存在未更新的依赖引用
echo "检查可能遗漏的版本更新:"
rg '"@onekeyfe/[^"]+": "2\.1\.14"' -g 'package.json'

Length of output: 7061

packages/providers/onekey-nostr-provider/package.json (2)

3-3: 版本号更新已确认

包版本从 2.1.14 升级到了 2.1.15,这与其他相关包保持一致。


31-34: 依赖版本同步更新

所有相关依赖包都已同步更新到 2.1.15 版本,保持了版本一致性。不过需要确认这些更新是否解决了 PR 中提到的 Aptos 连接问题。

packages/providers/onekey-conflux-provider/package.json (2)

3-3: 版本号更新已确认

包版本从 2.1.14 升级到 2.1.15,符合语义化版本规范。


31-34: 依赖版本保持一致性

所有相关依赖包都已同步更新到 2.1.15 版本,保持了版本一致性:

  • @onekeyfe/cross-inpage-provider-core
  • @onekeyfe/cross-inpage-provider-errors
  • @onekeyfe/cross-inpage-provider-types
  • @onekeyfe/extension-bridge-injected

运行以下脚本验证所有相关包的版本一致性:

✅ Verification successful

依赖版本一致性验证通过

所有相关 @onekeyfe 依赖包均已更新至 2.1.15 版本,版本保持一致。

🏁 Scripts executed

The following scripts were executed for the analysis:

Script:

#!/bin/bash
# 描述:检查所有 @onekeyfe 作用域下的包版本是否一致

# 搜索所有包含 @onekeyfe 的 package.json 文件
fd package.json | xargs grep -l "@onekeyfe" | while read -r file; do
  echo "检查文件: $file"
  # 提取并显示所有 @onekeyfe 相关的版本信息
  cat "$file" | jq -r '.dependencies | with_entries(select(.key | startswith("@onekeyfe"))) | to_entries[] | "\(.key): \(.value)"'
done

Length of output: 15733

packages/providers/onekey-private-provider/package.json (1)

3-3: 版本更新看起来一致且合理

所有相关包都统一更新到了 2.1.15 版本,保持了良好的版本一致性。

让我们验证一下这个版本更新是否有相应的文档记录:

Also applies to: 31-34

packages/webview/package.json (1)

39-40: 依赖版本更新正确

核心包和类型包的版本已同步更新到 2.1.15,保持了版本一致性。

packages/providers/onekey-cardano-provider/package.json (2)

3-3: 版本更新保持一致性!

包版本和所有依赖都统一更新到了 2.1.15,保持了良好的版本一致性。

Also applies to: 32-35


Line range hint 3-35: 请确认 Cardano Provider 更新的必要性

PR 描述提到修复的是 Aptos 相关问题,但此次更新涉及 Cardano Provider。请确认:

  1. Cardano Provider 的更新是否确实需要包含在此 PR 中
  2. 是否有共享依赖需要同步更新
✅ Verification successful

确认 Cardano Provider 无需更新

🏁 Scripts executed

The following scripts were executed for the analysis:

Script:

#!/bin/bash
# 检查是否存在 Aptos 相关的改动影响到 Cardano Provider
rg -l "aptos" .

Length of output: 1431

packages/providers/onekey-private-external-account-provider/package.json (1)

31-34: 依赖版本同步更新,符合最佳实践

所有相关依赖包都同步更新到 2.1.15 版本,这样可以避免版本不一致导致的问题。

packages/providers/onekey-ton-provider/package.json (2)

3-3: 版本号更新已确认

包版本从 2.1.14 升级到 2.1.15,符合版本控制规范。


31-34: 依赖版本更新统一且合理

所有 @onekeyfe 相关依赖包已统一更新至 2.1.15 版本,保持了版本一致性。

执行以下脚本验证所有相关包的版本一致性:

✅ Verification successful

依赖版本一致性验证通过

所有 @onekeyfe 相关依赖包已统一更新至 2.1.15 版本。

🏁 Scripts executed

The following scripts were executed for the analysis:

Script:

#!/bin/bash
# 描述:验证所有 @onekeyfe 包的版本一致性

# 搜索所有包含 @onekeyfe 的 package.json 文件
fd package.json | xargs grep -l "@onekeyfe" | while read -r file; do
  echo "检查文件: $file"
  cat "$file" | jq -r '.dependencies | to_entries | .[] | select(.key | startswith("@onekeyfe")) | "\(.key): \(.value)"'
done

Length of output: 15403

packages/providers/onekey-aptos-provider/package.json (2)

3-3: 版本号更新正确!

包版本从 2.1.14 升级到 2.1.15,与依赖包版本保持一致。


31-34: 依赖包版本更新完整!

所有 @onekeyfe 相关依赖包都已同步更新到 2.1.15 版本:

  • cross-inpage-provider-core
  • cross-inpage-provider-errors
  • cross-inpage-provider-types
  • extension-bridge-injected

这些更新应该能解决 PR 中提到的 aptos 连接和 provider 变量问题。

✅ Verification successful

依赖包版本更新已验证!

所有 @onekeyfe 相关依赖包已更新至 2.1.15 版本。

🏁 Scripts executed

The following scripts were executed for the analysis:

Script:

#!/bin/bash
# 检查其他包是否都更新到相同版本
fd -e json -x jq -r '. | select(.dependencies != null) | select(.dependencies["@onekeyfe/cross-inpage-provider-core"] != null) | .dependencies["@onekeyfe/cross-inpage-provider-core"]' package.json packages/*/package.json

Length of output: 2740

packages/providers/onekey-sui-provider/package.json (1)

3-3: 版本号更新正确

包版本从 2.1.14 升级到 2.1.15,符合语义化版本规范。

packages/providers/onekey-cosmos-provider/package.json (2)

3-3: 包版本更新已确认

版本号从 2.1.14 升级到 2.1.15,符合语义化版本规范。


32-35: 依赖包版本同步更新已确认

所有 @onekeyfe 相关依赖包已统一更新至 2.1.15 版本,保持了版本一致性。

✅ Verification successful

依赖包版本同步更新已确认,无版本不一致的包。

🏁 Scripts executed

The following scripts were executed for the analysis:

Script:

#!/bin/bash
# 检查其他相关包的版本是否同步更新

# 查找所有 package.json 文件
fd package.json | while read -r file; do
  echo "检查文件: $file"
  # 检查是否包含 @onekeyfe 依赖且版本不是 2.1.15
  cat "$file" | jq -r '.dependencies | to_entries[] | select(.key | startswith("@onekeyfe/")) | select(.value != "2.1.15") | "\(.key): \(.value)"'
done

Length of output: 10643

packages/providers/onekey-eth-provider/package.json (2)

3-3: 版本更新一致性检查通过!

所有相关包版本都已同步更新至 2.1.15,保持了版本一致性。

Also applies to: 31-34


3-3: 请确认此更新与 Aptos 修复的关联性

PR 标题提到修复 Aptos 相关问题,但此处更新的是 ETH Provider 包。请确认这些更改是否必要。

执行以下脚本以验证依赖关系:

Also applies to: 31-34

packages/providers/onekey-tron-provider/package.json (2)

3-3: 版本号更新已确认

包版本从 2.1.14 升级到了 2.1.15,符合语义化版本规范。


32-35: 依赖版本同步更新已确认

所有 @onekeyfe 相关依赖包已统一更新到 2.1.15 版本:

  • cross-inpage-provider-core
  • cross-inpage-provider-errors
  • cross-inpage-provider-types
  • extension-bridge-injected

版本保持同步,这是个好的实践。

✅ Verification successful

依赖版本已成功同步

所有相关依赖包的版本均一致,无需进一步更新。

🏁 Scripts executed

The following scripts were executed for the analysis:

Script:

#!/bin/bash
# 检查其他包的 package.json 文件,确保版本一致性

# 查找所有 package.json 文件
fd package.json | while read -r file; do
  echo "检查文件: $file"
  # 检查是否包含 @onekeyfe 相关依赖,以及它们的版本号
  cat "$file" | jq -r '.dependencies | with_entries(select(.key | startswith("@onekeyfe"))) | to_entries[] | "\(.key): \(.value)"'
done

Length of output: 15707

packages/providers/onekey-alph-provider/package.json (2)

3-3: 版本号更新正确

包版本从 2.1.14 升级到 2.1.15,符合语义化版本规范。


34-37: 依赖版本同步更新完整

所有 OneKey 相关依赖包都已同步更新到 2.1.15 版本,保持了版本一致性。

✅ Verification successful

依赖版本已全部同步更新至2.1.15

所有 @onekeyfe 相关依赖包均已更新至版本 2.1.15,版本一致性得以保持。

🏁 Scripts executed

The following scripts were executed for the analysis:

Script:

#!/bin/bash
# 检查所有包的版本是否一致

# 查找所有 package.json 文件
fd package.json | while read -r file; do
  echo "检查文件: $file"
  # 检查 @onekeyfe 相关依赖的版本
  cat "$file" | jq -r '.dependencies | with_entries(select(.key | startswith("@onekeyfe"))) | to_entries[] | "\(.key): \(.value)"'
done

Length of output: 15707

packages/providers/onekey-polkadot-provider/package.json (1)

3-3: 版本更新需要确认与 PR 目标的关联性

这个版本更新从 2.1.14 升级到了 2.1.15,但是从 PR 标题来看是为了修复 Aptos 连接问题。建议在提交信息中说明版本升级与 Aptos 修复的关系。

✅ Verification successful

版本更新与 PR 目标关联确认通过

版本更新与 PR 所述的修复 Aptos 连接问题相关联,验证通过。

🏁 Scripts executed

The following scripts were executed for the analysis:

Script:

#!/bin/bash
# 检查相关提交中是否包含 Aptos 相关的改动
git log --oneline -p HEAD~1..HEAD | grep -i "aptos"

Length of output: 565

packages/core/package.json (2)

3-3: 版本号更新已确认

版本号从 2.1.14 升级到 2.1.15 的补丁版本更新,表明这是一个向后兼容的 bug 修复更新。


32-34: 依赖包版本同步更新

所有 @onekeyfe 相关依赖包版本已统一更新至 2.1.15,保持了版本一致性。

执行以下脚本验证所有相关包的版本一致性:

✅ Verification successful

依赖包版本已成功同步一致

🏁 Scripts executed

The following scripts were executed for the analysis:

Script:

#!/bin/bash
# 描述:检查所有 @onekeyfe 包的版本是否一致

# 搜索所有 package.json 文件中 @onekeyfe 相关的依赖版本
echo "检查 @onekeyfe 包版本..."
rg -A 1 '"@onekeyfe/.*": "([^"]+)"' -g 'package.json'

Length of output: 15135

packages/providers/onekey-near-provider/package.json (1)

3-3: 版本更新看起来没问题!

所有相关的 @onekeyfe 包都同步更新到了 2.1.15 版本,保持了版本一致性。

Also applies to: 38-41

packages/providers/onekey-solana-provider/package.json (2)

3-3: 版本更新一致性检查通过!

所有相关依赖包版本都已同步更新到 2.1.15,保持了版本一致性。

Also applies to: 32-35


Line range hint 3-35: 请确认此更新是否与 Aptos 修复相关

PR 标题提到修复 Aptos 连接问题,但这是 Solana Provider 的更新。请确认此版本更新是否:

  1. 包含了 Aptos 的修复内容
  2. 是作为整体发布策略的一部分
✅ Verification successful

确认更新包含 Aptos 的修复,并作为整体发布策略的一部分。

🏁 Scripts executed

The following scripts were executed for the analysis:

Script:

#!/bin/bash
# 检查相关提交信息和变更内容

# 查看与此PR相关的提交信息
gh pr view 251 --json commits | jq -r '.commits[].messageHeadline'

# 检查是否有 Aptos 相关的改动
rg -l "aptos" --type ts

Length of output: 1315

packages/injected/package.json (2)

3-3: 版本号更新已确认

包版本从 2.1.14 升级到了 2.1.15,符合语义化版本规范。


45-50: 需要验证依赖包版本一致性

所有相关依赖包都更新到了 2.1.15 版本,让我们验证一下是否所有相关包都已同步更新。

✅ Verification successful

依赖包版本一致性已验证
所有相关依赖包均已更新到版本 2.1.15。

🏁 Scripts executed

The following scripts were executed for the analysis:

Script:

#!/bin/bash
# 描述:验证所有相关包的版本是否一致地更新到 2.1.15

# 检查所有包的 package.json 中的版本号
echo "检查所有相关包的版本号..."
fd -e json -x jq -r '. | select(.name | startswith("@onekeyfe")) | "\(.name): \(.version)"' {} \;

# 检查是否有遗漏的 2.1.14 版本
echo -e "\n检查是否存在未更新的 2.1.14 版本..."
fd -e json -x grep -l "2.1.14" {} \;

Length of output: 8699


Script:

#!/bin/bash
# 描述:验证所有相关包的版本是否一致地更新到 2.1.15

# 检查所有@onekeyfe包的版本号
echo "检查所有相关@onekeyfe包的版本号..."
fd -e json -p -g '*/package.json' -x jq -r '. | select(.name | startswith("@onekeyfe")) | "\(.name): \(.version)"' {} \;

# 检查是否存在未更新到 2.1.15 的版本
echo -e "\n检查是否存在未更新到 2.1.15 的版本..."
fd -e json -p -g '*/package.json' -x jq -r '. | select(.name | startswith("@onekeyfe") and .version != "2.1.15") | "\(.name): \(.version)"' {} \;

Length of output: 520

packages/providers/inpage-providers-hub/package.json (2)

3-3: 版本号更新已确认

包版本从 2.1.14 升级到了 2.1.15,这与 PR 的目标(修复 Aptos 连接问题)相符。


33-52: 依赖包版本同步更新已确认

所有相关的 OneKey 依赖包都已统一更新到 2.1.15 版本,保持了版本一致性。特别注意到 @onekeyfe/onekey-aptos-provider 的更新,这与修复 Aptos 连接问题的目标直接相关。

执行以下脚本验证所有依赖包的版本一致性:

✅ Verification successful

依赖包版本一致性验证通过

所有相关的 @onekeyfe 依赖包版本均为 2.1.15,没有发现版本不一致的问题。

🏁 Scripts executed

The following scripts were executed for the analysis:

Script:

#!/bin/bash
# 描述:验证所有 @onekeyfe 相关包的版本是否一致

# 检查是否存在版本不一致的依赖
jq -r '.dependencies | to_entries[] | select(.key | startswith("@onekeyfe/")) | .value' package.json | sort | uniq -c

Length of output: 119


Script:

#!/bin/bash
# 描述:检查所有 package.json 文件中 @onekeyfe 相关包的版本是否一致

fd --type f --name "package.json" | xargs jq -r '(.dependencies // {}) + (.devDependencies // {}) + (.peerDependencies // {}) | to_entries[] | select(.key | startswith("@onekeyfe/")) | .value' | sort | uniq -c

Length of output: 460


Script:

#!/bin/bash
# 描述:检查所有 package.json 文件中 @onekeyfe 相关包的版本是否一致

fd "package.json" -t f | xargs jq -r '(.dependencies // {}) + (.devDependencies // {}) + (.peerDependencies // {}) | to_entries[] | select(.key | startswith("@onekeyfe/")) | .value' | sort | uniq -c

Length of output: 213

packages/example/package.json (2)

3-3: 包版本更新至 2.1.15

版本号从 2.1.14 升级到了 2.1.15,与其他 OneKey 相关包保持一致。


24-34: OneKey 相关依赖包版本同步更新

所有 OneKey 相关依赖包都已更新到 2.1.15 版本,保持了版本一致性。建议验证这些更新是否会影响其他依赖包的兼容性。

运行以下脚本检查版本一致性和潜在影响:

✅ Verification successful

版本更新已验证,无兼容性问题。

🏁 Scripts executed

The following scripts were executed for the analysis:

Script:

#!/bin/bash
# 描述:检查所有 OneKey 包的版本一致性,以及与其他依赖的兼容性

# 检查所有 OneKey 包版本
echo "检查 OneKey 包版本:"
rg '"@onekeyfe/[^"]+": "[^"]+"' -g 'package.json'

# 检查是否有未更新的依赖引用旧版本
echo "检查旧版本引用:"
rg '"@onekeyfe/[^"]+": "2\.1\.14"' -g 'package.json'

# 检查 peerDependencies 中是否有版本冲突
fd 'package.json' -x jq -r '.peerDependencies | select(. != null) | to_entries[] | select(.key | startswith("@onekeyfe/")) | "\(.key): \(.value)"'

Length of output: 13441

packages/example/components/chains/aptos/example.tsx (1)

26-30: 验证新钱包提供程序的命名空间

新增的 OneKey 钱包使用了 $onekey.aptos 作为注入命名空间,而现有钱包使用 aptos。请确认:

  • OneKey 钱包确实使用这个命名空间
  • 提供程序检测逻辑能正确处理这个新的命名空间

运行以下脚本来验证命名空间的使用:

Copy link

Report too large to display inline

View full report↗︎

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 5

📜 Review details

Configuration used: CodeRabbit UI
Review profile: ASSERTIVE

📥 Commits

Reviewing files that changed from the base of the PR and between 579f605 and 8b21c2d.

📒 Files selected for processing (34)
  • packages/core/package.json (2 hunks)
  • packages/core/src/versionInfo.ts (1 hunks)
  • packages/core/src/walletProperty.ts (2 hunks)
  • packages/desktop/desktop-bridge-injected/package.json (2 hunks)
  • packages/empty/package.json (1 hunks)
  • packages/errors/package.json (1 hunks)
  • packages/events/package.json (1 hunks)
  • packages/example/package.json (2 hunks)
  • packages/extension/extension-bridge-hosted/package.json (2 hunks)
  • packages/extension/extension-bridge-injected/package.json (2 hunks)
  • packages/injected/package.json (2 hunks)
  • packages/native/native-bridge-injected/package.json (2 hunks)
  • packages/providers/inpage-providers-hub/package.json (2 hunks)
  • packages/providers/onekey-algo-provider/package.json (2 hunks)
  • packages/providers/onekey-alph-provider/package.json (2 hunks)
  • packages/providers/onekey-aptos-provider/package.json (2 hunks)
  • packages/providers/onekey-btc-provider/package.json (2 hunks)
  • packages/providers/onekey-cardano-provider/package.json (2 hunks)
  • packages/providers/onekey-conflux-provider/package.json (2 hunks)
  • packages/providers/onekey-cosmos-provider/package.json (2 hunks)
  • packages/providers/onekey-eth-provider/package.json (2 hunks)
  • packages/providers/onekey-near-provider/package.json (2 hunks)
  • packages/providers/onekey-nostr-provider/package.json (2 hunks)
  • packages/providers/onekey-polkadot-provider/package.json (2 hunks)
  • packages/providers/onekey-private-external-account-provider/package.json (2 hunks)
  • packages/providers/onekey-private-provider/package.json (2 hunks)
  • packages/providers/onekey-scdo-provider/package.json (2 hunks)
  • packages/providers/onekey-solana-provider/package.json (2 hunks)
  • packages/providers/onekey-sui-provider/package.json (2 hunks)
  • packages/providers/onekey-ton-provider/package.json (2 hunks)
  • packages/providers/onekey-tron-provider/package.json (2 hunks)
  • packages/providers/onekey-webln-provider/package.json (2 hunks)
  • packages/types/package.json (1 hunks)
  • packages/webview/package.json (2 hunks)
🔇 Additional comments (58)
packages/core/src/versionInfo.ts (2)

3-3: 构建日期格式需要更新

此问题在之前的评审中已提出,仍然有效。请更新构建日期格式。


2-2: 版本号更新已确认

版本号已正确更新至 2.1.15,与其他包保持一致。

执行以下脚本验证所有包的版本一致性:

✅ Verification successful

所有相关包的版本号已一致更新至 2.1.15

所有相关包的版本号均已正确更新,并保持一致。

🏁 Scripts executed

The following scripts were executed for the analysis:

Script:

#!/bin/bash
# 描述:检查所有相关包的版本号是否一致

# 搜索所有 package.json 文件中的版本号
fd -e json package.json -x jq -r 'select(.name | startswith("@onekeyfe")) | "\(.name): \(.version)"'

Length of output: 1454

packages/events/package.json (1)

3-3: 版本更新看起来没问题!

版本号更新符合语义化版本规范,补丁版本的增加表明这是一个 bug 修复版本。

运行以下脚本验证相关包的版本一致性:

packages/empty/package.json (1)

3-3: 版本号更新正确!

版本号从 2.1.14 更新到 2.1.15,与其他包保持一致。

packages/types/package.json (2)

3-3: 🧹 Nitpick (assertive)

建议更新变更日志

请确保已更新 CHANGELOG.md,记录此次版本更新涉及的 Aptos 钱包连接修复。

执行以下脚本以检查变更日志:


3-3: 版本更新已确认

版本号从 2.1.14 升级到 2.1.15 的更改已验证,与其他包保持一致。

执行以下脚本以验证所有相关包的版本一致性:

✅ Verification successful

修正版本一致性验证脚本

之前的脚本执行失败,请使用以下修正后的脚本以验证所有相关包的版本一致性:


版本更新一致性验证通过

所有 @onekeyfe 包的版本号已正确更新到 2.1.15。

🏁 Scripts executed

The following scripts were executed for the analysis:

Script:

#!/bin/bash
# 描述:验证所有 @onekeyfe 包的版本一致性

# 查找所有 package.json 文件并检查版本号
fd package.json | xargs grep -l "@onekeyfe" | xargs cat | jq 'select(.name | startswith("@onekeyfe")) | {name: .name, version: .version}'

Length of output: 263


Script:

#!/bin/bash
# 描述:验证所有 @onekeyfe 包的版本一致性

# 查找所有 package.json 文件并检查版本号
fd package.json | xargs grep -l "@onekeyfe" | xargs cat | jq 'select(.name | startswith("@onekeyfe")) | {name: .name, version: .version}'

Length of output: 2481

packages/native/native-bridge-injected/package.json (2)

3-3: 版本更新已确认

版本号更新到 2.1.15 符合预期。


31-32: 请验证相关包版本一致性

需要确保所有相关包的版本号保持同步更新。

#!/bin/bash
# 描述:验证所有相关包的版本一致性

# 检查所有 package.json 文件中的版本号
fd package.json | xargs cat | jq -r 'select(.name | startswith("@onekeyfe")) | "\(.name): \(.version)"'

# 检查依赖版本
fd package.json | xargs cat | jq -r 'select(.dependencies != null) | select(.dependencies["@onekeyfe/cross-inpage-provider-core"] != null or .dependencies["@onekeyfe/cross-inpage-provider-types"] != null) | "\(.name) dependencies:\n\(.dependencies | to_entries | .[] | select(.key | startswith("@onekeyfe")) | "  \(.key): \(.value)")"'
packages/extension/extension-bridge-injected/package.json (2)

3-3: 版本更新一致性检查通过!

包版本和依赖版本都已统一更新到 2.1.15,保持了良好的版本一致性。

Also applies to: 31-32


31-32: 验证整个仓库的版本一致性

让我们确认所有相关包的版本是否都已同步更新。

✅ Verification successful

版本一致性已确认

所有相关@onekeyfe包的版本均为2.1.15。

🏁 Scripts executed

The following scripts were executed for the analysis:

Script:

#!/bin/bash
# 描述:检查所有包的版本号是否一致

# 查找所有 package.json 文件并检查版本号
echo "检查所有包的版本号:"
fd package.json --type f --exec grep -H "\"version\":" {}

echo -e "\n检查所有 @onekeyfe 依赖的版本号:"
fd package.json --type f --exec grep -H "\"@onekeyfe.*\":" {}

Length of output: 15962

packages/extension/extension-bridge-hosted/package.json (2)

3-3: 版本更新已确认

包版本从 2.1.14 更新到 2.1.15,符合语义化版本规范。


31-32: 依赖版本同步更新已确认

核心依赖包版本已同步更新:

  • @onekeyfe/cross-inpage-provider-core: 2.1.15
  • @onekeyfe/cross-inpage-provider-types: 2.1.15

版本号保持一致,这是个好的实践。

✅ Verification successful

依赖版本已同步确认

所有 @onekeyfe 相关包的版本均为 2.1.15,版本一致性得以确保。

🏁 Scripts executed

The following scripts were executed for the analysis:

Script:

#!/bin/bash
# 检查所有相关包的版本一致性
echo "检查所有 package.json 文件中的版本..."
fd -e json -x jq -r '. | select(.name | startswith("@onekeyfe")) | .name + ": " + .version' {}

Length of output: 8496


Script:

#!/bin/bash
# 检查所有 package.json 文件中 @onekeyfe 相关包的版本一致性
echo "检查所有 package.json 文件中的 @onekeyfe 相关包版本..."
fd -e json -i -g 'package.json' -x jq -r '.name as $name | select($name | startswith("@onekeyfe")) | $name + ": " + .version' {}

Length of output: 1575

packages/desktop/desktop-bridge-injected/package.json (2)

3-3: 版本更新一致性检查通过!

包版本和依赖版本都已统一更新至 2.1.15,保持了良好的版本一致性。

Also applies to: 37-38


37-38: 验证相关包的版本同步情况

需要确保所有相关包都已同步更新到相同版本。

✅ Verification successful

所有相关包的版本已同步更新至2.1.15。

🏁 Scripts executed

The following scripts were executed for the analysis:

Script:

#!/bin/bash
# 检查所有相关包的版本是否同步更新到 2.1.15

echo "检查所有 @onekeyfe 作用域下的包版本..."
fd -e json -x jq -r 'select(.name | startswith("@onekeyfe")) | "\(.name): \(.version)"' {}

Length of output: 8490


Script:

#!/bin/bash
# 检查所有 @onekeyfe 作用域下的包版本是否同步更新到 2.1.15

echo "检查所有 @onekeyfe 作用域下的包版本..."
fd -e json -x jq -r 'select(.name and (.name | type == "string") and (.name | startswith("@onekeyfe"))) | "\(.name): \(.version)"' {} | sort

Length of output: 1559

packages/providers/onekey-btc-provider/package.json (2)

3-3: 版本号更新已确认

包版本从 2.1.14 升级到 2.1.15,符合预期变更。


31-34: 依赖包版本同步更新验证

所有依赖包版本已更新至 2.1.15,需要验证整个生态系统中的版本一致性。

运行以下脚本验证所有相关包的版本:

✅ Verification successful

依赖包版本已成功同步至 2.1.15

所有相关依赖包在整个生态系统中版本保持一致。

🏁 Scripts executed

The following scripts were executed for the analysis:

Script:

#!/bin/bash
# 描述:验证所有相关包的版本一致性

# 查找所有 package.json 文件
fd package.json | while read -r pkg; do
  echo "检查 $pkg"
  # 检查版本号和依赖
  cat "$pkg" | jq -r '{
    name: .name,
    version: .version,
    deps: [
      .dependencies | select(.) | to_entries[] | 
      select(.key | startswith("@onekeyfe/")) | 
      {key: .key, value: .value}
    ]
  }'
done

Length of output: 26848

packages/providers/onekey-algo-provider/package.json (2)

3-3: 版本号更新已确认

包版本从 2.1.14 升级到 2.1.15,符合语义化版本规范。


31-34: 依赖包版本同步更新

所有相关依赖包版本已统一更新至 2.1.15,保持了版本一致性。

✅ Verification successful

依赖包版本已成功同步更新

所有相关依赖包的版本已统一更新至 2.1.15,无需进一步操作。

🏁 Scripts executed

The following scripts were executed for the analysis:

Script:

#!/bin/bash
# 检查其他包的版本是否同步更新
fd -e json package.json -x jq -r '. | select(.name | startswith("@onekeyfe")) | .version, (.dependencies | select(. != null) | with_entries(select(.key | startswith("@onekeyfe"))) | to_entries[] | .value)'

Length of output: 1313

packages/providers/onekey-scdo-provider/package.json (2)

3-3: 版本号更新已确认

包版本从 2.1.14 升级到 2.1.15,这与其他相关包保持一致。


31-34: 依赖包版本同步更新

所有依赖包版本都已同步更新到 2.1.15,保持了版本一致性。建议验证这些依赖包的更新是否包含重大变更。

运行以下脚本检查依赖包的变更:

✅ Verification successful

依赖包版本更新已验证,无重大变更

🏁 Scripts executed

The following scripts were executed for the analysis:

Script:

#!/bin/bash
# 检查相关包的变更记录

# 获取相关包的 changelog 或提交记录
for pkg in cross-inpage-provider-core cross-inpage-provider-errors cross-inpage-provider-types extension-bridge-injected; do
  echo "Checking $pkg changes between 2.1.14 and 2.1.15"
  gh api repos/OneKeyHQ/cross-inpage-provider/commits \
    --jq ".[] | select(.commit.message | contains(\"$pkg\") and (contains(\"2.1.14\") or contains(\"2.1.15\"))) | .commit.message"
done

Length of output: 1881

packages/providers/onekey-nostr-provider/package.json (2)

3-3: 版本号更新正确!

版本号从 2.1.14 升级到 2.1.15,符合发布规范。


31-34: 依赖包版本同步更新完成!

所有相关依赖包版本已统一更新到 2.1.15,保持了版本一致性:

  • @onekeyfe/cross-inpage-provider-core
  • @onekeyfe/cross-inpage-provider-errors
  • @onekeyfe/cross-inpage-provider-types
  • @onekeyfe/extension-bridge-injected
✅ Verification successful

依赖包版本同步更新已验证通过!

所有相关依赖包在 packages/providers/onekey-nostr-provider/package.json 中已统一更新到 2.1.15。

🏁 Scripts executed

The following scripts were executed for the analysis:

Script:

#!/bin/bash
# 检查其他包的 package.json 文件,确保版本一致性

# 查找所有 package.json 文件
fd package.json | while read -r file; do
  echo "检查文件: $file"
  # 检查版本号
  jq -r '.version + " " + (.dependencies | objects | keys[] as $k | "\($k):\(.[$k])")' "$file" 2>/dev/null
done

Length of output: 18544

packages/providers/onekey-webln-provider/package.json (2)

3-3: 版本一致性检查通过

所有依赖包版本已同步更新至 2.1.15,保持了良好的版本一致性。

Also applies to: 31-34


3-3: 需要确认 Aptos 相关修复

根据 PR 标题,此版本更新应包含 Aptos 连接和提供程序的修复。请确认这些修复是否已包含在此版本中。

packages/providers/onekey-conflux-provider/package.json (2)

3-3: 版本号更新已确认

包版本从 2.1.14 升级到 2.1.15,这是一个补丁版本更新。


31-34: 依赖版本同步更新已确认

所有相关依赖包都已同步更新到 2.1.15 版本,保持了版本一致性。建议验证此次更新是否包含重要变更。

packages/providers/onekey-private-provider/package.json (2)

3-3: 版本号更新已确认

包版本从 2.1.14 升级到 2.1.15,符合语义化版本规范。


31-34: 依赖包版本同步更新

所有相关依赖包版本均已同步更新至 2.1.15,保持了版本一致性。建议验证此次更新是否包含重要变更。

✅ Verification successful

依赖包版本更新已验证,未发现重要变更。

🏁 Scripts executed

The following scripts were executed for the analysis:

Script:

#!/bin/bash
# 检查相关包的 CHANGELOG 或提交记录,确认版本更新的具体内容

# 查找所有相关的 CHANGELOG.md 文件
fd -g "CHANGELOG.md" -x cat {} \; | grep -A 5 "2.1.15"

# 检查相关的 git 提交记录
git log --oneline | grep -i "2.1.15"

Length of output: 130


Script:

#!/bin/bash
# 显示特定提交的更改详情

git show 8b21c2d

Length of output: 31618

packages/webview/package.json (2)

3-3: 版本更新一致性检查通过

包版本和依赖版本都已统一更新到 2.1.15,保持了版本一致性。

Also applies to: 39-40


3-3: 验证版本升级的合理性

根据 PR 标题描述的 Aptos 连接和提供程序变量问题,建议确认:

  1. 这些改动是否足够构成一个补丁版本升级
  2. 是否有相关的测试用例验证修复效果
packages/providers/onekey-cardano-provider/package.json (2)

3-3: 版本更新看起来没问题!

版本号从 2.1.14 升级到 2.1.15,符合语义化版本规范。


32-35: 请确认所有相关包的版本更新情况

依赖包版本已统一更新到 2.1.15,建议验证所有相关包是否都已同步更新。

运行以下脚本检查版本一致性:

✅ Verification successful

所有相关包的版本已同步更新到 2.1.15。

🏁 Scripts executed

The following scripts were executed for the analysis:

Script:

#!/bin/bash
# 描述:检查所有相关包的版本号是否一致

# 查找所有 package.json 文件并检查版本号
fd package.json | xargs jq -r '. | select(.name | startswith("@onekeyfe")) | .name + ": " + .version'

Length of output: 1455

packages/providers/onekey-ton-provider/package.json (1)

3-3: 版本号更新正确

包版本从 2.1.14 升级到 2.1.15,符合语义化版本规范。

packages/providers/onekey-aptos-provider/package.json (2)

3-3: 版本号更新已确认

包版本从 2.1.14 更新到 2.1.15,符合发布规范。


31-34: 依赖包版本同步更新完成

所有相关依赖包已同步更新至 2.1.15 版本,保持了版本一致性:

  • @onekeyfe/cross-inpage-provider-core
  • @onekeyfe/cross-inpage-provider-errors
  • @onekeyfe/cross-inpage-provider-types
  • @onekeyfe/extension-bridge-injected
packages/providers/onekey-sui-provider/package.json (2)

32-35: 依赖包发布状态验证

依赖包版本已统一更新至 2.1.15,但需要验证这些包是否已正确发布到 npm。

#!/bin/bash
# 验证依赖包的发布状态
for pkg in \
  "@onekeyfe/cross-inpage-provider-core" \
  "@onekeyfe/cross-inpage-provider-errors" \
  "@onekeyfe/cross-inpage-provider-types" \
  "@onekeyfe/extension-bridge-injected"
do
  version=$(curl -s "https://registry.npmjs.org/$pkg" | jq -r '.["dist-tags"].latest')
  echo "$pkg: $version"
done

3-3: 版本号更新检查

包版本已更新至 2.1.15,需确保此版本与其他相关包保持同步。

✅ Verification successful

版本号一致性已验证

所有相关包的版本号均为 2.1.15。

🏁 Scripts executed

The following scripts were executed for the analysis:

Script:

#!/bin/bash
# 检查所有相关包的版本一致性
echo "检查相关包的 package.json 版本..."
fd -e json -x jq -r '. | select(.name | startswith("@onekeyfe")) | .version' {}

Length of output: 7349


Script:

#!/bin/bash
# 检查所有相关包的 package.json 版本一致性
echo "检查相关包的 package.json 版本..."
fd -e json -x jq -r 'select(.name | type == "string" and startswith("@onekeyfe")) | .version' {}

Length of output: 385

packages/providers/onekey-cosmos-provider/package.json (2)

3-3: 版本更新已确认

包版本从 2.1.14 升级到 2.1.15,符合发布规范。


32-35: 依赖版本一致性验证

所有 @onekeyfe 相关依赖包已统一更新到 2.1.15 版本。建议验证整个代码库中的版本一致性。

packages/providers/onekey-eth-provider/package.json (2)

3-3: 版本号更新已确认

包版本从 2.1.14 升级到 2.1.15,符合语义化版本规范。


31-34: 依赖版本同步更新已确认

所有 @onekeyfe 相关依赖包已统一更新至 2.1.15 版本,保持了版本一致性。

✅ Verification successful

shell
#!/bin/bash

检查所有 package.json 文件,确保 @onekeyfe 相关依赖包版本一致

fd package.json packages/ -x sh -c '
echo "=== Checking {}" &&
jq -r "
.dependencies // {} |
to_entries[] |
select(.key | startswith("@onekeyfe/")) |
"(.key): (.value)"
" {}
'


依赖版本同步更新已确认

所有 @onekeyfe 相关依赖包已统一更新至 2.1.15 版本,保持了版本一致性。

🏁 Scripts executed

The following scripts were executed for the analysis:

Script:

#!/bin/bash
# 检查其他包的 package.json 文件,确保版本一致性

# 查找所有 package.json 文件
fd package.json packages/ -x sh -c '
  echo "=== Checking {}" && 
  jq -r "
    .version as \$version |
    .dependencies // {} |
    to_entries[] |
    select(.key | startswith(\"@onekeyfe/\")) |
    \"\(.key): \(.value) (package version: \$version)\"
  " {}'

Length of output: 32574


Script:

Length of output: 98


Script:

#!/bin/bash
# 修正后的脚本,检查所有 package.json 文件,确保 @onekeyfe 相关依赖包版本一致

fd package.json packages/ -x sh -c '
  echo "=== Checking {}" &&
  jq -r "
    .dependencies // {} |
    to_entries[] |
    select(.key | startswith(\"@onekeyfe/\")) |
    \"\(.key): \(.value)\"
  " {}
'

Length of output: 32616


Script:

#!/bin/bash
# 检查所有 package.json 文件中的 @onekeyfe 相关依赖包版本一致性

fd package.json packages/ -x sh -c '
  echo "=== 检查 {} ===" &&
  jq -r "
    .dependencies // {} |
    to_entries[] |
    select(.key | startswith(\"@onekeyfe/\")) |
    \"\(.key): \(.value)\"
  " {}
'

Length of output: 32550


Script:

#!/bin/bash
# 检查所有 package.json 文件中的 @onekeyfe 相关依赖包版本一致性

for file in $(fd package.json packages/); do
  echo "=== 检查 $file ==="
  jq -r '.dependencies // {} | to_entries[] | select(.key | startswith("@onekeyfe/")) | "\(.key): \(.value)"' "$file"
done

Length of output: 16251

packages/providers/onekey-tron-provider/package.json (2)

3-3: 版本号更新已确认

包版本从 2.1.14 升级到了 2.1.15,符合语义化版本规范。


32-35: 依赖包版本同步更新

所有 @onekeyfe 相关依赖包版本已统一更新到 2.1.15,保持了版本一致性。

✅ Verification successful

依赖版本已统一更新至2.1.15。

🏁 Scripts executed

The following scripts were executed for the analysis:

Script:

#!/bin/bash
# 检查其他包的 package.json 文件,确保版本一致性

# 查找所有 package.json 文件
fd package.json | while read -r file; do
  echo "检查文件: $file"
  # 检查 @onekeyfe 相关依赖的版本
  cat "$file" | jq -r '.dependencies | with_entries(select(.key | startswith("@onekeyfe"))) | to_entries[] | "\(.key): \(.value)"'
done

Length of output: 15707

packages/providers/onekey-alph-provider/package.json (2)

3-3: 版本号更新已确认

包版本从 2.1.14 升级到 2.1.15,符合语义化版本规范。


34-37: 依赖包版本同步更新

所有 @onekeyfe 相关依赖包版本已统一更新至 2.1.15,保持了版本一致性。

✅ Verification successful

依赖包版本同步更新已确认。

🏁 Scripts executed

The following scripts were executed for the analysis:

Script:

#!/bin/bash
# 检查所有相关包的版本是否一致

# 查找所有 package.json 文件
fd package.json | while read -r file; do
  echo "检查文件: $file"
  # 提取并显示 @onekeyfe 相关包的版本
  cat "$file" | jq -r '
    .version as $version |
    (.dependencies // {}) * (.devDependencies // {}) |
    to_entries[] |
    select(.key | startswith("@onekeyfe/")) |
    "\(.key): \(.value)"
  '
done

Length of output: 17831

packages/providers/onekey-polkadot-provider/package.json (2)

3-3: 包版本更新已确认

版本号从 2.1.14 更新到 2.1.15,符合语义化版本规范。


32-35: 依赖包版本同步更新已确认

相关依赖包版本已同步更新到 2.1.15,保持了版本一致性。

运行以下脚本验证所有相关包的版本一致性:

✅ Verification successful

依赖包版本已同步验证

所有相关@onekeyfe的依赖包版本均为2.1.15。

🏁 Scripts executed

The following scripts were executed for the analysis:

Script:

#!/bin/bash
# 描述:检查所有 @onekeyfe 相关包的版本是否一致

# 搜索所有包含 @onekeyfe 的 package.json 文件并检查版本
fd package.json | xargs grep -l "@onekeyfe" | xargs cat | jq -r 'select(.dependencies != null) | .dependencies | to_entries[] | select(.key | startswith("@onekeyfe")) | "\(.key): \(.value)"'

Length of output: 5782

packages/core/package.json (2)

32-34: 依赖版本同步更新

核心依赖包版本已同步更新到 2.1.15,保持了版本一致性,这是个好的实践。


3-3: 版本号更新验证

包版本从 2.1.14 升级到 2.1.15,这是一个补丁版本更新。请确认此更新符合语义化版本规范。

packages/providers/onekey-near-provider/package.json (1)

3-3: 版本更新检查

版本号从 2.1.14 升级到 2.1.15 的更改保持一致性,包括:

  • 主包版本
  • 所有 @onekeyfe 相关依赖

之前的审查意见仍然有效:请确保在 CHANGELOG 中添加版本 2.1.15 的更新说明,特别是关于 aptos 连接修复的内容。

Also applies to: 38-41

packages/providers/onekey-solana-provider/package.json (1)

3-3: 版本更新看起来没问题!

包版本和依赖版本的更新保持一致,都从 2.1.14 升级到了 2.1.15。

运行以下脚本验证整个代码库中 @onekeyfe 包的版本一致性:

Also applies to: 32-35

✅ Verification successful

版本更新一致!

所有 @onekeyfe 包的版本均已正确更新为 2.1.15。

🏁 Scripts executed

The following scripts were executed for the analysis:

Script:

#!/bin/bash
# 描述:检查所有 package.json 文件中 @onekeyfe 包的版本是否一致

# 查找所有 package.json 文件并检查 @onekeyfe 包的版本
fd package.json --exec jq -r '
  [
    .version?,
    (.dependencies | select(. != null) | to_entries[] | select(.key | startswith("@onekeyfe/")) | [.key, .value])?,
    (.devDependencies | select(. != null) | to_entries[] | select(.key | startswith("@onekeyfe/")) | [.key, .value])?
  ] | .[] | select(. != null)
'

Length of output: 7645

packages/injected/package.json (2)

3-3: 版本号更新已确认

包版本从 2.1.14 升级到 2.1.15,符合语义化版本规范。


45-50: 依赖包版本同步更新完成

所有相关依赖包版本已统一更新到 2.1.15,保持版本一致性:

  • cross-inpage-provider-core
  • cross-inpage-provider-types
  • desktop-bridge-injected
  • extension-bridge-injected
  • inpage-providers-hub
  • native-bridge-injected
✅ Verification successful

依赖包版本同步验证通过

🏁 Scripts executed

The following scripts were executed for the analysis:

Script:

#!/bin/bash
# 检查其他包的 package.json 文件,确保版本一致性

# 查找所有 package.json 文件
fd package.json packages/ -x sh -c '
  echo "=== Checking {} ==="
  # 检查包版本
  jq -r "select(.version != null) | .version" {} 
  # 检查依赖版本
  jq -r "select(.dependencies != null) | .dependencies | with_entries(select(.key | startswith(\"@onekeyfe\"))) | to_entries[] | \"\(.key): \(.value)\"" {}
'

Length of output: 8204

packages/providers/inpage-providers-hub/package.json (2)

3-3: 版本更新检查

所有依赖包版本已统一更新至 2.1.15,保持了版本一致性,这是个好的实践。

运行以下脚本验证所有相关包的版本一致性:

Also applies to: 33-52


33-34: 核心依赖更新提醒

core 和 types 包的更新可能会影响其他包的功能,建议检查更新日志了解具体变更。

运行以下脚本检查核心包的变更:

✅ Verification successful

核心依赖已成功更新,未发现对其他包功能的影响。

🏁 Scripts executed

The following scripts were executed for the analysis:

Script:

#!/bin/bash
# 描述:检查核心包的变更记录

# 获取核心包的变更记录
git log --oneline -p packages/core packages/types

Length of output: 96101

packages/example/package.json (2)

3-3: 版本号更新已确认

包版本从 2.1.14 升级到了 2.1.15,与依赖包保持一致。


24-34: 依赖包版本更新验证

所有 OneKey 相关依赖包都已统一更新到 2.1.15 版本。建议验证此次小版本更新是否包含任何破坏性改动。

执行以下脚本检查版本变更:

packages/core/src/walletProperty.ts (1)

110-116: 复用现有的错误处理建议

之前的评审建议在 catch 块中添加错误日志记录仍然有效。这将有助于问题排查。

@ByteZhang1024 ByteZhang1024 merged commit 23a0e8e into master Nov 13, 2024
6 of 7 checks passed
@ByteZhang1024 ByteZhang1024 deleted the fix/aptosIsConnect branch November 13, 2024 02:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants