Skip to content

Fix OOG reversion from sending to cold account - #30

Merged
danyalprout merged 1 commit into
mainfrom
fix-revert-from-first-tx
Jul 25, 2026
Merged

Fix OOG reversion from sending to cold account#30
danyalprout merged 1 commit into
mainfrom
fix-revert-from-first-tx

Conversation

@mbeutt-cb

Copy link
Copy Markdown
Collaborator

Summary

  • There is currently a bug in our gas estimation that doesn't account for an eth transfer to a cold account
  • This adds additional headroom to our existing estimates (9k value-transfer surcharge and 25k for CallNewAccountGas)
  • In practice this headroom is almost certainly more that we'll need, but this will give us sufficient cushion to avoid persistent OOG errors

@cb-heimdall

cb-heimdall commented Jul 24, 2026

Copy link
Copy Markdown
Collaborator

✅ Heimdall Review Status

Requirement Status More Info
Reviews 1/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

@vercel

vercel Bot commented Jul 24, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
omni-ui Ready Ready Preview, Comment Jul 24, 2026 11:12pm

Request Review

@danyalprout
danyalprout merged commit 7a87950 into main Jul 25, 2026
6 checks passed
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