Skip to content

Rango: Add Sonic#456

Merged
j0ntz merged 1 commit into
masterfrom
jon/rango-sonic-sui
Jun 15, 2026
Merged

Rango: Add Sonic#456
j0ntz merged 1 commit into
masterfrom
jon/rango-sonic-sui

Conversation

@j0ntz

@j0ntz j0ntz commented Jun 8, 2026

Copy link
Copy Markdown
Contributor

CHANGELOG

Does this branch warrant an entry to the CHANGELOG?

  • Yes
  • No

Dependencies

none

Description

Asana task

Recreates the partner PR with no changes found to be needed:


Note

Low Risk
Mapping-only change with no swap logic or auth changes; same pattern as prior Rango chain additions like SUI.

Overview
Enables Sonic on the Rango DEX aggregator by wiring Rango’s SONIC network to Edge’s sonic plugin in both mapping directions.

scripts/mappings/rangoMappings.ts changes SONIC from null to 'sonic', and the generated src/mappings/rango.ts maps sonicSONIC (was null). The unreleased CHANGELOG notes “(Rango) Sonic support.”

Reviewed by Cursor Bugbot for commit c60bced. Bugbot is set up for automated code reviews on this repo. Configure here.

This was referenced Jun 8, 2026

@cursor cursor Bot left a comment

Copy link
Copy Markdown

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 1 potential issue.

Fix All in Cursor

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

Reviewed by Cursor Bugbot for commit 65da00d. Configure here.

Comment thread src/mappings/rango.ts
@j0ntz j0ntz force-pushed the jon/rango-sonic-sui branch from 65da00d to c4d9620 Compare June 8, 2026 23:24
@j0ntz j0ntz changed the title feat: add sonic to rango (consolidates #438 sui + #439 sonic) Rango: Add Sonic Jun 9, 2026
@j0ntz

j0ntz commented Jun 9, 2026

Copy link
Copy Markdown
Contributor Author

📸 Test evidence — Sonic swap in-app (sim)

pp rango on

pp rango on

nn quote

nn quote

qq preswap

qq preswap

rr slider

rr slider

Captured by the agent's in-app test run (build-and-test).

@peachbits peachbits left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good, my only feedback is about the provided test evidence. Ideally, we test both directions but if I had to choose one it's more appropriate to test selling the new asset since it will go through the more complicated transaction creation flow. Parse the swap partners address, calculate fees, etc.

@j0ntz j0ntz force-pushed the jon/rango-sonic-sui branch from c4d9620 to c60bced Compare June 15, 2026 17:27
@j0ntz j0ntz merged commit 3ca2904 into master Jun 15, 2026
4 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.

2 participants