Update logger

.eslintrc

@@ -21,6 +21,6 @@
     "prefer-const": ["warn", { "destructuring": "all" }],
     "no-constant-condition": ["error", { "checkLoops": false }],
     "import/extensions": ["warn", "always", { "ts": "never" }],
-    "no-throw-literal": "error"
+    "no-throw-literal": "error",
   }
 }

API/stats/bestiary.js

@@ -72,7 +72,7 @@ function getBestiary(userProfile) {
       maxMilestone: totalTiers / 10,
     };
   } catch (error) {
-    console.log(error);
+    console.error(error);
     return null;
   }
 }

API/stats/chocolateFactory.js

@@ -18,7 +18,7 @@ module.exports = (profile) => {
       level: profile.events?.easter?.chocolate_level || 0,
     };
   } catch (error) {
-    console.log(error);
+    console.error(error);
     return null;
   }
 };

API/stats/crimson.js

@@ -1,5 +1,4 @@
 // CREDITS: by @Kathund (https://github.com/Kathund)
-
 const { titleCase } = require("../constants/functions.js");
 
 module.exports = (profile) => {
@@ -62,7 +61,7 @@ module.exports = (profile) => {
       trophyFishing: getTrophyFish(profile),
     };
   } catch (error) {
-    console.log(error);
+    console.error(error);
     return null;
   }
 };

API/stats/dungeons.js

@@ -95,7 +95,7 @@ module.exports = (profile) => {
       },
     };
   } catch (error) {
-    console.log(error);
+    console.error(error);
     return null;
   }
 };

API/stats/hotm.js

@@ -34,7 +34,7 @@ module.exports = (profile) => {
           forgeItem.timeFinishedText =
             timeFinished < Date.now() ? "Finished" : `ending ${moment(timeFinished).fromNow()}`;
         } else {
-          console.log(item);
+          console.error(item);
           forgeItem.name = "Unknown Item";
           forgeItem.id = `UNKNOWN-${item.id}`;
         }
@@ -66,7 +66,7 @@ module.exports = (profile) => {
       forge: forgeItems,
     };
   } catch (error) {
-    console.log(error);
+    console.error(error);
     return null;
   }
 };

API/stats/talismans.js

@@ -75,7 +75,7 @@ module.exports = async (profile) => {
       return null;
     }
   } catch (error) {
-    console.log(error);
+    console.error(error);
     return null;
   }
 };

index.js

@@ -1,3 +1,4 @@
+/* eslint-disable no-console */
 process.on("uncaughtException", (error) => console.log(error));
 const app = require("./src/Application.js");
 

package.json

@@ -6,10 +6,10 @@
   "scripts": {
     "start": "node index.js",
     "test": "jest --detectOpenHandles --forceExit",
-    "lint": "npx eslint .",
-    "lint:fix": "npx eslint . --fix",
-    "prettier": "npx prettier --check .",
-    "prettier:fix": "npx prettier --write .",
+    "lint": "npx eslint src/",
+    "lint:fix": "npx eslint src/ --fix",
+    "prettier": "npx prettier src/ --check",
+    "prettier:fix": "npx prettier src/ --write",
     "nodemon": "nodemon index"
   },
   "repository": {

src/Application.js

@@ -7,6 +7,7 @@ class Application {
   constructor() {
     require("./Configuration.js");
     require("./Updater.js");
+    require("./Logger.js");
     if (!existsSync("./data/")) mkdirSync("./data/", { recursive: true });
     if (!existsSync("./data/linked.json")) writeFileSync("./data/linked.json", JSON.stringify({}));
   }

src/Logger.js

@@ -1,8 +1,8 @@
+/* eslint-disable no-console */
 const customLevels = { discord: 0, minecraft: 1, web: 2, warn: 3, error: 4, broadcast: 5, max: 6 };
 const { createLogger, format, transports } = require("winston");
 const config = require("../config.json");
 const chalk = require("chalk");
-
 const discordTransport = new transports.File({ level: "discord", filename: "./logs/discord.log" });
 const minecraftTransport = new transports.File({ level: "minecraft", filename: "./logs/minecraft.log" });
 const webTransport = new transports.File({ level: "web", filename: "./logs/web.log" });
@@ -18,7 +18,7 @@ const discordLogger = createLogger({
     format.timestamp({ format: getCurrentTime }),
     format.printf(({ timestamp, level, message }) => {
       return `[${timestamp}] ${level.toUpperCase()} > ${message}`;
-    })
+    }),
   ),
   transports: [discordTransport, combinedTransport],
 });
@@ -30,7 +30,7 @@ const minecraftLogger = createLogger({
     format.timestamp({ format: getCurrentTime }),
     format.printf(({ timestamp, level, message }) => {
       return `[${timestamp}] ${level.toUpperCase()} > ${message}`;
-    })
+    }),
   ),
   transports: [minecraftTransport, combinedTransport],
 });
