refactor: remove jupiter swapper (replaced by relay)#12265
Conversation
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
|
Important Review skippedReview was skipped due to path filters ⛔ Files ignored due to path filters (1)
CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
📝 WalkthroughWalkthroughThis pull request removes all Jupiter swapper integration from the codebase, including the Jupiter API dependency, environment variables, feature flags, type definitions, and all Jupiter-specific implementations across swapper, widget, and main application layers. Changes
Estimated code review effort🎯 4 (Complex) | ⏱️ ~60 minutes Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Description
Remove the Jupiter swapper entirely — Relay (which uses Jupiter under the hood) has fully replaced it.
Deleted:
packages/swapper/src/swappers/JupiterSwapper/— full implementation (swapper, endpoints, IDL, helpers, constants)headers/csps/jupiter.ts— CSP config for Jupiter APIsrc/components/.../SwapperIcon/jupiter-icon.svg— icon assetsrc/constants/urls.ts— only contained Jupiter URLsRemoved from config/state:
SwapperName.Jupiterenum member and all switch/record branchesJupiterSwapfeature flag (preferencesSlice,config.ts,vite-env.d.ts,.env, mock store)VITE_JUPITER_API_URLfrom swapper config type, app config, public-api config/envjupiterQuoteResponsefield from trade step typevalidateTradeQuote.ts@jup-ag/apidependency from swapper package and root pnpm overrideisSolBuyAssetIdparameter fromgetEnabledSwappers(only used by Jupiter)Relocated:
COMPUTE_UNIT_MARGIN_MULTIPLIER→ inlined insolana-utils/getUnsignedSolanaTransaction.ts(still needed by Relay)SolanaLogsError→ moved topackages/swapper/src/utils.ts(still needed by error handling UI)Risk
Medium — removes a swapper that is already superseded by Relay. No new on-chain transaction types introduced. Relay continues to handle all Solana swaps.
Testing
Engineering
SwapperName.Jupiterremain in sourceOperations
🤖 Generated with Claude Code
Summary by CodeRabbit
Release Notes