@flighthq/node 0.3.1-next.625.675880d → 0.3.1-next.704.69286f6

package/dist/nodeColorAdjustment.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"nodeColorAdjustment.d.ts","sourceRoot":"","sources":["../src/nodeColorAdjustment.ts"],"names":[],"mappings":"AAQA,OAAO,KAAK,EAAE,UAAU,EAA0B,IAAI,EAAe,MAAM,0BAA0B,CAAC;AAKtG;;;;;GAKG;AACH,wBAAgB,sBAAsB,CAAC,MAAM,SAAS,MAAM,EAAE,MAAM,EAAE,IAAI,CAAC,MAAM,CAAC,EAAE,UAAU,EAAE,UAAU,GAAG,IAAI,CAMhH;AAED,wFAAwF;AACxF,wBAAgB,uBAAuB,CAAC,MAAM,SAAS,MAAM,EAC3D,MAAM,EAAE,QAAQ,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC,GAC7B,SAAS,UAAU,EAAE,GAAG,IAAI,CAE9B;AAED;;;;;GAKG;AACH,wBAAgB,uBAAuB,CAAC,MAAM,SAAS,MAAM,EAC3D,MAAM,EAAE,IAAI,CAAC,MAAM,CAAC,EACpB,KAAK,EAAE,SAAS,UAAU,EAAE,GAAG,IAAI,GAClC,IAAI,CAKN;AAED,wEAAwE;AACxE,wBAAgB,2BAA2B,CAAC,MAAM,SAAS,MAAM,EAAE,MAAM,EAAE,IAAI,CAAC,MAAM,CAAC,EAAE,IAAI,EAAE,MAAM,GAAG,IAAI,CAE3G"}
+{"version":3,"file":"nodeColorAdjustment.d.ts","sourceRoot":"","sources":["../src/nodeColorAdjustment.ts"],"names":[],"mappings":"AAOA,OAAO,KAAK,EAAE,UAAU,EAA0B,IAAI,EAAe,MAAM,0BAA0B,CAAC;AAKtG;;;;;GAKG;AACH,wBAAgB,sBAAsB,CAAC,MAAM,SAAS,MAAM,EAAE,MAAM,EAAE,IAAI,CAAC,MAAM,CAAC,EAAE,UAAU,EAAE,UAAU,GAAG,IAAI,CAMhH;AAED,wFAAwF;AACxF,wBAAgB,uBAAuB,CAAC,MAAM,SAAS,MAAM,EAC3D,MAAM,EAAE,QAAQ,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC,GAC7B,SAAS,UAAU,EAAE,GAAG,IAAI,CAE9B;AAED;;;;;GAKG;AACH,wBAAgB,uBAAuB,CAAC,MAAM,SAAS,MAAM,EAC3D,MAAM,EAAE,IAAI,CAAC,MAAM,CAAC,EACpB,KAAK,EAAE,SAAS,UAAU,EAAE,GAAG,IAAI,GAClC,IAAI,CAKN;AAED,wEAAwE;AACxE,wBAAgB,2BAA2B,CAAC,MAAM,SAAS,MAAM,EAAE,MAAM,EAAE,IAAI,CAAC,MAAM,CAAC,EAAE,IAAI,EAAE,MAAM,GAAG,IAAI,CAE3G"}

package/dist/nodeColorAdjustment.js

@@ -1,4 +1,4 @@
-import { COLOR_ADJUSTMENT_CHANNEL_MIXING, COLOR_ADJUSTMENT_NONE, createTintAdjustment, resolveColorAdjustmentsColorMatrix, resolveColorAdjustmentsColorScaleBias, } from '@flighthq/adjustments/contract';
+import { COLOR_ADJUSTMENT_CHANNEL_MIXING, createTintAdjustment, resolveColorAdjustmentsColorMatrix, resolveColorAdjustmentsColorScaleBias, } from '@flighthq/adjustments/contract';
 import { createColorScaleBias } from '@flighthq/materials/contract';
 import { getNodeRuntime } from './node';
 import { invalidateNodeAppearance } from './revision';
