fix(script): prevent actors from hitting themselves (#64)

data/base.yml

@@ -34,10 +34,12 @@ worlds:
                   seen: |-
                     You see a {{item.meta.name}}: {{item.meta.desc}} ({{item.meta.id}}).
               hit:
-                type: |-
-                  {{cmd.target}} is not an actor!
                 item: |-
                   You cannot hit {{target.meta.name}}, you are not holding anything!
+                self: |-
+                  You cannot hit yourself!
+                type: |-
+                  {{cmd.target}} is not an actor!
               drop:
                 type: |-
                   {{cmd.target}} is not an item!

src/service/script/common/ActorStep.ts

@@ -119,6 +119,11 @@ export async function ActorStepHit(this: Actor, context: ScriptContext): Promise
     return;
   }
 
+  if (this === target) {
+    await context.focus.show('actor.step.hit.self', { cmd });
+    return;
+  }
+
   if (this.items.length === 0) {
     await context.focus.show('actor.step.hit.item', { target });
     return;

src/service/state/LocalStateService.ts

@@ -1,4 +1,4 @@
-import { constructorName, isNil, mustExist, NotFoundError } from '@apextoaster/js-utils';
+import { constructorName, InvalidArgumentError, isNil, mustExist, NotFoundError } from '@apextoaster/js-utils';
 import { EventEmitter } from 'events';
 import { BaseOptions, Container, Inject, Logger } from 'noicejs';
 
@@ -240,7 +240,7 @@ export class LocalStateService extends EventEmitter implements StateService {
       type: ACTOR_TYPE,
     });
     if (!isActor(player)) {
-      throw new Error('invalid focus actor');
+      throw new InvalidArgumentError('invalid focus actor');
     }
 
     const input = await this.getActorInput(player);