[CI] Create lint-and-build action. (#14)

.github/workflows/lint-and-build.yml

@@ -0,0 +1,32 @@
+name: Lint and Build
+
+on:
+  push:
+    branches: [master]
+  pull_request:
+
+jobs:
+  lint-and-build:
+    runs-on: ubuntu-latest
+
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v4
+
+      - name: Setup Node.js
+        uses: actions/setup-node@v4
+        with:
+          node-version: "24"
+          cache: npm
+
+      - name: Install dependencies
+        run: npm ci
+
+      - name: Lint
+        run: npm run lint
+
+      - name: Type check
+        run: npm run typecheck
+
+      - name: Build
+        run: npm run build

package.json

@@ -29,6 +29,8 @@
   "scripts": {
     "dev": "tsx watch --enable-source-maps src/index.ts",
     "start": "node dist/index.js",
+    "lint": "npx eslint .",
+    "typecheck": "npx tsc --noEmit",
     "build": "npm run clean && node scripts/build.js && npm run copyfiles",
     "clean": "rimraf dist",
     "copyfiles": "copyfiles -u 1 \"./src/**/*.html\" ./dist",

src/index.ts

@@ -1,13 +1,13 @@
 import { Client as DiscordClient, GatewayIntentBits, MessageFlags, Partials } from 'discord.js';
 import { config } from './config';
 import events from './events';
+import { ScheduledTasks, Scheduler } from './scheduler';
 import { Database } from './shared/Database';
 import { openRedisClient } from './shared/RedisClient';
 import { Handler } from './types';
 import { initIfNecessary, loadHandlersFrom, refreshCommands } from './utils';
 import { initializeWebserver } from './webserver';
 import { Encrypter } from './shared/Encrypter';
-import tasks from './tasks';
 
 
 let ready = false;
@@ -31,6 +31,8 @@ const client = new DiscordClient({
   }
 });
 
+const scheduler: Scheduler = new Scheduler(client);
+
 const commands: Handler[] = [];
 const contextMenus: Handler[] = [];
 const buttons: Handler[] = [];
@@ -139,10 +141,7 @@ client.on('clientReady', async () => {
 
   await initializeWebserver(client);
 
-  tasks.forEach(async (task) => {
+  ScheduledTasks.forEach(task => scheduler.add(task));
-    if (task.firstRun) await task.handler(client);
-    setInterval(async () => await task.handler(client), task.interval);
-  });
 
   events.forEach(event =>
     client.on(event.event, (...args) =>

src/scheduler/index.ts

@@ -0,0 +1,23 @@
+import { Client } from 'discord.js';
+import { Task } from '../types';
+import ScheduledTasks from './tasks';
+
+class Scheduler {
+  private tasks = new Map<Task, NodeJS.Timeout>();
+  private context: Client;
+
+  constructor(context: Client) {
+    this.context = context;
+  }
+
+  add(task: Task) {
+    const interval = setInterval(() => task.handle(this.context), task.interval);
+    if (task.firstRun) interval._onTimeout();
+
+    this.tasks.set(task, interval);
+  }
+}
+
+export {
+  ScheduledTasks, Scheduler
+};

src/scheduler/tasks/check-expired-bans-task.ts

@@ -0,0 +1,14 @@
+import { Client } from 'discord.js';
+import { Task } from '../../types';
+import { checkExpiredBans } from '../../utils';
+
+class CheckExpiredBansTask implements Task {
+  interval: number = 300000;
+  firstRun: boolean = true;
+
+  async handle(context: Client): Promise<void> {
+    await checkExpiredBans(context);
+  }
+}
+
+export default new CheckExpiredBansTask();

src/scheduler/tasks/close-stale-tickets-task.ts

@@ -0,0 +1,14 @@
+import { Client } from 'discord.js';
+import { Task } from '../../types';
+import { closeOldTickets } from '../../utils';
+
+class CloseStaleTicketsTask implements Task {
+  interval: number = 3.6e6;
+  firstRun: boolean = true;
+
+  async handle(context: Client): Promise<void> {
+    await closeOldTickets(context);
+  }
+}
+
+export default new CloseStaleTicketsTask();

src/scheduler/tasks/index.ts

@@ -0,0 +1,9 @@
+import checkExpiredBansTask from './check-expired-bans-task';
+import closeStaleTicketsTask from './close-stale-tickets-task';
+import pruneOldMessagesTask from './prune-old-messages-task';
+
+export default [
+  closeStaleTicketsTask,
+  checkExpiredBansTask,
+  pruneOldMessagesTask
+];

src/scheduler/tasks/prune-old-messages-task.ts

@@ -0,0 +1,13 @@
+import { Task } from '../../types';
+import { Database } from '../../shared/Database';
+
+class PruneOldMessagesTask implements Task {
+  interval: number = 3.6e6;
+  firstRun: boolean = true;
+
+  async handle(): Promise<void> {
+    await Database.pruneOldMessages();
+  }
+}
+
+export default new PruneOldMessagesTask();

src/tasks/checkExpiredBansTask.ts

@@ -1,11 +0,0 @@
-import { Client } from 'discord.js';
-import { checkExpiredBans } from '../utils';
-
-export default {
-  interval: 300000,
-  firstRun: true,
-
-  handler: async (context: Client) => {
-    await checkExpiredBans(context);
-  },
-};

src/tasks/closeStaleTicketsTask.ts

@@ -1,11 +0,0 @@
-import { Client } from 'discord.js';
-import { closeOldTickets } from '../utils';
-
-export default {
-  interval: 3.6e6,
-  firstRun: true,
-
-  handler: async (context: Client) => {
-    await closeOldTickets(context);
-  },
-};

src/tasks/index.ts

@@ -1,9 +0,0 @@
-import checkExpiredBansTask from './checkExpiredBansTask';
-import closeStaleTicketsTask from './closeStaleTicketsTask';
-import pruneOldMessagesTask from './pruneOldMessagesTask';
-
-export default [
-  closeStaleTicketsTask,
-  checkExpiredBansTask,
-  pruneOldMessagesTask
-];

src/tasks/pruneOldMessagesTask.ts

@@ -1,11 +0,0 @@
-import { Client } from 'discord.js';
-import { Database } from '../shared/Database';
-
-export default {
-  interval: 3.6e6,
-  firstRun: true,
-
-  handler: async (context: Client) => {
-    await Database.pruneOldMessages();
-  },
-};

src/types/index.d.ts

@@ -3,3 +3,4 @@ export * from './database-types.d';
 export * from './e621-types.d';
 export * from './handler.d';
 export * from './helper-types.d';
+export * from './scheduler.d';

src/types/scheduler.d.ts

@@ -0,0 +1,8 @@
+import { Client } from 'discord.js';
+
+export interface Task {
+  interval: number;
+  firstRun: boolean;
+
+  handle(context: Client): Promise<void>;
+};

src/utils/embed-utils.ts

@@ -1,7 +1,7 @@
 import { APIEmbed, Client } from 'discord.js';
 
-type EmbedSeverity =
+type EmbedSeverity
-  | 'info'
+  = | 'info'
   | 'warning'
   | 'error'
   | 'success'