@masonator/coolify-mcp 2.4.0 → 2.5.0

package/README.md

@@ -43,7 +43,7 @@ The server uses **85% fewer tokens** than a naive implementation (6,600 vs 43,00
 ### Prerequisites
 
 - Node.js >= 18
-- A running Coolify instance
+- A running Coolify instance (tested with v4.0.0-beta.460)
 - Coolify API access token (generate in Coolify Settings > API)
 
 ### Claude Desktop

package/dist/__tests__/mcp-server.test.js

@@ -6,7 +6,7 @@
  * These tests verify MCP server instantiation and structure.
  */
 import { describe, it, expect, beforeEach } from '@jest/globals';
-import { CoolifyMcpServer } from '../lib/mcp-server.js';
+import { CoolifyMcpServer, truncateLogs } from '../lib/mcp-server.js';
 describe('CoolifyMcpServer v2', () => {
     let server;
     beforeEach(() => {
@@ -140,3 +140,49 @@ describe('CoolifyMcpServer v2', () => {
         });
     });
 });
+describe('truncateLogs', () => {
+    it('should return logs unchanged when within limits', () => {
+        const logs = 'line1\nline2\nline3';
+        const result = truncateLogs(logs, 200, 50000);
+        expect(result).toBe(logs);
+    });
+    it('should truncate to last N lines', () => {
+        const logs = 'line1\nline2\nline3\nline4\nline5';
+        const result = truncateLogs(logs, 3, 50000);
+        expect(result).toBe('line3\nline4\nline5');
+    });
+    it('should truncate by character limit when lines are huge', () => {
+        const hugeLine = 'x'.repeat(100);
+        const logs = `${hugeLine}\n${hugeLine}\n${hugeLine}`;
+        const result = truncateLogs(logs, 200, 50);
+        expect(result.length).toBeLessThanOrEqual(50);
+        expect(result.startsWith('...[truncated]...')).toBe(true);
+    });
+    it('should not add truncation prefix when under char limit', () => {
+        const logs = 'line1\nline2\nline3';
+        const result = truncateLogs(logs, 200, 50000);
+        expect(result.startsWith('...[truncated]...')).toBe(false);
+    });
+    it('should handle empty logs', () => {
+        const result = truncateLogs('', 200, 50000);
+        expect(result).toBe('');
+    });
+    it('should use default limits when not specified', () => {
+        const logs = 'line1\nline2';
+        const result = truncateLogs(logs);
+        expect(result).toBe(logs);
+    });
+    it('should respect custom line limit', () => {
+        const lines = Array.from({ length: 300 }, (_, i) => `line${i + 1}`).join('\n');
+        const result = truncateLogs(lines, 50, 50000);
+        const resultLines = result.split('\n');
+        expect(resultLines.length).toBe(50);
+        expect(resultLines[0]).toBe('line251');
+        expect(resultLines[49]).toBe('line300');
+    });
+    it('should respect custom char limit', () => {
+        const logs = 'x'.repeat(1000);
+        const result = truncateLogs(logs, 200, 100);
+        expect(result.length).toBe(100);
+    });
+});

package/dist/lib/mcp-server.d.ts

@@ -5,6 +5,11 @@
 import { McpServer } from '@modelcontextprotocol/sdk/server/mcp.js';
 import type { Transport } from '@modelcontextprotocol/sdk/shared/transport.js';
 import type { CoolifyConfig } from '../types/coolify.js';
+/**
+ * Truncate logs by line count and character count.
+ * Exported for testing.
+ */
+export declare function truncateLogs(logs: string, lineLimit?: number, charLimit?: number): string;
 export declare class CoolifyMcpServer extends McpServer {
     private readonly client;
     constructor(config: CoolifyConfig);

package/dist/lib/mcp-server.js

@@ -2,11 +2,10 @@
  * Coolify MCP Server v2.4.0
  * Consolidated tools for efficient token usage
  */
-/* eslint-disable @typescript-eslint/no-explicit-any */
 import { McpServer } from '@modelcontextprotocol/sdk/server/mcp.js';
 import { z } from 'zod';
 import { CoolifyClient, } from './coolify-client.js';
-const VERSION = '2.4.0';
+const VERSION = '2.5.0';
 /** Wrap handler with error handling */
 function wrap(fn) {
     return fn()
@@ -22,6 +21,24 @@ function wrap(fn) {
         ],
     }));
 }
