Refactor: Decouple component logging from local storage flag

Author: leandrowilliam

src/cli/facade/dev.ts

@@ -204,7 +204,7 @@ const dev = ({ local, logger }: { logger: Logger; local: Local }) =>
           hotReloading,
           liveReloadPort: liveReload.port,
           local: true,
-          enableComponentConsoleOutput: true,
+          componentConsole: console,
           postRequestPayloadSize,
           components: opts.components,
           path: path.resolve(componentsDir),

src/registry/domain/options-sanitiser.ts

@@ -2,6 +2,7 @@ import zlib from 'node:zlib';
 import { compileSync } from 'oc-client-browser';
 import settings from '../../resources/settings';
 import type { Config } from '../../types';
+import createNoopConsole from '../../utils/noop-console';
 import * as auth from './authentication';
 
 const DEFAULT_NODE_KEEPALIVE_MS = 5000;
@@ -174,8 +175,8 @@ export default function optionsSanitiser(input: RegistryOptions): Config {
     options.tarExtractMode = 766;
   }
 
-  if (typeof options.enableComponentConsoleOutput === 'undefined') {
-    options.enableComponentConsoleOutput = false;
+  if (!options.componentConsole) {
+    options.componentConsole = createNoopConsole();
   }
 
   if (

src/registry/routes/helpers/get-component.ts

@@ -70,10 +70,6 @@ export interface GetComponentResult {
 }
 
 export const stream = Symbol('stream');
-const noop = () => {};
-const noopConsole = Object.fromEntries(
-  Object.keys(console).map((key) => [key, noop])
-);
 
 /**
  * Converts the plugins to a function that returns a record of plugins with the context applied
@@ -676,9 +672,7 @@ export default function getComponent(conf: Config, repository: Repository) {
                       exports: {} as Record<string, (...args: any[]) => any>
                     },
                     exports: {} as Record<string, (...args: any[]) => any>,
-                    console: conf.enableComponentConsoleOutput
-                      ? console
-                      : noopConsole,
+                    console: conf.componentConsole,
                     setTimeout,
                     Buffer,
                     AbortController: globalThis?.AbortController,

src/types.ts

@@ -290,13 +290,23 @@ export interface Config<T = any> {
    */
   local: boolean;
   /**
-   * Enables component console output (console.log, console.error, etc.)
-   * during component execution. Useful for debugging in development and
-   * lower environments.
+   * Console interface provided to components during execution.
+   * Allows for flexible logging strategies: pass the real console, a custom
+   * implementation that sends logs to a monitoring provider, or a no-op console.
    *
-   * @default false
+   * @example
+   * // Log to console
+   * componentConsole: console
+   * @example
+   * // Log to monitoring provider
+   * componentConsole: createCustomConsole(monitoringClient)
+   * @example
+   * // Disable component logs
+   * componentConsole: createNoopConsole()
+   *
+   * @default createNoopConsole() - a no-op console that discards all logs
    */
-  enableComponentConsoleOutput: boolean;
+  componentConsole: Partial<Console>;
   /**
    * File and directory mode (octal) applied when extracting tarballs during
    * publishing.

src/utils/noop-console.ts

@@ -0,0 +1,12 @@
+/**
+ * Creates a no-op console object that silently discards all logging calls.
+ * Useful as a default console implementation when component logging is disabled.
+ *
+ * @returns Console object with all methods mapped to no-op functions
+ */
+export function createNoopConsole(): Partial<Console> {
+  const noop = () => {};
+  return Object.fromEntries(Object.keys(console).map((key) => [key, noop]));
+}
+
+export default createNoopConsole;