Dependencies: Update dependencies and lint tooling

Update runtime and development dependencies to current compatible
versions, keeping TypeScript on 5.9 instead of taking the TS 6
migration in this change.

Move ESLint to flat config for ESLint 10 and add narrow overrides
for transitive ws/flatted fixes that stay within upstream semver
ranges.

Author: gmta

.eslintignore

@@ -0,0 +1,25 @@
+import js from "@eslint/js";
+import typescriptEslint from "@typescript-eslint/eslint-plugin";
+import eslintConfigPrettier from "eslint-config-prettier";
+import prettier from "eslint-plugin-prettier";
+import unusedImports from "eslint-plugin-unused-imports";
+
+export default [
+  {
+    ignores: ["node_modules/**", "build/**"],
+  },
+  js.configs.recommended,
+  ...typescriptEslint.configs["flat/recommended"],
+  eslintConfigPrettier,
+  {
+    files: ["**/*.ts"],
+    plugins: {
+      prettier,
+      "unused-imports": unusedImports,
+    },
+    rules: {
+      "prettier/prettier": "error",
+      camelcase: "error",
+    },
+  },
+];

.eslintrc

@@ -27,23 +27,26 @@
   },
   "homepage": "https://github.com/LadybirdBrowser/discord-bot#readme",
   "devDependencies": {
-    "@types/bun": "latest",
-    "@typescript-eslint/eslint-plugin": "^7.15.0",
-    "@typescript-eslint/parser": "^7.15.0",
-    "eslint": "^8.56.0",
-    "eslint-config-prettier": "^8.8.0",
-    "eslint-plugin-prettier": "^5.1.3",
-    "eslint-plugin-unused-imports": "^4.0.0",
-    "prettier": "^3.3.2"
+    "@eslint/js": "^10.0.1",
+    "@types/bun": "^1.3.14",
+    "@typescript-eslint/eslint-plugin": "^8.60.1",
+    "@typescript-eslint/parser": "^8.60.1",
+    "eslint": "^10.4.1",
+    "eslint-config-prettier": "^10.1.8",
+    "eslint-plugin-prettier": "^5.5.6",
+    "eslint-plugin-unused-imports": "^4.4.1",
+    "prettier": "^3.8.3",
+    "typescript": "^5.9.3"
   },
   "dependencies": {
-    "@octokit/plugin-throttling": "^9.3.0",
-    "@octokit/rest": "^21.0.0",
-    "axios": "^1.4.0",
-    "discord.js": "^14.16.3",
-    "zod": "^3.23.8"
+    "@octokit/plugin-throttling": "^11.0.3",
+    "@octokit/rest": "^22.0.1",
+    "axios": "^1.17.0",
+    "discord.js": "^14.26.4",
+    "zod": "^4.4.3"
   },
-  "peerDependencies": {
-    "typescript": "^5.5.3"
+  "overrides": {
+    "flatted": "^3.4.0",
+    "ws": "^8.20.1"
   }
 }

bun.lockb

@@ -59,7 +59,6 @@ class GithubAPI {
   private readonly octokit: Octokit;
 
   constructor() {
-    // @ts-expect-error -- TODO: types need fixes
     this.octokit = new (Octokit.plugin(OctokitThrottling))({
       userAgent: "Ladybird-Bot",
       auth: env.GITHUB_TOKEN,

eslint.config.mjs

@@ -77,7 +77,8 @@ export default class CommandHandler {
     )} from '${interaction.user.tag}'`;
     console.log(msg);
 
-    if (!interaction.isCommand()) throw new Error("Invalid command interaction");
+    if (!interaction.isChatInputCommand() && !interaction.isContextMenuCommand())
+      throw new Error("Invalid command interaction");
 
     if (interaction.commandName === "help") {
       await interaction.reply({
@@ -120,7 +121,7 @@ export default class CommandHandler {
       throw new Error(`${matchedCommand.constructor.name}: Missing handleContextMenu handler`);
     }
 
-    // NOTE: At this point we can we be sure that it's a command
+    // NOTE: At this point we can be sure that it's a chat input command.
     return this.callInteractionHandler(matchedCommand, matchedCommand.handleCommand, interaction);
   }
 

package.json

@@ -11,7 +11,6 @@ import { getLadybird, getMakemore, getSadCaret } from "@/util/emoji";
 import Command from "./command";
 import githubAPI from "@/apis/githubAPI";
 
-/* eslint-disable camelcase */
 interface Result {
   commit_timestamp: number;
   run_timestamp: number;
@@ -25,7 +24,6 @@ interface Result {
     };
   };
 }
-/* eslint-enable camelcase */
 
 interface TestVariant {
   description: string;

src/apis/githubAPI.ts

@@ -49,14 +49,14 @@ export class UserCommand extends Command {
 
     if (username === null) {
       const message = await noMatchingFoundMessage(interaction);
-      await interaction.editReply(message);
+      await interaction.editReply({ content: message.content });
       throw new Error(message.content);
     }
 
     const response = await githubAPI.fetchUserIssuesAndPulls(username).catch(() => null);
     if (!response || (response.issues.length === 0 && response.pulls.length === 0)) {
       const message = await noMatchingFoundMessage(interaction);
-      await interaction.editReply(message);
+      await interaction.editReply({ content: message.content });
       throw new Error(message.content);
     }
 

src/commandHandler.ts

@@ -10,25 +10,16 @@ import z from "zod";
 const envSchema = z.object({
   NODE_ENV: z.enum(["development", "production", "test"]).default("development"),
   DISCORD_TOKEN: z
-    .string({
-      // eslint-disable-next-line camelcase -- Param isn't camelcase
-      required_error: "DISCORD_TOKEN was not found, either it or the .env file are missing",
-    })
+    .string("DISCORD_TOKEN was not found, either it or the .env file are missing")
     .trim()
     .min(1, "DISCORD_TOKEN was not of the minimum length it is either malformed or incomplete"),
   GITHUB_TOKEN: z
-    .string({
-      // eslint-disable-next-line camelcase -- Param isn't camelcase
-      required_error: "GITHUB_TOKEN was not found, either it or the .env file are missing",
-    })
+    .string("GITHUB_TOKEN was not found, either it or the .env file are missing")
     .trim()
     .min(1, "GITHUB_TOKEN was not of the minimum length it is either malformed or incomplete")
     .optional(),
   GUILD_ID: z
-    .string({
-      // eslint-disable-next-line camelcase -- Param isn't camelcase
-      required_error: "GUILD_ID was not found, either it or the .env file are missing",
-    })
+    .string("GUILD_ID was not found, either it or the .env file are missing")
     .trim()
     .min(1, "GUILD_ID was not of the minimum length it is either malformed or incomplete"),
 });
@@ -52,7 +43,7 @@ type EnvSchemaType = z.infer<typeof envSchema>;
 declare global {
   // eslint-disable-next-line @typescript-eslint/no-namespace -- intentional global override of process.env.
   namespace NodeJS {
-    // eslint-disable-next-line @typescript-eslint/no-empty-interface -- Just an overwrite of types
+    // eslint-disable-next-line @typescript-eslint/no-empty-object-type -- Just an overwrite of types
     interface ProcessEnv extends EnvSchemaType {}
   }
 }