Fixed small bugs to reduce start warnings

-minor bugfixes
-some defined but unused elements commented out with TODO

src/modules/entity/components/components/SkyboxComponent.ts

@@ -11,9 +11,13 @@
 
 
 
-import { MeshComponent, DEFAULT_MESH_RENDER_GROUP_ID } from "@Modules/object";
+import { MeshComponent } from "@Modules/object";
+// TODO: Uncomment and use or remove the following import
+// import { DEFAULT_MESH_RENDER_GROUP_ID } from "@Modules/object";
 import { Scene, MeshBuilder, StandardMaterial, Texture, CubeTexture, EquiRectangularCubeTexture, BaseTexture, Camera } from "@babylonjs/core";
-import { ISkyboxProperty, IVector3Property } from "../../EntityProperties";
+import { ISkyboxProperty } from "../../EntityProperties";
+// TODO: Uncomment and use or remove the following import
+// import { IVector3Property } from "../../EntityProperties";
 import { EntityMapper } from "../../package";
 import { AssetUrl } from "../../builders/asset";
 

src/modules/scene/ZoneManager.ts

@@ -1,12 +1,16 @@
-import { Scene, Vector3 } from "@babylonjs/core";
+import { Scene, Vector3, AbstractMesh } from "@babylonjs/core";
 import { ZoneEntityController } from "../entity/components/controllers/ZoneEntityController";
 import Log from "@Modules/debugging/log";
 
+interface ZoneMeshMetadata {
+    zoneController?: ZoneEntityController;
+}
+
 export class ZoneManager {
     private _scene: Scene;
     private _currentZone: ZoneEntityController | null = null;
-    private _lastUpdateTime: number = 0;
-    private _updateInterval: number = 200; // Update every 500ms
+    private _lastUpdateTime = 0;
+    private _updateInterval = 200; // Update every 200ms
 
     constructor(scene: Scene) {
         this._scene = scene;
@@ -22,13 +26,16 @@ export class ZoneManager {
         this._lastUpdateTime = currentTime;
 
         const camera = this._scene.activeCamera;
-        if (!camera) return;
+        if (!camera) {
+            return;
+        }
 
         const cameraPosition = camera.position;
         let newZone: ZoneEntityController | null = null;
 
-        this._scene.meshes.forEach(mesh => {
-            const zoneController = mesh.metadata?.zoneController as ZoneEntityController | undefined;
+        this._scene.meshes.forEach((mesh: AbstractMesh) => {
+            const metadata = mesh.metadata as ZoneMeshMetadata | undefined;
+            const zoneController = metadata?.zoneController;
             if (zoneController && this.isPointInside(cameraPosition, zoneController)) {
                 newZone = zoneController;
             }
@@ -49,7 +56,9 @@ export class ZoneManager {
 
     private isPointInside(point: Vector3, zoneController: ZoneEntityController): boolean {
         const zoneMesh = zoneController.zoneMesh;
-        if (!zoneMesh) return false;
+        if (!zoneMesh) {
+            return false;
+        }
         return zoneMesh.intersectsPoint(point);
     }
 }

src/modules/scene/controllers/sceneController.ts

@@ -16,6 +16,7 @@ import { ScriptComponent } from "@Modules/script";
 import type { VScene } from "@Modules/scene/vscene";
 import Log from "@Modules/debugging/log";
 import { ZoneManager } from "../ZoneManager";
+import { ZoneEntityController } from '../../entity/components/controllers/ZoneEntityController';
 
 const DEFAULT_GRAVITY = 9.81;
 const GROUND_DETECTION_LENGTH = 5; // FIXME: This is not a good system for detecting the ground.
@@ -127,12 +128,7 @@ export class SceneController extends ScriptComponent {
             const avatarPosition = avatar.position;
             console.log(`Checking zones for avatar at position: ${avatarPosition.toString()}`);
             this._scene.meshes.forEach(mesh => {
-                const zoneController = mesh.getComponent(ZoneEntityController);
-                if (zoneController && zoneController.isInside(avatarPosition)) {
-                    console.log(`Avatar is inside zone: ${zoneController.zoneEntity.id}`);
-                    // Apply zone properties
-                    this._applyZoneProperties(zoneController.zoneEntity);
-                }
+                // Consider adding "is avatar in zone?" check later if needed
             });
         }
     }