@flighthq/render 0.2.1-next.841.373388f → 0.3.0-next.1032.e2b99fc

package/src/sceneRender.test.ts

@@ -1,28 +1,37 @@
-import { createCamera3D, createPerspectiveProjection, setCamera3DViewMatrix4FromLookAt } from '@flighthq/camera';
-import { unpackColorToLinear } from '@flighthq/color';
-import { createAabb, createMatrix4 } from '@flighthq/geometry';
+import {
+  createCamera3D,
+  createPerspectiveProjection,
+  setCamera3DViewMatrix4FromLookAt,
+} from '@flighthq/camera/contract';
+import { unpackColorToLinear } from '@flighthq/color/contract';
+import { createAabb, createMatrix4 } from '@flighthq/geometry/contract';
 import {
   createAmbientLight,
   createDirectionalLight,
   createHemisphereLight,
   createPointLight,
   createSpotLight,
-} from '@flighthq/lighting';
+} from '@flighthq/lighting/contract';
 import {
   CANONICAL_MESH_GEOMETRY_LAYOUT,
   computeMeshGeometryBounds,
   createBoxMeshGeometry,
   createMeshGeometry,
-} from '@flighthq/mesh';
-import { addNodeChild, getNodeWorldMatrix4, invalidateNodeLocalTransform, setNodeLocalMatrix4 } from '@flighthq/node';
-import { createMesh, createNode3D, Node3DKind, getNode3DWorldAlpha } from '@flighthq/scene3d';
-import type { Camera3D, Material, MeshGeometry, Scene3DLightBlock, Scene3DLightsLike } from '@flighthq/types';
+} from '@flighthq/mesh/contract';
+import {
+  addNodeChild,
+  getNodeWorldMatrix4,
+  invalidateNodeLocalTransform,
+  setNodeLocalMatrix4,
+} from '@flighthq/node/contract';
+import { createMesh, createNode3D, Node3DKind, getNode3DWorldAlpha } from '@flighthq/scene3d/contract';
+import type { Camera3D, Material, MeshGeometry, Scene3DLightBlock, Scene3DLightsLike } from '@flighthq/types/contract';
 import {
   SCENE_LIGHT_BLOCK_FLOATS,
   SCENE_LIGHT_HEMISPHERE_OFFSET,
   SCENE_LIGHT_POINT_OFFSET,
   SCENE_LIGHT_SPOT_OFFSET,
-} from '@flighthq/types';
+} from '@flighthq/types/contract';
 import { describe, expect, it } from 'vitest';
 
 import { createRenderState } from './renderState';
@@ -235,6 +244,19 @@ describe('prepareScene3DRender', () => {
     expect(list.viewProjection.m[15]).not.toBe(1);
   });
 
