add even more logging

Author: graphiteisaac

src/data/Thread.ts

@@ -1122,18 +1122,28 @@ export class Thread {
   }
 
   public async getDMChannel(): Promise<DMChannel> {
-    const user = await bot.users.fetch(this.user_id);
-    const dmChannel = await user.createDM();
+    try {
+      const user = await bot.users.fetch(this.user_id);
+      const dmChannel = await user.createDM();
 
-    return dmChannel;
+      return dmChannel;
+    } catch (err) {
+      logger.error({ thread_id: this.id, user_id: this.user_id, err });
+      throw err;
+    }
   }
 
   public async getThreadChannel(): Promise<SendableChannels> {
-    const channel = await bot.channels.fetch(this.channel_id);
+    try {
+      const channel = await bot.channels.fetch(this.channel_id);
 
-    if (channel?.isSendable()) return channel;
+      if (channel?.isSendable()) return channel;
 
-    throw "it was impossible to retrieve the thread channel";
+      throw "it was impossible to retrieve the thread channel";
+    } catch (err) {
+      logger.error({ thread_id: this.id, user_id: this.user_id, err });
+      throw err;
+    }
   }
 
   public async sendInfoHeader(

src/data/threads.ts

@@ -168,10 +168,14 @@ export async function createNewThreadForUser(
         // We can safely discard this error, because it just means we couldn't find the member in the guild
         // Which - for obvious reasons - is completely okay.
         if ((e as DiscordAPIError).code !== 10007) {
-          logger.error({
+          logger.debug({
             discord_api_code: (e as DiscordAPIError).code,
             err: e,
           });
+        } else {
+          logger.error({
+            err: e,
+          });
         }
       }
     }

src/index.ts

@@ -9,6 +9,7 @@ import { start } from "./main";
 import { migrateAllUp } from "./migrate";
 import { PluginInstallationError } from "./PluginInstallationError";
 import web from "./web";
+import logger from "./logger";
 
 const bunVersion = process.versions.bun.split(".").map(parseInt) as [
   number,
@@ -112,6 +113,8 @@ modules.forEach((mod) => {
 (async () => {
   await migrateAllUp();
 
+  logger.info(`Pino opened on level ${process.env.PINO_LOG_LEVEL}`);
+
   // Start the bot
   start(bot);