@elitedcs/ghl-mcp 3.64.0 → 3.64.1

package/CHANGELOG.md

@@ -1,5 +1,21 @@
 # Changelog
 
+## 3.64.1 — three more calls that failed quietly
+
+- **Fix: client pricing was mangled on the way into a build plan.** The Blueprint
+  intake parsed price answers by splitting on line breaks only, and read the
+  hyphen in a price range as the separator between a product and its price. An
+  answer typed as a sentence — "Contouring packages 1200-3500. Injectables
+  400-900." — became a single entry named "Contouring packages 1200" priced at
+  "3500", with the second offering discarded entirely, and the brief still
+  reported as valid. Prices now survive as ranges, sentences and semicolons
+  separate entries, and nothing is silently dropped.
+- **Fix: listing invoices always failed.** GoHighLevel requires both a limit and
+  an offset on that endpoint and rejects the call without them, so the ordinary
+  "show me my invoices" request never worked. Both are now always sent.
+- **Fix: blog authors and categories always failed**, for the same reason and
+  with the same fix. The paths were never wrong — only the request was.
+
 ## 3.64.0 — four silent failures fixed, four more guides open
 
 Every fix here is the same shape: a call that succeeded, returned

package/dist/index.js

@@ -2854,7 +2854,7 @@ var require_package = __commonJS({
   "package.json"(exports2, module2) {
     module2.exports = {
       name: "@elitedcs/ghl-mcp",
-      version: "3.64.0",
+      version: "3.64.1",
       mcpName: "io.github.drjerryrelth/ghl-command",
       description: "GoHighLevel MCP Server for Claude. 235 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",
@@ -8606,11 +8606,11 @@ function registerInvoiceTools(server2, client) {
   safeTool(
     server2,
     "list_invoices",
-    "List invoices for a location. Offset-based pagination via limit/offset.",
+    "List invoices for a location. Offset-based pagination via limit/offset; both are always sent because GHL requires them.",
     {
       locationId: import_zod20.z.string().optional().describe("GHL Location ID (optional if GHL_LOCATION_ID is set)"),
-      limit: import_zod20.z.number().optional().describe("Max number of invoices to return"),
-      offset: import_zod20.z.number().optional().describe("Offset for pagination"),
+      limit: import_zod20.z.number().min(1).optional().describe("Max number of invoices to return (must be at least 1). Defaults to 10."),
+      offset: import_zod20.z.number().min(0).optional().describe("Offset for pagination. Defaults to 0."),
       status: import_zod20.z.enum(["draft", "sent", "paid", "void", "partially_paid"]).optional().describe("Filter by invoice status"),
       contactId: import_zod20.z.string().optional().describe("Filter by contact ID"),
       startAt: import_zod20.z.string().optional().describe("Start date filter (ISO 8601)"),
@@ -8624,8 +8624,8 @@ function registerInvoiceTools(server2, client) {
         altId: resolvedLocationId,
         altType: "location"
       };
-      if (limit !== void 0) params.limit = limit;
-      if (offset !== void 0) params.offset = offset;
+      params.limit = limit ?? 10;
+      params.offset = offset ?? 0;
       if (status !== void 0) params.status = status;
       if (contactId !== void 0) params.contactId = contactId;
       if (startAt !== void 0) params.startAt = startAt;
@@ -9214,28 +9214,36 @@ function registerBlogTools(server2, client) {
   safeTool(
     server2,
     "get_blog_authors",
-    "List blog authors for a location",
+    "List blog authors for a location.",
     {
-      locationId: import_zod27.z.string().optional().describe("GHL Location ID (optional if GHL_LOCATION_ID is set)")
+      locationId: import_zod27.z.string().optional().describe("GHL Location ID (optional if GHL_LOCATION_ID is set)"),
+      limit: import_zod27.z.number().min(1).max(50).optional().describe("How many authors to return (1-50). Defaults to 50."),
+      offset: import_zod27.z.number().min(0).optional().describe("How many to skip. Defaults to 0.")
     },
-    async ({ locationId: locationId2 }) => {
+    // limit and offset are REQUIRED by GHL even though they read as optional
+    // paging: omitting them returns 422 "limit should not be empty", which
+    // surfaced as "blog authors are broken" (live-verified 2026-08-08).
+    async ({ locationId: locationId2, limit, offset }) => {
       const resolvedLocationId = client.resolveLocationId(locationId2);
       return client.get("/blogs/authors", {
-        params: { locationId: resolvedLocationId }
+        params: { locationId: resolvedLocationId, limit: limit ?? 50, offset: offset ?? 0 }
       });
     }
   );
   safeTool(
     server2,
     "get_blog_categories",
-    "List blog categories for a location",
+    "List blog categories for a location.",
     {
-      locationId: import_zod27.z.string().optional().describe("GHL Location ID (optional if GHL_LOCATION_ID is set)")
+      locationId: import_zod27.z.string().optional().describe("GHL Location ID (optional if GHL_LOCATION_ID is set)"),
+      limit: import_zod27.z.number().min(1).max(50).optional().describe("How many categories to return (1-50). Defaults to 50."),
+      offset: import_zod27.z.number().min(0).optional().describe("How many to skip. Defaults to 0.")
     },
-    async ({ locationId: locationId2 }) => {
+    // Same required-paging contract as /blogs/authors — see the note there.
+    async ({ locationId: locationId2, limit, offset }) => {
       const resolvedLocationId = client.resolveLocationId(locationId2);
       return client.get("/blogs/categories", {
-        params: { locationId: resolvedLocationId }
+        params: { locationId: resolvedLocationId, limit: limit ?? 50, offset: offset ?? 0 }
       });
     }
   );
@@ -15956,18 +15964,20 @@ function splitList(text) {
 }
 function parsePricePoints(text) {
   const out = [];
-  for (const line of text.split(/\r?\n/)) {
-    const t = line.trim();
+  const segments = text.split(/\r?\n|;|\.\s+/);
+  const ENTRY = /^(.*?)\s*[-–—:|]?\s*\$?\s*([\d,]+(?:\.\d+)?)\s*(?:[-–—]\s*\$?\s*([\d,]+(?:\.\d+)?))?\s*$/;
+  const clean = (s) => s.replace(/[\s.,:;|–—-]+$/, "").trim();
+  const num = (s) => s.replace(/,/g, "");
+  for (const segment of segments) {
+    const t = segment.trim().replace(/\.$/, "");
     if (!t) continue;
-    const m = t.match(/^(.*?)[\s]*[-—:|]\s*\$?\s*([\d,]+(?:\.\d+)?)/);
-    if (m) {
-      out.push({ name: m[1].trim(), price: m[2].replace(/,/g, "") });
+    const m = t.match(ENTRY);
+    if (m && m[2]) {
+      const name = clean(m[1]);
+      const price = m[3] ? `${num(m[2])}-${num(m[3])}` : num(m[2]);
+      out.push({ name: name || t, price });
     } else {
-      const priceOnly = t.match(/\$?\s*([\d,]+(?:\.\d+)?)\s*$/);
-      out.push({
-        name: priceOnly ? t.slice(0, priceOnly.index).trim() || t : t,
-        price: priceOnly ? priceOnly[1].replace(/,/g, "") : ""
-      });
+      out.push({ name: t, price: "" });
     }
   }
   return out;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@elitedcs/ghl-mcp",
-  "version": "3.64.0",
+  "version": "3.64.1",
   "mcpName": "io.github.drjerryrelth/ghl-command",
   "description": "GoHighLevel MCP Server for Claude. 235 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",