Don't clobber readonly "roles" array when using it as active player list.

2 files changed, 3 insertions, 3 deletions

diff --git a/rules.js b/rules.js
index ced9de7..e068bbf 100644
--- a/rules.js
+++ b/rules.js
@@ -292,7 +292,7 @@ function get_next_active(p) {
         return p.map((faction) => faction_player_map[faction]);
     }
     if (p === 'all') {
-        return exports.roles;
+        return exports.roles.slice();
     }
     if (p === 'None') {
         return 'None';
diff --git a/rules.ts b/rules.ts
index 98b10b9..1939d74 100644
--- a/rules.ts
+++ b/rules.ts
@@ -122,7 +122,7 @@ const fascist_decks = {
 
 export const scenarios = ['Standard'];
 
-export const roles: Player[] = [ANARCHIST, COMMUNIST, MODERATE];
+export const roles: readonly Player[] = [ANARCHIST, COMMUNIST, MODERATE];
 
 function gen_action(action: string, argument?: number | string) {
   if (argument === undefined) {
@@ -443,7 +443,7 @@ function get_next_active(p: StateNode['p']): Player | Player[] | 'None' {
     return p.map((faction) => faction_player_map[faction]);
   }
   if (p === 'all') {
-    return roles;
+    return roles.slice();
   }
   if (p === 'None') {
     return 'None';