fix(types): Always allow player ID array as active players argument

Fixes #1041

Author: delucis

src/core/flow.ts

@@ -748,7 +748,7 @@ export function Flow({
   function SetActivePlayersEvent(
     state: State,
     _playerID: PlayerID,
-    arg: ActivePlayersArg | PlayerID[]
+    arg: ActivePlayersArg
   ): State {
     return Process(state, [{ fn: UpdateActivePlayers, arg }]);
   }

src/core/turn-order.ts

@@ -18,10 +18,7 @@ import type {
 } from '../types';
 import { supportDeprecatedMoveLimit } from './backwards-compatibility';
 
-export function SetActivePlayers(
-  ctx: Ctx,
-  arg: ActivePlayersArg | PlayerID[]
-): Ctx {
+export function SetActivePlayers(ctx: Ctx, arg: ActivePlayersArg): Ctx {
   let activePlayers: typeof ctx.activePlayers = {};
   let _prevActivePlayers: typeof ctx._prevActivePlayers = [];
   let _nextActivePlayers: ActivePlayersArg | null = null;

src/types.ts

@@ -69,18 +69,20 @@ export type StageArg =
       maxMoves?: number;
     };
 
-export interface ActivePlayersArg {
-  currentPlayer?: StageArg;
-  others?: StageArg;
-  all?: StageArg;
-  value?: Record<PlayerID, StageArg>;
-  minMoves?: number;
-  maxMoves?: number;
-  /** @deprecated Use `minMoves` and `maxMoves` instead. */
-  moveLimit?: number;
-  revert?: boolean;
-  next?: ActivePlayersArg;
-}
+export type ActivePlayersArg =
+  | PlayerID[]
+  | {
+      currentPlayer?: StageArg;
+      others?: StageArg;
+      all?: StageArg;
+      value?: Record<PlayerID, StageArg>;
+      minMoves?: number;
+      maxMoves?: number;
+      /** @deprecated Use `minMoves` and `maxMoves` instead. */
+      moveLimit?: number;
+      revert?: boolean;
+      next?: ActivePlayersArg;
+    };
 
 export interface ActivePlayers {
   [playerID: string]: StageName;