Skip to content

fix(snapshots): calculate full download from history window - #156

Open
haardikk21 wants to merge 2 commits into
mainfrom
fix/full-snapshot-download-size
Open

fix(snapshots): calculate full download from history window#156
haardikk21 wants to merge 2 commits into
mainfrom
fix/full-snapshot-download-size

Conversation

@haardikk21

@haardikk21 haardikk21 commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

Summary

  • calculate the Full preset from complete State and Headers plus every Transactions, Receipts, and Account/Storage Changesets chunk overlapping the final 1,339,200 blocks
  • derive the history cutoff from the manifest's actual snapshot block, so a partial newest chunk correctly causes three or four files to be counted
  • keep Senders and RocksDB excluded from Full, while leaving Archive and Minimal totals unchanged
  • cover both three- and four-file history windows with unit tests

Screenshots

Snapshots Full preset with corrected download size

Test plan

  • npm test (338 tests)
  • npm run typecheck
  • npx eslint app/snapshots/data.ts app/snapshots/data.test.ts app/snapshots/r2.ts app/snapshots/r2.test.ts app/snapshots/SnapshotsClient.tsx
  • npm run build
  • validated the current mainnet manifest: block 50,758,927 produces cutoff 49,419,727 and selects four chunks per historical component
  • visually verified /snapshots at 1440×1000

@vercel

vercel Bot commented Sep 4, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
omni-ui Ready Ready Preview Sep 4, 2026 6:25pm UTC

Request Review

@cb-heimdall

Copy link
Copy Markdown
Collaborator

🟡 Heimdall Review Status

Requirement Status More Info
Reviews 🟡 0/1
Denominator calculation
Show calculation
1 if user is bot 0
1 if user is external 0
2 if repo is sensitive 0
From .codeflow.yml 1
Additional review requirements
Show calculation
Max 0
0
From CODEOWNERS 0
Global minimum 0
Max 1
1
1 if commit is unverified 0
Sum 1

@haardikk21 haardikk21 changed the title fix(snapshots): calculate full download from tail chunks fix(snapshots): calculate full download from history window Sep 4, 2026
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