Stu Mason
Stu Mason

Activity

Pull Request Opened

PR #77 opened: fix: strip MCP-internal fields before API calls

Summary

  • Fixes the application tool passing action, uuid, delete_volumes to the Coolify API
  • Fixes the service tool with the same issue
  • Adds tests documenting client pass-through behavior

Root Cause

The consolidated tools use an action parameter for internal routing (e.g., create_public, update, delete). The handlers were passing the full args object to client methods, but the Coolify API rejects action with "This field is not allowed."

Solution

Destructure args to separate MCP-internal fields from API payload:

const { action, uuid, delete_volumes, ...apiData } = args;
// Now use apiData for API calls

Test Plan

  • All 186 tests passing
  • Build successful
  • Lint clean

Fixes #76

Stu Mason + AI [email protected]

+98
additions
-13
deletions
2
files changed