decision: bring the token-list enrichment tooling from f7a2437 into the repo? #495

Closed
opened 2026-10-07 07:59:55 +02:00 by clawbot · 2 comments
Collaborator

Your commit f7a2437 ("Add token-list enrichment tooling (scripts/)", branch chore/token-list-enrichment) never landed: its three files under scripts/ are not on next, and only the token names and URLs they produced are in src/shared/tokenList.js. The branch was deleted on 2026-09-09; #491 restores it at that commit so the work cannot be lost. Found while pruning branches for #167.

Options:

  • (a) Bring the tooling in under script/ as its own unit, reviewed like any other, so the token list can be regenerated from the repo.
  • (b) Keep the restored branch as reference only.
  • (c) Drop it and delete the branch.

Recommendation: (a). The token list ships in the extension, and today nothing in the repo shows how it was made, the same gap #378 closed for the icons.

Model: opus-5-5

Your commit `f7a2437` ("Add token-list enrichment tooling (scripts/)", branch `chore/token-list-enrichment`) never landed: its three files under `scripts/` are not on `next`, and only the token names and URLs they produced are in `src/shared/tokenList.js`. The branch was deleted on 2026-09-09; https://git.eeqj.de/sneak/AutistMask/pulls/491 restores it at that commit so the work cannot be lost. Found while pruning branches for https://git.eeqj.de/sneak/AutistMask/issues/167. Options: - (a) Bring the tooling in under `script/` as its own unit, reviewed like any other, so the token list can be regenerated from the repo. - (b) Keep the restored branch as reference only. - (c) Drop it and delete the branch. Recommendation: (a). The token list ships in the extension, and today nothing in the repo shows how it was made, the same gap https://git.eeqj.de/sneak/AutistMask/issues/378 closed for the icons. Model: opus-5-5
sneak was assigned by clawbot 2026-10-07 07:59:55 +02:00
Author
Collaborator

Settled as recommended, option (a): it is your own tooling, the token list ships in the extension, and nothing in the repo shows how the list was made. Easy to reverse.

Plan: bring the three files from f7a2437 in under script/ (the repo's entrypoint directory), formatted and linted like the rest of the tree, with no new dependency, and name them in README.md's Entrypoints section. Done when running them on the committed data reproduces the names and URLs now in src/shared/tokenList.js, or each difference is explained on the PR. The branch chore/token-list-enrichment stays until the unit lands.

Model: opus-5-5

Settled as recommended, option (a): it is your own tooling, the token list ships in the extension, and nothing in the repo shows how the list was made. Easy to reverse. Plan: bring the three files from `f7a2437` in under `script/` (the repo's entrypoint directory), formatted and linted like the rest of the tree, with no new dependency, and name them in `README.md`'s Entrypoints section. Done when running them on the committed data reproduces the names and URLs now in `src/shared/tokenList.js`, or each difference is explained on the PR. The branch `chore/token-list-enrichment` stays until the unit lands. Model: opus-5-5
sneak was unassigned by clawbot 2026-10-14 03:59:16 +02:00
clawbot self-assigned this 2026-10-14 03:59:16 +02:00
Author
Collaborator

Built in #531: the two programs and the data file from f7a2437, under script/lib/ and named in the README's Entrypoints section. On the committed data the apply step leaves src/shared/tokenList.js unchanged. One judgement call is open for review there: script/check-censored now also allows the MUSD token's on-chain name in the data file. The branch chore/token-list-enrichment can go once the PR lands.

Model: opus-5-5

Built in https://git.eeqj.de/sneak/AutistMask/pulls/531: the two programs and the data file from `f7a2437`, under `script/lib/` and named in the README's Entrypoints section. On the committed data the apply step leaves `src/shared/tokenList.js` unchanged. One judgement call is open for review there: `script/check-censored` now also allows the MUSD token's on-chain name in the data file. The branch `chore/token-list-enrichment` can go once the PR lands. Model: opus-5-5
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: sneak/AutistMask#495