feat: allow injecting user-facing logger

node/bundler.ts

@@ -34,6 +34,7 @@ export interface BundleOptions {
   onAfterDownload?: OnAfterDownloadHook
   onBeforeDownload?: OnBeforeDownloadHook
   systemLogger?: LogFunction
+  userLogger?: LogFunction
   vendorDirectory?: string
 }
 
@@ -52,11 +53,12 @@ export const bundle = async (
     internalSrcFolder,
     onAfterDownload,
     onBeforeDownload,
+    userLogger,
     systemLogger,
     vendorDirectory,
   }: BundleOptions = {},
 ) => {
-  const logger = getLogger(systemLogger, debug)
+  const logger = getLogger(systemLogger, userLogger, debug)
   const featureFlags = getFlags(inputFeatureFlags)
   const options: DenoOptions = {
     debug,

node/logger.test.ts

@@ -13,12 +13,12 @@ afterEach(() => {
   console.log = consoleLog
 })
 
-test('Prints user logs to stdout', () => {
+test('Prints user logs to stdout if no user logger is provided', () => {
   const mockConsoleLog = vi.fn()
   console.log = mockConsoleLog
 
-  const logger1 = getLogger(noopLogger, true)
-  const logger2 = getLogger(noopLogger, false)
+  const logger1 = getLogger(noopLogger, undefined, true)
+  const logger2 = getLogger(noopLogger, undefined, false)
 
   logger1.user('Hello with `debug: true`')
   logger2.user('Hello with `debug: false`')
@@ -28,13 +28,23 @@ test('Prints user logs to stdout', () => {
   expect(mockConsoleLog).toHaveBeenNthCalledWith(2, 'Hello with `debug: false`')
 })
 
+test('Prints user logs to user logger provided', () => {
+  const userLogger = vi.fn()
+  const logger = getLogger(noopLogger, userLogger, true)
+
+  logger.user('Hello!')
+
+  expect(userLogger).toHaveBeenCalledTimes(1)
+  expect(userLogger).toHaveBeenNthCalledWith(1, 'Hello!')
+})
+
 test('Prints system logs to the system logger provided', () => {
   const mockSystemLog = vi.fn()
   const mockConsoleLog = vi.fn()
   console.log = mockSystemLog
 
-  const logger1 = getLogger(mockSystemLog, true)
-  const logger2 = getLogger(mockSystemLog, false)
+  const logger1 = getLogger(mockSystemLog, undefined, true)
+  const logger2 = getLogger(mockSystemLog, undefined, false)
 
   logger1.system('Hello with `debug: true`')
   logger2.system('Hello with `debug: false`')
@@ -49,8 +59,8 @@ test('Prints system logs to stdout if there is no system logger provided and `de
   const mockConsoleLog = vi.fn()
   console.log = mockConsoleLog
 
-  const logger1 = getLogger(undefined, true)
-  const logger2 = getLogger(undefined, false)
+  const logger1 = getLogger(undefined, undefined, true)
+  const logger2 = getLogger(undefined, undefined, false)
 
   logger1.system('Hello with `debug: true`')
   logger2.system('Hello with `debug: false`')

node/logger.ts

@@ -9,15 +9,16 @@ interface Logger {
   user: LogFunction
 }
 
-const getLogger = (systemLogger?: LogFunction, debug = false): Logger => {
+const getLogger = (systemLogger?: LogFunction, userLogger?: LogFunction, debug = false): Logger => {
   // If there is a system logger configured, we'll use that. If there isn't,
   // we'll pipe system logs to stdout if `debug` is enabled and swallow them
   // otherwise.
   const system = systemLogger ?? (debug ? console.log : noopLogger)
+  const user = userLogger ?? console.log
 
   return {
     system,
-    user: console.log,
+    user,
   }
 }