mcp-wordpress 1.4.0 → 1.5.0

package/src/client/managers/AuthenticationManager.ts

@@ -20,42 +20,42 @@ export class AuthenticationManager extends BaseManager {
       (process.env.WORDPRESS_AUTH_METHOD as AuthMethod) || "app-password";
 
     switch (method) {
-    case "app-password":
-      return {
-        method: "app-password",
-        username: process.env.WORDPRESS_USERNAME || "",
-        appPassword: process.env.WORDPRESS_APP_PASSWORD || "",
-      };
-
-    case "jwt":
-      return {
-        method: "jwt",
-        username: process.env.WORDPRESS_USERNAME || "",
-        password:
+      case "app-password":
+        return {
+          method: "app-password",
+          username: process.env.WORDPRESS_USERNAME || "",
+          appPassword: process.env.WORDPRESS_APP_PASSWORD || "",
+        };
+
+      case "jwt":
+        return {
+          method: "jwt",
+          username: process.env.WORDPRESS_USERNAME || "",
+          password:
             process.env.WORDPRESS_JWT_PASSWORD ||
             process.env.WORDPRESS_PASSWORD ||
             "",
-        secret: process.env.WORDPRESS_JWT_SECRET || "",
-      };
-
-    case "basic":
-      return {
-        method: "basic",
-        username: process.env.WORDPRESS_USERNAME || "",
-        password: process.env.WORDPRESS_PASSWORD || "",
-      };
-
-    case "api-key":
-      return {
-        method: "api-key",
-        apiKey: process.env.WORDPRESS_API_KEY || "",
-      };
-
-    default:
-      throw new AuthenticationError(
-        `Unsupported authentication method: ${method}`,
-        method,
-      );
+          secret: process.env.WORDPRESS_JWT_SECRET || "",
+        };
+
+      case "basic":
+        return {
+          method: "basic",
+          username: process.env.WORDPRESS_USERNAME || "",
+          password: process.env.WORDPRESS_PASSWORD || "",
+        };
+
+      case "api-key":
+        return {
+          method: "api-key",
+          apiKey: process.env.WORDPRESS_API_KEY || "",
+        };
+
+      default:
+        throw new AuthenticationError(
+          `Unsupported authentication method: ${method}`,
+          method,
+        );
     }
   }
 
@@ -66,49 +66,49 @@ export class AuthenticationManager extends BaseManager {
     const auth = this.config.auth;
 
     switch (auth.method) {
-    case "app-password":
-      if (!auth.username || !auth.appPassword) {
-        throw new AuthenticationError(
-          "Username and app password are required",
-          auth.method,
-        );
-      }
-
-      const credentials = Buffer.from(
-        `${auth.username}:${auth.appPassword}`,
-      ).toString("base64");
-      return { Authorization: `Basic ${credentials}` };
-
-    case "jwt":
-      if (!this.jwtToken) {
-        await this.authenticateJWT();
-      }
-      return { Authorization: `Bearer ${this.jwtToken}` };
-
-    case "basic":
-      if (!auth.username || !auth.password) {
+      case "app-password":
+        if (!auth.username || !auth.appPassword) {
+          throw new AuthenticationError(
+            "Username and app password are required",
+            auth.method,
+          );
+        }
+
+        const credentials = Buffer.from(
+          `${auth.username}:${auth.appPassword}`,
+        ).toString("base64");
+        return { Authorization: `Basic ${credentials}` };
+
+      case "jwt":
+        if (!this.jwtToken) {
+          await this.authenticateJWT();
+        }
+        return { Authorization: `Bearer ${this.jwtToken}` };
+
+      case "basic":
+        if (!auth.username || !auth.password) {
+          throw new AuthenticationError(
+            "Username and password are required",
+            auth.method,
+          );
+        }
+
+        const basicCredentials = Buffer.from(
+          `${auth.username}:${auth.password}`,
+        ).toString("base64");
+        return { Authorization: `Basic ${basicCredentials}` };
+
+      case "api-key":
+        if (!auth.apiKey) {
+          throw new AuthenticationError("API key is required", auth.method);
+        }
+        return { "X-API-Key": auth.apiKey };
+
+      default:
         throw new AuthenticationError(
-          "Username and password are required",
+          `Unsupported authentication method: ${auth.method}`,
           auth.method,
         );
-      }
-
-      const basicCredentials = Buffer.from(
-        `${auth.username}:${auth.password}`,
-      ).toString("base64");
-      return { Authorization: `Basic ${basicCredentials}` };
-
-    case "api-key":
-      if (!auth.apiKey) {
-        throw new AuthenticationError("API key is required", auth.method);
-      }
-      return { "X-API-Key": auth.apiKey };
-
-    default:
-      throw new AuthenticationError(
-        `Unsupported authentication method: ${auth.method}`,
-        auth.method,
-      );
     }
   }
 
