@jgardner04/ghost-mcp-server 1.12.1 → 1.12.3

package/src/__tests__/mcp_server_pages.test.js

@@ -125,12 +125,12 @@ vi.mock('path', async () => {
   };
 });
 
-describe('mcp_server_improved - ghost_get_pages tool', () => {
+describe('mcp_server - ghost_get_pages tool', () => {
   beforeEach(async () => {
     vi.clearAllMocks();
     // Import the module to register tools (only first time)
     if (mockTools.size === 0) {
-      await import('../mcp_server_improved.js');
+      await import('../mcp_server.js');
     }
   });
 
@@ -206,11 +206,11 @@ describe('mcp_server_improved - ghost_get_pages tool', () => {
   });
 });
 
-describe('mcp_server_improved - ghost_get_page tool', () => {
+describe('mcp_server - ghost_get_page tool', () => {
   beforeEach(async () => {
     vi.clearAllMocks();
     if (mockTools.size === 0) {
-      await import('../mcp_server_improved.js');
+      await import('../mcp_server.js');
     }
   });
 
@@ -270,11 +270,11 @@ describe('mcp_server_improved - ghost_get_page tool', () => {
   });
 });
 
-describe('mcp_server_improved - ghost_create_page tool', () => {
+describe('mcp_server - ghost_create_page tool', () => {
   beforeEach(async () => {
     vi.clearAllMocks();
     if (mockTools.size === 0) {
-      await import('../mcp_server_improved.js');
+      await import('../mcp_server.js');
     }
   });
 
@@ -345,11 +345,11 @@ describe('mcp_server_improved - ghost_create_page tool', () => {
   });
 });
 
-describe('mcp_server_improved - ghost_update_page tool', () => {
+describe('mcp_server - ghost_update_page tool', () => {
   beforeEach(async () => {
     vi.clearAllMocks();
     if (mockTools.size === 0) {
-      await import('../mcp_server_improved.js');
+      await import('../mcp_server.js');
     }
   });
 
@@ -411,11 +411,11 @@ describe('mcp_server_improved - ghost_update_page tool', () => {
   });
 });
 
-describe('mcp_server_improved - ghost_delete_page tool', () => {
+describe('mcp_server - ghost_delete_page tool', () => {
   beforeEach(async () => {
     vi.clearAllMocks();
     if (mockTools.size === 0) {
-      await import('../mcp_server_improved.js');
+      await import('../mcp_server.js');
     }
   });
 
@@ -451,11 +451,11 @@ describe('mcp_server_improved - ghost_delete_page tool', () => {
   });
 });
 
-describe('mcp_server_improved - ghost_search_pages tool', () => {
+describe('mcp_server - ghost_search_pages tool', () => {
   beforeEach(async () => {
     vi.clearAllMocks();
     if (mockTools.size === 0) {
-      await import('../mcp_server_improved.js');
+      await import('../mcp_server.js');
     }
   });
 

package/src/errors/index.js

@@ -40,6 +40,13 @@ export class ValidationError extends BaseError {
     this.errors = errors;
   }
 
+  toJSON() {
+    return {
+      ...super.toJSON(),
+      ...(this.errors.length > 0 && { errors: this.errors }),
+    };
+  }
+
   static fromJoi(joiError) {
     const errors = joiError.details.map((detail) => ({
       field: detail.path.join('.'),

package/src/index.js

@@ -5,7 +5,6 @@ import dotenv from 'dotenv';
 import postRoutes from './routes/postRoutes.js'; // Import post routes
 import imageRoutes from './routes/imageRoutes.js'; // Import image routes
 import tagRoutes from './routes/tagRoutes.js'; // Import tag routes
-import { startMCPServer } from './mcp_server.js'; // Import MCP server start function
 import { createContextLogger } from './utils/logger.js';
 
 // Load environment variables from .env file
@@ -16,7 +15,6 @@ const logger = createContextLogger('main');
 
 const app = express();
 const restApiPort = process.env.PORT || 3000;
-const mcpPort = process.env.MCP_PORT || 3001; // Allow configuring MCP port
 
 // Apply security headers with Helmet (OWASP recommended)
 app.use(
@@ -91,9 +89,8 @@ app.use((err, req, res, _next) => {
   });
 });
 
-// Start both servers
-const startServers = async () => {
-  // Start Express server
+// Start Express server
+const startServer = () => {
   app.listen(restApiPort, () => {
     logger.info('Express REST API server started', {
       port: restApiPort,
@@ -101,16 +98,6 @@ const startServers = async () => {
       type: 'server_start',
     });
   });
-
-  // Start MCP server
-  await startMCPServer(mcpPort);
 };
 
-startServers().catch((error) => {
-  logger.error('Failed to start servers', {
-    error: error.message,
-    stack: error.stack,
-    type: 'server_start_error',
-  });
-  process.exit(1);
-});
+startServer();