[fengtality]

Closes #713.

Cause

src/connectors/pancakeswap/clmm-routes/executeSwap.ts set sqrtPriceLimitX96 with encodeSqrtRatioX96 from @uniswap/v3-sdk, given quote.trade.executionPrice.numerator/denominator. A PancakeSwap SDK Price holds native bigint values. The Uniswap encoder calls JSBI.BigInt(arg), which accepts numbers, strings, booleans and objects, and throws TypeError: Cannot convert <value> to a BigInt for a native bigint. So every PancakeSwap CLMM swap (Ethereum and BSC) failed before it was sent. Reproduced directly:

new Price(WBNB, USDT, 10n**18n, 999429662315995114n).numerator  // typeof 'bigint'
encodeSqrtRatioX96(p.numerator, p.denominator)
// TypeError: Cannot convert 999429662315995114 to a BigInt

Fix

Pass sqrtPriceLimitX96: '0' (no limit), as the Uniswap CLMM route does since bb44ffd. Encoding the trade's average execution price as a limit is wrong too: the pool's price crosses that average partway through the swap, so it would partial-fill and revert on the min-out check. Slippage protection stays on amountOutMinimum / amountInMaximum from the quote.

The same change is inside #680 (and #647), which bundle it with the MasterChef staking work. This PR is the minimal fix for #713 alone; credit to those PRs for spotting it.

Tests

• New test/connectors/pancakeswap/clmm-routes/execute-swap.test.ts: SELL and BUY with a real PancakeSwap SDK Price built from the issue's numerator. Both fail on development with the exact error from the issue, and pass with the fix.
• test/connectors/pancakeswap: 14 suites, 58 tests pass.

🤖 Generated with Claude Code

[greptile-apps]

Confidence Score: 5/5

[Critical impact] The PR appears safe to merge.

Summary

Replaces the PancakeSwap CLMM price encoder with sqrtPriceLimitX96: '0', avoiding the native-bigint conversion error.

• PancakeSwap CLMM swaps run without a separate price limit.

_{Reviews (1) · Last reviewed commit: "fix(pancakeswap): send CLMM swaps with n..." · Reviewed by Greptile}

[fengtality]

Bundled into #735 with the other fix PRs.