chore: rename battleLockManager to lockManager

Author: t7ru

src/commands/battle.ts

@@ -14,7 +14,7 @@ import { createCanvas, loadImage } from "canvas";
 import path from "path";
 import { generateFighter, Fighter } from "../utils/fighterGenerator.js";
 import { BattleStatsManager } from "../utils/battleStatsManager.js";
-import { BattleLockManager } from "../utils/battleLockManager.js";
+import { LockManager } from "../utils/lockManager.js";
 
 export const data = new SlashCommandBuilder()
   .setName("battle")
@@ -693,7 +693,7 @@ export async function execute(
     return;
   }
 
-  if (BattleLockManager.isLocked(interaction.guildId!, "battle")) {
+  if (LockManager.isLocked(interaction.guildId!, "battle")) {
     console.log(
       `[DEATHBATTLE] Battle already active in guild ${interaction.guildId}, rejecting new battle`,
     );
@@ -706,8 +706,8 @@ export async function execute(
   }
 
   if (
-    BattleLockManager.isUserBusy(fighter1User.id) ||
-    BattleLockManager.isUserBusy(fighter2User.id)
+    LockManager.isUserBusy(fighter1User.id) ||
+    LockManager.isUserBusy(fighter2User.id)
   ) {
     console.log(
       `[DEATHBATTLE] One of the fighters is already in a battle elsewhere`,
@@ -720,11 +720,10 @@ export async function execute(
     return;
   }
 
-  const lockAcquired = BattleLockManager.acquireLock(
-    interaction.guildId!,
-    "battle",
-    [fighter1User.id, fighter2User.id],
-  );
+  const lockAcquired = LockManager.acquireLock(interaction.guildId!, "battle", [
+    fighter1User.id,
+    fighter2User.id,
+  ]);
 
   if (!lockAcquired) {
     // This is a fallback, should never happen as it should be caught by the checks above
@@ -973,7 +972,7 @@ export async function execute(
       components: [],
     });
   } finally {
-    BattleLockManager.releaseLock(interaction.guildId!, "battle");
+    LockManager.releaseLock(interaction.guildId!, "battle");
     console.log(
       `[DEATHBATTLE] Released battle lock for guild ${interaction.guildId}`,
     );

src/commands/russian.ts

@@ -13,7 +13,7 @@ import {
 import { createCanvas, loadImage } from "canvas";
 import path from "path";
 import { RussianStatsManager } from "../utils/russianStatsManager.js";
-import { BattleLockManager } from "../utils/battleLockManager.js";
+import { LockManager } from "../utils/lockManager.js";
 
 export const data = new SlashCommandBuilder()
   .setName("russian")
@@ -226,7 +226,7 @@ export async function execute(
     return;
   }
 
-  if (BattleLockManager.isLocked(interaction.guildId!, "russian")) {
+  if (LockManager.isLocked(interaction.guildId!, "russian")) {
     await interaction.reply({
       content:
         "**A game of Russian Roulette is already in progress in this server!**",
@@ -236,8 +236,8 @@ export async function execute(
   }
 
   if (
-    BattleLockManager.isUserBusy(inviterUser.id) ||
-    BattleLockManager.isUserBusy(targetUser.id)
+    LockManager.isUserBusy(inviterUser.id) ||
+    LockManager.isUserBusy(targetUser.id)
   ) {
     await interaction.reply({
       content: "**One of the players is already in a game!**",
@@ -246,7 +246,7 @@ export async function execute(
     return;
   }
 
-  const lockAcquired = BattleLockManager.acquireLock(
+  const lockAcquired = LockManager.acquireLock(
     interaction.guildId!,
     "russian",
     [inviterUser.id, targetUser.id],
@@ -267,7 +267,7 @@ export async function execute(
   );
 
   if (!consentGiven) {
-    BattleLockManager.releaseLock(interaction.guildId!, "russian");
+    LockManager.releaseLock(interaction.guildId!, "russian");
     await interaction.editReply({
       content: "**Game cancelled! One or both players did not accept.**",
       embeds: [],
@@ -498,7 +498,7 @@ export async function execute(
   });
 
   collector.on("end", (collected, reason) => {
-    BattleLockManager.releaseLock(interaction.guildId!, "russian");
+    LockManager.releaseLock(interaction.guildId!, "russian");
     if (turnTimer) clearTimeout(turnTimer);
     if (reason === "time") {
       interaction.editReply({

src/index.ts

@@ -19,7 +19,7 @@ import {
   handleReactionAdd,
   handleReactionRemove,
 } from "./utils/eventHandlers.js";
-import { BattleLockManager } from "./utils/battleLockManager.js";
+import { LockManager } from "./utils/lockManager.js";
 
 class AltershaperBot {
   private client: Client;
@@ -289,7 +289,7 @@ class AltershaperBot {
     console.log("🔒 Active Battle Locks:");
     console.log("================================");
 
-    const allLocks = BattleLockManager.getAllLocks();
+    const allLocks = LockManager.getAllLocks();
 
     if (allLocks.size === 0) {
       console.log("No active battle locks\n");

src/utils/battleLockManager.ts src/utils/lockManager.tssrc/utils/battleLockManager.ts renamed to src/utils/lockManager.ts

@@ -9,7 +9,7 @@ interface GameLock {
 
 const activeLocks = new Map<string, Map<GameType, GameLock>>();
 
-export class BattleLockManager {
+export class LockManager {
   private static readonly TIMEOUT = 1000 * 60 * 5; // 5 minutes
 
   // Checks if a specific game type is active in a server