Skip to content

fix(swap): raise default --slippage from 50 to 100 bps for thin-liquidity broadcasts #20

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
Hiksang merged 1 commit into from
May 7, 2026
Merged

fix(swap): raise default --slippage from 50 to 100 bps for thin-liquidity broadcasts #20

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
9 changes: 8 additions & 1 deletion ts/packages/defi-cli/src/commands/swap.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -219,7 +219,14 @@ export function registerSwap(
.requiredOption("--to <token>", "Output token symbol or address")
.requiredOption("--amount <amount>", "Amount of input token in wei")
.option("--provider <name>", "Aggregator: kyber, openocean, liquid, lifi, relay", "kyber")
.option("--slippage <bps>", "Slippage tolerance in bps", "50")
// Default 100 bps (1%) is the SSOT 7.3 ceiling for "safe default" and
// the sweet spot for thin-liquidity chains like Monad where the previous
// 50 bps default produced revert-on-broadcast even when the dry-run
// simulated successfully. Live re-verification on 2026-05-07 reproduced
// the issue: kyber dry-run succeeded but the actual on-chain tx reverted
// inside the router's minOut check (Monad tx 0x4fe39977…). Users wanting
// tighter control still pass --slippage explicitly.
.option("--slippage <bps>", "Slippage tolerance in bps", "100")
.action(async (opts) => {
const executor = makeExecutor();
const chainName = requireChain(parent, getOpts);
Expand Down