@integrity-labs/agt-cli 0.28.421 → 0.28.422

package/dist/bin/agt.js

@@ -40,7 +40,7 @@ import {
   success,
   table,
   warn
-} from "../chunk-HVVEMQWZ.js";
+} from "../chunk-6JC5YLU5.js";
 import {
   AnchorSessionClient,
   CHANNEL_REGISTRY,
@@ -70,7 +70,7 @@ import {
   renderTemplate,
   resolveChannels,
   serializeManifestForSlackCli
-} from "../chunk-B4NX2T2H.js";
+} from "../chunk-XFC7YVGS.js";
 import "../chunk-XWVM4KPK.js";
 
 // src/bin/agt.ts
@@ -4829,7 +4829,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.421" : "dev";
+var cliVersion = true ? "0.28.422" : "dev";
 async function fetchLatestVersion() {
   const host2 = getHost();
   if (!host2) return null;
@@ -6001,7 +6001,7 @@ function handleError(err) {
 }
 
 // src/bin/agt.ts
-var cliVersion2 = true ? "0.28.421" : "dev";
+var cliVersion2 = true ? "0.28.422" : "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-HVVEMQWZ.js → chunk-6JC5YLU5.js}

@@ -25,7 +25,7 @@ import {
   resolveConnectivityProbe,
   worseConnectivityOutcome,
   wrapScheduledTaskPrompt
-} from "./chunk-B4NX2T2H.js";
+} from "./chunk-XFC7YVGS.js";
 import {
   parsePsRows
 } from "./chunk-XWVM4KPK.js";
