@rpgjs/server 5.0.0-alpha.25 → 5.0.0-alpha.27

package/src/rooms/map.ts

@@ -1,5 +1,5 @@
-import { Action, MockConnection, Request, Room, RoomOnJoin } from "@signe/room";
-import { Hooks, IceMovement, ModulesToken, ProjectileMovement, ProjectileType, RpgCommonMap, Direction, RpgCommonPlayer, RpgShape } from "@rpgjs/common";
+import { Action, MockConnection, Request, Room, RoomMethods, RoomOnJoin } from "@signe/room";
+import { Hooks, IceMovement, ModulesToken, ProjectileMovement, ProjectileType, RpgCommonMap, Direction, RpgCommonPlayer, RpgShape, findModules } from "@rpgjs/common";
 import { WorldMapsManager, type WorldMapConfig } from "@rpgjs/common";
 import { RpgPlayer, RpgEvent } from "../Player/Player";
 import { generateShortUUID, sync, type, users } from "@signe/sync";
@@ -13,6 +13,7 @@ import { COEFFICIENT_ELEMENTS, DAMAGE_CRITICAL, DAMAGE_PHYSIC, DAMAGE_SKILL } fr
 import { z } from "zod";
 import { EntityState } from "@rpgjs/physic";
 import { MapOptions } from "../decorators/map";
+import { BaseRoom } from "./BaseRoom";
 
 /**
  * Interface for input controls configuration
@@ -113,7 +114,7 @@ export class RpgMap extends RpgCommonMap<RpgPlayer> implements RoomOnJoin {
    * ```
    */
   @users(RpgPlayer) players = signal({});
-  
+
   /** 
    * Synchronized signal containing all events (NPCs, objects) on the map
    * 
@@ -130,7 +131,7 @@ export class RpgMap extends RpgCommonMap<RpgPlayer> implements RoomOnJoin {
    * ```
    */
   @sync(RpgPlayer) events = signal({});
-  
+
   /** 
    * Signal containing the map's database of items, classes, and other game data
    * 
@@ -148,7 +149,7 @@ export class RpgMap extends RpgCommonMap<RpgPlayer> implements RoomOnJoin {
    * ```
    */
   database = signal({});
-  
+
   /** 
    * Array of map configurations - can contain MapOptions objects or instances of map classes
    * 
@@ -156,7 +157,7 @@ export class RpgMap extends RpgCommonMap<RpgPlayer> implements RoomOnJoin {
    * It's populated when the map is loaded via `updateMap()`.
    */
   maps: (MapOptions | any)[] = []
-  
+
   /** 
    * Array of sound IDs to play when players join the map
    * 
@@ -170,7 +171,7 @@ export class RpgMap extends RpgCommonMap<RpgPlayer> implements RoomOnJoin {
    * ```
    */
   sounds: string[] = []
-  
+
   /** 
    * BehaviorSubject that completes when the map data is ready
    * 
@@ -186,7 +187,7 @@ export class RpgMap extends RpgCommonMap<RpgPlayer> implements RoomOnJoin {
    * ```
    */
   dataIsReady$ = new BehaviorSubject<void>(undefined);
-  
+
   /** 
    * Global configuration object for the map
    * 
@@ -194,7 +195,7 @@ export class RpgMap extends RpgCommonMap<RpgPlayer> implements RoomOnJoin {
    * It's populated from the map data when `updateMap()` is called.
    */
   globalConfig: any = {}
-  
+
   /** 
    * Damage formulas configuration for the map
    * 
@@ -212,12 +213,24 @@ export class RpgMap extends RpgCommonMap<RpgPlayer> implements RoomOnJoin {
   /** Enable/disable automatic tick processing (useful for unit tests) */
   private _autoTickEnabled: boolean = true;
 
