use port env variable instead

.env.example

@@ -7,4 +7,5 @@ IMGUR_CLIENT_ID="imgur client id" # get it from imgur, needed to parse imgur url
 VOTE_WEBHOOK_URL="your vote webhook url" # for sending that someone voted for the bot (optional)
 NETWORK_API_KEY="your network api key" # for posting to global chat (ask devoid)
 NODE_ENV=development # change to production when deploying
-DEBUG=false # set to true to enable debug logging
+DEBUG=false # set to true to enable debug logging
+PORT=3000 # or anything else for production

src/api/index.ts

@@ -3,7 +3,6 @@ import Logger from '../utils/Logger.js';
 import express from 'express';
 import dblRoute from './routes/dbl.js';
 import nsfwRouter from './routes/nsfw.js';
-import { API_PORT } from '../utils/Constants.js';
 
 // to start the server
 export const startApi = (data: { voteManager: VoteManager }) => {
@@ -14,5 +13,6 @@ export const startApi = (data: { voteManager: VoteManager }) => {
   app.use(nsfwRouter);
   if (data.voteManager) app.use(dblRoute(data.voteManager));
 
-  app.listen(API_PORT, () => Logger.info(`API listening on port http://localhost:${API_PORT}.`));
+  const port = process.env.PORT;
+  app.listen(port, () => Logger.info(`API listening on port http://localhost:${port}.`));
 };

src/core/Client.ts

@@ -96,27 +96,21 @@ export default class SuperClient extends Client {
     SuperClient.instance = this;
 
     // initialize i18n for localization
-    await loadLocales('locales/src/locales');
+    loadLocales('locales/src/locales');
 
     // load commands
     await loadCommandFiles();
 
     await syncConnectionCache();
     this._connectionCachePopulated = true;
 
-    this.scheduler.addRecurringTask(
-      'populateConnectionCache',
-      60_000 * 5,
-      syncConnectionCache,
-    );
-
-    // store network message timestamps to connectedList every minute
+    this.scheduler.addRecurringTask('populateConnectionCache', 60_000 * 5, syncConnectionCache);
     this.scheduler.addRecurringTask('storeMsgTimestamps', 60 * 1_000, () => {
+      // store network message timestamps to connectedList every minute
       storeMsgTimestamps(messageTimestamps);
       messageTimestamps.clear();
     });
 
-
     await this.login(process.env.TOKEN);
   }
 
@@ -135,7 +129,7 @@ export default class SuperClient extends Client {
   async fetchGuild(guildId: Snowflake): Promise<RemoveMethods<Guild> | undefined> {
     const fetch = (await this.cluster.broadcastEval(
       (client, guildID) => client.guilds.cache.get(guildID),
-      { context: guildId },
+      { guildId, context: guildId },
     )) as Guild[];
 
     return fetch ? SuperClient.resolveEval(fetch) : undefined;

src/scripts/network/storeMessageData.ts

@@ -44,19 +44,18 @@ export default async (
     if (!message.inGuild()) return;
 
     // store message data in db
-    await db.originalMessages
-      .create({
-        data: {
-          messageId: message.id,
-          authorId: message.author.id,
-          serverId: message.guildId,
-          messageReference: dbReference?.messageId,
-          createdAt: message.createdAt,
-          broadcastMsgs: { createMany: { data: messageDataObj } },
-          hub: { connect: { id: hubId } },
-          reactions: {},
-        },
-      });
+    await db.originalMessages.create({
+      data: {
+        messageId: message.id,
+        authorId: message.author.id,
+        serverId: message.guildId,
+        messageReference: dbReference?.messageId,
+        createdAt: message.createdAt,
+        broadcastMsgs: { createMany: { data: messageDataObj } },
+        hub: { connect: { id: hubId } },
+        reactions: {},
+      },
+    });
   }
 
   // store message timestamps to push to db later

src/utils/Constants.ts

@@ -17,7 +17,6 @@ export const isDevBuild = process.env.NODE_ENV === 'development';
 
 export const PROJECT_VERSION = require('../../package.json').version ?? 'Unknown';
 export const CLIENT_ID = isDevBuild ? '798748015435055134' : '769921109209907241';
-export const API_PORT = isDevBuild ? 3000 : 443;
 export const SUPPORT_SERVER_ID = '770256165300338709';
 export const VOTER_ROLE_ID = '985153241727770655';
 

src/utils/HubLogger/Report.ts

@@ -50,17 +50,16 @@ const genJumpLink = async (
   });
   if (!messageInDb) return null;
 
-
   // fetch the reports server ID from the log channel's ID
   const reportsServerId = SuperClient.resolveEval(
     await client.cluster.broadcastEval(
-      async (cl, ctx) => {
+      async (cl, channelId) => {
         const channel = (await cl.channels
-          .fetch(ctx.reportsChannelId)
+          .fetch(channelId)
           .catch(() => null)) as GuildTextBasedChannel | null;
         return channel?.guild.id;
       },
-      { context: { reportsChannelId } },
+      { context: reportsChannelId },
     ),
   );
 

src/utils/NSFWDetection.ts

@@ -1,5 +1,3 @@
-import { API_PORT } from './Constants.js';
-
 export declare type predictionType = {
   className: 'Drawing' | 'Hentai' | 'Neutral' | 'Porn' | 'Sexy';
   probability: number;
@@ -10,7 +8,7 @@ export declare type predictionType = {
  * @returns The predictions object
  */
 export const analyzeImageForNSFW = async (imageUrl: string): Promise<predictionType[] | null> => {
-  const res = await fetch(`http://localhost:${API_PORT}/nsfw`, {
+  const res = await fetch(`http://localhost:${process.env.PORT}/nsfw`, {
     method: 'POST',
     headers: { 'Content-Type': 'application/json' },
     body: JSON.stringify({ imageUrl }),

src/utils/Utils.ts

@@ -416,18 +416,17 @@ export const modifyUserRole = async (
   await cluster.broadcastEval(
     async (client, ctx) => {
       const guild = client.guilds.cache.get(ctx.guildId);
-      const role = guild?.roles.cache.find(({ id }) => id === ctx.roleId);
+      if (!guild) return;
 
-      if (!guild || !role) return null;
+      const role = await guild.roles.fetch(ctx.roleId);
+      if (!role) return;
 
       // add or remove role
-      const member = await guild.members.fetch(ctx.userId).catch(() => null);
-      return await member?.roles[ctx.action](role).catch(() => null);
+      const member = await guild.members.fetch(ctx.userId);
+      await member?.roles[ctx.action](role);
     },
-    { guildId, context: { userId, roleId, guildId, action } },
+    { context: { userId, roleId, guildId, action } },
   );
-
-  return;
 };
 
 /**