mcp-wordpress 2.4.2 → 2.5.1

package/src/dxt-entry.cjs

@@ -8,37 +8,45 @@
 try {
     require('./ajv-patch.js');
 } catch (error) {
-    console.error("DEBUG: AJV patch failed to load:", error.message);
+    // Use stderr to avoid interfering with STDIO
+    process.stderr.write(`[DXT] AJV patch failed to load: ${error.message}\n`);
 }
 
-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}`);
+// Use stderr for debug output to avoid interfering with STDIO
+const debug = (message) => {
+    if (process.env.DEBUG || process.env.NODE_ENV === 'development') {
+        process.stderr.write(`[DXT] ${message}\n`);
+    }
+};
+
+debug("CommonJS entry point starting...");
+debug(`Current working directory: ${process.cwd()}`);
+debug(`__dirname: ${__dirname}`);
+debug(`Node version: ${process.version}`);
 
 // Set DXT mode environment variable
 process.env.NODE_ENV = "dxt";
 
-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'}`);
+debug("Environment variables passed from DXT:");
+debug(`  WORDPRESS_SITE_URL: ${process.env.WORDPRESS_SITE_URL ? 'SET' : 'NOT SET'}`);
+debug(`  WORDPRESS_USERNAME: ${process.env.WORDPRESS_USERNAME ? 'SET' : 'NOT SET'}`);
+debug(`  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...");
+        debug("Attempting to import ES module...");
         const { MCPWordPressServer } = await import("./index.js");
         
-        console.error("DEBUG: Creating MCPWordPressServer instance from DXT entry point...");
+        debug("Creating MCPWordPressServer instance from DXT entry point...");
         const server = new MCPWordPressServer();
         
-        console.error("DEBUG: Starting server (DXT mode - fast startup)...");
+        debug("Starting server (DXT mode - fast startup)...");
         await server.run();
         
         // Handle graceful shutdown
         const shutdown = async () => {
-            console.error("DEBUG: Received shutdown signal in DXT entry point");
+            debug("Received shutdown signal in DXT entry point");
             await server.shutdown();
             process.exit(0);
         };
@@ -47,12 +55,14 @@ async function startDXTServer() {
         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.stderr.write(`[DXT] FATAL: Server failed to start: ${error instanceof Error ? error.message : String(error)}\n`);
+        if (error instanceof Error && error.stack) {
+            process.stderr.write(`[DXT] Stack trace: ${error.stack}\n`);
+        }
         process.exit(1);
     }
 }
 
 // Always run when loaded as DXT entry point
-console.error("DEBUG: Calling startDXTServer...");
+debug("Calling startDXTServer...");
 startDXTServer();

package/src/dxt-entry.ts

@@ -4,35 +4,29 @@
  * 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 { LoggerFactory } from "./utils/logger.js";
 import { MCPWordPressServer } from "./index.js";
 
+const logger = LoggerFactory.server();
+
+logger.debug("DXT entry point starting...");
+logger.debug(`Current working directory: ${process.cwd()}`);
+logger.debug(`__dirname equivalent: ${import.meta.url}`);
+logger.debug("Environment variables passed from DXT:");
+logger.debug(`  WORDPRESS_SITE_URL: ${process.env.WORDPRESS_SITE_URL ? "SET" : "NOT SET"}`);
+logger.debug(`  WORDPRESS_USERNAME: ${process.env.WORDPRESS_USERNAME ? "SET" : "NOT SET"}`);
+logger.debug(`  WORDPRESS_APP_PASSWORD: ${process.env.WORDPRESS_APP_PASSWORD ? "SET" : "NOT SET"}`);
+
 async function startDXTServer() {
   try {
-    console.error(
-      "DEBUG: Creating MCPWordPressServer instance from DXT entry point...",
-    );
+    logger.debug("Creating MCPWordPressServer instance from DXT entry point...");
     const server = new MCPWordPressServer();
-    console.error("DEBUG: Starting server...");
+    logger.debug("Starting server...");
     await server.run();
 
     // Handle graceful shutdown
     const shutdown = async () => {
-      console.error("DEBUG: Received shutdown signal in DXT entry point");
+      logger.debug("Received shutdown signal in DXT entry point");
       await server.shutdown();
       process.exit(0);
     };
@@ -40,16 +34,12 @@ async function startDXTServer() {
     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"}`,
-    );
+    logger.fatal(`DXT server failed to start: ${error instanceof Error ? error.message : String(error)}`);
+    logger.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...");
+logger.debug("Calling startDXTServer...");
 startDXTServer();

package/src/index.ts

@@ -6,6 +6,9 @@ import { ServerConfiguration, SiteConfig } from "./config/ServerConfiguration.js
 import { ToolRegistry } from "./server/ToolRegistry.js";
 import { ConnectionTester } from "./server/ConnectionTester.js";
 import { getErrorMessage } from "./utils/error.js";
+import { LoggerFactory } from "./utils/logger.js";
+import { ConfigHelpers } from "./config/Config.js";
+import { McpConfigType } from "./config/ServerConfiguration.js";
 
 // --- Constants ---
 const SERVER_VERSION = "1.1.8"; // Technical debt resolution and modular architecture
