feat: adding VerboseLogger to avoid restrictions on DefaultLogger

Signed-off-by: Kevin Long <[email protected]>

Author: kevinmlong

packages/shared/src/hooks/logging-hook.ts

@@ -3,7 +3,7 @@ import type { BaseHook } from './hook';
 import type { BeforeHookContext, HookContext, HookHints } from './hooks';
 import type { FlagValue, EvaluationDetails } from '../evaluation';
 
-import { DefaultLogger, SafeLogger } from '../logger';
+import { VerboseLogger, SafeLogger } from '../logger';
 
 type LoggerPayload = Record<string, unknown>;
 
@@ -20,7 +20,7 @@ const VALUE_KEY = 'value';
 
 export class LoggingHook implements BaseHook {
   readonly includeEvaluationContext: boolean = false;
-  readonly logger = new SafeLogger(new DefaultLogger());
+  readonly logger = new SafeLogger(new VerboseLogger());
 
   constructor(includeEvaluationContext: boolean = false) {
     this.includeEvaluationContext = !!includeEvaluationContext;

packages/shared/src/logger/default-logger.ts

@@ -11,11 +11,7 @@ export class DefaultLogger implements Logger {
     console.warn(...args);
   }
 
-  info(...args: unknown[]): void {
-    console.info(...args);
-  }
+  info(): void {}
 
-  debug(...args: unknown[]): void {
-    console.debug(...args);
-  }
+  debug(): void {}
 }

packages/shared/src/logger/index.ts

@@ -1,3 +1,4 @@
 export * from './logger';
 export * from './default-logger';
 export * from './safe-logger';
+export * from './verbose-logger';

packages/shared/src/logger/verbose-logger.ts

@@ -0,0 +1,19 @@
+import type { Logger } from './logger';
+
+export class VerboseLogger implements Logger {
+  error(...args: unknown[]): void {
+    console.error(...args);
+  }
+
+  warn(...args: unknown[]): void {
+    console.warn(...args);
+  }
+
+  info(...args: unknown[]): void {
+    console.info(...args);
+  }
+  
+  debug(...args: unknown[]): void {
+    console.debug(...args);
+  }
+}