tjdavey / geojson-map-fit-mercator / 34011167263

95%

main: 95%

push

github

Fix broken CommonJS bundle caused by incorrect default-export interop

The @turf/* packages were imported as default imports but never resolved
by Rollup, so it fell back to `interop: 'default'` and assumed each
module's `module.exports` was the default export. Turf v7 ships an ES
module shape (`__esModule: true` with `.default` alongside named
exports), so the CommonJS bundle emitted `polygonToLine(convexHull)`
against the namespace object rather than the function.

Both public exports route through that call, so every CommonJS consumer
failed with `TypeError: polygonToLine is not a function`. The ESM bundle
was unaffected, and the Jest suite runs against the TypeScript sources so
it never exercised the built output.

- Import transformRotate, centroid, polygonToLine and envelope by name,
  matching the other turf imports and removing the interop ambiguity.
- Derive `external` from package.json dependencies. The previous list was
  stale, naming @turf/turf (removed in 90fa5bb) and a
  geojson-minimum-bounding-rectangle package that is not a dependency,
  while omitting everything actually imported. Externalisation is now
  declared rather than relying on the unresolved-module fallback, which
  also clears the "Unresolved dependencies" build warning.
- Add test/smoke.mjs, run via `npm run test:dist` and in CI after the
  build, loading both entry points as a consumer would so a packaging
  fault cannot ship undetected again.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01WLSej5VbHu1WJuvsbgap5F

42 of 48 branches covered (87.5%)

Branch coverage included in aggregate %.

4 of 4 new or added lines in 1 file covered. (100.0%)

106 of 107 relevant lines covered (99.07%)

11.81 hits per line