Skip to content

proto: add block marker support - #138

Open
emwang-jump wants to merge 1 commit into
mainfrom
emwang/block-marker-fuzz
Open

emwang-jump wants to merge 1 commit into
mainfrom
emwang/block-marker-fuzz

Conversation

@emwang-jump

Copy link
Copy Markdown

No description provided.

Copilot AI balanced review requested due to automatic review settings September 24, 2026 18:19

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot review overview

🟡 Changes recommended

The schema and checked-in generated Rust are out of sync, causing the CI regeneration check to fail.

Get a fresh assessment by requesting another Copilot review.

Review effort: Balanced
Findings: 1 High severity

Open (1)
What changed in this PR

Adds protobuf support for Alpenglow block-marker differential fuzzing.

Changes:

  • Defines block-marker inputs, outputs, certificates, and parse results.
  • Adds Nanopb sizing and pointer options.
  • Adds generated Rust protobuf bindings.
File Description
proto/​block_marker.proto Defines the block-marker fixture schema.
proto/​block_marker.options Configures Nanopb representations and limits.
src/​generated/​org.solana.sealevel.v1.rs Exposes generated Rust message types.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread proto/block_marker.proto
@emwang-jump
emwang-jump force-pushed the emwang/block-marker-fuzz branch from 3679df6 to 61f64ed Compare September 24, 2026 18:23
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