fix(#523): dual-read wrap-mapper fees on router unwrap_output #1056

Merged

PlasticDigits merged 2 commits from feat/523-router-unwrap-split-fees into main

2026-08-15 12:01:37 +00:00

PlasticDigits commented

2026-08-15 11:54:28 +00:00

(Migrated from gitlab.com)

Closes #523

Summary

Router unwrap_output settlement dual-reads wrap-mapper Config with the same W13 rules as the dApp: fee_unwrap_bps after ustr-cmm#9 migrate; legacy fee_bps only when both split fields are absent.
#[cw_serde] ConfigResponse no longer requires fee_bps, so native-out Swap no longer reverts when mapper drops that field (!1054 follow-up).
Router crate bumped to 1.1.0 for store + migrate in the same window as wrap-mapper migrate (ustr-cmm#13).

Test plan

make verify-issue-523 (5/5)
cargo test -p dex-common wrap_mapper
test_unwrap_output_split_fee_config_no_fee_bps (post-migrate Config, no fee_bps)
existing test_unwrap_minimum_receive_* (legacy { fee_bps })
Store + migrate router wasm before or with wrap-mapper migrate + set_fees 200/51

Closes #523 ## Summary - Router `unwrap_output` settlement dual-reads wrap-mapper `Config` with the same W13 rules as the dApp: `fee_unwrap_bps` after ustr-cmm#9 migrate; legacy `fee_bps` only when both split fields are absent. - `#[cw_serde]` `ConfigResponse` no longer requires `fee_bps`, so native-out Swap no longer reverts when mapper drops that field (!1054 follow-up). - Router crate bumped to **1.1.0** for store + migrate in the same window as wrap-mapper migrate ([ustr-cmm#13](https://gitlab.com/PlasticDigits2/ustr-cmm/-/work_items/13)). ## Test plan - [x] `make verify-issue-523` (5/5) - [x] `cargo test -p dex-common wrap_mapper` - [x] `test_unwrap_output_split_fee_config_no_fee_bps` (post-migrate Config, no `fee_bps`) - [x] existing `test_unwrap_minimum_receive_*` (legacy `{ fee_bps }`) - [ ] Store + migrate router wasm before or with wrap-mapper migrate + `set_fees` 200/51

PlasticDigits commented

2026-08-15 11:59:55 +00:00

(Migrated from gitlab.com)

mentioned in issue #523

PlasticDigits commented

2026-08-15 12:01:23 +00:00

(Migrated from gitlab.com)

added 3 commits

f120f2da...8ff700ee - 2 commits from branch main
1d5360e0 - Merge branch 'main' into feat/523-router-unwrap-split-fees

Compare with previous version

added 3 commits <ul><li>f120f2da...8ff700ee - 2 commits from branch <code>main</code></li><li>1d5360e0 - Merge branch 'main' into feat/523-router-unwrap-split-fees</li></ul> [Compare with previous version](/PlasticDigits/cl8y-dex-terraclassic/-/merge_requests/338/diffs?diff_id=1963812743&start_sha=f120f2daf842fe058d7a5092fa13535465af9ae3)