Fix Webpack warnings for optional imports (#186)

* Add pino to peer dependencies

* Add webpackIgnore comments

Author: itssimon

package.json

@@ -221,7 +221,8 @@
     "fastify-plugin": "^4 || ^5",
     "h3": "^2",
     "hono": "^4",
-    "koa": "^2 || ^3"
+    "koa": "^2 || ^3",
+    "pino": "^8 || ^9 || ^10"
   },
   "peerDependenciesMeta": {
     "@adonisjs/core": {
@@ -265,6 +266,9 @@
     },
     "koa": {
       "optional": true
+    },
+    "pino": {
+      "optional": true
     }
   },
   "devDependencies": {

src/common/sentry.ts

@@ -5,7 +5,7 @@ let sentry: typeof Sentry | undefined;
 // Initialize Sentry when the module is loaded
 (async () => {
   try {
-    sentry = await import("@sentry/node");
+    sentry = await import(/* webpackIgnore: true */ "@sentry/node");
   } catch (e) {
     // Sentry SDK is not installed, ignore
   }

src/loggers/nestjs.ts

@@ -20,7 +20,7 @@ export async function patchNestLogger(
   }
 
   try {
-    const { Logger } = await import("@nestjs/common");
+    const { Logger } = await import(/* webpackIgnore: true */ "@nestjs/common");
     const logMethods: LogLevel[] = [
       "log",
       "error",

src/loggers/pino.ts

@@ -42,7 +42,7 @@ export async function patchPinoLogger(
 
     const originalStream = logger[originalStreamSym];
     if (originalStream) {
-      const pino = await import("pino");
+      const pino = await import(/* webpackIgnore: true */ "pino");
       const captureStream = new ApitallyLogCaptureStream(
         logsContext,
         logger[messageKeySym],

src/loggers/winston.ts

@@ -18,8 +18,10 @@ export async function patchWinston(
   }
 
   try {
-    // @ts-expect-error - file is not typed
-    const loggerModule = await import("winston/lib/winston/logger.js");
+    const loggerModule = await import(
+      // @ts-expect-error - file is not typed
+      /* webpackIgnore: true */ "winston/lib/winston/logger.js"
+    );
     if (loggerModule.default?.prototype?.write) {
       const originalWrite = loggerModule.default.prototype.write;
       loggerModule.default.prototype.write = function (info: any) {