@integrity-labs/agt-cli 0.28.327 → 0.28.329

package/dist/bin/agt.js

@@ -38,7 +38,7 @@ import {
   success,
   table,
   warn
-} from "../chunk-EQMRT6OT.js";
+} from "../chunk-AOSBSVNB.js";
 import {
   AnchorSessionClient,
   CHANNEL_REGISTRY,
@@ -68,7 +68,7 @@ import {
   renderTemplate,
   resolveChannels,
   serializeManifestForSlackCli
-} from "../chunk-BSTFCC62.js";
+} from "../chunk-J2WYEOBH.js";
 import "../chunk-XWVM4KPK.js";
 
 // src/bin/agt.ts
@@ -4827,7 +4827,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.327" : "dev";
+var cliVersion = true ? "0.28.329" : "dev";
 async function fetchLatestVersion() {
   const host2 = getHost();
   if (!host2) return null;
@@ -5931,7 +5931,7 @@ function handleError(err) {
 }
 
 // src/bin/agt.ts
-var cliVersion2 = true ? "0.28.327" : "dev";
+var cliVersion2 = true ? "0.28.329" : "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-EQMRT6OT.js → chunk-AOSBSVNB.js}

@@ -19,7 +19,7 @@ import {
   resolveConnectivityProbe,
   worseConnectivityOutcome,
   wrapScheduledTaskPrompt
-} from "./chunk-BSTFCC62.js";
+} from "./chunk-J2WYEOBH.js";
 import {
   parsePsRows
 } from "./chunk-XWVM4KPK.js";