@@ -45,12 +45,6 @@ function resolveNodeColorAdjustments(runtime) {
     }
     const out = runtime.resolvedColorScaleBias ?? createColorScaleBias();
     const status = resolveColorAdjustmentsColorScaleBias(adjustments, out);
-    if (status === COLOR_ADJUSTMENT_NONE) {
-        runtime.resolvedColorScaleBias = null;
-        runtime.resolvedColorMatrix = null;
-        runtime.colorAdjustmentsUnsupported = false;
-        return;
-    }
     runtime.resolvedColorScaleBias = out;
     runtime.resolvedColorMatrix =
         status === COLOR_ADJUSTMENT_CHANNEL_MIXING ? resolveColorAdjustmentsColorMatrix(adjustments) : null;

package/dist/nodeColorAdjustment.js.map

@@ -1 +1 @@
-{"version":3,"file":"nodeColorAdjustment.js","sourceRoot":"","sources":["../src/nodeColorAdjustment.ts"],"names":[],"mappings":"AAAA,OAAO,EACL,+BAA+B,EAC/B,qBAAqB,EACrB,oBAAoB,EACpB,kCAAkC,EAClC,qCAAqC,GACtC,MAAM,gCAAgC,CAAC;AACxC,OAAO,EAAE,oBAAoB,EAAE,MAAM,8BAA8B,CAAC;AAGpE,OAAO,EAAE,cAAc,EAAE,MAAM,QAAQ,CAAC;AACxC,OAAO,EAAE,wBAAwB,EAAE,MAAM,YAAY,CAAC;AAEtD;;;;;GAKG;AACH,MAAM,UAAU,sBAAsB,CAAwB,MAAoB,EAAE,UAAsB;IACxG,MAAM,OAAO,GAAG,cAAc,CAAC,MAAM,CAAwB,CAAC;IAC9D,MAAM,OAAO,GAAG,OAAO,CAAC,gBAAgB,CAAC;IACzC,OAAO,CAAC,gBAAgB,GAAG,OAAO,KAAK,IAAI,CAAC,CAAC,CAAC,CAAC,UAAU,CAAC,CAAC,CAAC,CAAC,CAAC,GAAG,OAAO,EAAE,UAAU,CAAC,CAAC;IACtF,2BAA2B,CAAC,OAAO,CAAC,CAAC;IACrC,wBAAwB,CAAC,MAAM,CAAC,CAAC;AACnC,CAAC;AAED,wFAAwF;AACxF,MAAM,UAAU,uBAAuB,CACrC,MAA8B;IAE9B,OAAO,cAAc,CAAC,MAAM,CAAC,CAAC,gBAAgB,CAAC;AACjD,CAAC;AAED;;;;;GAKG;AACH,MAAM,UAAU,uBAAuB,CACrC,MAAoB,EACpB,KAAmC;IAEnC,MAAM,OAAO,GAAG,cAAc,CAAC,MAAM,CAAwB,CAAC;IAC9D,OAAO,CAAC,gBAAgB,GAAG,KAAK,CAAC;IACjC,2BAA2B,CAAC,OAAO,CAAC,CAAC;IACrC,wBAAwB,CAAC,MAAM,CAAC,CAAC;AACnC,CAAC;AAED,wEAAwE;AACxE,MAAM,UAAU,2BAA2B,CAAwB,MAAoB,EAAE,IAAY;IACnG,uBAAuB,CAAC,MAAM,EAAE,CAAC,oBAAoB,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;AAChE,CAAC;AAED,SAAS,2BAA2B,CAAC,OAA+B;IAClE,MAAM,WAAW,GAAG,OAAO,CAAC,gBAAgB,CAAC;IAC7C,IAAI,WAAW,KAAK,IAAI,IAAI,WAAW,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;QACrD,OAAO,CAAC,sBAAsB,GAAG,IAAI,CAAC;QACtC,OAAO,CAAC,mBAAmB,GAAG,IAAI,CAAC;QACnC,OAAO,CAAC,2BAA2B,GAAG,KAAK,CAAC;QAC5C,OAAO;IACT,CAAC;IACD,MAAM,GAAG,GAAG,OAAO,CAAC,sBAAsB,IAAI,oBAAoB,EAAE,CAAC;IACrE,MAAM,MAAM,GAAG,qCAAqC,CAAC,WAAW,EAAE,GAAG,CAAC,CAAC;IACvE,IAAI,MAAM,KAAK,qBAAqB,EAAE,CAAC;QACrC,OAAO,CAAC,sBAAsB,GAAG,IAAI,CAAC;QACtC,OAAO,CAAC,mBAAmB,GAAG,IAAI,CAAC;QACnC,OAAO,CAAC,2BAA2B,GAAG,KAAK,CAAC;QAC5C,OAAO;IACT,CAAC;IACD,OAAO,CAAC,sBAAsB,GAAG,GAAG,CAAC;IACrC,OAAO,CAAC,mBAAmB;QACzB,MAAM,KAAK,+BAA+B,CAAC,CAAC,CAAC,kCAAkC,CAAC,WAAW,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC;IACtG,OAAO,CAAC,2BAA2B;QACjC,MAAM,KAAK,+BAA+B,IAAI,OAAO,CAAC,mBAAmB,KAAK,IAAI,CAAC;AACvF,CAAC"}
+{"version":3,"file":"nodeColorAdjustment.js","sourceRoot":"","sources":["../src/nodeColorAdjustment.ts"],"names":[],"mappings":"AAAA,OAAO,EACL,+BAA+B,EAC/B,oBAAoB,EACpB,kCAAkC,EAClC,qCAAqC,GACtC,MAAM,gCAAgC,CAAC;AACxC,OAAO,EAAE,oBAAoB,EAAE,MAAM,8BAA8B,CAAC;AAGpE,OAAO,EAAE,cAAc,EAAE,MAAM,QAAQ,CAAC;AACxC,OAAO,EAAE,wBAAwB,EAAE,MAAM,YAAY,CAAC;AAEtD;;;;;GAKG;AACH,MAAM,UAAU,sBAAsB,CAAwB,MAAoB,EAAE,UAAsB;IACxG,MAAM,OAAO,GAAG,cAAc,CAAC,MAAM,CAAwB,CAAC;IAC9D,MAAM,OAAO,GAAG,OAAO,CAAC,gBAAgB,CAAC;IACzC,OAAO,CAAC,gBAAgB,GAAG,OAAO,KAAK,IAAI,CAAC,CAAC,CAAC,CAAC,UAAU,CAAC,CAAC,CAAC,CAAC,CAAC,GAAG,OAAO,EAAE,UAAU,CAAC,CAAC;IACtF,2BAA2B,CAAC,OAAO,CAAC,CAAC;IACrC,wBAAwB,CAAC,MAAM,CAAC,CAAC;AACnC,CAAC;AAED,wFAAwF;AACxF,MAAM,UAAU,uBAAuB,CACrC,MAA8B;IAE9B,OAAO,cAAc,CAAC,MAAM,CAAC,CAAC,gBAAgB,CAAC;AACjD,CAAC;AAED;;;;;GAKG;AACH,MAAM,UAAU,uBAAuB,CACrC,MAAoB,EACpB,KAAmC;IAEnC,MAAM,OAAO,GAAG,cAAc,CAAC,MAAM,CAAwB,CAAC;IAC9D,OAAO,CAAC,gBAAgB,GAAG,KAAK,CAAC;IACjC,2BAA2B,CAAC,OAAO,CAAC,CAAC;IACrC,wBAAwB,CAAC,MAAM,CAAC,CAAC;AACnC,CAAC;AAED,wEAAwE;AACxE,MAAM,UAAU,2BAA2B,CAAwB,MAAoB,EAAE,IAAY;IACnG,uBAAuB,CAAC,MAAM,EAAE,CAAC,oBAAoB,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;AAChE,CAAC;AAED,SAAS,2BAA2B,CAAC,OAA+B;IAClE,MAAM,WAAW,GAAG,OAAO,CAAC,gBAAgB,CAAC;IAC7C,IAAI,WAAW,KAAK,IAAI,IAAI,WAAW,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;QACrD,OAAO,CAAC,sBAAsB,GAAG,IAAI,CAAC;QACtC,OAAO,CAAC,mBAAmB,GAAG,IAAI,CAAC;QACnC,OAAO,CAAC,2BAA2B,GAAG,KAAK,CAAC;QAC5C,OAAO;IACT,CAAC;IACD,MAAM,GAAG,GAAG,OAAO,CAAC,sBAAsB,IAAI,oBAAoB,EAAE,CAAC;IACrE,MAAM,MAAM,GAAG,qCAAqC,CAAC,WAAW,EAAE,GAAG,CAAC,CAAC;IACvE,OAAO,CAAC,sBAAsB,GAAG,GAAG,CAAC;IACrC,OAAO,CAAC,mBAAmB;QACzB,MAAM,KAAK,+BAA+B,CAAC,CAAC,CAAC,kCAAkC,CAAC,WAAW,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC;IACtG,OAAO,CAAC,2BAA2B;QACjC,MAAM,KAAK,+BAA+B,IAAI,OAAO,CAAC,mBAAmB,KAAK,IAAI,CAAC;AACvF,CAAC"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@flighthq/node",
-  "version": "0.3.1-next.625.675880d",
+  "version": "0.3.1-next.704.69286f6",
   "author": "Joshua Granick and other contributors",
   "license": "MIT",
   "repository": {
@@ -38,12 +38,12 @@
     "clean:dist": "tsx ../../scripts/clean-package-dist.ts"
   },
   "dependencies": {
-    "@flighthq/adjustments": "0.3.1-next.625.675880d",
-    "@flighthq/entity": "0.3.1-next.625.675880d",
-    "@flighthq/geometry": "0.3.1-next.625.675880d",
-    "@flighthq/materials": "0.3.1-next.625.675880d",
-    "@flighthq/signals": "0.3.1-next.625.675880d",
-    "@flighthq/types": "0.3.1-next.625.675880d"
+    "@flighthq/adjustments": "0.3.1-next.704.69286f6",
+    "@flighthq/entity": "0.3.1-next.704.69286f6",
+    "@flighthq/geometry": "0.3.1-next.704.69286f6",
+    "@flighthq/materials": "0.3.1-next.704.69286f6",
+    "@flighthq/signals": "0.3.1-next.704.69286f6",
+    "@flighthq/types": "0.3.1-next.704.69286f6"
   },
   "devDependencies": {
     "typescript": "^5.3.0"

package/src/boundsRectangle.test.ts

@@ -42,8 +42,10 @@ import { initBoundsRectangleRuntimeTrait, initBoundsRectangleTrait } from './has
 
 function getEntityRuntime(
   source: TestNode,
-): NodeRuntime<HasBoundsRectangle & HasTransform2D> & HasBoundsRectangleRuntime {
-  return _getRuntime(source) as NodeRuntime<HasBoundsRectangle & HasTransform2D> & HasBoundsRectangleRuntime;
+): NodeRuntime<HasBoundsRectangle & HasTransform2D> & HasBoundsRectangleRuntime & HasTransform2DRuntime {
+  return _getRuntime(source) as NodeRuntime<HasBoundsRectangle & HasTransform2D> &
+    HasBoundsRectangleRuntime &
+    HasTransform2DRuntime;
 }
 
 function createTestNode(): TestNode {
@@ -90,6 +92,14 @@ describe('computeNodeBoundsRectangle', () => {
     expect(equalsRectangle(out, { x: 5, y: 5, width: 100, height: 100 })).toBe(true);
   });
 
+  it('defaults a null target coordinate space to the source', () => {
+    const out = createRectangle();
+
+    computeNodeBoundsRectangle(out, child, null);
+
+    expect(equalsRectangle(out, { x: 5, y: 5, width: 100, height: 100 })).toBe(true);
+  });
+
   it('should compute bounds relative to parent', () => {
     const out = createRectangle();
     computeNodeBoundsRectangle(out, child, root);
@@ -99,6 +109,14 @@ describe('computeNodeBoundsRectangle', () => {
     expect(out.height).toBeCloseTo(100);
   });
 
+  it('uses parent-space bounds directly when the parent is nested', () => {
+    const out = createRectangle();
+
+    computeNodeBoundsRectangle(out, grandChild, child);
+
+    expect(equalsRectangle(out, getNodeParentBoundsRectangle(grandChild))).toBe(true);
+  });
+
   it('should compute bounds relative to nested child', () => {
     const out = createRectangle();
     computeNodeBoundsRectangle(out, root, grandChild);
@@ -153,6 +171,17 @@ describe('computeNodeBoundsRectangle', () => {
     expect(equalsRectangle(out, getNodeWorldBoundsRectangle(child))).toBe(true);
   });
 
+  it('converts through world space for a nested unrelated target', () => {
+    const unrelatedRoot = createTestNode();
+    const unrelatedTarget = createTestNode();
+    addNodeChild(unrelatedRoot, unrelatedTarget);
+    const out = createRectangle();
+
+    computeNodeBoundsRectangle(out, grandChild, unrelatedTarget);
+
+    expect(equalsRectangle(out, getNodeWorldBoundsRectangle(grandChild))).toBe(true);
+  });
+
   it('should return world bounds if the target coordinate space is root', () => {
     const out = createRectangle();
     computeNodeBoundsRectangle(out, child, root);
@@ -197,6 +226,39 @@ describe('computeNodeRootLocalBoundsRectangle', () => {
     expect(out.width).toBeCloseTo(18);
     expect(out.height).toBeCloseTo(24);
   });
+
+  it('composes transforms at every descendant depth', () => {
+    const root = createTestNode();
+    const child = createTestNode();
+    const grandchild = createTestNode();
+    setRectangle(getNodeLocalBoundsRectangle(root), 0, 0, 1, 1);
+    setRectangle(getNodeLocalBoundsRectangle(child), 0, 0, 1, 1);
+    setRectangle(getNodeLocalBoundsRectangle(grandchild), 0, 0, 2, 2);
+    child.x = 10;
+    grandchild.x = 20;
+    addNodeChild(root, child);
+    addNodeChild(child, grandchild);
+
+    const out = createRectangle();
+    computeNodeRootLocalBoundsRectangle(out, root);
+
+    expect(out).toMatchObject({ x: 0, y: 0, width: 32, height: 2 });
+  });
+
+  it('excludes disabled descendants', () => {
+    const root = createTestNode();
+    const child = createTestNode();
+    setRectangle(getNodeLocalBoundsRectangle(root), 0, 0, 10, 10);
+    setRectangle(getNodeLocalBoundsRectangle(child), 0, 0, 20, 20);
+    child.x = 100;
+    child.enabled = false;
+    addNodeChild(root, child);
+
+    const out = createRectangle();
+    computeNodeRootLocalBoundsRectangle(out, root);
+
+    expect(out).toMatchObject({ x: 0, y: 0, width: 10, height: 10 });
+  });
 });
 
 describe('ensureNodeLocalBoundsRectangle', () => {
@@ -316,6 +378,20 @@ describe('ensureNodeWorldBoundsRectangle', () => {
     expect(runtime.worldBoundsRectangle).not.toBeNull();
   });
 
+  it('recomputes when bounds are cached before the world matrix exists', () => {
+    const object = createTestNode();
+    const runtime = getEntityRuntime(object);
+    getNodeLocalBoundsRectangle(object);
+    runtime.worldBoundsRectangle = createRectangle(5, 5, 10, 10);
+    runtime.worldBoundsUsingLocalBoundsId = runtime.localBoundsId;
+    expect(runtime.worldMatrix).toBeNull();
+
+    ensureNodeWorldBoundsRectangle(object);
+
+    expect(runtime.worldMatrix).not.toBeNull();
+    expect(runtime.worldBoundsRectangle).toMatchObject({ x: 0, y: 0, width: 0, height: 0 });
+  });
+
   it('should not recalculate if localBoundsId and worldTransformId are unchanged', () => {
     const object = createTestNode();
     const runtime = getEntityRuntime(object);
@@ -500,6 +576,20 @@ describe('getNodeWorldBoundsRectangle', () => {
     expect(bounds.width).toBeGreaterThan(10);
     expect(bounds.height).toBeGreaterThan(10);
   });
+
+  it('ignores an enabled child with zero-area world bounds', () => {
+    const parent = createTestNode();
+    const child = createTestNode();
+    addNodeChild(parent, child);
+    setRectangle(getNodeLocalBoundsRectangle(parent) as Rectangle, 0, 0, 10, 10);
+    setRectangle(getNodeLocalBoundsRectangle(child) as Rectangle, 0, 0, 0, 5);
+    child.x = 100;
+    invalidateNodeLocalTransform(child);
+
+    const bounds = getNodeWorldBoundsRectangle(parent);
+
+    expect(bounds).toMatchObject({ x: 0, y: 0, width: 10, height: 10 });
+  });
 });
 
 describe('setNodeHeight', () => {

package/src/hierarchy.test.ts

@@ -67,6 +67,13 @@ describe('addNodeChild', () => {
     expect(() => addNodeChild(container, container)).toThrow(TypeError);
   });
 
+  it('rejects a child disallowed by the parent runtime', () => {
+    getEntityRuntime(container).canAddChild = () => false;
+
+    expect(() => addNodeChild(container, childA)).toThrow('The specified parent object cannot add this child');
+    expect(getNodeParent(childA)).toBeNull();
+  });
+
   it('removes child from previous parent before adding', () => {
     const other = createNode(NodeKind);
 
@@ -222,6 +229,15 @@ describe('addNodeChildAt', () => {
     expect(getNodeParentReferenceRevision(childA)).toBe(parentReferenceId);
   });
 
+  it('repairs a stale parent reference whose child-list entry is absent', () => {
+    getEntityRuntime(childA).parent = container;
+
+    addNodeChildAt(container, childA, 0);
+
+    expect(getNodeChildAt(container, 0)).toBe(childA);
+    expect(getNodeParent(childA)).toBe(container);
+  });
+
   it('calls onParentChanged on the child', () => {
     let called = false;
     connectSignal(enableNodeSignals(childA).onParentChanged, () => {
@@ -531,6 +547,16 @@ describe('removeNodeChild', () => {
     expect(getNodeParent(childA)).toBeNull();
   });
 
+  it('clears a stale parent reference when the child-list entry is absent', () => {
+    getEntityRuntime(container).children = [];
+    getEntityRuntime(childA).parent = container;
+
+    removeNodeChild(container, childA);
+
+    expect(getNodeParent(childA)).toBeNull();
+    expect(getNodeChildCount(container)).toBe(0);
+  });
+
   it('calls onParentChanged on the child', () => {
     addNodeChild(container, childA);
     let called = false;
@@ -605,6 +631,11 @@ describe('removeNodeChildAt', () => {
 });
 
 describe('removeNodeChildren', () => {
+  it('does nothing when the node has never had children', () => {
+    expect(() => removeNodeChildren(container)).not.toThrow();
+    expect(getNodeChildCount(container)).toBe(0);
+  });
+
   it('removeChildren removes all children by default', () => {
     addNodeChild(container, childA);
     addNodeChild(container, childB);
@@ -908,6 +939,11 @@ describe('replaceNodeChild', () => {
 });
 
 describe('setNodeChildIndex', () => {
+  it('does nothing when the node has never had children', () => {
+    expect(() => setNodeChildIndex(container, childA, 0)).not.toThrow();
+    expect(getNodeChildCount(container)).toBe(0);
+  });
+
   it('advances the child-list revision when order changes', () => {
     addNodeChild(container, childA);
     addNodeChild(container, childB);
@@ -948,6 +984,25 @@ describe('setNodeChildIndex', () => {
     expect(getChildren(container)[0]).toBe(childA);
   });
 
+  it('does not advance the revision when the child is already at the destination', () => {
+    addNodeChild(container, childA);
+    const childrenId = getNodeChildrenRevision(container);
+
+    setNodeChildIndex(container, childA, 0);
+
+    expect(getNodeChildrenRevision(container)).toBe(childrenId);
+    expect(getNodeChildAt(container, 0)).toBe(childA);
+  });
+
+  it('ignores a stale parent reference whose child-list entry is absent', () => {
+    addNodeChild(container, childB);
+    getEntityRuntime(childA).parent = container;
+
+    setNodeChildIndex(container, childA, 0);
+
+    expect(getChildren(container)).toEqual([childB]);
+  });
+
   it('calls onChildrenOrderChanged on the parent', () => {
     addNodeChild(container, childA);
     addNodeChild(container, childB);
@@ -962,6 +1017,16 @@ describe('setNodeChildIndex', () => {
 });
 
 describe('swapNodeChildren', () => {
+  it('does not advance the revision when both arguments are the same child', () => {
+    addNodeChild(container, childA);
+    const childrenId = getNodeChildrenRevision(container);
+
+    swapNodeChildren(container, childA, childA);
+
+    expect(getNodeChildrenRevision(container)).toBe(childrenId);
+    expect(getNodeChildAt(container, 0)).toBe(childA);
+  });
+
   it('advances the child-list revision when order changes', () => {
     addNodeChild(container, childA);
     addNodeChild(container, childB);
@@ -1007,6 +1072,21 @@ describe('swapNodeChildren', () => {
 });
 
 describe('swapNodeChildrenAt', () => {
+  it('does nothing when the node has never had children', () => {
+    expect(() => swapNodeChildrenAt(container, 0, 1)).not.toThrow();
+    expect(getNodeChildCount(container)).toBe(0);
+  });
+
+  it('does not advance the revision when both indices are equal', () => {
+    addNodeChild(container, childA);
+    const childrenId = getNodeChildrenRevision(container);
+
+    swapNodeChildrenAt(container, 0, 0);
+
+    expect(getNodeChildrenRevision(container)).toBe(childrenId);
+    expect(getNodeChildAt(container, 0)).toBe(childA);
+  });
+
   it('advances the child-list revision when order changes', () => {
     addNodeChild(container, childA);
     addNodeChild(container, childB);

package/src/node.test.ts

@@ -1,5 +1,5 @@
 import { connectSignal } from '@flighthq/signals/contract';
-import type { Node, NodeData, NodeRuntime, PartialNode } from '@flighthq/types/contract';
+import type { InteractionSignals, Node, NodeData, NodeRuntime, PartialNode } from '@flighthq/types/contract';
 import { NodeKind } from '@flighthq/types/contract';
 
 import { addNodeChild, getNodeChildCount, getNodeParent } from './hierarchy';
@@ -165,6 +165,16 @@ describe('disposeNode', () => {
     expect(getNodeSignals(node)).toBeNull();
   });
 
+  it('releases the interaction signal registry after disposal', () => {
+    const node = createNode(NodeKind);
+    const runtime = getNodeRuntime(node) as NodeRuntime;
+    runtime.interactionSignals = {} as InteractionSignals;
+
+    disposeNode(node);
+
+    expect(runtime.interactionSignals).toBeNull();
+  });
+
   it('is safe to call on a root node with no parent', () => {
     const node = createNode(NodeKind);
     expect(() => disposeNode(node)).not.toThrow();

package/src/nodeColorAdjustment.test.ts

@@ -19,6 +19,17 @@ describe('addNodeColorAdjustment', () => {
     expect(getNodeColorAdjustments(node)).toEqual([adjustment]);
     expect(getNodeRuntime(node).resolvedColorScaleBias?.redScale).toBeCloseTo(0x7f / 255);
   });
+
+  it('appends to an existing stack without replacing earlier adjustments', () => {
+    const node = createNode(NodeKind);
+    const first = createTintAdjustment(0xff0000ff);
+    const second = createTintAdjustment(0x00ff00ff);
+
+    addNodeColorAdjustment(node, first);
+    addNodeColorAdjustment(node, second);
+
+    expect(getNodeColorAdjustments(node)).toEqual([first, second]);
+  });
 });
 
 describe('getNodeColorAdjustments', () => {

package/src/nodeOrderList.test.ts

@@ -614,4 +614,13 @@ describe('swapNodeOrderListEntries', () => {
 
     expect(getNodeOrderListEntrySortKey(list, childA)).toBe(1);
   });
+
+  it('does nothing when the first node has no entry', () => {
+    const list = createNodeOrderList();
+    addNodeOrderListEntry(list, childB, 2);
+
+    swapNodeOrderListEntries(list, childA, childB);
+
+    expect(getNodeOrderListEntrySortKey(list, childB)).toBe(2);
+  });
 });

package/src/nodeTransform2d.test.ts

@@ -166,6 +166,28 @@ describe('ensureNodeLocalMatrix', () => {
     ensureNodeLocalMatrix(node);
     expect(equalsMatrix(runtime.localMatrix, cache)).toBe(true);
   });
+
+  it('normalizes rotations above 180 degrees', () => {
+    const runtime = getEntityRuntime(node) as HasTransform2DRuntime;
+    node.rotation = 270;
+    invalidateNodeLocalTransform(node);
+
+    ensureNodeLocalMatrix(node);
+
+    expect(runtime.rotationAngle).toBe(-90);
+    expect(runtime.localMatrix?.b).toBeCloseTo(-1);
+  });
+
+  it('normalizes rotations below -180 degrees', () => {
+    const runtime = getEntityRuntime(node) as HasTransform2DRuntime;
+    node.rotation = -270;
+    invalidateNodeLocalTransform(node);
+
+    ensureNodeLocalMatrix(node);
+
+    expect(runtime.rotationAngle).toBe(90);
+    expect(runtime.localMatrix?.b).toBeCloseTo(1);
+  });
 });
 
 describe('ensureNodeWorldMatrix', () => {

package/src/traversal.test.ts

@@ -1,8 +1,8 @@
-import type { Node, NodeTraits } from '@flighthq/types/contract';
+import type { Node, NodeRuntime, NodeTraits } from '@flighthq/types/contract';
 import { NodeKind } from '@flighthq/types/contract';
 
 import { addNodeChild } from './hierarchy';
-import { createNode } from './node';
+import { createNode, getNodeRuntime } from './node';
 import {
   findNode,
   findNodeByName,
@@ -190,6 +190,12 @@ describe('getNodeNextSibling', () => {
   it('returns the next sibling', () => {
     expect(getNodeNextSibling(childA)).toBe(childB);
   });
+
+  it('returns null for a stale parent whose child list is absent', () => {
+    (getNodeRuntime(root) as NodeRuntime).children = null;
+
+    expect(getNodeNextSibling(childA)).toBeNull();
+  });
 });
 
 describe('getNodePreviousSibling', () => {
@@ -204,6 +210,12 @@ describe('getNodePreviousSibling', () => {
   it('returns the previous sibling', () => {
     expect(getNodePreviousSibling(childB)).toBe(childA);
   });
+
+  it('returns null for a stale parent whose child list is absent', () => {
+    (getNodeRuntime(root) as NodeRuntime).children = null;
+
+    expect(getNodePreviousSibling(childB)).toBeNull();
+  });
 });
 
 describe('walkNodeDescendants', () => {
@@ -242,4 +254,16 @@ describe('walkNodeDescendants', () => {
     expect(visited).not.toContain(grandchild);
     expect(visited).not.toContain(childB);
   });
+
+  it('propagates an early stop from a nested descendant', () => {
+    const visited: Node<NodeTraits>[] = [];
+
+    const result = walkNodeDescendants(root, (node) => {
+      visited.push(node);
+      return node !== grandchild;
+    });
+
+    expect(result).toBe(false);
+    expect(visited).toEqual([childA, grandchild]);
+  });
 });