Skip to content

chore: Upgrade dependencies for v4 - #7919

Merged
camdecoster merged 23 commits into
v4.0from
cam/7918/update-deps-for-v4
Jul 28, 2026
Merged

chore: Upgrade dependencies for v4#7919
camdecoster merged 23 commits into
v4.0from
cam/7918/update-deps-for-v4

Conversation

@camdecoster

@camdecoster camdecoster commented Jul 23, 2026

Copy link
Copy Markdown
Contributor

Description

Update dependencies that can be updated without breaking the library.

Closes #7918.

Changes

  • Update dependencies
  • Update devtools calls per new libraries
  • Refactor calls to color-normalize to instead use Color.normalize, which then points to color-normalize. This will make it easier to remove this dependency in the future.
  • Stop using arguments in favor of ...args to enable newer version of TS
  • Update calls to support new version of maplibre-gl
  • Replace color-rgba with existing color functions
  • Refactor some scripts to module JS to support ESM imports for updated dependencies
  • Replace madge dependency with strongly-connected-components for circular dependency check
  • Update Biome config per update, disabling rule
  • Update tsconfig per TS upgrade, remove dead config options
  • Replace ts-node with tsx
  • Replace prettysize with pretty-bytes
    • This library uses the SI standard for measuring file size (1 KB == 1000 B). The previous library uses the binary standard (1KB == 1024 B).
  • Pin to-px to avoid breaking baseline image generation with kaleido

Testing

  • Check CI for passing tests
  • Load some mocks and see if things look right

@camdecoster camdecoster added the no-draftlog The associated PR doesn't require a draftlog and the CI check will be skipped. label Jul 23, 2026
@camdecoster camdecoster self-assigned this Jul 27, 2026
@camdecoster
camdecoster marked this pull request as ready for review July 27, 2026 19:35
@emilykl

emilykl commented Jul 28, 2026

Copy link
Copy Markdown
Contributor

@camdecoster Any context on the switch from ecstatic to sirv? No objection, just wondering.

@camdecoster

Copy link
Copy Markdown
Contributor Author

ecstatic is no longer maintained and I kept seeing warning messages saying "don't use this" when installing deps with npm ci.

@emilykl emilykl left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! Thanks @camdecoster.

It looks like plotly.min.js has grown by about 7% compared to a fresh build off of v4.0. Not a huge deal but might be worth looking into before a full release. The plotly-map.min.js bundle has grown by a larger % so it's possible MapLibre v5 is a bit larger than v4.

@camdecoster
camdecoster merged commit 0cb10fe into v4.0 Jul 28, 2026
165 of 166 checks passed
@camdecoster
camdecoster deleted the cam/7918/update-deps-for-v4 branch July 28, 2026 20:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-draftlog The associated PR doesn't require a draftlog and the CI check will be skipped.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants