@integrity-labs/agt-cli 0.28.1094 → 0.28.1095

package/dist/bin/agt.js

@@ -40,7 +40,7 @@ import {
   success,
   table,
   warn
-} from "../chunk-JJYOF5KM.js";
+} from "../chunk-OHKWHHTS.js";
 import {
   getProjectDir,
   isSessionResumeDisabled,
@@ -5474,7 +5474,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.1094" : "dev";
+var cliVersion = true ? "0.28.1095" : "dev";
 async function fetchLatestVersion() {
   const host2 = getHost();
   if (!host2) return null;
@@ -6734,7 +6734,7 @@ function handleError(err) {
 }
 
 // src/bin/agt.ts
-var cliVersion2 = true ? "0.28.1094" : "dev";
+var cliVersion2 = true ? "0.28.1095" : "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-JJYOF5KM.js → chunk-OHKWHHTS.js}

@@ -7122,7 +7122,7 @@ function exchangeFailureKind(err) {
 }
 
 // src/lib/api-client.ts
-var agtCliVersion = true ? "0.28.1094" : "dev";
+var agtCliVersion = true ? "0.28.1095" : "dev";
 var lastConfigHash = null;
 function setConfigHash(hash) {
   lastConfigHash = hash && hash.length > 0 ? hash : null;
@@ -12095,4 +12095,4 @@ export {
   managerInstallSystemUnitCommand,
   managerUninstallSystemUnitCommand
 };
-//# sourceMappingURL=chunk-JJYOF5KM.js.map
+//# sourceMappingURL=chunk-OHKWHHTS.js.map

package/dist/lib/manager-worker.js

@@ -67,7 +67,7 @@ import {
   safeWriteJsonAtomic,
   setConfigHash,
   tripClass
-} from "../chunk-JJYOF5KM.js";
+} from "../chunk-OHKWHHTS.js";
 import {
   getProjectDir as getProjectDir2,
   getReadyTasks,
@@ -14900,7 +14900,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.1094" : "dev";
+var agtCliVersion = true ? "0.28.1095" : "dev";
 function resolveBrewPath(execFileSync3) {
   try {
     const out = execFileSync3("which", ["brew"], { timeout: 5e3 }).toString().trim();

package/dist/mcp/index.js

@@ -50003,6 +50003,66 @@ Give your manager this link so they can accept or decline it: ${data.review_url}
     }
   }
 );
+server.tool(
+  "routine_retire",
+  "Ask for approval to RETIRE one of your routines, so it stops for good. The card goes to the person who accepted the routine (your manager if nobody did). Use this when a routine should no longer run at all: a duplicate, a responsibility that has ended, or one its owner has asked you to delete. This is the only way to end a routine - schedule_delete refuses a routine's schedules, and routine_revise only changes what it does. Nothing stops until they approve: they get an approve/deny card, and the routine keeps running until they decide. On approval the platform retires the routine and turns off every schedule in it at once. Poll the returned request_id with check_approval. To change WHAT a routine does, use routine_revise instead; to change WHEN it runs, send only schedule fields to schedule_update.",
+  {
+    routine_id: external_exports.string().trim().min(1).describe("The routine to retire. Get this from routine_list. A schedule_id from schedule_list is also accepted and resolved server-side. It must be a routine YOU own; anything else reads as not-found."),
+    reason: external_exports.string().trim().min(1).max(500).describe("Why this routine should stop, and who asked if someone did. The approver reads this on the card before deciding. Required."),
+    idempotency_key: external_exports.string().trim().min(1).max(200).describe('A stable key for this request so a retry does not file a second card (e.g. "retire-<routine_id>").')
+  },
+  async (params) => {
+    let data;
+    try {
+      data = await apiPost("/host/routines/retire", {
+        agent_id: AGT_AGENT_ID,
+        routine_id: params.routine_id,
+        reason: params.reason,
+        idempotency_key: params.idempotency_key
+      });
+    } catch (err) {
+      const msg = err instanceof Error ? err.message : String(err);
+      if (/returned\s+[45]\d\d/.test(msg)) {
+        let detail = msg;
+        const jsonStart = msg.indexOf("{");
+        if (jsonStart !== -1) {
+          try {
+            const parsedErr = JSON.parse(msg.slice(jsonStart));
+            if (parsedErr.error) detail = parsedErr.error;
+          } catch {
+          }
+        }
+        return { content: [{ type: "text", text: `Could not request the retire: ${detail}` }], isError: true };
+      }
+      return {
+        content: [
+          {
+            type: "text",
+            text: `The retire request's outcome is unknown (${msg}); it may already have been filed. Retry with the SAME idempotency_key: that re-attaches to the existing request if there is one instead of filing a second card. Do not use a new key. If it keeps failing, tell your operator.`
+          }
+        ],
+        isError: true
+      };
+    }
+    if (!data.ok) {
+      return {
+        content: [{ type: "text", text: `Could not request the retire: ${data.error ?? "unknown error"}` }],
+        isError: true
+      };
+    }
+    const name = data.routine_name ? `"${data.routine_name}"` : "the routine";
+    const idLine = data.request_id ? ` Request id: ${data.request_id} (poll it with check_approval).` : "";
+    const unpaged = data.notification_status === "failed" ? " WARNING: the approval card could not be delivered, so nobody has been paged yet. Tell your operator." : "";
+    return {
+      content: [
+        {
+          type: "text",
+          text: `Asked for approval to retire ${name}.${idLine} It keeps running until they approve; on approval it is retired and every schedule in it stops. Do not re-send while it is pending.${unpaged}`
+        }
+      ]
+    };
+  }
+);
 server.tool(
   "routine_proposal_status",
   "Read the outcome of the routine revisions YOU proposed with routine_revise: pending (your manager has not resolved it yet), accepted, declined, or expired - plus the summary and the FULL replacement description you submitted, so you can see what is actually waiting on them. A pending proposal also returns a review_url to hand to your manager, and is flagged `stale` when the routine has changed since you proposed (accepting it would apply a rewrite of text the routine no longer says - propose again against the current prompt instead). Call this when routine_list shows a revision pending, when routine_revise refuses because one already is, or to close the loop on a proposal you filed in an earlier session. No decline reason is recorded anywhere on this path, so a declined proposal tells you THAT it was declined and not why - ask the person rather than reporting a reason you do not have.",
@@ -50965,6 +51025,9 @@ var LOCAL_TOOL_NAMES = /* @__PURE__ */ new Set([
   // ENG-10494: the verdict loop for routine_revise. Without it a proposed
   // revision is write-only, and an unseen one blocks that routine for good.
   "routine_proposal_status",
+  // CS-1964: the ending half. Files a routine.retirement approval; there was no
+  // agent path to stop a routine at all.
+  "routine_retire",
   // ENG-10276: knowledge_get is the untruncated single-entry read. The list and
   // search tools serve a stored SUMMARY column, not the entry, so this is the
   // only one that can be trusted before a knowledge_update.

package/dist/mcp/xero.js

@@ -22704,12 +22704,12 @@ var require_lineAmountTypes = __commonJS({
     "use strict";
     Object.defineProperty(exports, "__esModule", { value: true });
     exports.LineAmountTypes = void 0;
-    var LineAmountTypes;
-    (function(LineAmountTypes2) {
-      LineAmountTypes2[LineAmountTypes2["Exclusive"] = "Exclusive"] = "Exclusive";
-      LineAmountTypes2[LineAmountTypes2["Inclusive"] = "Inclusive"] = "Inclusive";
-      LineAmountTypes2[LineAmountTypes2["NoTax"] = "NoTax"] = "NoTax";
-    })(LineAmountTypes || (exports.LineAmountTypes = LineAmountTypes = {}));
+    var LineAmountTypes3;
+    (function(LineAmountTypes4) {
+      LineAmountTypes4[LineAmountTypes4["Exclusive"] = "Exclusive"] = "Exclusive";
+      LineAmountTypes4[LineAmountTypes4["Inclusive"] = "Inclusive"] = "Inclusive";
+      LineAmountTypes4[LineAmountTypes4["NoTax"] = "NoTax"] = "NoTax";
+    })(LineAmountTypes3 || (exports.LineAmountTypes = LineAmountTypes3 = {}));
   }
 });
 
@@ -114893,11 +114893,28 @@ function messageForStatus(status, detail) {
       return detail ? redactSensitiveText(detail) : "An error occurred while communicating with Xero.";
   }
 }
+var MAX_VALIDATION_MESSAGES = 5;
 function readDetail(body) {
-  if (body && typeof body === "object" && "Detail" in body) {
-    const d = body.Detail;
-    return typeof d === "string" ? d : null;
-  }
+  if (!body || typeof body !== "object") return null;
+  const b = body;
+  if (typeof b.Detail === "string" && b.Detail.trim() !== "") return b.Detail;
+  const messages = [];
+  if (Array.isArray(b.Elements)) {
+    for (const element of b.Elements) {
+      const errors = element?.ValidationErrors;
+      if (!Array.isArray(errors)) continue;
+      for (const e of errors) {
+        const m = e?.Message;
+        if (typeof m === "string" && m.trim() !== "" && !messages.includes(m.trim())) messages.push(m.trim());
+      }
+    }
+  }
+  if (messages.length > 0) {
+    const shown = messages.slice(0, MAX_VALIDATION_MESSAGES);
+    const more = messages.length - shown.length;
+    return `Xero rejected the request: ${shown.join("; ")}${more > 0 ? ` (and ${more} more)` : ""}`;
+  }
+  if (typeof b.Message === "string" && b.Message.trim() !== "") return b.Message;
   return null;
 }
 
@@ -115495,7 +115512,7 @@ var create_credit_note_tool_default = CreateCreditNoteTool;
 
 // src/handlers/create-xero-invoice.handler.ts
 var import_xero_node5 = __toESM(require_dist2(), 1);
-async function createInvoice(contactId, lineItems, type, reference, date3) {
+async function createInvoice(contactId, lineItems, type, reference, date3, lineAmountTypes) {
   await xeroClient.authenticate();
   const invoice = {
     type,
@@ -115508,7 +115525,11 @@ async function createInvoice(contactId, lineItems, type, reference, date3) {
     dueDate: new Date(Date.now() + 30 * 24 * 60 * 60 * 1e3).toISOString().split("T")[0],
     // 30 days from now
     ...type === import_xero_node5.Invoice.TypeEnum.ACCPAY ? { invoiceNumber: reference } : { reference },
-    status: import_xero_node5.Invoice.StatusEnum.DRAFT
+    status: import_xero_node5.Invoice.StatusEnum.DRAFT,
+    // CS-1974: omitted unless asked for, so Xero's own default (Exclusive)
+    // still applies to every existing caller. Inclusive lets a tax-inclusive
+    // price list be entered as-is instead of as retail/1.1 with cent rounding.
+    ...lineAmountTypes !== void 0 ? { lineAmountTypes } : {}
   };
   const response = await xeroClient.accountingApi.createInvoices(
     xeroClient.tenantId,
@@ -115527,14 +115548,15 @@ async function createInvoice(contactId, lineItems, type, reference, date3) {
   const createdInvoice = response.body.invoices?.[0];
   return createdInvoice;
 }
-async function createXeroInvoice(contactId, lineItems, type = import_xero_node5.Invoice.TypeEnum.ACCREC, reference, date3) {
+async function createXeroInvoice(contactId, lineItems, type = import_xero_node5.Invoice.TypeEnum.ACCREC, reference, date3, lineAmountTypes) {
   try {
     const createdInvoice = await createInvoice(
       contactId,
       lineItems,
       type,
       reference,
-      date3
+      date3,
+      lineAmountTypes
     );
     if (!createdInvoice) {
       throw new Error("Invoice creation failed.");
@@ -115569,6 +115591,11 @@ var lineItemSchema3 = external_exports.object({
   itemCode: external_exports.string().describe("The item code of the line item - can be obtained from the list-items tool     If the item is not listed, add without an item code and ask the user if they would like to add an item code.").optional(),
   tracking: external_exports.array(trackingSchema).describe("Up to 2 tracking categories and options can be added to the line item.     Can be obtained from the list-tracking-categories tool.     Only use if prompted by the user.").optional()
 });
+var LINE_AMOUNT_TYPES = {
+  Exclusive: import_xero_node6.LineAmountTypes.Exclusive,
+  Inclusive: import_xero_node6.LineAmountTypes.Inclusive,
+  NoTax: import_xero_node6.LineAmountTypes.NoTax
+};
 var CreateInvoiceTool = CreateXeroTool(
   "create-invoice",
   "Create an invoice in Xero. When an invoice is created, a deep link to the invoice in Xero is returned.  This deep link can be used to view the invoice in Xero directly.  This link should be displayed to the user.",
@@ -115577,11 +115604,19 @@ var CreateInvoiceTool = CreateXeroTool(
     lineItems: external_exports.array(lineItemSchema3),
     type: external_exports.enum(["ACCREC", "ACCPAY"]).describe("The type of invoice to create.       ACCREC is for sales invoices, Accounts Receivable, or customer invoices.       ACCPAY is for purchase invoices, Accounts Payable invoices, supplier invoices, or bills.       If the type is not specified, the default is ACCREC."),
     reference: external_exports.string().describe("A reference number for the invoice.").optional(),
-    date: external_exports.string().describe("The date the invoice was created (YYYY-MM-DD format).").optional()
+    date: external_exports.string().describe("The date the invoice was created (YYYY-MM-DD format).").optional(),
+    lineAmountTypes: external_exports.enum(["Exclusive", "Inclusive", "NoTax"]).describe("Whether the line unitAmounts INCLUDE tax.       Inclusive: the amounts already include tax (e.g. retail prices incl. GST) - use this rather than dividing by 1.1.       Exclusive: tax is added on top (Xero's default when omitted).       NoTax: no tax applies.").optional()
   },
-  async ({ contactId, lineItems, type, reference, date: date3 }) => {
+  async ({ contactId, lineItems, type, reference, date: date3, lineAmountTypes }) => {
     const xeroInvoiceType = type === "ACCREC" ? import_xero_node6.Invoice.TypeEnum.ACCREC : import_xero_node6.Invoice.TypeEnum.ACCPAY;
-    const result = await createXeroInvoice(contactId, lineItems, xeroInvoiceType, reference, date3);
+    const result = await createXeroInvoice(
+      contactId,
+      lineItems,
+      xeroInvoiceType,
+      reference,
+      date3,
+      lineAmountTypes === void 0 ? void 0 : LINE_AMOUNT_TYPES[lineAmountTypes]
+    );
     if (result.isError) {
       return {
         content: [
@@ -115631,7 +115666,8 @@ async function createItem(itemDetails) {
     purchaseDetails: itemDetails.purchaseDetails ? {
       unitPrice: itemDetails.purchaseDetails.unitPrice,
       taxType: itemDetails.purchaseDetails.taxType,
-      accountCode: itemDetails.purchaseDetails.accountCode
+      accountCode: itemDetails.purchaseDetails.accountCode,
+      cOGSAccountCode: itemDetails.purchaseDetails.cOGSAccountCode
     } : void 0,
     salesDetails: itemDetails.salesDetails ? {
       unitPrice: itemDetails.salesDetails.unitPrice,
@@ -115679,7 +115715,9 @@ async function createXeroItem(itemDetails) {
 var purchaseDetailsSchema = external_exports.object({
   unitPrice: external_exports.number(),
   taxType: external_exports.string().optional(),
-  accountCode: external_exports.string().optional()
+  accountCode: external_exports.string().optional(),
+  // CS-1974: without this a tracked item cannot be created at all.
+  cOGSAccountCode: external_exports.string().optional().describe("Cost of goods sold account code. REQUIRED by Xero when the item is tracked as inventory (isTrackedAsInventory true): Xero rejects a tracked item without it. Use a cost of sales / COGS account from list-accounts.")
 });
 var salesDetailsSchema = external_exports.object({
   unitPrice: external_exports.number(),
@@ -115696,8 +115734,8 @@ var CreateItemTool = CreateXeroTool(
     purchaseDescription: external_exports.string().optional(),
     purchaseDetails: purchaseDetailsSchema.optional(),
     salesDetails: salesDetailsSchema.optional(),
-    isTrackedAsInventory: external_exports.boolean().optional(),
-    inventoryAssetAccountCode: external_exports.string().optional()
+    isTrackedAsInventory: external_exports.boolean().optional().describe("Track quantity and value. A tracked item needs inventoryAssetAccountCode AND purchaseDetails.cOGSAccountCode, and Xero records purchases against the inventory asset account rather than purchaseDetails.accountCode."),
+    inventoryAssetAccountCode: external_exports.string().optional().describe("Inventory asset account code. Required when isTrackedAsInventory is true.")
   },
   async ({
     code,
@@ -120080,7 +120118,9 @@ async function updateXeroItem(itemId, itemDetails) {
 var purchaseDetailsSchema2 = external_exports.object({
   unitPrice: external_exports.number().optional(),
   taxType: external_exports.string().optional(),
-  accountCode: external_exports.string().optional()
+  accountCode: external_exports.string().optional(),
+  // CS-1974: lets an existing item be switched to tracked inventory.
+  cOGSAccountCode: external_exports.string().optional().describe("Cost of goods sold account code. REQUIRED by Xero when the item is tracked as inventory (isTrackedAsInventory true): Xero rejects a tracked item without it. Use a cost of sales / COGS account from list-accounts.")
 });
 var salesDetailsSchema2 = external_exports.object({
   unitPrice: external_exports.number().optional(),

package/package.json

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