@integrity-labs/agt-cli 0.28.635 → 0.28.636

package/dist/bin/agt.js

@@ -40,10 +40,10 @@ import {
   success,
   table,
   warn
-} from "../chunk-J6KQDGWM.js";
+} from "../chunk-KIOTRBOI.js";
 import {
   readDirectChatSessionState
-} from "../chunk-GNTCOXZG.js";
+} from "../chunk-4HVSMJFM.js";
 import {
   AnchorSessionClient,
   CHANNEL_REGISTRY,
@@ -73,7 +73,7 @@ import {
   requiredMcpWildcard,
   resolveChannels,
   serializeManifestForSlackCli
-} from "../chunk-FKG7DIE2.js";
+} from "../chunk-O4ADFO57.js";
 import "../chunk-XWVM4KPK.js";
 
 // src/bin/agt.ts
@@ -4909,7 +4909,7 @@ import { execFileSync, execSync } from "child_process";
 import { existsSync as existsSync10, realpathSync as realpathSync2 } from "fs";
 import chalk18 from "chalk";
 import ora16 from "ora";
-var cliVersion = true ? "0.28.635" : "dev";
+var cliVersion = true ? "0.28.636" : "dev";
 async function fetchLatestVersion() {
   const host2 = getHost();
   if (!host2) return null;
@@ -6089,7 +6089,7 @@ function handleError(err) {
 }
 
 // src/bin/agt.ts
-var cliVersion2 = true ? "0.28.635" : "dev";
+var cliVersion2 = true ? "0.28.636" : "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-GNTCOXZG.js → chunk-4HVSMJFM.js}

@@ -16,7 +16,7 @@ import {
   rotateDailySession,
   sessionFileExists,
   todayLocalIso
-} from "./chunk-FKG7DIE2.js";
+} from "./chunk-O4ADFO57.js";
 import {
   reapOrphanChannelMcps
 } from "./chunk-XWVM4KPK.js";
@@ -4313,4 +4313,4 @@ export {
   stopAllSessionsAndWait,
   getProjectDir
 };
-//# sourceMappingURL=chunk-GNTCOXZG.js.map
+//# sourceMappingURL=chunk-4HVSMJFM.js.map

package/dist/{chunk-J6KQDGWM.js → chunk-KIOTRBOI.js}

