fix(community-tax): tax official-router hops (improved option 2, #607) #1127

Merged

PlasticDigits merged 1 commit from feat/607-router-tax-option-2 into main

2026-08-24 02:41:59 +00:00

PlasticDigits commented

2026-08-24 00:28:58 +00:00

(Migrated from gitlab.com)

Summary

Reverses the #607 option 1 disclose-only close. Official-router Send+Swap is Sell: extra-debit authenticated Cw20HookMsg::Swap.trader (honored only when from == config.router). Pair-direct ignores a spoofed trader. Missing / protocol-exempt trader fail-closes (RouterTraderRequired).
Buy-leg specified: pair→router stays 1:1; official router→user is outbound split. Pair/router swap math unchanged (H-01). Hybrid (#596) stays on.
dApp Max extra-debits on router-hop sells too. Copy: Buy/sell tax applies on every listed-pair swap. / Sell tax extra / Buy tax applies.

Test plan

make verify-issue-607 first pass + retest (docs, frontend Vitest, token multitest, inverted PoC)
cargo test -p cl8y-community-tax-token --lib (65 tests)
Swap pair-direct 11611 sell: Sell tax extra + Max leaves room for extra-debit
Swap multi-hop / hybrid: same extra-debit Max (not full CW20) + Sell tax extra
Trade market pair-direct vs router-ops hints match
Create/Manage: Buy/sell tax applies on every listed-pair swap.
Live 11611 store + CMM migrate (ops follow-up; pin still Honest hops until then)

Closes #607 (choice + implement). #603 importers may promise bps on official Swap only after this wasm.

## Summary - Reverses the #607 option 1 disclose-only close. Official-router `Send+Swap` is **Sell**: extra-debit authenticated `Cw20HookMsg::Swap.trader` (honored only when `from == config.router`). Pair-direct ignores a spoofed `trader`. Missing / protocol-exempt trader fail-closes (`RouterTraderRequired`). - Buy-leg specified: pair→router stays 1:1; official router→user is outbound split. Pair/router swap math unchanged (H-01). Hybrid (#596) stays on. - dApp Max extra-debits on router-hop sells too. Copy: `Buy/sell tax applies on every listed-pair swap.` / `Sell tax extra` / `Buy tax applies`. ## Test plan - [x] `make verify-issue-607` first pass + retest (docs, frontend Vitest, token multitest, inverted PoC) - [x] `cargo test -p cl8y-community-tax-token --lib` (65 tests) - [ ] Swap pair-direct 11611 sell: `Sell tax extra` + Max leaves room for extra-debit - [ ] Swap multi-hop / hybrid: same extra-debit Max (not full CW20) + `Sell tax extra` - [ ] Trade market pair-direct vs router-ops hints match - [ ] Create/Manage: `Buy/sell tax applies on every listed-pair swap.` - [ ] Live 11611 store + CMM migrate (ops follow-up; pin still Honest hops until then) Closes #607 (choice + implement). #603 importers may promise bps on official Swap only after this wasm.

Rows
Columns