@elitedcs/ghl-mcp 3.72.2 → 3.72.3

package/CHANGELOG.md

@@ -1,5 +1,29 @@
 # Changelog
 
+## 3.72.3 — the waits in a built account can actually go live
+
+Blueprint builds a client's account and then asks whether to publish the workflows or
+leave them as drafts for review. Say publish, and the ones with a wait of a day or more
+were refused by GoHighLevel and stayed as drafts. The build still reported them as
+created, so the account looked finished when the follow-up sequences were not running.
+
+The cause was one word. GoHighLevel takes `minutes` and `hour`, but for days it wants
+`days`, and we were sending `day`. It is not a rule you can guess: a workflow that is
+nothing but a wait accepts the singular quite happily, which is why this went unnoticed.
+It only fails on the shape a real nurture has, a message followed by a wait, which is
+every follow-up sequence the presets build. **123 of the waits in the shipped presets
+were affected.**
+
+Workflows already in an account are untouched and keep working; this was only ever about
+newly built ones.
+
+**The second half is the reporting.** A workflow that could not be published was still
+counted as built, and the summary read the same as a run where nobody asked to publish at
+all: it suggested turning on publishing, to someone who had just turned it on. The real
+reason sat further down the report where nobody looks first. A build now says plainly
+which workflows could not be published, what GoHighLevel said about each, that they are
+sitting as drafts and not running, and that repeating the build will not help.
+
 ## 3.72.2 — a form you read back can be saved again
 
 A subscriber tried to edit one of her forms the documented way: read it with

package/dist/index.js

