@elitedcs/ghl-mcp 3.78.0 → 3.78.1

package/CHANGELOG.md

@@ -1,5 +1,37 @@
 # Changelog
 
+## 3.78.1 — the template installer stops guessing which funnel it just made
+
+Yesterday's release shipped `install_ghl_template` with the originals kept by default,
+because the tool worked out which funnel it had created by comparing the account before
+and after, and on a busy sub-account that comparison cannot be certain. Keeping the
+originals was the safe answer to an open question.
+
+The question now has an answer. GoHighLevel's install response names what it created: the
+funnel id and its first page. The tool reads that id and waits for that funnel, so it
+cannot land on one somebody else happened to be building at the same moment.
+
+**It does not simply believe it.** The name only counts when your account agrees the
+funnel is genuinely new, and the tool checks the response against a complete picture of
+the account taken before the install. Two independent confirmations, because that one
+field is what authorises removing anything. If the name and the account ever disagree, the
+name is thrown away, nothing is removed, and the result says so.
+
+With both confirmations in hand the originals go back to being removed once every branded
+page has been read back and measured, which is how the tool was meant to work: one clean
+funnel in your client's colours, not a doubled one you have to tidy. Pass
+`keepOriginals: true` if you would rather see both.
+
+Anything less and the originals stay, whatever you passed: no name in the response, a name
+the account cannot corroborate, or an account with so many funnels that the before picture
+was incomplete.
+
+A split-test step, where one step holds more than one page, still stops any removal. Only
+the first page of such a step is rebuilt, so removing the step would take the variants
+with it.
+
+Counts unchanged: 250 tools, 51 modules, free tier 112.
+
 ## 3.78.0 — GoHighLevel's own template library, repainted in your client's colours
 
 GoHighLevel ships more than 900 designed funnel templates. You could always install

package/README.md

@@ -897,7 +897,7 @@ Built by **[Elite DCs, LLC](https://elitedcs.com)** — a digital marketing and
 
 **Tech stack:** TypeScript, Node.js, esbuild, MCP SDK, Zod, GHL API v2, Firebase Auth
 
-**Version:** 3.78.0
+**Version:** 3.78.1
 
 ---
 

package/dist/index.js

@@ -10713,6 +10713,15 @@ async function templateFacets(api, product = "funnels") {
   const names = (list2) => (list2 ?? []).map((x) => typeof x === "string" ? x : x.name ?? "").filter(Boolean);
   return { categories: names(out.categories), tags: names(out.tags), features: names(out.features) };
 }
+function installedFunnelId(loaded) {
+  const r = loaded;
+  if (!r || typeof r !== "object") return void 0;
+  if (r.status !== "ok") return void 0;
+  if (r.data?.status !== "completed") return void 0;
+  const t = r.data?.target;
+  if (!t || t.asset !== "funnels") return void 0;
+  return typeof t.assetId === "string" && GHL_ID.test(t.assetId) ? t.assetId : void 0;
+}
 async function installTemplate(api, templateId) {
   if (!/^[A-Za-z0-9]{12,40}$/.test(templateId)) throw new Error(`Not a template id: ${JSON.stringify(templateId)}`);
   return call(api, "/templates/template/load", {
@@ -10737,12 +10746,13 @@ function rankForBrand(rows, brand) {
     return { ...t, why: parts.join("; ") };
   });
 }
-var SVC;
+var SVC, GHL_ID;
 var init_ghl_templates = __esm({
   "src/command-os/design/ghl-templates.ts"() {
     "use strict";
     init_color();
     SVC = "https://services.leadconnectorhq.com";
+    GHL_ID = /^[A-Za-z0-9]{15,40}$/;
   }
 });
 
@@ -13550,7 +13560,7 @@ function installedFunnelTracker(before) {
   };
 }
 function removalHoldBack(opts) {
-  if (opts.truncated) return `this sub-account has at least ${opts.funnelPageSize} funnels, so "which funnel is new" could not be established with certainty`;
+  if (opts.truncated && !opts.identifiedFromResponse) return `this sub-account's funnels could not all be read back before the install, so "which funnel is new" could not be established with certainty`;
   if (opts.multiPageStepNames.length > 0) return `${opts.multiPageStepNames.length} step(s) carry more than one page (${opts.multiPageStepNames.join(", ")}) and only the first of each was rebuilt`;
   return null;
 }
@@ -13563,12 +13573,23 @@ function registerPageStudioTools(server2, builderClient) {
     locationId: () => client.locationId
   });
   const FUNNEL_PAGE = 100;
