@integrity-labs/agt-cli 0.28.748 → 0.28.750

package/dist/lib/manager-worker.js

@@ -57,7 +57,7 @@ import {
   safeWriteJsonAtomic,
   setConfigHash,
   tripClass
-} from "../chunk-G7GWXCPV.js";
+} from "../chunk-5GJ2JEFA.js";
 import {
   getProjectDir as getProjectDir2,
   getReadyTasks,
@@ -1695,7 +1695,28 @@ async function runConnectivityProbes(integrations, options = {}) {
       // assumed. Defaults to 'none' rather than being omitted — an absent field
       // from an older manager and a probe that contacted nothing must land in
       // the same, conservative place.
-      evidence: outcome.evidence ?? "none"
+      evidence: outcome.evidence ?? "none",
+      // ENG-9173's other half. That ticket added `last_connectivity_cause` and
+      // the API-side write for it, and the probes compute a cause today
+      // (`causeForHttp` in connectivity-http-probes.ts, and the did-not-run
+      // branches in cli-probe.ts). This line is what puts it ON THE WIRE.
+      //
+      // Without it `ConnectivityReport` had no `cause` field at all, so the
+      // host lane never sent one and the API's
+      // `last_connectivity_cause: item.cause ?? 'unknown'` resolved to
+      // `'unknown'` for EVERY host-probed row on the fleet, forever. The column
+      // added to stop an escalation boundary matching prose was itself never
+      // populated by the lane that does almost all the probing — the identical
+      // parse-then-drop the ENG-9173 header describes, one layer further up.
+      //
+      // Omitted rather than defaulted when absent: unlike `evidence`, where a
+      // missing value and "contacted nothing" must both land conservatively on
+      // `'none'`, the API already coerces an absent or unrecognised cause to
+      // `'unknown'` (`isConnectivityCause`, integration-connectivity-report.ts).
+      // Sending an explicit `'unknown'` would be indistinguishable from an older
+      // manager that cannot classify at all, and that distinction is worth
+      // keeping while this rolls out across hosts.
+      ...outcome.cause ? { cause: outcome.cause } : {}
     });
   }
   return { reports, due: due.length, probed: batch.length, skipped };
@@ -4010,6 +4031,39 @@ function detectMissedIdentifiers(turns, candidates) {
   }
   return out;
 }
+async function salvageIdentifiers(args) {
+  const forced = detectMissedIdentifiers(args.turns, []);
+  if (forced.length === 0) return 0;
+  const scrubbed = forced.map((c) => ({
+    ...c,
+    content: scrubSensitive(c.content),
+    rationale: scrubSensitive(c.rationale)
+  }));
+  try {
+    const res = await args.api.post(
+      "/host/memories/identifiers",
+      {
+        agent_id: args.agentId,
+        conversation_id: args.conversationId,
+        candidates: scrubbed
+      }
+    );
+    if (!res?.ok) return 0;
+    return res.promoted ?? 0;
+  } catch {
+    return 0;
+  }
+}
+async function salvageOnce(st, conversationId, fn) {
+  if (st.salvaged.has(conversationId)) return 0;
+  st.salvaged.add(conversationId);
+  return fn();
+}
+function pruneSalvaged(st, stillPending) {
+  for (const id of st.salvaged) {
+    if (!stillPending.has(id)) st.salvaged.delete(id);
+  }
+}
 async function maybeExtractMemories(args) {
   const { api: api2, backend, codeName, agentId, log: log2 } = args;
   const now = args.now ?? /* @__PURE__ */ new Date();
@@ -4018,7 +4072,8 @@ async function maybeExtractMemories(args) {
   if (existing && nowMs - existing.lastCheckedAt < MIN_CHECK_INTERVAL_MS5) {
     return;
   }
-  state5.set(codeName, { lastCheckedAt: nowMs });
+  const st = { lastCheckedAt: nowMs, salvaged: existing?.salvaged ?? /* @__PURE__ */ new Set() };
+  state5.set(codeName, st);
   let pending2;
   try {
     const resp = await api2.get(
@@ -4029,6 +4084,7 @@ async function maybeExtractMemories(args) {
     log2(`[memory-extract] ${codeName}: pending fetch failed: ${err.message}`);
     return;
   }
+  pruneSalvaged(st, new Set(pending2.map((p) => p.conversation_id)));
   if (pending2.length === 0) return;
   const dir = args.transcriptDir ?? sessionTranscriptDir(getProjectDir(codeName));
   const allTurns = readRecentTurns(dir, nowMs);
@@ -4038,6 +4094,7 @@ async function maybeExtractMemories(args) {
     }
     return;
   }
+  let salvaged = 0;
   for (const conv of pending2) {
     const windowStart = Date.parse(conv.started_at) - WINDOW_PAD_MS2;
     const windowEnd = Date.parse(conv.last_message_at) + WINDOW_PAD_MS2;
@@ -4058,10 +4115,20 @@ async function maybeExtractMemories(args) {
     } catch (err) {
       log2(`[memory-extract] ${codeName}: extraction failed: ${err.message}`);
       args.onModelCallError?.(err);
+      salvaged += await salvageOnce(
+        st,
+        conv.conversation_id,
+        () => salvageIdentifiers({ api: api2, agentId, conversationId: conv.conversation_id, turns, log: log2, codeName })
+      );
       continue;
     }
     if (candidates === null) {
       log2(`[memory-extract] ${codeName}: unparseable output for ${conv.conversation_id.slice(0, 8)} \u2014 will retry`);
+      salvaged += await salvageOnce(
+        st,
+        conv.conversation_id,
+        () => salvageIdentifiers({ api: api2, agentId, conversationId: conv.conversation_id, turns, log: log2, codeName })
+      );
       continue;
     }
     const forced = detectMissedIdentifiers(turns, candidates);
@@ -4098,6 +4165,9 @@ async function maybeExtractMemories(args) {
       onModelCallError: args.onModelCallError
     });
   }
+  if (salvaged > 0) {
+    log2(`[memory-extract] ${codeName}: identifiers_salvaged=${salvaged} (model call failed)`);
+  }
 }
 async function reportSkip2(api2, agentId, conversationId, log2, codeName) {
   try {
@@ -13321,7 +13391,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.748" : "dev";
+var agtCliVersion = true ? "0.28.750" : "dev";
 function resolveBrewPath(execFileSync3) {
   try {
     const out = execFileSync3("which", ["brew"], { timeout: 5e3 }).toString().trim();