@energy8platform/game-engine 0.7.1 → 0.8.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@energy8platform/game-engine",
-  "version": "0.7.1",
+  "version": "0.8.0",
   "description": "Universal casino game engine built on PixiJS v8 and @energy8platform/game-sdk",
   "type": "module",
   "main": "./dist/index.cjs.js",

package/src/core/SceneManager.ts

@@ -25,12 +25,14 @@ interface SceneManagerEvents {
  * ```
  */
 export class SceneManager extends EventEmitter<SceneManagerEvents> {
+  private static MAX_TRANSITION_DEPTH = 10;
+
   /** Root container that scenes are added to */
   public root!: Container;
 
   private registry = new Map<string, SceneConstructor>();
   private stack: SceneEntry[] = [];
-  private _transitioning = false;
+  private _transitionDepth = 0;
 
   /** Current viewport dimensions — set by ViewportManager */
   private _width = 0;
@@ -64,7 +66,7 @@ export class SceneManager extends EventEmitter<SceneManagerEvents> {
 
   /** Whether a scene transition is in progress */
   get isTransitioning(): boolean {
-    return this._transitioning;
+    return this._transitionDepth > 0;
   }
 
   /**
@@ -75,6 +77,9 @@ export class SceneManager extends EventEmitter<SceneManagerEvents> {
     data?: unknown,
     transition?: TransitionConfig,
   ): Promise<void> {
+    if (this._transitionDepth >= SceneManager.MAX_TRANSITION_DEPTH) {
+      throw new Error('[SceneManager] Max transition depth exceeded — possible infinite loop');
+    }
     const prevKey = this.currentKey;
 
     // Exit all current scenes
@@ -96,6 +101,9 @@ export class SceneManager extends EventEmitter<SceneManagerEvents> {
     data?: unknown,
     transition?: TransitionConfig,
   ): Promise<void> {
+    if (this._transitionDepth >= SceneManager.MAX_TRANSITION_DEPTH) {
+      throw new Error('[SceneManager] Max transition depth exceeded — possible infinite loop');
+    }
     const prevKey = this.currentKey;
     await this.pushInternal(key, data, transition);
     this.emit('change', { from: prevKey, to: key });
@@ -109,6 +117,9 @@ export class SceneManager extends EventEmitter<SceneManagerEvents> {
       console.warn('[SceneManager] Cannot pop the last scene');
       return;
     }
+    if (this._transitionDepth >= SceneManager.MAX_TRANSITION_DEPTH) {
+      throw new Error('[SceneManager] Max transition depth exceeded — possible infinite loop');
+    }
     const prevKey = this.currentKey;
     await this.popInternal(true, transition);
     this.emit('change', { from: prevKey, to: this.currentKey! });
@@ -122,6 +133,9 @@ export class SceneManager extends EventEmitter<SceneManagerEvents> {
     data?: unknown,
     transition?: TransitionConfig,
   ): Promise<void> {
+    if (this._transitionDepth >= SceneManager.MAX_TRANSITION_DEPTH) {
+      throw new Error('[SceneManager] Max transition depth exceeded — possible infinite loop');
+    }
     const prevKey = this.currentKey;
     await this.popInternal(false);
     await this.pushInternal(key, data, transition);
@@ -177,7 +191,7 @@ export class SceneManager extends EventEmitter<SceneManagerEvents> {
     data?: unknown,
     transition?: TransitionConfig,
   ): Promise<void> {
-    this._transitioning = true;
+    this._transitionDepth++;
 
     const scene = this.createScene(key);
     this.root.addChild(scene.container);
@@ -195,7 +209,7 @@ export class SceneManager extends EventEmitter<SceneManagerEvents> {
 
     await scene.onEnter?.(data);
 
-    this._transitioning = false;
+    this._transitionDepth--;
   }
 
   private async popInternal(
@@ -205,7 +219,7 @@ export class SceneManager extends EventEmitter<SceneManagerEvents> {
     const entry = this.stack.pop();
     if (!entry) return;
 
-    this._transitioning = true;
+    this._transitionDepth++;
 
     await entry.scene.onExit?.();
 
@@ -216,7 +230,7 @@ export class SceneManager extends EventEmitter<SceneManagerEvents> {
     entry.scene.onDestroy?.();
     entry.scene.container.destroy({ children: true });
 
-    this._transitioning = false;
+    this._transitionDepth--;
   }
 
   private async transitionIn(

package/src/state/StateMachine.ts

@@ -58,10 +58,12 @@ interface StateMachineEvents {
  * ```
  */
 export class StateMachine<TContext = Record<string, unknown>> extends EventEmitter<StateMachineEvents> {
+  private static MAX_TRANSITION_DEPTH = 10;
+
   private _states = new Map<string, StateConfig<TContext>>();
   private _guards = new Map<string, (ctx: TContext) => boolean>();
   private _current: string | null = null;
-  private _transitioning = false;
+  private _transitionDepth = 0;
   private _context: TContext;
 
   constructor(context: TContext) {
@@ -76,7 +78,7 @@ export class StateMachine<TContext = Record<string, unknown>> extends EventEmitt
 
   /** Whether a transition is in progress */
   get isTransitioning(): boolean {
-    return this._transitioning;
+    return this._transitionDepth > 0;
   }
 
   /** State machine context (shared data) */
@@ -131,9 +133,10 @@ export class StateMachine<TContext = Record<string, unknown>> extends EventEmitt
    * @returns true if the transition succeeded, false if blocked by a guard
    */
   async transition(to: string, data?: unknown): Promise<boolean> {
-    if (this._transitioning) {
-      console.warn('[StateMachine] Transition already in progress');
-      return false;
+    if (this._transitionDepth >= StateMachine.MAX_TRANSITION_DEPTH) {
+      throw new Error(
+        '[StateMachine] Max transition depth exceeded — possible infinite loop',
+      );
     }
 
     const from = this._current;
@@ -152,7 +155,7 @@ export class StateMachine<TContext = Record<string, unknown>> extends EventEmitt
       throw new Error(`[StateMachine] State "${to}" not registered.`);
     }
 
-    this._transitioning = true;
+    this._transitionDepth++;
 
     try {
       // Exit current state
@@ -170,7 +173,7 @@ export class StateMachine<TContext = Record<string, unknown>> extends EventEmitt
       this.emit('error', err instanceof Error ? err : new Error(String(err)));
       throw err;
     } finally {
-      this._transitioning = false;
+      this._transitionDepth--;
     }
 
     return true;
@@ -213,7 +216,7 @@ export class StateMachine<TContext = Record<string, unknown>> extends EventEmitt
       await state?.exit?.(this._context);
     }
     this._current = null;
-    this._transitioning = false;
+    this._transitionDepth = 0;
   }
 
   /**