Update to chia-blockchain 2.7.0 - #462
Conversation
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
|
Warning Review the following alerts detected in dependencies. According to your organization's Security Policy, it is recommended to resolve "Warn" alerts. Learn more about Socket for GitHub.
|
There was a problem hiding this comment.
Pull request overview
Updates this service’s Chia dependencies and adapts wallet/full-node RPC integrations to the chia-blockchain 2.7.0 request/response types.
Changes:
- Bump
chia-blockchainfrom 2.5.6 to 2.7.0 (and update the lockfile accordingly). - Migrate several wallet RPC calls to typed request objects (e.g.,
GetWallets,GetTransactions,SelectCoins,CreateSignedTransaction). - Update CI test-cache version used by the GitHub Actions workflow.
Reviewed changes
Copilot reviewed 7 out of 8 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
pyproject.toml |
Bumps chia-blockchain dependency to 2.7.0. |
poetry.lock |
Updates resolved dependency graph for chia-blockchain 2.7.0 (incl. Python >=3.10 constraints and new deps). |
app/crud/chia.py |
Switches CoinRecord import to chia_rs to align with updated chia types. |
app/core/utils.py |
Adds puzzle-hash-to-address helper and updates wallet RPC calls to typed request objects. |
app/core/climate_wallet/wallet.py |
Updates wallet RPC calls (select coins, memos handling) and sets to_address for detokenization tx creation. |
app/api/v1/transactions.py |
Updates transactions endpoints to typed RPC request/response objects. |
app/api/v1/keys.py |
Updates GetPrivateKey call to use named parameter. |
.github/workflows/tests.yaml |
Updates BLOCKS_AND_PLOTS_VERSION used for cached test data. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| wallet_objs: list[ChiaJsonObject] = await wallet_rpc_client.get_wallets( | ||
| wallet_type=WalletType.CAT, | ||
| wallets_response = await wallet_rpc_client.get_wallets( | ||
| GetWallets(type=uint16(WalletType.CAT)), |
There was a problem hiding this comment.
GetWallets(type=uint16(WalletType.CAT)) is inconsistent with the rest of the codebase which treats WalletType as an enum and uses .value when an int is required (e.g., wallet_info.type != WalletType.CAT.value). Passing the enum instance here may fail if WalletType is not an IntEnum. Use the numeric value consistently (e.g., uint16(WalletType.CAT.value)), or cast explicitly with int(WalletType.CAT) if that’s the intended contract.
| GetWallets(type=uint16(WalletType.CAT)), | |
| GetWallets(type=uint16(WalletType.CAT.value)), |
Update to chia-blockchain 2.7.0