@integrity-labs/agt-cli 0.28.770 → 0.28.772

package/dist/bin/agt.js

@@ -41,7 +41,7 @@ import {
   success,
   table,
   warn
-} from "../chunk-E4YS6E4T.js";
+} from "../chunk-SBAX57SN.js";
 import {
   getProjectDir,
   isSessionResumeDisabled,
@@ -5467,7 +5467,7 @@ import { execFileSync, execSync } from "child_process";
 import { existsSync as existsSync11, realpathSync as realpathSync2 } from "fs";
 import chalk18 from "chalk";
 import ora16 from "ora";
-var cliVersion = true ? "0.28.770" : "dev";
+var cliVersion = true ? "0.28.772" : "dev";
 async function fetchLatestVersion() {
   const host2 = getHost();
   if (!host2) return null;
@@ -6658,7 +6658,7 @@ function handleError(err) {
 }
 
 // src/bin/agt.ts
-var cliVersion2 = true ? "0.28.770" : "dev";
+var cliVersion2 = true ? "0.28.772" : "dev";
 var program = new Command();
 program.name("agt").description("Augmented CLI \u2014 agent provisioning and management").version(cliVersion2).option("--json", "Emit machine-readable JSON output (suppress spinners and colors)").option("--skip-update-check", "Skip the automatic update check on startup");
 program.hook("preAction", async (thisCommand, actionCommand) => {

package/dist/{chunk-E4YS6E4T.js → chunk-SBAX57SN.js}

@@ -6452,7 +6452,7 @@ function exchangeFailureKind(err) {
 }
 
 // src/lib/api-client.ts
-var agtCliVersion = true ? "0.28.770" : "dev";
+var agtCliVersion = true ? "0.28.772" : "dev";
 var lastConfigHash = null;
 function setConfigHash(hash) {
   lastConfigHash = hash && hash.length > 0 ? hash : null;
@@ -10655,4 +10655,4 @@ export {
   managerInstallSystemUnitCommand,
   managerUninstallSystemUnitCommand
 };
-//# sourceMappingURL=chunk-E4YS6E4T.js.map
+//# sourceMappingURL=chunk-SBAX57SN.js.map

package/dist/lib/manager-worker.js

@@ -58,7 +58,7 @@ import {
   safeWriteJsonAtomic,
   setConfigHash,
   tripClass
-} from "../chunk-E4YS6E4T.js";
+} from "../chunk-SBAX57SN.js";
 import {
   getProjectDir as getProjectDir2,
   getReadyTasks,
@@ -13530,7 +13530,7 @@ var agentRestartTimezoneInputs = /* @__PURE__ */ new Map();
 var lastVersionCheckAt = 0;
 var VERSION_CHECK_INTERVAL_MS = 5 * 60 * 1e3;
 var lastResponsivenessProbeAt = 0;
-var agtCliVersion = true ? "0.28.770" : "dev";
+var agtCliVersion = true ? "0.28.772" : "dev";
 function resolveBrewPath(execFileSync2) {
   try {
     const out = execFileSync2("which", ["brew"], { timeout: 5e3 }).toString().trim();

package/dist/mcp/xero.js

@@ -116594,6 +116594,81 @@ async function getXeroRepeatingInvoice(repeatingInvoiceID) {
   }
 }
 
+// src/helpers/repeating-invoice-generation.ts
+function normaliseStatus(v) {
+  if (v === void 0 || v === null) return null;
+  if (typeof v !== "string" && typeof v !== "number") return null;
+  const s = String(v).trim();
+  return s.length > 0 ? s.toUpperCase() : null;
+}
+function generationVerdict(t) {
+  const status = normaliseStatus(t.status);
+  if (!status) {
+    return {
+      willGenerate: false,
+      reason: "GENERATION UNKNOWN: Xero returned no status for this template."
+    };
+  }
+  if (status === "DELETED" || status === "VOIDED") {
+    return {
+      willGenerate: false,
+      reason: `WILL NOT GENERATE: template is ${status}.`
+    };
+  }
+  if (status !== "AUTHORISED") {
+    return {
+      willGenerate: false,
+      reason: `WILL NOT GENERATE: status is ${status}. Only AUTHORISED templates generate invoices. Set it with update-repeating-invoice \u2014 and confirm the amount, cadence and counterparty with a human first, because authorising a template starts billing someone on a schedule.`
+    };
+  }
+  const end = t.schedule?.endDate;
+  const next = t.schedule?.nextScheduledDate;
+  if (end && isBoundaryAmbiguous(end)) {
+    return {
+      willGenerate: false,
+      reason: `GENERATION UNKNOWN: AUTHORISED and the schedule ends ${end}, which is within a day of now. Xero generates in the organisation's time zone and this check compares in UTC, so the two can disagree by a calendar day at the boundary. Confirm in Xero before acting on this one.`
+    };
+  }
+  if (end && isPast(end)) {
+    return {
+      willGenerate: false,
+      reason: `WILL NOT GENERATE: AUTHORISED but the schedule ended ${end}.`
+    };
+  }
+  if (!next) {
+    return {
+      willGenerate: false,
+      reason: "GENERATION UNKNOWN: AUTHORISED but Xero returned no next scheduled date, so the next run cannot be confirmed from here."
+    };
+  }
+  return {
+    willGenerate: true,
+    reason: `Generates: AUTHORISED, next invoice ${next}${end ? `, ends ${end}` : ""}.`
+  };
+}
+function isPast(dateStr) {
+  const t = Date.parse(dateStr);
+  if (Number.isNaN(t)) return false;
+  const endOfDay = new Date(t);
+  endOfDay.setUTCHours(23, 59, 59, 999);
+  return endOfDay.getTime() < Date.now();
+}
+var MAX_TZ_AHEAD_MS = 14 * 60 * 60 * 1e3;
+var MAX_TZ_BEHIND_MS = 12 * 60 * 60 * 1e3;
+function isBoundaryAmbiguous(dateStr) {
+  const t = Date.parse(dateStr);
+  if (Number.isNaN(t)) return false;
+  const endOfDayUtc = new Date(t);
+  endOfDayUtc.setUTCHours(23, 59, 59, 999);
+  const end = endOfDayUtc.getTime();
+  const now = Date.now();
+  return now >= end - MAX_TZ_AHEAD_MS && now <= end + MAX_TZ_BEHIND_MS;
+}
+function sendSettingsLine(t) {
+  const flag = (v) => v === void 0 ? "unknown" : v ? "yes" : "no";
+  return `Sending: approvedForSending=${flag(t.approvedForSending)}, sendCopy=${flag(t.sendCopy)}, markAsSent=${flag(t.markAsSent)}, includePDF=${flag(t.includePDF)}` + (t.approvedForSending === false ? " \u2014 NOT approved for sending, so generated invoices are not emailed to the contact." : "");
+}
+
 // src/tools/get/get-repeating-invoice.tool.ts
 var GetRepeatingInvoiceTool = CreateXeroTool(
   "get-repeating-invoice",
@@ -116630,6 +116705,8 @@ var GetRepeatingInvoiceTool = CreateXeroTool(
           type: "text",
           text: [
             `RepeatingInvoiceID: ${t.repeatingInvoiceID}`,
+            // CS-1635: state the conclusion, not just the evidence.
+            generationVerdict(t).reason,
             t.contact ? `Contact: ${t.contact.name} (${t.contact.contactID})` : null,
             t.type ? `Type: ${t.type}` : null,
             t.status ? `Status: ${t.status}` : null,
@@ -116638,6 +116715,7 @@ var GetRepeatingInvoiceTool = CreateXeroTool(
             t.subTotal !== void 0 ? `Sub Total: ${t.subTotal}` : null,
             t.totalTax !== void 0 ? `Total Tax: ${t.totalTax}` : null,
             t.currencyCode ? `Currency: ${t.currencyCode}` : null,
+            sendSettingsLine(t),
             t.lineAmountTypes ? `Line Amount Types: ${t.lineAmountTypes}` : null,
             t.schedule ? `Schedule: every ${t.schedule.period ?? "?"} ${t.schedule.unit ?? ""}, start ${t.schedule.startDate ?? "?"}, due ${t.schedule.dueDate ?? "?"} (${t.schedule.dueDateType ?? "?"}), next ${t.schedule.nextScheduledDate ?? "?"}, end ${t.schedule.endDate ?? "open-ended"}` : null,
             t.lineItems?.length ? `Line Items:
@@ -116786,6 +116864,15 @@ async function listXeroAgedPayablesByContact(contactId, reportDate, invoicesFrom
   }
 }
 
+// src/helpers/unreconciled-blind-spot.ts
+var UNRECONCILED_BLIND_SPOT_NOTE = "INCOMPLETE BY DESIGN \u2014 read before quoting any figure from this result.\nXero's Accounting API exposes LEDGER records. Bank STATEMENT LINES that have never been matched into the ledger are not exposed at all, and neither are any notes left on them. Money can therefore be in the bank account and absent from this data.\nSo any receivables, payables, cashflow or aged-debtors figure derived from this may be INCOMPLETE OR INACCURATE, and THE SIZE AND DIRECTION OF THE ERROR CANNOT BE DETERMINED FROM THIS RESPONSE \u2014 an unmatched line might be a customer receipt, a supplier payment or a bank fee, and nothing here says which, or how many there are. An invoice can read AUTHORISED and fully unpaid while the customer's payment is already sitting in the bank account.\nWhen reporting a figure from this data, say that unmatched bank activity is not included and that you cannot see it. Do not describe a customer as overdue, and do not request or draft a payment chase, on this evidence alone: check the Xero bank reconciliation screen, or ask a human to. (CS-1638)";
+function unreconciledBlindSpotBlock() {
+  return { type: "text", text: UNRECONCILED_BLIND_SPOT_NOTE };
+}
+function withUnreconciledBlindSpotNote(result) {
+  return { ...result, content: [...result.content, unreconciledBlindSpotBlock()] };
+}
+
 // src/tools/list/list-aged-payables-by-contact.tool.ts
 var ListAgedPayablesByContact = CreateXeroTool(
   "list-aged-payables-by-contact",
@@ -116811,7 +116898,7 @@ var ListAgedPayablesByContact = CreateXeroTool(
     }
     const agedPayablesReport = response.result;
     const filter2 = formatAgedReportFilter(invoicesFromDate, invoicesToDate);
-    return {
+    return withUnreconciledBlindSpotNote({
       content: [
         {
           type: "text",
@@ -116830,7 +116917,7 @@ var ListAgedPayablesByContact = CreateXeroTool(
           text: JSON.stringify(agedPayablesReport.rows, null, 2)
         }
       ]
-    };
+    });
   }
 );
 var list_aged_payables_by_contact_tool_default = ListAgedPayablesByContact;
@@ -116902,7 +116989,7 @@ var ListAgedReceivablesByContact = CreateXeroTool(
     }
     const agedReceivablesReport = response.result;
     const filter2 = formatAgedReportFilter(invoicesFromDate, invoicesToDate);
-    return {
+    return withUnreconciledBlindSpotNote({
       content: [
         {
           type: "text",
@@ -116921,7 +117008,7 @@ var ListAgedReceivablesByContact = CreateXeroTool(
           text: JSON.stringify(agedReceivablesReport.rows, null, 2)
         }
       ]
-    };
+    });
   }
 );
 var list_aged_receivables_by_contact_tool_default = ListAgedReceivablesByContact;
@@ -116982,7 +117069,14 @@ var formatLineItem = (lineItem) => {
 // src/tools/list/list-bank-transactions.tool.ts
 var ListBankTransactionsTool = CreateXeroTool(
   "list-bank-transactions",
-  `List all bank transactions in Xero.
+  `List Xero BankTransaction LEDGER records (spend-money / receive-money), each
+  flagged Reconciled or Unreconciled.
+  This is NOT the bank account, and the distinction is the point. Xero's
+  Accounting API exposes BankTransaction records only. Bank STATEMENT LINES that
+  have never been matched into the ledger are not exposed at all, so money can be
+  in the bank account and absent from this list entirely. A balance, receivables
+  or cashflow figure derived from this may be incomplete, and nothing here reveals
+  by how much. (CS-1638)
   Ask the user if they want to see bank transactions for a specific bank account,
   or to see all bank transactions before running.
   Ask the user if they want the next page of quotes after running this tool if
@@ -117006,7 +117100,7 @@ var ListBankTransactionsTool = CreateXeroTool(
       };
     }
     const bankTransactions = response.result;
-    return {
+    return withUnreconciledBlindSpotNote({
       content: [
         {
           type: "text",
@@ -117032,7 +117126,7 @@ var ListBankTransactionsTool = CreateXeroTool(
           ].filter(Boolean).join("\n")
         })) || []
       ]
-    };
+    });
   }
 );
 var list_bank_transactions_tool_default = ListBankTransactionsTool;
@@ -117289,7 +117383,7 @@ async function listXeroInvoices(page = 1, contactIds, invoiceNumbers) {
 // src/tools/list/list-invoices.tool.ts
 var ListInvoicesTool = CreateXeroTool(
   "list-invoices",
-  "List invoices in Xero. This includes Draft, Submitted, and Paid invoices.   Ask the user if they want to see invoices for a specific contact,   invoice number, or to see all invoices before running.   Ask the user if they want the next page of invoices after running this tool   if 10 invoices are returned.   If they want the next page, call this tool again with the next page number   and the contact or invoice number if one was provided in the previous call.",
+  `List invoices in Xero. This includes Draft, Submitted, and Paid invoices.   CS-1638: an invoice reads AUTHORISED and fully unpaid until the customer's   payment is RECONCILED, so "unpaid" here means "not yet matched in Xero",   not "the money has not arrived". Do not chase on this evidence alone.   Ask the user if they want to see invoices for a specific contact,   invoice number, or to see all invoices before running.   Ask the user if they want the next page of invoices after running this tool   if 10 invoices are returned.   If they want the next page, call this tool again with the next page number   and the contact or invoice number if one was provided in the previous call.`,
   {
     page: external_exports.number(),
     contactIds: external_exports.array(external_exports.string()).optional(),
@@ -117309,7 +117403,7 @@ var ListInvoicesTool = CreateXeroTool(
     }
     const invoices = response.result;
     const returnLineItems = (invoiceNumbers?.length ?? 0) > 0;
-    return {
+    return withUnreconciledBlindSpotNote({
       content: [
         {
           type: "text",
@@ -117344,7 +117438,7 @@ var ListInvoicesTool = CreateXeroTool(
           ].filter(Boolean).join("\n")
         })) || []
       ]
-    };
+    });
   }
 );
 var list_invoices_tool_default = ListInvoicesTool;
@@ -119042,7 +119136,7 @@ async function listXeroRepeatingInvoices() {
 // src/tools/list/list-repeating-invoices.tool.ts
 var ListRepeatingInvoicesTool = CreateXeroTool(
   "list-repeating-invoices",
-  "List every repeating (recurring) invoice template configured on the Xero tenant. Each template defines an invoice that Xero auto-generates on a schedule (weekly/monthly/etc.). Use this to look up the RepeatingInvoiceID before calling get-repeating-invoice or update-repeating-invoice. The endpoint does not support pagination \u2014 all templates are returned.",
+  "List every repeating (recurring) invoice template configured on the Xero tenant. Each template defines an invoice that Xero auto-generates on a schedule (weekly/monthly/etc.). Use this to look up the RepeatingInvoiceID before calling get-repeating-invoice or update-repeating-invoice. The endpoint does not support pagination \u2014 all templates are returned. CS-1635: every row states outright whether it WILL GENERATE and why, plus its next scheduled date and its sending settings. A template can be perfectly configured, sit in DRAFT, and generate nothing \u2014 read the verdict line rather than inferring from Status.",
   {},
   async () => {
     const response = await listXeroRepeatingInvoices();
@@ -119067,6 +119161,10 @@ var ListRepeatingInvoicesTool = CreateXeroTool(
           type: "text",
           text: [
             `RepeatingInvoiceID: ${t.repeatingInvoiceID}`,
+            // CS-1635: the conclusion, first, because it is the question being
+            // asked. Printing Status and leaving the reader to know that DRAFT
+            // means "silent" is how 14 of 15 templates sat dead unnoticed.
+            generationVerdict(t).reason,
             t.contact ? `Contact: ${t.contact.name} (${t.contact.contactID})` : null,
             t.type ? `Type: ${t.type}` : null,
             t.status ? `Status: ${t.status}` : null,
@@ -119074,6 +119172,12 @@ var ListRepeatingInvoicesTool = CreateXeroTool(
             t.total !== void 0 ? `Total: ${t.total}` : null,
             t.currencyCode ? `Currency: ${t.currencyCode}` : null,
             t.schedule ? `Schedule: every ${t.schedule.period ?? "?"} ${t.schedule.unit ?? ""} starting ${t.schedule.startDate ?? "?"}` : null,
+            t.schedule?.nextScheduledDate ? `Next scheduled: ${t.schedule.nextScheduledDate}` : null,
+            t.schedule?.endDate ? `Schedule ends: ${t.schedule.endDate}` : null,
+            // CS-1635 asks for this by name: the send setting was not exposed
+            // anywhere, so a template could generate and never reach the
+            // customer with nothing in any tool saying so.
+            sendSettingsLine(t),
             t.lineAmountTypes ? `Line Amount Types: ${t.lineAmountTypes}` : null
           ].filter(Boolean).join("\n")
         }))
@@ -120609,12 +120713,12 @@ async function updateXeroRepeatingInvoice(repeatingInvoiceID, patch) {
         error: `RepeatingInvoice ${repeatingInvoiceID} not found.`
       };
     }
-    const noChange = patch.reference === void 0 && patch.lineItems === void 0 && patch.status === void 0 && patch.contactId === void 0;
+    const noChange = patch.reference === void 0 && patch.lineItems === void 0 && patch.status === void 0 && patch.contactId === void 0 && patch.approvedForSending === void 0;
     if (noChange) {
       return {
         result: null,
         isError: true,
-        error: "No fields supplied to update. Pass at least one of reference, lineItems, status, or contactId."
+        error: "No fields supplied to update. Pass at least one of reference, lineItems, status, contactId, or approvedForSending."
       };
     }
     const xeroPatch = {};
@@ -120625,6 +120729,9 @@ async function updateXeroRepeatingInvoice(repeatingInvoiceID, patch) {
     if (patch.contactId !== void 0) {
       xeroPatch.contact = { contactID: patch.contactId };
     }
+    if (patch.approvedForSending !== void 0) {
+      xeroPatch.approvedForSending = patch.approvedForSending;
+    }
     if (patch.lineItems !== void 0) {
       xeroPatch.lineItems = patch.lineItems.map((li) => ({
         description: li.description,
@@ -120674,15 +120781,19 @@ var UpdateRepeatingInvoiceTool = CreateXeroTool(
     status: external_exports.enum(["DRAFT", "AUTHORISED"]).optional().describe(
       "Template status. DRAFT templates do NOT generate invoices; AUTHORISED templates do. Flipping a draft to AUTHORISED is high-risk and should be explicitly confirmed with the user."
     ),
-    contactId: external_exports.string().optional().describe("Replace the counterparty. Look up the ContactID via list-contacts first.")
+    contactId: external_exports.string().optional().describe("Replace the counterparty. Look up the ContactID via list-contacts first."),
+    approvedForSending: external_exports.boolean().optional().describe(
+      'CS-1635: whether Xero emails each generated invoice to the contact. A template can generate invoices perfectly and never send them, which looks identical to "working" from every other field. Setting this true starts emailing a customer on a schedule, so confirm it with a human first.'
+    )
   },
   async (params) => {
-    const { repeatingInvoiceID, reference, lineItems, status, contactId } = params;
+    const { repeatingInvoiceID, reference, lineItems, status, contactId, approvedForSending } = params;
     const result = await updateXeroRepeatingInvoice(repeatingInvoiceID, {
       reference,
       lineItems,
       status,
-      contactId
+      contactId,
+      approvedForSending
     });
     if (result.isError) {
       return {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@integrity-labs/agt-cli",
-  "version": "0.28.770",
+  "version": "0.28.772",
   "description": "Augmented Team CLI — agent provisioning and management",
   "type": "module",
   "engines": {