@flighthq/geometry 0.5.1-next.973.788d3d7 → 0.6.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@flighthq/geometry",
-  "version": "0.5.1-next.973.788d3d7",
+  "version": "0.6.0",
   "author": "Joshua Granick and other contributors",
   "license": "MIT",
   "repository": {
@@ -32,19 +32,20 @@
   "scripts": {
     "build": "tsc -b",
     "clean": "tsc -b --clean",
+    "generate": "tsx ../../scripts/generate-rust-package.ts --crate flight-geometry --package geometry",
+    "generate:check": "tsx ../../scripts/generate-rust-package.ts --crate flight-geometry --package geometry --check",
     "test": "vitest run --config vitest.config.ts",
     "test:watch": "vitest --watch --config vitest.config.ts",
     "prepack": "npm run clean && npm run clean:dist && npm run build",
     "clean:dist": "tsx ../../scripts/clean-package-dist.ts"
   },
   "dependencies": {
-    "@flighthq/entity": "0.5.1-next.973.788d3d7",
-    "@flighthq/log": "0.5.1-next.973.788d3d7",
-    "@flighthq/math": "0.5.1-next.973.788d3d7",
-    "@flighthq/types": "0.5.1-next.973.788d3d7"
+    "@flighthq/entity": "0.6.0",
+    "@flighthq/log": "0.6.0",
+    "@flighthq/math": "0.6.0"
   },
   "devDependencies": {
-    "typescript": "^5.3.0"
+    "typescript": "^7.0.2"
   },
   "description": "2D/3D math primitives: vectors, matrices, rectangles, and object pools",
   "sideEffects": false

package/src/aabb.test.ts

@@ -4,6 +4,7 @@ import {
   createAabb,
   createBoundingSphere,
   createMatrix4,
+  createMatrix4FromValues,
   createVector3,
   expandAabbByPoint,
   expandAabbBySphere,
@@ -22,7 +23,7 @@ import {
   unionAabb,
 } from '@flighthq/geometry/contract';
 
-import { initializeAabb } from './aabb';
+import { initializeAabb } from './aabb.ts';
 
 describe('cloneAabb', () => {
   it('creates an independent copy with independent corner vectors', () => {
@@ -351,7 +352,7 @@ describe('transformAabbByMatrix4', () => {
     const a = createAabb(-1, -1, -1, 1, 1, 1);
     // Column-major 45-degree z rotation.
     const c = Math.SQRT1_2;
-    const m = createMatrix4(c, c, 0, 0, -c, c, 0, 0, 0, 0, 1, 0, 0, 0, 0, 1);
+    const m = createMatrix4FromValues(c, c, 0, 0, -c, c, 0, 0, 0, 0, 1, 0, 0, 0, 0, 1);
     const out = createAabb();
     transformAabbByMatrix4(out, a, m);
     // Rotated unit box spans +/- sqrt(2) in x and y.

package/src/boundingSphere.test.ts

@@ -4,6 +4,7 @@ import {
   createAabb,
   createBoundingSphere,
   createMatrix4,
+  createMatrix4FromValues,
   createVector3,
   containsBoundingSpherePoint,
   getClosestPointOnBoundingSphere,
@@ -15,7 +16,7 @@ import {
   transformBoundingSphereByMatrix4,
 } from '@flighthq/geometry/contract';
 
-import { initializeBoundingSphere } from './boundingSphere';
+import { initializeBoundingSphere } from './boundingSphere.ts';
 
 describe('cloneBoundingSphere', () => {
   it('creates an independent copy with an independent center', () => {
@@ -264,7 +265,7 @@ describe('transformBoundingSphereByMatrix4', () => {
   it('scales the radius by the largest axis scale', () => {
     const s = createBoundingSphere(0, 0, 0, 1);
     // Column-major non-uniform scale diag(2,3,4).
-    const m = createMatrix4(2, 0, 0, 0, 0, 3, 0, 0, 0, 0, 4, 0, 0, 0, 0, 1);
+    const m = createMatrix4FromValues(2, 0, 0, 0, 0, 3, 0, 0, 0, 0, 4, 0, 0, 0, 0, 1);
     const out = createBoundingSphere();
     transformBoundingSphereByMatrix4(out, s, m);
     expect(out.radius).toBeCloseTo(4, 6);
@@ -285,7 +286,7 @@ describe('transformBoundingSphereByMatrix4', () => {
     // move it further from zero and, under a scale below 1, could carry it up past zero into a
     // real sphere that was never there.
     const empty = createBoundingSphere(0, 0, 0, -1);
-    const m = createMatrix4(3, 0, 0, 0, 0, 3, 0, 0, 0, 0, 3, 0, 0, 0, 0, 1);
+    const m = createMatrix4FromValues(3, 0, 0, 0, 0, 3, 0, 0, 0, 0, 3, 0, 0, 0, 0, 1);
     const out = createBoundingSphere();
     transformBoundingSphereByMatrix4(out, empty, m);
     expect(out.radius).toBe(-1);

package/src/capsule.test.ts

@@ -12,7 +12,7 @@ import {
   setCapsule,
 } from '@flighthq/geometry/contract';
 
-import { initializeCapsule } from './capsule';
+import { initializeCapsule } from './capsule.ts';
 
 describe('createCapsule', () => {
   it('stores all fields', () => {

package/src/enableGeometryPoolGuards.test.ts

@@ -1,35 +1,40 @@
+import type { LogState } from '@flighthq/log/contract';
 import {
   addLogSink,
-  clearLogOnceKeys,
+  clearMemoryLogSink,
+  createLogState,
   createMemoryLogSink,
   getMemoryLogSinkEntries,
-  removeLogSink,
 } from '@flighthq/log/contract';
-import type { LogEntry } from '@flighthq/types/contract';
+import type { LogEntry } from '@flighthq/log/contract';
 
 import {
   areGeometryPoolGuardsEnabled,
   disableGeometryPoolGuards,
   enableGeometryPoolGuards,
-} from './enableGeometryPoolGuards';
-import { acquireMatrix3, clearMatrix3Pool, releaseMatrix3 } from './matrix3Pool';
-import { acquireMatrix4, clearMatrix4Pool, releaseMatrix4 } from './matrix4Pool';
-import { acquireMatrix, clearMatrixPool, releaseMatrix } from './matrixPool';
-import { acquireQuaternion, clearQuaternionPool, releaseQuaternion } from './quaternionPool';
-import { acquireRectangle, clearRectanglePool, releaseRectangle } from './rectanglePool';
-import { acquireVector2, clearVector2Pool, releaseVector2 } from './vector2Pool';
-import { acquireVector3, clearVector3Pool, releaseVector3 } from './vector3Pool';
-import { acquireVector4, clearVector4Pool, releaseVector4 } from './vector4Pool';
+} from './enableGeometryPoolGuards.ts';
+import { acquireMatrix3, clearMatrix3Pool, releaseMatrix3 } from './matrix3Pool.ts';
+import { acquireMatrix4, clearMatrix4Pool, releaseMatrix4 } from './matrix4Pool.ts';
+import { acquireMatrix, clearMatrixPool, releaseMatrix } from './matrixPool.ts';
+import { acquireQuaternion, clearQuaternionPool, releaseQuaternion } from './quaternionPool.ts';
+import { acquireRectangle, clearRectanglePool, releaseRectangle } from './rectanglePool.ts';
+import { acquireVector2, clearVector2Pool, releaseVector2 } from './vector2Pool.ts';
+import { acquireVector3, clearVector3Pool, releaseVector3 } from './vector3Pool.ts';
+import { acquireVector4, clearVector4Pool, releaseVector4 } from './vector4Pool.ts';
 
+// Each test gets its own caller-owned log state, so once-keys never leak between tests.
 function captureLog(run: () => void): readonly LogEntry[] {
   const sink = createMemoryLogSink(16);
-  addLogSink(sink.sink);
-  try {
-    run();
-    return getMemoryLogSinkEntries(sink);
-  } finally {
-    removeLogSink(sink.sink);
-  }
+  addLogSink(logState, sink.sink);
+  run();
+  const entries: LogEntry[] = [];
+  getMemoryLogSinkEntries(entries, sink);
+  clearMemoryLogSink(sink);
+  return entries;
+}
+
+function clock(): number {
+  return 0;
 }
 
 function clearPools(): void {
@@ -55,7 +60,7 @@ function messageOf(entry: Readonly<LogEntry>): string {
 }
 
 beforeEach(() => {
-  clearLogOnceKeys();
+  logState = createLogState();
   clearPools();
 });
 
@@ -67,7 +72,7 @@ afterEach(() => {
 describe('areGeometryPoolGuardsEnabled', () => {
   it('reports whether the pool release diagnostic is installed', () => {
     expect(areGeometryPoolGuardsEnabled()).toBe(false);
-    enableGeometryPoolGuards();
+    enableGeometryPoolGuards(logState, clock);
     expect(areGeometryPoolGuardsEnabled()).toBe(true);
     disableGeometryPoolGuards();
     expect(areGeometryPoolGuardsEnabled()).toBe(false);
@@ -76,7 +81,7 @@ describe('areGeometryPoolGuardsEnabled', () => {
 
 describe('disableGeometryPoolGuards', () => {
   it('restores the production-default silent release path', () => {
-    enableGeometryPoolGuards();
+    enableGeometryPoolGuards(logState, clock);
     disableGeometryPoolGuards();
     const entries = captureLog(() => doubleRelease(acquireVector2, releaseVector2));
     expect(entries).toHaveLength(0);
@@ -84,9 +89,9 @@ describe('disableGeometryPoolGuards', () => {
 });
 
 describe('enableGeometryPoolGuards', () => {
-  // ONE firing test deliberately consumes all eight process-lifetime logOnce keys.
+  // ONE firing test deliberately consumes all eight logOnce keys of this test's log state.
   it('warns once for a double release from every geometry pool and stays silent for valid brackets', () => {
-    enableGeometryPoolGuards();
+    enableGeometryPoolGuards(logState, clock);
 
     const pairedEntries = captureLog(() => {
       releaseMatrix(acquireMatrix());
@@ -128,3 +133,5 @@ describe('enableGeometryPoolGuards', () => {
     }
   });
 });
+
+let logState: LogState = createLogState();

package/src/frustum.test.ts

@@ -3,6 +3,7 @@ import {
   createBoundingSphere,
   createFrustum,
   createMatrix4,
+  createMatrix4FromValues,
   createPerspectiveMatrix4,
   createVector3,
   getFrustumCorners,
@@ -16,9 +17,9 @@ import {
   setFrustumFromMatrix4,
   setMatrix4LookAt,
 } from '@flighthq/geometry/contract';
-import type { Matrix4 } from '@flighthq/types/contract';
 
-import { initializeFrustum } from './frustum';
+import { initializeFrustum } from './frustum.ts';
+import type { Matrix4 } from './types.ts';
 
 // View-projection for a camera at (0,0,5) looking toward the origin down -z.
 function createTestViewProjection(): Matrix4 {
@@ -89,7 +90,7 @@ describe('getFrustumCorners', () => {
     // point at infinity. Passing the coordinates through undivided keeps the corner finite, so a
     // caller fitting bounds around these corners gets a wrong box rather than an infinite one.
     const corners = Array.from({ length: 8 }, () => createVector3());
-    getFrustumCorners(corners, createMatrix4(1, 0, 0, 0, 0, 1, 0, 0, 0, 0, 1, 0, 0, 0, 0, 0));
+    getFrustumCorners(corners, createMatrix4FromValues(1, 0, 0, 0, 0, 1, 0, 0, 0, 0, 1, 0, 0, 0, 0, 0));
     for (const corner of corners) {
       expect(Number.isFinite(corner.x)).toBe(true);
       expect(Number.isFinite(corner.y)).toBe(true);
@@ -219,7 +220,7 @@ describe('setFrustumFromMatrix4', () => {
     // culls against this frustum gets consistent rejections instead of NaN comparisons, which are
     // false in both directions and would let a box pass and fail the same test.
     const f = createFrustum();
-    setFrustumFromMatrix4(f, createMatrix4(0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0));
+    setFrustumFromMatrix4(f, createMatrix4FromValues(0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0));
     for (const plane of [f.near, f.far, f.left, f.right, f.top, f.bottom]) {
       expect(Number.isFinite(plane.a)).toBe(true);
       expect(Number.isFinite(plane.b)).toBe(true);

package/src/geometryPoolGuards.test.ts

@@ -1,9 +1,22 @@
-import { geometryPoolReleaseGuard, setGeometryPoolReleaseGuard } from './geometryPoolGuards';
+import {
+  geometryPoolReleaseGuard,
+  getGeometryPoolReleaseGuard,
+  setGeometryPoolReleaseGuard,
+} from './geometryPoolGuards.ts';
 
 afterEach(() => {
   setGeometryPoolReleaseGuard(null);
 });
 
+describe('getGeometryPoolReleaseGuard', () => {
+  it('reads the installed release callback, or null when none is installed', () => {
+    expect(getGeometryPoolReleaseGuard()).toBeNull();
+    const guard = (): void => {};
+    setGeometryPoolReleaseGuard(guard);
+    expect(getGeometryPoolReleaseGuard()).toBe(guard);
+  });
+});
+
 describe('setGeometryPoolReleaseGuard', () => {
   it('installs and removes the internal release callback', () => {
     const guard = (): void => {};

package/src/matrix.test.ts

@@ -41,9 +41,9 @@ import {
   translateMatrixByVectorXY,
   writeMatrixToFloat32Array,
 } from '@flighthq/geometry/contract';
-import type { Matrix, Matrix3Like, Matrix4Like } from '@flighthq/types/contract';
 
-import { initializeMatrix } from './matrix';
+import { initializeMatrix } from './matrix.ts';
+import type { Matrix, Matrix3Like, Matrix4Like } from './types.ts';
 
 describe('cloneMatrix', () => {
   it('should clone the matrix3x2 correctly', () => {