feat: updates DefaultLogger per GitHub issue thread

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 { VerboseLogger, SafeLogger } from '../logger';
+import { DefaultLogger, 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 VerboseLogger());
+  readonly logger = new SafeLogger(new DefaultLogger(true, true));
 
   constructor(includeEvaluationContext: boolean = false) {
     this.includeEvaluationContext = !!includeEvaluationContext;

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

@@ -3,6 +3,15 @@
 import type { Logger } from './logger';
 
 export class DefaultLogger implements Logger {
+  
+  private readonly showInfo : boolean = false;
+  private readonly showDebug : boolean = false;
+
+  constructor(showInfo: boolean = false, showDebug: boolean = false){
+    this.showInfo = showInfo;
+    this.showDebug = showDebug;
+  }
+
   error(...args: unknown[]): void {
     console.error(...args);
   }
@@ -11,7 +20,15 @@ export class DefaultLogger implements Logger {
     console.warn(...args);
   }
 
-  info(): void {}
+  info(...args: unknown[]): void {
+    if(this.showInfo) {
+      console.info(...args);
+    }
+  }
 
-  debug(): void {}
+  debug(...args: unknown[]): void {
+    if(this.showDebug) {
+      console.debug(...args);
+    }
+  }
 }

packages/shared/src/logger/index.ts

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

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

@@ -1,8 +1,7 @@
 import { GeneralError } from '../src/errors';
 import type { HookContext } from '../src/hooks/hooks';
 import { LoggingHook } from '../src/hooks/logging-hook';
-import { SafeLogger } from '../src/logger';
-import { VerboseLogger } from '../src/logger';
+import { DefaultLogger, SafeLogger } from '../src/logger';
 
 describe('LoggingHook', () => {
   const FLAG_KEY = 'some-key';
@@ -22,7 +21,7 @@ describe('LoggingHook', () => {
   const ERROR_MESSAGE_KEY = 'error_message';
 
   let hookContext: HookContext;
-  const logger : SafeLogger = new SafeLogger(new VerboseLogger());
+  const logger : SafeLogger = new SafeLogger(new DefaultLogger(true, true));
 
   beforeEach(() => {
     const mockProviderMetaData = { name: PROVIDER_NAME };