@elitedcs/ghl-mcp 3.84.1 → 3.84.2

package/CHANGELOG.md

@@ -2,6 +2,52 @@
 
 ## Unreleased
 
+## 3.84.2 — done means read back, not echoed
+
+Every tool that reports a thing as done, verified, ready or published was checked against a second,
+independent read of what GoHighLevel actually holds. Thirteen places were claiming an outcome from
+the write's own response, a counter, or the wrong field. Each now derives the claim from a read taken
+after the write, or says plainly what could not be confirmed.
+
+**A build report no longer says "Published N workflows live" for a workflow GoHighLevel kept as a
+draft.** The publish step already read the workflow back and said NOT LIVE when it was not; the
+build threw that answer away and counted it anyway. It is counted now, and a workflow that published
+with every trigger still inactive is named as such rather than called running. The publish tool's
+step and trigger counts come from the read after the write, with the number of active triggers
+beside them.
+
+**The workflow audit no longer says "ok" while dropping four kinds of warning.** Steps GoHighLevel
+will not accept on save, waits in the unit it no longer takes, an alert set to notify a specific
+person with nobody chosen, and steps needing a social account that is not connected were computed
+for every workflow and then left out of the roll-up. They are in it, and in the client-facing report,
+in plain words.
+
+**"Trained." means the pages are trained.** Training a knowledge base settled on a status shared
+with discovery, so it said Trained the moment discovery finished, for a failed run, and for pages
+queued behind an earlier batch. It now settles only when every requested page is in the trained
+bucket, and a crawl GoHighLevel stopped says so instead of "Discovery finished".
+
+**The morning brief no longer calls a morning quiet when your own calendar could not be read.**
+A connected Google Calendar that failed to open produced the same "Nothing needs you this morning"
+as an empty one. The headline now says the calendar could not be read.
+
+**Readiness, calendars, smart lists, keys and pages tell the truth.** A phone-agent readiness check
+no longer says READY on a number pool whose numbers it cannot read. Renaming a calendar carries the
+assigned staff back in the shape GoHighLevel validates and reports the stored team-member count from
+a read after the update. A contact smart list is refused on the opportunity service instead of being
+created where the Contacts screen never shows it. "Removed" for a sub-account key is said only when
+the key really left this computer's file. A page save whose baseline could not be read is reported
+as unverified rather than visible, and a page build is verified on the pointer GoHighLevel serves
+the page from. Switching accounts says the workflow-builder credentials were found, not tested.
+
+**Also in this release:** the eleven small findings deferred from the 3.84.1 reviews. Among them:
+a survey trigger is checked against the account instead of refused as a dead end; three more trigger
+slots are read back; a plan handed back by hand is held to the same confirmed selection as a saved
+one; a half-applied older build is told to keep its undo and plan only the rest; a number an agent
+held only in the old non-routing shape is named when a bind starts routing it; and the brand repaint
+leaves a tint or tone as it was only when that makes the text readable, saying so honestly, and
+repaints every spelling of a colour, not only the six-digit hex.
+
 ## 3.84.1 — the phone agent answers, the repaint tells the truth, and the build waits to be told
 
 **A phone agent you connect now actually answers.** Connecting a number to a voice agent wrote a

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.84.1
+**Version:** 3.84.2
 
 ---
 

package/dist/index.js

