@elitedcs/ghl-mcp 3.65.1 → 3.65.2

package/CHANGELOG.md

@@ -1,5 +1,43 @@
 # Changelog
 
+## 3.65.2 — invoicing never worked, and the guide stopped being one long page
+
+- **Fix: creating an invoice failed every single time.** The call sent five
+  fields; GoHighLevel needs eleven, and it names line-item amounts differently
+  than we did. Given a payload it could not read, their server crashed rather
+  than telling us what was missing, so this looked like an outage on their side
+  rather than a bug on ours. It has never worked for anyone. Invoices now carry
+  the currency, the dates, your business name and the client's own details, and
+  are created as real payable drafts unless you ask for a test one.
+
+**The user guide is a library again, not a scroll.**
+
+- **Cards open their own page.** The guide had drifted into a single document
+  that printed the index and then every guide beneath it, growing by one whole
+  guide each release. Opening a card now shows that guide on its own, from the
+  top, and the back button works.
+- **The category menu is back**, above the cards.
+- **It stays light.** The guide used to follow your computer into dark mode,
+  which is wrong for something people read at length, print, and save as a PDF.
+- **It prints properly now.** One guide per job, no navigation, no buttons, no
+  index bleeding into the page.
+- Fixed a layout bug that had been chopping numbered steps into narrow columns
+  whenever a step contained a command. The install guide's first step was the
+  worst hit, which is the first thing a new customer reads.
+
+**Three new guides, each proven live before shipping:**
+
+- **What your AI bot knows, and where to put it.** Custom values resolve in a
+  bot's prompt and are silently ignored in its knowledge base. Asked a question
+  whose answer sat behind an unresolved token, a test bot named a nearby
+  sentence as the person responsible. Worse, if the same fact also sits in the
+  prompt the bot quietly repairs the answer, so the bug passes exactly the test
+  you would run.
+- **Bill a client and see what you were paid.** Invoices, products, coupons, and
+  the difference between money invoiced and money received.
+- **Reuse a build across client accounts.** Snapshots, and the honest list of
+  what does not travel with one.
+
 ## 3.65.1 — the guide caught up with the product
 
 No code changes. The built-in user guide ships inside the install, so it goes

package/dist/index.js

@@ -2854,7 +2854,7 @@ var require_package = __commonJS({
   "package.json"(exports2, module2) {
     module2.exports = {
       name: "@elitedcs/ghl-mcp",
-      version: "3.65.1",
+      version: "3.65.2",
       mcpName: "io.github.drjerryrelth/ghl-command",
       description: "GoHighLevel MCP Server for Claude. 238 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",
@@ -8648,6 +8648,38 @@ function registerProductTools(server2, client) {
 
 // src/tools/invoices.ts
 var import_zod20 = require("zod");
+function buildCreateInvoiceBody(args, resolvedLocationId, contact, businessName, today) {
+  const currency = args.currency ?? args.items[0]?.currency ?? "USD";
+  return {
+    altId: resolvedLocationId,
+    altType: "location",
+    name: args.name,
+    title: args.title ?? args.name,
+    currency,
+    // GHL's line items are `amount` and `qty`. Sending price/quantity leaves
+    // both undefined on their side, which is the crash.
+    items: args.items.map((i) => ({
+      name: i.name,
+      description: i.description ?? "",
+      currency: i.currency ?? currency,
+      amount: i.price,
+      qty: i.quantity
+    })),
+    discount: args.discount ?? { type: "percentage", value: 0 },
+    termsNotes: args.termsNotes ?? "",
+    businessDetails: { name: businessName },
+    contactDetails: {
+      id: contact.id,
+      name: contact.name ?? "",
+      email: contact.email ?? "",
+      phoneNo: contact.phone ?? ""
+    },
+    issueDate: args.issueDate ?? today,
+    dueDate: args.dueDate ?? args.issueDate ?? today,
+    // Real, payable invoice unless the caller explicitly asks for a test one.
+    liveMode: args.liveMode ?? true
+  };
+}
 function registerInvoiceTools(server2, client) {
   safeTool(
     server2,
@@ -8713,30 +8745,47 @@ function registerInvoiceTools(server2, client) {
       contactId: import_zod20.z.string().describe("Contact ID for the invoice"),
       items: import_zod20.z.array(import_zod20.z.object({
         name: import_zod20.z.string().describe("Item name"),
-        description: import_zod20.z.string().describe("Item description"),
+        description: import_zod20.z.string().optional().describe("Item description"),
         quantity: import_zod20.z.number().describe("Item quantity"),
-        price: import_zod20.z.number().describe("Item price"),
-        currency: import_zod20.z.string().describe("Currency code (e.g. USD)")
-      })).describe("Invoice line items"),
+        price: import_zod20.z.number().describe("Unit price, in the invoice currency."),
+        currency: import_zod20.z.string().optional().describe("Currency code (e.g. USD). Defaults to the invoice currency.")
+      })).min(1).describe("Invoice line items"),
+      currency: import_zod20.z.string().optional().describe("Invoice currency code, e.g. 'USD'. Defaults to the first item's currency, then USD."),
       discount: import_zod20.z.object({
         type: import_zod20.z.string().optional().describe("Discount type: 'percentage' or 'fixed'."),
         value: import_zod20.z.number().optional().describe("Discount amount: percent (0-100) when type=percentage, currency amount when type=fixed.")
       }).optional().describe("Discount to apply"),
       termsNotes: import_zod20.z.string().optional().describe("Terms and notes for the invoice"),
-      title: import_zod20.z.string().optional().describe("Invoice title")
+      title: import_zod20.z.string().optional().describe("Invoice title. Defaults to the invoice name."),
+      issueDate: import_zod20.z.string().optional().describe("Issue date, YYYY-MM-DD. Defaults to today."),
+      dueDate: import_zod20.z.string().optional().describe("Due date, YYYY-MM-DD. Defaults to the issue date."),
+      liveMode: import_zod20.z.boolean().optional().describe("true (default) creates a real, payable invoice. false creates a test-mode invoice that cannot be paid.")
     },
-    async ({ locationId: locationId2, name, contactId, items, discount, termsNotes, title }) => {
+    async ({ locationId: locationId2, name, contactId, items, currency, discount, termsNotes, title, issueDate, dueDate, liveMode }) => {
       const resolvedLocationId = client.resolveLocationId(locationId2);
-      const body = {
-        altId: resolvedLocationId,
-        altType: "location",
-        name,
-        contactId,
-        items
+      const contactRes = await client.get(`/contacts/${contactId}`);
+      const c = contactRes?.contact ?? contactRes ?? {};
+      const contact = {
+        id: contactId,
+        name: [c.firstName, c.lastName].filter(Boolean).join(" ") || c.name || "",
+        email: c.email ?? "",
+        phone: c.phone ?? ""
       };
-      if (discount !== void 0) body.discount = discount;
-      if (termsNotes !== void 0) body.termsNotes = termsNotes;
-      if (title !== void 0) body.title = title;
+      let businessName = "";
+      try {
+        const loc = await client.get(`/locations/${resolvedLocationId}`);
+        const l = loc?.location ?? loc ?? {};
+        businessName = l.name ?? "";
+      } catch {
+      }
+      const today = (/* @__PURE__ */ new Date()).toISOString().slice(0, 10);
+      const body = buildCreateInvoiceBody(
+        { name, contactId, items, currency, discount, termsNotes, title, issueDate, dueDate, liveMode },
+        resolvedLocationId,
+        contact,
+        businessName,
+        today
+      );
       return client.post("/invoices/", { body });
     }
   );