@elitedcs/ghl-mcp 3.34.9 → 3.35.0

package/CHANGELOG.md

@@ -1,5 +1,52 @@
 # Changelog
 
+## 3.35.0 — `create_social_post` is now draft-first (closes the silent "everything published at once" trap)
+
+**Behavior change.** `create_social_post` no longer defers go-live to GoHighLevel's
+server-side default. Before, when `status` was omitted the field was dropped from the
+request and GHL decided whether the post went live — an undocumented default that, in
+a loop, is exactly the "all posts fired at once, no error, no warning" failure operators
+hit. The most expensive failures are the silent ones, so the tool now takes a defensive
+stance by default.
+
+- **Draft-first default.** Omit `status` and the post is created as a `draft` (editable,
+  never live). Going live is now an explicit choice (`status:"published"`).
+- **Scheduling is coupled to status.** Pass `scheduledAt` with no `status` and it
+  auto-resolves to `"scheduled"`. Pass `scheduledAt` alongside any *other* status, or
+  `status:"scheduled"` with no `scheduledAt`, and the call is rejected with an actionable
+  error instead of silently publishing now or falling back to the server default. This
+  kills the "I thought it was scheduled" class of failure (CHANGELOG 3.18.0 always
+  required both fields; nothing enforced the pairing until now).
+- Logic extracted to the pure, tested `resolvePostStatus(status, scheduledAt)` helper;
+  tool and field descriptions updated to document the contract. 5 regression tests added.
+- **Migration:** any caller that relied on an un-statused post going live must now pass
+  `status:"published"` explicitly. No other tool is affected.
+
+## 3.34.10 — `get_free_slots` accepts ISO dates; `update_form` name fallback for fresh forms
+
+Two small fixes that close rough edges found while live-verifying v3.34.9 through
+the MCP tools. Neither changes a proven capability; both have trivial workarounds.
+
+- **`get_free_slots` now accepts ISO dates (its documented input) — not just epoch
+  milliseconds.** GHL's `GET /calendars/{id}/free-slots` requires `startDate` /
+  `endDate` as epoch **milliseconds** and 422s (`startDate must be a number…`) on an
+  ISO date, yet the tool's own schema documented `YYYY-MM-DD` and forwarded it
+  unconverted — so the documented input always failed. The tool now accepts a bare
+  `YYYY-MM-DD` (taken as start-of-day for `startDate`, end-of-day for `endDate`, in
+  the supplied `timezone`, default UTC), a full ISO datetime (`Date.parse`), or an
+  already-numeric epoch-millis value (passed through, back-compat) and converts to
+  the epoch ms GHL needs. Logic extracted to `buildFreeSlotsParams` / `toEpochMillis`
+  with tests (incl. the live Phoenix anchor `2026-06-15 → 1781506800000`).
+- **`update_form` no longer throws on the natural `create_form` → `update_form`
+  sequence.** When `name` is omitted, `update_form` reads the current name to
+  preserve it. A form freshly created by `create_form` has no `name` in its builder
+  doc until its first save, so that read returned none and the call threw
+  (`Could not resolve current form name`). It now falls back to the public forms list
+  (which carries the name) and only errors — with actionable guidance to pass `name`
+  — if both sources come up empty. The form-builder tools now also receive the public
+  `GHLClient` for this lookup (it already had Firebase auth for the builder routes).
+  Helpers `pickFormName` / `findFormNameInList` extracted, with tests.
+
 ## 3.34.9 — Calendar availability: create_calendar / update_calendar now set `openHours`
 
 Closes the on-camera "assign availability hours" gap. `create_calendar` and

package/dist/index.js