@@ -948,20 +948,33 @@ var init_token_registry = __esm({
         const token2 = this.data.tokens[locationId2];
         if (!token2) return false;
         if (token2.companyId === companyId) return true;
+        const previous = token2.companyId;
         token2.companyId = companyId;
         this.save();
+        if (this.lastSaveError) {
+          if (previous === void 0) delete token2.companyId;
+          else token2.companyId = previous;
+          return false;
+        }
         return true;
       }
       /**
-       * Remove a location from the registry
+       * Remove a location from the registry. Returns true only when the removal
+       * reached the key file on disk: save() swallows a write failure into
+       * lastSaveError, and a key still sitting in .ghl-tokens.json comes back in
+       * full at the next start. A caller that reports "Removed" must not be told
+       * that by a delete from an in-memory map alone.
        */
       removeLocation(locationId2) {
-        if (this.data.tokens[locationId2]) {
-          delete this.data.tokens[locationId2];
-          this.save();
-          return true;
+        const existing = this.data.tokens[locationId2];
+        if (!existing) return false;
+        delete this.data.tokens[locationId2];
+        this.save();
+        if (this.lastSaveError) {
+          this.data.tokens[locationId2] = existing;
+          return false;
         }
-        return false;
+        return true;
       }
       /**
        * List all registered locations
@@ -2610,19 +2623,27 @@ ${errorBody}`
         const result = await this.updateWorkflow(workflowId, { status: "published", triggers });
         let returned = null;
         let readBackFailed = null;
+        let savedSteps = null;
+        let savedTriggers = null;
         try {
           const after = await this.getWorkflow(workflowId);
           returned = typeof after.status === "string" ? after.status : null;
+          savedSteps = (after.workflowData?.templates ?? []).length;
+          savedTriggers = after.triggers ?? [];
         } catch (e) {
           readBackFailed = e instanceof Error ? e.message : String(e);
           returned = isRecord2(result) && typeof result.status === "string" ? result.status : null;
         }
+        const activeTriggers = savedTriggers ? savedTriggers.filter((t) => t.active === true).length : null;
+        const inactiveTriggers = savedTriggers ? savedTriggers.filter((t) => t.active === false).length : 0;
         const publish_check = {
           published: returned === "published",
           status_returned: returned ?? "(GHL returned no status)",
           status_source: readBackFailed ? "the publish response (re-read failed)" : "the saved workflow, read back",
-          steps: steps.length,
-          triggers: triggers.length
+          counts_source: savedTriggers ? "the saved workflow, read back" : "the read taken BEFORE the write (the re-read failed, so these counts do not prove what is saved)",
+          steps: savedSteps ?? steps.length,
+          triggers: savedTriggers ? savedTriggers.length : triggers.length,
+          triggers_active: activeTriggers
         };
         if (readBackFailed) {
           publish_check.warning = `NOT VERIFIED. The publish was accepted but reading the workflow back failed (${readBackFailed}), so this status is only what the publish call echoed, not the saved state. Confirm with get_workflow_full.`;
@@ -2630,7 +2651,10 @@ ${errorBody}`
         if (returned !== "published") {
           publish_check.warning = `NOT LIVE. GHL returned status ${returned === null ? "nothing" : `"${returned}"`} rather than "published", so do not report this workflow as running.`;
         }
-        if (triggers.length === 0) {
+        if (returned === "published" && inactiveTriggers > 0) {
+          publish_check.warning = `TRIGGER NOT ACTIVE. The workflow saved as "published", but ${inactiveTriggers} of ${savedTriggers.length} trigger(s) still read active:false, so ${inactiveTriggers === savedTriggers.length ? "nothing enrols contacts into it on its own" : "those triggers enrol nobody"}. Check the trigger in GHL before reporting this workflow as running.`;
+        }
+        if ((savedTriggers ? savedTriggers.length : triggers.length) === 0) {
           publish_check.note = "This workflow has NO TRIGGER, so nothing starts it on its own. That is only correct if another workflow adds contacts to it; otherwise its trigger did not save.";
         }
         return isRecord2(result) ? { ...result, publish_check } : { result, publish_check };
@@ -5197,8 +5221,40 @@ function registerCalendarTools(server2, client) {
           `update_calendar: could not read calendar ${args.calendarId} before updating it (${e instanceof Error ? e.message : String(e)}). Refusing to write: GHL's calendar update does not merge, so a partial write here would silently reset this calendar's slot length, buffer and availability hours.`
         );
       }
+      if (existing && existing.calendarType === "event") existing = { ...existing, teamMembers: void 0 };
+      if (existing && args.teamMembers === void 0 && Array.isArray(existing.teamMembers)) {
+        const stripped = existing.teamMembers.map((m) => {
+          const r = m ?? {};
+          const configs = Array.isArray(r.locationConfigurations) ? r.locationConfigurations.map((c) => {
+            const x = c ?? {};
+            return { kind: x.kind, position: x.position, ...typeof x.location === "string" ? { location: x.location } : {} };
+          }) : r.locationConfigurations;
+          return { userId: r.userId, priority: r.priority, isPrimary: r.isPrimary, selected: r.selected, locationConfigurations: configs };
+        });
+        const parsed2 = import_zod12.z.array(CalendarTeamMemberSchema).safeParse(stripped);
+        if (!parsed2.success) {
+          const issue = parsed2.error.issues[0];
+          throw new Error(
+            `update_calendar: this calendar's stored team members cannot be carried over in the shape GoHighLevel's update validates (${issue ? `${issue.path.join(".")}: ${issue.message}` : "unknown"}). Nothing was changed, because a partial write here would unassign the staff. Pass teamMembers explicitly (userId, priority 0/0.5/1, isPrimary, selected, locationConfigurations with at least one entry) and run the update again.`
+          );
+        }
+        existing = { ...existing, teamMembers: parsed2.data };
+      }
       const body = buildUpdateCalendarBody(mergeCalendarUpdate(existing, args));
-      return await client.put(`/calendars/${args.calendarId}`, { body });
+      const updated = await client.put(`/calendars/${args.calendarId}`, { body });
+      let storedTeamMembers;
+      try {
+        const after = await client.get(`/calendars/${args.calendarId}`);
+        const cal = after?.calendar ?? after;
+        storedTeamMembers = Array.isArray(cal?.teamMembers) ? cal.teamMembers.length : 0;
+      } catch (e) {
+        storedTeamMembers = `NOT VERIFIED \u2014 the calendar could not be read back after the update (${e instanceof Error ? e.message : String(e)}), so its team-member assignment is unconfirmed. Run get_calendar to check.`;
+      }
+      return {
+        updated,
+        stored_team_members: storedTeamMembers,
+        stored_team_members_source: "GET /calendars/{id} taken AFTER the update \u2014 a read of what GoHighLevel now holds, not the update's own response."
+      };
     }
   );
   safeTool(
@@ -5401,7 +5457,13 @@ var init_calendars = __esm({
       "appointmentPerDay",
       "allowBookingAfter",
       "allowBookingAfterUnit",
-      "availabilityType"
+      "availabilityType",
+      "teamMembers",
+      "description",
+      "slug",
+      "widgetSlug",
+      "eventTitle",
+      "eventColor"
     ];
   }
 });
@@ -8255,6 +8317,16 @@ function normalizeCrawlStatus(raw) {
     urls
   };
 }
+function trainedOfRequested(s, urlIds) {
+  const bucketOf = new Map(s.urls.map((u) => [u.id, u.bucket]));
+  const isTrained = (id) => TRAINED_BUCKET.test(bucketOf.get(id) ?? "");
+  const pending = urlIds.filter((id) => !isTrained(id));
+  return {
+    trained: urlIds.filter(isTrained),
+    pending,
+    buckets: [...new Set(pending.map((id) => bucketOf.get(id) ?? "not listed"))]
+  };
+}
 async function readCrawlStatus(client, locationId2, knowledgeBaseId, operationId) {
   return normalizeCrawlStatus(await client.get("/knowledge-bases/crawler/status", { version: AI_API_VERSION, params: { locationId: locationId2, operationId, knowledgeBaseId } }));
 }
@@ -8306,7 +8378,7 @@ async function crawlAndTrain(client, opts) {
   const d = await settleCrawl(client, sleep7, opts.locationId, opts.knowledgeBaseId, operationId, opts.waitDiscover ?? 90, (x) => isSettled(x.status), { operationId, status: typeof rec.status === "string" ? rec.status : "Processing", mode, url: opts.url });
   if (d.pollError) notes.push(`a status poll failed (${d.pollError})`);
   if (!d.settled && d.s.urls.length > 0) notes.push(`${d.s.urls.length} page(s) listed while the operation still read "${d.s.status}" \u2014 the list may be incomplete; the pages below are what was trained`);
-  let trained = 0, trainStatus = "not started", trainSettled = false;
+  let trained = 0, trainedPages = 0, trainStatus = "not started", trainSettled = false;
   const chosen = mode === "Exact" ? [] : pickPages(d.s.urls, opts.maxPages ?? 8);
   if (mode === "Exact") {
     notes.push("Exact mode: the single page trains itself.");
@@ -8314,17 +8386,23 @@ async function crawlAndTrain(client, opts) {
     notes.push("no pages were listed to train");
   } else {
     await client.post("/knowledge-bases/crawler/train", { ...v, body: { locationId: opts.locationId, urlIds: chosen.map((c) => c.id), knowledgeBaseId: opts.knowledgeBaseId, operationId }, noRetry: true });
-    const t = await settleCrawl(client, sleep7, opts.locationId, opts.knowledgeBaseId, operationId, opts.waitTrain ?? 120, (x) => isSettled(x.status) || x.trainedUrlsCount >= chosen.length, { operationId, status: "Training" });
+    const chosenIds = chosen.map((c) => c.id);
+    const t = await settleCrawl(client, sleep7, opts.locationId, opts.knowledgeBaseId, operationId, opts.waitTrain ?? 120, (x) => trainSettledOn(x, chosenIds), { operationId, status: "Training" });
     trained = t.s.trainedUrlsCount;
     trainStatus = t.s.status;
     trainSettled = t.settled;
+    trainedPages = trainedOfRequested(t.s, chosenIds).trained.length;
     if (t.pollError) notes.push(`a training poll failed (${t.pollError})`);
-    if (!t.settled) notes.push(`training still read "${t.s.status}" after ${opts.waitTrain ?? 120}s`);
+    else if (wasStopped(t.s.status)) notes.push(`GoHighLevel stopped this operation ("${t.s.status}"), so nothing here confirms these pages trained`);
+    else if (!t.settled) {
+      const { pending, buckets } = trainedOfRequested(t.s, chosenIds);
+      notes.push(`${pending.length} of ${chosenIds.length} page(s) are still not in a "Trained" bucket after ${opts.waitTrain ?? 120}s (operation reads "${t.s.status}"; those pages read ${buckets.join(", ")})`);
+    }
   }
   const listed = unwrapEnvelope(await client.get("/knowledge-bases/crawler", { ...v, params: { knowledgeBaseId: opts.knowledgeBaseId, locationId: opts.locationId, page: 1, pageLength: 50 } }));
   const rows2 = isRecord4(listed) && Array.isArray(listed.urls) ? listed.urls : [];
   const pages = rows2.filter(isRecord4).map((r) => ({ id: String(r.id ?? r._id ?? ""), url: String(r.url ?? ""), title: typeof r.title === "string" ? r.title : void 0, content: typeof r.content === "string" ? r.content : void 0 }));
-  return { operationId, discovered: d.s.urls.length, discoverStatus: d.s.status, discoverSettled: d.settled, trained, trainStatus, trainSettled, pages, notes };
+  return { operationId, discovered: d.s.urls.length, discoverStatus: d.s.status, discoverSettled: d.settled, trained, trainedPages, trainStatus, trainSettled, pages, notes };
 }
 function registerKnowledgeBaseTools(server2, client, deps = {}) {
   const sleep7 = deps.sleep ?? realSleep;
@@ -8431,13 +8509,19 @@ function registerKnowledgeBaseTools(server2, client, deps = {}) {
       const startStatus = typeof rec.status === "string" ? rec.status : "Processing";
       const { s, polls, settled, pollError } = await settle2(loc, knowledgeBaseId, operationId, wait, (x) => isSettled(x.status), { operationId, status: startStatus, mode: option, url });
       const listed = s.urls.length;
+      const outcome = crawlOutcome(s.status, settled);
       return {
         operationId,
         ...s,
         polls,
         settled,
+        outcome,
         ...pollError ? { pollError } : {},
-        next: option === "Exact" ? "This page trains itself \u2014 do NOT call train_kb_urls on it (GoHighLevel answers 500). Check list_kb_urls in a moment." : pollError ? `A status poll failed (${pollError}); discovery may still be running \u2014 poll get_kb_crawl_status with operationId "${operationId}".` : settled ? `Discovery finished with ${listed} page(s). Pick the ones the agent should know and pass their ids to train_kb_urls with operationId "${operationId}".` : listed > 0 ? `${listed} page(s) listed so far, but the operation still reads "${s.status}" after ${wait}s, so the list may be incomplete. Train these now with train_kb_urls (operationId "${operationId}"), or poll get_kb_crawl_status for more first.` : `Discovery is still running after ${wait}s with nothing listed yet \u2014 poll get_kb_crawl_status with operationId "${operationId}".`
+        next: (
+          // The refusal goes first, Exact included: a page that "trains itself"
+          // trains nothing when GoHighLevel stopped the operation.
+          outcome === "stopped" ? `GoHighLevel stopped this crawl (${s.status}) \u2014 it did not finish reading ${url}${listed ? `, and the ${listed} page(s) listed below are whatever it had before it stopped` : ", and nothing was listed"}. Nothing here says the site was read, so do not train from this run: fix what stopped it and crawl again.` : option === "Exact" ? "This page trains itself \u2014 do NOT call train_kb_urls on it (GoHighLevel answers 500). Check list_kb_urls in a moment." : pollError ? `A status poll failed (${pollError}); discovery may still be running \u2014 poll get_kb_crawl_status with operationId "${operationId}".` : settled ? `Discovery finished with ${listed} page(s). Pick the ones the agent should know and pass their ids to train_kb_urls with operationId "${operationId}".` : listed > 0 ? `${listed} page(s) listed so far, but the operation still reads "${s.status}" after ${wait}s, so the list may be incomplete. Train these now with train_kb_urls (operationId "${operationId}"), or poll get_kb_crawl_status for more first.` : `Discovery is still running after ${wait}s with nothing listed yet \u2014 poll get_kb_crawl_status with operationId "${operationId}".`
+        )
       };
     }
   );
@@ -8457,16 +8541,24 @@ function registerKnowledgeBaseTools(server2, client, deps = {}) {
       const queued = await client.post("/knowledge-bases/crawler/train", { ...v, body: { locationId: loc, urlIds, knowledgeBaseId, operationId }, noRetry: true });
       const wait = waitSeconds ?? 60;
       if (wait === 0) return { queued, note: "Queued. Poll get_kb_crawl_status for trainedUrlsCount." };
-      const { s, polls, settled, pollError } = await settle2(loc, knowledgeBaseId, operationId, wait, (x) => isSettled(x.status) || x.trainedUrlsCount >= urlIds.length, { operationId, status: "Training" });
+      const { s, polls, settled, pollError } = await settle2(loc, knowledgeBaseId, operationId, wait, (x) => trainSettledOn(x, urlIds), { operationId, status: "Training" });
+      const { trained, pending, buckets } = trainedOfRequested(s, urlIds);
+      const outcome = crawlOutcome(s.status, settled);
       return {
         queued,
         status: s.status,
+        outcome,
+        trained,
+        pending,
+        // The operation's own counter, kept because it is what GoHighLevel
+        // reports — it counts every page trained against this operationId,
+        // including earlier train calls, so it is not a result for THIS one.
         trainedUrlsCount: s.trainedUrlsCount,
         requested: urlIds.length,
         polls,
         settled,
         ...pollError ? { pollError } : {},
-        note: pollError ? `Training was queued, then a status poll failed (${pollError}) \u2014 poll get_kb_crawl_status with operationId "${operationId}".` : settled ? `Trained. list_kb_urls shows each page with a link to the text the agent learned.` : `Still training after ${wait}s \u2014 poll get_kb_crawl_status.`
+        note: pollError ? `Training was queued, then a status poll failed (${pollError}) \u2014 poll get_kb_crawl_status with operationId "${operationId}".` : outcome === "stopped" ? `GoHighLevel stopped this operation (${s.status}). ${trained.length} of ${urlIds.length} requested page(s) are in a "Trained" bucket; the other ${pending.length} are not, so the agent has not learned them.` : settled ? `Trained. list_kb_urls shows each page with a link to the text the agent learned.` : `Still training after ${wait}s \u2014 ${pending.length} of ${urlIds.length} requested page(s) are not in a "Trained" bucket yet (they read ${buckets.join(", ")}; the operation reads "${s.status}"). Poll get_kb_crawl_status with operationId "${operationId}".`
       };
     }
   );
@@ -8573,7 +8665,7 @@ function registerKnowledgeBaseTools(server2, client, deps = {}) {
     }
   );
 }
-var import_zod34, AI_API_VERSION, KB_CAP_PER_LOCATION, realSleep, FINISHED_STATUS, isSettled, PAGE_PRIORITY, HOME_PATH, SKIP;
+var import_zod34, AI_API_VERSION, KB_CAP_PER_LOCATION, realSleep, FINISHED_STATUS, isSettled, STOPPED_STATUS, wasStopped, crawlOutcome, TRAINED_BUCKET, trainSettledOn, PAGE_PRIORITY, HOME_PATH, SKIP;
 var init_knowledge_base = __esm({
   "src/tools/knowledge-base.ts"() {
     "use strict";
@@ -8584,6 +8676,11 @@ var init_knowledge_base = __esm({
     realSleep = (ms) => new Promise((r) => setTimeout(r, ms));
     FINISHED_STATUS = /^(successful|success|failed|failure|existing|restricted|cancelled|canceled|aborted|completed?)$/i;
     isSettled = (status) => FINISHED_STATUS.test(status.trim());
+    STOPPED_STATUS = /^(failed|failure|restricted|cancelled|canceled|aborted)$/i;
+    wasStopped = (status) => STOPPED_STATUS.test(status.trim());
+    crawlOutcome = (status, settled) => !settled ? "running" : wasStopped(status) ? "stopped" : "finished";
+    TRAINED_BUCKET = /^trained$/i;
+    trainSettledOn = (s, urlIds) => wasStopped(s.status) || trainedOfRequested(s, urlIds).pending.length === 0;
     PAGE_PRIORITY = /about|service|treatment|program|plan|product|menu|special|offer|pricing|price|cost|fee|insurance|contact|location|direction|faq|question|help|hour|book|schedule|appointment|new[- ]?patient|patient|client|team|staff|doctor|provider|review|testimonial/i;
     HOME_PATH = /^\/(index(\.html?)?|home(\.html?)?)?\/?$/i;
     SKIP = /privacy|terms|cookie|legal|login|sign[- ]?in|cart|checkout|account|unsubscribe|\.(pdf|jpe?g|png|gif|svg|xml|json|css|js)(\?|$)|\/blog\/|\/news\/|\/tag\/|\/category\/|\/author\/|\/page\/\d/i;
@@ -9260,13 +9357,17 @@ function registerAiEmployeeDemoTools(server2, shared, deps = {}) {
         return {
           runId: run.runId,
           bot: { id: botId, name: upd.after.name, mode: upd.after.mode, knowledgeBaseIds: upd.after.knowledgeBaseIds, businessName: upd.after.businessName },
-          knowledgeBase: { id: kbId, discovered: crawl.discovered, trained: crawl.trained, learned, notes: crawl.notes },
+          // trainedPages = pages GoHighLevel holds in a Trained bucket; the
+          // operation counter climbs across every train call and is named for
+          // what it is (delta review 3).
+          knowledgeBase: { id: kbId, discovered: crawl.discovered, trainedPages: crawl.trainedPages ?? 0, operationTrainedCount: crawl.trained, learned, notes: crawl.notes },
           prospect: { contactId, phone: prospectPhone, fieldsVerified: Object.keys(values).length },
           numberToText,
           primaryOnSms: primary,
           armed: run.armed,
           readiness: [
-            crawl.trained > 0 ? `the bot has learned ${crawl.trained} page(s)` : "the bot has learned NO pages yet",
+            // Per-page buckets, not the operation counter (sweep review 7).
+            (crawl.trainedPages ?? 0) > 0 ? `the bot has learned ${crawl.trainedPages} page(s)` : "the bot has learned NO pages yet",
             armedNow && wasAutoPilot ? "WARNING: the bot was ALREADY on auto-pilot before this demo and answers on its own; end_ai_employee_demo puts it back to auto-pilot, because that is its before-state" : armedNow ? "the bot is on auto-pilot (armed here)" : armAutoPilot ? `WARNING: arming was requested but the bot reads back as "${String(upd.after.mode)}" \u2014 it is NOT answering on its own` : "the bot is NOT armed \u2014 pass armAutoPilot: true to let it answer",
             primary === true ? "the bot is primary on SMS" : primary === false ? "the bot is NOT primary on SMS \u2014 it will not answer texts until it is" : "could not read whether the bot is primary on SMS",
             numberToText ? `text ${numberToText}` : "this sub-account has no phone number",
@@ -9803,13 +9904,16 @@ function registerVoiceAiTools(server2, shared) {
         for (const n of held) if (!holders.has(n)) holders.set(n, { agentId: String(a.id), agentName: String(a.agentName), via: `pool ${pool}` });
         return { agentId: String(a.id), agentName: String(a.agentName), poolId: pool, poolNumbers: held.length ? held : "unknown \u2014 this API does not list the pool's numbers" };
       });
-      const unresolvedPool = poolBound.some((p) => typeof p.poolNumbers === "string");
+      const poolRouting = poolBound.filter((p) => Array.isArray(p.poolNumbers) && p.poolNumbers.length > 0);
+      const poolUnproven = poolBound.filter((p) => typeof p.poolNumbers === "string");
+      const unresolvedPool = poolUnproven.length > 0;
       const numberRows = numbers.map((n) => ({ number: n, answeredBy: holders.get(n) ?? null }));
       const free = unresolvedPool ? [] : numberRows.filter((r) => !r.answeredBy && !heldUnproven.has(r.number)).map((r) => r.number);
       const rota = agents.filter(inBackupRota).map((a) => ({ agentId: String(a.id), agentName: String(a.agentName) }));
-      const ready = numberRows.some((r) => r.answeredBy) || poolBound.length > 0;
+      const ready = numberRows.some((r) => r.answeredBy) || poolRouting.length > 0;
       const unprovenLine = unproven.length ? ` ${unproven.length} agent(s) carry a number binding NOT proven to route (${unproven.map((u) => `"${u.agentName}" on ${u.numbers.join(", ")}`).join("; ")}): see unprovenBindings for the fix.` : "";
-      const verdict = numbers.length === 0 ? "NOT READY: no phone number on this sub-account." : ready ? `READY: ${numberRows.filter((r) => r.answeredBy).length} number(s) bound to a voice agent${poolBound.length ? `, and ${poolBound.length} agent(s) answering through a number pool` : ""}${rota.length ? `; ${rota.length} agent(s) also sit in the backup rota` : ""}.${unprovenLine}` : unproven.length ? `NOT PROVEN TO ROUTE: no number is bound in the shape GoHighLevel routes to an agent.${unprovenLine}${rota.length ? ` ${rota.length} agent(s) sit in the backup rota (${rota.map((r) => r.agentName).join(", ")}) and may be handed unanswered calls.` : ""}` : `NUMBER PRESENT, NO AGENT BOUND BY API: ${numbers.length} number(s), none carried in inboundNumbers or as numberPoolId by any voice agent.${rota.length ? ` ${rota.length} agent(s) sit in the backup rota (${rota.map((r) => r.agentName).join(", ")}) and may be handed unanswered calls.` : ""} Calls may still reach an agent through the number's own routing, which this API cannot read.`;
+      const poolUnprovenLine = poolUnproven.length ? ` ${poolUnproven.length} agent(s) answer through a number pool this API does not list that pool's numbers for (${poolUnproven.map((p) => `"${p.agentName}" on pool ${p.poolId}`).join("; ")}), so nothing here proves a call lands: open the pool in GoHighLevel and check it holds a number.` : "";
+      const verdict = numbers.length === 0 ? "NOT READY: no phone number on this sub-account." : ready ? `READY: ${numberRows.filter((r) => r.answeredBy).length} number(s) bound to a voice agent${poolRouting.length ? `, and ${poolRouting.length} agent(s) answering through a number pool` : ""}${rota.length ? `; ${rota.length} agent(s) also sit in the backup rota` : ""}.${unprovenLine}${poolUnprovenLine}` : unproven.length || poolUnproven.length ? `NOT PROVEN TO ROUTE: no number is bound in the shape GoHighLevel routes to an agent.${unprovenLine}${poolUnprovenLine}${rota.length ? ` ${rota.length} agent(s) sit in the backup rota (${rota.map((r) => r.agentName).join(", ")}) and may be handed unanswered calls.` : ""}` : `NUMBER PRESENT, NO AGENT BOUND BY API: ${numbers.length} number(s), none carried in inboundNumbers or as numberPoolId by any voice agent.${rota.length ? ` ${rota.length} agent(s) sit in the backup rota (${rota.map((r) => r.agentName).join(", ")}) and may be handed unanswered calls.` : ""} Calls may still reach an agent through the number's own routing, which this API cannot read.`;
       return {
         locationId: loc,
         verdict,
@@ -9939,6 +10043,7 @@ function registerVoiceAiTools(server2, shared) {
       if (numberPoolId !== void 0) change.numberPoolId = numberPoolId;
       if (Object.keys(change).length === 0 && inboundNumber === void 0) throw new Error("Nothing to update \u2014 pass at least one field to change.");
       const before = await readVoiceAgent(client, loc, agentId);
+      const bindNotes = [];
       if (typeof inboundNumber === "string") {
         const current = Array.isArray(before.inboundNumbers) ? before.inboundNumbers.filter((n) => typeof n === "string" && n.length > 0) : [];
         change.inboundNumbers = [.../* @__PURE__ */ new Set([...current, inboundNumber])];
@@ -9965,8 +10070,26 @@ function registerVoiceAiTools(server2, shared) {
         const { numbers, pools, poolObjects } = await readNumbers(client, loc);
         if (typeof inboundNumber === "string") {
           if (!numbers.includes(inboundNumber)) throw new Error(numbers.length ? `${inboundNumber} is not a number on this sub-account. Its numbers: ${numbers.join(", ")}.` : NO_NUMBER_TASK);
-          const clash = bindingClash(inboundNumber, await listAllVoiceAgents(client, loc), agentId, poolObjects);
+          const agents = await listAllVoiceAgents(client, loc);
+          const clash = bindingClash(inboundNumber, agents, agentId, poolObjects);
           if (clash) throw new Error(clash);
+          const dormant = unprovenNumbers(before).filter((n) => n !== inboundNumber);
+          const kept = [];
+          for (const n of change.inboundNumbers) {
+            if (n === inboundNumber) {
+              kept.push(n);
+              continue;
+            }
+            const holder = agents.find((a) => a.id !== agentId && boundNumbers(a).includes(n));
+            if (holder && dormant.includes(n)) {
+              bindNotes.push(`${n} was held by this agent in the shape not proven to route and now answers for agent "${String(holder.agentName)}" (${String(holder.id)}), so it was left off this agent rather than written onto two.`);
+              continue;
+            }
+            if (holder) bindNotes.push(`${n} also answers for agent "${String(holder.agentName)}" (${String(holder.id)}) in GoHighLevel. It stays on this agent, which was routing it; GoHighLevel decides which of the two takes a call, so unbind it on one of them.`);
+            kept.push(n);
+            if (dormant.includes(n)) bindNotes.push(`${n} was held by this agent in the shape not proven to route (BUG-2026-09-15-02) and starts routing to it with this write.`);
+          }
+          change.inboundNumbers = kept;
         }
         if (typeof numberPoolId === "string") {
           if (!pools.includes(numberPoolId)) throw new Error(pools.length ? `Number pool ${numberPoolId} is not on this sub-account. Its pools: ${pools.join(", ")}.` : "This sub-account has no number pools (list_number_pools).");
@@ -9991,7 +10114,7 @@ function registerVoiceAiTools(server2, shared) {
       if (!rb.agent) return { sent: Object.keys(change), changed: [], before: {}, agent: void 0, answersOn: [], pool: null, inBackupRota: void 0, readBack: "ASSERTED", differs: void 0, notes: [READBACK_FAILED("update", agentId, rb.error ?? "unknown error")] };
       const after = rb.agent;
       const changed = diffVoiceAgentSnapshots(before, after);
-      const notes = [];
+      const notes = [...bindNotes];
       const differs = agentWriteDifferences(change, after);
       for (const k of differs) notes.push(`${k}: sent ${JSON.stringify(change[k])}, reads back ${JSON.stringify(after[k])}.`);
       let asserted2 = differs.length > 0;
@@ -11619,8 +11742,10 @@ ${await response.text()}`);
     async ({ pageId, content, funnelId, isPublished }) => {
       try {
         let meta = {};
+        let preReadOk = false;
         try {
           meta = await funnelRequest("GET", `/page/${pageId}?locationId=${client.locationId}`);
+          preReadOk = true;
         } catch (metaErr) {
           if (!funnelId) throw metaErr;
         }
@@ -11656,14 +11781,24 @@ ${await response.text()}`);
           }
           const afterVersion = typeof after.pageVersion === "number" ? after.pageVersion : null;
           const becameVisible = afterUrl !== "" && afterUrl !== beforeUrl;
-          save_check = {
-            visible: becameVisible,
-            pageVersion_sent: pageVersion,
-            pageVersion_now: afterVersion,
-            ...becameVisible ? {} : {
-              warning: "SAVED BUT NOT VISIBLE. GHL accepted the write and stored the content, but the page's live pointer did not move, so visitors still see the previous version. This is GHL's behaviour on a page that already has content, not a transport error. Rebuild on a fresh step with create_funnel_page and save onto that instead \u2014 do NOT report this page as updated."
-            }
-          };
+          if (!preReadOk) {
+            const versionMismatch = afterVersion !== null && afterVersion !== pageVersion;
+            save_check = {
+              visible: null,
+              pageVersion_sent: pageVersion,
+              pageVersion_now: afterVersion,
+              warning: "SAVE NOT VERIFIED. The page could not be read before the save, so whether the new version is the one visitors see is not verified \u2014 with no baseline pointer, a page that never moved off its old content is indistinguishable from one that took the save. Read it with get_page_full and compare before reporting it as updated." + (versionMismatch ? ` GoHighLevel now reports this page at pageVersion ${afterVersion} while the save was sent as pageVersion ${pageVersion}: the page already held content this save did not build on, which is exactly the case where GHL stores the write and keeps serving the previous version.` : "")
+            };
+          } else {
+            save_check = {
+              visible: becameVisible,
+              pageVersion_sent: pageVersion,
+              pageVersion_now: afterVersion,
+              ...becameVisible ? {} : {
+                warning: "SAVED BUT NOT VISIBLE. GHL accepted the write and stored the content, but the page's live pointer did not move, so visitors still see the previous version. This is GHL's behaviour on a page that already has content, not a transport error. Rebuild on a fresh step with create_funnel_page and save onto that instead \u2014 do NOT report this page as updated."
+              }
+            };
+          }
         } catch (verifyErr) {
           save_check = {
             visible: null,
@@ -12468,6 +12603,10 @@ function pageFaces(envelope) {
 function derivePalette(envelope) {
   return derivePaletteFromUsage(usageOrHexes(envelope), pageFaces(envelope));
 }
+function paintsAs(literal, from) {
+  if (/^#[0-9a-f]{8}$/i.test(literal)) return { hit: literal.slice(0, 7).toLowerCase() === from, keep: literal.slice(7) };
+  return { hit: normaliseColour(literal)?.toLowerCase() === from, keep: "" };
+}
 function sectionPairs(envelope) {
   const groups = /* @__PURE__ */ new Map();
   for (const p of renderedPalette(envelope).paints) {
@@ -12517,28 +12656,47 @@ function rethemeEnvelope(envelope, brand, derived = derivePalette(envelope)) {
     return out;
   };
   const sayLoss = (l) => `${l.where}: ${target(l.t).toUpperCase()} type on ${target(l.bg).toUpperCase()} would be ${l.after.toFixed(2)}:1 (it was ${l.before.toFixed(2)}:1); WCAG AA wants 4.5.`;
-  for (let pass = 0; pass <= plan.length; pass++) {
-    const hit = losses().find((l) => l.involved.some((x) => x.guarded));
-    if (!hit) break;
-    for (const step of hit.involved.filter((x) => x.guarded)) {
-      skipped.push(`${step.role} ${step.from}: left as it was, because ${sayLoss(hit)}`);
-      plan.splice(plan.indexOf(step), 1);
+  const guardedAtStart = plan.filter((x) => x.guarded).length;
+  for (let pass = 0; pass < guardedAtStart; pass++) {
+    let removed = false;
+    for (const hit of losses()) {
+      for (const step of hit.involved.filter((x) => x.guarded)) {
+        const without = (hex) => hex === step.from ? hex : target(hex);
+        if (contrastHex(without(hit.t), without(hit.bg)) < 4.5) continue;
+        skipped.push(`${step.role} ${step.from}: left as it was, because ${sayLoss(hit)}`);
+        plan.splice(plan.indexOf(step), 1);
+        removed = true;
+        break;
+      }
+      if (removed) break;
     }
+    if (!removed) break;
   }
   const seen = /* @__PURE__ */ new Set();
   for (const l of losses()) {
-    const line3 = `${sayLoss(l)} Kept, because ${l.involved.map((x) => `the ${x.role}`).join(" and ")} ${l.involved.length > 1 ? "are" : "is"} the brand's own colour: check this section before a client sees it.`;
+    const brandRoles = l.involved.filter((x) => !x.guarded).map((x) => `the ${x.role}`);
+    const guardedRoles = l.involved.filter((x) => x.guarded).map((x) => `the ${x.role}`);
+    const because = [
+      brandRoles.length ? `${brandRoles.join(" and ")} ${brandRoles.length > 1 ? "are" : "is"} the brand's own colour` : "",
+      guardedRoles.length ? `leaving ${guardedRoles.join(" and ")} as it was would not make it readable` : ""
+    ].filter(Boolean).join(", and ");
+    const line3 = `${sayLoss(l)} Kept, because ${because}: check this section before a client sees it.`;
     if (!seen.has(`${target(l.t)} ${target(l.bg)}`)) warnings.push(line3);
     seen.add(`${target(l.t)} ${target(l.bg)}`);
   }
   let text2 = JSON.stringify(envelope);
   const colourTokens = [];
   plan.forEach((step, i) => {
-    const re = new RegExp(escapeRe(step.from), "gi");
-    const occurrences = (text2.match(re) ?? []).length;
-    if (occurrences === 0) return;
     const token2 = `\0COLOUR${i}\0`;
-    text2 = text2.replace(re, token2);
+    let occurrences = 0;
+    const swapped = text2.replace(COLOUR_LITERAL_RE, (lit) => {
+      const { hit, keep } = paintsAs(lit, step.from);
+      if (!hit) return lit;
+      occurrences++;
+      return token2 + keep;
+    });
+    if (occurrences === 0) return;
+    text2 = swapped;
     colourTokens.push({ token: token2, to: step.to });
     swaps.push({ role: step.role, from: step.from, to: step.to, occurrences });
   });
@@ -12581,7 +12739,7 @@ function derivePaletteAcross(envelopes) {
   for (const env2 of envelopes) for (const f of pageFaces(env2)) if (!fonts.includes(f)) fonts.push(f);
   return derivePaletteFromUsage(pooled2, fonts.slice(0, 2));
 }
-var ICON_FONTS, BRAND_CHROMA, TONE_CHROMA, escapeRe;
+var ICON_FONTS, BRAND_CHROMA, TONE_CHROMA, escapeRe, COLOUR_LITERAL_RE;
 var init_retheme = __esm({
   "src/command-os/design/retheme.ts"() {
     "use strict";
@@ -12593,6 +12751,7 @@ var init_retheme = __esm({
     BRAND_CHROMA = 0.06;
     TONE_CHROMA = 0.02;
     escapeRe = (s) => s.replace(/[.*+?^${}()|[\]\\]/g, "\\$&");
+    COLOUR_LITERAL_RE = /#[0-9a-f]{3,8}(?![0-9a-f])|rgba?\([^()]*\)|hsla?\([^()]*\)/gi;
   }
 });
 
@@ -17229,21 +17388,24 @@ ${await response.text()}`);
       integrations: computeIntegrations({ sections })
     };
     const saved = await autosaveRequest(pre.pageId, { funnelId, pageVersion: savePageVersion, pageData });
-    const savedDataUrl = typeof saved.pageDataUrl === "string" ? saved.pageDataUrl : null;
+    const savedDataUrl = typeof saved.pageDataUrl === "string" && saved.pageDataUrl.length > 0 ? saved.pageDataUrl : null;
     let verified = false;
+    let servedUrl = "";
     for (let attempt = 1; attempt <= 8 && !verified; attempt++) {
       await new Promise((r) => setTimeout(r, 4e3));
       try {
         const now = await funnelRequest("GET", `/page/${pre.pageId}?locationId=${client.locationId}`);
-        const nowUrl = typeof now.pageDataUrl === "string" ? now.pageDataUrl : "";
-        if (savedDataUrl !== null && nowUrl === savedDataUrl) verified = true;
+        servedUrl = typeof now.pageDataDownloadUrl === "string" ? now.pageDataDownloadUrl : "";
+        if (savedDataUrl !== null && servedUrl !== "" && servedUrl !== pre.priorDataUrl) verified = true;
       } catch {
       }
     }
     return {
       saved,
       verified,
-      ...verified ? {} : { verifyNote: "The save was accepted but the page has not yet reported the new content (eventual consistency). Re-check get_page_full in a minute before publishing; if it never advances, the save did not take." }
+      ...verified ? {} : {
+        verifyNote: savedDataUrl === null ? "NOT VERIFIED: GoHighLevel's autosave answered without a pageDataUrl, so this save produced no pointer to confirm against" + (servedUrl === "" ? ", and the page still reports no stored content." : ", though the page does now report stored content.") + " Re-check get_page_full in a minute before publishing; if it never advances, the save did not take." : "The save was accepted but the page has not yet reported the new content (eventual consistency). Re-check get_page_full in a minute before publishing; if it never advances, the save did not take."
+      }
     };
   }
   async function savePageGuarded(input) {
@@ -20541,6 +20703,10 @@ function registerValidatorTools(server2, client, builderClient) {
               merge_field_warnings_total: 0,
               workflows_with_shape_warnings: 0,
               shape_warnings_total: 0,
+              workflows_with_save_blocker_warnings: 0,
+              save_blocker_warnings_total: 0,
+              workflows_with_integration_warnings: 0,
+              integration_warnings_total: 0,
               workflows_unscannable: 0,
               workflows_zero_references: 0,
               // Nothing was scanned, so no reference category was loaded or needed.
@@ -20551,6 +20717,8 @@ function registerValidatorTools(server2, client, builderClient) {
             workflows_with_issues: [],
             merge_field_warnings: [],
             shape_warnings: [],
+            save_blocker_warnings: [],
+            integration_warnings: [],
             unscannable: [],
             notes: catalog2.complete ? [] : [incompleteCatalogNote(catalog2.stopReason, 0, catalog2.reportedTotal)]
           });
@@ -20587,6 +20755,12 @@ function registerValidatorTools(server2, client, builderClient) {
         const warningsTotal = withWarnings.reduce((n, w) => n + w.warnings.length, 0);
         const withShapeWarnings = results.filter((r) => r.findings.some((f) => f.severity === "warning" && f.category === "action_shape")).map((r) => ({ workflowId: r.id, workflowName: r.name, status: r.status, warnings: r.findings.filter((f) => f.severity === "warning" && f.category === "action_shape") })).sort((a, b) => (a.status === "published" ? -1 : 1) - (b.status === "published" ? -1 : 1));
         const shapeWarningsTotal = withShapeWarnings.reduce((n, w) => n + w.warnings.length, 0);
+        const SAVE_BLOCKER_CATEGORIES = /* @__PURE__ */ new Set(["legacy_step_id", "legacy_wait_unit", "missing_required_attribute"]);
+        const bucket = (match) => results.filter((r) => r.findings.some(match)).map((r) => ({ workflowId: r.id, workflowName: r.name, status: r.status, warnings: r.findings.filter(match) })).sort((a, b) => (a.status === "published" ? -1 : 1) - (b.status === "published" ? -1 : 1));
+        const withSaveBlockerWarnings = bucket((f) => f.severity === "warning" && SAVE_BLOCKER_CATEGORIES.has(f.category));
+        const saveBlockerWarningsTotal = withSaveBlockerWarnings.reduce((n, w) => n + w.warnings.length, 0);
+        const withIntegrationWarnings = bucket((f) => f.severity === "warning" && f.category === "missing_integration");
+        const integrationWarningsTotal = withIntegrationWarnings.reduce((n, w) => n + w.warnings.length, 0);
         const unverifiedCats = ALL_CATEGORIES.filter((c) => lookups.status[c] !== "loaded");
         const unverifiedRefs = results.reduce((n, r) => n + r.findings.filter((f) => f.severity === "unverified").length, 0);
         return jsonResponse({
@@ -20606,14 +20780,24 @@ function registerValidatorTools(server2, client, builderClient) {
             merge_field_warnings_total: warningsTotal,
             workflows_with_shape_warnings: withShapeWarnings.length,
             shape_warnings_total: shapeWarningsTotal,
+            workflows_with_save_blocker_warnings: withSaveBlockerWarnings.length,
+            save_blocker_warnings_total: saveBlockerWarningsTotal,
+            workflows_with_integration_warnings: withIntegrationWarnings.length,
+            integration_warnings_total: integrationWarningsTotal,
             workflows_unscannable: unscannable.length,
             workflows_zero_references: zeroRefCount,
             unverified: { categories_unloaded: unverifiedCats, references_unverified: unverifiedRefs },
+            // Unchanged on purpose: `status` has meant "are there ERRORS" since it
+            // was written, and these four categories are warnings by ruling — the
+            // workflows genuinely still run. What was broken is that they were
+            // invisible, not that they were ranked too low.
             status: errorsTotal > 0 ? "issues_found" : "ok"
           },
           workflows_with_issues: withErrors,
           merge_field_warnings: withWarnings,
           shape_warnings: withShapeWarnings,
+          save_blocker_warnings: withSaveBlockerWarnings,
+          integration_warnings: withIntegrationWarnings,
           unscannable,
           notes: [
             // NEVER advise re-running for "the next batch": GHL's list endpoint ignores
@@ -20624,6 +20808,8 @@ function registerValidatorTools(server2, client, builderClient) {
             ...unverifiedCats.length ? [`Could not fully load: ${unverifiedCats.join(", ")} \u2014 references to those are 'unverified', not 'broken'. Re-run.`] : [],
             ...warningsTotal ? [`${warningsTotal} {{contact.X}} merge tag(s) reference a field that no longer exists \u2014 these render BLANK in the message but do NOT stop the workflow (warning, not a break). See merge_field_warnings.`] : [],
             ...shapeWarningsTotal ? [`${shapeWarningsTotal} Create/Update Opportunity step(s) have only pipeline + stage \u2014 they cannot create a card for a new contact (see shape_warnings).`] : [],
+            ...saveBlockerWarningsTotal ? [`${saveBlockerWarningsTotal} step(s) across ${withSaveBlockerWarnings.length} workflow(s) either stop the workflow being SAVED (a legacy step id, or the singular wait unit GoHighLevel no longer accepts \u2014 it still runs, it just cannot be edited) or save and run while doing nothing (a notification step with nobody selected). See save_blocker_warnings.`] : [],
+            ...integrationWarningsTotal ? [`${integrationWarningsTotal} step(s) across ${withIntegrationWarnings.length} workflow(s) need a social account this location has not connected \u2014 they save, publish and run, and do nothing when reached. See integration_warnings.`] : [],
             "workflow_goal, goto, and unrecognized condition types are not deeply checked in this version."
           ]
         });
@@ -27524,6 +27710,7 @@ Use these authoritative normalized intake answers when provided: ${JSON.stringif
 8. ${STAFF_NOTIFICATION_RULE} In a hand-authored fallback plan, a notification's "to" is that user id, or "" when the account has no users.
 9. ${MISSING_STEP_RULE}
 10. ${NO_INVENTED_CAUSE_RULE}
+If apply_build_plan answers phase:"scope" (nothing is confirmed or selected for this client, or the selection changed), STOP: build nothing else and finish with RESULT: {"ok":false,"error":"<its error, word for word>"}.
 Finish with EXACTLY one final line:
 RESULT: {"ok":true,"summary":"<what was created: counts of pipelines/fields/tags/workflows, and whether the plan came from the preset or the fallback>","issues":["<anything skipped or needing a human, including every notification step waiting for a staff member>"]}` + loadPlanGuide(ctx?.industry)
       },
@@ -31568,7 +31755,7 @@ var require_package = __commonJS({
   "package.json"(exports2, module2) {
     module2.exports = {
       name: "@elitedcs/ghl-mcp",
-      version: "3.84.1",
+      version: "3.84.2",
       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",
@@ -35255,8 +35442,8 @@ var init_capability_runs = __esm({
       staff: ["get_users", "get_user", "get_calendars"],
       phone: ["list_phone_numbers", "list_number_pools", "get_location"],
       workflows: ["get_workflows", "get_workflow_full", "get_trigger_registry", "get_users", "get_calendars", "get_pipelines", "get_pipeline_full", "get_custom_fields", "get_custom_values", "get_location_tags", "list_email_templates", "list_message_templates", "validate_workflow", "create_workflow", "update_workflow_actions", "audit_workflows"],
-      reactivation: ["get_contact_count", "search_contacts", "get_location_tags", "create_location_tag", "get_workflows", "get_workflow_full", "validate_workflow", "create_workflow", "update_workflow_actions", "create_smart_list", "list_smart_lists"],
-      // no bulk tagging: that is a mass write the operator does by hand
+      reactivation: ["get_contact_count", "search_contacts", "get_location_tags", "create_location_tag", "get_workflows", "get_workflow_full", "validate_workflow", "create_workflow", "update_workflow_actions", "create_smart_list", "list_smart_lists", "create_contact_smart_list", "get_contact_smart_list"],
+      // contact segments live on the contacts service (3.84.2); no bulk tagging: that is a mass write the operator does by hand
       templates: ["list_email_templates", "create_email_template", "update_email_template", "list_message_templates", "create_sms_template", "get_custom_values"],
       broadcasts: ["list_email_templates", "create_email_template", "update_email_template", "list_smart_lists", "get_email_campaigns"],
       // drafts the email as a template; sending or scheduling is the operator's click
@@ -39987,12 +40174,12 @@ function routeFirebaseForCompany(builderClient, registry2, companyId) {
       companyId
     });
     return `
-Firebase: using ${cfg.name || `company ${companyId}`} credentials \u2014 workflow builder enabled here.`;
+Firebase: found stored ${cfg.name || `company ${companyId}`} credentials and pointed the workflow builder at them \u2014 not tested this session. To confirm they still work, run register_company_firebase for this company with test_location_id, which makes a real workflow-builder call.`;
   }
   builderClient.resetToHomeFirebase();
   const homeCompanyId = builderClient.getCurrentCompanyId();
   if (homeCompanyId === companyId) {
-    return "\nFirebase: home company \u2014 workflow builder enabled.";
+    return "\nFirebase: this location's company is your home company, so the home credentials apply \u2014 not tested this session. To confirm the workflow builder really works here, run register_company_firebase for this company with test_location_id, which makes a real workflow-builder call.";
   }
   const registeredCompanies = registry2?.listCompanyFirebases?.() ?? [];
   if (registeredCompanies.length > 0) {
@@ -40223,7 +40410,17 @@ Agency-wide tools now available: list_snapshots, create_snapshot_share_link, and
           isError: true
         };
       }
-      registry2.removeLocation(locationId2);
+      const removed = registry2.removeLocation(locationId2);
+      const saveError = registry2.getLastSaveError();
+      if (!removed || saveError) {
+        return {
+          content: [{
+            type: "text",
+            text: `${token2.name} (${locationId2}) could NOT be removed: this computer's key file could not be written${saveError ? ` (${saveError})` : ""}, so that sub-account's API key is still in it and will be back at the next start. Check storage permissions on the GHL Command app-data folder and run unregister_location again.`
+          }],
+          isError: true
+        };
+      }
       return {
         content: [{ type: "text", text: `Removed: ${token2.name} (${locationId2}) from token registry.` }]
       };
@@ -40513,11 +40710,13 @@ ${lines.join("\n")}
       try {
         const agencyClient = new GHLClient({ apiKey: agencyKey });
         await agencyClient.delete(`/locations/${location_id}`, { params: { deleteTwilioAccount: delete_twilio ?? false } });
+        const hadKey = Boolean(registry2?.getToken(location_id));
         const wasRegistered = registry2?.removeLocation(location_id) ?? false;
+        const keyLine = wasRegistered ? " Removed from the token registry." : hadKey ? ` Its API key could NOT be removed from this computer's key file${registry2?.getLastSaveError() ? ` (${registry2.getLastSaveError()})` : ""}; the key is dead with the sub-account, but run unregister_location once the app-data folder is writable so it does not linger.` : "";
         return {
           content: [{
             type: "text",
-            text: `Sub-account ${location_id} permanently deleted.${wasRegistered ? " Removed from the token registry." : ""}`
+            text: `Sub-account ${location_id} permanently deleted.${keyLine}`
           }]
         };
       } catch (error) {
@@ -41781,6 +41980,28 @@ function buildContactSmartListBody(args) {
     locationId: args.locationId
   };
 }
+function findSmartListRecord(payload) {
+  const looksLikeRecord = (v) => {
+    if (!v || typeof v !== "object" || Array.isArray(v)) return false;
+    const o = v;
+    return "objectKey" in o || "filters" in o || "id" in o || "_id" in o || "name" in o;
+  };
+  if (looksLikeRecord(payload)) return payload;
+  if (payload && typeof payload === "object") {
+    for (const key of ["list", "smartList", "smartlist", "data", "record"]) {
+      const nested = payload[key];
+      if (looksLikeRecord(nested)) return nested;
+    }
+  }
+  return void 0;
+}
+function contactServiceNote(payload) {
+  const rec = findSmartListRecord(payload);
+  if (!rec) return void 0;
+  const reason2 = rec.objectKey === "contacts" ? 'this record carries objectKey "contacts"' : rec.objectKey === "opportunity" ? void 0 : !("filters" in rec) ? "this record came back with no `filters` key at all" : void 0;
+  if (!reason2) return void 0;
+  return `This id was read from GoHighLevel's OPPORTUNITY smart-list service (/lists/dynamic) and ${reason2}. Contact smart lists do not live here and the Contacts screen never shows a record created here \u2014 create one with create_contact_smart_list, read it with get_contact_smart_list, and delete it with delete_contact_smart_list.`;
+}
 function registerSmartListTools(server2, builderClient) {
   if (!builderClient) return;
   const client = builderClient;
@@ -41909,7 +42130,8 @@ ${await res.text()}`);
       try {
         const loc = locationId2 ?? client.locationId;
         const result = await smartListRequest("GET", `/${loc}/${listId}`);
-        return jsonResponse(result);
+        const note = contactServiceNote(result);
+        return note ? jsonResponse({ smart_list: result, note }) : jsonResponse(result);
       } catch (error) {
         return errorResponse(error);
       }
@@ -41917,10 +42139,10 @@ ${await res.text()}`);
   );
   server2.tool(
     "create_smart_list",
-    "Create a new smart list (dynamic filter list). Required: name + objectKey. Filters define the saved-search criteria \u2014 pass an empty array to create an empty list and add filters later via update_smart_list. The shape of filters/columns is opaque here; query an existing smart list with get_smart_list to see the format GHL expects.",
+    'Create a new OPPORTUNITY smart list (dynamic filter list). objectKey "contacts" is refused here \u2014 this service cannot hold a contact segment; use create_contact_smart_list for that. Required: name + objectKey. Filters define the saved-search criteria \u2014 pass an empty array to create an empty list and add filters later via update_smart_list. The shape of filters/columns is opaque here; query an existing smart list with get_smart_list to see the format GHL expects.',
     {
       name: import_zod59.z.string().describe("Display name for the smart list."),
-      objectKey: import_zod59.z.enum(OBJECT_KEYS).describe("Object type the list segments over. 'contacts' or 'opportunity'."),
+      objectKey: import_zod59.z.enum(OBJECT_KEYS).describe("Object type the list segments over. 'opportunity' \u2014 'contacts' is refused here and is created with create_contact_smart_list instead."),
       filters: import_zod59.z.array(import_zod59.z.record(import_zod59.z.string(), import_zod59.z.unknown())).optional().describe("Array of filter objects. Each object has fields like {field, operator, value} \u2014 exact shape varies by filter type. See get_smart_list on an existing list to learn the format."),
       columns: import_zod59.z.array(import_zod59.z.record(import_zod59.z.string(), import_zod59.z.unknown())).optional().describe("Array of column definitions for the smart list view in GHL UI. Each defines which contact/opportunity field shows as a column. Defaults to GHL's standard columns if omitted."),
       pipelineIds: import_zod59.z.array(import_zod59.z.string()).optional().describe("(opportunity objectKey only) Pipeline IDs to restrict this smart list to. Empty array = all pipelines."),
@@ -41929,6 +42151,11 @@ ${await res.text()}`);
     },
     async ({ name, objectKey, filters, columns, pipelineIds, defaultInPipelines, locationId: locationId2 }) => {
       try {
+        if (objectKey === "contacts") {
+          throw new Error(
+            "create_smart_list writes to GoHighLevel's opportunity smart-list service, which cannot hold a contact segment \u2014 create a contact smart list with create_contact_smart_list instead."
+          );
+        }
         const loc = locationId2 ?? client.locationId;
         const body = { name, objectKey };
         if (filters !== void 0) body.filters = filters;
@@ -41966,8 +42193,13 @@ ${await res.text()}`);
         if (Object.keys(body).length === 0) {
           throw new Error("Pass at least one of: name, filters, columns, pipelineIds, defaultInPipelines. Nothing to update.");
         }
+        let note;
+        try {
+          note = contactServiceNote(await smartListRequest("GET", `/${loc}/${listId}`));
+        } catch {
+        }
         const result = await smartListRequest("PUT", `/${loc}/${listId}`, body);
-        return jsonResponse(result);
+        return note ? jsonResponse({ updated: result, note }) : jsonResponse(result);
       } catch (error) {
         return errorResponse(error);
       }
@@ -41984,8 +42216,13 @@ ${await res.text()}`);
     async ({ listId, locationId: locationId2 }) => {
       try {
         const loc = locationId2 ?? client.locationId;
+        let note;
+        try {
+          note = contactServiceNote(await smartListRequest("GET", `/${loc}/${listId}`));
+        } catch {
+        }
         const result = await smartListRequest("DELETE", `/${loc}/${listId}`);
-        return jsonResponse(result);
+        return note ? jsonResponse({ deleted: result, note }) : jsonResponse(result);
       } catch (error) {
         return errorResponse(error);
       }
@@ -48965,7 +49202,10 @@ var init_verify_findings = __esm({
       tag: { path: (loc) => `/locations/${loc}/tags`, ids: (raw) => idsOf(raw, "tags"), paged: true },
       calendar: { path: () => "/calendars/", params: byLocation, ids: (raw) => idsOf(raw, "calendars") },
       form: { path: () => "/forms/", params: byLocation, ids: (raw) => idsOf(raw, "forms"), paged: true },
-      user: { path: () => "/users/", params: byLocation, ids: (raw) => idsOf(raw, "users") }
+      user: { path: () => "/users/", params: byLocation, ids: (raw) => idsOf(raw, "users") },
+      // The list get_surveys reads; it takes limit/skip, so it is read the way
+      // forms are, page by page and never assumed complete (3.84.2).
+      survey: { path: () => "/surveys/", params: byLocation, ids: (raw) => idsOf(raw, "surveys"), paged: true }
     };
     CHECKABLE_KIND_NAMES = Object.keys(CHECKABLE_KINDS).join(", ");
     neverLookedAt = (id) => `"${id}" was never looked at. A step aimed at an unverified id silently kills every action after it, so this is refused rather than attempted.`;
@@ -49332,12 +49572,13 @@ function scanArgs(args, maxNodes = MAX_ARG_NODES) {
       continue;
     }
     if (!isRecord10(v)) continue;
-    const nodeType = typeof v.type === "string" && v.type ? v.type : parentType;
+    const isNode = typeof v.type === "string" && v.type && (isRecord10(v.attributes) || Array.isArray(v.conditions));
+    const nodeType = isNode ? v.type : parentType;
     const refs = [];
     if (typeof v.type === "string" && isRecord10(v.attributes)) extractFromAction(v, refs);
     if (Array.isArray(v.conditions)) extractFromTrigger(v, refs);
     for (const r of refs) {
-      if (r.byKey) continue;
+      if (r.byKey || !r.id.trim()) continue;
       const kind = AUDIT_KIND[r.kind];
       if (kind) out.ids.push({ kind, id: r.id, key: r.where });
       else out.uncheckable.push({ kind: r.kind, id: r.id, key: r.where });
@@ -49410,6 +49651,8 @@ var init_arg_ids = __esm({
       formId: "form",
       formIds: "form",
       // a form_submission goal's extras (build_goal_event)
+      surveyId: "survey",
+      // get_surveys reads /surveys/, so a survey is read back like a form (3.84.2)
       userId: "user",
       userIds: "user",
       assignedUserId: "user",
@@ -49424,6 +49667,12 @@ var init_arg_ids = __esm({
       ...ID_ARG_KINDS,
       "form.id": "form",
       "calendar.id": "calendar",
+      // The payment and task triggers name their form, calendar and assignee under
+      // their own prefixes (src/trigger-schemas.ts); unread until 3.84.2.
+      "payment.form.id": "form",
+      "payment.calendar.id": "calendar",
+      "task.assignedTo": "user",
+      "survey.id": "survey",
       "opportunity.pipelineId": "pipeline",
       "opportunity.pipelineStageId": "pipeline_stage",
       "opportunity.assignedTo": "user",
@@ -49438,7 +49687,7 @@ var init_arg_ids = __esm({
       workflow: "workflow",
       form: "form",
       calendar: "calendar",
-      survey: null
+      survey: "survey"
     });
     NOT_CHECKABLE_ID_ARGS = Object.freeze({
       // The account itself, not an object inside it. One account per plan is
@@ -49499,7 +49748,6 @@ var init_arg_ids = __esm({
       linkId: "trigger links have no read-back list here",
       blogId: "blogs have no read-back list here",
       categoryId: "categories have no read-back list here",
-      surveyId: "surveys have no read-back list here",
       submissionId: "form submissions have no read-back list here",
       mediaId: "media files have no read-back list here",
       photoAssetId: "media files have no read-back list here",
@@ -49556,6 +49804,10 @@ var init_arg_ids = __esm({
 // src/tools/agency-build.ts
 function accountMismatch(a, active) {
   if (!a.locationId) {
+    const applied = a.applied.filter((s) => s.state === "applied").length;
+    if (applied > 0) {
+      return `This build was saved before builds recorded the sub-account their ids were checked against, and ${applied} of its ${a.steps.length} steps ${applied === 1 ? "is" : "are"} already applied, so nothing can say which account they belong to and re-planning it whole would run those steps twice. Keep the undo notes you have, run abandon_automation to list what is live, then plan only the remaining step${a.steps.length - applied === 1 ? "" : "s"} with plan_automation and approve that plan.`;
+    }
     return "This build was saved before builds recorded the sub-account their ids were checked against, so nothing can say which account they belong to: plan it again (plan_automation) and approve the new plan.";
   }
   if (active === a.locationId) return null;
@@ -50235,7 +50487,7 @@ function dailyBrief(input) {
       );
     }
   }
-  const quiet = !unread.length && !halfRead.length && !today2.length && !waitingOnYou.length && !runningNow.length && !neverStarted.length && !renewals.length && !underFloor.length && !newLeads.length && !payingButQuiet.length && !floorUncheckable.length && !renewalUnknown.length && !yourDay.length;
+  const quiet = !unread.length && !halfRead.length && !today2.length && !waitingOnYou.length && !runningNow.length && !neverStarted.length && !renewals.length && !underFloor.length && !newLeads.length && !payingButQuiet.length && !floorUncheckable.length && !renewalUnknown.length && !yourDay.length && !yourDayNote;
   let headline3;
   if (today2.length === 1) {
     headline3 = `You have one call today \u2014 ${today2[0].who ?? today2[0].client} at ${today2[0].at}.`;
@@ -50266,6 +50518,8 @@ function dailyBrief(input) {
     headline3 = `Nothing needs you in what I could read \u2014 but ${unread[0].client} could not be read this morning.`;
   } else if (unread.length > 1) {
     headline3 = `Nothing needs you in what I could read \u2014 but ${unread.length} accounts could not be read this morning.`;
+  } else if (yourDayNote) {
+    headline3 = "Nothing needs you in what I could read \u2014 but your own calendar could not be read this morning.";
   } else if (floorUncheckable.length) {
     const c = floorUncheckable[0].client;
     headline3 = floorUncheckable.length === 1 ? `Nothing needs you \u2014 but ${c}'s price cannot be checked against any floor, so nothing here can tell you whether they are under it.` : `Nothing needs you \u2014 but ${floorUncheckable.length} clients' prices cannot be checked against any floor, ${c} among them.`;
@@ -50916,6 +51170,45 @@ function translateWorkflowAudit(audit, inputs = {}) {
       evidence: [{ tool: "audit_workflows", detail: `${s.shape_warnings_total ?? shapeCount} steps` }]
     });
   }
+  const blockerCount = s.workflows_with_save_blocker_warnings ?? 0;
+  if (blockerCount > 0) {
+    const rows2 = audit.save_blocker_warnings ?? [];
+    const names = rows2.map((r) => r.workflowName ?? "").filter(Boolean);
+    const cats = new Set(rows2.flatMap((r) => (r.warnings ?? []).map((w) => w.category ?? "")).filter(Boolean));
+    const editBlocked = cats.has("legacy_step_id") || cats.has("legacy_wait_unit");
+    const alertsNobody = cats.has("missing_required_attribute");
+    const unknownKind = !editBlocked && !alertsNobody;
+    const editSentence = ` Some of their steps were built before GoHighLevel tightened what it accepts. The automation runs exactly as it always has; what breaks is CHANGING it \u2014 the save is refused, and the refusal names steps nobody touched, so it reads as unrelated to the edit that triggered it.`;
+    const alertSentence = ` One or more alert steps are set to notify a specific person with nobody chosen. That step saves, publishes and runs, and simply never reaches anyone \u2014 so the automation looks healthy while the person waiting on the alert hears nothing.`;
+    out.push({
+      id: "steps-that-block-saving",
+      title: alertsNobody && !editBlocked ? `${blockerCount} automation${blockerCount === 1 ? " sends" : "s send"} an alert that reaches nobody` : editBlocked && !alertsNobody ? `${blockerCount} automation${blockerCount === 1 ? " cannot" : "s cannot"} be changed without the save being refused` : `${blockerCount} automation${blockerCount === 1 ? " carries a step" : "s carry steps"} that block changes, or alert nobody`,
+      what: `${nameList(names, blockerCount)} ${blockerCount === 1 ? "has" : "have"} at least one step of this kind.` + (editBlocked || unknownKind ? editSentence : "") + (alertsNobody || unknownKind ? alertSentence : ""),
+      severity: "high",
+      impact: alertsNobody && !editBlocked ? "leads" : "efficiency",
+      money: { confidence: "unknown", assumptions: ["Not costed: what this costs depends on which automation and who was waiting on the alert, and we would rather say so than guess."] },
+      remedy: alertsNobody && !editBlocked ? "Pick the person each alert should go to, or set it to notify everyone. Then re-run this check." : editBlocked && !alertsNobody ? "Rebuild those steps so GoHighLevel will accept a save again \u2014 nobody part-way through an automation can be moved while that is done. Then re-run this check." : "Rebuild those steps so GoHighLevel will accept a save again \u2014 nobody part-way through an automation can be moved while that is done \u2014 and pick a person for any alert with nobody selected. Then re-run this check.",
+      fix: "guided",
+      minutes: Math.max(10, blockerCount * 8),
+      evidence: [{ tool: "audit_workflows", detail: `${s.save_blocker_warnings_total ?? blockerCount} steps across ${blockerCount} workflows` }]
+    });
+  }
+  const integrationCount = s.workflows_with_integration_warnings ?? 0;
+  if (integrationCount > 0) {
+    const names = (audit.integration_warnings ?? []).map((r) => r.workflowName ?? "").filter(Boolean);
+    out.push({
+      id: "steps-without-their-account",
+      title: `${integrationCount} automation${integrationCount === 1 ? " has a step" : "s have steps"} waiting on a social account that is not connected`,
+      what: `${nameList(names, integrationCount)} include a step that messages or replies through a social account this location has not connected. The automation saves, publishes and runs; that step simply does nothing when it is reached, and nothing in the history says so.`,
+      severity: "high",
+      impact: "leads",
+      money: { confidence: "unknown", assumptions: ["Not costed: it depends how many people reach that step, which we cannot read."] },
+      remedy: "Connect the account in Settings, then re-run this check. (A connected account with no messaging permission fails the same way and this check cannot see it, so send one test message through the step as well.)",
+      fix: "manual",
+      minutes: Math.max(10, integrationCount * 5),
+      evidence: [{ tool: "audit_workflows", detail: `${s.integration_warnings_total ?? integrationCount} steps` }]
+    });
+  }
   return out;
 }
 function notCheckedFrom(audit) {
@@ -51655,6 +51948,15 @@ 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("; ");
+    const inert = r.publishFailed.filter((p) => p.kind === "inert"), unverified = r.publishFailed.filter((p) => p.kind === "not_verified");
+    const drafts = r.publishFailed.filter((p) => p.kind !== "inert" && p.kind !== "not_verified");
+    if (!drafts.length) {
+      const parts = [
+        inert.length ? `${inert.length} workflow(s) PUBLISHED but with every trigger still inactive, so nothing enrols contacts into them: ${inert.map((p) => `${p.name} (${p.error})`).join("; ")}. Open each in GHL and check its trigger.` : "",
+        unverified.length ? `${unverified.length} workflow(s) had the publish accepted but it could not be confirmed: ${unverified.map((p) => `${p.name} (${p.error})`).join("; ")}. Open each in GHL and check it shows Published before reporting it as running.` : ""
+      ].filter(Boolean).join(" ");
+      return `${lead}${parts}${r.published.length ? " Any gated workflow stays DRAFT until its handoff is met." : ""}`;
+    }
     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) {
@@ -51668,6 +51970,8 @@ function publishNote(r) {
 function publishSummaryClause(r) {
   if (r.publishFailed.length) {
     const also = r.published.length ? ` (${r.published.length} did publish)` : "";
+    const drafts = r.publishFailed.filter((p) => p.kind !== "inert" && p.kind !== "not_verified").length;
+    if (!drafts) return ` ${r.publishFailed.length} workflow(s) published but NOT confirmed running${also} (inactive trigger or unconfirmed publish), see publishNote.`;
     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.`;
@@ -52864,8 +53168,14 @@ async function executeBackbone(plan, deps, opts = {}) {
     }
     if (opts.publishWorkflows && !opts.gatedWorkflowRefs?.has(wf.ref)) {
       try {
-        await deps.publishWorkflow(wfId);
-        published.push(wf.ref);
+        const verdict = publishVerdict(await deps.publishWorkflow(wfId));
+        if (!verdict.live) {
+          publishFailed.push({ ref: wf.ref, name: wf.name, error: verdict.reason, kind: verdict.kind });
+          manual.push({ ref: wf.ref, type: "workflow-publish", name: wf.name, reason: `[${wf.name}] ${verdict.kind === "inert" ? `published, but every trigger on it still reads inactive, so nothing enrols contacts into it \u2014 open the workflow in GHL and check its trigger (${verdict.reason})` : verdict.kind === "not_verified" ? `the publish was accepted but could not be confirmed (${verdict.reason}) \u2014 open the workflow in GHL and check it shows Published` : `built successfully but not live (${verdict.reason}) \u2014 publish it manually in GHL`}.` });
+        } else {
+          published.push(wf.ref);
+          if (verdict.note) manual.push({ ref: wf.ref, type: "workflow-trigger", name: wf.name, reason: `[${wf.name}] published, but ${verdict.note}` });
+        }
       } 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.` });
@@ -52985,6 +53295,21 @@ async function createWithReadback(create, read2, name, beforeIds, poll, wait) {
 function msg2(e) {
   return accountErrorText(e);
 }
+function publishVerdict(outcome) {
+  if (typeof outcome !== "object" || outcome === null) return { live: true };
+  const check = outcome.publish_check;
+  if (typeof check !== "object" || check === null) return { live: true };
+  const c = check;
+  const warning = typeof c.warning === "string" && c.warning ? c.warning : "";
+  if (warning.startsWith("NOT VERIFIED")) return { live: false, kind: "not_verified", reason: warning };
+  if (warning.startsWith("TRIGGER NOT ACTIVE")) {
+    const active = typeof c.triggers_active === "number" ? c.triggers_active : 0;
+    return active > 0 ? { live: true, note: warning } : { live: false, kind: "inert", reason: warning };
+  }
+  if (c.published === true) return { live: true };
+  const status = typeof c.status_returned === "string" ? c.status_returned : "(GHL returned no status)";
+  return { live: false, kind: "not_live", reason: warning || `NOT LIVE. The workflow read back as ${status} rather than "published", so it is not running.` };
+}
 var norm5, sleep5, PREVIEW_SNIPPET_LENGTH, JournalHookError;
 var init_execute = __esm({
   "src/intake-to-build/execute.ts"() {
@@ -53549,9 +53874,12 @@ function makeExecuteDeps(client, builderClient, locationId2, registry2) {
     deleteWorkflow: async (workflowId) => {
       await builderClient.deleteWorkflow(workflowId);
     },
-    publishWorkflow: async (workflowId) => {
-      await builderClient.publishWorkflow(workflowId);
-    }
+    // The result is RETURNED, not awaited-and-dropped. It carries the client's
+    // own `publish_check` — the status read back from the SAVED workflow after
+    // the write, with a NOT LIVE / NOT VERIFIED sentence when the publish did
+    // not land. Discarding it is what let a workflow left in Draft be counted
+    // into "Published N workflow(s) live"; the executor now reads it.
+    publishWorkflow: (workflowId) => builderClient.publishWorkflow(workflowId)
   };
 }
 function makeRevertDeps(client, builderClient, locationId2) {
@@ -53759,7 +54087,7 @@ function registerIntakeToBuildTools(server2, client, builderClient, registry2) {
           if (saved2?.wholePreset) {
             scopeSource = "cockpit";
           } else if (!saved2 && Array.isArray(board.assessmentPurchases?.[activeLocation])) {
-            return jsonResponse({ ok: false, phase: "scope", scopeSource: "cockpit", error: NOT_CONFIRMED_SCOPE });
+            return jsonResponse({ ok: false, phase: "scope", scopeSource: "cockpit", error: notConfirmedAsk(activeLocation) });
           } else if (saved2 && Array.isArray(saved2.features)) {
             const features = saved2.features.filter(isFeatureId);
             if (!features.length) {
@@ -53817,11 +54145,11 @@ function registerIntakeToBuildTools(server2, client, builderClient, registry2) {
       if (!gate2.ok) return jsonResponse({ ok: false, phase: "plan", error: gate2.error, savedPlan: saved ? { planId: saved.planId, savedAt: saved.savedAt } : void 0 });
       const plan = gate2.plan;
       const planSource = gate2.source;
-      if (planSource === "saved") {
+      if (!fromPreset) {
         const board = readBoardState();
         const current = board.scope?.[activeLocation];
         if (!current && Array.isArray(board.assessmentPurchases?.[activeLocation])) {
-          return jsonResponse({ ok: false, phase: "scope", scopeSource: "cockpit", error: NOT_CONFIRMED_SCOPE });
+          return jsonResponse({ ok: false, phase: "scope", scopeSource: "cockpit", error: notConfirmedAsk(activeLocation) });
         }
         if (current && !current.wholePreset && Array.isArray(current.features) && !current.features.some(isFeatureId)) {
           return jsonResponse({
@@ -53831,7 +54159,7 @@ function registerIntakeToBuildTools(server2, client, builderClient, registry2) {
             error: "No features are selected for this client, so nothing was built, not even from the saved plan. Open the client's Review screen, tick the features to build, confirm, and run the build again."
           });
         }
-        if (!savedPlanMatchesScope(saved, current)) {
+        if (saved && planSource !== "argument-replaces-saved" && !savedPlanMatchesScope(saved, current)) {
           const planned = saved?.scope ? [...saved.scope.built, ...(saved.scope.omitted ?? []).map((f) => f.id)].join(", ") || "no features" : "everything in the preset";
           const now = current?.wholePreset ? "everything in the preset (confirmed)" : current ? current.features.filter(isFeatureId).join(", ") : "everything in the preset (nothing is saved)";
           return jsonResponse({
@@ -54523,7 +54851,7 @@ function intakeFormMatches(expected, actual) {
   const keys = ["id", "tag", "type", "hiddenFieldQueryKey", "required", "label", "options", "picklistOptions"];
   return expected.every((field, i) => keys.every((key) => field[key] === void 0 || JSON.stringify(field[key]) === JSON.stringify(actual[i]?.[key])));
 }
-var import_zod81, customFieldItemSchema, sleep6, NOT_CONFIRMED_SCOPE;
+var import_zod81, customFieldItemSchema, sleep6, NOT_CONFIRMED_SCOPE, WHOLE_PRESET_NOT_KEPT, notConfirmedAsk;
 var init_intake_to_build = __esm({
   "src/tools/intake-to-build.ts"() {
     "use strict";
@@ -54570,6 +54898,12 @@ var init_intake_to_build = __esm({
     }).passthrough();
     sleep6 = (ms) => new Promise((r) => setTimeout(r, ms));
     NOT_CONFIRMED_SCOPE = "What this client bought is recorded, but nobody has confirmed what to build, so nothing was built. Open the client's Review screen: the features are ticked from what they bought. Change them if you need to, save to confirm, and run the build again.";
+    WHOLE_PRESET_NOT_KEPT = "Note: an earlier build on this client used everything in the preset, and that choice was not kept, so the Review screen ticks only what they bought. Tick everything again if that is still what you want.";
+    notConfirmedAsk = (locationId2) => {
+      const sp = readSavedPlan(locationId2);
+      const fromPreset = typeof sp?.plan?.preset === "string";
+      return sp && !sp.scope && fromPreset ? `${NOT_CONFIRMED_SCOPE} ${WHOLE_PRESET_NOT_KEPT}` : NOT_CONFIRMED_SCOPE;
+    };
   }
 });
 

package/package.json

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