feat(protocol): expose the NU7 consensus branch - #474
Merged
ValarDragon merged 4 commits intoSep 23, 2026
Merged
ValarDragon merged 4 commits into
ValarDragon merged 4 commits into
Conversation
p0mvn
force-pushed
the
feat/nu7-protocol-activation
branch
from
September 21, 2026 11:15
5e39b52 to
ce40dbd
Compare
This was referenced Sep 21, 2026
p0mvn
force-pushed
the
feat/nu7-protocol-activation
branch
from
September 21, 2026 11:25
ce40dbd to
1929e9f
Compare
Adapted from valargroup/librustzcash#76. Co-authored-by: Evan Forbes <evan.samuel.forbes@gmail.com>
p0mvn
force-pushed
the
feat/nu7-protocol-activation
branch
from
September 21, 2026 11:27
1929e9f to
659cfb5
Compare
ValarDragon
reviewed
Sep 22, 2026
p0mvn
marked this pull request as ready for review
September 23, 2026 06:22
|
Note Complete: Audit complete. V12 did not find any issues that need review. Open the full results here. Analyzed four files, diff |
ValarDragon
merged commit Sep 23, 2026
b8bcca5
into
agent/import-zakura-protocol-20260921
51 checks passed
ValarDragon
added a commit
that referenced
this pull request
Sep 23, 2026
* Import zcash_protocol 0.10.5 into common * Record zakura-protocol fork provenance * Add changelog for zakura-protocol import * Fix imported package policy checks * Import address and transparent protocol crates * Fix imported transparent README whitespace * Import ZIP 321 payment requests * Fix imported ZIP 321 policy checks * Fix imported README lint findings * feat(protocol): expose the NU7 consensus branch (#474) * feat(protocol): expose NU7 consensus branch Adapted from valargroup/librustzcash#76. Co-authored-by: Evan Forbes <evan.samuel.forbes@gmail.com> * Add changelog for NU7 protocol activation * fix(primitives): satisfy clippy for NU7 branch matching * Apply suggestion from @ValarDragon --------- Co-authored-by: Evan Forbes <evan.samuel.forbes@gmail.com> Co-authored-by: Dev Ojha <ValarDragon@users.noreply.github.com> --------- Co-authored-by: Evan Forbes <evan.samuel.forbes@gmail.com> Co-authored-by: Dev Ojha <ValarDragon@users.noreply.github.com>
Closed
ebfull
added a commit
that referenced
this pull request
Sep 23, 2026
Bump the workspace version and intra-workspace requirements from 1.3.0-alpha.1 to 1.3.0 and assemble the v1.3.0 changelog sections, consuming all 17 pending fragments and folding the 1.3.0-alpha.1 sections into the stable sections. Editorial changes to the assembled entries: - Record the #471 dependency swap in zakura-primitives and zakura-keys. - Link #492 from the #474 NU7 entry: the release ships the 0x77190AD9 consensus branch ID that the fragment documented. - Drop the stale `zcash_unstable="nu7"` qualifier from the #458 entry. - Remove internal-only wording from the #437 and #451 halo2_proofs entries. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
ebfull
added a commit
that referenced
this pull request
Sep 23, 2026
Bump the workspace version and intra-workspace requirements from 1.3.0-alpha.1 to 2.0.0 and assemble the v2.0.0 changelog sections, consuming all 17 pending fragments. The bump is major because #471 replaced the `zcash_protocol`, `zcash_transparent`, and `zcash_address` crates behind types in the public APIs of zakura-primitives and zakura-keys, and #458 removed the `zip-233` feature and its APIs. No stable 1.3.0 ships, so the published `1.3.0-alpha.1` sections belong to the 2.0.0 delta from 1.2.0: they were folded with `./scripts/changelog.py release v1.3.0` and the section renamed to 2.0.0. Editorial changes to the assembled entries: - Record the #471 dependency swap in zakura-primitives and zakura-keys. - Link #492 from the #474 NU7 entry: the release ships the 0x77190AD9 consensus branch ID that the fragment documented. - Drop the stale `zcash_unstable="nu7"` qualifier from the #458 entry. - Remove internal-only wording from the #437 and #451 halo2_proofs entries. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Merged
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
2 participants
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.
Summary
NetworkUpgrade::Nu7,BranchId::Nu7, andLocalNetwork::nu7in every build0x77190AD8while leaving Mainnet and Testnet activation heights unsetzcash_unstable="nu7"Motivation
This ports the applicable parts of valargroup/librustzcash#76 onto the Zakura-owned Common crates introduced by #471. Common already removed the obsolete experimental ZIP 233 field in #458, so this adaptation preserves that fork decision while carrying Evan's NU7 branch activation behavior.
Tests
cargo test --locked -p zakura-protocol -p zakura-primitives --all-features— 99 unit tests and one compile-fail doctest passedcargo fmt --all -- --checkgit diff --checkAttribution
Adapted from work by Evan Forbes in valargroup/librustzcash#76; the implementation commit retains Evan as a co-author.
AI assistance
Codex assisted with adapting the source changes to the current Common fork and validating them.