Zero call sites since #18522 (Aug 2024) removed the last callers but
left the method. Deletes the Blockchain implementation, the
BlockchainInterface protocol declaration, and the test mock.
Co-authored-by: Cursor <cursoragent@cursor.com>
The current AddressManager serialization (serialize_bytes /
deserialize_bytes) shipped in #19467 (May 2025) and the peers file is
re-serialized routinely during operation, so any node run since then
holds the new format. This removes the old-format deserializer that was
kept for migration, and with it the quirk that every startup parsed the
peers file with the OLD format first and reached the current one only
via exception handler.
A node whose peers file predates May 2025 falls back to a fresh
AddressManager repopulated from DNS introducers - the same self-healing
path as a corrupt file.
Co-authored-by: Cursor <cursoragent@cursor.com>
* Remove config keys that nothing reads from initial-config.yaml
Eight keys in the default config template are read by no code. Dated by
when their readers were removed: ssh_filename (SSH UI deleted Mar 2020),
pool_share_threshold / ips_estimate / max_connection_time (Jan 2021),
nft_metadata_cache_path + _hash_length (reader reverted in #13660,
Oct 2022), peer_db_path / wallet_peers_path (migration removed in #17517,
Feb 2024; both were already comment-marked deprecated).
Template-only change: existing user configs are unaffected; new installs
stop inheriting dead keys. The GUI repo was searched for all eight keys
(GitHub code search): zero references.
Co-authored-by: Cursor <cursoragent@cursor.com>
* Reword vdf_clients comment after ips_estimate removal
Co-authored-by: Cursor <cursoragent@cursor.com>
---------
Co-authored-by: Cursor <cursoragent@cursor.com>
* Use a separate config for pooling information
* New PlotNFT drivers
* PlotNFT2 Wallet
* PlotNFT V2 RPCs and CLI
* Integrate v2 pooling protocol into farmer
* Comment by @cursor
* Fix test
* Comments by @cursor
* whoops
* Comments by @cursor
* Comments by @cursor
* Comments by @cursor
* Some tidying
* fix test for memo adjustment
* tweak additional memos again
* empirical testing with v1 pools
* Add expiration to `GetAuthRequest`
* Remove pyjwt dep
* Revert `get_login_link` to old behavior
* Add an extra derivation to auth key for v2
* unhardened
* Fix login link test
* Fix login link test again
* Fix network protocol data
* Comments by cursor
* Fix test
* Use correct pool url after redirect
* Fix test for coverage
* Upstream wallet fixes
* Add `REMARK` option to `launch`
* pre-commit
* chmod
* test coverage
* Add wallet name
* test coverage
* moar test coverage
* commentsby @cursor
* moar test coverage
* Add transfer endpoint
* Add melt endpoint
* Modify tests to start at cmdline
* fix custody architecture namespace
* Add test for cli parsing
* executable bit
* Add test for errors
* Add wallet deletion
* test coverage
* Comments by @matt-o-how
* Fix /GET farmer to omit the signature rather than None
* Robust-ness for out of order coin states
* Reorg safety for transfers/melts
* Comments by @cursor
* Reorganize test to have transfer/melt as main part
* Comments by @cursor
* Remove reorg testing
* Fix protocol file?
* diff minimization
* pre-commit
* Readd reorg tests
* Handle config removal during wallet deletion
* Comments by @cursor
* catch one level higher
* Add comment to transfer_plotnft
* Delete unused function
* [LABS-485] Extract coin_state level try/catch to its own function
* Comments by @cursor
* Move peer.closed check into exception handler
* Comments by @cursor
* [LABS-482] Introduce `WalletSyncScope`
* Comments by @cursor
* Fix pickling Tx records
* Comments by @cursor
* Fix pre-commit
* Remove executable bit
* Use `action_scope` in `save_trade`
* [LABS-162] Replace `Streamable` in `ActionScope` with `copy.deepcopy`
* Fix tests
* Remove serialization methods
* Comments by @cursor
* Sync scope outside db_wrapper
* Revert ordering of writer and sync_Scope
* extend the log_mempool configuration option to only log transactions that time out
* improve test
* parse log_mempool in full_node
* refactor spend_bundle_id computation
* add more test cases to test_log_mempool_mode_normalization
Source hash: 2371eec356
Remaining commits: 0
<!-- CURSOR_SUMMARY -->
---
> [!NOTE]
> **Low Risk**
> No substantive code changes appear in the diff; risk is limited to
merge/checkpoint process rather than altered runtime behavior.
>
> **Overview**
> This PR is a **checkpoint merge** of `release/2.7.4` into `main` at
source hash `2371eec356e61bc3e771eca06b2b7b7aa099e027`, with **0
remaining commits** on the release branch relative to that point.
>
> The supplied diff contains **no file changes**—only an empty addition
to `/dev/null`—so there is no new product code, config, or documentation
in this merge from the perspective of the diff snapshot. It aligns
branch pointers / release bookkeeping rather than landing additional
patches beyond what is already at that commit.
>
> <sup>Reviewed by [Cursor Bugbot](https://cursor.com/bugbot) for commit
58bbe9af0c. Bugbot is set up for automated
code reviews on this repo. Configure
[here](https://www.cursor.com/dashboard/bugbot).</sup>
<!-- /CURSOR_SUMMARY -->
Source hash: 119707aa8b
Remaining commits: 1
<!-- CURSOR_SUMMARY -->
---
> [!NOTE]
> **Medium Risk**
> Merging a full release branch into main can touch many areas at once;
risk depends on the commits on `release/2.7.4`, which are not visible in
this diff.
>
> **Overview**
> This PR is a **release checkpoint** that merges `release/2.7.4` into
`main` at source commit `119707aa8bd836b0ffea9ac3c752e68a5b5324f2`, with
**one commit** still listed as remaining on the release line.
>
> No file-level diff was included in the review payload (only an empty
`/dev/null` hunk), so the concrete code changes are whatever that
release branch contains relative to `main` at merge time—typically a
batched set of fixes and release-prep work for **2.7.4**, not a single
feature PR.
>
> <sup>Reviewed by [Cursor Bugbot](https://cursor.com/bugbot) for commit
deb85e9d38. Bugbot is set up for automated
code reviews on this repo. Configure
[here](https://www.cursor.com/dashboard/bugbot).</sup>
<!-- /CURSOR_SUMMARY -->
Source hash: 3d1b328b8a
Remaining commits: 2
<!-- CURSOR_SUMMARY -->
---
> [!NOTE]
> **Low Risk**
> No code changes appear in the provided diff; risk is limited to
merge/checkpoint bookkeeping unless the omitted commits touch critical
paths.
>
> **Overview**
> **Checkpoint PR** to bring `main` in line with `release/2.7.4` at
source commit `3d1b328` (per title/description). The supplied diff
contains **no file hunks**, so there is nothing concrete to review in
the patch itself.
>
> Treat this as a **release-branch sync or merge checkpoint** with **2
commits** still noted as remaining on the description—not as a feature
or fix PR with reviewable code deltas in this bundle.
>
> <sup>Reviewed by [Cursor Bugbot](https://cursor.com/bugbot) for commit
baaa8783a5. Bugbot is set up for automated
code reviews on this repo. Configure
[here](https://www.cursor.com/dashboard/bugbot).</sup>
<!-- /CURSOR_SUMMARY -->
* [LABS-485] Extract coin_state level try/catch to its own function
* Comments by @cursor
* Move peer.closed check into exception handler
* Comments by @cursor
CHIA-4405 Handle ConnectionError from send_bytes in test_large_message_disconnect_and_ban (#21333)
Handle ConnectionError from send_bytes in test_large_message_disconnect_and_ban.
<!-- CURSOR_SUMMARY -->
> [!NOTE]
> **Medium Risk**
> Refactor of wallet coin identification used during sync. Logic is
relocated rather than redesigned, but mistakes in wiring could mis-route
CAT/DID/NFT/clawback coins.
>
> **Overview**
> Moves coin-type identification out of `WalletStateManager` into the
wallets that own that logic. `determine_coin_type` still classifies
spends, then delegates to classmethod `identify` on CAT, DID, NFT,
PlotNFT, and VC wallets, plus `ClawbackManager.identify`.
>
> WSM drops `handle_cat` / `handle_did` / `handle_nft` /
`handle_clawback` / `handle_vc` and the PlotNFT matching block. Behavior
is intended to stay the same: auto-create/convert wallets, stray CAT
tracking, DID/NFT caps, clawback records. Clawback identification now
takes `add_interested_coin_ids` as a callback instead of calling WSM
directly.
>
> NFT auto-add tests now patch and call `NFTWallet.identify`.
>
> <sup>Reviewed by [Cursor Bugbot](https://cursor.com/bugbot) for commit
f488a6b870. Bugbot is set up for automated
code reviews on this repo. Configure
[here](https://www.cursor.com/dashboard/bugbot).</sup>
<!-- /CURSOR_SUMMARY -->
<!-- CURSOR_SUMMARY -->
> [!NOTE]
> **Low Risk**
> Governance-only change to review routing; no runtime or application
code is affected.
>
> **Overview**
> Removes the **CODEOWNERS** entry that required
`@Chia-Network/actions-reviewers` on all paths under `.github/**/*`.
>
> Workflow and GitHub config changes in `.github` will now follow only
the repo-wide `*` rule (`@Chia-Network/required-reviewers`), without an
extra mandatory actions team review.
>
> <sup>Reviewed by [Cursor Bugbot](https://cursor.com/bugbot) for commit
bbeb9da95a. Bugbot is set up for automated
code reviews on this repo. Configure
[here](https://www.cursor.com/dashboard/bugbot).</sup>
<!-- /CURSOR_SUMMARY -->
Source hash: ab5512dccf
Remaining commits: 0
<!-- CURSOR_SUMMARY -->
---
> [!NOTE]
> **Low Risk**
> The visible diff only updates the changelog; documented fixes touch
wallet GUI and offers but are typical release-integration scope for a
checkpoint merge.
>
> **Overview**
> **Release checkpoint** merging `release/2.7.3` into `main` and adding
the **2.7.3** (2026-07-16) section to `CHANGELOG.md`.
>
> **Changed:** Read-only WalletConnect calls `chia_getHeightInfo` and
`chia_getCoinRecordsByNames` no longer require confirmation prompts.
>
> **Fixed:** Linux/Wayland wallet GUI confirmation dialogs when
`ready-to-show` never fires; Plot NFT key assignment via `launcherId`
hex normalization; reference wallet log viewer (Electron main-process
imports); NFT sell offer confirmation (“Asset type is not valid”).
>
> <sup>Reviewed by [Cursor Bugbot](https://cursor.com/bugbot) for commit
6e8d74ff50. Bugbot is set up for automated
code reviews on this repo. Configure
[here](https://www.cursor.com/dashboard/bugbot).</sup>
<!-- /CURSOR_SUMMARY -->
dependabot[bot]
Matt Hauff
Richard KissandCursor
ChiaAutomation
Arvid Norberg
Earle Lowe
Amine Khaldi
ibutterbot
