@jgardner04/ghost-mcp-server 1.12.3 → 1.12.5

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jgardner04/ghost-mcp-server",
-  "version": "1.12.3",
+  "version": "1.12.5",
   "description": "A Model Context Protocol (MCP) server for interacting with Ghost CMS via the Admin API",
   "author": "Jonathan Gardner",
   "type": "module",
@@ -73,7 +73,8 @@
     "sharp": "^0.34.1",
     "uuid": "^11.1.0",
     "winston": "^3.17.0",
-    "zod": "^3.25.76"
+    "zod": "^3.25.76",
+    "zod-to-json-schema": "^3.25.0"
   },
   "keywords": [
     "ghost",

package/src/errors/index.js

@@ -439,7 +439,7 @@ export async function retryWithBackoff(fn, options = {}) {
       }
 
       const delay = ErrorHandler.getRetryDelay(attempt, error);
-      console.log(`Retry attempt ${attempt}/${maxAttempts} after ${delay}ms`);
+      console.error(`Retry attempt ${attempt}/${maxAttempts} after ${delay}ms`);
 
       await new Promise((resolve) => setTimeout(resolve, delay));
       onRetry(attempt, error);

package/src/mcp_server_enhanced.js

@@ -24,7 +24,7 @@ import {
 
 dotenv.config();
 
-console.log('Initializing Enhanced MCP Server...');
+console.error('Initializing Enhanced MCP Server...');
 
 // Initialize components
 const resourceManager = new ResourceManager(ghostService);
@@ -48,7 +48,7 @@ const mcpServer = new MCPServer({
 
 // --- Register Resources with Enhanced Fetching ---
 
-console.log('Registering enhanced resources...');
+console.error('Registering enhanced resources...');
 
 // Ghost Post Resource
 const postResource = resourceManager.registerResource(
@@ -127,7 +127,7 @@ mcpServer.addResource(tagResource);
 
 // --- Enhanced Tools ---
 
-console.log('Registering enhanced tools...');
+console.error('Registering enhanced tools...');
 
 // Batch Operations Tool
 const batchOperationsTool = new Tool({
@@ -434,7 +434,7 @@ const subscriptionTool = new Tool({
         input.uri,
         (event) => {
           // In a real implementation, this would send events to the client
-          console.log('Resource update:', event);
+          console.error('Resource update:', event);
         },
         input.options || {}
       );
@@ -468,13 +468,13 @@ mcpServer.addTool(subscriptionTool);
 
 const startEnhancedMCPServer = async (transport = 'http', options = {}) => {
   try {
-    console.log(`Starting Enhanced MCP Server with ${transport} transport...`);
+    console.error(`Starting Enhanced MCP Server with ${transport} transport...`);
 
     switch (transport) {
       case 'stdio': {
         const stdioTransport = new StdioServerTransport();
         await mcpServer.connect(stdioTransport);
-        console.log('Enhanced MCP Server running on stdio transport');
+        console.error('Enhanced MCP Server running on stdio transport');
         break;
       }
 
@@ -553,10 +553,10 @@ const startEnhancedMCPServer = async (transport = 'http', options = {}) => {
         await mcpServer.connect(sseTransport);
 
         const server = app.listen(port, () => {
-          console.log(`Enhanced MCP Server (SSE) listening on port ${port}`);
-          console.log(`Health: http://localhost:${port}/health`);
-          console.log(`Resources: http://localhost:${port}/resources`);
-          console.log(`SSE: http://localhost:${port}/mcp/sse`);
+          console.error(`Enhanced MCP Server (SSE) listening on port ${port}`);
+          console.error(`Health: http://localhost:${port}/health`);
+          console.error(`Resources: http://localhost:${port}/resources`);
+          console.error(`SSE: http://localhost:${port}/mcp/sse`);
         });
 
         mcpServer._httpServer = server;
@@ -574,7 +574,7 @@ const startEnhancedMCPServer = async (transport = 'http', options = {}) => {
         const wss = new WebSocketServer({ port: wsPort });
 
         wss.on('connection', async (ws) => {
-          console.log('New WebSocket connection');
+          console.error('New WebSocket connection');
 
           const wsTransport = new WebSocketServerTransport(ws);
           await mcpServer.connect(wsTransport);
@@ -616,7 +616,7 @@ const startEnhancedMCPServer = async (transport = 'http', options = {}) => {
           });
         });
 
-        console.log(`Enhanced MCP Server (WebSocket) listening on port ${wsPort}`);
+        console.error(`Enhanced MCP Server (WebSocket) listening on port ${wsPort}`);
         mcpServer._wss = wss;
         break;
       }
@@ -626,18 +626,18 @@ const startEnhancedMCPServer = async (transport = 'http', options = {}) => {
     }
 
     // Log capabilities
-    console.log('Server Capabilities:');
-    console.log(
+    console.error('Server Capabilities:');
+    console.error(
       '- Resources:',
       mcpServer.listResources().map((r) => r.name)
     );
-    console.log(
+    console.error(
       '- Tools:',
       mcpServer.listTools().map((t) => t.name)
     );
-    console.log('- Cache enabled with LRU eviction');
-    console.log('- Subscription support for real-time updates');
-    console.log('- Batch operations for efficiency');
+    console.error('- Cache enabled with LRU eviction');
+    console.error('- Subscription support for real-time updates');
+    console.error('- Batch operations for efficiency');
   } catch (error) {
     errorLogger.logError(error);
     console.error('Failed to start Enhanced MCP Server:', error);
@@ -647,7 +647,7 @@ const startEnhancedMCPServer = async (transport = 'http', options = {}) => {
 
 // Graceful shutdown
 const shutdown = async () => {
-  console.log('\nShutting down Enhanced MCP Server...');
+  console.error('\nShutting down Enhanced MCP Server...');
 
   // Clear all subscriptions
   resourceManager.subscriptionManager.subscriptions.clear();

package/src/middleware/__tests__/errorMiddleware.test.js

@@ -299,7 +299,7 @@ describe('errorMiddleware', () => {
 
     describe('logInfo', () => {
       it('should log info message when level allows', async () => {
-        const consoleSpy = vi.spyOn(console, 'log').mockImplementation(() => {});
+        const consoleSpy = vi.spyOn(console, 'error').mockImplementation(() => {});
 
         const logger = new ErrorLogger({ logLevel: 'info', enableFileLogging: false });
         await logger.logInfo('Info message');
@@ -309,7 +309,7 @@ describe('errorMiddleware', () => {
       });
 
       it('should not log when level is higher', async () => {
-        const consoleSpy = vi.spyOn(console, 'log').mockImplementation(() => {});
+        const consoleSpy = vi.spyOn(console, 'error').mockImplementation(() => {});
 
         const logger = new ErrorLogger({ logLevel: 'error', enableFileLogging: false });
         await logger.logInfo('Info message');
@@ -333,7 +333,7 @@ describe('errorMiddleware', () => {
 
     describe('logDebug', () => {
       it('should log debug message when level is debug', async () => {
-        const consoleSpy = vi.spyOn(console, 'log').mockImplementation(() => {});
+        const consoleSpy = vi.spyOn(console, 'error').mockImplementation(() => {});
 
         const logger = new ErrorLogger({ logLevel: 'debug', enableFileLogging: false });
         await logger.logDebug('Debug message');
@@ -343,7 +343,7 @@ describe('errorMiddleware', () => {
       });
 
       it('should not log when level is not debug', async () => {
-        const consoleSpy = vi.spyOn(console, 'log').mockImplementation(() => {});
+        const consoleSpy = vi.spyOn(console, 'error').mockImplementation(() => {});
 
         const logger = new ErrorLogger({ logLevel: 'info', enableFileLogging: false });
         await logger.logDebug('Debug message');

package/src/middleware/errorMiddleware.js

@@ -111,7 +111,7 @@ export class ErrorLogger {
 
   async logInfo(message, meta = {}) {
     if (['info', 'debug'].includes(this.logLevel)) {
-      console.log(`[INFO] ${message}`);
+      console.error(`[INFO] ${message}`);
       const entry = this.formatLogEntry('info', message, meta);
       await this.writeToFile('app', entry);
     }
@@ -127,7 +127,7 @@ export class ErrorLogger {
 
   async logDebug(message, meta = {}) {
     if (this.logLevel === 'debug') {
-      console.log(`[DEBUG] ${message}`);
+      console.error(`[DEBUG] ${message}`);
       const entry = this.formatLogEntry('debug', message, meta);
       await this.writeToFile('debug', entry);
     }
@@ -446,11 +446,11 @@ export class GracefulShutdown {
     if (this.isShuttingDown) return;
 
     this.isShuttingDown = true;
-    console.log('Graceful shutdown initiated...');
+    console.error('Graceful shutdown initiated...');
 
     // Stop accepting new connections
     server.close(() => {
-      console.log('Server closed to new connections');
+      console.error('Server closed to new connections');
     });
 
     // Close existing connections

package/src/resources/ResourceManager.js

@@ -168,7 +168,7 @@ class ResourceFetcher {
     // Check cache
     const cached = this.cache.get(cacheKey);
     if (cached) {
-      console.log(`Cache hit for ${cacheKey}`);
+      console.error(`Cache hit for ${cacheKey}`);
       return cached;
     }
 
@@ -221,7 +221,7 @@ class ResourceFetcher {
     // Check cache
     const cached = this.cache.get(cacheKey);
     if (cached) {
-      console.log(`Cache hit for posts query`);
+      console.error(`Cache hit for posts query`);
       return cached;
     }
 
@@ -277,7 +277,7 @@ class ResourceFetcher {
     // Check cache
     const cached = this.cache.get(cacheKey);
     if (cached) {
-      console.log(`Cache hit for ${cacheKey}`);
+      console.error(`Cache hit for ${cacheKey}`);
       return cached;
     }
 
@@ -324,7 +324,7 @@ class ResourceFetcher {
     // Check cache
     const cached = this.cache.get(cacheKey);
     if (cached) {
-      console.log(`Cache hit for tags query`);
+      console.error(`Cache hit for tags query`);
       return cached;
     }
 
@@ -400,7 +400,7 @@ class ResourceSubscriptionManager extends EventEmitter {
       this.startPolling(subscriptionId, pollingInterval);
     }
 
-    console.log(`Created subscription ${subscriptionId} for ${uri}`);
+    console.error(`Created subscription ${subscriptionId} for ${uri}`);
 
     return subscriptionId;
   }
@@ -418,7 +418,7 @@ class ResourceSubscriptionManager extends EventEmitter {
     // Remove subscription
     this.subscriptions.delete(subscriptionId);
 
-    console.log(`Removed subscription ${subscriptionId}`);
+    console.error(`Removed subscription ${subscriptionId}`);
   }
 
   startPolling(subscriptionId, interval) {
@@ -539,7 +539,7 @@ export class ResourceManager {
     try {
       const parsed = ResourceURIParser.parse(uri);
 
-      console.log('Fetching resource:', { uri: uri.substring(0, 100), parsed });
+      console.error('Fetching resource:', { uri: uri.substring(0, 100), parsed });
 
       // Route to appropriate fetcher
       switch (parsed.resourceType) {
@@ -605,7 +605,7 @@ export class ResourceManager {
    */
   invalidateCache(pattern = null) {
     this.cache.invalidate(pattern);
-    console.log(`Cache invalidated${pattern ? ` for pattern: ${pattern}` : ''}`);
+    console.error(`Cache invalidated${pattern ? ` for pattern: ${pattern}` : ''}`);
   }
 
   /**

package/src/resources/__tests__/ResourceManager.test.js

@@ -836,7 +836,7 @@ describe('ResourceManager', () => {
       });
 
       it('should log invalidation', async () => {
-        const consoleSpy = vi.spyOn(console, 'log').mockImplementation(() => {});
+        const consoleSpy = vi.spyOn(console, 'error').mockImplementation(() => {});
 
         resourceManager.invalidateCache();
         expect(consoleSpy).toHaveBeenCalledWith(expect.stringContaining('Cache invalidated'));

package/src/services/ghostServiceImproved.js

@@ -54,7 +54,7 @@ const handleApiRequest = async (resource, action, data = {}, options = {}, confi
   // Main execution function
   const executeRequest = async () => {
     try {
-      console.log(`Executing Ghost API request: ${operation}`);
+      console.error(`Executing Ghost API request: ${operation}`);
 
       let result;
 
@@ -78,7 +78,7 @@ const handleApiRequest = async (resource, action, data = {}, options = {}, confi
           result = await api[resource][action](data);
       }
 
-      console.log(`Successfully executed Ghost API request: ${operation}`);
+      console.error(`Successfully executed Ghost API request: ${operation}`);
       return result;
     } catch (error) {
       // Transform Ghost API errors into our error types
@@ -96,12 +96,12 @@ const handleApiRequest = async (resource, action, data = {}, options = {}, confi
     return await retryWithBackoff(wrappedExecute, {
       maxAttempts: maxRetries,
       onRetry: (attempt, _error) => {
-        console.log(`Retrying ${operation} (attempt ${attempt}/${maxRetries})`);
+        console.error(`Retrying ${operation} (attempt ${attempt}/${maxRetries})`);
 
         // Log circuit breaker state if relevant
         if (useCircuitBreaker) {
           const state = ghostCircuitBreaker.getState();
-          console.log(`Circuit breaker state:`, state);
+          console.error(`Circuit breaker state:`, state);
         }
       },
     });