Skip to content

Remove unused config options / features#4156

Merged
MartinquaXD merged 4 commits intomainfrom
remove-stuff
Feb 13, 2026
Merged

Remove unused config options / features#4156
MartinquaXD merged 4 commits intomainfrom
remove-stuff

Conversation

@MartinquaXD
Copy link
Contributor

Description

This started out as an investigation into speeding up deny-listed token filtering in the auction building process but I realized that there is still some unfinished work of migrating away from the tracing based bad token detection.

We already moved away from that bad token detection mechanism a while ago but I didn't fully delete all the related code yet in case the new mechanism doesn't work well enough. Since then we've been running the services without the feature for many months so it's time to remove that stuff now.

We are deleting so much here because only the tracing detector was using all the logic to find owners of certain tokens onchain so that also went with it.
Once that was done I also went ahead and tried to find any other args that are unused.

Changes

  • remove tracing_url parameter which made the tracing bad token detector unused
  • removed tracing detector which made token owner finding stuff unused
  • removed token owner finding stuff which made liquidity source configs unused
  • removed any other args that weren't used

How to test

compiler

@MartinquaXD MartinquaXD requested a review from a team as a code owner February 13, 2026 15:44
Copy link
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

No critical issues found.

@MartinquaXD MartinquaXD added this pull request to the merge queue Feb 13, 2026
Merged via the queue into main with commit 95b5017 Feb 13, 2026
19 checks passed
@MartinquaXD MartinquaXD deleted the remove-stuff branch February 13, 2026 17:12
@github-actions github-actions bot locked and limited conversation to collaborators Feb 13, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants