From 8dd5f654ff03c7b99fff9e60e9d46f78dcf50528 Mon Sep 17 00:00:00 2001
From: Evan <evanpelle@gmail.com>
Date: Fri, 28 Mar 2025 14:02:29 -0700
Subject: [PATCH] Attacks don't create embargos against bots because they can't
 trade anyways

---
 src/core/execution/AttackExecution.ts | 9 ++++++++-
 1 file changed, 8 insertions(+), 1 deletion(-)

diff --git a/src/core/execution/AttackExecution.ts b/src/core/execution/AttackExecution.ts
index 08dc212d2..364ddef51 100644
--- a/src/core/execution/AttackExecution.ts
+++ b/src/core/execution/AttackExecution.ts
@@ -82,7 +82,14 @@ export class AttackExecution implements Execution {
         : mg.player(this._targetID);
 
     if (this.target && this.target.isPlayer()) {
-      (this.target as Player).addEmbargo(this._owner.id());
+      const targetPlayer = this.target as Player;
+      if (
+        targetPlayer.type() != PlayerType.Bot &&
+        this._owner.type() != PlayerType.Bot
+      ) {
+        // Don't let bots embargo since they can't trade anyways.
+        targetPlayer.addEmbargo(this._owner.id());
+      }
     }
 
     if (this._owner == this.target) {