@integrity-labs/agt-cli 0.28.487 → 0.28.489

package/dist/{chunk-H5H4IVQC.js → chunk-46GY6S7I.js}

@@ -30,7 +30,7 @@ import {
   resolveConnectivityProbe,
   worseConnectivityOutcome,
   wrapScheduledTaskPrompt
-} from "./chunk-XZI5IHQP.js";
+} from "./chunk-34E3FHOS.js";
 import {
   parsePsRows
 } from "./chunk-XWVM4KPK.js";
@@ -3094,6 +3094,25 @@ function buildMcpJson(input) {
       args: [localXeroMcpPath],
       env: {
         ...brokerMode ? {} : { XERO_CLIENT_BEARER_TOKEN: "${XERO_ACCESS_TOKEN}" },
+        // ENG-8264: broker mode must ALSO declare which env var holds the
+        // credential, or dropping the `${XERO_ACCESS_TOKEN}` reference above
+        // silently re-classifies it. `findMcpServersUsingVars` only sees vars
+        // referenced as `${VAR}`, so with that reference gone the manager's
+        // `envOnlyRespawnVars` treats XERO_ACCESS_TOKEN as env-only and
+        // schedules a FULL SESSION RESPAWN on every OAuth rotation (~30 min) -
+        // trading the ~20-min child kill ENG-5318 fixed for something worse.
+        // Observed live on stirling at 2026-08-02T07:29:34Z.
+        //
+        // This is a declaration of fact, not a carve-out: in broker mode the
+        // server fetches the freshest token from the credential broker on every
+        // call (that is what AGT_INTEGRATION_ID/AGT_AGENT_ID above are for), so
+        // a rotation genuinely needs no restart - exactly what
+        // AGT_REMOTE_MCP_TOKEN_VAR means. `liveProxyTokenVars` reads this key
+        // off ANY mcpServers entry, not only remote-proxy ones, so the manager
+        // filters the var out of the respawn set. The `${VAR}` reference stays
+        // absent, so stale-mcp-reaper stays quiet too: both restart paths
+        // satisfied at once.
+        ...brokerMode ? { AGT_REMOTE_MCP_TOKEN_VAR: "XERO_ACCESS_TOKEN" } : {},
         XERO_TENANT_ID: "${XERO_TENANT_ID}",
         AGT_HOST: "${AGT_HOST}",
         AGT_TOKEN: "${AGT_TOKEN}",
@@ -4285,6 +4304,13 @@ ${sections}`
       const brokerMode = Boolean(agentId && xeroIntegration.id);
       const xeroEnv = {
         ...brokerMode ? {} : { XERO_CLIENT_BEARER_TOKEN: "${XERO_ACCESS_TOKEN}" },
+        // ENG-8264: declare the credential var in broker mode so the manager
+        // does not re-classify it as env-only and respawn the whole session on
+        // every rotation. Mirrors the buildMcpJson path above - see that block
+        // for the full rationale. Kept in both renderers deliberately: they
+        // produce the same entry by two routes, and a fix in only one of them
+        // is a fix that holds until whichever path the agent happens to take.
+        ...brokerMode ? { AGT_REMOTE_MCP_TOKEN_VAR: "XERO_ACCESS_TOKEN" } : {},
         XERO_TENANT_ID: "${XERO_TENANT_ID}",
         // ENG-4922: writes go through xero-broker → read-only vendor server.
         ...hasXeroBroker ? { XERO_WRITES_VIA_BROKER: "true" } : {},
@@ -4938,7 +4964,7 @@ function exchangeFailureKind(err) {
 }
 
 // src/lib/api-client.ts
-var agtCliVersion = true ? "0.28.487" : "dev";
+var agtCliVersion = true ? "0.28.489" : "dev";
 var lastConfigHash = null;
 function setConfigHash(hash) {
   lastConfigHash = hash && hash.length > 0 ? hash : null;
@@ -7583,6 +7609,12 @@ async function executeConnectivityProbe(target, deps = {}) {
         message: `${target.definitionId}: built-in module \u2014 no live check available, not verified`,
         evidence: "none"
       };
+    case "host_unprobeable":
+      return {
+        status: "unverified",
+        message: descriptor.label,
+        evidence: "none"
+      };
     case "unsupported":
     default:
       return null;
@@ -7667,4 +7699,4 @@ export {
   managerInstallSystemUnitCommand,
   managerUninstallSystemUnitCommand
 };
-//# sourceMappingURL=chunk-H5H4IVQC.js.map
+//# sourceMappingURL=chunk-46GY6S7I.js.map