Skip to content

docs(evm): explain Gravity withdrawal invariant - #390

Merged
Richard1048576 merged 1 commit into
Galxe:mainfrom
AshinGau:docs/explain-gravity-withdrawals
Jul 17, 2026
Merged

docs(evm): explain Gravity withdrawal invariant#390
Richard1048576 merged 1 commit into
Galxe:mainfrom
AshinGau:docs/explain-gravity-withdrawals

Conversation

@AshinGau

@AshinGau AshinGau commented Jul 13, 2026

Copy link
Copy Markdown
Collaborator

close https://github.com/Galxe/gravity-audit/issues/839

Summary

  • document why Gravity ordered blocks cannot carry proposer-controlled withdrawals
  • explain that divergent local execution results cannot join an honest commit quorum
  • clarify that standard withdrawal processing remains necessary for Ethereum mainnet history replay

This addresses the exploitability assumption raised in Galxe/gravity-audit#839 without changing execution semantics.

Documentation-only change; no runtime behavior changed.

@Richard1048576 Richard1048576 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.

LGTM

@Richard1048576
Richard1048576 merged commit cc00451 into Galxe:main Jul 17, 2026
49 of 51 checks passed
nekomoto911 added a commit that referenced this pull request Jul 20, 2026
Advances the merge-base so PR #396 no longer conflicts with main. Brings in the
4 commits that landed after the 2026-07-09 sync: #383 (randomness from canonical
headers), #385 (EIP-7702 cross-account nonce halt + recoverable-error panic
hardening), #382 (mint precompile gas guard), #390 (withdrawal-invariant docs).

The 11 conflicts are all v2.3.0/revm40 API-form divergence against the
already-adapted #367-#374 batch; resolved take-ours (revm40 accessor methods,
PrecompileHalt, AccountInfo::account_id, no StateCacheDbRefMutWrapper). #382's
gas guard re-adapted from revm36 PrecompileError to revm40 PrecompileHalt.

Verified: cargo check --workspace --all-features = 0 err; fmt clean; pipe-exec
and gravity-storage --tests clean.
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