Skip to content

chore: removes all code related to the normalized state diff hash#1244

Merged
mds1 merged 1 commit into
mainfrom
chore/remove-normalized-state-diff-hash
Oct 10, 2025
Merged

chore: removes all code related to the normalized state diff hash#1244
mds1 merged 1 commit into
mainfrom
chore/remove-normalized-state-diff-hash

Conversation

@mds1
Copy link
Copy Markdown
Contributor

@mds1 mds1 commented Oct 9, 2025

The normalized state diff hash was introduced briefly as an experiment: Between upgrades 13 through 16, we had many upgrades for many chains with many state diffs, and at this time we required that VALIDATION.md files list all state diffs for every task. This was a lot of state diffs, so it was very tedious for task developers to create these files, and for signers to validate them.

As a result, we tried experimenting with a "normalized state diff hash" which commits to a state diff at the time a task is created, normalizing for any state variables that are not constant, such as timestamps. The idea was that signers would know that if that hash hasn't changed, the state diff has not changed since the task was created. To further lighten the load for task developers, we did state diff audits where the auditors would act as an independent third party that would generate and attest to the state diff.

This experiment worked ok, but ultimately it was not something we needed to continue, as we realized that for tasks using an OPCM we actually do not need to list every state diff. As a result, we no longer use normalized state diff hashes, therefore we can remove all of this code and the corresponding documentation from superchain-ops tasks going forward.

@mds1 mds1 requested review from a team as code owners October 9, 2025 21:45
@mds1 mds1 requested a review from smartcontracts October 9, 2025 21:45
@Ethnical
Copy link
Copy Markdown
Contributor

Ethnical commented Oct 10, 2025

From my end, I have simulated and check the changes on previous tasks
And this is looks good to me, this is nice was confusing people a bit sometimes, especially when were missing some override during tasks simulation in sequencing.
c4685f386295a145e19c8ecb90415200bfd9259db2a03659669665fae72808fb

@mds1 mds1 added this pull request to the merge queue Oct 10, 2025
Merged via the queue into main with commit 9defeb9 Oct 10, 2025
20 checks passed
@mds1 mds1 deleted the chore/remove-normalized-state-diff-hash branch October 10, 2025 13:40
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