@@ -17,28 +20,25 @@ class MCPWordPressServer {
   private initialized: boolean = false;
   private siteConfigs: SiteConfig[] = [];
   private toolRegistry: ToolRegistry;
+  private logger = LoggerFactory.server();
 
-  constructor(mcpConfig?: any) {
+  constructor(mcpConfig?: McpConfigType) {
     this.loadConfiguration(mcpConfig);
 
     if (this.wordpressClients.size === 0) {
       // In test environments, don't exit the process
-      if (
-        process.env.NODE_ENV === "test" ||
-        process.env.CI === "true" ||
-        (globalThis as any).__EXECUTION_CONTEXT__ === "jest"
-      ) {
-        console.error("WARNING: No WordPress sites configured in test environment");
+      if (ConfigHelpers.isCI() || ConfigHelpers.isTest() || (globalThis as Record<string, unknown>).__EXECUTION_CONTEXT__ === "jest") {
+        this.logger.warn("No WordPress sites configured in test environment");
         // Create a dummy client for testing
-        this.wordpressClients.set("test", {} as any);
+        this.wordpressClients.set("test", {} as WordPressClient);
       } else {
-        console.error(
-          "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");
+        this.logger.fatal("No WordPress sites configured. Server cannot start.", {
+          expectedEnvVars: [
+            "WORDPRESS_SITE_URL",
+            "WORDPRESS_USERNAME", 
+            "WORDPRESS_APP_PASSWORD"
+          ]
+        });
         process.exit(1);
       }
     }
@@ -52,7 +52,7 @@ class MCPWordPressServer {
     this.setupTools();
   }
 
-  private loadConfiguration(mcpConfig?: any) {
+  private loadConfiguration(mcpConfig?: McpConfigType) {
     const serverConfig = ServerConfiguration.getInstance();
     const { clients, configs } = serverConfig.loadClientConfigurations(mcpConfig);
 
@@ -65,35 +65,45 @@ class MCPWordPressServer {
   }
 
   private async testClientConnections(): Promise<void> {
-    await ConnectionTester.testClientConnections(this.wordpressClients);
+    // Use optimized connection testing with timeouts and concurrency control
+    await ConnectionTester.testClientConnections(this.wordpressClients, {
+      timeout: ConfigHelpers.getTimeout("test"),
+      maxConcurrent: ConfigHelpers.isCI() ? 2 : 3 // Reduce concurrency in CI
+    });
     this.initialized = true;
   }
 
   async run() {
     // Skip connection testing in DXT environment to prevent timeouts
-    const isDXTMode = process.env.NODE_ENV === "dxt" || process.argv[0]?.includes("dxt-entry");
+    const isDXTMode = ConfigHelpers.isDXT() || process.argv[0]?.includes("dxt-entry");
 
     if (!this.initialized && !isDXTMode) {
-      console.error("INFO: Testing connections to configured WordPress sites...");
+      this.logger.info("Testing connections to configured WordPress sites...", {
+        siteCount: this.wordpressClients.size
+      });
       try {
         await this.testClientConnections();
       } catch (error) {
-        console.error(`WARNING: Connection test failed: ${getErrorMessage(error)}`);
-        console.error("INFO: Continuing with server startup. Tools will be available but connections may fail.");
+        this.logger.warn("Connection test failed - continuing with server startup", {
+          error: getErrorMessage(error)
+        });
       }
     } else if (isDXTMode) {
-      console.error("INFO: DXT mode detected - skipping connection tests for faster startup");
+      this.logger.info("DXT mode detected - skipping connection tests for faster startup");
       this.initialized = true;
     }
 
-    console.error("INFO: Starting MCP WordPress Server...");
+    this.logger.info("Starting MCP WordPress Server...", {
+      version: SERVER_VERSION,
+      sites: this.wordpressClients.size
+    });
 
     // Connect to stdio transport with timeout
     const transport = new StdioServerTransport();
 
     // Add timeout protection for server connection
     const connectionTimeout = setTimeout(() => {
-      console.error("ERROR: Server connection timed out after 30 seconds");
+      this.logger.fatal("Server connection timed out", { timeoutMs: 30000 });
       process.exit(1);
     }, 30000);
 
@@ -101,7 +111,9 @@ class MCPWordPressServer {
       await this.server.connect(transport);
       clearTimeout(connectionTimeout);
 
-      console.error(`INFO: Server started and connected. Tools available for ${this.wordpressClients.size} site(s).`);
+      this.logger.info("Server started and connected successfully", {
+        sites: this.wordpressClients.size
+      });
 
       // Keep the process alive
       process.stdin.resume();
@@ -112,14 +124,16 @@ class MCPWordPressServer {
   }
 
   async shutdown() {
-    console.error("INFO: Shutting down MCP WordPress Server...");
+    this.logger.info("Shutting down MCP WordPress Server...");
     await this.server.close();
-    console.error("INFO: Server stopped.");
+    this.logger.info("Server stopped");
   }
 }
 
 // --- Main Execution ---
 async function main() {
+  const mainLogger = LoggerFactory.server();
+  
   try {
     const mcpServer = new MCPWordPressServer();
     await mcpServer.run();
@@ -132,7 +146,7 @@ async function main() {
     process.on("SIGINT", shutdown);
     process.on("SIGTERM", shutdown);
   } catch (error) {
-    console.error(`FATAL: Failed to start server: ${getErrorMessage(error)}`);
+    mainLogger.fatal("Failed to start server", { error: getErrorMessage(error) });
     process.exit(1);
   }
 }