@elitedcs/ghl-mcp 3.34.7 → 3.34.8

package/CHANGELOG.md

@@ -1,5 +1,35 @@
 # Changelog
 
+## 3.34.8 — create_opportunity + get_funnel_pages no longer 422 on omitted required params
+
+Two real defects found during a full end-to-end build test on a live sub-account
+(every advertised object created, read back, and deleted). Both are cases where GHL
+requires a parameter the tool treated as optional, so the call 422'd unless the
+caller happened to pass it.
+
+- **`create_opportunity` now defaults `status` to `"open"`.** GHL's
+  `POST /opportunities/` rejects a create with no status (`422 "status should not
+  be empty"`). The tool exposed `status` as optional and omitted it from the body,
+  so a bare create always failed. It now sends `status: "open"` when the caller
+  doesn't specify one. Logic extracted to `buildCreateOpportunityBody` with tests.
+- **`get_funnel_pages` now defaults `limit`/`offset`.** GHL's `GET /funnels/page`
+  requires both (`422 "limit should not be empty" / "offset should not be empty"`)
+  and caps `limit` at 20. The tool only sent them when provided, so listing a
+  funnel's pages failed by default. It now sends `limit: 20, offset: 0` when
+  omitted. (`get_funnels` / `GET /funnels/funnel/list` does not require them and is
+  unchanged.) Logic extracted to `buildFunnelPagesParams` with tests.
+
+Verified working in the same test pass (no change needed): create_funnel,
+create_funnel_page, create_form, create_calendar (incl. team-member assignment),
+create_custom_field, create_location_tag, create_contact, and the full workflow
+lifecycle (create → update actions → validate → publish). Suite 280 pass / 1 skip.
+
+Known follow-up (not in this release): exposing calendar availability / `openHours`.
+Investigation showed round-robin "OptimizeForAvailability" calendars derive
+availability from the assigned user's working hours, not a calendar `openHours`
+field (every live round-robin calendar inspected had `openHours: {}`), so this needs
+a design + live write-verification pass before shipping rather than a guessed shape.
+
 ## 3.34.7 — Audit now sees UUID pipeline-stage ids (dead-stage silent failure)
 
 - **`audit_workflows` / `validate_workflow` now catch a dead pipeline STAGE

package/dist/index.js

@@ -31,7 +31,7 @@ var require_package = __commonJS({
   "package.json"(exports2, module2) {
     module2.exports = {
       name: "@elitedcs/ghl-mcp",
-      version: "3.34.7",
+      version: "3.34.8",
       mcpName: "io.github.drjerryrelth/ghl-command",
       description: "GoHighLevel MCP Server for Claude. 218 tools \u2014 full CRM, automation, marketing control, account-wide workflow audit, and the only programmatic GHL workflow builder, now multi-tenant across client accounts.",
       main: "dist/index.js",
@@ -2540,6 +2540,20 @@ function registerConversationTools(server2, client) {
 var import_zod8 = require("zod");
 var statusEnum = import_zod8.z.enum(["open", "won", "lost", "abandoned"]).describe("Opportunity status");
 var statusEnumWithAll = import_zod8.z.enum(["open", "won", "lost", "abandoned", "all"]).describe("Opportunity status filter (use 'all' to include every status)");
+function buildCreateOpportunityBody(args, locationId2) {
+  const body = {
+    locationId: locationId2,
+    pipelineId: args.pipelineId,
+    pipelineStageId: args.pipelineStageId,
+    contactId: args.contactId,
+    name: args.name,
+    status: args.status ?? "open"
+  };
+  if (args.monetaryValue !== void 0) body.monetaryValue = args.monetaryValue;
+  if (args.assignedTo !== void 0) body.assignedTo = args.assignedTo;
+  if (args.source !== void 0) body.source = args.source;
+  return body;
+}
 function registerOpportunityTools(server2, client) {
   safeTool(
     server2,
@@ -2612,25 +2626,14 @@ function registerOpportunityTools(server2, client) {
       pipelineStageId: import_zod8.z.string().describe("Pipeline stage ID (required)"),
       contactId: import_zod8.z.string().describe("Contact ID (required)"),
       name: import_zod8.z.string().describe("Opportunity name (required)"),
-      status: statusEnum.optional().describe("Opportunity status"),
+      status: statusEnum.optional().describe("Opportunity status. Defaults to 'open' (GHL requires a non-empty status on create)."),
       monetaryValue: import_zod8.z.number().optional().describe("Monetary value of the opportunity"),
       assignedTo: import_zod8.z.string().optional().describe("User ID to assign the opportunity to"),
       source: import_zod8.z.string().optional().describe("Lead source")
     },
     async (args) => {
       const locationId2 = client.resolveLocationId(args.locationId);
-      const body = {
-        locationId: locationId2,
-        pipelineId: args.pipelineId,
-        pipelineStageId: args.pipelineStageId,
-        contactId: args.contactId,
-        name: args.name
-      };
-      if (args.status !== void 0) body.status = args.status;
-      if (args.monetaryValue !== void 0)
-        body.monetaryValue = args.monetaryValue;
-      if (args.assignedTo !== void 0) body.assignedTo = args.assignedTo;
-      if (args.source !== void 0) body.source = args.source;
+      const body = buildCreateOpportunityBody(args, locationId2);
       return await client.post("/opportunities/", { body });
     }
   );
@@ -3289,6 +3292,14 @@ function registerWorkflowTools(server2, client) {
 
 // src/tools/funnels.ts
 var import_zod12 = require("zod");
+function buildFunnelPagesParams(args) {
+  return {
+    locationId: args.locationId,
+    funnelId: args.funnelId,
+    limit: args.limit ?? 20,
+    offset: args.offset ?? 0
+  };
+}
 function registerFunnelTools(server2, client) {
   safeTool(
     server2,
@@ -3314,17 +3325,12 @@ function registerFunnelTools(server2, client) {
     {
       locationId: import_zod12.z.string().optional().describe("GHL Location ID (optional if GHL_LOCATION_ID is set)"),
       funnelId: import_zod12.z.string().describe("The funnel ID"),
-      limit: import_zod12.z.number().optional().describe("Max number of pages to return"),
-      offset: import_zod12.z.number().optional().describe("Offset for pagination")
+      limit: import_zod12.z.number().optional().describe("Max number of pages to return (1-20). Defaults to 20 (GHL requires it and caps at 20)."),
+      offset: import_zod12.z.number().optional().describe("Offset for pagination. Defaults to 0 (GHL requires it).")
     },
     async ({ locationId: locationId2, funnelId, limit, offset }) => {
       const resolvedLocationId = client.resolveLocationId(locationId2);
-      const params = {
-        locationId: resolvedLocationId,
-        funnelId
-      };
-      if (limit !== void 0) params.limit = limit;
-      if (offset !== void 0) params.offset = offset;
+      const params = buildFunnelPagesParams({ locationId: resolvedLocationId, funnelId, limit, offset });
       return client.get("/funnels/page", { params });
     }
   );

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@elitedcs/ghl-mcp",
-  "version": "3.34.7",
+  "version": "3.34.8",
   "mcpName": "io.github.drjerryrelth/ghl-command",
   "description": "GoHighLevel MCP Server for Claude. 218 tools — full CRM, automation, marketing control, account-wide workflow audit, and the only programmatic GHL workflow builder, now multi-tenant across client accounts.",
   "main": "dist/index.js",