+  it('uses a draw-time viewport aspect without mutating the camera projection', () => {
+    const state = createRenderState();
+    const scene = createNode3D(Node3DKind);
+    const camera = frontCamera();
+
+    const tallXScale = prepareScene3DRender(state, scene, camera, emptyLights(), 0.5).viewProjection.m[0];
+    const wideXScale = prepareScene3DRender(state, scene, camera, emptyLights(), 2).viewProjection.m[0];
+
+    expect(tallXScale).toBeCloseTo(wideXScale * 4);
+    expect(camera.projection.kind).toBe('perspective');
+    if (camera.projection.kind === 'perspective') expect(camera.projection.aspect).toBe(1);
+  });
+
   it('culls a mesh placed far behind the camera', () => {
     const state = createRenderState();
     const scene = createNode3D(Node3DKind);

package/dist/renderColorTransform.d.ts

@@ -1,3 +0,0 @@
-import type { RenderProxy, RenderState } from '@flighthq/types';
-export declare function updateRenderProxyColorTransform(state: RenderState, data: RenderProxy, _parentData?: RenderProxy): void;
-//# sourceMappingURL=renderColorTransform.d.ts.map

package/dist/renderColorTransform.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"renderColorTransform.d.ts","sourceRoot":"","sources":["../src/renderColorTransform.ts"],"names":[],"mappings":"AACA,OAAO,KAAK,EAAoB,WAAW,EAAE,WAAW,EAAE,MAAM,iBAAiB,CAAC;AAelF,wBAAgB,+BAA+B,CAC7C,KAAK,EAAE,WAAW,EAClB,IAAI,EAAE,WAAW,EACjB,WAAW,CAAC,EAAE,WAAW,GACxB,IAAI,CAMN"}

package/dist/renderColorTransform.js

@@ -1,21 +0,0 @@
-import { getNodeRuntime } from '@flighthq/node';
-import { getRenderStateRuntime } from './renderState';
-// Hands the node's resolved color transform to the render node. A node's color-adjustment stack
-// (NodeRuntime.colorAdjustments) is fused once into the affine `resolvedColorTransform` cache by the
-// set-accessors on change — never per frame — so this hot-path visitor is a single field read, identical
-// in cost to reading the old `.colorTransform`. Non-inheriting: a node uses its own resolved value (or
-// none → null). The value is the Adjustment-tier fold input; it is not a material and does not key the
-// batch.
-//
-// When the fused stack carries off-diagonal channel-mixing terms the 8-float inline fold cannot represent
-// yet (the 4×5 path is deferred), the node is flagged `colorAdjustmentsChannelMixing`: only the affine
-// part is applied, and the render state's shakeable guard (enableColorAdjustmentGuards) is invoked so the
-// deferral is reported rather than silent.
-export function updateRenderProxyColorTransform(state, data, _parentData) {
-    const runtime = getNodeRuntime(data.source);
-    data.colorTransform = runtime.resolvedColorTransform ?? null;
-    if (runtime.colorAdjustmentsChannelMixing) {
-        getRenderStateRuntime(state).colorAdjustmentChannelMixingGuard?.(state, data.source);
-    }
-}
-//# sourceMappingURL=renderColorTransform.js.map

package/dist/renderColorTransform.js.map

@@ -1 +0,0 @@
-{"version":3,"file":"renderColorTransform.js","sourceRoot":"","sources":["../src/renderColorTransform.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,cAAc,EAAE,MAAM,gBAAgB,CAAC;AAGhD,OAAO,EAAE,qBAAqB,EAAE,MAAM,eAAe,CAAC;AAEtD,gGAAgG;AAChG,qGAAqG;AACrG,yGAAyG;AACzG,uGAAuG;AACvG,uGAAuG;AACvG,SAAS;AACT,EAAE;AACF,0GAA0G;AAC1G,uGAAuG;AACvG,0GAA0G;AAC1G,2CAA2C;AAC3C,MAAM,UAAU,+BAA+B,CAC7C,KAAkB,EAClB,IAAiB,EACjB,WAAyB;IAEzB,MAAM,OAAO,GAAG,cAAc,CAAC,IAAI,CAAC,MAAc,CAAC,CAAC;IACpD,IAAI,CAAC,cAAc,GAAG,OAAO,CAAC,sBAAsB,IAAI,IAAI,CAAC;IAC7D,IAAI,OAAO,CAAC,6BAA6B,EAAE,CAAC;QAC1C,qBAAqB,CAAC,KAAK,CAAC,CAAC,iCAAiC,EAAE,CAAC,KAAK,EAAE,IAAI,CAAC,MAAoB,CAAC,CAAC;IACrG,CAAC;AACH,CAAC"}

package/src/renderColorTransform.test.ts

@@ -1,45 +0,0 @@
-import { createColorTransformAdjustment } from '@flighthq/adjustments';
-import { createColorTransform } from '@flighthq/materials';
-import { createDisplayObject, getNode2DRuntime, setNode2DColorAdjustments } from '@flighthq/scene2d';
-import type { Renderable } from '@flighthq/types';
-
-import { updateRenderProxyColorTransform } from './renderColorTransform';
-import { createRenderProxy } from './renderProxy';
-import { createRenderState } from './renderState';
-
-describe('updateRenderProxyColorTransform', () => {
-  it('resolves a node color-adjustment stack onto the render node as an affine ColorTransform', () => {
-    const state = createRenderState();
-    const node = createDisplayObject();
-    setNode2DColorAdjustments(node, [
-      createColorTransformAdjustment(createColorTransform({ redMultiplier: 0.5, greenMultiplier: 0 })),
-    ]);
-    const data = createRenderProxy(state, node as unknown as Renderable);
-    updateRenderProxyColorTransform(state, data);
-    expect(data.colorTransform).not.toBeNull();
-    expect(data.colorTransform!.redMultiplier).toBe(0.5);
-    expect(data.colorTransform!.greenMultiplier).toBe(0);
-  });
-
-  it('resolves to null when the node carries no adjustments', () => {
-    const state = createRenderState();
-    const node = createDisplayObject();
-    const data = createRenderProxy(state, node as unknown as Renderable);
-    updateRenderProxyColorTransform(state, data);
-    expect(data.colorTransform).toBeNull();
-  });
-
-  it('reads the cache the accessor fused once (the walk never re-fuses)', () => {
-    const state = createRenderState();
-    const node = createDisplayObject();
-    setNode2DColorAdjustments(node, [createColorTransformAdjustment(createColorTransform({ redMultiplier: 0.25 }))]);
-    // The set-accessor fused the stack once; the runtime already holds the cached resolved value.
-    const cached = getNode2DRuntime(node).resolvedColorTransform;
-    expect(cached).not.toBeNull();
-    const data = createRenderProxy(state, node as unknown as Renderable);
-    updateRenderProxyColorTransform(state, data);
-    expect(data.colorTransform).toBe(cached);
-    updateRenderProxyColorTransform(state, data);
-    expect(data.colorTransform).toBe(cached);
-  });
-});