@flighthq/lighting 0.1.0 → 0.2.0

package/dist/index.d.ts

@@ -6,5 +6,6 @@ export * from './environment';
 export * from './hemisphereLight';
 export * from './lightAnalysis';
 export * from './pointLight';
+export * from './sceneLights';
 export * from './spotLight';
 //# sourceMappingURL=index.d.ts.map

package/dist/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":"AAAA,cAAc,gBAAgB,CAAC;AAC/B,cAAc,aAAa,CAAC;AAC5B,cAAc,mBAAmB,CAAC;AAClC,cAAc,oBAAoB,CAAC;AACnC,cAAc,eAAe,CAAC;AAC9B,cAAc,mBAAmB,CAAC;AAClC,cAAc,iBAAiB,CAAC;AAChC,cAAc,cAAc,CAAC;AAC7B,cAAc,aAAa,CAAC"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":"AAAA,cAAc,gBAAgB,CAAC;AAC/B,cAAc,aAAa,CAAC;AAC5B,cAAc,mBAAmB,CAAC;AAClC,cAAc,oBAAoB,CAAC;AACnC,cAAc,eAAe,CAAC;AAC9B,cAAc,mBAAmB,CAAC;AAClC,cAAc,iBAAiB,CAAC;AAChC,cAAc,cAAc,CAAC;AAC7B,cAAc,eAAe,CAAC;AAC9B,cAAc,aAAa,CAAC"}

package/dist/index.js

@@ -6,5 +6,6 @@ export * from './environment';
 export * from './hemisphereLight';
 export * from './lightAnalysis';
 export * from './pointLight';
+export * from './sceneLights';
 export * from './spotLight';
 //# sourceMappingURL=index.js.map

package/dist/index.js.map

@@ -1 +1 @@
-{"version":3,"file":"index.js","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":"AAAA,cAAc,gBAAgB,CAAC;AAC/B,cAAc,aAAa,CAAC;AAC5B,cAAc,mBAAmB,CAAC;AAClC,cAAc,oBAAoB,CAAC;AACnC,cAAc,eAAe,CAAC;AAC9B,cAAc,mBAAmB,CAAC;AAClC,cAAc,iBAAiB,CAAC;AAChC,cAAc,cAAc,CAAC;AAC7B,cAAc,aAAa,CAAC"}
+{"version":3,"file":"index.js","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":"AAAA,cAAc,gBAAgB,CAAC;AAC/B,cAAc,aAAa,CAAC;AAC5B,cAAc,mBAAmB,CAAC;AAClC,cAAc,oBAAoB,CAAC;AACnC,cAAc,eAAe,CAAC;AAC9B,cAAc,mBAAmB,CAAC;AAClC,cAAc,iBAAiB,CAAC;AAChC,cAAc,cAAc,CAAC;AAC7B,cAAc,eAAe,CAAC;AAC9B,cAAc,aAAa,CAAC"}

package/dist/sceneLights.d.ts

@@ -0,0 +1,3 @@
+import type { SceneLights } from '@flighthq/types';
+export declare function createSceneLights(options?: Readonly<Partial<SceneLights>>): SceneLights;
+//# sourceMappingURL=sceneLights.d.ts.map

package/dist/sceneLights.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"sceneLights.d.ts","sourceRoot":"","sources":["../src/sceneLights.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,WAAW,EAAE,MAAM,iBAAiB,CAAC;AAYnD,wBAAgB,iBAAiB,CAAC,OAAO,CAAC,EAAE,QAAQ,CAAC,OAAO,CAAC,WAAW,CAAC,CAAC,GAAG,WAAW,CAQvF"}

package/dist/sceneLights.js

@@ -0,0 +1,20 @@
+// Constructs a `SceneLights` draw-argument, filling every absent slot: the single ambient/directional
+// terms default to `null` and the punctual arrays to empty. Prefer this over a bare object literal —
+// a literal that omits a slot passes `undefined` where the packer expects a light-or-null, which the
+// strict `!== null` presence check reads as "present" and then dereferences (the classic
+// `undefined.direction` crash). Going through the constructor makes an omitted slot unrepresentable.
+//
+// `SceneLights` is a per-draw argument (lights are not scene members), so it is NOT an Entity: it
+// carries no GPU binding identity of its own — the packed `SceneLightBlock` is what a backend caches,
+// keyed off the render state. The individual lights it references (`createPointLight`, …) are the
+// entities. This is the constructor for the plain-data collection, not an entity factory.
+export function createSceneLights(options) {
+    return {
+        ambient: options?.ambient ?? null,
+        directional: options?.directional ?? null,
+        hemisphere: options?.hemisphere ?? [],
+        point: options?.point ?? [],
+        spot: options?.spot ?? [],
+    };
+}
+//# sourceMappingURL=sceneLights.js.map

