fix(frontend): raise ADD_LIQUIDITY_GAS_LIMIT above measured provide_liquidity gas #843

Merged

Brouie merged 1 commit from qa/addliq-gas-limit into main

2026-06-09 07:05:30 +00:00

Brouie commented

2026-06-09 02:16:00 +00:00

(Migrated from gitlab.com)

Finding (during #337 local QA pass)

Add-liquidity fails from the dapp with "Transaction needed more gas than estimated" on every attempt. provide_liquidity returns the fixed ADD_LIQUIDITY_GAS_LIMIT = 500000, but the real cost is higher.

Measured on a localnet EMBER/CORAL pair (terrad, --gas auto):

op	gas_used	dapp limit	result
`provide_liquidity`	507,509	500,000	OOG
`withdraw_liquidity`	544,545	600,000	ok

The limit is fixed (not simulated), and block variance is far below the ~7.5k shortfall, so the "try again" hint can't recover it — add-liquidity is effectively unusable from the UI. Contract path itself is fine (provide_liquidity, transfer_from x2, mint, code 0).

Fix

Raise ADD_LIQUIDITY_GAS_LIMIT 500000 -> 650000 (~28% over measured, in line with the headroom REMOVE already carries). transactions.test.ts assertion updated. REMOVE (600k) and CREATE_PAIR (800k) left as-is.

Same gas-underestimate family as #343 (different path — no native/unwrap here). Found verifying #337 (PL-00-02).

## Finding (during #337 local QA pass) Add-liquidity fails from the dapp with **"Transaction needed more gas than estimated"** on every attempt. `provide_liquidity` returns the fixed `ADD_LIQUIDITY_GAS_LIMIT = 500000`, but the real cost is higher. Measured on a localnet EMBER/CORAL pair (terrad, `--gas auto`): | op | gas_used | dapp limit | result | |----|----------|-----------|--------| | `provide_liquidity` | **507,509** | 500,000 | **OOG** | | `withdraw_liquidity` | 544,545 | 600,000 | ok | The limit is fixed (not simulated), and block variance is far below the ~7.5k shortfall, so the "try again" hint can't recover it — add-liquidity is effectively unusable from the UI. Contract path itself is fine (`provide_liquidity, transfer_from x2, mint`, code 0). ## Fix Raise `ADD_LIQUIDITY_GAS_LIMIT` 500000 -> 650000 (~28% over measured, in line with the headroom REMOVE already carries). `transactions.test.ts` assertion updated. REMOVE (600k) and CREATE_PAIR (800k) left as-is. Same gas-underestimate family as #343 (different path — no native/unwrap here). Found verifying #337 (PL-00-02).

Rows
Columns