fix(app): leaderboard label said one thing and measured another - #6
Open
OxToF wants to merge 1 commit into
Open
Conversation
Two corrections to the same two lines. The subtitle claimed "Not sybil-filtered". That was true before wallet_verdicts landed and false since: app/api/leaderboard hides every wallet the scan marked non-HUMAN_LIKE from this view. Right now that is 1874 qualified wallets, 450 hidden, 1424 shown. Publishing a filter claim that the code contradicts is the kind of detail a partner or a grant reviewer checks. The label read "N on-chain wallets", which readers take as "N users". It is neither: it counts wallets with at least one chain-verified action, minus the sybil-hidden ones. "verified testers" says what the number is, and it is the figure that survives scrutiny among the three available (3539 wallets touched a quest, 1874 have an on-chain action, 1424 pass the filter). Locale pinned to en-US on the thousands separator: the UI is English throughout, and a French visitor was getting "1 425" inside an English sentence. Note the sybil scan itself has not run since 2026-07-07, so the hidden set is two weeks stale. Re-running it will move the published number. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
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
1 participant
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.
Two lines in
components/Leaderboard.tsx, both wrong in different ways.The subtitle was factually false
It read "Not sybil-filtered". True before
wallet_verdictslanded, false since:app/api/leaderboardhides every wallet the scan marked non-HUMAN_LIKEfrom this view.Publishing a filtering claim that the code contradicts is the kind of detail a partner or a grant reviewer checks.
The label did not mean what readers read
"N on-chain wallets" is taken as "N users". It is neither: it counts wallets with at least one chain-verified action, minus the sybil-hidden ones.
verified testerssays what the number is. It is also the figure that survives scrutiny among the three available: 3539 wallets have touched a quest, 1874 have an on-chain action, 1424 pass the filter.Also
Locale pinned to
en-USon the thousands separator. The UI is English throughout, and a French visitor was getting1 425inside an English sentence.Verified
Rendered on the Airdrop page, which is where this component actually appears and which renders without a connected wallet. New copy reads:
Consistent with the "Obvious sybil clusters are excluded" line already sitting higher on the same page.
yarn buildpasses.Worth knowing
The sybil scan has not run since 2026-07-07, so the hidden set is two weeks stale. Re-running it will move the published number.
🤖 Generated with Claude Code