Enable the @typescript-eslint/no-unused-vars eslint rule

eslint.config.js

@@ -41,7 +41,7 @@ export default [
       // Disable rules that would fail. The failures should be fixed, and the entries here removed.
       "@typescript-eslint/no-explicit-any": "off",
       "@typescript-eslint/no-unused-expressions": "off",
-      "@typescript-eslint/no-unused-vars": "off",
+      "no-unused-vars": "off",
     },
   },
   {
@@ -50,6 +50,13 @@ export default [
       "@typescript-eslint/prefer-nullish-coalescing": "error",
       eqeqeq: "error",
       "no-case-declarations": "error",
+      "@typescript-eslint/no-unused-vars": [
+        "error",
+        {
+          args: "none",
+          caughtErrors: "none",
+        },
+      ],
     },
   },
 ];

src/client/components/Difficulties.ts

@@ -74,6 +74,7 @@ export class DifficultyDisplay extends LitElement {
       ></path>
     </svg>`;
 
+    // eslint-disable-next-line @typescript-eslint/no-unused-vars
     const kingSkull = html`<svg
       stroke="currentColor"
       fill="currentColor"

src/client/graphics/fx/SpriteFx.ts

@@ -77,7 +77,6 @@ export class SpriteFx implements Fx {
 
     if (!this.animatedSprite.isActive() && !this.waitToTheEnd) return false;
 
-    const t = this.elapsedTime / this.duration;
     this.animatedSprite.update(frameTime);
     this.animatedSprite.draw(ctx, this.x, this.y);
     return true;

src/client/graphics/layers/RadialMenuElements.ts

@@ -106,6 +106,7 @@ export enum Slot {
   Delete = "delete",
 }
 
+// eslint-disable-next-line @typescript-eslint/no-unused-vars
 const infoChatElement: MenuElement = {
   id: "info_chat",
   name: "chat",
@@ -123,6 +124,7 @@ const infoChatElement: MenuElement = {
       })),
 };
 
+// eslint-disable-next-line @typescript-eslint/no-unused-vars
 const allyTargetElement: MenuElement = {
   id: "ally_target",
   name: "target",
@@ -138,6 +140,7 @@ const allyTargetElement: MenuElement = {
   },
 };
 
+// eslint-disable-next-line @typescript-eslint/no-unused-vars
 const allyTradeElement: MenuElement = {
   id: "ally_trade",
   name: "trade",
@@ -153,6 +156,7 @@ const allyTradeElement: MenuElement = {
   },
 };
 
+// eslint-disable-next-line @typescript-eslint/no-unused-vars
 const allyEmbargoElement: MenuElement = {
   id: "ally_embargo",
   name: "embargo",
@@ -204,6 +208,7 @@ const allyBreakElement: MenuElement = {
   },
 };
 
+// eslint-disable-next-line @typescript-eslint/no-unused-vars
 const allyDonateGoldElement: MenuElement = {
   id: "ally_donate_gold",
   name: "donate gold",
@@ -217,6 +222,7 @@ const allyDonateGoldElement: MenuElement = {
   },
 };
 
+// eslint-disable-next-line @typescript-eslint/no-unused-vars
 const allyDonateTroopsElement: MenuElement = {
   id: "ally_donate_troops",
   name: "donate troops",
@@ -230,6 +236,7 @@ const allyDonateTroopsElement: MenuElement = {
   },
 };
 
+// eslint-disable-next-line @typescript-eslint/no-unused-vars
 const infoPlayerElement: MenuElement = {
   id: "info_player",
   name: "player",
@@ -241,6 +248,7 @@ const infoPlayerElement: MenuElement = {
   },
 };
 