+const TRUNCATION_PREFIX = '...[truncated]...\n';
+/**
+ * Truncate logs by line count and character count.
+ * Exported for testing.
+ */
+export function truncateLogs(logs, lineLimit = 200, charLimit = 50000) {
+    // First: limit by lines
+    const logLines = logs.split('\n');
+    const limitedLines = logLines.slice(-lineLimit);
+    let truncatedLogs = limitedLines.join('\n');
+    // Second: limit by characters (safety net for huge lines)
+    if (truncatedLogs.length > charLimit) {
+        // Account for prefix length to stay within limit
+        const prefixLen = TRUNCATION_PREFIX.length;
+        truncatedLogs = TRUNCATION_PREFIX + truncatedLogs.slice(-(charLimit - prefixLen));
+    }
+    return truncatedLogs;
+}
 export class CoolifyMcpServer extends McpServer {
     constructor(config) {
         super({ name: 'coolify', version: VERSION });
@@ -205,8 +222,7 @@ export class CoolifyMcpServer extends McpServer {
             // Delete fields
             delete_volumes: z.boolean().optional(),
         }, async (args) => {
-            // Strip MCP-internal fields before passing to API (fixes #76)
-            const { action, uuid, delete_volumes, ...apiData } = args;
+            const { action, uuid, delete_volumes } = args;
             switch (action) {
                 case 'create_public':
                     if (!args.project_uuid ||
@@ -224,7 +240,19 @@ export class CoolifyMcpServer extends McpServer {
                             ],
                         };
                     }
-                    return wrap(() => this.client.createApplicationPublic(apiData));
+                    return wrap(() => this.client.createApplicationPublic({
+                        project_uuid: args.project_uuid,
+                        server_uuid: args.server_uuid,
+                        git_repository: args.git_repository,
+                        git_branch: args.git_branch,
+                        build_pack: args.build_pack,
+                        ports_exposes: args.ports_exposes,
+                        environment_name: args.environment_name,
+                        environment_uuid: args.environment_uuid,
+                        name: args.name,
+                        description: args.description,
+                        fqdn: args.fqdn,
+                    }));
                 case 'create_github':
                     if (!args.project_uuid ||
                         !args.server_uuid ||
@@ -240,7 +268,20 @@ export class CoolifyMcpServer extends McpServer {
                             ],
                         };
                     }
-                    return wrap(() => this.client.createApplicationPrivateGH(apiData));
+                    return wrap(() => this.client.createApplicationPrivateGH({
+                        project_uuid: args.project_uuid,
+                        server_uuid: args.server_uuid,
+                        github_app_uuid: args.github_app_uuid,
+                        git_repository: args.git_repository,
+                        git_branch: args.git_branch,
+                        build_pack: args.build_pack,
+                        ports_exposes: args.ports_exposes,
+                        environment_name: args.environment_name,
+                        environment_uuid: args.environment_uuid,
+                        name: args.name,
+                        description: args.description,
+                        fqdn: args.fqdn,
+                    }));
                 case 'create_key':
                     if (!args.project_uuid ||
                         !args.server_uuid ||
@@ -256,7 +297,20 @@ export class CoolifyMcpServer extends McpServer {
                             ],
                         };
                     }
-                    return wrap(() => this.client.createApplicationPrivateKey(apiData));
+                    return wrap(() => this.client.createApplicationPrivateKey({
+                        project_uuid: args.project_uuid,
+                        server_uuid: args.server_uuid,
+                        private_key_uuid: args.private_key_uuid,
+                        git_repository: args.git_repository,
+                        git_branch: args.git_branch,
+                        build_pack: args.build_pack,
+                        ports_exposes: args.ports_exposes,
+                        environment_name: args.environment_name,
+                        environment_uuid: args.environment_uuid,
+                        name: args.name,
+                        description: args.description,
+                        fqdn: args.fqdn,
+                    }));
                 case 'create_dockerimage':
                     if (!args.project_uuid ||
                         !args.server_uuid ||
@@ -271,11 +325,25 @@ export class CoolifyMcpServer extends McpServer {
                             ],
                         };
                     }
-                    return wrap(() => this.client.createApplicationDockerImage(apiData));
-                case 'update':
+                    return wrap(() => this.client.createApplicationDockerImage({
+                        project_uuid: args.project_uuid,
+                        server_uuid: args.server_uuid,
+                        docker_registry_image_name: args.docker_registry_image_name,
+                        ports_exposes: args.ports_exposes,
+                        docker_registry_image_tag: args.docker_registry_image_tag,
+                        environment_name: args.environment_name,
+                        environment_uuid: args.environment_uuid,
+                        name: args.name,
+                        description: args.description,
+                        fqdn: args.fqdn,
+                    }));
+                case 'update': {
                     if (!uuid)
                         return { content: [{ type: 'text', text: 'Error: uuid required' }] };
-                    return wrap(() => this.client.updateApplication(uuid, apiData));
+                    // eslint-disable-next-line @typescript-eslint/no-unused-vars
+                    const { action: _, uuid: __, delete_volumes: ___, ...updateData } = args;
+                    return wrap(() => this.client.updateApplication(uuid, updateData));
+                }
                 case 'delete':
                     if (!uuid)
                         return { content: [{ type: 'text', text: 'Error: uuid required' }] };