@@ -2392,6 +2392,17 @@ function provisionStopHook(codeName) {
     'chmod 600 "$RECENT_FILE" 2>/dev/null || true',
     `trap 'rm -f "$RECENT_FILE" 2>/dev/null || true' EXIT`,
     'tail -400 "$TRANSCRIPT_PATH" > "$RECENT_FILE" 2>/dev/null || true',
+    "# ENG-8142: collect the tool_use ids whose tool_result came back",
+    "# `is_error: true`. A REFUSED reply is not a delivered reply, but the",
+    "# replied_this_conv_* predicates below only ever saw the tool_use half of",
+    '# the pair, so any errored exit read as "the agent replied" \u2014 returning',
+    "# early before the recovery outbox is written AND setting SL_REPLIED=yes so",
+    "# emit_block_if_obligated skipped the owed-reply re-prompt. Both halves of",
+    "# the safety net stood down and the human just saw silence. slack.reply",
+    "# alone has eight isError branches, so this is a whole class, not one bug.",
+    "# Computed once here (not per predicate) to hold the ENG-6288 budget of a",
+    "# single jq pass over the snapshot.",
+    `ERRORED_TOOL_IDS=$(jq -r '(.message.content // .content // []) | if type=="array" then .[] else empty end | select((type=="object") and (.type=="tool_result") and (.is_error==true)) | (.tool_use_id // empty)' "$RECENT_FILE" 2>/dev/null | sort -u || true)`,
     `LAST_ASSISTANT=$(jq -cs '[.[] | select(.type == "assistant" or .role == "assistant")] | last // empty' "$RECENT_FILE" 2>/dev/null || true)`,
     'if [ -z "$LAST_ASSISTANT" ] || [ "$LAST_ASSISTANT" = "null" ]; then exit 0; fi',
     "# Assistant content is array-shaped today; normalize anyway so a shape",
@@ -2467,19 +2478,24 @@ function provisionStopHook(codeName) {
     "# (the chronic sherlock symptom). These scope the check to the conversation",
     "# keyed by the inbound tag, comparing the reply tool's input against it.",
     '# Output "yes"/"no" (never a jq nonzero exit) to stay clear of the ERR trap.',
+    "# ENG-8142: `$bad` is the errored-tool_use id set; a tool_use whose id is in",
+    "# it was REFUSED, so it must not count as a reply. A tool_use with no `.id`",
+    "# cannot be correlated and deliberately still counts as a reply \u2014 that is the",
+    "# pre-ENG-8142 behaviour, and it is the safe direction, because the failure",
+    "# mode of guessing the other way is double-delivering to the customer.",
     "replied_this_conv_slack() {",
     '  local channel="$1" thread_ts="$2" hit',
-    `  hit=$(printf '%s' "$LAST_ASSISTANT" | jq -r --arg ch "$channel" --arg th "$thread_ts" '(.message.content // .content // []) | (if type=="array" then . else [] end) | (if any(.[]; (type=="object") and (.type=="tool_use") and ((.name|tostring)|test("(^|__)slack[._]reply$")) and ((.input.channel // "") as $c | (.input.thread_ts // "") as $t | (.input.message_ts // "") as $m | (($th != "" and ($t==$th or $m==$th)) or ($th=="" and $c!="" and $c==$ch)))) then "yes" else "no" end)' 2>/dev/null || echo no)`,
+    `  hit=$(printf '%s' "$LAST_ASSISTANT" | jq -r --arg ch "$channel" --arg th "$thread_ts" --arg errids "$ERRORED_TOOL_IDS" '(($errids // "") | split("\\n") | map(select(length > 0))) as $bad | (.message.content // .content // []) | (if type=="array" then . else [] end) | (if any(.[]; (type=="object") and (.type=="tool_use") and ((.name|tostring)|test("(^|__)slack[._]reply$")) and ((.id // "") as $tid | ($bad | index($tid)) == null) and ((.input.channel // "") as $c | (.input.thread_ts // "") as $t | (.input.message_ts // "") as $m | (($th != "" and ($t==$th or $m==$th)) or ($th=="" and $c!="" and $c==$ch)))) then "yes" else "no" end)' 2>/dev/null || echo no)`,
     '  [ "$hit" = "yes" ]',
     "}",
     "replied_this_conv_telegram() {",
     '  local chat_id="$1" hit',
-    `  hit=$(printf '%s' "$LAST_ASSISTANT" | jq -r --arg cid "$chat_id" '(.message.content // .content // []) | (if type=="array" then . else [] end) | (if any(.[]; (type=="object") and (.type=="tool_use") and ((.name|tostring)|test("(^|__)telegram[._](reply|send_message)$")) and (((.input.chat_id // "")|tostring)==$cid)) then "yes" else "no" end)' 2>/dev/null || echo no)`,
+    `  hit=$(printf '%s' "$LAST_ASSISTANT" | jq -r --arg cid "$chat_id" --arg errids "$ERRORED_TOOL_IDS" '(($errids // "") | split("\\n") | map(select(length > 0))) as $bad | (.message.content // .content // []) | (if type=="array" then . else [] end) | (if any(.[]; (type=="object") and (.type=="tool_use") and ((.name|tostring)|test("(^|__)telegram[._](reply|send_message)$")) and ((.id // "") as $tid | ($bad | index($tid)) == null) and (((.input.chat_id // "")|tostring)==$cid)) then "yes" else "no" end)' 2>/dev/null || echo no)`,
     '  [ "$hit" = "yes" ]',
     "}",
     "replied_this_conv_teams() {",
     '  local conversation_id="$1" hit',
-    `  hit=$(printf '%s' "$LAST_ASSISTANT" | jq -r --arg cid "$conversation_id" '(.message.content // .content // []) | (if type=="array" then . else [] end) | (if any(.[]; (type=="object") and (.type=="tool_use") and ((.name|tostring)|test("(^|__)teams[._]reply$")) and (((.input.conversation_id // "")|tostring)==$cid)) then "yes" else "no" end)' 2>/dev/null || echo no)`,
+    `  hit=$(printf '%s' "$LAST_ASSISTANT" | jq -r --arg cid "$conversation_id" --arg errids "$ERRORED_TOOL_IDS" '(($errids // "") | split("\\n") | map(select(length > 0))) as $bad | (.message.content // .content // []) | (if type=="array" then . else [] end) | (if any(.[]; (type=="object") and (.type=="tool_use") and ((.name|tostring)|test("(^|__)teams[._]reply$")) and ((.id // "") as $tid | ($bad | index($tid)) == null) and (((.input.conversation_id // "")|tostring)==$cid)) then "yes" else "no" end)' 2>/dev/null || echo no)`,
     '  [ "$hit" = "yes" ]',
     "}",
     '# ENG-7808 (ENG-6722 fm4): "did the agent reply to THIS direct-chat session in',
@@ -6011,7 +6027,7 @@ function requireHost() {
 }
 
 // src/lib/api-client.ts
-var agtCliVersion = true ? "0.28.421" : "dev";
+var agtCliVersion = true ? "0.28.422" : "dev";
 var lastConfigHash = null;
 function setConfigHash(hash) {
   lastConfigHash = hash && hash.length > 0 ? hash : null;
@@ -8515,4 +8531,4 @@ export {
   managerInstallSystemUnitCommand,
   managerUninstallSystemUnitCommand
 };
-//# sourceMappingURL=chunk-HVVEMQWZ.js.map
+//# sourceMappingURL=chunk-6JC5YLU5.js.map