package/dist/sceneLights.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"sceneLights.js","sourceRoot":"","sources":["../src/sceneLights.ts"],"names":[],"mappings":"AAEA,sGAAsG;AACtG,qGAAqG;AACrG,qGAAqG;AACrG,yFAAyF;AACzF,qGAAqG;AACrG,EAAE;AACF,kGAAkG;AAClG,sGAAsG;AACtG,kGAAkG;AAClG,0FAA0F;AAC1F,MAAM,UAAU,iBAAiB,CAAC,OAAwC;IACxE,OAAO;QACL,OAAO,EAAE,OAAO,EAAE,OAAO,IAAI,IAAI;QACjC,WAAW,EAAE,OAAO,EAAE,WAAW,IAAI,IAAI;QACzC,UAAU,EAAE,OAAO,EAAE,UAAU,IAAI,EAAE;QACrC,KAAK,EAAE,OAAO,EAAE,KAAK,IAAI,EAAE;QAC3B,IAAI,EAAE,OAAO,EAAE,IAAI,IAAI,EAAE;KAC1B,CAAC;AACJ,CAAC"}

package/package.json

@@ -1,6 +1,11 @@
 {
   "name": "@flighthq/lighting",
-  "version": "0.1.0",
+  "version": "0.2.0",
+  "repository": {
+    "type": "git",
+    "url": "https://github.com/flighthq/flight.git",
+    "directory": "packages/lighting"
+  },
   "type": "module",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",
@@ -27,9 +32,9 @@
     "clean:dist": "tsx ../../scripts/clean-package-dist.ts"
   },
   "dependencies": {
-    "@flighthq/entity": "0.1.0",
-    "@flighthq/geometry": "0.1.0",
-    "@flighthq/types": "0.1.0"
+    "@flighthq/entity": "0.2.0",
+    "@flighthq/geometry": "0.2.0",
+    "@flighthq/types": "0.2.0"
   },
   "devDependencies": {
     "typescript": "^5.3.0"

package/src/sceneLights.test.ts

@@ -0,0 +1,36 @@
+import { describe, expect, it } from 'vitest';
+
+import { createAmbientLight } from './ambientLight';
+import { createDirectionalLight } from './directionalLight';
+import { createPointLight } from './pointLight';
+import { createSceneLights } from './sceneLights';
+
+describe('createSceneLights', () => {
+  it('fills every absent slot — singles to null, punctual arrays to empty', () => {
+    const lights = createSceneLights();
+    expect(lights.ambient).toBeNull();
+    expect(lights.directional).toBeNull();
+    expect(lights.point).toEqual([]);
+    expect(lights.spot).toEqual([]);
+    expect(lights.hemisphere).toEqual([]);
+  });
+
+  it('carries the provided lights and defaults the omitted ones (no undefined slot)', () => {
+    const point = createPointLight({ intensity: 2 });
+    const lights = createSceneLights({ point: [point] });
+    // The omitted directional must be null, not undefined — a literal that dropped it would crash the
+    // packer's strict `!== null` check; the constructor makes that unrepresentable.
+    expect(lights.directional).toBeNull();
+    expect(lights.ambient).toBeNull();
+    expect(lights.point).toEqual([point]);
+    expect(Object.prototype.hasOwnProperty.call(lights, 'directional')).toBe(true);
+  });
+
+  it('passes through ambient + directional when supplied', () => {
+    const ambient = createAmbientLight({ intensity: 0.2 });
+    const directional = createDirectionalLight({ intensity: 3 });
+    const lights = createSceneLights({ ambient, directional });
+    expect(lights.ambient).toBe(ambient);
+    expect(lights.directional).toBe(directional);
+  });
+});