Skip to content

fix: normalize imported VS Code theme slugs - #301

Open
PathGao wants to merge 8 commits into
alecdotdev:masterfrom
PathGao:fix/normalize-imported-theme-slugs
Open

fix: normalize imported VS Code theme slugs#301
PathGao wants to merge 8 commits into
alecdotdev:masterfrom
PathGao:fix/normalize-imported-theme-slugs

Conversation

@PathGao

@PathGao PathGao commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

Fixes #187

VS Code theme labels now normalize punctuation runs into one separator before matching the vscodethemes.com URL slug. This makes the documented SynthWave '84 label match the synthwave-84 URL segment.

Depends on #300. Please merge in this order: #286#287#288#293#296#298#300 → this PR.

Validation:

  • Reproduced the mismatch against the upstream SynthWave package manifest (SynthWave '84 versus synthwave-84).
  • npm run check
  • npm test (133 passing)
  • cargo test (21 passing)

@PathGao

PathGao commented Jul 31, 2026

Copy link
Copy Markdown
Contributor Author

@alecdotdev This is the next scoped issue fix in the single chain. It includes a Rust regression test and is based on #300.

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.

Theme import fails

1 participant