snarkVM icon indicating copy to clipboard operation
snarkVM copied to clipboard

[Do Not Merge] Fall back to int deserialization for block metadata

Open vicsn opened this issue 6 months ago • 1 comments

Motivation

This old PR: https://github.com/ProvableHQ/snarkVM/pull/2559 introduced a backwards-incompatible change. We just changed serialization of data. This PR tries both versions in the deserialization.

Alternatively to this PR, explorers and indexers re-index their data to use the new format. Keen to hear your feedback on the preferred way forward. CC @miazn @HarukaMa

Test Plan

  • [ ] Test that deserialization and snarkos developer scan now succeeds.

vicsn avatar Jun 23 '25 11:06 vicsn

Looks like this is only affecting JSON block format. AleoScan doesn't consume JSON format; it currently only accepts data in binary format and directly from local rocksdb, so it's not affected by this change. For example, the explorer won't directly support upcoming upgradable program, as the binary format of programs will be changed to include a new constructor element.

Internally AleoScan does have a JSON serializer to produce data for the frontend to consume, but it's not designed to be consumed back by snarkVM as it didn't store the subdag data from blocks, thus can't reproduce complete blocks from the database.

HarukaMa avatar Jun 24 '25 00:06 HarukaMa

Also has been fixed for explorer.provable.com - thx @miazn !!!

vicsn avatar Jul 07 '25 20:07 vicsn