Wind Extent.polygon() counterclockwise

(re: #296)

This also adds "@esbuild/darwin-arm64" optional dependency

Author: bhousel

package-lock.json

@@ -36,6 +36,9 @@
     "typescript": "^5.7.3",
     "typescript-eslint": "^8.21.0"
   },
+  "optionalDependencies": {
+    "@esbuild/darwin-arm64": "^0.25.6"
+  },
   "scripts": {
     "all": "run-s clean build test",
     "build": "npm run build --workspaces",

package.json

@@ -119,17 +119,17 @@ export class Extent {
     return { minX: this.min[0], minY: this.min[1], maxX: this.max[0], maxY: this.max[1] };
   }
 
-  /** Returns a polygon representing the extent wound clockwise.
+  /** Returns a polygon representing the extent wound counterclockwise.
    * @returns Polygon array
    * @example
-   * new Extent([0, 0], [5, 10]).polygon();  // returns [[0, 0], [0, 10], [5, 10], [5, 0], [0, 0]]
+   * new Extent([0, 0], [5, 10]).polygon();  // returns [[0, 0], [5, 0], [5, 10], [0, 10], [0, 0]]
    */
   polygon(): Vec2[] {
     return [
       [this.min[0], this.min[1]],
-      [this.min[0], this.max[1]],
-      [this.max[0], this.max[1]],
       [this.max[0], this.min[1]],
+      [this.max[0], this.max[1]],
+      [this.min[0], this.max[1]],
       [this.min[0], this.min[1]]
     ];
   }

packages/math/src/Extent.ts

@@ -100,7 +100,7 @@ describe('math/extent', () => {
       const e = new Extent([0, 0], [5, 10]);
       const result = e.polygon();
       assert.ok(result instanceof Array);
-      assert.deepEqual(result, [ [0, 0], [0, 10], [5, 10], [5, 0], [0, 0] ]);
+      assert.deepEqual(result, [[0, 0], [5, 0], [5, 10], [0, 10], [0, 0]]);
     });
   });
 

packages/math/test/Extent.test.js

@@ -281,7 +281,7 @@ describe('math/geom', () => {
       const points = [[0, -1], [5, 1], [10, -1], [15, 1]];
       const ssr = test.geomGetSmallestSurroundingRectangle(points);
       assert.ok(ssr instanceof Object);
-      assert.deepEqual(ssr.poly, [[0, -1], [0, 1], [15, 1], [15, -1], [0, -1]]);
+      assert.deepEqual(ssr.poly, [[0, -1], [15, -1], [15, 1], [0, 1], [0, -1]]);
       assert.equal(ssr.angle, 0);
     });
   });