[DO NOT MERGE] Bump librustzcash pin to include ZeWIF keypool-exposure fix#653
Draft
nullcopy wants to merge 1 commit into
Draft
[DO NOT MERGE] Bump librustzcash pin to include ZeWIF keypool-exposure fix#653nullcopy wants to merge 1 commit into
nullcopy wants to merge 1 commit into
Conversation
nullcopy
marked this pull request as draft
July 18, 2026 03:19
nullcopy
force-pushed
the
fix/migrate-transparent-change-gap
branch
3 times, most recently
from
July 18, 2026 05:31
8b3a574 to
1990779
Compare
Pins the librustzcash stack, in the root and both backend workspaces, to the head of the ZeWIF importer fix branch (nullcopy/fix/zewif-import-keypool-exposure, the source of the upstream librustzcash PR). The importer no longer marks never-exposed transparent addresses as exposed: it was marking pre-generated change (internal-scope) keypool reserves exposed at the account birthday during a `migrate-zcashd-wallet` import, exhausting the small internal gap limit and permanently blocking transparent change-address reservation (`z_sendmany` fails with `ReachedGapLimit`). Fixes #637. Lockfiles regenerated with utils/sync-lockfiles.sh; lockstep holds across all three workspaces. The rev is hosted on a fork pending the upstream librustzcash PR; repoint the [patch.crates-io] entries in all three manifests to zcash/librustzcash once it merges. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_012Fi1HoNsLCHbubXyPtYTei
nullcopy
force-pushed
the
fix/migrate-transparent-change-gap
branch
from
July 18, 2026 11:52
1990779 to
a927938
Compare
nullcopy
marked this pull request as ready for review
July 18, 2026 11:54
nullcopy
changed the title
nullcopy
marked this pull request as draft
July 18, 2026 11:56
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
DO NOT MERGE YET - Need to update LRZ pin after zcash/librustzcash#2666 merges
Depends on zcash/librustzcash#2666
Updates librustzcash pin to pull in fixes to the ZeWIF importer.
Closes #637
COR-1588