Skip to content

fix: restore map SDK externals in build scripts, bump to v0.4.2#17

Merged
DuncanAForbes merged 1 commit into
mainfrom
fix/restore-externals
Apr 12, 2026
Merged

fix: restore map SDK externals in build scripts, bump to v0.4.2#17
DuncanAForbes merged 1 commit into
mainfrom
fix/restore-externals

Conversation

@DuncanAForbes
Copy link
Copy Markdown
Contributor

@DuncanAForbes DuncanAForbes commented Apr 12, 2026

PR #15 (version bump) accidentally reverted the build script changes from PR #14 — the --external flags for optional map SDKs were lost, causing every build to fail.

This restores:

  • --external mapbox-gl --external @maptiler/sdk --external maplibre-gl --external radar-sdk-js in build/dev scripts
  • Optional peerDependencies for all map SDKs
  • Version bump to 0.4.2 (v0.4.1 tag was already used)

Last one — merge and I'll create the v0.4.2 release.

Summary by CodeRabbit

  • Chores
    • Patched version to 0.4.2
    • Optimised build configuration to support flexible map library integration, allowing optional selection of mapping libraries to suit your project needs.

PR #15 (version bump) accidentally reverted the build script changes
from PR #14 that added --external flags for optional map SDKs.

Restores:
- --external mapbox-gl, @maptiler/sdk, maplibre-gl, radar-sdk-js
- Optional peerDependencies for all map SDKs
- Bumps to v0.4.2 (v0.4.1 tag already used)
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 12, 2026

Caution

Review failed

Pull request was closed or merged during review

Walkthrough

Version bumped from 0.4.1 to 0.4.2. Build and dev scripts updated to treat mapbox-gl, @maptiler/sdk, maplibre-gl, and radar-sdk-js as external dependencies in tsup configuration. These packages added to peerDependencies with version ranges and marked as optional via peerDependenciesMeta.

Changes

Cohort / File(s) Summary
Dependency and Build Configuration
package.json
Version incremented to 0.4.2. Build and dev scripts expanded with external dependency declarations for mapping and location services libraries. Four new packages added to peerDependencies with optional status in peerDependenciesMeta, whilst lucide-react remains non-optional.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 A version hops up, from point-four-one to two,
External friends dance in the build scripts anew,
Peer dependencies shuffle, optional and free,
Maps and location services, wild as can be!
One tiny change, yet ecosystems align—
This rabbit's delighted by dependency design. 🗺️

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately reflects the main changes: restoring map SDK externals in build scripts and bumping the version to 0.4.2, matching the PR's primary objectives.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/restore-externals

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@DuncanAForbes DuncanAForbes merged commit 1437a3f into main Apr 12, 2026
1 of 2 checks passed
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