@@ -7,7 +7,7 @@ import {
   expandTemplateVars,
   isolationMode,
   parseEnvIntegrations
-} from "./chunk-GNTCOXZG.js";
+} from "./chunk-4HVSMJFM.js";
 import {
   BIND_FAILURE_QUARANTINE_THRESHOLD,
   INTEGRATIONS_SECTION_END,
@@ -33,6 +33,7 @@ import {
   handshakeToolsListOutcome,
   isControlPlaneOnlyConfigKey,
   isDefaultRemoteMcpConnection,
+  isSessionToolBindCandidate,
   isSlackInboundTransportDown,
   listFlagDefinitions,
   mcpWildcardsForServers,
@@ -51,7 +52,7 @@ import {
   resolveConnectivityProbe,
   worseConnectivityOutcome,
   wrapScheduledTaskPrompt
-} from "./chunk-FKG7DIE2.js";
+} from "./chunk-O4ADFO57.js";
 import {
   parsePsRows
 } from "./chunk-XWVM4KPK.js";
@@ -6069,7 +6070,7 @@ function exchangeFailureKind(err) {
 }
 
 // src/lib/api-client.ts
-var agtCliVersion = true ? "0.28.635" : "dev";
+var agtCliVersion = true ? "0.28.636" : "dev";
 var lastConfigHash = null;
 function setConfigHash(hash) {
   lastConfigHash = hash && hash.length > 0 ? hash : null;
@@ -8300,6 +8301,17 @@ function resolveIntegrationServerKeys(definitionId, declaredKeys, hint) {
   for (const c of candidates) if (declared.has(c)) out.add(c);
   return [...out];
 }
+function hasDeliverySignal(integ) {
+  return DELIVERY_FIELDS.some((k) => Object.prototype.hasOwnProperty.call(integ, k));
+}
+var DELIVERY_FIELDS = [
+  "source_type",
+  "provider",
+  "mcp_command",
+  "mcp_url",
+  "cli_binary",
+  "cli_package"
+];
 function isDue(last, now, intervalMs) {
   if (!last) return true;
   const t = Date.parse(last);
@@ -8320,6 +8332,8 @@ async function runSessionToolBindProbes(integrations, options) {
   const reports = [];
   const rebindCandidates = [];
   let skipped = 0;
+  let notApplicable = 0;
+  const skippedDetail = [];
   let probed = 0;
   for (const integ of batch) {
     const keys = resolveIntegrationServerKeys(
@@ -8328,7 +8342,29 @@ async function runSessionToolBindProbes(integrations, options) {
       integ.mcp_server_key
     );
     if (keys.length === 0) {
+      if (!hasDeliverySignal(integ)) {
+        skipped += 1;
+        skippedDetail.push({ definition_id: integ.definition_id, reason: "delivery-unknown" });
+        continue;
+      }
+      if (!isSessionToolBindCandidate({
+        sourceType: integ.source_type,
+        provider: integ.provider,
+        mcpCommand: integ.mcp_command,
+        mcpUrl: integ.mcp_url,
+        cliBinary: integ.cli_binary,
+        cliPackage: integ.cli_package
+      })) {
+        notApplicable += 1;
+        reports.push({
+          integration_id: integ.id,
+          scope: integ.scope,
+          status: "not_applicable"
+        });
+        continue;
+      }
       skipped += 1;
+      skippedDetail.push({ definition_id: integ.definition_id, reason: "no-declared-server" });
       continue;
     }
     probed += 1;
@@ -8358,6 +8394,7 @@ async function runSessionToolBindProbes(integrations, options) {
     });
     if (status === "unknown") {
       skipped += 1;
+      skippedDetail.push({ definition_id: integ.definition_id, reason: "inconclusive" });
       continue;
     }
     reports.push({ integration_id: integ.id, scope: integ.scope, status });
@@ -8384,7 +8421,7 @@ async function runSessionToolBindProbes(integrations, options) {
       }
     }
   }
-  return { reports, rebindCandidates, due: due.length, probed, skipped };
+  return { reports, rebindCandidates, due: due.length, probed, skipped, skippedDetail, notApplicable };
 }
 
 // src/lib/connectivity-probe-context.ts
@@ -8651,35 +8688,58 @@ import { execFileSync as syncExecFile } from "child_process";
 import { readFileSync as readFileSync7 } from "fs";
 import { join as join6 } from "path";
 import { dirname as dirname4 } from "path";
+function deliveryFields(i) {
+  const keys = ["source_type", "provider", "mcp_command", "mcp_url", "cli_binary", "cli_package"];
+  const out = {};
+  for (const k of keys) {
+    if (Object.prototype.hasOwnProperty.call(i, k)) out[k] = i[k];
+  }
+  return out;
+}
 async function gatherSessionToolBindProbe(agent, integrations, projectDir, opts) {
   if (integrations.length === 0) return null;
-  let mcpJson = null;
+  let mcpRaw = null;
   try {
-    mcpJson = JSON.parse(readFileSync7(join6(projectDir, ".mcp.json"), "utf-8"));
+    mcpRaw = readFileSync7(join6(projectDir, ".mcp.json"), "utf-8");
   } catch {
-    return null;
+    mcpRaw = null;
   }
-  if (!mcpJson?.mcpServers || Object.keys(mcpJson.mcpServers).length === 0) return null;
+  let parsed = null;
+  if (mcpRaw !== null) {
+    try {
+      parsed = JSON.parse(mcpRaw);
+    } catch {
+      return null;
+    }
+  }
+  const declaredServers = parsed?.mcpServers && typeof parsed.mcpServers === "object" ? parsed.mcpServers : {};
+  const mcpJson = {
+    ...parsed ?? {},
+    mcpServers: declaredServers
+  };
+  const declaredServerCount = Object.keys(declaredServers).length;
   const isolated = isolationMode(agent.code_name) === "docker";
   const hasDeclaredStdio = Object.values(mcpJson.mcpServers).some(
     (entry) => typeof entry?.command === "string"
   );
   let missingStdioKeys = /* @__PURE__ */ new Set();
-  try {
-    const psOutput = isolated ? syncExecFile("docker", ["exec", `agt-${agent.code_name}`, "ps", "-eo", "pid,ppid,args"], {
-      encoding: "utf-8",
-      timeout: 8e3
-    }) : syncExecFile("ps", ["-eo", "pid,ppid,args"], { encoding: "utf-8", timeout: 5e3 });
-    missingStdioKeys = new Set(
-      findMissingMcpServers({
-        rows: parsePsRows(psOutput),
-        codeName: agent.code_name,
-        mcpJson,
-        inContainer: isolated
-      })
-    );
-  } catch {
-    if (hasDeclaredStdio) return null;
+  if (declaredServerCount > 0) {
+    try {
+      const psOutput = isolated ? syncExecFile("docker", ["exec", `agt-${agent.code_name}`, "ps", "-eo", "pid,ppid,args"], {
+        encoding: "utf-8",
+        timeout: 8e3
+      }) : syncExecFile("ps", ["-eo", "pid,ppid,args"], { encoding: "utf-8", timeout: 5e3 });
+      missingStdioKeys = new Set(
+        findMissingMcpServers({
+          rows: parsePsRows(psOutput),
+          codeName: agent.code_name,
+          mcpJson,
+          inContainer: isolated
+        })
+      );
+    } catch {
+      if (hasDeclaredStdio) return null;
+    }
   }
   const probeEnv = buildProbeEnv(projectDir, agent.agent_id);
   const probeDeps = buildConnectivityProbeDeps(projectDir, probeEnv);
@@ -8696,6 +8756,15 @@ async function gatherSessionToolBindProbe(agent, integrations, projectDir, opts)
       definition_id: i.definition_id,
       scope: i.scope,
       last_session_tool_bind_at: i.last_session_tool_bind_at ?? null,
+      // ENG-9114: forwarded by PRESENCE, not by value. `hasDeliverySignal` in
+      // the runner asks "did the API speak about delivery at all?", and spreading
+      // these unconditionally would have made it impossible to answer: a key
+      // written as `undefined` is still an own property, so an old API's omitted
+      // columns would have looked identical to a current API's explicit nulls.
+      // Both readings are needed and they mean opposite things - explicit null is
+      // a real answer (this toolkit sets no mcp_command), absence is no answer at
+      // all. Omitting the key here is what keeps them distinguishable. (CodeRabbit)
+      ...deliveryFields(i),
       // Server-key hint for managed/remote-MCP kinds; the runner also tries the
       // raw + sanitised definition_id against the real `.mcp.json` keys.
       mcp_server_key: deriveMcpServerKey({
@@ -9630,4 +9699,4 @@ export {
   managerInstallSystemUnitCommand,
   managerUninstallSystemUnitCommand
 };
-//# sourceMappingURL=chunk-J6KQDGWM.js.map
+//# sourceMappingURL=chunk-KIOTRBOI.js.map