@integrity-labs/agt-cli 0.28.770 → 0.28.771

package/dist/bin/agt.js

@@ -41,7 +41,7 @@ import {
   success,
   table,
   warn
-} from "../chunk-E4YS6E4T.js";
+} from "../chunk-BOYUCA64.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.771" : "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.771" : "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-BOYUCA64.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.771" : "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-BOYUCA64.js.map

package/dist/lib/manager-worker.js

@@ -58,7 +58,7 @@ import {
   safeWriteJsonAtomic,
   setConfigHash,
   tripClass
-} from "../chunk-E4YS6E4T.js";
+} from "../chunk-BOYUCA64.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.771" : "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:
@@ -119042,7 +119120,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 +119145,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 +119156,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 +120697,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 +120713,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 +120765,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.771",
   "description": "Augmented Team CLI — agent provisioning and management",
   "type": "module",
   "engines": {