Skip to content

fix: migrate styles when downloading#96

Merged
gmaclennan merged 4 commits intomainfrom
fix/migrate-downloaded-styles
Mar 20, 2026
Merged

fix: migrate styles when downloading#96
gmaclennan merged 4 commits intomainfrom
fix/migrate-downloaded-styles

Conversation

@gmaclennan
Copy link
Member

StyleDownloader now calls migrate() from @maplibre/maplibre-gl-style-spec before validating styles, allowing v7 styles to be auto-upgraded to v8. Previously, v7 styles would fail validation in StyleDownloader and never reach the Writer (which already handles migration).

Includes StyleDownloader test suite covering migration, constructor, getStyle, getSprites, getGlyphs, getTiles, and source inlining.

StyleDownloader now calls migrate() from @maplibre/maplibre-gl-style-spec
before validating styles, allowing v7 styles to be auto-upgraded to v8.
Previously, v7 styles would fail validation in StyleDownloader and never
reach the Writer (which already handles migration).

Includes StyleDownloader test suite covering migration, constructor,
getStyle, getSprites, getGlyphs, getTiles, and source inlining.
@gmaclennan gmaclennan enabled auto-merge (squash) March 20, 2026 22:05
@gmaclennan gmaclennan self-assigned this Mar 20, 2026
…d-styles

* origin/main:
  feat: exclude locally-rendered glyph ranges requirements (#95)
  feat: add `--dedupe` option to `smp download` CLI command (#97)
  feat: add SMP validator with `validate()` API and `smp validate` CLI (#93)
  chore: remove LFS objects (moved to R2 bucket) (#94)
@gmaclennan gmaclennan merged commit 6ce4486 into main Mar 20, 2026
15 checks passed
@gmaclennan gmaclennan deleted the fix/migrate-downloaded-styles branch March 20, 2026 22:33
This was referenced Mar 20, 2026
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.

1 participant