Skip to content

Update known mints to the top 63 by 12-month trade count - #203

Merged
kaze-cow merged 2 commits into
mainfrom
update-token-list
Oct 13, 2026
Merged

kaze-cow merged 2 commits into
mainfrom
update-token-list

Conversation

@kaze-cow

@kaze-cow kaze-cow commented Oct 13, 2026 •

Copy link
Copy Markdown
Contributor

Description

Replaces the 63 entries in KNOWN_MINTS with the 63 most-traded Solana mints over the last 12 full months, as generated by this Dune query. 33 mints carry over and 30 are swapped.

Motivation

The old list was a one-off snapshot of Jupiter's verified tokens, which picked up a lot of short-lived names (pre-IPO tokens, single-month memecoins). The Dune query ranks by trade count and only keeps mints that traded in every one of the 12 months, so the known-buffer fast path should go to mints that are actually settled consistently. The query link now lives in the file header so the list can be regenerated the same way.

How to test

just test
just bench   # bench-report.json should be unchanged

🤖 Generated with Claude Code

Replace the Jupiter-sourced snapshot with the 63 most-traded Solana mints
over the last 12 full months, filtered for staying power, as generated by
https://dune.com/queries/8905140. 33 mints carry over; 30 are swapped.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@kaze-cow
kaze-cow requested a review from a team as a code owner October 13, 2026 08:28

@fedgiac fedgiac 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.

The tokens are valid and the comment matches the mint. The list looks reasonable at a glance and the way it's generated makes sense. I can't verify that the list matches, I reran the query and it timed out after 60min, apparently it's a heavy one. Anyway, I see it's similar to the previous one and all reasonable tokens are still there, so I think it's fine to merge.

Comment thread bench-report.json
Comment on lines -81 to +84
"settle/pulls_from_multiple_orders": 13366,
"settle/pulls_funds_to_destination": 7277,
"settle/pulls_to_multiple_destinations": 8427,
"settle/pushes_a_single_order": 6124,
"settle/pulls_from_multiple_orders": 13361,
"settle/pulls_funds_to_destination": 7272,
"settle/pulls_to_multiple_destinations": 8422,
"settle/pushes_a_single_order": 6119,

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.

That's interesting! Do you have an explanation for it?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

yea I saw it too. I have no explanation 😅

@kaze-cow
kaze-cow merged commit 6697956 into main Oct 13, 2026
18 checks passed
@kaze-cow
kaze-cow deleted the update-token-list branch October 13, 2026 13:33
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