@@ -31,7 +31,7 @@ var require_package = __commonJS({
   "package.json"(exports2, module2) {
     module2.exports = {
       name: "@elitedcs/ghl-mcp",
-      version: "3.34.9",
+      version: "3.35.0",
       mcpName: "io.github.drjerryrelth/ghl-command",
       description: "GoHighLevel MCP Server for Claude. 218 tools \u2014 full CRM, automation, marketing control, account-wide workflow audit, and the only programmatic GHL workflow builder, now multi-tenant across client accounts.",
       main: "dist/index.js",
@@ -2779,6 +2779,61 @@ function buildUpdateCalendarBody(args) {
   applyCalendarFields(body, args);
   return body;
 }
+function tzOffsetMillis(utcMillis, timeZone) {
+  const dtf = new Intl.DateTimeFormat("en-US", {
+    timeZone,
+    hourCycle: "h23",
+    year: "numeric",
+    month: "2-digit",
+    day: "2-digit",
+    hour: "2-digit",
+    minute: "2-digit",
+    second: "2-digit"
+  });
+  const parts = dtf.formatToParts(new Date(utcMillis));
+  const get = (type) => Number(parts.find((p) => p.type === type)?.value);
+  const asUTC = Date.UTC(get("year"), get("month") - 1, get("day"), get("hour"), get("minute"), get("second"));
+  return asUTC - utcMillis;
+}
+function zonedStartOfDayMillis(year, month, day, timeZone) {
+  const utcGuess = Date.UTC(year, month - 1, day, 0, 0, 0, 0);
+  if (!timeZone) return utcGuess;
+  return utcGuess - tzOffsetMillis(utcGuess, timeZone);
+}
+function zonedDayBoundaryMillis(year, month, day, endOfDay, timeZone) {
+  if (endOfDay) return zonedStartOfDayMillis(year, month, day + 1, timeZone) - 1;
+  return zonedStartOfDayMillis(year, month, day, timeZone);
+}
+function toEpochMillis(value, opts = {}) {
+  const trimmed = value.trim();
+  if (/^\d+$/.test(trimmed)) {
+    if (trimmed.length === 13) return trimmed;
+    throw new Error(`Ambiguous numeric date "${value}" for free-slots \u2014 pass epoch MILLISECONDS (13 digits), a YYYY-MM-DD date, or a full ISO datetime.`);
+  }
+  const dateOnly = /^(\d{4})-(\d{2})-(\d{2})$/.exec(trimmed);
+  if (dateOnly) {
+    const millis = zonedDayBoundaryMillis(
+      Number(dateOnly[1]),
+      Number(dateOnly[2]),
+      Number(dateOnly[3]),
+      Boolean(opts.endOfDay),
+      opts.timeZone
+    );
+    return String(millis);
+  }
+  const parsed = Date.parse(trimmed);
+  if (!Number.isNaN(parsed)) return String(parsed);
+  throw new Error(`Invalid date "${value}" for free-slots \u2014 use YYYY-MM-DD, a full ISO datetime, or epoch milliseconds.`);
+}
+function buildFreeSlotsParams(args) {
+  const params = {
+    startDate: toEpochMillis(args.startDate, { timeZone: args.timezone }),
+    endDate: toEpochMillis(args.endDate, { endOfDay: true, timeZone: args.timezone })
+  };
+  if (args.timezone !== void 0) params.timezone = args.timezone;
+  if (args.userId !== void 0) params.userId = args.userId;
+  return params;
+}
 function registerCalendarTools(server2, client) {
   safeTool(
     server2,
@@ -2881,15 +2936,13 @@ function registerCalendarTools(server2, client) {
     "Get available free slots for a calendar",
     {
       calendarId: import_zod9.z.string().describe("The calendar ID"),
-      startDate: import_zod9.z.string().describe("Start date in ISO format (YYYY-MM-DD)"),
-      endDate: import_zod9.z.string().describe("End date in ISO format (YYYY-MM-DD)"),
+      startDate: import_zod9.z.string().describe("Start of the window. Accepts a date (YYYY-MM-DD), a full ISO datetime, or epoch milliseconds. A bare date is taken as start-of-day in `timezone` (UTC if unset). GHL's API needs epoch ms; the tool converts for you."),
+      endDate: import_zod9.z.string().describe("End of the window. Accepts a date (YYYY-MM-DD), a full ISO datetime, or epoch milliseconds. A bare date is taken as end-of-day in `timezone` (UTC if unset)."),
       timezone: import_zod9.z.string().optional().describe("Timezone (e.g. America/New_York)"),
       userId: import_zod9.z.string().optional().describe("Filter by user ID")
     },
     async ({ calendarId, startDate, endDate, timezone, userId }) => {
-      const params = { startDate, endDate };
-      if (timezone !== void 0) params.timezone = timezone;
-      if (userId !== void 0) params.userId = userId;
+      const params = buildFreeSlotsParams({ startDate, endDate, timezone, userId });
       return await client.get(`/calendars/${calendarId}/free-slots`, {
         params
       });
@@ -4001,6 +4054,21 @@ function registerMediaTools(server2, client) {
 
 // src/tools/social-planner.ts
 var import_zod21 = require("zod");
+function resolvePostStatus(status, scheduledAt) {
+  if (scheduledAt !== void 0 && status !== void 0 && status !== "scheduled") {
+    throw new Error(
+      `create_social_post: scheduledAt was provided but status is "${status}". A scheduled post requires status:"scheduled" (or omit status to auto-schedule).`
+    );
+  }
+  if (status === "scheduled" && scheduledAt === void 0) {
+    throw new Error(
+      'create_social_post: status:"scheduled" requires scheduledAt (an ISO publish time).'
+    );
+  }
+  if (status !== void 0) return status;
+  if (scheduledAt !== void 0) return "scheduled";
+  return "draft";
+}
 function registerSocialPlannerTools(server2, client) {
   safeTool(
     server2,
@@ -4072,7 +4140,7 @@ function registerSocialPlannerTools(server2, client) {
   safeTool(
     server2,
     "create_social_post",
-    "Create a new social media post",
+    'Create a social media post. Defaults to a draft (never publishes live) unless you pass status explicitly. To schedule, pass scheduledAt and the status auto-resolves to "scheduled". To post immediately, pass status:"published".',
     {
       locationId: import_zod21.z.string().optional().describe("GHL Location ID (optional if GHL_LOCATION_ID is set)"),
       accountIds: import_zod21.z.array(import_zod21.z.string()).describe("Array of social account IDs to post to"),
@@ -4080,8 +4148,12 @@ function registerSocialPlannerTools(server2, client) {
       summary: import_zod21.z.string().optional().describe("The post text/caption"),
       media: import_zod21.z.array(import_zod21.z.string()).optional().describe("Array of media URLs to attach"),
       type: import_zod21.z.enum(["post", "story", "reel"]).optional().describe("Post type (default post)"),
-      status: import_zod21.z.enum(["in_review", "scheduled", "draft", "published"]).optional().describe("Post status"),
-      scheduledAt: import_zod21.z.string().optional().describe("Scheduled publish time (ISO string)"),
+      status: import_zod21.z.enum(["in_review", "scheduled", "draft", "published"]).optional().describe(
+        'Post status. Omit to default to "draft" (safe \u2014 never goes live). Use "published" to post immediately, "scheduled" (with scheduledAt) to schedule, or "in_review" for an approval queue.'
+      ),
+      scheduledAt: import_zod21.z.string().optional().describe(
+        'Scheduled publish time (ISO string). When set, status auto-resolves to "scheduled" if you omit it; passing any other status alongside scheduledAt is rejected.'
+      ),
       tags: import_zod21.z.array(import_zod21.z.string()).optional().describe("Tags for the post"),
       ogData: import_zod21.z.object({
         title: import_zod21.z.string().optional(),
@@ -4119,7 +4191,7 @@ function registerSocialPlannerTools(server2, client) {
         };
         body.media = media.map((url) => ({ url, type: mimeFor(url) }));
       }
-      if (status !== void 0) body.status = status;
+      body.status = resolvePostStatus(status, scheduledAt);
       if (scheduledAt !== void 0) body.scheduleDate = scheduledAt;
       if (tags !== void 0) body.tags = tags;
       if (ogData !== void 0) body.ogData = ogData;
@@ -6007,13 +6079,34 @@ ${text2}`);
 
 // src/tools/form-builder.ts
 var import_zod35 = require("zod");
+function pickFormName(builderForm) {
+  if (builderForm && typeof builderForm === "object" && "form" in builderForm) {
+    const name = builderForm.form?.name;
+    if (typeof name === "string" && name.length > 0) return name;
+  }
+  return void 0;
+}
+function findFormNameInList(list, formId) {
+  const forms = list?.forms;
+  if (Array.isArray(forms)) {
+    for (const f of forms) {
+      if (f && typeof f === "object") {
+        const entry = f;
+        if ((entry.id === formId || entry._id === formId) && typeof entry.name === "string" && entry.name.length > 0) {
+          return entry.name;
+        }
+      }
+    }
+  }
+  return void 0;
+}
 function buildUpdateFormPath(formId, locationId2) {
   return `/${formId}?locationId=${locationId2}`;
 }
 function buildUpdateFormBody(name, formData) {
   return { name, formData };
 }
-function registerFormBuilderTools(server2, builderClient) {
+function registerFormBuilderTools(server2, builderClient, publicClient) {
   const client = builderClient;
   if (!client) return;
   async function formRequest(method, path7, body) {
@@ -6067,11 +6160,28 @@ ${text2}`);
         let resolvedName = name;
         if (resolvedName === void 0) {
           const current = await formRequest("GET", `/${formId}?locationId=${client.locationId}`);
-          const currentName = typeof current === "object" && current !== null && "form" in current ? current.form?.name : void 0;
-          if (typeof currentName !== "string") {
-            throw new Error(`Could not resolve current form name for ${formId} (GET /forms/${formId} returned no string name).`);
+          resolvedName = pickFormName(current);
+          if (resolvedName === void 0 && publicClient) {
+            try {
+              const pageSize = 100;
+              const maxForms = 5e3;
+              for (let skip = 0; skip < maxForms; skip += pageSize) {
+                const list = await publicClient.get("/forms/", {
+                  params: { locationId: client.locationId, limit: pageSize, skip }
+                });
+                resolvedName = findFormNameInList(list, formId);
+                if (resolvedName !== void 0) break;
+                const forms = list?.forms;
+                if (!Array.isArray(forms) || forms.length < pageSize) break;
+              }
+            } catch {
+            }
+          }
+          if (resolvedName === void 0) {
+            throw new Error(
+              `Could not resolve the current name for form ${formId}. Pass the \`name\` argument explicitly \u2014 a form created via create_form has no builder-doc name until its first save.`
+            );
           }
-          resolvedName = currentName;
         }
         const result = await formRequest(
           "POST",
@@ -9545,7 +9655,6 @@ var publicApiTools = [
 var internalApiTools = [
   [registerWorkflowBuilderTools, "workflow-builder"],
   [registerFunnelBuilderTools, "funnel-builder"],
-  [registerFormBuilderTools, "form-builder"],
   [registerPipelineBuilderTools, "pipeline-builder"],
   [registerWorkflowClonerTools, "workflow-cloner"],
   [registerSmartListTools, "smart-lists"],
@@ -9558,9 +9667,11 @@ var VALIDATORS_MODULE = "validators";
 var DIAGNOSTICS_MODULE = "diagnostics";
 var LOCATION_SWITCHER_MODULE = "location-switcher";
 var SNAPSHOTS_MODULE = "snapshots";
+var FORM_BUILDER_MODULE = "form-builder";
 var KNOWN_MODULES = /* @__PURE__ */ new Set([
   ...publicApiTools.map(([, label]) => label),
   ...internalApiTools.map(([, label]) => label),
+  FORM_BUILDER_MODULE,
   VALIDATORS_MODULE,
   DIAGNOSTICS_MODULE,
   LOCATION_SWITCHER_MODULE,
@@ -9578,6 +9689,7 @@ function registerAllTools(server2, client, registry2, mcpVersion, env = process.
   for (const [register, moduleName] of internalApiTools) {
     register(wrap(moduleName), builderClient);
   }
+  registerFormBuilderTools(wrap(FORM_BUILDER_MODULE), builderClient, client);
   registerValidatorTools(wrap(VALIDATORS_MODULE), client, builderClient);
   registerDiagnosticTools(
     wrap(DIAGNOSTICS_MODULE),

package/package.json

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