@elitedcs/ghl-mcp 3.39.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.39.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",
@@ -10921,6 +10961,18 @@ function validateBuildPlan(input) {
       seen.add(key);
     }
   }
+  for (const fn of plan.funnels ?? []) {
+    const seen = /* @__PURE__ */ new Set();
+    for (const pg of fn.pages) {
+      const key = pg.name.trim().toLowerCase();
+      if (seen.has(key)) {
+        allErrors.push(
+          `funnel "${fn.ref}" has duplicate page name "${pg.name}" \u2014 page names must be unique within a funnel so each page ref resolves to a single step`
+        );
+      }
+      seen.add(key);
+    }
+  }
   const nameGroups = [
     ["pipelines", (plan.pipelines ?? []).map((x) => ({ ref: x.ref, name: x.name }))],
     ["customFields", (plan.customFields ?? []).map((x) => ({ ref: x.ref, name: x.name }))],
@@ -11437,6 +11489,10 @@ function renderReport(plan, result, ctx) {
     any = true;
     L.push(`  \u2022 [calendar] ${c.reason}`);
   }
+  for (const fn of plan.funnels ?? []) {
+    any = true;
+    L.push(`  \u2022 [funnel] Design + populate the pages of funnel "${fn.name}" (Blueprint builds the funnel + steps; page content/design is manual).`);
+  }
   for (const w of result.workflows) {
     for (const m of w.manual) {
       any = true;
@@ -11459,6 +11515,9 @@ function renderReport(plan, result, ctx) {
 // src/intake-to-build/execute.ts
 var norm2 = (s) => s.trim().toLowerCase();
 var sleep = (ms) => new Promise((r) => setTimeout(r, ms));
+function slugifyName(s) {
+  return s.toLowerCase().split(/[^a-z0-9]+/).filter(Boolean).join("-");
+}
 async function executeBackbone(plan, deps, opts = {}) {
   const retries = opts.verifyRetries ?? 4;
   const backoff = opts.verifyBackoffMs ?? 500;
@@ -11667,6 +11726,85 @@ async function executeBackbone(plan, deps, opts = {}) {
     idMap[form.ref] = formId;
     built.push({ ref: form.ref, type: "form", name: form.name, status: "created", realId: formId });
   }
+  const formNameByRef = new Map((plan.forms ?? []).map((f) => [f.ref, f.name]));
+  const calNameByRef = new Map((plan.calendars ?? []).map((c) => [c.ref, c.name]));
+  const funnelContentReason = (fn) => {
+    const pageBits = fn.pages.map((pg) => {
+      const hosts = [];
+      if (pg.formRef) hosts.push(`hosts form "${formNameByRef.get(pg.formRef) ?? pg.formRef}"`);
+      if (pg.calendarRef) hosts.push(`links calendar "${calNameByRef.get(pg.calendarRef) ?? pg.calendarRef}"`);
+      return `page "${pg.name}"${hosts.length ? ` (${hosts.join(", ")})` : ""}`;
+    });
+    return `Design + publish the pages of funnel "${fn.name}": ${pageBits.join("; ")}. (Blueprint built the funnel + steps; page content/design is manual.)`;
+  };
+  for (const fn of plan.funnels ?? []) {
+    let funnels;
+    try {
+      funnels = await deps.listFunnels();
+    } catch (e) {
+      return halt(fn.ref, "funnel", `could not read existing funnels: ${msg(e)}`);
+    }
+    const matches = funnels.filter((f) => norm2(f.name) === norm2(fn.name));
+    if (matches.length > 1) {
+      return halt(fn.ref, "funnel", `${matches.length} existing funnels are named "${fn.name}" \u2014 ambiguous, cannot safely bind ${fn.ref}. Resolve the duplicate in GHL, then re-run.`);
+    }
+    if (matches.length === 1) {
+      const ex = matches[0];
+      const pageBindings = [];
+      const problems = [];
+      for (const pg of fn.pages) {
+        const stepMatches = ex.steps.filter((s) => norm2(s.name) === norm2(pg.name));
+        if (stepMatches.length === 1) pageBindings.push({ ref: pg.ref, name: pg.name, id: stepMatches[0].id });
+        else if (stepMatches.length === 0) problems.push(`page "${pg.name}" has no matching step`);
+        else problems.push(`page "${pg.name}" matches ${stepMatches.length} steps`);
+      }
+      if (problems.length) {
+        return halt(fn.ref, "funnel", `a funnel named "${fn.name}" already exists but its steps don't match the plan: ${problems.join("; ")}. Align the steps in GHL or remove the funnel, then re-run. (Blueprint never modifies an existing funnel.)`);
+      }
+      idMap[fn.ref] = ex.id;
+      built.push({ ref: fn.ref, type: "funnel", name: fn.name, status: "existing", realId: ex.id });
+      for (const b of pageBindings) {
+        idMap[b.ref] = b.id;
+        built.push({ ref: b.ref, type: "page", name: b.name, status: "existing", realId: b.id });
+      }
+      manual.push({ ref: fn.ref, type: "funnel-page-content", name: fn.name, reason: funnelContentReason(fn) });
+      continue;
+    }
+    let funnelId;
+    try {
+      funnelId = await deps.createFunnel(fn.name);
+    } catch (e) {
+      return halt(fn.ref, "funnel", `create failed: ${msg(e)}`);
+    }
+    if (!funnelId) return halt(fn.ref, "funnel", "funnel create returned no id");
+    const stepBindings = [];
+    try {
+      for (const pg of fn.pages) {
+        const stepId = await deps.createFunnelStep(funnelId, pg.name, `/${slugifyName(pg.name)}`);
+        if (!stepId) throw new Error(`step "${pg.name}" returned no id`);
+        stepBindings.push({ ref: pg.ref, name: pg.name, id: stepId });
+      }
+    } catch (e) {
+      let rolledBack = false;
+      try {
+        await deps.deleteFunnel(funnelId);
+        rolledBack = true;
+      } catch {
+      }
+      return halt(
+        fn.ref,
+        "funnel",
+        rolledBack ? `step create failed; the partial funnel was rolled back (deleted): ${msg(e)}` : `step create failed AND the rollback delete also failed \u2014 manually delete the orphan funnel (id ${funnelId}) in GHL before re-running: ${msg(e)}`
+      );
+    }
+    idMap[fn.ref] = funnelId;
+    built.push({ ref: fn.ref, type: "funnel", name: fn.name, status: "created", realId: funnelId });
+    for (const sb of stepBindings) {
+      idMap[sb.ref] = sb.id;
+      built.push({ ref: sb.ref, type: "page", name: sb.name, status: "created", realId: sb.id });
+    }
+    manual.push({ ref: fn.ref, type: "funnel-page-content", name: fn.name, reason: funnelContentReason(fn) });
+  }
   return { ok: true, idMap, built, manual, deferred: deferredSections(plan) };
 }
 async function buildSimple(objects, type, list, create, nameOf, pollForNew, idMap, built) {
@@ -11703,7 +11841,6 @@ async function buildSimple(objects, type, list, create, nameOf, pollForNew, idMa
 }
 function deferredSections(plan) {
   const out = [];
-  if (plan.funnels?.length) out.push({ section: "funnels", count: plan.funnels.length });
   if (plan.workflows?.length) out.push({ section: "workflows", count: plan.workflows.length });
   return out;
 }
@@ -11840,6 +11977,38 @@ function pickPipelines(raw) {
   }
   return out;
 }
+function pickFunnels(raw) {
+  const root = raw && typeof raw === "object" ? raw : {};
+  const list = Array.isArray(root.funnels) ? root.funnels : Array.isArray(raw) ? raw : [];
+  const out = [];
+  for (const item of list) {
+    if (!item || typeof item !== "object") continue;
+    const f = item;
+    const id = typeof f._id === "string" ? f._id : typeof f.id === "string" ? f.id : void 0;
+    const name = typeof f.name === "string" ? f.name : void 0;
+    if (!id || !name) continue;
+    const stepsRaw = Array.isArray(f.steps) ? f.steps : [];
+    const steps = stepsRaw.filter((s) => !!s && typeof s === "object").map((s) => ({
+      id: typeof s._id === "string" ? s._id : typeof s.id === "string" ? s.id : "",
+      name: typeof s.name === "string" ? s.name : ""
+    })).filter((s) => s.id && s.name);
+    out.push({ id, name, steps });
+  }
+  return out;
+}
+function extractFunnelId(result) {
+  if (!result || typeof result !== "object") return void 0;
+  const r = result;
+  if (typeof r._id === "string") return r._id;
+  if (typeof r.id === "string") return r.id;
+  const funnel = r.funnel;
+  if (funnel && typeof funnel === "object") {
+    const f = funnel;
+    if (typeof f._id === "string") return f._id;
+    if (typeof f.id === "string") return f.id;
+  }
+  return void 0;
+}
 function makeExecuteDeps(client, builderClient, locationId2) {
   const pipelineApi = async (method, path7, body) => {
     const headers = await builderClient.buildHeaders();
@@ -11850,6 +12019,27 @@ function makeExecuteDeps(client, builderClient, locationId2) {
     if (!response.ok) {
       const text2 = await response.text();
       throw new Error(`Pipeline API ${response.status}: ${method} ${path7}
+${text2.slice(0, 300)}`);
+    }
+    const text = await response.text();
+    if (!text) return {};
+    try {
+      return JSON.parse(text);
+    } catch {
+      return JSON.parse(text.replace(/[\x00-\x1F\x7F]/g, ""));
+    }
+  };
+  const funnelApi = async (method, path7, body) => {
+    const headers = await builderClient.buildHeaders();
+    headers.Origin = "https://app.gohighlevel.com";
+    headers.Referer = "https://app.gohighlevel.com/";
+    const url = `https://backend.leadconnectorhq.com/funnels${path7}`;
+    const options = { method, headers };
+    if (body && (method === "POST" || method === "PUT")) options.body = JSON.stringify(body);
+    const response = await fetch(url, options);
+    if (!response.ok) {
+      const text2 = await response.text();
+      throw new Error(`Funnel API ${response.status}: ${method} ${path7}
 ${text2.slice(0, 300)}`);
     }
     const text = await response.text();
@@ -11959,6 +12149,49 @@ ${text2.slice(0, 300)}`);
         }
         throw err;
       }
+    },
+    // ── Funnels ────────────────────────────────────────────────────────────
+    // Internal funnel API. Origin/Referer are REQUIRED on writes (else 401 "Error
+    // calling IAM service") — mirrors funnel-builder.ts.
+    // Paginate so never-clobber sees ALL funnels (a single page would miss a
+    // same-named funnel beyond it → duplicate). Dedup by id and stop when a page
+    // adds nothing new — robust whether or not the endpoint honors `offset`.
+    listFunnels: async () => {
+      const byId = /* @__PURE__ */ new Map();
+      const pageSize = 100;
+      const maxFunnels = 5e3;
+      for (let offset = 0; offset < maxFunnels; offset += pageSize) {
+        const page = pickFunnels(await funnelApi("GET", `/funnel/list?locationId=${locationId2}&limit=${pageSize}&offset=${offset}&getStats=true`));
+        let added = 0;
+        for (const f of page) if (!byId.has(f.id)) {
+          byId.set(f.id, f);
+          added++;
+        }
+        if (page.length < pageSize || added === 0) break;
+      }
+      return [...byId.values()];
+    },
+    createFunnel: async (name) => {
+      const result = await funnelApi("POST", `/funnel/create?locationId=${locationId2}`, {
+        name,
+        locationId: locationId2,
+        type: "funnel",
+        steps: []
+      });
+      const id = extractFunnelId(result);
+      if (!id) throw new Error(`create_funnel returned no id: ${JSON.stringify(result).slice(0, 200)}`);
+      return id;
+    },
+    createFunnelStep: async (funnelId, name, url) => {
+      const stepId = crypto.randomUUID();
+      await funnelApi("POST", `/funnel/create-step`, {
+        funnelId,
+        step: { id: stepId, name, url, pages: [], control_traffic: 100, split: false, type: "optin_funnel_page" }
+      });
+      return stepId;
+    },
+    deleteFunnel: async (funnelId) => {
+      await funnelApi("POST", `/funnel/delete`, { funnelId, locationId: locationId2, userId: builderClient.getUserId() });
     }
   };
 }
@@ -12017,7 +12250,7 @@ function registerIntakeToBuildTools(server2, client, builderClient) {
   );
   server2.tool(
     "apply_build_plan",
-    `Take an APPROVED Blueprint \xA75 build plan + the CURRENT confirmed sub-account and build it. mode:"dry_run" (default) writes NOTHING \u2014 it resolves refs, expands each workflow's logical actions to native GHL JSON, runs the NEVER-CLOBBER existing-asset scan, and returns a two-part report. Run it FIRST. mode:"execute" performs LIVE writes for the CRM backbone (pipelines+stages, custom fields, tags, custom values), calendars, AND forms: never clobbers (same-named objects are bound to their existing id, never modified), verifies each create by read-back before resolving its ref, halts on the first failure returning the partial idMap, and is idempotent (re-run = no-op). Staff-requiring calendars (round_robin etc.) auto-build only when you are the sole account user (auto-assigns you as the team member); with 0 or 2+ users they're surfaced as a manual step, not auto-staffed to a guess. Forms build with their standard + custom fields (custom fieldRefs resolve to the real fields created earlier in the run). Funnels/workflows are surfaced as manual next steps, not auto-built yet. Always confirms the active location and validates the plan before any write.`,
+    `Take an APPROVED Blueprint \xA75 build plan + the CURRENT confirmed sub-account and build it. mode:"dry_run" (default) writes NOTHING \u2014 it resolves refs, expands each workflow's logical actions to native GHL JSON, runs the NEVER-CLOBBER existing-asset scan, and returns a two-part report. Run it FIRST. mode:"execute" performs LIVE writes for the CRM backbone (pipelines+stages, custom fields, tags, custom values), calendars, AND forms: never clobbers (same-named objects are bound to their existing id, never modified), verifies each create by read-back before resolving its ref, halts on the first failure returning the partial idMap, and is idempotent (re-run = no-op). Staff-requiring calendars (round_robin etc.) auto-build only when you are the sole account user (auto-assigns you as the team member); with 0 or 2+ users they're surfaced as a manual step, not auto-staffed to a guess. Forms build with their standard + custom fields (custom fieldRefs resolve to the real fields created earlier in the run). Funnels build structurally (funnel + named steps), with each funnel emitting a manual 'design + populate the pages' step (page content/HTML is not auto-built \u2014 plans carry outlines). Workflows are surfaced as manual next steps, not auto-built yet. Always confirms the active location and validates the plan before any write.`,
     {
       plan: import_zod53.z.record(import_zod53.z.unknown()).describe("The approved \xA75 Build Plan object."),
       mode: import_zod53.z.enum(["dry_run", "execute"]).optional().describe("dry_run (default) = resolve/expand/scan/report, no writes. execute = live writes (not yet enabled)."),
@@ -12092,7 +12325,7 @@ function registerIntakeToBuildTools(server2, client, builderClient) {
           }
           const deps = makeExecuteDeps(client, builderClient, activeLocation);
           const exec = await executeBackbone(typedPlan, deps);
-          const calendarManualLines = exec.manual.map((m) => `[calendar] ${m.reason}`);
+          const execManualLines = exec.manual.map((m) => `[${m.type}] ${m.reason}`);
           const manualLines = result.workflows.flatMap((w) => [
             ...w.manual.map((m) => `[${w.name}] ${m.reason}`),
             ...w.needsContent.map((c) => `[${w.name}] ${c.reason}`)
@@ -12110,9 +12343,9 @@ function registerIntakeToBuildTools(server2, client, builderClient) {
             manual: exec.manual,
             idMap: exec.idMap,
             deferred: exec.deferred,
-            deferredNote: "execute builds the CRM backbone (pipelines, custom fields, tags, custom values), calendars, and forms live. Staff-requiring calendars (round_robin etc.) auto-build only when you are the sole user; otherwise they're listed under manual steps. Funnels/workflows are planned but NOT auto-built yet \u2014 create them via the GHL UI or the dedicated tools, in this order: funnels \u2192 workflows.",
-            nextManualSteps: [...calendarManualLines, ...manualLines, ...handoffLines],
-            summary: exec.ok ? `Built ${exec.built.filter((b) => b.status === "created").length} new object(s), bound ${exec.built.filter((b) => b.status === "existing").length} existing.${exec.manual.length ? ` ${exec.manual.length} calendar(s) need manual staff assignment.` : ""}${exec.deferred.length ? " Deferred: " + exec.deferred.map((d) => `${d.count} ${d.section}`).join(", ") + " (manual)." : ""}` : `HALTED at ${exec.halted?.atRef} (${exec.halted?.reason}). ${exec.built.length} object(s) were created before the halt \u2014 see idMap to resume or clean up. NEVER-CLOBBER means a re-run will bind those, not duplicate them.`
+            deferredNote: "execute builds the CRM backbone (pipelines, custom fields, tags, custom values), calendars, forms, and funnels (funnel + named steps) live. Staff-requiring calendars (round_robin etc.) auto-build only when you are the sole user, else manual. Funnel PAGE CONTENT/design is not auto-built (plans carry outlines, not HTML) \u2014 each funnel lists a manual 'design + populate the pages' step. Workflows are planned but NOT auto-built yet.",
+            nextManualSteps: [...execManualLines, ...manualLines, ...handoffLines],
+            summary: exec.ok ? `Built ${exec.built.filter((b) => b.status === "created").length} new object(s), bound ${exec.built.filter((b) => b.status === "existing").length} existing.${exec.manual.length ? ` ${exec.manual.length} item(s) need a manual step (see nextManualSteps).` : ""}${exec.deferred.length ? " Deferred: " + exec.deferred.map((d) => `${d.count} ${d.section}`).join(", ") + " (manual)." : ""}` : `HALTED at ${exec.halted?.atRef} (${exec.halted?.reason}). ${exec.built.length} object(s) were created before the halt \u2014 see idMap to resume or clean up. NEVER-CLOBBER means a re-run will bind those, not duplicate them.`
           });
         }
         const collisions = result.items.filter((i) => i.status === "existing");
@@ -12150,7 +12383,7 @@ function registerIntakeToBuildTools(server2, client, builderClient) {
           calendarsManual: result.calendarsManual,
           handoffs: result.handoffs,
           report,
-          next: 'Review the report. When it looks right, re-run with mode:"execute" to build the CRM backbone + calendars live (pipelines, fields, tags, custom values, calendars). Forms/funnels/workflows are listed as manual next steps.'
+          next: 'Review the report. When it looks right, re-run with mode:"execute" to build live: the CRM backbone (pipelines, fields, tags, custom values), calendars, forms, and funnels (funnel + named steps). Funnel page content/design and workflows are listed as manual next steps.'
         });
       } catch (error) {
         return errorResponse(error);

package/package.json

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