fix: estimate a request even when the site sends its gas limit (closes #554) #557

Merged
clawbot merged 1 commits from issue-554-estimate-with-site-gas into next 2026-10-14 19:46:02 +02:00
1 Commits
Author SHA1 Message Date
sneak 7d6dad20cc fix: estimate a request even when the site sends its gas limit (closes #554)
check / check (push) Waiting to run
e2e / e2e-chrome (push) Waiting to run
e2e / e2e-firefox (push) Waiting to run
ethers asks the node for a gas estimate only when the request has no gas
limit, and that estimate is the wallet's only check that a transaction
does not revert. Since a site's `gas` became the gas limit, a reverting
request carrying one opened the approval window and could be signed and
mined to fail. Such a request is now estimated first, with the site's
limit and under the same timeout; a failure goes back to the site with no
window, and the site's gas limit is still the one shown and signed. A
limit too low for the call now fails the same way.

Model: opus-5-5
2026-10-14 16:38:23 +00:00