Fixes

Author: VariableVince

src/core/GameRunner.ts

@@ -196,15 +196,14 @@ export class GameRunner {
     playerID: PlayerID,
     x?: number,
     y?: number,
-    units?: readonly PlayerBuildableUnitType[] | null,
+    units?: PlayerBuildableUnitType[] | null,
   ): PlayerActions {
     const player = this.game.player(playerID);
     const tile =
       x !== undefined && y !== undefined ? this.game.ref(x, y) : null;
     const actions = {
       canAttack: tile !== null && player.canAttack(tile),
-      buildableUnits:
-        units === null ? [] : player.buildableUnits(tile, units),
+      buildableUnits: units === null ? [] : player.buildableUnits(tile, units),
       canSendEmojiAllPlayers: player.canSendEmoji(AllPlayers),
       canEmbargoAll: player.canEmbargoAll(),
     } as PlayerActions;

src/core/execution/PlayerExecution.ts

@@ -1,5 +1,12 @@
 import { Config } from "../configuration/Config";
-import { Cell, Execution, Game, isStructureType, Player, UnitType } from "../game/Game";
+import {
+  Cell,
+  Execution,
+  Game,
+  isStructureType,
+  Player,
+  UnitType,
+} from "../game/Game";
 import { TileRef } from "../game/GameMap";
 import { calculateBoundingBox, getMode, inscribed, simpleHash } from "../Util";
 
@@ -35,7 +42,7 @@ export class PlayerExecution implements Execution {
   tick(ticks: number) {
     this.player.decayRelations();
     for (const u of this.player.units()) {
-      if (!isStructureType(u.type())){
+      if (!isStructureType(u.type())) {
         continue;
       }
 

src/core/game/Game.ts

@@ -656,7 +656,7 @@ export interface Player {
   unitsOwned(type: UnitType): number;
   buildableUnits(
     tile: TileRef | null,
-    units?: readonly PlayerBuildableUnitType[],
+    units?: PlayerBuildableUnitType[],
   ): BuildableUnit[];
   canBuild(type: UnitType, targetTile: TileRef): TileRef | false;
   buildUnit<T extends UnitType>(

src/core/game/GameView.ts

@@ -406,7 +406,7 @@ export class PlayerView {
 
   async actions(
     tile?: TileRef,
-    units?: readonly PlayerBuildableUnitType[] | null,
+    units?: PlayerBuildableUnitType[] | null,
   ): Promise<PlayerActions> {
     return this.game.worker.playerInteraction(
       this.id(),

src/core/worker/WorkerClient.ts

@@ -165,7 +165,7 @@ export class WorkerClient {
     playerID: PlayerID,
     x?: number,
     y?: number,
-    units?: readonly PlayerBuildableUnitType[] | null,
+    units?: PlayerBuildableUnitType[] | null,
   ): Promise<PlayerActions> {
     return new Promise((resolve, reject) => {
       if (!this.isInitialized) {

src/core/worker/WorkerMessages.ts

@@ -63,7 +63,7 @@ export interface PlayerActionsMessage extends BaseWorkerMessage {
   playerID: PlayerID;
   x?: number;
   y?: number;
-  units?: readonly PlayerBuildableUnitType[];
+  units?: PlayerBuildableUnitType[] | null;
 }
 
 export interface PlayerActionsResultMessage extends BaseWorkerMessage {