Potential fix for pull request finding 'Comparison between inconvertible types'

Co-authored-by: Copilot Autofix powered by AI <223894421+github-code-quality[bot]@users.noreply.github.com>
Signed-off-by: Alessandro Franceschi <[email protected]>

Author: alvagante

backend/src/integrations/ApiLogger.ts

@@ -334,43 +334,55 @@ export class ApiLogger {
    * @param body - Raw body
    * @returns Sanitized body
    */
-  private sanitizeBody(body: unknown): unknown {
-    if (!body) {
-      return body;
-    }
+    private sanitizeBody(body: unknown): unknown {
+      if (!body) {
+        return body;
+      }
 
-    // If body is a string, return as-is (already serialized)
-    if (typeof body === "string") {
-      return body;
-    }
+      // If body is a string, return as-is (already serialized)
+      if (typeof body === "string") {
+        return body;
+      }
 
-    // If body is an object, sanitize sensitive fields
-    if (typeof body === "object" && body !== null) {
-      const sanitized: Record<string, unknown> = {};
-      const sensitiveFields = [
-        "password",
-        "token",
-        "secret",
-        "api_key",
-        "apiKey",
-        "private_key",
-        "privateKey",
-      ];
-
-      for (const [key, value] of Object.entries(body)) {
-        const lowerKey = key.toLowerCase();
-        if (sensitiveFields.some((field) => lowerKey.includes(field))) {
-          sanitized[key] = "[REDACTED]";
-        } else {
-          sanitized[key] = value;
+      // If body is a plain object, sanitize sensitive fields
+      if (this.isPlainObject(body)) {
+        const sanitized: Record<string, unknown> = {};
+        const sensitiveFields = [
+          "password",
+          "token",
+          "secret",
+          "api_key",
+          "apiKey",
+          "private_key",
+          "privateKey",
+        ];
+
+        for (const [key, value] of Object.entries(body as Record<string, unknown>)) {
+          const lowerKey = key.toLowerCase();
+          if (sensitiveFields.some((field) => lowerKey.includes(field))) {
+            sanitized[key] = "[REDACTED]";
+          } else {
+            sanitized[key] = value;
+          }
         }
+
+        return sanitized;
       }
 
-      return sanitized;
+      return body;
     }
 
-    return body;
-  }
+
+    /**
+     * Check if value is a plain object
+     */
+    private isPlainObject(value: unknown): value is Record<string, unknown> {
+      if (Object.prototype.toString.call(value) !== "[object Object]") {
+        return false;
+      }
+      const prototype = Object.getPrototypeOf(value);
+      return prototype === null || prototype === Object.prototype;
+    }
 
   /**
    * Create a preview of the body for logging