Skip to content

try secondary route when primary one fails - #466

Open
rouzwelt wants to merge 2 commits into
2026-08-25-tx-task-updatefrom
2026-08-25-try-secondary-route
Open

try secondary route when primary one fails#466
rouzwelt wants to merge 2 commits into
2026-08-25-tx-task-updatefrom
2026-08-25-try-secondary-route

Conversation

@rouzwelt

@rouzwelt rouzwelt commented Aug 25, 2026

Copy link
Copy Markdown
Collaborator

Motivation

Router trade mode only ever tries the primary found route, certain times, a primary route can keep on rejecting while a secondary route might succeed, this PR implements that through finding a new route with excluding the primary route's DEX.

Solution

Checks

By submitting this for review, I'm confirming I've done the following:

  • made this PR as small as possible
  • unit-tested any new functionality
  • linked any relevant issues or PRs
  • included screenshots (if this involves a front-end change)

@rouzwelt rouzwelt self-assigned this Aug 25, 2026
@coderabbitai

coderabbitai Bot commented Aug 25, 2026

Copy link
Copy Markdown
Contributor

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: 500e1558-1aae-468f-8d8d-f7976de88aab

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

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.

1 participant