Deposit transactions need padding
Open
state: backlog
type: enhancement
- Dominant language
- TypeScript
- Stars
- 371
- Forks
- 248
- PR merge metrics
- No merged PRs in 30d
Description
We need to estimate gas and add additional padding because without it the algorithm wallets use to estimate gas will often result in a failed tx.
The optimism sdk does this here: https://github.com/ethereum-optimism/optimism/blob/develop/packages/sdk/src/cross-chain-messenger.ts#L2028
Contributor guide
Research direction
Start by reading the referenced Optimism SDK implementation at the linked cross-chain-messenger.ts location, then trace how deposit transactions estimate gas in op-viem. Done means adding the requested padding so wallet gas estimation no longer commonly produces failed transactions; add or update tests if the repository has coverage for this path.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- typescript
- Domain
- blockchain
- Issue type
- Bug
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100