@@ -42,7 +42,7 @@ const webLogger = createLogger({
     format.timestamp({ format: getCurrentTime }),
     format.printf(({ timestamp, level, message }) => {
       return `[${timestamp}] ${level.toUpperCase()} > ${message}`;
-    })
+    }),
   ),
   transports: [webTransport, combinedTransport],
 });
@@ -54,7 +54,7 @@ const warnLogger = createLogger({
     format.timestamp({ format: getCurrentTime }),
     format.printf(({ timestamp, level, message }) => {
       return `[${timestamp}] ${level.toUpperCase()} > ${message}`;
-    })
+    }),
   ),
   transports: [warnTransport, combinedTransport],
 });
@@ -66,7 +66,7 @@ const errorLogger = createLogger({
     format.timestamp({ format: getCurrentTime }),
     format.printf(({ timestamp, level, message }) => {
       return `[${timestamp}] ${level.toUpperCase()} > ${message}`;
-    })
+    }),
   ),
   transports: [errorTransport, combinedTransport],
 });
@@ -78,7 +78,7 @@ const broadcastLogger = createLogger({
     format.timestamp({ format: getCurrentTime }),
     format.printf(({ timestamp, level, message }) => {
       return `[${timestamp}] ${level.toUpperCase()} > ${message}`;
-    })
+    }),
   ),
   transports: [broadcastTransport, combinedTransport],
 });
@@ -115,12 +115,13 @@ function warnMessage(message) {
   return console.log(chalk.bgYellow.black(`[${getCurrentTime()}] Warning >`) + " " + chalk.yellow(message));
 }
 
