rayzee 7.26.3 → 7.26.4

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "rayzee",
-  "version": "7.26.3",
+  "version": "7.26.4",
   "type": "module",
   "description": "Real-time WebGPU path tracing engine built on Three.js",
   "main": "dist/rayzee.umd.js",

package/src/Processor/GeometryExtractor.js

@@ -6,6 +6,53 @@ const log = createLogger( 'geometry' );
 
 const MAX_TEXTURES_LIMIT = 128;
 
+/**
+ * glTF 2.0 alphaMode for a three.js material: 0 OPAQUE, 1 MASK, 2 BLEND.
+ *
+ * The single definition of this rule. alphaMode is the only alpha field the shader reads
+ * (MaterialTransmission takes the opaque fast path on 0 and gates its MASK branch on 1), so
+ * any control that changes alpha behaviour has to route through here or it does nothing.
+ *
+ * @param {import('three').Material} material
+ * @returns {0|1|2}
+ */
+export function deriveAlphaMode( material ) {
+
+	// Follow glTF 2.0 specification for alphaMode
+	// Check if material explicitly sets alphaMode (from glTF loader)
+	if ( material.userData?.gltfExtensions?.KHR_materials_unlit?.alphaMode ) {
+
+		const mode = material.userData.gltfExtensions.KHR_materials_unlit.alphaMode;
+		if ( mode === 'BLEND' ) return 2;
+		if ( mode === 'MASK' ) return 1;
+		return 0; // OPAQUE
+
+	}
+
+	// Fallback logic based on material properties
+	if ( material.alphaTest > 0.0 ) {
+
+		return 1; // MASK - alphaTest takes priority
+
+	}
+
+	if ( material.transparent && material.opacity < 1.0 ) {
+
+		return 2; // BLEND - transparent with opacity < 1
+
+	}
+
+	// Check for alpha in diffuse texture
+	if ( material.map && material.map.format === RGBAFormat && material.transparent ) {
+
+		return 2; // BLEND - has alpha texture and transparent flag
+
+	}
+
+	return 0; // OPAQUE
+
+}
+
 export class GeometryExtractor {
 
 	constructor() {
@@ -189,42 +236,6 @@ export class GeometryExtractor {
 
 	}
 
-	getMaterialAlphaMode( material ) {
-
-		// Follow glTF 2.0 specification for alphaMode
-		// Check if material explicitly sets alphaMode (from glTF loader)
-		if ( material.userData?.gltfExtensions?.KHR_materials_unlit?.alphaMode ) {
-
-			const mode = material.userData.gltfExtensions.KHR_materials_unlit.alphaMode;
-			if ( mode === 'BLEND' ) return 2;
-			if ( mode === 'MASK' ) return 1;
-			return 0; // OPAQUE
-
-		}
-
-		// Fallback logic based on material properties
-		if ( material.alphaTest > 0.0 ) {
-
-			return 1; // MASK - alphaTest takes priority
-
-		}
-
-		if ( material.transparent && material.opacity < 1.0 ) {
-
-			return 2; // BLEND - transparent with opacity < 1
-
-		}
-
-		// Check for alpha in diffuse texture
-		if ( material.map && material.map.format === RGBAFormat && material.transparent ) {
-
-			return 2; // BLEND - has alpha texture and transparent flag
-
-		}
-
-		return 0; // OPAQUE
-
-	}
 
 	getMaterialType( material ) {
 
@@ -424,7 +435,7 @@ export class GeometryExtractor {
 			// Transparency and alpha
 			transparent: material.transparent ? 1 : 0,
 			alphaTest: material.alphaTest ?? defaults.alphaTest,
-			alphaMode: this.getMaterialAlphaMode( material ),
+			alphaMode: deriveAlphaMode( material ),
 
 			// Rendering properties
 			side: this.getMaterialSide( material ),

package/src/index.js

@@ -17,6 +17,9 @@ export { EngineEvents } from './EngineEvents.js';
 // Asset URL / cache namespace overrides (call before constructing PathTracerApp)
 export { configureAssets, getAssetConfig } from './AssetConfig.js';
 
+// glTF alphaMode derivation — the one definition, shared with hosts that edit materials
+export { deriveAlphaMode } from './Processor/GeometryExtractor.js';
+
 // Scene-level authoring metadata embedded in model files (glTF `extras`)
 export { extractSceneMetadata, parseSceneMetadata } from './Processor/SceneMetadata.js';
 

package/src/managers/UniformManager.js

@@ -29,6 +29,9 @@ export class UniformManager {
 		/** @type {Set<string>} Uniforms that store boolean values as int 0/1 */
 		this._booleans = new Set();
 
+		/** @type {Set<string>} Uniforms already warned about an array value (warn once each) */
+		this._arrayWarned = new Set();
+
 		/** @type {Object} Light buffer uniformArray nodes */
 		this._lightBuffers = {};
 
@@ -64,6 +67,32 @@ export class UniformManager {
 
 		}
 
+		// Hosts wire UI controls straight to these, and slider components commonly emit [n]
+		// rather than n. A scalar uniform has no meaningful array value, and nothing downstream
+		// rejects one: the array lands in .value verbatim and poisons every later read. That is
+		// not hypothetical — maxBounces received [20], and PathTracer's `maxBounces +
+		// transmissiveBounces + maxSubsurfaceSteps` then evaluated to the STRING '2058', so the
+		// bounce loop ran to 2058 on any frame the survivor curve could not early-exit and hung
+		// the tab. Unwrap and warn once per uniform, so the caller is named rather than the
+		// symptom surfacing somewhere unrelated.
+		if ( Array.isArray( value ) && value.length === 1
+			&& ( this._booleans.has( name ) || typeof node.value === 'number' ) ) {
+
+			if ( ! this._arrayWarned.has( name ) ) {
+
+				this._arrayWarned.add( name );
+				console.warn(
+					`UniformManager: "${name}" was set to [${value[ 0 ]}] but expects a scalar — ` +
+					'unwrapping. Fix the caller; a slider handler is probably forwarding its ' +
+					'array value unchanged.'
+				);
+
+			}
+
+			value = value[ 0 ];
+
+		}
+
 		if ( this._booleans.has( name ) ) {
 
 			node.value = value ? 1 : 0;
@@ -324,6 +353,7 @@ export class UniformManager {
 
 		this._uniforms.clear();
 		this._booleans.clear();
+		this._arrayWarned.clear();
 		this._lightBuffers = {};
 
 	}