+  const FUNNEL_PAGE_CAP = 20;
   async function funnelSummary() {
     const out = /* @__PURE__ */ new Map();
-    const r = await funnelRequest("GET", `/funnel/list?locationId=${client.locationId}&limit=${FUNNEL_PAGE}`);
-    const rows = r.funnels ?? [];
-    for (const f of rows) out.set(f._id, `${f.name} (${(f.steps ?? []).length} steps)`);
-    return { map: out, truncated: rows.length >= FUNNEL_PAGE };
+    let truncated = false;
+    for (let page = 0; page < FUNNEL_PAGE_CAP; page++) {
+      const r = await funnelRequest("GET", `/funnel/list?locationId=${client.locationId}&limit=${FUNNEL_PAGE}&offset=${page * FUNNEL_PAGE}`);
+      const rows = r.funnels ?? [];
+      const fresh = rows.filter((f) => !out.has(f._id));
+      if (page > 0 && fresh.length === 0 && rows.length > 0) {
+        truncated = true;
+        break;
+      }
+      for (const f of rows) out.set(f._id, `${f.name} (${(f.steps ?? []).length} steps)`);
+      if (rows.length < FUNNEL_PAGE) return { map: out, truncated: false };
+      if (page === FUNNEL_PAGE_CAP - 1) truncated = true;
+    }
+    return { map: out, truncated };
   }
   async function funnelRequest(method, path36, body) {
     const headers = await client.buildHeaders();
@@ -13723,23 +13744,34 @@ ${await response.text()}`);
       templateId: import_zod42.z.string().describe("From find_ghl_template."),
       brandSlug: import_zod42.z.string().optional().describe("A stored brand to repaint the template in. Omit to install it in the designer's own colours."),
       confirm: import_zod42.z.literal("INSTALL").describe("Type INSTALL. This writes into the live sub-account you are switched to."),
-      keepOriginals: import_zod42.z.boolean().optional().describe("With brandSlug: leave the template's original unbranded steps beside the branded ones. DEFAULT TRUE \u2014 nothing is deleted unless you pass false. Deleting means trusting that the funnel this installed into is the one the template made, and on a busy sub-account that cannot be established with certainty (see the tool's note).")
+      keepOriginals: import_zod42.z.boolean().optional().describe("With brandSlug: leave the template's original unbranded steps beside the branded ones. Normally they are removed once every branded page has been read back and measured. They are KEPT automatically, whatever you pass, if GoHighLevel did not name the funnel it created in its install response and the identification had to fall back to comparing the account before and after.")
     },
     async ({ templateId, brandSlug, keepOriginals }) => {
       const brand = brandSlug ? houseLooks().find((x) => x.slug === brandSlug) ?? readBrand(slugify(brandSlug)) : null;
       const inventory = await funnelSummary();
       const before = inventory.map;
-      await installTemplate(templateApi(), templateId);
+      const named = installedFunnelId(await installTemplate(templateApi(), templateId));
+      const namedFunnelId = named && !before.has(named) ? named : void 0;
+      const namedButNotNew = Boolean(named) && named !== namedFunnelId;
       let funnel;
       let ambiguous = [];
       const tracker = installedFunnelTracker(before);
       for (let tries = 0; tries < 8; tries++) {
         const list2 = await funnelRequest("GET", `/funnel/list?locationId=${client.locationId}&limit=${FUNNEL_PAGE}`);
-        const seen = tracker.observe(list2.funnels ?? []);
-        if (seen.settled) {
-          if ("funnel" in seen) funnel = seen.funnel;
-          else ambiguous = seen.ambiguous;
-          break;
+        const rows = list2.funnels ?? [];
+        if (namedFunnelId) {
+          const mine = rows.find((f) => f._id === namedFunnelId);
+          if (mine && (mine.steps ?? []).length > 0) {
+            funnel = mine;
+            break;
+          }
+        } else {
+          const seen = tracker.observe(rows);
+          if (seen.settled) {
+            if ("funnel" in seen) funnel = seen.funnel;
+            else ambiguous = seen.ambiguous;
+            break;
+          }
         }
         await new Promise((r) => setTimeout(r, 1500));
       }
@@ -13839,9 +13871,10 @@ ${await response.text()}`);
         }
       }
       let removed = 0;
-      const removeOriginals = keepOriginals === false;
+      const certain = Boolean(namedFunnelId) && !inventory.truncated;
+      const removeOriginals = certain ? keepOriginals !== true : keepOriginals === false;
       const complete = problems.length === 0 && done.length === originals.length && done.length > 0;
-      const holdBack = removalHoldBack({ truncated: inventory.truncated, funnelPageSize: FUNNEL_PAGE, multiPageStepNames: multiPage.map((m) => m.name) });
+      const holdBack = removalHoldBack({ truncated: inventory.truncated, funnelPageSize: FUNNEL_PAGE, multiPageStepNames: multiPage.map((m) => m.name), identifiedFromResponse: certain });
       if (complete && removeOriginals && holdBack === null) {
         for (const d of done) {
           try {
@@ -13860,6 +13893,7 @@ ${await response.text()}`);
         pages: done,
         originalsRemoved: complete && removeOriginals ? removed : 0,
         ...problems.length > 0 && { problems, kept: "Every original step was LEFT IN PLACE because something failed. Nothing was deleted." },
+        ...namedButNotNew && { identification: "GoHighLevel named a funnel that this account ALREADY had before the install. That name was discarded and the account was compared instead, so the funnel above was identified the cautious way and originals are only removed if you asked for it explicitly. Worth reporting: it means the install response no longer means what it did." },
         ...holdBack !== null && { kept: `Every original step was LEFT IN PLACE on purpose: ${holdBack}. The branded pages are there beside them \u2014 remove the originals by hand once you have looked.` },
         warnings: rethemeEnvelope({}, brand, palette).warnings,
         next: done.length ? `Grade one before the client sees it: verify_design on ${String(done[0].preview)} with medium "site" and placement "ghl". GoHighLevel's own templates are designed but frequently fail contrast and tap-target rules.` : "Nothing was branded.",
@@ -38775,7 +38809,7 @@ var require_package = __commonJS({
   "package.json"(exports2, module2) {
     module2.exports = {
       name: "@elitedcs/ghl-mcp",
-      version: "3.78.0",
+      version: "3.78.1",
       mcpName: "io.github.drjerryrelth/ghl-command",
       description: "GoHighLevel MCP Server for Claude. 250 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.78.0",
+  "version": "3.78.1",
   "mcpName": "io.github.drjerryrelth/ghl-command",
   "description": "GoHighLevel MCP Server for Claude. 250 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",