This repository was archived by the owner on Apr 16, 2026. It is now read-only.
Bump protosol to v5.3.0, flatbuffers to v25.12.19-patches - #221
Merged
Conversation
- Update protosol submodule from v5.1.0 to v5.3.0 - Update flatbuffers submodule from v25.9.23-patches to v25.12.19-patches - Regenerate Python protobuf and flatbuffers bindings - Remove stake_delegations_t_1 references (field removed in protosol v5.3.0) Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
cmoyes-jump
requested review from
Copilot,
kbhargava-jump,
mjain-jump and
topointon-jump
April 6, 2026 23:21
There was a problem hiding this comment.
Pull request overview
Updates the repository’s schema/tooling dependencies (protosol + flatbuffers) and refreshes the Python protobuf bindings used by the test suite to match the new proto definitions.
Changes:
- Bump git submodule tracking branches for
shlr/protosolandshlr/flatbuffers. - Regenerate
src/test_suite/protos/*_pb2.pyto reflect updated.protodefinitions (including descriptor/layout changes). - Remove BlockContext stake-delegation base58 encode/decode handling that no longer corresponds to the updated protobuf schema.
Reviewed changes
Copilot reviewed 14 out of 14 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
src/test_suite/protos/vm_pb2.py |
Regenerated protobuf Python bindings (version/header + runtime validation updated). |
src/test_suite/protos/txn_pb2.py |
Regenerated bindings; updated serialized descriptors and offsets. |
src/test_suite/protos/shred_pb2.py |
Regenerated protobuf bindings (version/header + runtime validation updated). |
src/test_suite/protos/serialize_pb2.py |
Regenerated protobuf bindings (version/header + runtime validation updated). |
src/test_suite/protos/pack_pb2.py |
Regenerated protobuf bindings (version/header + runtime validation updated). |
src/test_suite/protos/metadata_pb2.py |
Regenerated protobuf bindings (version/header + runtime validation updated). |
src/test_suite/protos/invoke_pb2.py |
Regenerated protobuf bindings (version/header + runtime validation updated). |
src/test_suite/protos/gossip_pb2.py |
Regenerated protobuf bindings (version/header + runtime validation updated). |
src/test_suite/protos/context_pb2.py |
Regenerated bindings; descriptor content/offsets changed to match updated context schema. |
src/test_suite/protos/block_pb2.py |
Regenerated bindings; block schema changed (layout/fields) and offsets updated. |
src/test_suite/block/codec_utils.py |
Removes stake-delegation encode/decode helpers and loops aligned with schema removal. |
.gitmodules |
Updates submodule branch tracking to protosol v5.3.0 and flatbuffers v25.12.19-patches. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
- Pin buf plugin to v34.1 (was v34.0) to match protobuf==7.34.1 runtime - Regenerate Python protobuf and flatbuffers with correct tool versions - Fix CreateXXHash -> CreateXxhash rename from flatc v25.12.19 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…ication - check-protobuf-version: ensures generated pb2 files match pyproject.toml pin - check-quick-import: catches broken imports from flatc API renames - CI: git diff --exit-code after regeneration to catch stale committed code Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
The committed file was empty but codegen writes a comment line, causing the CI git-diff check to fail. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
drubin-fd
approved these changes
Apr 9, 2026
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 subscribe to this conversation on GitHub.
Already have an account?
Sign in.
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.
No description provided.