@integrity-labs/agt-cli 0.28.468 → 0.28.470

package/dist/bin/agt.js

@@ -40,7 +40,7 @@ import {
   success,
   table,
   warn
-} from "../chunk-Y2XU2XWM.js";
+} from "../chunk-BFCHF4XF.js";
 import {
   AnchorSessionClient,
   CHANNEL_REGISTRY,
@@ -4830,7 +4830,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.468" : "dev";
+var cliVersion = true ? "0.28.470" : "dev";
 async function fetchLatestVersion() {
   const host2 = getHost();
   if (!host2) return null;
@@ -6002,7 +6002,7 @@ function handleError(err) {
 }
 
 // src/bin/agt.ts
-var cliVersion2 = true ? "0.28.468" : "dev";
+var cliVersion2 = true ? "0.28.470" : "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-Y2XU2XWM.js → chunk-BFCHF4XF.js}

@@ -4932,7 +4932,7 @@ function exchangeFailureKind(err) {
 }
 
 // src/lib/api-client.ts
-var agtCliVersion = true ? "0.28.468" : "dev";
+var agtCliVersion = true ? "0.28.470" : "dev";
 var lastConfigHash = null;
 function setConfigHash(hash) {
   lastConfigHash = hash && hash.length > 0 ? hash : null;
@@ -7661,4 +7661,4 @@ export {
   managerInstallSystemUnitCommand,
   managerUninstallSystemUnitCommand
 };
-//# sourceMappingURL=chunk-Y2XU2XWM.js.map
+//# sourceMappingURL=chunk-BFCHF4XF.js.map

package/dist/lib/manager-worker.js

@@ -44,7 +44,7 @@ import {
   resolveEffectivePinRaw,
   safeWriteJsonAtomic,
   setConfigHash
-} from "../chunk-Y2XU2XWM.js";
+} from "../chunk-BFCHF4XF.js";
 import {
   getProjectDir as getProjectDir2,
   getReadyTasks,
@@ -10038,7 +10038,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.468" : "dev";
+var agtCliVersion = true ? "0.28.470" : "dev";
 function resolveBrewPath(execFileSync2) {
   try {
     const out = execFileSync2("which", ["brew"], { timeout: 5e3 }).toString().trim();

package/dist/mcp/slack-channel.js

@@ -38733,6 +38733,31 @@ function describeChannelRedirect(input) {
   return `Heads up: this was delivered to ${effectiveChannel}, NOT the ${requestedChannel} you named${why}. routed_to=${effectiveChannel}. If ${effectiveChannel} is not where you intended this to go, the destination is misconfigured; do not assume it posted to ${requestedChannel}.`;
 }
 
+// src/channel-send-destination.ts
+function describeSendDestination(dest) {
+  const where = dest.threadTs ? `channel=${dest.channel} thread_ts=${dest.threadTs}` : `channel=${dest.channel} at the channel root (no thread)`;
+  return `\u2192 ${where}`;
+}
+function describeSendFallback(dest) {
+  if (!dest.fellBack) return null;
+  switch (dest.reason) {
+    case "unresolved_inbound": {
+      const id = dest.requestedInboundId ? ` "${dest.requestedInboundId}"` : "";
+      const landed = dest.threadTs ? `Your coordinates were used instead and it threaded correctly.` : `Your coordinates did not name a thread either, so it posted at the CHANNEL ROOT \u2014 if you meant to continue a thread, resend with an explicit thread_ts.`;
+      return `WARNING: the inbound_id${id} you named could not be resolved (it may be stale, or this process restarted since that message was answered). ${landed}`;
+    }
+    case "scheduled":
+      return `WARNING: this was routed to the active scheduled-task destination rather than the channel you named. If you meant to answer a person, name their inbound_id.`;
+    default:
+      return null;
+  }
+}
+function describeSendOutcome(dest) {
+  const warning = describeSendFallback(dest);
+  const where = describeSendDestination(dest);
+  return warning ? `${warning} ${where}` : where;
+}
+
 // src/slack-reply-binding-telemetry.ts
 import { readFileSync as readFileSync17, writeFileSync as writeFileSync11 } from "fs";
 import { join as join17 } from "path";
@@ -38918,11 +38943,15 @@ function scheduledDestOverridesNamedChannel(opts) {
   return opts.scheduledTargetChannel !== opts.namedChannel;
 }
 function resolveSlackScheduledPriority(opts) {
-  const inboundWins = !!(opts.inboundId || opts.enforceBound);
-  const effectiveChannel = inboundWins ? opts.bindingChannel ?? opts.fallbackChannel : opts.scheduledDest ? opts.scheduledDest.channel : opts.fallbackChannel;
-  const effectiveThreadTs = inboundWins ? opts.bindingThreadTs ?? void 0 : opts.scheduledDest ? opts.scheduledDest.thread_ts : opts.fallbackThreadTs ?? void 0;
+  const inboundUnresolved = !!opts.inboundId && opts.inboundResolved === false;
+  const inboundWins = !!(opts.inboundId && !inboundUnresolved || opts.enforceBound);
+  const effectiveChannel = inboundWins ? opts.bindingChannel ?? opts.fallbackChannel : opts.scheduledDest && !inboundUnresolved ? opts.scheduledDest.channel : opts.fallbackChannel;
+  const effectiveThreadTs = opts.forcedRoot ? void 0 : inboundWins ? opts.bindingThreadTs ?? void 0 : opts.scheduledDest && !inboundUnresolved ? opts.scheduledDest.thread_ts : opts.fallbackThreadTs ?? void 0;
   const settleThreadTs = inboundWins ? opts.bindingThreadTs ?? void 0 : opts.fallbackSettleThreadTs ?? void 0;
-  return { effectiveChannel, effectiveThreadTs, settleThreadTs };
+  const scheduledRetarget = !inboundWins && !inboundUnresolved && !!opts.scheduledDest;
+  const fellBack = inboundUnresolved || scheduledRetarget;
+  const reason = inboundUnresolved ? "unresolved_inbound" : scheduledRetarget ? "scheduled" : opts.forcedRoot ? "forced_root" : inboundWins ? "bound" : "fallback";
+  return { effectiveChannel, effectiveThreadTs, settleThreadTs, reason, fellBack };
 }
 
 // src/slack-allowlist-source.ts
@@ -42695,7 +42724,7 @@ mcp.setRequestHandler(ListToolsRequestSchema, async () => ({
           // WS2 (cross-thread reply routing): the preferred way to target a reply.
           inbound_id: {
             type: "string",
-            description: `The inbound_id attribute of the <channel> tag you are answering (e.g. "ib_a1b2c3d4e5f6"). ALWAYS include it when replying to a message - it is STRONGLY PREFERRED over channel/thread_ts/message_ts because the server resolves the exact destination (channel AND thread) from it, so your reply can never land in another user's channel or thread. Copy it verbatim from the tag of the message you are replying to. Omit it only for a proactive (self-initiated) send.`
+            description: `The inbound_id attribute of the <channel> tag you are answering (e.g. "ib_a1b2c3d4e5f6"). ALWAYS include it when replying to a message - it is STRONGLY PREFERRED over channel/thread_ts/message_ts because the server resolves the exact destination (channel AND thread) from it, so your reply can never land in another user's channel or thread. Copy it verbatim from the tag of the message you are replying to. Omit it only for a proactive (self-initiated) send. NOTE (ENG-8280): an inbound_id can go stale - it stops resolving if this process restarted after you already answered that message. A stale one no longer silently posts at the channel root: the server falls back to your channel/thread_ts coordinates and the response tells you it did. For a self-initiated FOLLOW-UP into a thread you already answered, prefer passing thread_ts explicitly rather than re-using the old inbound_id.`
           },
           proactive: {
             type: "boolean",
@@ -42707,7 +42736,7 @@ mcp.setRequestHandler(ListToolsRequestSchema, async () => ({
           // conversation is a separate, deliberate action signalled here.
           to_channel_root: {
             type: "boolean",
-            description: "Set true to DELIBERATELY post a new top-level message at the channel root instead of threading into the conversation you are currently working in. Use it only when you genuinely want to start a new topic in the channel (not a reply to, or a continuation of, an existing thread). Leave it off (default) for anything that belongs in an ongoing thread."
+            description: "Set true to DELIBERATELY post a new top-level message at the channel root instead of threading into the conversation you are currently working in. Use it only when you genuinely want to start a new topic in the channel (not a reply to, or a continuation of, an existing thread). Leave it off (default) for anything that belongs in an ongoing thread. It now overrides every other destination signal including inbound_id (ENG-8280 - previously it was silently ignored whenever inbound_id was also passed)."
           }
         },
         required: ["channel", "text"]
@@ -43119,8 +43148,26 @@ mcp.setRequestHandler(CallToolRequestSchema, async (req) => {
         }
       }
     }
-    const { effectiveChannel, effectiveThreadTs, settleThreadTs } = resolveSlackScheduledPriority({
+    const {
+      effectiveChannel,
+      effectiveThreadTs,
+      settleThreadTs,
+      reason: routingReason,
+      fellBack: routingFellBack
+    } = resolveSlackScheduledPriority({
       inboundId: inbound_id,
+      // ENG-8280: tell the resolver whether the inbound_id actually resolved.
+      // `binding.unknownInboundId` was already computed here and fed to
+      // telemetry, but never to routing — so an id the registry could not find
+      // was still treated as an authoritative "post nowhere", discarding the
+      // message_ts / active-card / hot-thread fallbacks and landing the reply
+      // at the channel root with the agent told only `sent`.
+      inboundResolved: !binding.unknownInboundId,
+      // ENG-8280: an explicit to_channel_root (or the thread_ts:null sentinel)
+      // was previously honoured ONLY on the plain-reply branch, so pairing it
+      // with an inbound_id silently ignored it. It is the least ambiguous
+      // instruction this tool takes and now wins everywhere.
+      forcedRoot,
       enforceBound,
       scheduledDest: scheduledDest ?? null,
       bindingChannel: binding.channel,
@@ -43289,6 +43336,19 @@ mcp.setRequestHandler(CallToolRequestSchema, async (req) => {
           );
         }
       }
+      base += `. ${describeSendOutcome({
+        channel: effectiveChannel,
+        threadTs: effectiveThreadTs,
+        reason: routingReason,
+        fellBack: routingFellBack,
+        requestedInboundId: inbound_id
+      })}`;
+      if (routingFellBack) {
+        process.stderr.write(
+          `slack-channel(${AGENT_CODE_NAME}): send_fell_back reason=${routingReason} channel=${redactSlackId(effectiveChannel)} thread=${redactSlackId(effectiveThreadTs)} inbound_id=${redactSlackId(inbound_id ?? void 0)}
+`
+        );
+      }
       const sentTs = typeof data.ts === "string" ? data.ts : void 0;
       await maybeSendSlackAccountWarn({ channel: effectiveChannel, threadTs: effectiveThreadTs });
       return {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@integrity-labs/agt-cli",
-  "version": "0.28.468",
+  "version": "0.28.470",
   "description": "Augmented Team CLI — agent provisioning and management",
   "type": "module",
   "engines": {