[codex] harden Easy Edits burner reliability#41
Merged
Conversation
JamesCarnley
force-pushed
the
codex/easy-edits-reliability
branch
from
June 25, 2026 20:01
bf87f4b to
f5666d6
Compare
JamesCarnley
marked this pull request as ready for review
June 26, 2026 04:42
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
NEXT_PUBLIC_SEPOLIA_RPC_URL, plus devnet/local equivalents) ahead of Alchemy, so operator-provided paid/proxied RPCs still win.Context
app.efs.eth.limois serving the merged Easy Edits code, but promptless Sepolia writes were still routed by the burner signer through the chain's default public RPC unlessNEXT_PUBLIC_SEPOLIA_RPC_URLwas set. That can make uploads feel much slower or flakier than manual wallet flows. The Easy Edits click path also deferred the actualconnect()call to an effect, whereas manually choosing Burner Wallet calls the connector directly from the user's action.This keeps the feature as a disposable signer layer only. It does not change identity/account semantics or write attribution.
Verification
yarn workspace @se-2/nextjs exec node --test --test-reporter=spec utils/scaffold-eth/chainAwareBurnerWallet.test.ts utils/scaffold-eth/instantBurner.test.ts(29 pass, 0 fail)yarn workspace @se-2/nextjs test --test-reporter=spec(156 pass, 0 fail)yarn next:lint --max-warnings=0yarn next:check-typeshttp://localhost:3003: fallback Chrome Playwright check cleared storage, clicked Easy Edits, generatedburnerWallet.pk, connected burner on Sepolia, and observedaria-pressed="true"with no wallet modal.https://app.efs.eth.limois allowed for/faucet/drip; localhost is intentionally not allowed, so local faucet POST fails are not expected on production.Expert Review
Four specialist review passes were run after the first PR version. One RPC/performance reviewer found that the first transport patch let Alchemy outrank an explicit
NEXT_PUBLIC_SEPOLIA_RPC_URL; this head fixes that and adds a regression test. The other reviewers found no blocking state-machine, guardrail, or user-flow issues. Remaining non-blocking gap: there is no React-level unit test for the Easy Edits click itself; browser verification covers it for this PR.Notes
The in-app browser bridge timed out when navigating both the live and local EFS app, even though
about:blankworked and curl/local Chrome loaded the app. I used it as far as it would go, then used Chrome-channel Playwright for click-level verification.