fix: mixed hybrid router gas stays ~192 LUNC (#679) #1182

Merged

PlasticDigits merged 2 commits from feat/679-hybrid-mixed-hop-gas into main

2026-08-27 04:52:42 +00:00

PlasticDigits commented

2026-08-27 00:37:15 +00:00

(Migrated from gitlab.com)

Summary

Mixed hybrid + pool router hops no longer add HYBRID_SWAP_GAS_LIMIT (15M) per hop that omits hybrid. The live 4-hop cLUNC→USTR shape (book on hop 1, pool on hops 2–4) is now 6,785,500 gas (~192 LUNC), above columbus-5 gas_used 5,026,176 and well below 15M.
Swap Network fee (est.) uses the same indexer operations as submit, so the hint matches broadcast (not the old pool-only ~108 LUNC).
Station mobile/WC auto-gas residual is documented: dApp row is the source of truth; atomic WC is unchanged; mainnet SEC-E08 stays off.

Closes #679

Test plan

make verify-issue-679 (unit + swarm lockstep + docs; child #475 / #587 / #596; retest)
Mixed 4-hop envelope is 6,785,500, not 46,785,500; fee LUNC < 400
Hint = broadcast for indexer-equivalent ops
Wrap+2hop / hybrid-on-every-hop / direct-pair book_input=0 unchanged
Station extension on columbus-5: wallet Tx Fee matches dApp Network fee when preferNoSetFee is honored
Station mobile / WC: either the same match, or residual auto-gas with dApp row still ~192 LUNC
Optional LocalTerra: VERIFY_ISSUE_679_CHAIN=1 — ≥2-hop CW20 with hybrid on one hop only (Keplr / simulated wallet, not Station)

## Summary - Mixed hybrid + pool router hops no longer add `HYBRID_SWAP_GAS_LIMIT` (15M) per hop that omits `hybrid`. The live 4-hop cLUNC→USTR shape (book on hop 1, pool on hops 2–4) is now **6,785,500** gas (~192 LUNC), above columbus-5 `gas_used` **5,026,176** and well below 15M. - Swap **Network fee (est.)** uses the same indexer `operations` as submit, so the hint matches broadcast (not the old pool-only ~108 LUNC). - Station mobile/WC auto-gas residual is documented: dApp row is the source of truth; atomic WC is unchanged; mainnet SEC-E08 stays off. Closes #679 ## Test plan - [x] `make verify-issue-679` (unit + swarm lockstep + docs; child #475 / #587 / #596; retest) - [x] Mixed 4-hop envelope is 6,785,500, not 46,785,500; fee LUNC < 400 - [x] Hint = broadcast for indexer-equivalent ops - [x] Wrap+2hop / hybrid-on-every-hop / direct-pair book_input=0 unchanged - [ ] Station extension on columbus-5: wallet Tx Fee matches dApp Network fee when `preferNoSetFee` is honored - [ ] Station mobile / WC: either the same match, or residual auto-gas with dApp row still ~192 LUNC - [ ] Optional LocalTerra: `VERIFY_ISSUE_679_CHAIN=1` — ≥2-hop CW20 with hybrid on one hop only (Keplr / simulated wallet, not Station)

PlasticDigits commented

2026-08-27 00:37:34 +00:00

(Migrated from gitlab.com)

mentioned in issue #679

PlasticDigits commented

2026-08-27 04:52:25 +00:00

(Migrated from gitlab.com)

added 13 commits

3e2d809f...972d46f9 - 12 commits from branch main
85b7c248 - Merge origin/main into feat/679-hybrid-mixed-hop-gas.

Compare with previous version

added 13 commits <ul><li>3e2d809f...972d46f9 - 12 commits from branch <code>main</code></li><li>85b7c248 - Merge origin/main into feat/679-hybrid-mixed-hop-gas.</li></ul> [Compare with previous version](/PlasticDigits/cl8y-dex-terraclassic/-/merge_requests/464/diffs?diff_id=1986966260&start_sha=3e2d809fdd7ea2bb47c08bff7ea60ed87381295a)

PlasticDigits commented

2026-08-27 04:52:42 +00:00

(Migrated from gitlab.com)

mentioned in commit 1fbee5b4d5

mentioned in commit 1fbee5b4d529875467ae6a2467f73ddf813943ef