mcp-wordpress 2.6.3 → 2.7.0

package/src/security/AISecurityScanner.ts

@@ -170,9 +170,9 @@ export class AISecurityScanner {
       });
 
       return result;
-    } catch (error) {
-      this.logger.error("Security scan failed", { scanId, error: String(error) });
-      throw new SecurityValidationError("Security scan failed", [{ message: String(error) }]);
+    } catch (_error) {
+      this.logger.error("Security scan failed", { scanId, _error: String(_error) });
+      throw new SecurityValidationError("Security scan failed", [{ message: String(_error) }]);
     }
   }
 
@@ -204,9 +204,9 @@ export class AISecurityScanner {
           await this.scanFile(fullPath);
         }
       }
-    } catch (error) {
+    } catch (_error) {
       // Directory might not exist or be accessible
-      this.logger.warn("Cannot scan directory", { dirPath, error: String(error) });
+      this.logger.warn("Cannot scan directory", { dirPath, _error: String(_error) });
     }
   }
 
@@ -238,8 +238,8 @@ export class AISecurityScanner {
       this.scanForCSRF(filePath, content, lines);
       this.scanForInfoDisclosure(filePath, content, lines);
       this.scanForInsecureConfiguration(filePath, content, lines);
-    } catch (error) {
-      this.logger.warn("Cannot scan file", { filePath, error: String(error) });
+    } catch (_error) {
+      this.logger.warn("Cannot scan file", { filePath, _error: String(_error) });
     }
   }
 

package/src/security/AutomatedRemediation.ts

@@ -377,16 +377,16 @@ export class AutomatedRemediation {
             details: result.details,
           });
         }
-      } catch (error) {
+      } catch (_error) {
         this.logger.error("Remediation action threw error", {
           actionId: action.id,
-          error: String(error),
+          _error: String(_error),
         });
         results.push({
           vulnerabilityId: action.id,
           success: false,
           action: action.type,
-          details: `Error: ${error instanceof Error ? error.message : String(error)}`,
+          details: `Error: ${_error instanceof Error ? _error.message : String(_error)}`,
           timestamp: new Date(),
         });
       }
@@ -460,12 +460,12 @@ export class AutomatedRemediation {
         backupPath,
         validationResult,
       };
-    } catch (error) {
+    } catch (_error) {
       return {
         vulnerabilityId: action.id,
         success: false,
         action: action.type,
-        details: `Remediation failed: ${error instanceof Error ? error.message : String(error)}`,
+        details: `Remediation failed: ${_error instanceof Error ? _error.message : String(_error)}`,
         timestamp: new Date(),
       };
     }
@@ -621,10 +621,10 @@ export class AutomatedRemediation {
       }
 
       return true;
-    } catch (error) {
+    } catch (_error) {
       this.logger.warn("Validation failed for action", {
         actionId: action.id,
-        error: String(error),
+        _error: String(_error),
       });
       return false;
     }
@@ -667,8 +667,10 @@ export class AutomatedRemediation {
         targetFile,
         backupPath,
       });
-    } catch (error) {
-      throw new SecurityValidationError(`Rollback failed: ${error instanceof Error ? error.message : String(error)}`);
+    } catch (_error) {
+      throw new SecurityValidationError(
+        `Rollback failed: ${_error instanceof Error ? _error.message : String(_error)}`,
+      );
     }
   }
 
@@ -689,8 +691,8 @@ export class AutomatedRemediation {
           this.logger.debug("Cleaned up old backup", { file });
         }
       }
-    } catch (error) {
-      this.logger.warn("Backup cleanup failed", { error: String(error) });
+    } catch (_error) {
+      this.logger.warn("Backup cleanup failed", { _error: String(_error) });
     }
   }
 }

package/src/security/InputValidator.ts

@@ -175,26 +175,27 @@ export function validateSecurity(schema: z.ZodSchema) {
         logger.info(`Security validation passed for ${propertyName}`, {
           timestamp: new Date().toISOString(),
           method: propertyName,
-          paramCount: Object.keys(validatedParams).length,
+          paramCount: typeof validatedParams === 'object' && validatedParams ? Object.keys(validatedParams).length : 0,
         });
 
         // Call original method with validated params
         return await method.call(this, validatedParams, ...args.slice(1));
-      } catch (error) {
+      } catch (_error) {
         // Log security validation failure
         logger.error(`Security validation failed for ${propertyName}`, {
           timestamp: new Date().toISOString(),
           method: propertyName,
-          error: error instanceof z.ZodError ? error.errors : error instanceof Error ? error.message : String(error),
+          _error:
+            _error instanceof z.ZodError ? _error.issues : _error instanceof Error ? _error.message : String(_error),
         });
 
         throw new SecurityValidationError(
           `Security validation failed for ${propertyName}`,
-          error instanceof z.ZodError
-            ? error.errors
+          _error instanceof z.ZodError
+            ? _error.issues
             : [
                 {
-                  message: error instanceof Error ? error.message : String(error),
+                  message: _error instanceof Error ? _error.message : String(_error),
                 },
               ],
         );
@@ -209,12 +210,12 @@ export function validateSecurity(schema: z.ZodSchema) {
  * Custom security validation error
  */
 export class SecurityValidationError extends Error {
-  public readonly errors: Array<z.ZodIssue | { message: string }>;
+  public readonly issues: Array<z.ZodIssue | { message: string }>;
 
-  constructor(message: string, errors: Array<z.ZodIssue | { message: string }> = []) {
+  constructor(message: string, issues: Array<z.ZodIssue | { message: string }> = []) {
     super(message);
     this.name = "SecurityValidationError";
-    this.errors = errors;
+    this.issues = issues;
   }
 }