feat: enhance logging with environment-based configuration (#530)

# Enhance logging system with environment-based configuration

This PR improves the edge worker logging system by adding environment-based auto-configuration:

- Added environment detection to automatically set appropriate log formats:
  - Local development: Uses 'fancy' format with colors and icons
  - Production: Uses 'simple' format with key=value pairs

- Implemented smart defaults for log levels:
  - Local development: 'verbose' level for detailed logs
  - Production: 'info' level for standard operational logs

- Added support for the NO_COLOR standard to disable colored output

- Created explicit configuration overrides:
  - `EDGE_WORKER_LOG_FORMAT`: Override the auto-detected format ('fancy'|'simple')
  - `EDGE_WORKER_LOG_LEVEL`: Override the environment-based log level

- Modified the SupabasePlatformAdapter to initialize the logging factory with environment variables

- Added comprehensive tests for all new configuration options

Author: jumski

pkgs/edge-worker/src/platform/SupabasePlatformAdapter.ts

@@ -50,8 +50,8 @@ export class SupabasePlatformAdapter implements PlatformAdapter<SupabaseResource
   private queries: Queries;
   private deps: SupabasePlatformDeps;
 
-  // Logging factory with dynamic workerId support
-  private loggingFactory = createLoggingFactory();
+  // Logging factory with dynamic workerId support (initialized in constructor)
+  private loggingFactory!: ReturnType<typeof createLoggingFactory>;
 
   constructor(
     options?: { sql?: Sql; connectionString?: string },
@@ -73,9 +73,9 @@ export class SupabasePlatformAdapter implements PlatformAdapter<SupabaseResource
     // Create abort controller for shutdown signal
     this.abortController = new AbortController();
 
-    // Set initial log level
-    const logLevel = this.validatedEnv.EDGE_WORKER_LOG_LEVEL || 'info';
-    this.loggingFactory.setLogLevel(logLevel);
+    // Create logging factory with environment for auto-configuration
+    // (log level, format, and colors are determined from env)
+    this.loggingFactory = createLoggingFactory(env);
 
     // startWorker logger with a default module name
     this.logger = this.loggingFactory.createLogger('SupabasePlatformAdapter');

pkgs/edge-worker/src/platform/logging.ts

@@ -1,12 +1,38 @@
 import type { Logger } from './types.js';
+import { isLocalSupabaseEnv } from '../shared/localDetection.js';
 
 /**
- * Creates a logging factory with dynamic workerId support
+ * Log format type: 'fancy' for local dev with colors/icons, 'simple' for hosted with key=value
  */
-export function createLoggingFactory() {
+export type LogFormat = 'fancy' | 'simple';
+
+/**
+ * Environment record type for logging configuration
+ */
+export type LoggingEnv = Record<string, string | undefined>;
+
+/**
+ * Creates a logging factory with dynamic workerId support and environment-based configuration
+ * @param env - Optional environment variables for auto-configuration (NO_COLOR, EDGE_WORKER_LOG_FORMAT, etc.)
+ */
+export function createLoggingFactory(env?: LoggingEnv) {
+  // Determine if colors should be enabled (NO_COLOR standard: any value disables colors)
+  const colorsEnabled = env?.NO_COLOR === undefined;
+
+  // Determine if this is a local Supabase environment
+  const isLocal = env ? isLocalSupabaseEnv(env) : false;
+
+  // Determine log format: explicit override > auto-detect from environment
+  const explicitFormat = env?.EDGE_WORKER_LOG_FORMAT as LogFormat | undefined;
+  const format: LogFormat = explicitFormat ?? (isLocal ? 'fancy' : 'simple');
+
+  // Determine log level: explicit override > environment-based default (local=verbose, hosted=info)
+  const explicitLevel = env?.EDGE_WORKER_LOG_LEVEL;
+  const defaultLevel = isLocal ? 'verbose' : 'info';
+  let logLevel = explicitLevel ?? defaultLevel;
+
   // Shared state for all loggers
   let sharedWorkerId = 'unknown';
-  let logLevel = 'info';
 
   // All created logger instances - using Map for efficient lookup
   const loggers: Map<string, Logger> = new Map();
@@ -101,5 +127,15 @@ export function createLoggingFactory() {
     createLogger,
     setWorkerId,
     setLogLevel,
+    // Expose configuration for inspection/testing
+    get colorsEnabled() {
+      return colorsEnabled;
+    },
+    get format() {
+      return format;
+    },
+    get logLevel() {
+      return logLevel;
+    },
   };
 }

pkgs/edge-worker/tests/unit/platform/logging.test.ts

@@ -127,3 +127,102 @@ Deno.test('createLoggingFactory - level hierarchy: error < warn < info < verbose
     restore();
   }
 });
+
+// ============================================================
+// Environment Configuration Tests (Phase 2)
+// ============================================================
+
+Deno.test('createLoggingFactory - colorsEnabled defaults to true', () => {
+  const factory = createLoggingFactory();
+
+  assertEquals(factory.colorsEnabled, true);
+});
+
+Deno.test('createLoggingFactory - colorsEnabled is false when NO_COLOR env var is set', () => {
+  const factory = createLoggingFactory({ NO_COLOR: '1' });
+
+  assertEquals(factory.colorsEnabled, false);
+});
+
+Deno.test('createLoggingFactory - colorsEnabled is false when NO_COLOR is any truthy value', () => {
+  // NO_COLOR standard: presence of the variable (any value) disables colors
+  const factory1 = createLoggingFactory({ NO_COLOR: 'true' });
+  assertEquals(factory1.colorsEnabled, false);
+
+  const factory2 = createLoggingFactory({ NO_COLOR: '' });
+  // Empty string is still "set" per NO_COLOR standard
+  assertEquals(factory2.colorsEnabled, false);
+});
+
+Deno.test('createLoggingFactory - format defaults to simple when env not provided', () => {
+  const factory = createLoggingFactory();
+
+  assertEquals(factory.format, 'simple');
+});
+
+Deno.test('createLoggingFactory - format is fancy for local Supabase environment', () => {
+  const localEnv = {
+    SUPABASE_ANON_KEY: 'eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJzdXBhYmFzZS1kZW1vIiwicm9sZSI6ImFub24iLCJleHAiOjE5ODM4MTI5OTZ9.CRXP1A7WOeoJeXxjNni43kdQwgnWNReilDMblYTn_I0',
+  };
+  const factory = createLoggingFactory(localEnv);
+
+  assertEquals(factory.format, 'fancy');
+});
+
+Deno.test('createLoggingFactory - format is simple for hosted Supabase environment', () => {
+  const hostedEnv = {
+    SUPABASE_ANON_KEY: 'some-production-key',
+  };
+  const factory = createLoggingFactory(hostedEnv);
+
+  assertEquals(factory.format, 'simple');
+});
+
+Deno.test('createLoggingFactory - EDGE_WORKER_LOG_FORMAT overrides auto-detection', () => {
+  // Even in local env, explicit format override should win
+  const localEnv = {
+    SUPABASE_ANON_KEY: 'eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJzdXBhYmFzZS1kZW1vIiwicm9sZSI6ImFub24iLCJleHAiOjE5ODM4MTI5OTZ9.CRXP1A7WOeoJeXxjNni43kdQwgnWNReilDMblYTn_I0',
+    EDGE_WORKER_LOG_FORMAT: 'simple',
+  };
+  const factory = createLoggingFactory(localEnv);
+
+  assertEquals(factory.format, 'simple');
+});
+
+Deno.test('createLoggingFactory - EDGE_WORKER_LOG_FORMAT can set fancy in hosted env', () => {
+  const hostedEnv = {
+    SUPABASE_ANON_KEY: 'some-production-key',
+    EDGE_WORKER_LOG_FORMAT: 'fancy',
+  };
+  const factory = createLoggingFactory(hostedEnv);
+
+  assertEquals(factory.format, 'fancy');
+});
+
+Deno.test('createLoggingFactory - default log level is verbose for local env', () => {
+  const localEnv = {
+    SUPABASE_ANON_KEY: 'eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJzdXBhYmFzZS1kZW1vIiwicm9sZSI6ImFub24iLCJleHAiOjE5ODM4MTI5OTZ9.CRXP1A7WOeoJeXxjNni43kdQwgnWNReilDMblYTn_I0',
+  };
+  const factory = createLoggingFactory(localEnv);
+
+  assertEquals(factory.logLevel, 'verbose');
+});
+
+Deno.test('createLoggingFactory - default log level is info for hosted env', () => {
+  const hostedEnv = {
+    SUPABASE_ANON_KEY: 'some-production-key',
+  };
+  const factory = createLoggingFactory(hostedEnv);
+
+  assertEquals(factory.logLevel, 'info');
+});
+
+Deno.test('createLoggingFactory - EDGE_WORKER_LOG_LEVEL overrides default', () => {
+  const localEnv = {
+    SUPABASE_ANON_KEY: 'eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJzdXBhYmFzZS1kZW1vIiwicm9sZSI6ImFub24iLCJleHAiOjE5ODM4MTI5OTZ9.CRXP1A7WOeoJeXxjNni43kdQwgnWNReilDMblYTn_I0',
+    EDGE_WORKER_LOG_LEVEL: 'debug',
+  };
+  const factory = createLoggingFactory(localEnv);
+
+  assertEquals(factory.logLevel, 'debug');
+});