@rpgjs/server 5.0.0-alpha.32 → 5.0.0-alpha.33

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@rpgjs/server",
-  "version": "5.0.0-alpha.32",
+  "version": "5.0.0-alpha.33",
   "main": "./dist/index.js",
   "types": "./dist/index.d.ts",
   "publishConfig": {
@@ -11,14 +11,14 @@
   "license": "MIT",
   "description": "",
   "dependencies": {
-    "@rpgjs/common": "5.0.0-alpha.32",
-    "@rpgjs/physic": "5.0.0-alpha.32",
-    "@rpgjs/testing": "5.0.0-alpha.32",
+    "@rpgjs/common": "5.0.0-alpha.33",
+    "@rpgjs/physic": "5.0.0-alpha.33",
+    "@rpgjs/testing": "5.0.0-alpha.33",
     "@rpgjs/database": "^4.3.0",
-    "@signe/di": "^2.8.2",
-    "@signe/reactive": "^2.8.2",
-    "@signe/room": "^2.8.2",
-    "@signe/sync": "^2.8.2",
+    "@signe/di": "^2.8.3",
+    "@signe/reactive": "^2.8.3",
+    "@signe/room": "^2.8.3",
+    "@signe/sync": "^2.8.3",
     "rxjs": "^7.8.2",
     "zod": "^4.3.5"
   },

package/src/Gui/ShopGui.ts

@@ -3,9 +3,10 @@ import { Gui } from './Gui'
 import { RpgPlayer } from '../Player/Player'
 
 export type ShopSellList = Record<string, number> | Array<{ id: string; multiplier: number }>
+export type ShopItemInput = string | { id?: string; [key: string]: any }
 
 export interface ShopGuiOptions {
-    items: any[]
+    items: ShopItemInput[]
     sell?: ShopSellList
     sellMultiplier?: number
     message?: string
@@ -16,7 +17,7 @@ export interface ShopGuiOptions {
 }
 
 export class ShopGui extends Gui {
-    private itemsInput: any[] = []
+    private itemsInput: ShopItemInput[] = []
     private sellMultipliers: Record<string, number> = {}
     private baseSellMultiplier = 0.5
     private messageInput?: string
@@ -65,7 +66,7 @@ export class ShopGui extends Gui {
             return undefined
         }
 
-        const buildItemData = (item, overrides: { price?: number; quantity?: number } = {}) => {
+        const buildItemData = (item: ShopItemInput, overrides: { price?: number; quantity?: number } = {}) => {
             const rawId = typeof item === 'string'
                 ? item
                 : (typeof item?.id === 'function' ? item.id() : (item?.id ?? item?.name))

package/src/Player/ClassManager.ts

@@ -6,8 +6,8 @@ type ActorClass = any;
 interface PlayerWithMixins extends RpgCommonPlayer {
   databaseById(id: string): any;
   addParameter(name: string, { start, end }: { start: number, end: number }): void;
-  addItem(item: any): void;
-  equip(item: any, equip: boolean): void;
+  addItem(item: any): any;
+  equip(itemId: string, equip?: boolean | 'auto'): void;
 }
 
 /**
@@ -108,8 +108,11 @@ export function WithClassManager<TBase extends PlayerCtor>(Base: TBase) {
         (this as any).addParameter(param, actor.parameters[param]);
       }
       for (let item of actor.startingEquipment) {
-        (this as any).addItem(item);
-        (this as any).equip(item, true);
+        const inventory = (this as any).addItem(item);
+        const itemId = inventory?.id?.();
+        if (itemId) {
+          (this as any).equip(itemId, true);
+        }
       }
       if (actor.class) this.setClass(actor.class);
       (this as any)["execMethod"]("onSet", [this], actor);

package/src/Player/ItemManager.ts

@@ -581,15 +581,19 @@ export function WithItemManager<TBase extends PlayerCtor>(Base: TBase) {
     }
 
     equip(
-      itemClass: ItemClass | string,
-      equip: boolean = true
+      itemId: string,
+      equip: boolean | 'auto' = true
     ): void {
-      const itemId = isString(itemClass) ? itemClass : (itemClass as any).name;
-      const inventory: Item = this.getItem(itemClass);
+      const autoAdd = equip === 'auto';
+      const equipState = equip === 'auto' ? true : equip;
+      const data = (this as any).databaseById(itemId);
+      let inventory: Item = this.getItem(itemId);
+      if (!inventory && autoAdd) {
+        inventory = this.addItem(itemId, 1);
+      }
       if (!inventory) {
         throw ItemLog.notInInventory(itemId);
       }
-      const data = (this as any).databaseById(itemId);
       if (data._type == "item") {
         throw ItemLog.invalidToEquiped(itemId);
       }
@@ -607,11 +611,11 @@ export function WithItemManager<TBase extends PlayerCtor>(Base: TBase) {
 
       const item = inventory;
 
-      if ((item as any).equipped && equip) {
+      if ((item as any).equipped && equipState) {
         throw ItemLog.isAlreadyEquiped(itemId);
       }
-      (item as any).equipped = equip;
-      if (!equip) {
+      (item as any).equipped = equipState;
+      if (!equipState) {
         const index = this.equipments().findIndex((it) => it.id() == item.id());
         this.equipments().splice(index, 1);
       } else {
@@ -619,7 +623,7 @@ export function WithItemManager<TBase extends PlayerCtor>(Base: TBase) {
       }
       // Call onEquip hook - use stored instance if available
       const hookTarget = (item as any)._itemInstance || item;
-      this["execMethod"]("onEquip", [this, equip], hookTarget);
+      this["execMethod"]("onEquip", [this, equipState], hookTarget);
     }
   } as unknown as TBase;
 }
@@ -852,12 +856,13 @@ export interface IItemManager {
   /**
    * Equips a weapon or armor on a player
    * 
-   * Think first to add the item in the inventory with the `addItem()` method before equipping the item.
+   * Think first to add the item in the inventory with the `addItem()` method before equipping the item,
+   * or pass `"auto"` to add the item if it is missing and equip it.
    * 
    * The `onEquip()` method is called on the ItemClass when the item is equipped or unequipped.
    * 
-   * @param itemClass - Item class or string identifier. If string, it's the item ID
-   * @param equip - Equip the item if `true`, unequip if `false` (default: `true`)
+   * @param itemId - Item identifier to resolve from the database
+   * @param equip - Equip the item if `true`, unequip if `false`, or `"auto"` to add then equip (default: `true`)
    * @throws {Object} ItemLog.notInInventory - If the item is not in the inventory
    *   - `id`: `ITEM_NOT_INVENTORY`
    *   - `msg`: Error message
@@ -870,19 +875,17 @@ export interface IItemManager {
    * 
    * @example
    * ```ts
-   * import Sword from 'your-database/sword'
-   * 
    * try {
-   *   player.addItem(Sword)
-   *   player.equip(Sword)
+   *   player.addItem('sword')
+   *   player.equip('sword')
    *   // Later, unequip it
-   *   player.equip(Sword, false)
+   *   player.equip('sword', false)
    * } catch (err) {
    *   console.log(err)
    * }
    * ```
    */
-  equip(itemClass: ItemClass | string, equip?: boolean): void;
+  equip(itemId: string, equip?: boolean | 'auto'): void;
 
   /**
    * Get the player's attack (sum of items equipped)

package/tests/item.spec.ts

@@ -385,6 +385,24 @@ describe("Item Management - Equipment", () => {
     expect((item as any).equipped).toBe(true);
   });
 
+  test("should auto add and equip item", () => {
+    player.equip("TestSword", "auto");
+    const item = player.getItem("TestSword");
+    expect(item).toBeDefined();
+    expect(item?.quantity()).toBe(1);
+    expect((item as any).equipped).toBe(true);
+    expect(player.equipments().some((eq) => eq.id() === "TestSword")).toBe(
+      true
+    );
+  });
+
+  test("should not add duplicate when auto equipping existing item", () => {
+    player.addItem("TestSword", 2);
+    player.equip("TestSword", "auto");
+    const item = player.getItem("TestSword");
+    expect(item?.quantity()).toBe(2);
+  });
+
   test("should throw error when equipping non-existent item", () => {
     expect(() => {
       player.equip("TestSword", true);
@@ -588,4 +606,4 @@ describe("Item Management - Edge Cases", () => {
     expect(player.getItem("TestSword")?.quantity()).toBe(1);
     expect(player.getItem("TestArmor")?.quantity()).toBe(2);
   });
-});
+});