@optimizely-opal/opal-tool-ocp-sdk 1.0.0-OCP-1442.5 → 1.0.0-OCP-1449.1

package/src/logging/ToolLogger.ts

@@ -64,8 +64,8 @@ export class ToolLogger {
     }
 
     if (typeof data === 'string') {
-      return data.length > ToolLogger.MAX_PARAM_LENGTH
-        ? `${data.substring(0, ToolLogger.MAX_PARAM_LENGTH)}... (truncated, ${data.length} chars total)`
+      return data.length > this.MAX_PARAM_LENGTH
+        ? `${data.substring(0, this.MAX_PARAM_LENGTH)}... (truncated, ${data.length} chars total)`
         : data;
     }
 
@@ -74,10 +74,10 @@ export class ToolLogger {
     }
 
     if (Array.isArray(data)) {
-      const truncated = data.slice(0, ToolLogger.MAX_ARRAY_ITEMS);
-      const result = truncated.map((item) => ToolLogger.redactSensitiveData(item, maxDepth - 1));
-      if (data.length > ToolLogger.MAX_ARRAY_ITEMS) {
-        result.push(`... (${data.length - ToolLogger.MAX_ARRAY_ITEMS} more items truncated)`);
+      const truncated = data.slice(0, this.MAX_ARRAY_ITEMS);
+      const result = truncated.map((item) => this.redactSensitiveData(item, maxDepth));
+      if (data.length > this.MAX_ARRAY_ITEMS) {
+        result.push(`... (${data.length - this.MAX_ARRAY_ITEMS} more items truncated)`);
       }
       return result;
     }
@@ -86,12 +86,12 @@ export class ToolLogger {
       const result: any = {};
       for (const [key, value] of Object.entries(data)) {
         // Check if this field contains sensitive data
-        const isSensitive = ToolLogger.isSensitiveField(key);
+        const isSensitive = this.isSensitiveField(key);
 
         if (isSensitive) {
           result[key] = '[REDACTED]';
         } else {
-          result[key] = ToolLogger.redactSensitiveData(value, maxDepth - 1);
+          result[key] = this.redactSensitiveData(value, maxDepth - 1);
         }
       }
       return result;
@@ -105,7 +105,7 @@ export class ToolLogger {
    */
   private static isSensitiveField(fieldName: string): boolean {
     const lowerKey = fieldName.toLowerCase();
-    return ToolLogger.SENSITIVE_FIELDS.some((sensitiveField) =>
+    return this.SENSITIVE_FIELDS.some((sensitiveField) =>
       lowerKey.includes(sensitiveField)
     );
   }
@@ -118,7 +118,7 @@ export class ToolLogger {
       return null;
     }
 
-    return ToolLogger.redactSensitiveData(params);
+    return this.redactSensitiveData(params);
   }
 
   /**
@@ -146,7 +146,8 @@ export class ToolLogger {
     const requestLog = {
       event: 'opal_tool_request',
       path: req.path,
-      parameters: ToolLogger.createParameterSummary(params)
+      method: req.method,
+      parameters: this.createParameterSummary(params)
     };
 
     // Log with Zaius audience so developers only see requests for accounts they have access to
@@ -167,7 +168,7 @@ export class ToolLogger {
       duration: processingTimeMs ? `${processingTimeMs}ms` : undefined,
       status: response.status,
       contentType: response.headers?.get('content-type') || 'unknown',
-      contentLength: ToolLogger.calculateContentLength(response),
+      contentLength: this.calculateContentLength(response),
       success: response.status >= 200 && response.status < 300
     };