Skip to content

Rango: Add Sui support#448

Merged
j0ntz merged 1 commit intomasterfrom
jon/rango-sui-support
Mar 28, 2026
Merged

Rango: Add Sui support#448
j0ntz merged 1 commit intomasterfrom
jon/rango-sui-support

Conversation

@j0ntz
Copy link
Copy Markdown
Contributor

@j0ntz j0ntz commented Mar 26, 2026

CHANGELOG

Does this branch warrant an entry to the CHANGELOG?

  • Yes
  • No

Dependencies

none

Description

Initial commit is identical to an external contribution: #406


Note

Medium Risk
Adds a new chain-specific transaction path (SUI) and changes how swap orders are constructed, which could affect swap execution/metadata across providers if the new parsing or shared return structure is incorrect.

Overview
Adds SUI support to the Rango DEX plugin by accepting tx.type: 'SUI' responses, decoding unsignedPtbBase64, and returning a MakeTx-based swap order with standard swap metadata.

Refactors swap-order construction to share common fields (swapOrderBase) and reuse it across EVM/Solana/Cosmos/Sui flows, while preserving the existing spendInfo vs makeTxParams behavior. Updates the CHANGELOG Unreleased section to note Rango SUI support.

Written by Cursor Bugbot for commit e6f81b0. This will update automatically on new commits. Configure here.


@j0ntz j0ntz force-pushed the jon/rango-sui-support branch from 3afa7af to 7b9b651 Compare March 26, 2026 23:15
Copy link
Copy Markdown

@cursor cursor bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 2 potential issues.

Fix All in Cursor

Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Comment thread src/swap/defi/rango.ts
Comment thread src/swap/defi/rango.ts Outdated
@j0ntz j0ntz force-pushed the jon/rango-sui-support branch from cc692f8 to e6f81b0 Compare March 28, 2026 01:08
@j0ntz j0ntz merged commit 98f538c into master Mar 28, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants