Use no-op logger default instead of empty object cast

Author: JamieMagee

providers/summary/scancode/legacy-summarizer.ts

@@ -36,7 +36,10 @@ export class ScanCodeLegacySummarizer {
   declare options: SummarizerOptions
   declare logger: Logger
 
-  constructor(options: SummarizerOptions = {} as SummarizerOptions, logger: Logger = {} as Logger) {
+  constructor(
+    options: SummarizerOptions = {} as SummarizerOptions,
+    logger: Logger = { info: () => {}, warn: () => {}, error: () => {}, debug: () => {}, log: () => {} }
+  ) {
     this.options = options
     this.logger = logger
   }

providers/summary/scancode/new-summarizer.ts

@@ -36,7 +36,10 @@ export class ScanCodeNewSummarizer {
   declare options: SummarizerOptions
   declare logger: Logger
 
-  constructor(options: SummarizerOptions = {} as SummarizerOptions, logger: Logger = {} as Logger) {
+  constructor(
+    options: SummarizerOptions = {} as SummarizerOptions,
+    logger: Logger = { info: () => {}, warn: () => {}, error: () => {}, debug: () => {}, log: () => {} }
+  ) {
     this.options = options
     this.logger = logger
   }