-  constructor() {
+  autoSync: boolean = true;
+
+  constructor(room) {
     super();
     this.hooks.callHooks("server-map-onStart", this).subscribe();
-    this.throttleSync = this.isStandalone ? 0 : 50; // Reduced from 100ms to 50ms for better responsiveness
-    this.throttleStorage = this.isStandalone ? 0 : 1000;
-    this.sessionExpiryTime = 1000 * 60 * 5; //5 minutes
+    const isTest = room.env.TEST === 'true' ? true : false;
+    if (isTest) {
+      this.autoSync = false;
+      this.setAutoTick(false);
+      this.autoTickEnabled = false;
+      this.throttleSync = 0;
+      this.throttleStorage = 0;
+    }
+    else {
+      this.throttleSync = this.isStandalone ? 1 : 50
+      this.throttleStorage = this.isStandalone ? 1 : 50
+    };
+    this.sessionExpiryTime = 1000 * 60 * 5;
     this.setupCollisionDetection();
     if (this._autoTickEnabled) {
       this.loop();
@@ -291,7 +304,7 @@ export class RpgMap extends RpgCommonMap<RpgPlayer> implements RoomOnJoin {
         // One of the entities is a shape
         const shape = shapeA || shapeB;
         const otherEntity = shapeA ? entityB : entityA;
-        
+
         if (shape) {
           const shapeKey = `${otherEntity.uuid}-${shape.name}`;
           if (!activeShapeCollisions.has(shapeKey)) {
@@ -327,7 +340,7 @@ export class RpgMap extends RpgCommonMap<RpgPlayer> implements RoomOnJoin {
       if (event) {
         // Mark this collision as processed
         activeCollisions.add(collisionKey);
-        
+
         // Trigger the onPlayerTouch hook on the event
         event.execMethod('onPlayerTouch', [player]);
       }
@@ -350,7 +363,7 @@ export class RpgMap extends RpgCommonMap<RpgPlayer> implements RoomOnJoin {
         // One of the entities is a shape
         const shape = shapeA || shapeB;
         const otherEntity = shapeA ? entityB : entityA;
-        
+
         if (shape) {
           const shapeKey = `${otherEntity.uuid}-${shape.name}`;
           if (activeShapeCollisions.has(shapeKey)) {
@@ -467,7 +480,11 @@ export class RpgMap extends RpgCommonMap<RpgPlayer> implements RoomOnJoin {
    * ```
    */
   onJoin(player: RpgPlayer, conn: MockConnection) {
-    player.map = this;
+    if (player.setMap) {
+      player.setMap(this);
+    } else {
+      player.map = this;
+    }
     player.context = context;
     player.conn = conn;
     player._onInit()
@@ -477,17 +494,17 @@ export class RpgMap extends RpgCommonMap<RpgPlayer> implements RoomOnJoin {
         if ((this as any).stopAllSoundsBeforeJoin) {
           player.stopAllSounds();
         }
-        
-        this.sounds.forEach(sound => player.playSound(sound,{ loop: true }));
-        
+
+        this.sounds.forEach(sound => player.playSound(sound, { loop: true }));
+
         // Execute global map hooks (from RpgServer.map)
         await lastValueFrom(this.hooks.callHooks("server-map-onJoin", player, this));
-        
+
         // // Execute map-specific hooks (from @MapData or MapOptions)
         if (typeof (this as any)._onJoin === 'function') {
           await (this as any)._onJoin(player);
         }
-        
+
         // Execute player hooks
         await lastValueFrom(this.hooks.callHooks("server-player-onJoinMap", player, this));
       })
@@ -520,12 +537,12 @@ export class RpgMap extends RpgCommonMap<RpgPlayer> implements RoomOnJoin {
   async onLeave(player: RpgPlayer, conn: MockConnection) {
     // Execute global map hooks (from RpgServer.map)
     await lastValueFrom(this.hooks.callHooks("server-map-onLeave", player, this));
-    
+
     // Execute map-specific hooks (from @MapData or MapOptions)
     if (typeof (this as any)._onLeave === 'function') {
       await (this as any)._onLeave(player);
     }
-    
+
     // Execute player hooks
     await lastValueFrom(this.hooks.callHooks("server-player-onLeaveMap", player, this));
     player.pendingInputs = [];
@@ -549,89 +566,6 @@ export class RpgMap extends RpgCommonMap<RpgPlayer> implements RoomOnJoin {
     return inject<Hooks>(context, ModulesToken);
   }
 
-  /**
-   * Get the width of the map in pixels
-   * 
-   * @returns The width of the map in pixels, or 0 if not loaded
-   * 
-   * @example
-   * ```ts
-   * const width = map.widthPx;
-   * console.log(`Map width: ${width}px`);
-   * ```
-   */
-  get widthPx(): number {
-    return this.data()?.width ?? 0
-  }
-
-  /**
-   * Get the height of the map in pixels
-   * 
-   * @returns The height of the map in pixels, or 0 if not loaded
-   * 
-   * @example
-   * ```ts
-   * const height = map.heightPx;
-   * console.log(`Map height: ${height}px`);
-   * ```
-   */
-  get heightPx(): number {
-    return this.data()?.height ?? 0
-  }
-
-  /**
-   * Get the unique identifier of the map
-   * 
-   * @returns The map ID, or empty string if not loaded
-   * 
-   * @example
-   * ```ts
-   * const mapId = map.id;
-   * console.log(`Current map: ${mapId}`);
-   * ```
-   */
-  get id(): string {
-    return this.data()?.id ?? ''
-  }
-
-  /**
-   * Get the X position of this map in the world coordinate system
-   * 
-   * This is used when maps are part of a larger world map. The world position
-   * indicates where this map is located relative to other maps.
-   * 
-   * @returns The X position in world coordinates, or 0 if not in a world
-   * 
-   * @example
-   * ```ts
-   * const worldX = map.worldX;
-   * console.log(`Map is at world position (${worldX}, ${map.worldY})`);
-   * ```
-   */
-  get worldX(): number {
-    const worldMaps = this.getWorldMapsManager?.();
-    return worldMaps?.getMapInfo(this.id)?.worldX ?? 0
-  }
-  
-  /**
-   * Get the Y position of this map in the world coordinate system
-   * 
-   * This is used when maps are part of a larger world map. The world position
-   * indicates where this map is located relative to other maps.
-   * 
-   * @returns The Y position in world coordinates, or 0 if not in a world
-   * 
-   * @example
-   * ```ts
-   * const worldY = map.worldY;
-   * console.log(`Map is at world position (${map.worldX}, ${worldY})`);
-   * ```
-   */
-  get worldY(): number {
-    const worldMaps = this.getWorldMapsManager?.();
-    return worldMaps?.getMapInfo(this.id)?.worldY ?? 0
-  }
-
   /**
    * Handle GUI interaction from a player
    * 
@@ -649,7 +583,7 @@ export class RpgMap extends RpgCommonMap<RpgPlayer> implements RoomOnJoin {
    */
   @Action('gui.interaction')
   guiInteraction(player: RpgPlayer, value) {
-    //this.hooks.callHooks("server-player-guiInteraction", player, value);
+    this.hooks.callHooks("server-player-guiInteraction", player, value);
     player.syncChanges();
   }
 
@@ -791,6 +725,7 @@ export class RpgMap extends RpgCommonMap<RpgPlayer> implements RoomOnJoin {
     }
     await lastValueFrom(this.hooks.callHooks("server-maps-load", this))
     await lastValueFrom(this.hooks.callHooks("server-worldMaps-load", this))
+    await lastValueFrom(this.hooks.callHooks("server-databaseHooks-load", this))
 
     map.events = map.events ?? []
 
@@ -811,7 +746,7 @@ export class RpgMap extends RpgCommonMap<RpgPlayer> implements RoomOnJoin {
       else {
         this.sounds = map.sounds ?? []
       }
-      
+
       // Attach map-specific hooks from MapOptions or @MapData
       if (mapFound?.onLoad) {
         (this as any)._onLoad = mapFound.onLoad;
@@ -836,15 +771,15 @@ export class RpgMap extends RpgCommonMap<RpgPlayer> implements RoomOnJoin {
     }
 
     this.dataIsReady$.complete()
-    
+
     // Execute global map hooks (from RpgServer.map)
     await lastValueFrom(this.hooks.callHooks("server-map-onLoad", this))
-    
+
     // Execute map-specific hooks (from @MapData or MapOptions)
     if (typeof (this as any)._onLoad === 'function') {
       await (this as any)._onLoad();
     }
-    
+
     // TODO: Update map
   }
 
@@ -1079,7 +1014,7 @@ export class RpgMap extends RpgCommonMap<RpgPlayer> implements RoomOnJoin {
     if (this._inputLoopSubscription) {
       this._inputLoopSubscription.unsubscribe();
     }
-    
+
     this._inputLoopSubscription = this.tick$.pipe(
       throttleTime(50) // Throttle to 50ms for input processing
     ).subscribe(async ({ timestamp }) => {
@@ -1232,8 +1167,7 @@ export class RpgMap extends RpgCommonMap<RpgPlayer> implements RoomOnJoin {
   /**
    * Add data to the map's database
    * 
-   * This method allows you to dynamically add items, classes, or any data to the map's database.
-   * By default, if an ID already exists, the operation is ignored to prevent overwriting existing data.
+   * This method delegates to BaseRoom's implementation to avoid code duplication.
    * 
    * @param id - Unique identifier for the data
    * @param data - The data to store (can be a class, object, or any value)
@@ -1257,23 +1191,13 @@ export class RpgMap extends RpgCommonMap<RpgPlayer> implements RoomOnJoin {
    * ```
    */
   addInDatabase(id: string, data: any, options?: { force?: boolean }): boolean {
-    const database = this.database();
-    
-    // Check if ID already exists
-    if (database[id] !== undefined && !options?.force) {
-      // Ignore the addition if ID exists and force is not enabled
-      return false;
-    }
-    
-    // Add or overwrite the data
-    database[id] = data;
-    return true;
+    return BaseRoom.prototype.addInDatabase.call(this, id, data, options);
   }
 
   /**
    * Remove data from the map's database
    * 
-   * This method allows you to remove items or data from the map's database.
+   * This method delegates to BaseRoom's implementation to avoid code duplication.
    * 
    * @param id - Unique identifier of the data to remove
    * @returns true if data was removed, false if ID didn't exist
@@ -1291,16 +1215,7 @@ export class RpgMap extends RpgCommonMap<RpgPlayer> implements RoomOnJoin {
    * ```
    */
   removeInDatabase(id: string): boolean {
-    const database = this.database();
-    
-    // Check if ID exists
-    if (database[id] === undefined) {
-      return false;
-    }
-    
-    // Remove the data
-    delete database[id];
-    return true;
+    return BaseRoom.prototype.removeInDatabase.call(this, id);
   }
 
   /**
@@ -1406,9 +1321,8 @@ export class RpgMap extends RpgCommonMap<RpgPlayer> implements RoomOnJoin {
     eventInstance.map = this;
     eventInstance.context = context;
 
-    eventInstance.x.set(x);
-    eventInstance.y.set(y);
-    
+    await eventInstance.teleport({ x, y });
+
     this.events()[id] = eventInstance;
 
     await eventInstance.execMethod('onInit')
@@ -1708,6 +1622,20 @@ export class RpgMap extends RpgCommonMap<RpgPlayer> implements RoomOnJoin {
     }
   }
 
+  /**
+   * Apply sync to the client
+   * 
+   * This method applies sync to the client by calling the `$applySync()` method.
+   * 
+   * @example
+   * ```ts
+   * map.applySyncToClient();
+   * ```
+   */
+  applySyncToClient() {
+    this.$applySync();
+  }
+
   /**
    * Create a shape dynamically on the map
    * 
@@ -1784,7 +1712,7 @@ export class RpgMap extends RpgCommonMap<RpgPlayer> implements RoomOnJoin {
     properties?: Record<string, any>;
   }): RpgShape {
     const { x, y, width, height } = obj;
-    
+
     // Validate required parameters
     if (typeof x !== 'number' || typeof y !== 'number') {
       throw new Error('Shape x and y must be numbers');
@@ -2110,8 +2038,4 @@ export class RpgMap extends RpgCommonMap<RpgPlayer> implements RoomOnJoin {
   }
 }
 
-export interface RpgMap {
-  $send: (conn: MockConnection, data: any) => void;
-  $broadcast: (data: any) => void;
-  $sessionTransfer: (userOrPublicId: any | string, targetRoomId: string) => void;
-}
+export interface RpgMap extends RoomMethods { }

package/tests/change-map.spec.ts

@@ -67,6 +67,6 @@ test('Player can change map', async () => {
     expect(newMap).toBeDefined()
     expect(newMap?.id).toBe('map2')
     
-    expect(player.x()).toBe(200 - player.hitbox().h / 2)
-    expect(player.y()).toBe(200 - player.hitbox().w / 2)
+    expect(player.x()).toBe(200)
+    expect(player.y()).toBe(200)
 })

package/tests/event.spec.ts

@@ -0,0 +1,80 @@
+import { beforeEach, test, expect, afterEach } from 'vitest'
+import { testing, TestingFixture } from '@rpgjs/testing'
+import { defineModule, createModule } from '@rpgjs/common'
+import { RpgPlayer, RpgServer, Move } from '../src'
+import { RpgClient } from '../../client/src'
+
+const Event = () => {
+  return {
+    name: "EV-1",
+    onInit() {
+      this.setGraphic("hero");
+    }
+  }
+}
+
+// Define server module with two maps
+const serverModule = defineModule<RpgServer>({
+  maps: [
+    {
+      id: 'map1',
+      events: [{ event: Event(), x: 100, y: 150 }]
+    },
+  ],
+  player: {
+    async onConnected(player) {
+      await player.changeMap('map1', { x: 100, y: 126 })
+    }
+  }
+})
+
+// Define client module
+const clientModule = defineModule<RpgClient>({
+  // Client-side logic
+})
+
+let player: RpgPlayer
+let client: any
+let fixture: TestingFixture
+
+beforeEach(async () => {
+    const myModule = createModule('TestModule', [{
+        server: serverModule,
+        client: clientModule
+    }])
+    
+    fixture = await testing(myModule)
+    client = await fixture.createClient()
+    player = client.player
+})
+
+afterEach(() => {
+  fixture.clear()
+})
+
+test('Player to touch event', async () => {
+    player = await client.waitForMapChange('map1')
+    const map = player.getCurrentMap()
+    const event = map?.getEvents()[0]
+    expect(event).toBeDefined()
+    expect(event?.name()).toBe("EV-1")
+    expect(event?.x()).toBe(100)
+    expect(event?.y()).toBe(150)
+    await fixture.waitUntil(
+        player.moveRoutes([
+          Move.tileDown(2),
+        ], {
+          onStuck: () => false
+        })
+    )
+    expect(event?.x()).toBe(100)
+    expect(event?.y()).toBe(150)
+    await fixture.waitUntil(
+      event!.moveRoutes([
+        Move.down()
+      ])
+    )
+    expect(event?.x()).toBe(100)
+    expect(event?.y()).toBe(150 + event!.speed())
+   
+})