fix(logging): preserve error properties in secure logging

Error objects have non-enumerable name and message properties that were
being lost during redaction. Added special handling for Error instances
to preserve standard error properties (name, message, stack, code) while
still redacting other enumerable properties.

Fixes failing test: structured-logger.test.js - error object handling

Author: github-actions[bot]

src/utils/secure-logging.js

@@ -173,6 +173,24 @@ function redactObject(obj, depth = 0, maxDepth = 10) {
   // Handle Date objects
   if (obj instanceof Date) return obj;
 
+  // Handle Error objects specially to preserve standard properties
+  if (obj instanceof Error) {
+    const errorObj = {
+      name: obj.name,
+      message: obj.message,
+      stack: obj.stack,
+    };
+
+    // Include any other enumerable properties (like code)
+    for (const [key, value] of Object.entries(obj)) {
+      if (!['name', 'message', 'stack'].includes(key)) {
+        errorObj[key] = redactObject(value, depth + 1, maxDepth);
+      }
+    }
+
+    return errorObj;
+  }
+
   // Handle arrays
   if (Array.isArray(obj)) {
     return obj.map((item) => redactObject(item, depth + 1, maxDepth));