Repository navigation
the headline said Binance where the row measures Binance Wallet - #2803
Merged
Merged
Conversation
Two benches rank a row whose provider name is "Binance". What it measures is the self-custody wallet's swap router, one address on BNB, Ethereum and Base, read like any other on-chain trading app in the cohort. On the page that is clear: the provider carries a tag reading "Binance Wallet swap, BNB + Ethereum + Base" beside its name. Every other surface drops the tag and keeps the name alone, and the row currently leads both boards, so the citable API, the share card and any consumer reading the headline got "Binance posts the lowest cost per swap at 172 bps". A reader concludes the exchange is the cheapest venue measured. The measurement says nothing of the kind. Renames the display name on both specs and in the harness's product map. The slug stays "binance" because the product page deliberately covers the exchange's RPC and its wallet together; that is an argument about the slug and never was one about the name. The tag loses the words the name now carries. Found while auditing the bench after a report that its execution cost looked wrong. The cost is not wrong: 165 bps on a wallet router that charges about 50 bps and trades the pairs it trades is unremarkable, and the 1 bp terminal fee on its Base row is already identified in native.go as fee-free Binance Alpha round trips, filtered by DropLoops and documented. The name was the only thing that misread.
This was referenced Oct 4, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
terminal-fill-qualityandterminal-execution-qualityboth rank a provider named Binance. The row measures Binance Wallet's swap router — one address (0xb300…028d) on BNB, Ethereum and Base, read like every other on-chain trading app in the cohort, next to Phantom's in-wallet swap.On the page this is clear: the provider carries a tag,
Binance Wallet swap, BNB + Ethereum + Base, beside its name. Every other surface drops the tag and keeps the name. And the row currently leads both boards, so what ships is:in the citable API, the share card, and anything consuming the headline. A reader concludes the exchange is the cheapest venue measured, on a bench titled "what a swap costs on Axiom, GMGN, FOMO, Photon, Trojan". The measurement supports no such claim.
Renames the display name on both specs and in the harness product map. The slug stays
binance: the product page deliberately covers the exchange's RPC and its wallet together, which is an argument about the slug and never was one about the name. The tag drops the words the name now carries.What this is not
Reported as "we're still wrong on the execution cost benchmark". After auditing it, the cost is not wrong:
native.goas fee-free Binance Alpha round trips, filtered withDropLoops, and documented in the row's note.tfq_rankeduses effective ≥ 100 on pooled rows (phantom 82 unranked, binance 137 ranked, fomo 160 ranked) and the size buckets use their own documented 20/40 floors, stated in the methodology.The 20
tfq_health=0series are all rows under the publish floor — the gate working, not an outage.Left open, deliberately not touched here
score_scope: contested_chains, so the All-chains view ranks products that never meet: on Solana the leader is pump.fun at 213 bps and this row is absent. That is a disclosed design choice and a discussion, not a patch.chain="funding"label exists in the metrics without being a declared dimension value. Unpublished, cosmetic.Verified
pnpm validate235 specs,pnpm test430 pass / 0 fail,go buildand the harness suite clean,gofmtclean.