@@ -18523,7 +18523,7 @@ var init_executor = __esm({
     WAIT_UNIT_MAP = {
       minutes: "minutes",
       hours: "hour",
-      days: "day"
+      days: "days"
     };
     PENDING = (ref) => `__PENDING__:${ref}`;
     isPending = (v) => v.startsWith("__PENDING__:");
@@ -21493,6 +21493,36 @@ var init_plan_form = __esm({
   }
 });
 
+// src/intake-to-build/publish-report.ts
+function publishNote(r) {
+  if (r.publishFailed.length) {
+    const lead = r.published.length ? `Auto-published ${r.published.length} workflow(s) live, but ` : "";
+    const detail = r.publishFailed.map((p) => `${p.name} (${p.error})`).join("; ");
+    return `${lead}${r.publishFailed.length} workflow(s) were BUILT BUT COULD NOT BE PUBLISHED. They exist in GHL as DRAFT and are NOT running: ${detail}. You already opted in, so re-running with publishWorkflows will not help. If GHL rejected the workflow's contents, publishing by hand will hit the SAME error until the cause is fixed. Read the error above, fix it, then publish (see nextManualSteps).${r.published.length ? " Any gated workflow stays DRAFT until its handoff is met." : ""}`;
+  }
+  if (r.published.length) {
+    return `Auto-published ${r.published.length} workflow(s) live (you opted in). Any gated workflow stays DRAFT until its handoff is met.`;
+  }
+  if (r.anyWorkflowCreated) {
+    return "Workflows were built as DRAFT (the safe default). Review them in GHL and publish, or re-run with publishWorkflows:true to auto-publish ungated ones.";
+  }
+  return "No workflows were built in this run (none in the plan, or all already existed and were left untouched).";
+}
+function publishSummaryClause(r) {
+  if (r.publishFailed.length) {
+    const also = r.published.length ? ` (${r.published.length} did publish)` : "";
+    return ` ${r.publishFailed.length} workflow(s) BUILT BUT NOT PUBLISHED${also}. They are DRAFT and not running, see publishNote.`;
+  }
+  if (r.published.length) return ` Published ${r.published.length} workflow(s) live.`;
+  if (r.anyWorkflowCreated) return " Workflows are DRAFT (opt in with publishWorkflows to auto-publish).";
+  return "";
+}
+var init_publish_report = __esm({
+  "src/intake-to-build/publish-report.ts"() {
+    "use strict";
+  }
+});
+
 // src/intake-to-build/customization.ts
 function industryPack(slug3) {
   if (!slug3) return void 0;
@@ -23424,6 +23454,7 @@ async function executeBackbone(plan, deps, opts = {}) {
   const built = [];
   const manual = [];
   const published = [];
+  const publishFailed = [];
   const staff = [];
   const templates = { emails: [], sms: [] };
   let templatesUnverified = 0;
@@ -23447,6 +23478,7 @@ async function executeBackbone(plan, deps, opts = {}) {
     built,
     manual,
     published,
+    publishFailed,
     halted: { atRef, type, reason },
     deferred,
     staff,
@@ -24124,6 +24156,7 @@ async function executeBackbone(plan, deps, opts = {}) {
         await deps.publishWorkflow(wfId);
         published.push(wf.ref);
       } catch (e) {
+        publishFailed.push({ ref: wf.ref, name: wf.name, error: msg2(e) });
         manual.push({ ref: wf.ref, type: "workflow-publish", name: wf.name, reason: `[${wf.name}] built successfully but auto-publish failed (${msg2(e)}) \u2014 publish it manually in GHL.` });
       }
     }
@@ -24178,7 +24211,7 @@ async function executeBackbone(plan, deps, opts = {}) {
       }
     }
   }
-  return { ok: true, idMap, built, manual, published, deferred, staff, templates, templatesUnverified, templatesFormatNote, calendarPending, refreshed, missingSteps, recipients };
+  return { ok: true, idMap, built, manual, published, publishFailed, deferred, staff, templates, templatesUnverified, templatesFormatNote, calendarPending, refreshed, missingSteps, recipients };
 }
 async function buildSimple(objects, type, list2, create, nameOf, pollForNew, idMap, built) {
   if (objects.length === 0) return null;
@@ -25190,13 +25223,18 @@ function registerIntakeToBuildTools(server2, client, builderClient, registry2) {
             templatesFormatNote: exec.templatesFormatNote,
             templatesNote: exec.templates.emails.length + exec.templates.sms.length ? `The plan's email and SMS copy was also saved as account-level templates (Marketing \u2192 Emails \u2192 Templates; Conversations \u2192 Templates) \u2014 the reusable version your team edits. Templates are created in the editor's own format (vibe-editor) so they open and edit in Marketing \u2192 Emails \u2192 Templates. Workflow steps carry the same copy inline; editing a template does not change a step already built.${exec.templatesUnverified ? ` ${exec.templatesUnverified} email template(s) were saved but their body was NOT seen in GHL's preview \u2014 see templates.emails[].warning; do not report them as written.` : exec.templates.emails.some((t) => t.status === "created") ? ` Every email template created this run had its body verified in GHL's preview.` : ""}` : void 0,
             published: exec.published,
-            publishNote: exec.published.length ? `Auto-published ${exec.published.length} workflow(s) live (you opted in). Any gated workflow stays DRAFT until its handoff is met.` : exec.built.some((b) => b.type === "workflow" && b.status === "created") ? "Workflows were built as DRAFT (the safe default). Review them in GHL and publish, or re-run with publishWorkflows:true to auto-publish ungated ones." : "No workflows were built in this run (none in the plan, or all already existed and were left untouched).",
+            publishFailed: exec.publishFailed,
+            publishNote: publishNote({
+              published: exec.published,
+              publishFailed: exec.publishFailed,
+              anyWorkflowCreated: exec.built.some((b) => b.type === "workflow" && b.status === "created")
+            }),
             externalWiring,
             externalWiringNote: externalWiring ? "For each target:\"external\" funnel: plug these into your self-hosted site form + lead bridge (templates/external-funnel/). custom formFields carry the VERIFIED GHL field id your form's `custom` object must send (never name-guess keys); add a triggerTag to start the speed-to-lead workflow; bookingUrl is the GHL calendar widget for the CTA. Anything in `unresolved` wasn't built yet \u2014 re-run after it is." : void 0,
             deferred: exec.deferred,
             deferredNote: `execute builds the WHOLE plan live: staff users (plan.users, via the agency key \u2014 bound by email when they already exist), CRM backbone (pipelines, custom fields, tags, custom values), calendars (with the plan's named team members), forms, funnels (GHL-built = funnel + named steps), account-level email/SMS templates (the plan's reusable copy), and workflows (DRAFT, with all steps incl. opportunity create/move; notification / task steps are NEVER dropped \u2014 with no staff member yet they ship as placeholders flagged staff_pending). Remaining manual steps are surfaced per item: pending staff (no agency key), staff-requiring calendars whose team members are pending (workflows that point at such a calendar are STILL built \u2014 trigger unpinned, flagged calendar_pending \u2014 never a halt), GHL funnel page content/design, EXTERNAL funnels (target:"external" \u2014 the user builds + hosts the site themselves and wires it back; not built here), workflow triggers other than contact_tag, and publishing the DRAFT workflows.`,
             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" || b.status === "refreshed").length} existing${exec.refreshed.length ? ` (${exec.refreshed.length} workflow(s) refreshed in place \u2014 see refreshed[])` : ""}.${exec.staff.length ? ` Staff: ${exec.staff.filter((s) => s.status === "created").length} created, ${exec.staff.filter((s) => s.status === "existing").length} already in the account, ${exec.staff.filter((s) => s.status === "pending").length} pending.` : ""}${exec.templates.emails.length + exec.templates.sms.length ? ` Templates: ${exec.templates.emails.length} email (${exec.templates.emails.filter((t) => t.status === "created" && !t.warning).length} created with the body verified in the preview${exec.templatesUnverified ? `, ${exec.templatesUnverified} created but NOT verified \u2014 see templates.emails[].warning` : ""}${exec.templates.emails.filter((t) => t.status === "existing").length ? `, ${exec.templates.emails.filter((t) => t.status === "existing").length} already existed` : ""}${exec.templates.emails.filter((t) => t.status === "failed").length ? `, ${exec.templates.emails.filter((t) => t.status === "failed").length} failed` : ""}), ${exec.templates.sms.length} SMS.` : ""}${exec.published.length ? ` Published ${exec.published.length} workflow(s) live.` : exec.built.some((b) => b.type === "workflow" && b.status === "created") ? " Workflows are DRAFT (opt in with publishWorkflows to auto-publish)." : ""}${exec.manual.some((m) => m.type === "calendar") || exec.calendarPending.length ? ` Calendars: ${exec.manual.filter((m) => m.type === "calendar").length} deferred to manual; ${new Set(exec.calendarPending.filter((c) => c.status === "pending").map((c) => c.workflowRef)).size} workflow(s) built with the calendar unpinned${exec.calendarPending.some((c) => c.status === "calendar_ready") ? `, ${new Set(exec.calendarPending.filter((c) => c.status === "calendar_ready").map((c) => c.workflowRef)).size} ready to pin` : ""}.` : ""}${exec.manual.some((m) => m.type === "calendar_team") ? ` ${exec.manual.filter((m) => m.type === "calendar_team").length} calendar(s) were built WITHOUT the people the plan named \u2014 see nextManualSteps; do not report those calendars as staffed.` : ""}${exec.missingSteps.length ? ` NOT COMPLETE: ${exec.missingSteps.reduce((n, w) => n + w.steps.length, 0)} step(s) the plan asks for are still missing from ${exec.missingSteps.length} workflow(s) that already existed \u2014 add them by hand (see missingSteps).` : ""}${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.`
+            summary: exec.ok ? `Built ${exec.built.filter((b) => b.status === "created").length} new object(s), bound ${exec.built.filter((b) => b.status === "existing" || b.status === "refreshed").length} existing${exec.refreshed.length ? ` (${exec.refreshed.length} workflow(s) refreshed in place \u2014 see refreshed[])` : ""}.${exec.staff.length ? ` Staff: ${exec.staff.filter((s) => s.status === "created").length} created, ${exec.staff.filter((s) => s.status === "existing").length} already in the account, ${exec.staff.filter((s) => s.status === "pending").length} pending.` : ""}${exec.templates.emails.length + exec.templates.sms.length ? ` Templates: ${exec.templates.emails.length} email (${exec.templates.emails.filter((t) => t.status === "created" && !t.warning).length} created with the body verified in the preview${exec.templatesUnverified ? `, ${exec.templatesUnverified} created but NOT verified \u2014 see templates.emails[].warning` : ""}${exec.templates.emails.filter((t) => t.status === "existing").length ? `, ${exec.templates.emails.filter((t) => t.status === "existing").length} already existed` : ""}${exec.templates.emails.filter((t) => t.status === "failed").length ? `, ${exec.templates.emails.filter((t) => t.status === "failed").length} failed` : ""}), ${exec.templates.sms.length} SMS.` : ""}${publishSummaryClause({ published: exec.published, publishFailed: exec.publishFailed, anyWorkflowCreated: exec.built.some((b) => b.type === "workflow" && b.status === "created") })}${exec.manual.some((m) => m.type === "calendar") || exec.calendarPending.length ? ` Calendars: ${exec.manual.filter((m) => m.type === "calendar").length} deferred to manual; ${new Set(exec.calendarPending.filter((c) => c.status === "pending").map((c) => c.workflowRef)).size} workflow(s) built with the calendar unpinned${exec.calendarPending.some((c) => c.status === "calendar_ready") ? `, ${new Set(exec.calendarPending.filter((c) => c.status === "calendar_ready").map((c) => c.workflowRef)).size} ready to pin` : ""}.` : ""}${exec.manual.some((m) => m.type === "calendar_team") ? ` ${exec.manual.filter((m) => m.type === "calendar_team").length} calendar(s) were built WITHOUT the people the plan named \u2014 see nextManualSteps; do not report those calendars as staffed.` : ""}${exec.missingSteps.length ? ` NOT COMPLETE: ${exec.missingSteps.reduce((n, w) => n + w.steps.length, 0)} step(s) the plan asks for are still missing from ${exec.missingSteps.length} workflow(s) that already existed \u2014 add them by hand (see missingSteps).` : ""}${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");
@@ -25616,6 +25654,7 @@ var init_intake_to_build = __esm({
     init_emails();
     init_user_provisioning();
     init_plan_form();
+    init_publish_report();
     init_question_set();
     init_customization();
     init_intake_overlay();
@@ -26294,7 +26333,7 @@ var require_package = __commonJS({
   "package.json"(exports2, module2) {
     module2.exports = {
       name: "@elitedcs/ghl-mcp",
-      version: "3.72.2",
+      version: "3.72.3",
       mcpName: "io.github.drjerryrelth/ghl-command",
       description: "GoHighLevel MCP Server for Claude. 247 tools \u2014 full CRM, automation, marketing control, account-wide workflow audit, live funnel-capture verification, and the only programmatic GHL workflow builder, now multi-tenant across client accounts.",
       main: "dist/index.js",

package/package.json

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

package/templates/action-schemas.json

@@ -82,7 +82,7 @@
     "example_3days": {
       "type": "time",
       "startAfter": {
-        "type": "day",
+        "type": "days",
         "value": 3,
         "when": "after"
       },
@@ -92,7 +92,7 @@
       "convertToMultipath": false,
       "transitions": []
     },
-    "notes": "startAfter.type: 'minutes', 'hour', or 'day'. ALL 5 fields required (type, startAfter, isHybridAction, hybridActionType, transitions). Derived from working GHL UI-built workflows."
+    "notes": "startAfter.type: 'minutes', 'hour', or 'days' — NOT consistently singular, and 'day' is rejected with 400 INVALID_FIELD_VALUE on a workflow that has a real action before the wait (live-verified 2026-08-29). ALL 5 fields required (type, startAfter, isHybridAction, hybridActionType, transitions). Derived from working GHL UI-built workflows."
   },
   "internal_notification": {
     "example": {

package/templates/clinic-medspa.json

@@ -243,7 +243,7 @@
       "actions": [
         { "name": "Add Reactivation Tag", "type": "add_contact_tag", "attributes": { "tags": ["reactivation-candidate"] } },
         { "name": "Send Reactivation SMS", "type": "sms", "attributes": { "body": "Hi {{contact.first_name}}, it's been a while since your last visit to {{business_name}}. We miss you! Book your next session and get 10% off.", "attachments": [] } },
-        { "name": "Wait 3 Days", "type": "wait", "attributes": { "type": "time", "startAfter": { "type": "day", "value": 3, "when": "after" }, "name": "Wait", "isHybridAction": true, "hybridActionType": "wait", "convertToMultipath": false, "transitions": [] } },
+        { "name": "Wait 3 Days", "type": "wait", "attributes": { "type": "time", "startAfter": { "type": "days", "value": 3, "when": "after" }, "name": "Wait", "isHybridAction": true, "hybridActionType": "wait", "convertToMultipath": false, "transitions": [] } },
         { "name": "Send Reactivation Email", "type": "email", "attributes": { "subject": "We miss you, {{contact.first_name}}!", "html": "<p style=\"margin:0px;\">Hi {{contact.first_name}},</p><p style=\"margin:0px;\"><br></p><p style=\"margin:0px;\">It's been over 90 days since your last visit to {{business_name}}, and we wanted to check in.</p><p style=\"margin:0px;\"><br></p><p style=\"margin:0px;\">As a valued patient, we'd like to offer you 10% off your next treatment. Just mention this email when you book.</p><p style=\"margin:0px;\"><br></p><p style=\"margin:0px;\">See you soon!</p><p style=\"margin:0px;\">{{owner_name}}</p>", "trackingOptions": { "hasTrackingLinks": false, "hasUtmTracking": false, "hasTags": false } } }
       ]
     }