fix: address jgravois review comments on PR #79

- Remove redundant `!npoints` check now that default is 100
- Add src/ to package.json files array and update DEVELOPING.md
- Fix Browser ESM example: arc.GreatCircle -> GreatCircle, Arc(100) -> Arc()
- Fix CHANGELOG.md indentation in package.json files array

Author: thomas-hervey

DEVELOPING.md

@@ -75,6 +75,7 @@ npm publish   # prepublishOnly runs npm run build automatically
 ### What Gets Published
 
 - `dist/` folder (compiled ESM JS + TypeScript definitions)
+- `src/` folder (TypeScript source files)
 - `README.md`, `LICENSE.md`, `CHANGELOG.md`
 
 ## TypeScript Development

README.md

@@ -40,8 +40,8 @@ const line = gc.Arc(100);
 ```html
 <script type="module">
   import { GreatCircle } from 'https://cdn.skypack.dev/arc@1';
-  const gc = new arc.GreatCircle({x: -122, y: 48}, {x: -77, y: 39});
-  const line = gc.Arc(100);
+  const gc = new GreatCircle({x: -122, y: 48}, {x: -77, y: 39});
+  const line = gc.Arc();
 </script>
 ```
 

package.json

@@ -29,9 +29,10 @@
   },
   "files": [
     "dist/",
+    "src/",
     "README.md",
     "LICENSE.md",
-"CHANGELOG.md"
+    "CHANGELOG.md"
   ],
   "engines": {
     "node": ">=18"

src/great-circle.ts

@@ -97,7 +97,7 @@ export class GreatCircle {
     Arc(npoints: number = 100, _options?: ArcOptions): Arc {
         // NOTE: With npoints ≤ 2, no antimeridian splitting is performed.
         // A 2-point antimeridian route returns a single LineString spanning ±180°. Renderers that support coordinate wrapping (e.g. MapLibre GL JS) handle this correctly, whereas splitting would produce two disconnected straight-line stubs with no great-circle curvature — arguably worse behavior. This is a known limitation; open for maintainer discussion if a MultiLineString split is preferred.
-        if (!npoints || npoints <= 2) {
+        if (npoints <= 2) {
             const arc = new Arc(this.properties);
             const line = new _LineString();
             arc.geometries.push(line);