Handling unexpected error in publish pipeline.

Author: EMaher

src/lib/logger.ts

@@ -56,13 +56,22 @@ function isSensitiveKey(key: string): boolean {
 
 /**
  * Recursively sanitize a value, replacing sensitive fields with '***'.
- * Handles objects, arrays, and inline bearer tokens in strings.
+ * Handles objects, arrays, inline bearer tokens in strings, and Error objects.
  */
 function sanitize(value: unknown): unknown {
   if (typeof value === 'string') {
     // Redact inline bearer tokens (e.g. "Bearer eyJ...")
     return value.replace(/Bearer\s+[A-Za-z0-9\-._~+/]+=*/gi, 'Bearer ***');
   }
+  if (value instanceof Error) {
+    // Convert Error objects to a serializable form
+    return {
+      name: value.name,
+      message: value.message,
+      stack: value.stack,
+      ...(value.cause ? { cause: sanitize(value.cause) } : {}),
+    };
+  }
   if (Array.isArray(value)) {
     return value.map((item) => sanitize(item));
   }

tests/unit/lib/logger.test.ts

@@ -146,6 +146,27 @@ describe('Logger', () => {
       expect(output).toContain('test');
       expect(output).toContain('42');
     });
+
+    it('should serialize Error objects with message and stack', () => {
+      const error = new Error('Something went wrong');
+      loggerInstance.error('Error occurred:', error);
+      const output = stderrSpy.mock.calls[0]![0] as string;
+      expect(output).toContain('Something went wrong');
+      expect(output).toContain('"name":"Error"');
+      expect(output).toContain('"message":"Something went wrong"');
+      // Stack trace should be present (not testing exact format)
+      expect(output).toContain('"stack"');
+    });
+
+    it('should serialize Error objects with cause', () => {
+      const innerError = new Error('Inner error');
+      const outerError = new Error('Outer error', { cause: innerError });
+      loggerInstance.error('Nested error:', outerError);
+      const output = stderrSpy.mock.calls[0]![0] as string;
+      expect(output).toContain('Outer error');
+      expect(output).toContain('Inner error');
+      expect(output).toContain('"cause"');
+    });
   });
 
   describe('non-ASCII punctuation normalization', () => {