Skip to content

fix(wow): fail closed when graduated Uniswap quote is missing - #1421

Open
SashaMIT wants to merge 1 commit into
coinbase:mainfrom
SashaMIT:fix/wow-fail-closed-zero-quote
Open

fix(wow): fail closed when graduated Uniswap quote is missing#1421
SashaMIT wants to merge 1 commit into
coinbase:mainfrom
SashaMIT:fix/wow-fail-closed-zero-quote

Conversation

@SashaMIT

@SashaMIT SashaMIT commented Aug 9, 2026

Copy link
Copy Markdown

Summary

  • Graduated WOW buy/sell used Uniswap quoter amountOut as slippage floor
  • Quoter failures returned 0, so minTokens / minEth became 0 and the trade still submitted
  • Fail closed when the graduated quote is missing/zero (TS + Python); bonding-curve path unchanged

Test plan

  • Added TS unit: buy refuses to sendTransaction when getBuyQuote fails closed
  • CI

Tip: 0fe026bf.

Made with Cursor

Quoter errors returned amountOut=0, which became minOut/minEth=0 on
buy/sell for graduated tokens. Refuse the trade instead of submitting
unprotected slippage.
@SashaMIT
SashaMIT requested a review from murrlincoln as a code owner August 9, 2026 05:56
@cb-heimdall

Copy link
Copy Markdown

🟡 Heimdall Review Status

Requirement Status More Info
Reviews 🟡 0/1
Denominator calculation
Show calculation
1 if user is bot 0
1 if user is external 0
2 if repo is sensitive 0
From .codeflow.yml 1
Additional review requirements
Show calculation
Max 0
0
From CODEOWNERS 0
Global minimum 0
Max 1
1
1 if commit is unverified 0
Sum 1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Development

Successfully merging this pull request may close these issues.

2 participants