@elitedcs/ghl-mcp 3.40.0 → 3.41.0

package/dist/index.js

@@ -31,7 +31,7 @@ var require_package = __commonJS({
   "package.json"(exports2, module2) {
     module2.exports = {
       name: "@elitedcs/ghl-mcp",
-      version: "3.40.0",
+      version: "3.41.0",
       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",
@@ -705,6 +705,12 @@ function decodeFirebaseClaims(idToken) {
   }
 }
 
+// src/id-shape.ts
+var GHL_ID_SHAPE = /^([A-Za-z0-9]{17,}|[0-9a-fA-F]{8}-[0-9a-fA-F]{4}-[0-9a-fA-F]{4}-[0-9a-fA-F]{4}-[0-9a-fA-F]{12})$/;
+function isIdShaped(s) {
+  return typeof s === "string" && GHL_ID_SHAPE.test(s);
+}
+
 // src/trigger-schemas.ts
 var import_zod3 = require("zod");
 var TriggerActionSchema = import_zod3.z.object({
@@ -1170,6 +1176,31 @@ function normalizeRemoveFromWorkflowAction(action) {
   }
   return { ...action, attributes: next };
 }
+function normalizeInternalUpdateOpportunityAction(action) {
+  if (action.type !== "internal_update_opportunity") return action;
+  const attrIn = action.attributes && typeof action.attributes === "object" ? action.attributes : {};
+  const fieldsIn = Array.isArray(attrIn.__customInputFields__) ? attrIn.__customInputFields__ : [];
+  const __customInputFields__ = fieldsIn.map((raw) => {
+    const f = raw && typeof raw === "object" ? raw : {};
+    return {
+      ...f,
+      // shape-preserving — keep every key a real node carries
+      __customInputs__: f.__customInputs__ && typeof f.__customInputs__ === "object" ? f.__customInputs__ : {},
+      dataType: typeof f.dataType === "string" ? f.dataType : "SINGLE_OPTIONS",
+      valueFieldType: typeof f.valueFieldType === "string" ? f.valueFieldType : "select"
+    };
+  });
+  const attributes = {
+    ...attrIn,
+    // shape-preserving
+    type: "internal_update_opportunity",
+    allowBackward: typeof attrIn.allowBackward === "boolean" ? attrIn.allowBackward : false,
+    __customInputs__: attrIn.__customInputs__ && typeof attrIn.__customInputs__ === "object" ? attrIn.__customInputs__ : {},
+    __customInputFields__
+  };
+  delete attributes.workflowsActionType;
+  return { ...action, workflowsActionType: "INTERNAL", attributes };
+}
 function hasId(action) {
   return typeof action.id === "string" && action.id.length > 0;
 }
@@ -1185,7 +1216,7 @@ function getStringArray(value) {
   }
   return value;
 }
-function validateActionChain(actions) {
+function validateActionChain(actions, existingIds) {
   const byId = new Map(actions.filter(hasId).map((action) => [action.id, action]));
   for (const action of actions) {
     const attr = isRecord(action.attributes) ? action.attributes : void 0;
@@ -1207,12 +1238,24 @@ function validateActionChain(actions) {
         if (!attr.startAfter) throw new Error(`Wait action "${action.name}" missing required 'startAfter' in attributes.`);
         break;
       case "internal_update_opportunity": {
-        if (!hasId(action)) {
-          throw new Error(
-            `Action "${action.name}" (internal_update_opportunity / "Update Opportunity" / move-to-stage) cannot be created via the API yet: GHL's workflow builder rejects a synthesized node with "action has a corrupted type", which silently fails the whole save. Build this one step in the GHL UI (Workflow > add action > Update Opportunity), or pass through an existing one read via get_workflow_full (it keeps its id). Every other action type in this call works.`
-          );
+        const isRoundTripped = hasId(action) && (existingIds ? existingIds.has(action.id) : true);
+        if (!isRoundTripped) {
+          const cif = Array.isArray(attr.__customInputFields__) ? attr.__customInputFields__ : null;
+          if (!cif) {
+            throw new Error(
+              `Internal update opportunity action "${action.name}" missing '__customInputFields__' array (needs a pipelineId entry and a pipelineStageId entry). Use get_pipelines to find the IDs.`
+            );
+          }
+          const target = (ff) => cif.find((f) => f && f.filterField === ff);
+          for (const ff of ["pipelineId", "pipelineStageId"]) {
+            const t = target(ff);
+            if (!t || !isIdShaped(t.value)) {
+              throw new Error(
+                `Internal update opportunity action "${action.name}" needs a valid ${ff} entry (an id, not a name) in '__customInputFields__'. Use get_pipelines / list_pipelines_full to find the IDs. (A missing or non-existent id makes GHL silently fail this action and can kill the rest.)`
+              );
+            }
+          }
         }
-        if (!Array.isArray(attr.__customInputFields__)) throw new Error(`Internal update opportunity action "${action.name}" missing '__customInputFields__' array.`);
         break;
       }
       case "remove_from_workflow":
@@ -1686,7 +1729,8 @@ ${errorBody}`
     }
     const currentActions = current.workflowData?.templates || [];
     const newActions = updates.actions ?? currentActions;
-    const linkedActions = this.buildActionChain(newActions);
+    const existingActionIds = new Set(currentActions.filter(hasId).map((a) => a.id));
+    const linkedActions = this.buildActionChain(newActions, existingActionIds);
     const currentIds = new Set(currentActions.filter(hasId).map((a) => a.id));
     const newIds = new Set(linkedActions.filter(hasId).map((a) => a.id));
     const createdSteps = linkedActions.filter(hasId).filter((a) => !currentIds.has(a.id)).map((a) => a.id);
@@ -1831,9 +1875,9 @@ ${errorBody}`
    * saves correctly but subsequent actions get skipped during execution.
    * Only use arrays for branching nodes (if/else) that point to multiple targets.
    */
-  buildActionChain(actions) {
-    validateActionChain(actions);
-    const linked = actions.map(normalizeRemoveFromWorkflowAction).map((action, i) => {
+  buildActionChain(actions, existingIds) {
+    validateActionChain(actions, existingIds);
+    const linked = actions.map(normalizeRemoveFromWorkflowAction).map(normalizeInternalUpdateOpportunityAction).map((action, i) => {
       const copy = { ...action };
       if (!copy.id) {
         copy.id = crypto.randomUUID();
@@ -5512,7 +5556,7 @@ function registerWorkflowBuilderTools(server2, client) {
   );
   server2.tool(
     "update_workflow_actions",
-    "Update a workflow's actions (steps), triggers, name, or status. IMPORTANT: Call get_workflow_full first to see the current state before updating. Handles version tracking automatically. Uses the internal builder API (requires Firebase auth). Action types: sms, email, add_contact_tag, remove_contact_tag, wait, webhook, internal_update_opportunity, custom_code, update_contact_field, add_notes, internal_notification, task_notification, remove_from_workflow, add_to_workflow, goto, transition, workflow_goal. NOTE: internal_update_opportunity (GHL's 'Update Opportunity' / move-to-stage action) cannot be created fresh through this API \u2014 GHL's builder rejects a synthesized node with 'action has a corrupted type' and fails the whole save; add that single step in the GHL UI, or pass through one already read via get_workflow_full (it keeps its node id). For if/else, call build_if_else_branch and include its returned nodes; if_else is a node discriminator, not a standalone action. For goal events (exit-on-condition nodes), call build_goal_event to get a correctly-shaped workflow_goal node \u2014 wire it in by setting the prior action's `next` to the goal node's id. Trigger types: all 57 native GHL trigger types have typed validation; any unknown trigger type passes through via a permissive fallback so reads never crash.",
+    "Update a workflow's actions (steps), triggers, name, or status. IMPORTANT: Call get_workflow_full first to see the current state before updating. Handles version tracking automatically. Uses the internal builder API (requires Firebase auth). Action types: sms, email, add_contact_tag, remove_contact_tag, wait, webhook, internal_update_opportunity, custom_code, update_contact_field, add_notes, internal_notification, task_notification, remove_from_workflow, add_to_workflow, goto, transition, workflow_goal. NOTE: internal_update_opportunity (GHL's 'Create/Update Opportunity' / move-to-stage action) IS creatable from scratch \u2014 this tool normalizes it to GHL's exact node shape (workflowsActionType:'INTERNAL' hoisted to the NODE level; a nested copy is what makes GHL reject the node as 'action has a corrupted type'). A synthesized node needs both a pipelineId and a pipelineStageId entry in attributes.__customInputFields__ (use get_pipelines for the IDs); a node round-tripped via get_workflow_full keeps its id and passes through unchanged. For if/else, call build_if_else_branch and include its returned nodes; if_else is a node discriminator, not a standalone action. For goal events (exit-on-condition nodes), call build_goal_event to get a correctly-shaped workflow_goal node \u2014 wire it in by setting the prior action's `next` to the goal node's id. Trigger types: all 57 native GHL trigger types have typed validation; any unknown trigger type passes through via a permissive fallback so reads never crash.",
     {
       workflowId: import_zod33.z.string().describe("The workflow ID to update."),
       name: import_zod33.z.string().optional().describe("New workflow name."),
@@ -8667,10 +8711,6 @@ ${errors.join("\n")}` : "\nNo errors!",
 // src/tools/validators.ts
 var import_zod47 = require("zod");
 var ALL_CATEGORIES = ["pipeline", "stage", "custom_field", "user", "workflow", "form", "calendar", "survey"];
-var ID_SHAPE = /^([A-Za-z0-9]{17,}|[0-9a-fA-F]{8}-[0-9a-fA-F]{4}-[0-9a-fA-F]{4}-[0-9a-fA-F]{4}-[0-9a-fA-F]{12})$/;
-function isIdShaped(s) {
-  return typeof s === "string" && ID_SHAPE.test(s);
-}
 var STANDARD_CONTACT_FIELDS = /* @__PURE__ */ new Set([
   "first_name",
   "firstname",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@elitedcs/ghl-mcp",
-  "version": "3.40.0",
+  "version": "3.41.0",
   "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",

package/templates/action-schemas.json

@@ -165,7 +165,7 @@
       "workflowsActionType": "INTERNAL",
       "type": "internal_update_opportunity"
     },
-    "notes": "CANNOT CREATE FRESH VIA THIS API: GHL's builder rejects a synthesized internal_update_opportunity node with 'action has a corrupted type' and fails the whole save. Build this step in the GHL UI, or round-trip one read via get_workflow_full (it keeps its node id). The shape below is correct for READING / round-tripping. Use pipeline and stage IDs (not names). Use get_pipelines or list_pipelines_full to find IDs FIRST. CRITICAL: If the pipelineId or stageId don't exist in the target sub-account, GHL silently fails this action AND can kill subsequent actions in the workflow. Always verify IDs exist before deploying."
+    "notes": "CREATABLE from scratch (re-enabled v3.41.0). The discriminator workflowsActionType:'INTERNAL' MUST sit at the NODE level, never nested in attributes — a nested copy makes GHL reject the node as 'action has a corrupted type' and silently fail the whole save. update_workflow_actions normalizes this for you (hoists workflowsActionType to the node level, scaffolds allowBackward + __customInputs__, gives each __customInputFields__ entry an __customInputs__). The shape below (workflowsActionType at the node level alongside type/name/attributes) is correct for both creating and round-tripping. Use pipeline and stage IDs (not names) — get_pipelines / list_pipelines_full to find them FIRST. CRITICAL: if the pipelineId or pipelineStageId don't exist in the target sub-account, GHL silently fails this action AND can kill subsequent actions. A synthesized node needs BOTH a pipelineId and a pipelineStageId entry; a node round-tripped via get_workflow_full keeps its id and passes through unchanged."
   },
 
   "_if_else_branching": {