fix: internal logger usage

Author: notunderctrl

packages/commandkit/src/analytics/analytics-engine.ts

@@ -1,6 +1,5 @@
 import { CommandKit } from '../commandkit';
 import { Logger } from '../logger/Logger';
-import { warnUnstable } from '../utils/warn-unstable';
 import {
   AnalyticsEvent,
   AnalyticsProvider,
@@ -83,10 +82,7 @@ export class AnalyticsEngine {
     try {
       await this.#provider!.identify?.(this, event);
     } catch (error) {
-      Logger.error(
-        `Error identifying with provider ${this.#provider!.name}`,
-        error,
-      );
+      Logger.error`Error identifying with provider ${this.#provider!.name}: ${error}`;
     }
   }
 
@@ -102,10 +98,7 @@ export class AnalyticsEngine {
       if (await this.#doNotTrack(event)) return;
       await this.#provider!.track(this, event);
     } catch (error) {
-      Logger.error(
-        `Error tracking ${event.name} event with provider ${this.#provider!.name}`,
-        error,
-      );
+      Logger.error`Error tracking ${event.name} event with provider ${this.#provider!.name}: ${error}`;
     }
   }
 

packages/commandkit/src/app/handlers/AppCommandHandler.ts

@@ -729,7 +729,7 @@ export class AppCommandHandler {
 
       this.loadedMiddlewares.set(id, { middleware, data });
     } catch (error) {
-      Logger.error(`Failed to load middleware ${id}`, error);
+      Logger.error`Failed to load middleware ${id}: ${error}`;
     }
   }
 
@@ -865,7 +865,7 @@ export class AppCommandHandler {
         },
       });
     } catch (error) {
-      Logger.error(`Failed to load command ${command.name} (${id})`, error);
+      Logger.error`Failed to load command ${command.name} (${id}): ${error}`;
     }
   }
 

packages/commandkit/src/app/handlers/AppEventsHandler.ts

@@ -217,12 +217,9 @@ export class AppEventsHandler {
                 }
 
                 // Otherwise log the error as usual
-                Logger.error(
-                  `Error handling event ${name}${
-                    namespace ? ` of namespace ${namespace}` : ''
-                  }`,
-                  e,
-                );
+                Logger.error`Error handling event ${name}${
+                  namespace ? ` of namespace ${namespace}` : ''
+                }: ${e}`;
               }
             }
           },
@@ -284,12 +281,9 @@ export class AppEventsHandler {
                 }
 
                 // Otherwise log the error as usual
-                Logger.error(
-                  `Error handling event ${name}${
-                    namespace ? ` of namespace ${namespace}` : ''
-                  }`,
-                  e,
-                );
+                Logger.error`Error handling event ${name}${
+                  namespace ? ` of namespace ${namespace}` : ''
+                }: ${e}`;
               }
             },
           );

packages/commandkit/src/app/middlewares/permissions.ts

@@ -43,10 +43,7 @@ export async function beforeExecute(ctx: MiddlewareContext) {
         }
       }
     } catch (error) {
-      Logger.error(
-        `Could not send 'Server-only command' DM to user ${interaction?.user.id ?? message?.author.id} for command ${command.command.name}.`,
-        error,
-      );
+      Logger.error`Could not send 'Server-only command' DM to user ${interaction?.user.id ?? message?.author.id} for command ${command.command.name}: ${error}`;
     }
 
     return ctx.cancel(); // Stop the command from executing
@@ -153,10 +150,7 @@ export async function beforeExecute(ctx: MiddlewareContext) {
       });
     }
   } catch (error) {
-    Logger.error(
-      `Could not send 'Not enough permissions' reply to user ${interaction?.user.id ?? message?.author.id} for command ${command.command.name}.`,
-      error,
-    );
+    Logger.error`Could not send 'Not enough permissions' reply to user ${interaction?.user.id ?? message?.author.id} for command ${command.command.name}: ${error}`;
   }
 
   return ctx.cancel(); // Stop the command from executing

packages/commandkit/src/app/register/CommandRegistrar.ts

@@ -172,7 +172,7 @@ export class CommandRegistrar {
         `✨ Refreshed ${data.length} global application (/) commands`,
       );
     } catch (e) {
-      Logger.error('Failed to update global application (/) commands', e);
+      Logger.error`Failed to update global application (/) commands: ${e}`;
     }
   }
 
@@ -253,7 +253,7 @@ export class CommandRegistrar {
 
       Logger.info(`✨ Refreshed ${count} guild application (/) commands`);
     } catch (e) {
-      Logger.error('Failed to update guild application (/) commands', e);
+      Logger.error`Failed to update guild application (/) commands: ${e}`;
     }
   }
 }

packages/commandkit/src/commandkit.ts

@@ -207,7 +207,7 @@ export class CommandKit extends EventEmitter {
       try {
         await hook(this);
       } catch (e) {
-        Logger.error('Error while executing bootstrap hook: ', e);
+        Logger.error`Error while executing bootstrap hook: ${e}`;
       } finally {
         bootstrapHooks.delete(hook);
       }
@@ -222,7 +222,7 @@ export class CommandKit extends EventEmitter {
       try {
         await hook(this);
       } catch (e) {
-        Logger.error('Error while executing application bootstrap hook: ', e);
+        Logger.error`Error while executing application bootstrap hook: ${e}`;
       } finally {
         onApplicationBootstrapHooks.delete(hook);
       }

packages/commandkit/src/plugins/plugin-runtime/CommandKitPluginRuntime.ts

@@ -144,7 +144,7 @@ export class CommandKitPluginRuntime {
           return true;
         }
 
-        Logger.error(`Plugin "${plugin.name}" failed`, e?.stack || e);
+        Logger.error`Plugin "${plugin.name}" failed: ${e?.stack || e}`;
       }
     }