@@ -378,8 +446,7 @@ export class CoolifyMcpServer extends McpServer {
             docker_compose_raw: z.string().optional(),
             delete_volumes: z.boolean().optional(),
         }, async (args) => {
-            // Strip MCP-internal fields before passing to API (fixes #76)
-            const { action, uuid, delete_volumes, ...apiData } = args;
+            const { action, uuid, delete_volumes } = args;
             switch (action) {
                 case 'create':
                     if (!args.server_uuid || !args.project_uuid) {
@@ -389,11 +456,23 @@ export class CoolifyMcpServer extends McpServer {
                             ],
                         };
                     }
-                    return wrap(() => this.client.createService(apiData));
-                case 'update':
+                    return wrap(() => this.client.createService({
+                        project_uuid: args.project_uuid,
+                        server_uuid: args.server_uuid,
+                        type: args.type,
+                        name: args.name,
+                        description: args.description,
+                        environment_name: args.environment_name,
+                        instant_deploy: args.instant_deploy,
+                        docker_compose_raw: args.docker_compose_raw,
+                    }));
+                case 'update': {
                     if (!uuid)
                         return { content: [{ type: 'text', text: 'Error: uuid required' }] };
-                    return wrap(() => this.client.updateService(uuid, apiData));
+                    // eslint-disable-next-line @typescript-eslint/no-unused-vars
+                    const { action: _, uuid: __, delete_volumes: ___, ...updateData } = args;
+                    return wrap(() => this.client.updateService(uuid, updateData));
+                }
                 case 'delete':
                     if (!uuid)
                         return { content: [{ type: 'text', text: 'Error: uuid required' }] };
@@ -483,21 +562,18 @@ export class CoolifyMcpServer extends McpServer {
         // =========================================================================
         this.tool('list_deployments', 'List deployments (summary)', { page: z.number().optional(), per_page: z.number().optional() }, async ({ page, per_page }) => wrap(() => this.client.listDeployments({ page, per_page, summary: true })));
         this.tool('deploy', 'Deploy by tag/UUID', { tag_or_uuid: z.string(), force: z.boolean().optional() }, async ({ tag_or_uuid, force }) => wrap(() => this.client.deployByTagOrUuid(tag_or_uuid, force)));
-        this.tool('deployment', 'Manage deployment: get/cancel/list_for_app', {
+        this.tool('deployment', 'Manage deployment: get/cancel/list_for_app (logs truncated to last 200 lines/50K chars by default)', {
             action: z.enum(['get', 'cancel', 'list_for_app']),
             uuid: z.string(),
-            lines: z.number().optional(), // Limit log output to last N lines (for 'get' action)
-        }, async ({ action, uuid, lines }) => {
+            lines: z.number().optional(), // Limit log output to last N lines (default: 200)
+            max_chars: z.number().optional(), // Limit log output to last N chars (default: 50000)
+        }, async ({ action, uuid, lines, max_chars }) => {
             switch (action) {
                 case 'get':
                     return wrap(async () => {
                         const deployment = await this.client.getDeployment(uuid);
-                        // Truncate logs to last N lines if specified
-                        if (lines && deployment.logs) {
-                            const logLines = deployment.logs.split('\n');
-                            if (logLines.length > lines) {
-                                deployment.logs = logLines.slice(-lines).join('\n');
-                            }
+                        if (deployment.logs) {
+                            deployment.logs = truncateLogs(deployment.logs, lines ?? 200, max_chars ?? 50000);
                         }
                         return deployment;
                     });

package/dist/types/coolify.d.ts

@@ -224,11 +224,15 @@ export interface CreateApplicationPublicRequest {
     start_command?: string;
     instant_deploy?: boolean;
 }
-export interface CreateApplicationPrivateGHRequest extends CreateApplicationPublicRequest {
+export interface CreateApplicationPrivateGHRequest extends Omit<CreateApplicationPublicRequest, 'build_pack' | 'ports_exposes'> {
     github_app_uuid: string;
+    build_pack?: BuildPack;
+    ports_exposes?: string;
 }
-export interface CreateApplicationPrivateKeyRequest extends CreateApplicationPublicRequest {
+export interface CreateApplicationPrivateKeyRequest extends Omit<CreateApplicationPublicRequest, 'build_pack' | 'ports_exposes'> {
     private_key_uuid: string;
+    build_pack?: BuildPack;
+    ports_exposes?: string;
 }
 export interface CreateApplicationDockerfileRequest {
     project_uuid: string;

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@masonator/coolify-mcp",
   "scope": "@masonator",
-  "version": "2.4.0",
+  "version": "2.5.0",
   "description": "MCP server implementation for Coolify",
   "type": "module",
   "main": "./dist/index.js",
@@ -59,6 +59,7 @@
     "globals": "^17.0.0",
     "husky": "^9.0.11",
     "jest": "^29.7.0",
+    "jest-junit": "^16.0.0",
     "lint-staged": "^16.2.7",
     "markdownlint-cli2": "^0.20.0",
     "prettier": "^3.5.3",