-function errorMessage(message) {
+function errorMessage(error) {
+  const errorString = `${error.toString()}${error.stack?.replace(error.toString(), "")}`;
   if (config.other.logToFiles) {
-    errorLogger.log("error", message);
+    errorLogger.log("error", errorString);
   }
-  
-  return console.log(chalk.bgRedBright.black(`[${getCurrentTime()}] Error >`) + " " + chalk.redBright(message));
+
+  return console.log(chalk.bgRedBright.black(`[${getCurrentTime()}] Error >`) + " " + chalk.redBright(errorString));
 }
 
 function broadcastMessage(message, location) {
@@ -156,8 +157,8 @@ async function configUpdateMessage(message) {
   console.log();
   console.log(
     `${chalk.bgRedBright.black(`[${getCurrentTime()}] Config Update >`)} ${chalk.redBright("Added")} ${chalk.gray(
-      message
-    )} ${chalk.redBright("to config.json")}`
+      message,
+    )} ${chalk.redBright("to config.json")}`,
   );
 }
 
@@ -174,13 +175,14 @@ async function updateMessage() {
   console.log(chalk.bgRed.black(" ".repeat(columns).repeat(3)));
 }
 
+console.discord = discordMessage;
+console.minecraft = minecraftMessage;
+console.web = webMessage;
+console.warn = warnMessage;
+console.error = errorMessage;
+console.broadcast = broadcastMessage;
+
 module.exports = {
-  discordMessage,
-  minecraftMessage,
-  webMessage,
-  warnMessage,
-  errorMessage,
-  broadcastMessage,
   getCurrentTime,
   configUpdateMessage,
   updateMessage,

src/Updater.js

@@ -10,7 +10,7 @@ function updateCode() {
 
   exec("git pull", (error, stdout, stderr) => {
     if (error) {
-      console.error(`Git pull error: ${error}`);
+      console.error(error);
       return;
     }
 

src/contracts/API/mowojangAPI.js

@@ -28,7 +28,7 @@ async function getUUID(username) {
   } catch (error) {
     // eslint-disable-next-line no-throw-literal
     if (error.response.data === "Not found") throw "Invalid username.";
-    console.log(error);
+    console.error(error);
     throw error;
   }
 }
@@ -57,7 +57,7 @@ async function getUsername(uuid) {
 
     return data.name;
   } catch (error) {
-    console.log(error);
+    console.error(error);
     // eslint-disable-next-line no-throw-literal
     if (error.response?.data === "Not found") throw "Invalid UUID.";
     throw error;
@@ -75,7 +75,7 @@ async function resolveUsernameOrUUID(username) {
   } catch (error) {
     // eslint-disable-next-line no-throw-literal
     if (error.response.data === "Not found") throw "Invalid Username Or UUID.";
-    console.log(error);
+    console.error(error);
     throw error;
   }
 }

src/discord/CommandHandler.js

@@ -26,7 +26,7 @@ class CommandHandler {
 
     rest
       .put(Routes.applicationGuildCommands(clientID, config.discord.bot.serverID), { body: commands })
-      .catch(console.error);
+      .catch((e) => console.error(e));
   }
 }
 

src/discord/DiscordManager.js

@@ -6,7 +6,6 @@ const MessageHandler = require("./handlers/MessageHandler.js");
 const StateHandler = require("./handlers/StateHandler.js");
 const CommandHandler = require("./CommandHandler.js");
 const config = require("../../config.json");
-const Logger = require(".././Logger.js");
 const path = require("node:path");
 const fs = require("fs");
 
@@ -33,7 +32,7 @@ class DiscordManager extends CommunicationBridge {
     this.client.on("messageCreate", (message) => this.messageHandler.onMessage(message));
 
     this.client.login(config.discord.bot.token).catch((error) => {
-      Logger.errorMessage(error);
+      console.error(error);
     });
 
     client.commands = new Collection();
@@ -93,7 +92,7 @@ class DiscordManager extends CommunicationBridge {
     const mode = chat === "debugChannel" ? "minecraft" : config.discord.other.messageMode.toLowerCase();
     message = chat === "debugChannel" ? fullMessage : message;
     if (message !== undefined && chat !== "debugChannel") {
-      Logger.broadcastMessage(
+      console.broadcast(
         `${username} [${guildRank.replace(/§[0-9a-fk-or]/g, "").replace(/^\[|\]$/g, "")}]: ${message}`,
         `Discord`,
       );
@@ -106,7 +105,7 @@ class DiscordManager extends CommunicationBridge {
 
     const channel = await this.stateHandler.getChannel(chat || "Guild");
     if (channel === undefined) {
-      Logger.errorMessage(`Channel ${chat} not found!`);
+      console.error(`Channel ${chat} not found!`);
       return;
     }
     if (username === bot.username && message.endsWith("Check Discord Bridge for image.")) {
@@ -181,7 +180,7 @@ class DiscordManager extends CommunicationBridge {
   }
 
   async onBroadcastCleanEmbed({ message, color, channel }) {
-    Logger.broadcastMessage(message, "Event");
+    console.broadcast(message, "Event");
 
     channel = await this.stateHandler.getChannel(channel);
     channel.send({
@@ -195,7 +194,7 @@ class DiscordManager extends CommunicationBridge {
   }
 
   async onBroadcastHeadedEmbed({ message, title, icon, color, channel }) {
-    Logger.broadcastMessage(message, "Event");
+    console.broadcast(message, "Event");
 
     channel = await this.stateHandler.getChannel(channel);
     channel.send({
@@ -213,7 +212,7 @@ class DiscordManager extends CommunicationBridge {
   }
 
   async onPlayerToggle({ fullMessage, username, message, color, channel }) {
-    Logger.broadcastMessage(message, "Event");
+    console.broadcast(message, "Event");
     channel = await this.stateHandler.getChannel(channel);
     switch (config.discord.other.messageMode.toLowerCase()) {
       case "bot":

src/discord/commands/helpCommand.js

@@ -87,7 +87,7 @@ module.exports = {
         await interaction.followUp({ embeds: [embed] });
       }
     } catch (error) {
-      console.log(error);
+      console.error(error);
     }
   },
 };

src/discord/commands/infoCommand.js

@@ -95,7 +95,7 @@ module.exports = {
         });
       await interaction.followUp({ embeds: [infoEmbed] });
     } catch (e) {
-      console.log(e);
+      console.error(e);
     }
   },
 };

src/discord/commands/verifyCommand.js

@@ -94,7 +94,7 @@ module.exports = {
 
       await updateRolesCommand.execute(interaction, user);
     } catch (error) {
-      console.log(error);
+      console.error(error);
       // eslint-disable-next-line no-ex-assign
       error = error
         .toString()

src/discord/events/interactionCreate.js

@@ -3,7 +3,6 @@ const { ErrorEmbed } = require("../../contracts/embedHandler.js");
 // eslint-disable-next-line no-unused-vars
 const { CommandInteraction } = require("discord.js");
 const config = require("../../../config.json");
-const Logger = require("../.././Logger.js");
 
 module.exports = {
   name: "interactionCreate",
@@ -23,8 +22,8 @@ module.exports = {
         if (command === undefined) {
           return;
         }
-        
-        Logger.discordMessage(`${interaction.user.username} - [${interaction.commandName}]`);
+
+        console.discord(`${interaction.user.username} - [${interaction.commandName}]`);
 
         if (command.verificationCommand === true && config.verification.enabled === false) {
           throw new HypixelDiscordChatBridgeError("Verification is disabled.");
@@ -41,8 +40,7 @@ module.exports = {
         await command.execute(interaction);
       }
     } catch (error) {
-      console.log(error);
-
+      console.error(error);
       const errrorMessage =
         error instanceof HypixelDiscordChatBridgeError
           ? ""

src/discord/handlers/MessageHandler.js

@@ -68,7 +68,7 @@ class MessageHandler {
 
       this.discord.broadcastMessage(messageData);
     } catch (error) {
-      console.log(error);
+      console.error(error);
     }
   }
 
@@ -111,7 +111,7 @@ class MessageHandler {
 
       return mentionedUserName ?? null;
     } catch (error) {
-      console.log(error);
+      console.error(error);
       return null;
     }
   }