@@ -187,47 +187,47 @@ export class AuthenticationManager extends BaseManager {
     }
 
     switch (auth.method) {
-    case "app-password":
-      if (!auth.username || !auth.appPassword) {
-        throw new AuthenticationError(
-          "App password authentication requires username and appPassword",
-          "app-password",
-        );
-      }
-      break;
-
-    case "jwt":
-      if (!auth.username || !auth.password || !auth.secret) {
-        throw new AuthenticationError(
-          "JWT authentication requires username, password, and secret",
-          "jwt",
-        );
-      }
-      break;
-
-    case "basic":
-      if (!auth.username || !auth.password) {
+      case "app-password":
+        if (!auth.username || !auth.appPassword) {
+          throw new AuthenticationError(
+            "App password authentication requires username and appPassword",
+            "app-password",
+          );
+        }
+        break;
+
+      case "jwt":
+        if (!auth.username || !auth.password || !auth.secret) {
+          throw new AuthenticationError(
+            "JWT authentication requires username, password, and secret",
+            "jwt",
+          );
+        }
+        break;
+
+      case "basic":
+        if (!auth.username || !auth.password) {
+          throw new AuthenticationError(
+            "Basic authentication requires username and password",
+            "basic",
+          );
+        }
+        break;
+
+      case "api-key":
+        if (!auth.apiKey) {
+          throw new AuthenticationError(
+            "API key authentication requires apiKey",
+            "api-key",
+          );
+        }
+        break;
+
+      default:
         throw new AuthenticationError(
-          "Basic authentication requires username and password",
-          "basic",
-        );
-      }
-      break;
-
-    case "api-key":
-      if (!auth.apiKey) {
-        throw new AuthenticationError(
-          "API key authentication requires apiKey",
-          "api-key",
+          `Unsupported authentication method: ${auth.method}`,
+          auth.method,
         );
-      }
-      break;
-
-    default:
-      throw new AuthenticationError(
-        `Unsupported authentication method: ${auth.method}`,
-        auth.method,
-      );
     }
   }
 }

package/src/client/managers/RequestManager.ts

@@ -3,7 +3,7 @@
  * Handles all HTTP operations, rate limiting, and retries
  */
 
