fix(core): add geometryStamp to ngt instance

A new `geometryStamp` is added to `ngt.instanceStore`. This allows the
renderer to update the Local State when a geometry is applied via
`setProperty`. When this is the case, normal `nonObjects()` will not
work because the geometry isn't added via content of the parent.

However, we still need to track / rerun effects on geometries' changes
most of the times. This allows us to notify the Local State when the
geometry property is updated (via `[geometry]`)

A new `nonObjectsChanged` computed is a combination of `nonObjects()`
and `geometryStamp()`. Consumers using `localState.nonObjects()` will
automatically get the changes from `geometryStamp` as well.

Author: nartc

libs/core/src/lib/instance.ts

@@ -1,3 +1,4 @@
+import { computed } from '@angular/core';
 import { NgtAnyRecord, NgtInstanceNode, NgtLocalInstanceState, NgtLocalState } from './types';
 import { signalStore } from './utils/signal-store';
 import { checkUpdate } from './utils/update';
@@ -34,10 +35,19 @@ export function prepare<TInstance extends object = NgtAnyRecord>(
 				parent: null,
 				objects: [],
 				nonObjects: [],
+				geometryStamp: Date.now(),
 			}),
 			...rest
 		} = localState || {};
 
+		const nonObjects = instanceStore.select('nonObjects');
+		const geometryStamp = instanceStore.select('geometryStamp');
+
+		const nonObjectsChanged = computed(() => {
+			const [_nonObjects] = [nonObjects(), geometryStamp()];
+			return _nonObjects;
+		});
+
 		instance.__ngt__ = {
 			previousAttach: null,
 			store: null,
@@ -47,7 +57,7 @@ export function prepare<TInstance extends object = NgtAnyRecord>(
 			instanceStore,
 			parent: instanceStore.select('parent'),
 			objects: instanceStore.select('objects'),
-			nonObjects: instanceStore.select('nonObjects'),
+			nonObjects: nonObjectsChanged,
 			add(object, type) {
 				const current = instance.__ngt__.instanceStore.snapshot[type];
 				const foundIndex = current.indexOf((node: NgtInstanceNode) => object === node);
@@ -68,6 +78,9 @@ export function prepare<TInstance extends object = NgtAnyRecord>(
 			setParent(parent) {
 				instance.__ngt__.instanceStore.update({ parent });
 			},
+			updateGeometryStamp() {
+				instance.__ngt__.instanceStore.update({ geometryStamp: Date.now() });
+			},
 			...rest,
 		} as NgtLocalState;
 	}

libs/core/src/lib/renderer/index.ts

@@ -429,17 +429,23 @@ export class NgtRenderer implements Renderer2 {
 		const rS = el.__ngt_renderer__;
 		if (!rS || rS[NgtRendererClassId.destroyed]) return;
 
+		const localState = getLocalState(el);
+
 		if (rS[NgtRendererClassId.type] === 'three') {
 			if (name === SPECIAL_PROPERTIES.PARAMETERS) {
 				// NOTE: short-cut for null raycast to prevent upstream from creating a nullRaycast property
 				if ('raycast' in value && value['raycast'] === null) {
 					value['raycast'] = () => null;
 				}
 				applyProps(el, value);
+
+				if ('geometry' in value && value['geometry'].isBufferGeometry) {
+					localState?.updateGeometryStamp();
+				}
+
 				return;
 			}
 
-			const localState = getLocalState(el);
 			const parent = localState?.instanceStore.get('parent') || rS[NgtRendererClassId.parent];
 
 			// [rawValue]
@@ -469,6 +475,11 @@ export class NgtRenderer implements Renderer2 {
 			}
 
 			applyProps(el, { [name]: value });
+
+			if (name === 'geometry' && value.isBufferGeometry) {
+				localState?.updateGeometryStamp();
+			}
+
 			return;
 		}