StuMason/coolify-mcp
TypeScript
Pull Request Opened
PR #77 opened: fix: strip MCP-internal fields before API calls
Summary
- Fixes the
applicationtool passingaction,uuid,delete_volumesto the Coolify API - Fixes the
servicetool 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