@woosh/meep-engine 2.42.5 → 2.42.7

package/engine/graphics/render/forward_plus/plugin/ForwardPlusRenderingPlugin.js

@@ -3,6 +3,7 @@ import { EnginePlugin } from "../../../../plugin/EnginePlugin.js";
 import { MaterialTransformer } from "./MaterialTransformer.js";
 import { Vector2 as ThreeVector2 } from "three";
 
+
 export class ForwardPlusRenderingPlugin extends EnginePlugin {
     constructor() {
         super();
@@ -16,6 +17,13 @@ export class ForwardPlusRenderingPlugin extends EnginePlugin {
             light_manager: this.__light_manager,
             resolution: this.__resolution
         });
+
+        /**
+         * Track lifecycle state
+         * @type {boolean}
+         * @private
+         */
+        this.__state_is_running = false;
     }
 
     /**
@@ -36,6 +44,12 @@ export class ForwardPlusRenderingPlugin extends EnginePlugin {
     }
 
     async startup() {
+        if (this.__state_is_running) {
+            throw new Error("Already running");
+        }
+
+        this.__state_is_running = true;
+
         const lm = this.__light_manager;
 
         lm.setTileMapResolution(16, 8, 8);
@@ -52,14 +66,23 @@ export class ForwardPlusRenderingPlugin extends EnginePlugin {
 
         this.updateResolution();
 
-
         await super.startup();
     }
 
     async shutdown() {
+        if (!this.__state_is_running) {
+            throw new Error("Not running");
+        }
+
+        this.__state_is_running = false;
+
         const graphics = this.engine.graphics;
 
-        graphics.getMaterialManager().removeCompileStep(this.__material_transformer);
+        const is_material_removed = graphics.getMaterialManager().removeCompileStep(this.__material_transformer);
+
+        if (!is_material_removed) {
+            console.warn('Failed to remove material transformer');
+        }
 
         graphics.viewport.size.onChanged.remove(this.updateResolution, this);
         graphics.pixelRatio.onChanged.remove(this.updateResolution, this);

package/engine/plugin/EnginePluginManager.js

@@ -4,6 +4,7 @@ import { EnginePlugin } from "./EnginePlugin.js";
 import { PluginReferenceContext } from "./PluginReferenceContext.js";
 import { isSubclassOf } from "./isSubclassOf.js";
 import { assert } from "../../core/assert.js";
+import { IllegalStateException } from "../../core/fsm/exceptions/IllegalStateException.js";
 
 export class EnginePluginManager extends BaseProcess {
     constructor() {
@@ -246,6 +247,11 @@ export class EnginePluginManager extends BaseProcess {
 
         ctx.dependency_references.addAll(dependency_refs);
 
+        // Sanity check
+        if (this.__plugins.has(PluginClass)) {
+            throw new IllegalStateException('Plugin was already instantiated during acquisition');
+        }
+
         // all dependencies acquired, register the plugin
         this.__plugins.set(PluginClass, ctx);
 

package/package.json

@@ -5,7 +5,7 @@
   "productName": "Meep",
   "description": "production-ready JavaScript game engine based on Entity Component System Architecture",
   "author": "Alexander Goldring",
-  "version": "2.42.5",
+  "version": "2.42.7",
   "dependencies": {
     "gl-matrix": "3.4.3",
     "fast-levenshtein": "2.0.6",