-import fetch from "node-fetch";
+// Use native fetch in Node.js 18+
 import type {
   HTTPMethod,
   RequestOptions,

package/src/config/ServerConfiguration.ts

@@ -33,6 +33,14 @@ export class ServerConfiguration {
 
     // Load environment variables
     dotenv.config({ path: this.envPath });
+
+    // Debug output for DXT troubleshooting
+    console.error("DEBUG: ServerConfiguration initialized");
+    console.error(`DEBUG: Root directory: ${this.rootDir}`);
+    console.error(`DEBUG: Environment file path: ${this.envPath}`);
+    console.error(
+      `DEBUG: Environment file exists: ${fs.existsSync(this.envPath)}`,
+    );
   }
 
   /**
@@ -131,6 +139,23 @@ export class ServerConfiguration {
     configs: SiteConfig[];
   } {
     try {
+      // Debug output for DXT troubleshooting
+      console.error("DEBUG: loadSingleSiteFromEnv called");
+      console.error(`DEBUG: mcpConfig provided: ${mcpConfig ? "YES" : "NO"}`);
+      console.error("DEBUG: Current environment variables:");
+      console.error(
+        `  WORDPRESS_SITE_URL: ${process.env.WORDPRESS_SITE_URL || "NOT SET"}`,
+      );
+      console.error(
+        `  WORDPRESS_USERNAME: ${process.env.WORDPRESS_USERNAME || "NOT SET"}`,
+      );
+      console.error(
+        `  WORDPRESS_APP_PASSWORD: ${process.env.WORDPRESS_APP_PASSWORD ? "SET" : "NOT SET"}`,
+      );
+      console.error(
+        `  WORDPRESS_AUTH_METHOD: ${process.env.WORDPRESS_AUTH_METHOD || "NOT SET"}`,
+      );
+
       // Validate MCP config if provided
       const validatedMcpConfig = mcpConfig
         ? ConfigurationValidator.validateMcpConfig(mcpConfig)
@@ -157,6 +182,20 @@ export class ServerConfiguration {
         LOG_LEVEL: process.env.LOG_LEVEL,
       };
 
+      console.error("DEBUG: Final envConfig for validation:");
+      console.error(
+        `  WORDPRESS_SITE_URL: ${envConfig.WORDPRESS_SITE_URL || "NOT SET"}`,
+      );
+      console.error(
+        `  WORDPRESS_USERNAME: ${envConfig.WORDPRESS_USERNAME || "NOT SET"}`,
+      );
+      console.error(
+        `  WORDPRESS_APP_PASSWORD: ${envConfig.WORDPRESS_APP_PASSWORD ? "SET" : "NOT SET"}`,
+      );
+      console.error(
+        `  WORDPRESS_AUTH_METHOD: ${envConfig.WORDPRESS_AUTH_METHOD || "NOT SET"}`,
+      );
+
       // Validate environment configuration using Zod schema
       const validatedConfig =
         ConfigurationValidator.validateEnvironmentConfig(envConfig);

package/src/docs/MarkdownFormatter.ts

@@ -507,13 +507,13 @@ ${relatedTools.map((tool) => `- [\`${tool}\`](./${tool}.md)`).join("\n")}
     return `## Additional Examples
 
 ${examples
-    .map(
-      (example, index) => `### Example ${index + 2}
+  .map(
+    (example, index) => `### Example ${index + 2}
 \`\`\`json
 ${JSON.stringify(example, null, 2)}
 \`\`\`
 `,
-    )
-    .join("\n")}`;
+  )
+  .join("\n")}`;
   }
 }

package/src/dxt-entry.cjs

@@ -0,0 +1,55 @@
+#!/usr/bin/env node
+/* eslint-disable */
+/**
+ * CommonJS entry point for DXT package - ensures compatibility with Claude Desktop
+ */
+
+// Load AJV patch before any other modules
+try {
+    require('./ajv-patch.js');
+} catch (error) {
+    console.error("DEBUG: AJV patch failed to load:", error.message);
+}
+
+console.error("DEBUG: DXT CommonJS entry point starting...");
+console.error(`DEBUG: Current working directory: ${process.cwd()}`);
+console.error(`DEBUG: __dirname: ${__dirname}`);
+console.error(`DEBUG: Node version: ${process.version}`);
+
+console.error("DEBUG: Environment variables passed from DXT:");
+console.error(`  WORDPRESS_SITE_URL: ${process.env.WORDPRESS_SITE_URL ? 'SET' : 'NOT SET'}`);
+console.error(`  WORDPRESS_USERNAME: ${process.env.WORDPRESS_USERNAME ? 'SET' : 'NOT SET'}`);
+console.error(`  WORDPRESS_APP_PASSWORD: ${process.env.WORDPRESS_APP_PASSWORD ? 'SET' : 'NOT SET'}`);
+
+// Import and run the main server using dynamic import
+async function startDXTServer() {
+    try {
+        console.error("DEBUG: Attempting to import ES module...");
+        const { MCPWordPressServer } = await import("./index.js");
+        
+        console.error("DEBUG: Creating MCPWordPressServer instance from DXT entry point...");
+        const server = new MCPWordPressServer();
+        
+        console.error("DEBUG: Starting server...");
+        await server.run();
+        
+        // Handle graceful shutdown
+        const shutdown = async () => {
+            console.error("DEBUG: Received shutdown signal in DXT entry point");
+            await server.shutdown();
+            process.exit(0);
+        };
+        
+        process.on("SIGINT", shutdown);
+        process.on("SIGTERM", shutdown);
+        
+    } catch (error) {
+        console.error(`FATAL: DXT server failed to start: ${error instanceof Error ? error.message : String(error)}`);
+        console.error(`FATAL: Stack trace: ${error instanceof Error ? error.stack : 'No stack trace available'}`);
+        process.exit(1);
+    }
+}
+
+// Always run when loaded as DXT entry point
+console.error("DEBUG: Calling startDXTServer...");
+startDXTServer();

package/src/dxt-entry.ts

@@ -0,0 +1,55 @@
+#!/usr/bin/env node
+
+/**
+ * Entry point for DXT package - ensures proper initialization when run through Claude Desktop
+ */
+
+console.error("DEBUG: DXT entry point starting...");
+console.error(`DEBUG: Current working directory: ${process.cwd()}`);
+console.error(`DEBUG: __dirname equivalent: ${import.meta.url}`);
+console.error("DEBUG: Environment variables passed from DXT:");
+console.error(
+  `  WORDPRESS_SITE_URL: ${process.env.WORDPRESS_SITE_URL ? "SET" : "NOT SET"}`,
+);
+console.error(
+  `  WORDPRESS_USERNAME: ${process.env.WORDPRESS_USERNAME ? "SET" : "NOT SET"}`,
+);
+console.error(
+  `  WORDPRESS_APP_PASSWORD: ${process.env.WORDPRESS_APP_PASSWORD ? "SET" : "NOT SET"}`,
+);
+
+// Import and run the main server
+import { MCPWordPressServer } from "./index.js";
+
+async function startDXTServer() {
+  try {
+    console.error(
+      "DEBUG: Creating MCPWordPressServer instance from DXT entry point...",
+    );
+    const server = new MCPWordPressServer();
+    console.error("DEBUG: Starting server...");
+    await server.run();
+
+    // Handle graceful shutdown
+    const shutdown = async () => {
+      console.error("DEBUG: Received shutdown signal in DXT entry point");
+      await server.shutdown();
+      process.exit(0);
+    };
+
+    process.on("SIGINT", shutdown);
+    process.on("SIGTERM", shutdown);
+  } catch (error) {
+    console.error(
+      `FATAL: DXT server failed to start: ${error instanceof Error ? error.message : String(error)}`,
+    );
+    console.error(
+      `FATAL: Stack trace: ${error instanceof Error ? error.stack : "No stack trace available"}`,
+    );
+    process.exit(1);
+  }
+}
+
+// Always run when loaded as DXT entry point
+console.error("DEBUG: Calling startDXTServer...");
+startDXTServer();

package/src/index.ts

@@ -1,3 +1,5 @@
+console.error("DEBUG: MCP WordPress Server module loading started...");
+
 import { McpServer } from "@modelcontextprotocol/sdk/server/mcp.js";
 import { StdioServerTransport } from "@modelcontextprotocol/sdk/server/stdio.js";
 import { fileURLToPath } from "url";
@@ -10,6 +12,8 @@ import { ToolRegistry } from "./server/ToolRegistry.js";
 import { ConnectionTester } from "./server/ConnectionTester.js";
 import { getErrorMessage } from "./utils/error.js";
 
+console.error("DEBUG: All imports completed successfully");
+
 // --- Constants ---
 const SERVER_VERSION = "1.1.8"; // Technical debt resolution and modular architecture
 
@@ -22,12 +26,30 @@ class MCPWordPressServer {
   private toolRegistry: ToolRegistry;
 
   constructor(mcpConfig?: any) {
+    // Add debugging output for DXT troubleshooting
+    console.error("DEBUG: Starting MCP WordPress Server initialization...");
+    console.error("DEBUG: Environment variables:");
+    console.error(
+      `  WORDPRESS_SITE_URL: ${process.env.WORDPRESS_SITE_URL ? "SET" : "NOT SET"}`,
+    );
+    console.error(
+      `  WORDPRESS_USERNAME: ${process.env.WORDPRESS_USERNAME ? "SET" : "NOT SET"}`,
+    );
+    console.error(
+      `  WORDPRESS_APP_PASSWORD: ${process.env.WORDPRESS_APP_PASSWORD ? "SET" : "NOT SET"}`,
+    );
+    console.error(`  Working directory: ${process.cwd()}`);
+
     this.loadConfiguration(mcpConfig);
 
     if (this.wordpressClients.size === 0) {
       console.error(
-        "No WordPress sites were configured. Please create mcp-wordpress.config.json or set environment variables.",
+        "ERROR: No WordPress sites were configured. Please check that environment variables are set correctly.",
       );
+      console.error("Expected environment variables:");
+      console.error("  - WORDPRESS_SITE_URL");
+      console.error("  - WORDPRESS_USERNAME");
+      console.error("  - WORDPRESS_APP_PASSWORD");
       process.exit(1);
     }
 
@@ -71,6 +93,9 @@ class MCPWordPressServer {
     console.error(
       `INFO: Server started and connected. Tools available for ${this.wordpressClients.size} site(s).`,
     );
+
+    // Keep the process alive
+    process.stdin.resume();
   }
 
   async shutdown() {
@@ -82,8 +107,16 @@ class MCPWordPressServer {
 
 // --- Main Execution ---
 async function main() {
+  console.error("DEBUG: main() function started");
+  console.error(`DEBUG: process.argv: ${JSON.stringify(process.argv)}`);
+  console.error(
+    `DEBUG: fileURLToPath(import.meta.url): ${fileURLToPath(import.meta.url)}`,
+  );
+
   try {
+    console.error("DEBUG: Creating MCPWordPressServer instance...");
     const mcpServer = new MCPWordPressServer();
+    console.error("DEBUG: MCPWordPressServer created, calling run()...");
     await mcpServer.run();
 
     const shutdown = async () => {
@@ -95,12 +128,32 @@ async function main() {
     process.on("SIGTERM", shutdown);
   } catch (error) {
     console.error(`FATAL: Failed to start server: ${getErrorMessage(error)}`);
+    console.error(`FATAL: Error stack: ${(error as Error).stack}`);
     process.exit(1);
   }
 }
 
-if (process.argv[1] === fileURLToPath(import.meta.url)) {
+console.error(
+  `DEBUG: Checking if should run main - process.argv[1]: ${process.argv[1]}`,
+);
+console.error(
+  `DEBUG: fileURLToPath(import.meta.url): ${fileURLToPath(import.meta.url)}`,
+);
+
+// Check if running as main module - handle both direct execution and DXT entry point
+const isMainModule =
+  process.argv[1] === fileURLToPath(import.meta.url) ||
+  process.argv[1]?.endsWith("/index.js") ||
+  process.argv[1]?.endsWith("\\index.js") ||
+  !process.argv[1]; // When run through DXT, process.argv[1] might be undefined
+
+if (isMainModule) {
+  console.error("DEBUG: Running main function...");
   main();
+} else {
+  console.error(
+    "DEBUG: Not running main function - module imported, not executed directly",
+  );
 }
 
 export default MCPWordPressServer;

package/src/performance/MetricsCollector.ts

@@ -256,7 +256,7 @@ export class MetricsCollector {
     >;
     bestPerforming: string;
     worstPerforming: string;
-    } {
+  } {
     const sites = Array.from(this.clientInstances.keys());
     const comparison: any = {};
     const rankings: Array<{ site: string; score: number }> = [];
@@ -313,7 +313,7 @@ export class MetricsCollector {
     critical: string[];
     recommended: string[];
     optional: string[];
-    } {
+  } {
     const metrics = this.collectCurrentMetrics();
     const critical: string[] = [];
     const recommended: string[] = [];
@@ -382,7 +382,7 @@ export class MetricsCollector {
     };
     optimizations: any;
     alerts: any[];
-    } {
+  } {
     return {
       timestamp: new Date().toISOString(),
       overview: this.collectCurrentMetrics(),

package/src/performance/PerformanceAnalytics.ts

@@ -439,7 +439,7 @@ export class PerformanceAnalytics {
       implementationCost: "low" | "medium" | "high";
       timeToValue: number; // days
     };
-    } {
+  } {
     const insights = this.generateInsights();
 
     const quickWins = insights.filter(
@@ -506,7 +506,7 @@ export class PerformanceAnalytics {
     anomalies: PerformanceAnomaly[];
     predictions: any;
     optimizationPlan: any;
-    } {
+  } {
     const currentMetrics = this.collector.collectCurrentMetrics();
     const performanceScore = this.calculatePerformanceScore(currentMetrics);
 
@@ -823,20 +823,20 @@ export class PerformanceAnalytics {
     const current = this.collector.collectCurrentMetrics();
 
     switch (metricName) {
-    case "responseTime":
-      return current.requests.averageResponseTime;
-    case "cacheHitRate":
-      return current.cache.hitRate;
-    case "errorRate":
-      return current.requests.total > 0
-        ? current.requests.failed / current.requests.total
-        : 0;
-    case "memoryUsage":
-      return current.system.memoryUsage;
-    case "requestVolume":
-      return current.requests.requestsPerSecond;
-    default:
-      return 0;
+      case "responseTime":
+        return current.requests.averageResponseTime;
+      case "cacheHitRate":
+        return current.cache.hitRate;
+      case "errorRate":
+        return current.requests.total > 0
+          ? current.requests.failed / current.requests.total
+          : 0;
+      case "memoryUsage":
+        return current.system.memoryUsage;
+      case "requestVolume":
+        return current.requests.requestsPerSecond;
+      default:
+        return 0;
     }
   }
 

package/src/performance/PerformanceMonitor.ts

@@ -292,7 +292,7 @@ export class PerformanceMonitor {
     recommendations: string[];
     trends: string[];
     health: "excellent" | "good" | "warning" | "critical";
-    } {
+  } {
     const current = this.getMetrics();
     const health = this.calculateOverallHealth(current);
 

package/src/security/InputValidator.ts

@@ -228,11 +228,11 @@ export function validateSecurity(schema: z.ZodSchema) {
           error instanceof z.ZodError
             ? error.errors
             : [
-              {
-                message:
+                {
+                  message:
                     error instanceof Error ? error.message : String(error),
-              },
-            ],
+                },
+              ],
         );
       }
     };

package/src/security/SecurityConfig.ts

@@ -290,7 +290,7 @@ export function getEnvironmentSecurity(): {
   strictMode: boolean;
   verboseErrors: boolean;
   enforceHttps: boolean;
-  } {
+} {
   const isProduction = process.env.NODE_ENV === "production";
 
   return {