tokenList.js header comment says "511 tokens"; the array has 512 #242

Open
opened 2026-08-12 10:19:51 +02:00 by clawbot · 0 comments
Collaborator

The header comment at the top of src/shared/tokenList.js reads:

// 511 tokens.

TOKENS actually has 512 entries (512 address: keys, 512 unique addresses).
The list went from 511 to 512 in commit aca8c4b ("Add name and url fields to
all 512 tokens in tokenList.js") without the header being updated.

Same class of defect as
#239, which fixed the five
documentation locations; that unit was scoped docs-only and could not touch a
.js file, so the in-code comment was left.

Definition of done

  • The header comment agrees with the length of TOKENS, or states no count
    at all.
  • make check passes.
The header comment at the top of `src/shared/tokenList.js` reads: // 511 tokens. `TOKENS` actually has 512 entries (512 `address:` keys, 512 unique addresses). The list went from 511 to 512 in commit `aca8c4b` ("Add name and url fields to all 512 tokens in tokenList.js") without the header being updated. Same class of defect as https://git.eeqj.de/sneak/AutistMask/issues/239, which fixed the five documentation locations; that unit was scoped docs-only and could not touch a `.js` file, so the in-code comment was left. ## Definition of done - [ ] The header comment agrees with the length of `TOKENS`, or states no count at all. - [ ] `make check` passes.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: sneak/AutistMask#242