+// eslint-disable-next-line @typescript-eslint/no-unused-vars
 const infoEmojiElement: MenuElement = {
   id: "info_emoji",
   name: "emoji",

src/client/graphics/layers/RailroadLayer.ts

@@ -88,6 +88,7 @@ export class RailroadLayer implements Layer {
     this.canvas.width = this.game.width() * 2;
     this.canvas.height = this.game.height() * 2;
 
+    // eslint-disable-next-line @typescript-eslint/no-unused-vars
     for (const [_, rail] of this.existingRailroads) {
       this.paintRail(rail.tile);
     }
@@ -111,7 +112,9 @@ export class RailroadLayer implements Layer {
 
   private handleRailroadRendering(railUpdate: RailroadUpdate) {
     for (const railRoad of railUpdate.railTiles) {
+      // eslint-disable-next-line @typescript-eslint/no-unused-vars
       const x = this.game.x(railRoad.tile);
+      // eslint-disable-next-line @typescript-eslint/no-unused-vars
       const y = this.game.y(railRoad.tile);
       if (railUpdate.isActive) {
         this.paintRailroad(railRoad);

src/client/graphics/layers/StructureLayer.ts

@@ -246,6 +246,7 @@ export class StructureLayer implements Layer {
   ) {
     let color = unit.owner().borderColor();
     if (unit.type() === UnitType.Construction) {
+      // eslint-disable-next-line @typescript-eslint/no-unused-vars
       color = underConstructionColor;
     }
 

src/client/graphics/layers/TerritoryLayer.ts

@@ -447,12 +447,14 @@ export class TerritoryLayer implements Layer {
       return;
     }
     const owner = this.game.owner(tile) as PlayerView;
+    // eslint-disable-next-line @typescript-eslint/no-unused-vars
     const isHighlighted =
       this.highlightedTerritory &&
       this.highlightedTerritory.id() === owner.id();
     const myPlayer = this.game.myPlayer();
 
     if (this.game.isBorder(tile)) {
+      // eslint-disable-next-line @typescript-eslint/no-unused-vars
       const playerIsFocused = owner && this.game.focusedPlayer() === owner;
       if (myPlayer) {
         const alternativeColor = this.alternateViewColor(owner);

src/core/Schemas.ts

@@ -14,7 +14,6 @@ import {
   GameMapType,
   GameMode,
   GameType,
-  PlayerType,
   Quads,
   Trios,
   UnitType,
@@ -118,7 +117,6 @@ export type PlayerCosmeticRefs = z.infer<typeof PlayerCosmeticRefsSchema>;
 export type PlayerPattern = z.infer<typeof PlayerPatternSchema>;
 export type Flag = z.infer<typeof FlagSchema>;
 export type GameStartInfo = z.infer<typeof GameStartInfoSchema>;
-const PlayerTypeSchema = z.enum(PlayerType);
 
 export interface GameInfo {
   gameID: GameID;

src/core/configuration/DevConfig.ts

@@ -65,6 +65,7 @@ export class DevConfig extends DefaultConfig {
 
   unitInfo(type: UnitType): UnitInfo {
     const info = super.unitInfo(type);
+    // eslint-disable-next-line @typescript-eslint/no-unused-vars
     const oldCost = info.cost;
     // info.cost = (p: Player) => oldCost(p) / 1000000000;
     return info;

src/core/game/TeamAssignment.ts

@@ -33,6 +33,7 @@ export function assignTeams(
   );
 
   // First, assign clan players
+  // eslint-disable-next-line @typescript-eslint/no-unused-vars
   for (const [_, clanPlayers] of sortedClans) {
     // Try to keep the clan together on the team with fewer players
     let team: Team | null = null;

src/core/game/TerrainSearchMap.ts

@@ -18,8 +18,6 @@ export class TerrainSearchMap {
   node(x: number, y: number): SearchMapTileType {
     const packedByte = this.mapData[4 + y * this.width + x];
     const isLand = packedByte & 0b10000000;
-    const shoreline = !!(packedByte & 0b01000000);
-    const ocean = !!(packedByte & 0b00100000);
     const magnitude = packedByte & 0b00011111;
     if (isLand) {
       return SearchMapTileType.Land;

src/server/Master.ts

@@ -285,14 +285,13 @@ async function schedulePublicGame(playlist: MapPlaylist) {
     if (!response.ok) {
       throw new Error(`Failed to schedule public game: ${response.statusText}`);
     }
-
-    const data = await response.json();
   } catch (error) {
     log.error(`Failed to schedule public game on worker ${workerPath}:`, error);
     throw error;
   }
 }
 
+// eslint-disable-next-line @typescript-eslint/no-unused-vars
 function sleep(ms: number): Promise<void> {
   return new Promise((resolve) => setTimeout(resolve, ms));
 }

src/server/jwt.ts

@@ -27,7 +27,7 @@ export async function verifyClientToken(
     const issuer = config.jwtIssuer();
     const audience = config.jwtAudience();
     const key = await config.jwkPublicKey();
-    const { payload, protectedHeader } = await jwtVerify(token, key, {
+    const { payload } = await jwtVerify(token, key, {
       algorithms: ["EdDSA"],
       issuer,
       audience,

tests/ShellRandom.test.ts

@@ -85,7 +85,6 @@ describe("Shell Random Damage", () => {
       expect(damage).toBeLessThanOrEqual(maxExpectedDamage);
     });
 
-    const uniqueDamages = new Set(damages);
     expect(damages.length).toBeGreaterThan(0);
   });
 
@@ -231,16 +230,6 @@ describe("Shell Random Damage", () => {
 
     expect(damages.length).toBeGreaterThan(0);
 
-    const baseDamage = game.config().unitInfo(UnitType.Shell).damage ?? 250;
-    const expectedDamages = [
-      Math.round((baseDamage / 250) * 200),
-      Math.round((baseDamage / 250) * 225),
-      Math.round((baseDamage / 250) * 250),
-      Math.round((baseDamage / 250) * 275),
-      Math.round((baseDamage / 250) * 300),
-      Math.round((baseDamage / 250) * 325),
-    ];
-
     const uniqueDamages = new Set(damages);
     expect(uniqueDamages.size).toBeGreaterThan(0);
 
@@ -265,7 +254,6 @@ describe("Shell Random Damage", () => {
     );
     const initialHealth = target.health();
 
-    const seed = 12345;
     const shell1 = new ShellExecution(
       game.ref(coastX, 10),
       player1,

tests/core/executions/SAMLauncherExecution.test.ts

@@ -83,6 +83,7 @@ describe("SAM", () => {
     game.addExecution(new SAMLauncherExecution(defender, null, sam));
 
     // Sam will only target nukes it can destroy before it reaches its target
+    // eslint-disable-next-line @typescript-eslint/no-unused-vars
     const nuke = attacker.buildUnit(UnitType.AtomBomb, game.ref(1, 1), {
       targetTile: game.ref(3, 1),
       trajectory: [