fix: refactoring

Author: zemd

src/Logger.ts

@@ -2,13 +2,13 @@ import {Inject, Injectable, LoggerService} from "@nestjs/common";
 import * as util from "node:util";
 import type {Logger as PinoLogger} from "pino";
 import crypto from "node:crypto";
-import {PinoMessageSymbol} from "./logger.constants";
+import {PINO_LOGGER_INSTANCE, PinoMessageSymbol} from "./logger.constants";
 
 @Injectable()
 export class Logger implements LoggerService {
   private static cache = new Map<string, any>();
 
-  constructor(@Inject('PINO_LOGGER_INSTANCE') private readonly pinoInstance: PinoLogger) {
+  constructor(@Inject(PINO_LOGGER_INSTANCE) private readonly pinoInstance: PinoLogger) {
   }
 
   debug(message: any, ...optionalParams: any[]): any {

src/logger.module.ts

@@ -36,27 +36,22 @@ export class LoggerModule {
   }
 
   static forRootAsync(options: LoggerModuleAsyncOptions): DynamicModule {
-    const providers: Provider[] = [
-      Logger
-    ];
-
-    providers.push({
+    const optionsProvider = {
       provide: PINO_LOGGER_OPTIONS,
       useFactory: options.useFactory,
       inject: options.inject || [],
-    });
+    };
 
     const pinoProvider: Provider = {
       provide: PINO_LOGGER_INSTANCE,
-      useFactory: async (options: LoggerOptions) => pino(options),
+      useFactory: (options: LoggerOptions) => pino(options),
       inject: [PINO_LOGGER_OPTIONS],
-    }
-    providers.push(pinoProvider);
+    };
 
     return {
       module: LoggerModule,
       imports: options.imports ?? [],
-      providers,
+      providers: [optionsProvider, pinoProvider, Logger],
       exports: [pinoProvider, Logger]
     };
   }