Skip to content

Document Eden v0.4.0#146

Merged
jcstein merged 7 commits into
mainfrom
nuke-devx-24
Jun 9, 2026
Merged

Document Eden v0.4.0#146
jcstein merged 7 commits into
mainfrom
nuke-devx-24

Conversation

@nuke-web3

@nuke-web3 nuke-web3 commented Jun 1, 2026

Copy link
Copy Markdown
Member

Introduces a page on "why eden" that covers novel things ev-reth give over a generic EVM clone, specifically focused on the 0x76 tx type.

Resolves https://linear.app/celestia/issue/DEVX-24/document-eden-v040-app-based-gas-sponsorship-batch-tx-move-away-from

@nuke-web3 nuke-web3 self-assigned this Jun 1, 2026
@nuke-web3 nuke-web3 marked this pull request as ready for review June 4, 2026 20:18
@nuke-web3 nuke-web3 requested a review from jcstein June 4, 2026 20:18
@jcstein

jcstein commented Jun 4, 2026

Copy link
Copy Markdown
Member

Heads-up: #152 (bun standardization) just merged to main, so this branch needs a rebase — and the lockfiles need cleanup along the way. This PR currently touches bun.lock, bun.lockb, and yarn.lock; after rebasing, main should win on all three:

  • delete bun.lockb — it was generated by bun 1.1.x; the repo standardized on the text-based bun.lock (bun >= 1.2, see README). Committing both formats risks them silently drifting apart.
  • don't resurrect yarn.lock — it was removed in chore: standardize on bun for dev and CI #152.
  • take main's bun.lock — it was regenerated from scratch there (the yarn-migrated one didn't link bins). If this PR adds/changes deps, re-run bun install after the rebase with bun >= 1.2 (bun upgrade if needed) so the lockfile updates cleanly on top.

The bun >= 1.2 bit also explains the slow first install you hit on the other branch — 1.1.29 can't read bun.lock, so it did a full unpinned resolve.

@jcstein jcstein left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed the new /why-eden page and homepage changes. Good, well-scoped docs PR — I verified the 0x76 material against the ev-reth README and the core claims (batch atomicity, dual-signature scheme, refunds routing to sponsor) are accurate, and both external links resolve.

Two things I'd want resolved before merge (inline comments have details):

  1. The "No mempool, no MEV" framing overstates — ev-reth's own README documents a txpool, and "execute in the order submitted" rests on trusting the sequencer's discretion rather than a protocol guarantee.
  2. 128 KB contract limit needs confirmation — in ev-reth it's an opt-in chainspec setting (contractSizeLimit, with EIP-170 applying before the activation height), not a default. Worth confirming Eden mainnet's chainspec actually sets it before we promise it to developers.

The rest is polish — heading hierarchy, a few wording nits, and finishing the "Celestia Eden" → "Eden" rename on the homepage. The WIP callout gives some cover, but #1 and #2 are developer-facing promises people will design around.

Comment thread app/why-eden/page.mdx Outdated
Comment thread app/why-eden/page.mdx Outdated
Comment thread app/why-eden/page.mdx
Comment thread app/why-eden/page.mdx Outdated
Comment thread app/why-eden/page.mdx Outdated
Comment thread app/why-eden/page.mdx Outdated
Comment thread app/page.mdx
nuke-web3 and others added 2 commits June 5, 2026 16:27
Verified via eth_estimateGas on mainnet RPC: a creation tx returning
exactly 131,072 bytes of runtime code passes the size check, while
131,073 bytes fails with CreateContractSizeLimit (and a 28 KB deploy
estimates fine, so EIP-170 is off). Also reverts a blank line in the
wallets page that prettier strips.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@gbarros

gbarros commented Jun 8, 2026

Copy link
Copy Markdown
Collaborator

I didn't dive too much in the details (as in if a specific information is correct); but I read through it and it is looking good!
it's very clear and easy to grasp.

@jcstein jcstein left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

Comment thread app/why-eden/page.mdx
Comment thread app/why-eden/page.mdx Outdated
Signed-off-by: Josh Stein <46639943+jcstein@users.noreply.github.com>
@jcstein jcstein merged commit a0298dd into main Jun 9, 2026
@jcstein jcstein deleted the nuke-devx-24 branch June 9, 2026 17:32
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.

3 participants