Skip to content

chore: bump miden-protocol dependencies to 0.17.0-rc.4 - #2617

Merged
Mirko-von-Leipzig merged 4 commits into
nextfrom
chore/bump-miden-protocol-0.17.0-rc.4
Sep 15, 2026
Merged

Mirko-von-Leipzig merged 4 commits into
nextfrom
chore/bump-miden-protocol-0.17.0-rc.4

Conversation

@sofiazcoaga

@sofiazcoaga sofiazcoaga commented Sep 11, 2026

Copy link
Copy Markdown
Contributor

Summary

Upgrade to miden-protocol 0.17.0-rc.4. Needed to integrate the same version in the client.

Main changes:

  • Bump the seven miden-protocol crates from 0.17.0-rc.3 to 0.17.0-rc.4.
  • Replace every canonical TryFrom<proto> conversion with decode_fields() plus an explicit verify(), verify_with(context) or build_unchecked().
  • Add verify_field, verify_value, verify_optional and the verify! macro.

Changelog

changelog = "none"
reason    = "Internal change only."

@Mirko-von-Leipzig Mirko-von-Leipzig left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

As mentioned in one of the comments, we should be using the actual miden-protobuf traits to replace all of our own TryFrom implementations.

Its unclear how much extra work that entails, so I'm okay with merging this as an intermediate form to unblock the client.

Comment thread bin/network-monitor/src/deploy/mod.rs Outdated
Comment thread bin/network-monitor/src/deploy/mod.rs Outdated
Comment thread bin/network-monitor/src/counter.rs
Comment thread bin/network-monitor/src/funding.rs
Comment thread crates/proto/src/decode/mod.rs
Co-authored-by: Mirko <48352201+Mirko-von-Leipzig@users.noreply.github.com>
@Mirko-von-Leipzig
Mirko-von-Leipzig added this pull request to stack #2622 September 14, 2026 13:26
…ocol-0.17.0-rc.4

# Conflicts:
#	bin/benchmark/src/create_proofs.rs
#	bin/benchmark/src/rpc_state.rs
#	bin/large-account-benchmark/src/rpc.rs
#	bin/network-monitor/src/deploy/mod.rs
#	bin/node/src/commands/recover.rs
#	crates/block-producer/src/rpc_sync.rs
@sofiazcoaga
sofiazcoaga marked this pull request as ready for review September 14, 2026 14:15
…ocol-0.17.0-rc.4

# Conflicts:
#	crates/proto/src/domain/proof_request.rs
#	crates/rpc/src/tests.rs
@Mirko-von-Leipzig
Mirko-von-Leipzig merged commit 693adea into next Sep 15, 2026
32 checks passed
@Mirko-von-Leipzig
Mirko-von-Leipzig deleted the chore/bump-miden-protocol-0.17.0-rc.4 branch September 15, 2026 13:12
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