fix(viewer): restore appearance-change dirty-mark in GeometrySystem

The biome --unsafe autofix stripped [shading, textures, colorPreset,
sceneTheme] from the mount effect's dep array, treating them as unused.
They are deliberate re-run TRIGGERS: the effect re-marks every
def.geometry node dirty whenever an appearance value changes, so the
builders re-run and pick up the new shading / textures / color preset /
scene theme. Without them, switching render mode, toggling textures, or
changing the preset no longer rebuilds existing registry-driven geometry.

All four are primitives (stable by value), so listing them is safe;
added a scoped biome-ignore so the trigger-only deps survive future
autofixes.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

Author: sudhir9297

packages/viewer/src/systems/geometry/geometry-system.tsx

@@ -64,6 +64,14 @@ export const GeometrySystem = () => {
   // shelf dirties the shelf without altering its boards.
   const builtGeometryKeyRef = useRef<Map<string, string>>(new Map())
 
+  // Re-mark every geometry-backed node dirty whenever a viewer appearance
+  // value changes, so `def.geometry` builders re-run and pick up the new
+  // shading / texture / preset / theme. These four are deliberate re-run
+  // TRIGGERS, not values read in the body — the effect re-fires on any
+  // change. They're primitives (stable by value), so listing them is safe;
+  // biome flags them as "unnecessary" because the body doesn't reference
+  // them, but dropping them silently breaks appearance-mode switching.
+  // biome-ignore lint/correctness/useExhaustiveDependencies: shading/textures/colorPreset/sceneTheme are intentional re-run triggers; removing them stops geometry from rebuilding on appearance change.
   useEffect(() => {
     const nodes = useScene.getState().nodes
     for (const node of Object.values(nodes)) {
@@ -72,7 +80,7 @@ export const GeometrySystem = () => {
         useScene.getState().markDirty(node.id as AnyNodeId)
       }
     }
-  }, [])
+  }, [shading, textures, colorPreset, sceneTheme])
 
   useFrame(() => {
     if (dirtyNodes.size === 0) return