@flighthq/skeleton2d 0.5.1-next.3302.76ece6b → 0.5.1-next.4304.322dc96

package/README.md

@@ -13,5 +13,5 @@ Import the supported application-facing API from `@flighthq/skeleton2d`. The `@f
 This package is part of the locked-version Flight SDK graph. Applications may instead install and import `@flighthq/sdk` when package-level tree shaking is sufficient.
 
 - [Flight project](https://github.com/flighthq/flight)
-- [Source for @flighthq/skeleton2d@0.5.1-next.3302.76ece6b](https://github.com/flighthq/flight/tree/76ece6bee38e3d07816f341b7f017bfcec3e8e29/packages/skeleton2d)
-- [License](https://github.com/flighthq/flight/blob/76ece6bee38e3d07816f341b7f017bfcec3e8e29/LICENSE.md)
+- [Source for @flighthq/skeleton2d@0.5.1-next.4304.322dc96](https://github.com/flighthq/flight/tree/322dc963fc5a404a31bc0cc612e906de504ea024/packages/skeleton2d)
+- [License](https://github.com/flighthq/flight/blob/322dc963fc5a404a31bc0cc612e906de504ea024/LICENSE.md)

package/dist/enableSkeleton2DGuards.d.ts

@@ -1,3 +1,4 @@
+import type { LogState } from '@flighthq/log/contract';
 export declare function disableSkeleton2DGuards(): void;
 /**
  * Installs the caller-facing skeleton guards (opt-in, dev-only). Idempotent.
@@ -17,6 +18,8 @@ export declare function disableSkeleton2DGuards(): void;
  * The messages live here rather than beside the code that knows the fact, so a shipped build that never
  * imports this module carries neither them nor `@flighthq/log`. Every case also has a `has*`/`explain*`
  * query for a caller that wants the fact as data instead of a log line.
+ *
+ * Warnings go to the caller-owned `logState`, stamped with `nowMs`, the caller's clock reading at enable.
  */
-export declare function enableSkeleton2DGuards(): void;
+export declare function enableSkeleton2DGuards(logState: LogState, nowMs: number): void;
 //# sourceMappingURL=enableSkeleton2DGuards.d.ts.map

package/dist/enableSkeleton2DGuards.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"enableSkeleton2DGuards.d.ts","sourceRoot":"","sources":["../src/enableSkeleton2DGuards.ts"],"names":[],"mappings":"AAOA,wBAAgB,uBAAuB,IAAI,IAAI,CAG9C;AAED;;;;;;;;;;;;;;;;;;GAkBG;AACH,wBAAgB,sBAAsB,IAAI,IAAI,CAG7C"}
+{"version":3,"file":"enableSkeleton2DGuards.d.ts","sourceRoot":"","sources":["../src/enableSkeleton2DGuards.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,QAAQ,EAAE,MAAM,wBAAwB,CAAC;AAQvD,wBAAgB,uBAAuB,IAAI,IAAI,CAG9C;AAED;;;;;;;;;;;;;;;;;;;;GAoBG;AACH,wBAAgB,sBAAsB,CAAC,QAAQ,EAAE,QAAQ,EAAE,KAAK,EAAE,MAAM,GAAG,IAAI,CAG9E"}

package/dist/enableSkeleton2DGuards.js

@@ -24,18 +24,20 @@ export function disableSkeleton2DGuards() {
  * The messages live here rather than beside the code that knows the fact, so a shipped build that never
  * imports this module carries neither them nor `@flighthq/log`. Every case also has a `has*`/`explain*`
  * query for a caller that wants the fact as data instead of a log line.
+ *
+ * Warnings go to the caller-owned `logState`, stamped with `nowMs`, the caller's clock reading at enable.
  */
-export function enableSkeleton2DGuards() {
-    setSkeleton2DCoercedInterpolationGuard(warnOnCoercedInterpolation);
-    setSkeleton2DDeformLengthGuard(warnOnDeformLengthMismatch);
+export function enableSkeleton2DGuards(logState, nowMs) {
+    setSkeleton2DCoercedInterpolationGuard((report) => warnOnCoercedInterpolation(logState, nowMs, report));
+    setSkeleton2DDeformLengthGuard((report) => warnOnDeformLengthMismatch(logState, nowMs, report));
 }
-function warnOnCoercedInterpolation(report) {
-    logOnce(`skeleton2d:coerced-interpolation:${report.subject}`, LogLevel.Warn, {
+function warnOnCoercedInterpolation(logState, nowMs, report) {
+    logOnce(logState, nowMs, `skeleton2d:coerced-interpolation:${report.subject}`, LogLevel.Warn, {
         message: `A ${report.subject} channel states '${report.stated}' interpolation but is walked as '${report.applied}'. The value it carries cannot be blended — an attachment index between two table entries names art nobody authored, and a draw order between two orderings gives fractional sort keys — so the step is forced and the stated easing has no effect. Author the track as '${report.applied}' to say what actually happens, or drive a blendable property instead.`,
     }, 'skeleton2d');
 }
-function warnOnDeformLengthMismatch(report) {
-    logOnce(`skeleton2d:deform-length:${report.subject}`, LogLevel.Warn, {
+function warnOnDeformLengthMismatch(logState, nowMs, report) {
+    logOnce(logState, nowMs, `skeleton2d:deform-length:${report.subject}`, LogLevel.Warn, {
         message: `Deform offsets on '${report.subject}' carry ${report.offsets} values but the vertex stream they address needs ${report.addressed}. The offsets are ignored rather than partly applied: a stream of the wrong length has no correct prefix, so consuming what fits would deform some vertices and silently leave the rest in their setup pose. Re-export the attachment, or check that the deform timeline belongs to this attachment.`,
     }, 'skeleton2d');
 }

package/dist/enableSkeleton2DGuards.js.map

@@ -1 +1 @@
-{"version":3,"file":"enableSkeleton2DGuards.js","sourceRoot":"","sources":["../src/enableSkeleton2DGuards.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,OAAO,EAAE,MAAM,wBAAwB,CAAC;AAEjD,OAAO,EAAE,QAAQ,EAAE,MAAM,0BAA0B,CAAC;AAEpD,OAAO,EAAE,sCAAsC,EAAE,8BAA8B,EAAE,MAAM,uBAAuB,CAAC;AAE/G,6DAA6D;AAC7D,MAAM,UAAU,uBAAuB;IACrC,sCAAsC,CAAC,IAAI,CAAC,CAAC;IAC7C,8BAA8B,CAAC,IAAI,CAAC,CAAC;AACvC,CAAC;AAED;;;;;;;;;;;;;;;;;;GAkBG;AACH,MAAM,UAAU,sBAAsB;IACpC,sCAAsC,CAAC,0BAA0B,CAAC,CAAC;IACnE,8BAA8B,CAAC,0BAA0B,CAAC,CAAC;AAC7D,CAAC;AAED,SAAS,0BAA0B,CAAC,MAAgD;IAClF,OAAO,CACL,oCAAoC,MAAM,CAAC,OAAO,EAAE,EACpD,QAAQ,CAAC,IAAI,EACb;QACE,OAAO,EAAE,KAAK,MAAM,CAAC,OAAO,oBAAoB,MAAM,CAAC,MAAM,qCAAqC,MAAM,CAAC,OAAO,4QAA4Q,MAAM,CAAC,OAAO,wEAAwE;KACnd,EACD,YAAY,CACb,CAAC;AACJ,CAAC;AAED,SAAS,0BAA0B,CAAC,MAAgD;IAClF,OAAO,CACL,4BAA4B,MAAM,CAAC,OAAO,EAAE,EAC5C,QAAQ,CAAC,IAAI,EACb;QACE,OAAO,EAAE,sBAAsB,MAAM,CAAC,OAAO,WAAW,MAAM,CAAC,OAAO,oDAAoD,MAAM,CAAC,SAAS,sSAAsS;KACjb,EACD,YAAY,CACb,CAAC;AACJ,CAAC"}
+{"version":3,"file":"enableSkeleton2DGuards.js","sourceRoot":"","sources":["../src/enableSkeleton2DGuards.ts"],"names":[],"mappings":"AACA,OAAO,EAAE,OAAO,EAAE,MAAM,wBAAwB,CAAC;AAEjD,OAAO,EAAE,QAAQ,EAAE,MAAM,0BAA0B,CAAC;AAEpD,OAAO,EAAE,sCAAsC,EAAE,8BAA8B,EAAE,MAAM,uBAAuB,CAAC;AAE/G,6DAA6D;AAC7D,MAAM,UAAU,uBAAuB;IACrC,sCAAsC,CAAC,IAAI,CAAC,CAAC;IAC7C,8BAA8B,CAAC,IAAI,CAAC,CAAC;AACvC,CAAC;AAED;;;;;;;;;;;;;;;;;;;;GAoBG;AACH,MAAM,UAAU,sBAAsB,CAAC,QAAkB,EAAE,KAAa;IACtE,sCAAsC,CAAC,CAAC,MAAM,EAAE,EAAE,CAAC,0BAA0B,CAAC,QAAQ,EAAE,KAAK,EAAE,MAAM,CAAC,CAAC,CAAC;IACxG,8BAA8B,CAAC,CAAC,MAAM,EAAE,EAAE,CAAC,0BAA0B,CAAC,QAAQ,EAAE,KAAK,EAAE,MAAM,CAAC,CAAC,CAAC;AAClG,CAAC;AAED,SAAS,0BAA0B,CACjC,QAAkB,EAClB,KAAa,EACb,MAAgD;IAEhD,OAAO,CACL,QAAQ,EACR,KAAK,EACL,oCAAoC,MAAM,CAAC,OAAO,EAAE,EACpD,QAAQ,CAAC,IAAI,EACb;QACE,OAAO,EAAE,KAAK,MAAM,CAAC,OAAO,oBAAoB,MAAM,CAAC,MAAM,qCAAqC,MAAM,CAAC,OAAO,4QAA4Q,MAAM,CAAC,OAAO,wEAAwE;KACnd,EACD,YAAY,CACb,CAAC;AACJ,CAAC;AAED,SAAS,0BAA0B,CACjC,QAAkB,EAClB,KAAa,EACb,MAAgD;IAEhD,OAAO,CACL,QAAQ,EACR,KAAK,EACL,4BAA4B,MAAM,CAAC,OAAO,EAAE,EAC5C,QAAQ,CAAC,IAAI,EACb;QACE,OAAO,EAAE,sBAAsB,MAAM,CAAC,OAAO,WAAW,MAAM,CAAC,OAAO,oDAAoD,MAAM,CAAC,SAAS,sSAAsS;KACjb,EACD,YAAY,CACb,CAAC;AACJ,CAAC"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@flighthq/skeleton2d",
-  "version": "0.5.1-next.3302.76ece6b",
+  "version": "0.5.1-next.4304.322dc96",
   "author": "Joshua Granick and other contributors",
   "license": "MIT",
   "repository": {
@@ -38,15 +38,15 @@
     "clean:dist": "tsx ../../scripts/clean-package-dist.ts"
   },
   "dependencies": {
-    "@flighthq/animation": "0.5.1-next.3302.76ece6b",
-    "@flighthq/entity": "0.5.1-next.3302.76ece6b",
-    "@flighthq/geometry": "0.5.1-next.3302.76ece6b",
-    "@flighthq/log": "0.5.1-next.3302.76ece6b",
-    "@flighthq/node": "0.5.1-next.3302.76ece6b",
-    "@flighthq/math": "0.5.1-next.3302.76ece6b",
-    "@flighthq/path": "0.5.1-next.3302.76ece6b",
-    "@flighthq/registry": "0.5.1-next.3302.76ece6b",
-    "@flighthq/types": "0.5.1-next.3302.76ece6b"
+    "@flighthq/animation": "0.5.1-next.4304.322dc96",
+    "@flighthq/entity": "0.5.1-next.4304.322dc96",
+    "@flighthq/geometry": "0.5.1-next.4304.322dc96",
+    "@flighthq/log": "0.5.1-next.4304.322dc96",
+    "@flighthq/node": "0.5.1-next.4304.322dc96",
+    "@flighthq/math": "0.5.1-next.4304.322dc96",
+    "@flighthq/path": "0.5.1-next.4304.322dc96",
+    "@flighthq/registry": "0.5.1-next.4304.322dc96",
+    "@flighthq/types": "0.5.1-next.4304.322dc96"
   },
   "devDependencies": {
     "typescript": "^5.3.0"

package/src/deformPathAttachment2D.test.ts

@@ -1,6 +1,7 @@
 import { allocateEntity, finishEntity } from '@flighthq/entity/contract';
+import { createPath } from '@flighthq/path/contract';
 import type { Bone2D, Path, PathAttachment2D, Skeleton2DDeformLengthMismatch, Skin2D } from '@flighthq/types/contract';
-import { EntityRuntimeKey, PathAttachment2DKind, PathCommand, TransformMode2D } from '@flighthq/types/contract';
+import { PathAttachment2DKind, PathCommand, TransformMode2D } from '@flighthq/types/contract';
 import { describe, expect, it } from 'vitest';
 
 import { deformSkeleton2DPathAttachment } from './deformPathAttachment2D.ts';
@@ -26,7 +27,7 @@ function makeBone(overrides: Partial<Bone2D> = {}): Bone2D {
 }
 
 function emptyPath(): Path {
-  return { [EntityRuntimeKey]: undefined, commands: [], data: [], winding: 'nonZero' };
+  return createPath('nonZero');
 }
 
 function weightedPath(skin: Skin2D, pointCount: number, commands: number[]): PathAttachment2D {

package/src/enableSkeleton2DGuards.test.ts

@@ -1,28 +1,29 @@
-import { clearLogOnceKeys, setLogSink } from '@flighthq/log/contract';
+import type { LogState } from '@flighthq/log/contract';
+import { createLogState, setLogSink } from '@flighthq/log/contract';
 import type { LogEntry } from '@flighthq/types/contract';
 
 import { disableSkeleton2DGuards, enableSkeleton2DGuards } from './enableSkeleton2DGuards.ts';
 import { reportSkeleton2DCoercedInterpolation, reportSkeleton2DDeformLengthMismatch } from './skeleton2dGuards.ts';
 
-// Each case uses its own subject string. `logOnce` keys are process-wide with no reset, which is the
-// behaviour a shipped app wants — one warning per subject, not one per frame — but it means two cases
-// sharing a subject would have the second silently suppressed by the first.
+// Each case uses its own subject string. `logOnce` keys last for the LogState's lifetime, which is the
+// behaviour a shipped app wants — one warning per subject, not one per frame. Every case gets a fresh
+// state here, so no case can be silently suppressed by another.
 const entries: LogEntry[] = [];
+let logState: LogState;
 
 beforeEach(() => {
-  clearLogOnceKeys();
+  logState = createLogState();
   entries.length = 0;
-  setLogSink((entry) => void entries.push(entry));
+  setLogSink(logState, (entry) => void entries.push(entry));
 });
 
 afterEach(() => {
   disableSkeleton2DGuards();
-  setLogSink(null);
 });
 
 describe('disableSkeleton2DGuards', () => {
   it('silences both cases again', () => {
-    enableSkeleton2DGuards();
+    enableSkeleton2DGuards(logState, 0);
     disableSkeleton2DGuards();
 
     reportSkeleton2DCoercedInterpolation('DisabledChannel', 'Linear', 'Step');
@@ -40,7 +41,7 @@ describe('enableSkeleton2DGuards', () => {
   });
 
   it('names what was stated and what actually happens to a coerced channel', () => {
-    enableSkeleton2DGuards();
+    enableSkeleton2DGuards(logState, 0);
 
     reportSkeleton2DCoercedInterpolation('StatedAndApplied', 'Linear', 'Step');
 
@@ -52,7 +53,7 @@ describe('enableSkeleton2DGuards', () => {
   });
 
   it('reports both lengths on a deform mismatch, and why nothing was applied', () => {
-    enableSkeleton2DGuards();
+    enableSkeleton2DGuards(logState, 0);
 
     reportSkeleton2DDeformLengthMismatch('BothLengths', 6, 8);
 
@@ -63,7 +64,7 @@ describe('enableSkeleton2DGuards', () => {
   });
 
   it('keys the coercion message per subject, so two families both get heard', () => {
-    enableSkeleton2DGuards();
+    enableSkeleton2DGuards(logState, 0);
 
     reportSkeleton2DCoercedInterpolation('PerSubjectOne', 'Linear', 'Step');
     reportSkeleton2DCoercedInterpolation('PerSubjectTwo', 'Linear', 'Step');
@@ -74,8 +75,8 @@ describe('enableSkeleton2DGuards', () => {
   });
 
   it('installs one guard however many times it is enabled', () => {
-    enableSkeleton2DGuards();
-    enableSkeleton2DGuards();
+    enableSkeleton2DGuards(logState, 0);
+    enableSkeleton2DGuards(logState, 0);
 
     reportSkeleton2DDeformLengthMismatch('EnabledTwice', 6, 8);