@@ -2616,6 +2616,13 @@ function provisionStopHook(codeName) {
     'TG_MARKER_DIR="${AGENT_DIR}/telegram-pending-inbound"',
     'SL_MARKER_DIR="${AGENT_DIR}/slack-pending-inbound"',
     'MS_MARKER_DIR="${AGENT_DIR}/msteams-pending-inbound/.markers"',
+    "# ENG-7808 (ENG-6722 fm4): direct-chat markers (written by the direct-chat MCP",
+    "# into the CODE-NAME dir, at parity with the others). direct-chat has no",
+    "# recovery-outbox yet (that's ENG-7814/E), so it participates only in the",
+    "# block-turn-end paths, never in recover_*_for. Its markers carry neither",
+    "# discretionary nor undeliverable, so the owed-reply predicate always treats",
+    "# them as owed.",
+    'DC_MARKER_DIR="${AGENT_DIR}/direct-chat-pending-inbound"',
     "# CodeRabbit ENG-4569 round-3: latest-marker correlation could leak chat",
     "# A's composed reply into chat B if B arrived later. Now correlate by",
     "# scanning the transcript for the LAST channel-source <channel ...> tag",
@@ -2632,7 +2639,8 @@ function provisionStopHook(codeName) {
     'TG_PENDING=$(pending_markers_count "$TG_MARKER_DIR")',
     'SL_PENDING=$(pending_markers_count "$SL_MARKER_DIR")',
     'MS_PENDING=$(pending_markers_count "$MS_MARKER_DIR")',
-    'if [ "$TG_PENDING" = "0" ] && [ "$SL_PENDING" = "0" ] && [ "$MS_PENDING" = "0" ]; then exit 0; fi',
+    'DC_PENDING=$(pending_markers_count "$DC_MARKER_DIR")',
+    'if [ "$TG_PENDING" = "0" ] && [ "$SL_PENDING" = "0" ] && [ "$MS_PENDING" = "0" ] && [ "$DC_PENDING" = "0" ]; then exit 0; fi',
     "# ENG-6727 (failure mode 3 \u2014 hook slowness): read the transcript tail ONCE",
     "# into a temp file and reuse it for every jq pass below (last-assistant text,",
     "# channel-tag scan, and the per-source recency check). The old hook re-ran",
@@ -2697,7 +2705,7 @@ function provisionStopHook(codeName) {
     "# 'marker present but never recovered' mystery (no <channel> tag correlated).",
     "REPLY_IN_LAST=no",
     `if echo "$TOOL_NAMES" | grep -qE '(^|__)(slack|telegram|teams)[._](reply|send_message)$'; then REPLY_IN_LAST=yes; fi`,
-    'log_ghost "stop source=${TAG_SOURCE:-none} pending(tg=$TG_PENDING sl=$SL_PENDING ms=$MS_PENDING) text_len=${#TEXT} reply_tool_in_final_turn=$REPLY_IN_LAST"',
+    'log_ghost "stop source=${TAG_SOURCE:-none} pending(tg=$TG_PENDING sl=$SL_PENDING ms=$MS_PENDING dc=$DC_PENDING) text_len=${#TEXT} reply_tool_in_final_turn=$REPLY_IN_LAST"',
     'extract_attr() { echo "$1" | grep -oE "$2=\\"[^\\"]+\\"" | head -1 | sed -E "s/$2=\\"(.*)\\"/\\\\1/"; }',
     "# Atomic write helper: jq \u2192 tmp file in same dir, then rename. The",
     "# channel-side fs.watch only fires on rename, so the file is never",
@@ -2735,6 +2743,16 @@ function provisionStopHook(codeName) {
     `  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" = "yes" ]',
     "}",
+    '# ENG-7808 (ENG-6722 fm4): "did the agent reply to THIS direct-chat session in',
+    '# the final turn?" A direct_chat.reply OR direct_chat.consume settles the',
+    "# obligation (consume is the explicit handled-without-reply ack, the same",
+    "# session-scoped drain the MCP applies), so either counts as answered. session_id",
+    "# is unambiguous (one session per marker), so there is no cross-thread class here.",
+    "replied_this_conv_directchat() {",
+    '  local session_id="$1" hit',
+    `  hit=$(printf '%s' "$LAST_ASSISTANT" | jq -r --arg sid "$session_id" '(.message.content // .content // []) | (if type=="array" then . else [] end) | (if any(.[]; (type=="object") and (.type=="tool_use") and ((.name|tostring)|test("(^|__)direct_chat[._](reply|consume)$")) and (((.input.session_id // "")|tostring)==$sid)) then "yes" else "no" end)' 2>/dev/null || echo no)`,
+    '  [ "$hit" = "yes" ]',
+    "}",
     "recover_telegram_for() {",
     '  local chat_id="$1" msg_id="$2"',
     '  if [ -z "$chat_id" ] || [ -z "$msg_id" ]; then return; fi',
@@ -2888,6 +2906,39 @@ function provisionStopHook(codeName) {
     '  rm -f "$marker_path" 2>/dev/null || true',
     '  log_ghost "teams RECOVERED conv=$(safe_id "$conversation_id") text_len=${#TEXT}"',
     "}",
+    "# ENG-7814 (ENG-6722 slice E): always-on direct-chat recovery. Re-send the last",
+    "# assistant text to the (unambiguous) session via a direct-chat-recovery-outbox the",
+    "# direct-chat MCP consumes. NO recency/replied_other guard: session_id is unambiguous",
+    "# (one session per marker), so there is no cross-thread mis-correlation risk that the",
+    "# recover_slack_for recency guard exists to prevent. Confirm-before-clear: unlike the",
+    "# slack/telegram/teams recover_*_for above (which rm the marker here), this leaves the",
+    "# marker armed and caps re-fire with a per-marker ledger; the MCP consumer clears the",
+    "# marker only after /host/direct-chat/reply confirms, and removes the ledger entry on",
+    "# success OR failure (re-arm), so a failed re-send retries on a later Stop.",
+    "recover_directchat_for() {",
+    '  local session_id="$1" hex_session="$2"',
+    '  if [ -z "$session_id" ] || [ -z "$hex_session" ]; then return; fi',
+    '  local marker_path=""',
+    "  shopt -s nullglob",
+    '  for f in "$DC_MARKER_DIR"/${hex_session}__*.json; do',
+    '    if [ -z "$marker_path" ]; then marker_path="$f"; fi',
+    "  done",
+    '  if [ -z "$marker_path" ]; then log_ghost "direct-chat skip=no_pending_marker session=$(safe_id "$session_id")"; return; fi',
+    '  local marker_name; marker_name="$(basename "$marker_path")"',
+    "  # Already recovered once and awaiting delivery confirmation \u21D2 do not re-send.",
+    '  if [ -f "${DC_RECOVERY_LEDGER_DIR}/${marker_name}" ]; then log_ghost "direct-chat skip=recovery_in_flight session=$(safe_id "$session_id")"; return; fi',
+    "  # Persist the per-marker cap BEFORE the side effect (same fail-safe as",
+    "  # block-turn-end): if the ledger dir/file write fails we could re-recover every",
+    "  # Stop, so degrade to a logged skip and write no payload.",
+    '  if ! mkdir -p "$DC_RECOVERY_LEDGER_DIR" 2>/dev/null; then log_ghost "direct-chat skip=recovery_ledger_unwritable session=$(safe_id "$session_id")"; return; fi',
+    '  if ! : > "${DC_RECOVERY_LEDGER_DIR}/${marker_name}" 2>/dev/null; then log_ghost "direct-chat skip=recovery_ledger_unwritable session=$(safe_id "$session_id")"; return; fi',
+    "  local TS",
+    "  TS=$(date -u +%Y%m%dT%H%M%S%N)",
+    '  atomic_write_payload "$DC_RECOVERY_OUTBOX_DIR" "${TS}.json" \\',
+    `    '{session_id:$s, text:$t, marker_name:$mn, source:"ghost-reply-recovery"}' \\`,
+    '    --arg s "$session_id" --arg t "$TEXT" --arg mn "$marker_name"',
+    '  log_ghost "direct-chat RECOVERED session=$(safe_id "$session_id") text_len=${#TEXT}"',
+    "}",
     '# ENG-6467 / ADR-0024 Slice 2.5 \u2014 block-turn-end (the D1 "composed-but-unsent"',
     "# fix), gated dark behind AGT_CHANNEL_BLOCK_TURN_END_ENABLED (registry flag",
     "# channel-block-turn-end; the env var is the operator/canary override the bash",
@@ -2915,6 +2966,21 @@ function provisionStopHook(codeName) {
     'BLOCK_LEDGER_DIR="${AGENT_DIR}/.agt-block-turn-end-ledger"',
     "# GC stale ledger entries (>1 day) so the per-marker cap dir cannot grow unbounded.",
     'if [ -d "$BLOCK_LEDGER_DIR" ]; then find "$BLOCK_LEDGER_DIR" -type f -mtime +1 -delete 2>/dev/null || true; fi',
+    "# ENG-7814 (ENG-6722 slice E): always-on direct-chat recovery. When ON, an owed",
+    "# and unanswered direct-chat inbound that block-turn-end did NOT handle gets its",
+    "# last assistant text written to a direct-chat-recovery-outbox that the direct-chat",
+    "# MCP re-sends via /host/direct-chat/reply. Registry flag direct-chat-recovery; the",
+    "# env var is the operator/host override the bash reads directly. OFF (default) keeps",
+    "# today's behavior (direct-chat gets block-turn-end only, no recovery).",
+    "DC_RECOVERY_ON=0",
+    'case "${AGT_DIRECT_CHAT_RECOVERY_ENABLED:-}" in true|1|TRUE|True) DC_RECOVERY_ON=1;; esac',
+    'DC_RECOVERY_OUTBOX_DIR="${AGENT_DIR}/direct-chat-recovery-outbox"',
+    "# Per-marker recovery ledger: caps ONE in-flight recovery per inbound (so a marker",
+    "# left pending for confirm-before-clear is not re-recovered every Stop). The MCP",
+    "# consumer removes the entry on delivery success OR failure (re-arm); we GC stale",
+    "# entries here as a backstop, same as the block ledger.",
+    'DC_RECOVERY_LEDGER_DIR="${AGENT_DIR}/.agt-direct-chat-recovery-ledger"',
+    'if [ -d "$DC_RECOVERY_LEDGER_DIR" ]; then find "$DC_RECOVERY_LEDGER_DIR" -type f -mtime +1 -delete 2>/dev/null || true; fi',
     "# Returns 0 (after printing the block JSON to stdout) when it blocked; 1 otherwise.",
     '#   $1 = "yes"/"no" \u2014 did the agent reply to THIS conversation in the final turn?',
     "#        (ENG-6727: per-conversation, computed by the caller via replied_this_conv_*)",
@@ -2972,8 +3038,8 @@ function provisionStopHook(codeName) {
     '  if [ "$BLOCK_TURN_END_ON" != "1" ] || [ "$BLOCK_ALL_MARKERS_ON" != "1" ]; then return 1; fi',
     '  if [ "$STOP_ACTIVE" = "true" ]; then return 1; fi',
     "  local -a owed_paths=()",
-    '  local owed_lines="" owed_count=0 any_capped=0 entry src dir m d u replied ch th cid conv iid key led_name',
-    '  for entry in "slack:$SL_MARKER_DIR" "telegram:$TG_MARKER_DIR" "msteams:$MS_MARKER_DIR"; do',
+    '  local owed_lines="" owed_count=0 any_capped=0 entry src dir m d u replied ch th cid conv sess iid key led_name',
+    '  for entry in "slack:$SL_MARKER_DIR" "telegram:$TG_MARKER_DIR" "msteams:$MS_MARKER_DIR" "direct-chat:$DC_MARKER_DIR"; do',
     '    src="${entry%%:*}"; dir="${entry#*:}"',
     '    if [ ! -d "$dir" ]; then continue; fi',
     "    shopt -s nullglob",
@@ -2982,11 +3048,12 @@ function provisionStopHook(codeName) {
     `      d=$(jq -r '.discretionary // false' "$m" 2>/dev/null || echo true)`,
     `      u=$(jq -r '.undeliverable // false' "$m" 2>/dev/null || echo true)`,
     '      if [ "$d" = "true" ] || [ "$u" = "true" ]; then continue; fi',
-    '      replied=no; ch=""; th=""; cid=""; conv=""',
+    '      replied=no; ch=""; th=""; cid=""; conv=""; sess=""',
     '      case "$src" in',
     `        slack) ch=$(jq -r '.channel // ""' "$m" 2>/dev/null || echo ""); th=$(jq -r '.thread_ts // ""' "$m" 2>/dev/null || echo ""); if replied_this_conv_slack "$ch" "$th"; then replied=yes; fi ;;`,
     `        telegram) cid=$(jq -r '.chat_id // ""' "$m" 2>/dev/null || echo ""); if replied_this_conv_telegram "$cid"; then replied=yes; fi ;;`,
     `        msteams) conv=$(jq -r '.conversation_id // ""' "$m" 2>/dev/null || echo ""); if replied_this_conv_teams "$conv"; then replied=yes; fi ;;`,
+    `        direct-chat) sess=$(jq -r '.session_id // ""' "$m" 2>/dev/null || echo ""); if replied_this_conv_directchat "$sess"; then replied=yes; fi ;;`,
     "      esac",
     '      if [ "$replied" = "yes" ]; then continue; fi',
     '      led_name="$(basename "$m")"',
@@ -2996,6 +3063,7 @@ function provisionStopHook(codeName) {
     '        slack) key="thread=$(safe_id "$th")" ;;',
     '        telegram) key="chat=$(safe_id "$cid")" ;;',
     '        msteams) key="conv=$(safe_id "$conv")" ;;',
+    '        direct-chat) key="session=$(safe_id "$sess")" ;;',
     '        *) key="" ;;',
     "      esac",
     "      owed_count=$((owed_count + 1))",
@@ -3061,6 +3129,30 @@ function provisionStopHook(codeName) {
     `    if emit_block_if_obligated "$MS_REPLIED" "$MS_CAND" 'teams.reply'; then exit 0; fi`,
     "  fi",
     '  recover_teams_for "$CONVERSATION_ID" "$REPLY_TO_ID" "$SERVICE_URL"',
+    'elif [ "$TAG_SOURCE" = "direct-chat" ]; then',
+    "  # ENG-7808 (ENG-6722 fm4) + ENG-7814 (slice E): direct-chat backstop. First try",
+    "  # block-turn-end (make the MODEL re-send). If that does not fire and the flag",
+    "  # direct-chat-recovery is ON, re-send the composed text via the recovery-outbox",
+    "  # (ENG-7814). session_id is unambiguous (one session per marker), so there is no",
+    "  # cross-thread class and no recency guard is needed. With both off we just log the",
+    "  # miss and stand down.",
+    '  SESSION_ID=$(extract_attr "$CHANNEL_TAG" "session_id")',
+    '  if [ -n "$SESSION_ID" ]; then',
+    "    shopt -s nullglob",
+    "    # Marker filenames hex-encode the session (injective; matches the MCP",
+    "    # writer + the msteams pattern). Reproduce that hex here for the glob.",
+    '    HEX_SESSION=$(printf %s "$SESSION_ID" | od -An -tx1 | tr -d " \\n")',
+    "    DC_CAND=$(printf '%s\\n' \"${DC_MARKER_DIR}/${HEX_SESSION}__\"*.json)",
+    '    DC_REPLIED=no; if replied_this_conv_directchat "$SESSION_ID"; then DC_REPLIED=yes; fi',
+    `    if emit_block_if_obligated "$DC_REPLIED" "$DC_CAND" 'direct_chat.reply'; then exit 0; fi`,
+    '    if [ "$DC_REPLIED" = "no" ]; then',
+    '      if [ "$DC_RECOVERY_ON" = "1" ]; then',
+    '        recover_directchat_for "$SESSION_ID" "$HEX_SESSION"',
+    "      else",
+    '        log_ghost "direct-chat skip=no_recovery_path session=$(safe_id "$SESSION_ID") (block-turn-end off/capped; direct-chat-recovery off)"',
+    "      fi",
+    "    fi",
+    "  fi",
     'elif [ -z "$TAG_SOURCE" ]; then',
     "  # ENG-6467: TAG_SOURCE=none fallback (AC1). Guarded on an EMPTY source, not",
     "  # a catch-all else: a parsed-but-unsupported source (e.g. direct-chat, which",
@@ -6705,7 +6797,7 @@ function requireHost() {
 }
 
 // src/lib/api-client.ts
-var agtCliVersion = true ? "0.28.327" : "dev";
+var agtCliVersion = true ? "0.28.329" : "dev";
 var lastConfigHash = null;
 function setConfigHash(hash) {
   lastConfigHash = hash && hash.length > 0 ? hash : null;
@@ -9102,4 +9194,4 @@ export {
   managerInstallSystemUnitCommand,
   managerUninstallSystemUnitCommand
 };
-//# sourceMappingURL=chunk-EQMRT6OT.js.map
+//# sourceMappingURL=chunk-AOSBSVNB.js.map