fix(frontend): recalibrate router wrap and multi-hop gas (#353) #854

Merged

PlasticDigits merged 1 commit from local1/353-impl-router-gas into main

2026-06-10 05:31:49 +00:00

PlasticDigits commented

2026-06-10 03:37:23 +00:00

(Migrated from gitlab.com)

Fixes #353 — native wrap-swap and multi-hop router gas budgets.

Note

Medium Risk
Changes on-chain fee envelopes for router and wrap swaps; underestimation caused out-of-gas failures, but higher gas increases LUNC fees and must stay aligned with swarm + preflight estimates.

Overview
Fixes #353 by raising broadcast Fee.gas for paths that were under-budgeted vs measured LocalTerra usage.

Router vs direct pair: execute_swap_operations (native wrap, CW20→router) now uses new gasLimitForRouterExecuteSwapOperations: 1.4M single-hop, multi-hop floored at 900k/hop (+ buffer/padding/safety margin). Direct-to-pair swap and inner send→swap still use gasLimitForExecuteSwapOperations (840k one-hop).

Wrap: WRAP_GAS_LIMIT 300k → 400k (~301k measured).

Sync: Vitest expectations in transactions.test.ts / terraClassicFeeEstimate.test.ts, packages/localnet-trading-swarm/src/gas.ts, docs/frontend.md, and skills/AGENTS_TERRACLASSIC_GAS.md updated to match.

^{Reviewed by Cursor Bugbot for commit 4eaeba6d12. Bugbot is set up for automated code reviews on this repo. Configure here.}

Fixes #353 — native wrap-swap and multi-hop router gas budgets.  --- > [!NOTE] > **Medium Risk** > Changes on-chain fee envelopes for router and wrap swaps; underestimation caused out-of-gas failures, but higher gas increases LUNC fees and must stay aligned with swarm + preflight estimates. > > **Overview** > Fixes **#353** by raising broadcast **`Fee.gas`** for paths that were under-budgeted vs measured LocalTerra usage. > > **Router vs direct pair:** `execute_swap_operations` (native wrap, CW20→router) now uses new **`gasLimitForRouterExecuteSwapOperations`**: **1.4M** single-hop, multi-hop floored at **900k/hop** (+ buffer/padding/safety margin). Direct-to-pair **`swap`** and inner **`send`→swap** still use **`gasLimitForExecuteSwapOperations`** (**840k** one-hop). > > **Wrap:** **`WRAP_GAS_LIMIT`** **300k → 400k** (~301k measured). > > **Sync:** Vitest expectations in **`transactions.test.ts`** / **`terraClassicFeeEstimate.test.ts`**, **`packages/localnet-trading-swarm/src/gas.ts`**, **`docs/frontend.md`**, and **`skills/AGENTS_TERRACLASSIC_GAS.md`** updated to match. > > <sup>Reviewed by [Cursor Bugbot](https://cursor.com/bugbot) for commit 4eaeba6d1219bbf4ec62d936a837279d212cdeb4. Bugbot is set up for automated code reviews on this repo. Configure [here](https://www.cursor.com/dashboard/bugbot).</sup>

Rows
Columns