slack-term 1.51.2 → 1.53.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "slack-term",
-  "version": "1.51.2",
+  "version": "1.53.0",
   "repository": {
     "type": "git",
     "url": "https://github.com/snomiao/slack-term.git"

package/ts/askPush.ts

@@ -0,0 +1,83 @@
+// `slack ask --pending --push`: hand an answered question to the agent that
+// asked it, instead of waiting for that agent to poll.
+//
+// Why: a pill pressed while the asker had moved on was never read back
+// (2026-09-27: 13 of 71 asks). `--pending` lists them, but only to whoever
+// runs it; a wrapper outside slack-term pushed them over `ay`, and that now
+// lives here instead. The sent-log already holds the asking
+// agent's pid, so slack-term can tell it directly.
+import { spawnSync, type SpawnSyncReturns } from "node:child_process";
+import { closeSync, mkdirSync, openSync, readFileSync, unlinkSync, writeFileSync } from "node:fs";
+import { dirname } from "node:path";
+import { pidAlive } from "./sentlog.ts";
+
+/** ay rejects messages over 1024 chars. */
+export const PUSH_MAX = 1000;
+
+function clip(s: string, n: number): string {
+  const one = s.split(/\s+/).filter(Boolean).join(" ");
+  if (n <= 1) return "";
+  return one.length <= n ? one : `${one.slice(0, n - 1)}…`;
+}
+
+/** The one-line message the asker receives: link, answer, how, question. */
+export function askPushMessage(p: { link: string; answer: string; how: string; question: string; freeText: boolean; collected?: boolean }): string {
+  const head = `[ask-answer] ${p.link} →`;
+  const ans = clip(p.answer, 300);
+  let tail = p.how ? ` (${p.how})` : "";
+  if (p.freeText) tail += " — a free-text reply, not a choice: the question stays open";
+  else if (p.collected) tail += " — collected ✅";
+  const q = clip(p.question, PUSH_MAX - head.length - ans.length - tail.length - 8);
+  return `${head} ${ans}${tail}. Q: ${q}`.slice(0, PUSH_MAX);
+}
+
+type Run = (cmd: string, args: string[]) => Pick<SpawnSyncReturns<string>, "status" | "stdout" | "stderr" | "error">;
+const runSync: Run = (cmd, args) => spawnSync(cmd, args, { encoding: "utf8", timeout: 60_000 });
+
+/** Parent pid, or 0. */
+export function parentPid(pid: number): number {
+  try {
+    const stat = readFileSync(`/proc/${pid}/stat`, "utf8");
+    return Number(stat.slice(stat.lastIndexOf(")") + 2).split(" ")[1]) || 0;
+  } catch {
+    const r = spawnSync("ps", ["-o", "ppid=", "-p", String(pid)], { encoding: "utf8", timeout: 2000 });
+    return Number((r.stdout ?? "").trim()) || 0;
+  }
+}
+
+/** Send `msg` to the agent at `pid` with `ay send --force`. The sent-log keeps
+ *  the agent's own pid, while ay may know the wrapper above it, so up to three
+ *  parents are tried too. Returns the pid that took it, or undefined. QUEUED
+ *  and NOT SUBMITTED count as delivered: both reach the agent, and treating
+ *  them as failures re-sent the same answer on every pass. */
+export function notifyAgent(pid: number, msg: string, run: Run = runSync, parent = parentPid): number | undefined {
+  for (let i = 0; i < 4 && pid > 1; i++, pid = parent(pid)) {
+    const r = run(process.env.SLACK_TERM_AY || "ay", ["send", String(pid), "--force", msg]);
+    if (r.error) return undefined; // no ay on this machine
+    const out = `${r.stdout ?? ""}${r.stderr ?? ""}`;
+    if (/no agent matched/i.test(out)) continue;
+    if (r.status === 0 || /QUEUED|sent to pid|NOT SUBMITTED/.test(out)) return pid;
+  }
+  return undefined;
+}
+
+/** One push pass at a time per machine: a watcher and a one-off run must not
+ *  both push the same answer. Returns a release function, or undefined when a
+ *  live pass holds the lock (a dead holder's lock is taken over). */
+export function acquirePushLock(path: string): (() => void) | undefined {
+  mkdirSync(dirname(path), { recursive: true });
+  for (let attempt = 0; attempt < 2; attempt++) {
+    try {
+      const fd = openSync(path, "wx");
+      writeFileSync(fd, String(process.pid));
+      closeSync(fd);
+      return () => { try { unlinkSync(path); } catch { /* gone */ } };
+    } catch {
+      let holder = 0;
+      try { holder = Number(readFileSync(path, "utf8").trim()); } catch { /* vanished */ }
+      if (holder && holder !== process.pid && pidAlive(holder)) return undefined;
+      try { unlinkSync(path); } catch { /* raced */ }
+    }
+  }
+  return undefined;
+}

package/ts/botIntent.ts

@@ -30,11 +30,12 @@ export function isWriteCommand(path: (string | number)[]): boolean {
   return WRITES.has(a) || WRITES.has(`${a} ${b}`);
 }
 
-/** Read-only modes of a write command. `ask --ls` only reads the local
- *  sent-log and the questions' live state. (`ask --waitFor` and
- *  `poll --results` are NOT here: both can edit the posted message.) */
+/** Read-only modes of a write command. `ask --ls` and `ask --pending` only
+ *  read the local sent-log and the questions' live state. (`ask --waitFor`,
+ *  `ask --pending --push` and `poll --results` are NOT here: each can edit the
+ *  posted message.) */
 export function isReadMode(path: (string | number)[], argv: Record<string, unknown>): boolean {
-  return String(path[0] ?? "") === "ask" && argv.ls === true;
+  return String(path[0] ?? "") === "ask" && (argv.ls === true || (argv.pending === true && argv.push !== true));
 }
 
 export type BotIntent = {

package/ts/cli.ts

@@ -3,18 +3,20 @@
 
 import { createHash } from "node:crypto";
 import { existsSync, readFileSync, mkdirSync, realpathSync, writeFileSync } from "node:fs";
+import { spawnSync } from "node:child_process";
 import { homedir } from "node:os";
 import { dirname, join, resolve } from "node:path";
 
 import yargs, { type Options } from "yargs";
 import { hideBin } from "yargs/helpers";
-import { guardUrlBoundaries } from "./urlGuard.ts";
+import { fixUrlBoundaries, guardUrlBoundaries } from "./urlGuard.ts";
 import { listProfiles, removeProfile, resolveBotToken, resolveCookie, resolveToken, useProfile, type Profile } from "./profiles.ts";
 import { diagnoseBotMessaging, formatDiagnosis } from "./botdoctor.ts";
 import { cmdAuthLogin, cmdAuthLoginChrome, cmdAuthChrome, cmdAuthFirefox, cmdAuthToken, cmdAuthApp, cmdAuthSave, cmdAuthTokens } from "./auth.ts";
 import { cmdTail, parseSince as parseDuration } from "./tail.ts";
-import { runStream, webClient, type StreamMatch } from "./stream.ts";
+import { phiDelay, runStream, webClient, type StreamMatch } from "./stream.ts";
 import { relaySubscriber, relayTransport } from "./relay.ts";
+import { acquirePushLock, askPushMessage, notifyAgent } from "./askPush.ts";
 import { DEFAULT_DIR, initRelay, initReport, relayHealth } from "./relayInit.ts";
 import { agentCommands } from "./agent.ts";
 
@@ -70,7 +72,7 @@ import {
 } from "./poll.ts";
 import { seedReactionsInOrder } from "./reactionSeed.ts";
 import { buildListSchema, cellValue, completedColumn, findColumn, parseColumnSpec, parseListSchema, parseSets, type ListColumn } from "./lists.ts";
-import { attributionEnabled, attributionMetadata, captureAttribution, markAskCollected, markAskDelivered, parseSince, querySent, recordSent, SENT_LOG_UNAVAILABLE, sentDbPath, senderAlive, uncollectedAsks, type RecordSent, type SentKind, type SentRow } from "./sentlog.ts";
+import { attributionEnabled, attributionMetadata, captureAttribution, markAskCollected, markAskDelivered, markAskPushed, parseSince, querySent, recordSent, SENT_LOG_UNAVAILABLE, sentDbPath, senderAlive, uncollectedAsks, type RecordSent, type SentKind, type SentRow } from "./sentlog.ts";
 import {
   authTest,
   authScopes,
@@ -4247,6 +4249,10 @@ interface AskPendingItem {
   alive: boolean;
   /** The ready-to-run `--waitFor` line, cursor included. */
   collect: string;
+  /** Where a delivery of this free-text reply ends (`--after` for the next look). */
+  cursor?: string;
+  /** Set by `--push`: the pid that took the answer, and whether it was collected. */
+  pushed?: { pid: number; collected: boolean };
 }
 
 /** `slack ask --pending [--since 7d] [--json]` — questions THIS machine asked
@@ -4265,16 +4271,24 @@ interface AskPendingItem {
  *  a person) runs the collect line. */
 async function cmdAskPending(
   tokens: { user?: string; bot?: string; cookie?: string },
-  opts: { sinceMs: number; json: boolean; channel?: string },
+  opts: {
+    sinceMs: number; json: boolean; channel?: string; push?: boolean; selfArgs?: string[];
+    /** `--watch`: only these questions (`channel:ts`), and print only what was pushed. */
+    only?: Set<string>; watch?: boolean;
+  },
 ): Promise<void> {
+  // Never blocks: one look per question and no waiting out 429s. A throttled
+  // pass says how many it skipped; the next pass picks them up.
+  setRateLimitWait(0);
   const all = uncollectedAsks(opts.sinceMs);
   if (!all) {
     console.error(`Error: ${SENT_LOG_UNAVAILABLE}\n  (${sentDbPath()})`);
     process.exit(ASK_EXIT_ERROR);
   }
-  const rows = opts.channel ? all.filter((r) => r.channel === opts.channel || r.target === opts.channel) : all;
+  const rows = all.filter((r) =>
+    (!opts.channel || r.channel === opts.channel || r.target === opts.channel) && (!opts.only || opts.only.has(`${r.channel}:${r.ts}`)));
   const pending: AskPendingItem[] = [];
-  const count = { unanswered: 0, ambiguous: 0, resolved: 0, void: 0, explain: 0 };
+  const count = { unanswered: 0, ambiguous: 0, resolved: 0, void: 0, explain: 0, throttled: 0 };
   /** Open questions someone pressed ℹ️ on — reported only: the thread note is
    *  the waiting session's to post (`--pending` writes nothing). */
   const explainAsked: string[] = [];
@@ -4287,7 +4301,7 @@ async function cmdAskPending(
     : tokens.bot ? (await selfIdentity(tokens.bot))?.team ?? "" : "";
   const otherTeams = new Map<string, number>();
 
-  for (const row of rows) {
+  for (const [i, row] of rows.entries()) {
     if (row.team && currentTeam && row.team !== currentTeam) {
       otherTeams.set(row.team, (otherTeams.get(row.team) ?? 0) + 1);
       continue;
@@ -4330,19 +4344,30 @@ async function cmdAskPending(
       if (found) {
         pending.push({ ...base, answer: found.answer, how: found.how, freeText: false });
       } else if (unchosen && !unchosen.ambiguous && unchosen.text !== row.delivered_answer) {
-        pending.push({ ...base, answer: unchosen.text, how: "reply (not a choice)", freeText: true });
+        pending.push({ ...base, answer: unchosen.text, how: "reply (not a choice)", freeText: true, cursor: askNextCursor(row.ts, after, unchosen.ts) });
       } else if (unchosen?.ambiguous) {
         count.ambiguous++;
       } else {
         count.unanswered++;
       }
     } catch (e: unknown) {
+      if (e instanceof RateLimitError) {
+        // Probing on would only collect more 429s for the same method.
+        count.throttled = rows.length - i;
+        problems.push(`rate-limited by Slack (${e.message}): ${count.throttled} question(s) not checked this pass`);
+        break;
+      }
       problems.push(`${link}: ${e instanceof Error ? e.message : String(e)}`);
     }
   }
 
+  if (opts.push) askPushAll(pending, opts.selfArgs ?? [], problems);
+
+  // A watcher's log carries what it did, not the whole list on every pass.
+  const shown = opts.watch ? pending.filter((it) => it.pushed) : pending;
+  if (opts.watch && !shown.length && !problems.length) return;
   if (opts.json) {
-    for (const it of pending) {
+    for (const it of shown) {
       const r = it.row;
       console.log(JSON.stringify({
         id: r.id, link: it.link, channel: r.channel, ts: r.ts, asked_at: r.sent_at,
@@ -4350,12 +4375,13 @@ async function cmdAskPending(
         sender: { session_id: r.session_id ?? null, cli: r.cli ?? null, agent_pid: r.agent_pid ?? null, pid: r.pid, alive: it.alive, cwd: r.cwd, git_branch: r.git_branch ?? null, host: r.host ?? null },
         delivered_at: r.delivered_at ?? null,
         collect: it.collect,
+        ...(opts.push ? { pushed: it.pushed ?? null } : {}),
       }));
     }
   } else {
     // Grouped by the session that asked — the unit that has to be told.
     const groups = new Map<string, AskPendingItem[]>();
-    for (const it of pending) {
+    for (const it of shown) {
       const key = it.row.session_id ? `session ${it.row.session_id}` : `pid ${it.row.agent_pid ?? it.row.pid}`;
       groups.set(key, [...(groups.get(key) ?? []), it]);
     }
@@ -4370,6 +4396,7 @@ async function cmdAskPending(
         console.log(`    Q: ${stripTerminalControls(askFlatten(it.question))}`);
         console.log(`    A: ${stripTerminalControls(askFlatten(it.answer))}  — ${stripTerminalControls(it.how)}${it.freeText ? " — not a decision" : ""}`);
         console.log(`    collect: ${stripTerminalControls(it.collect)}`);
+        if (it.pushed) console.log(`    pushed to pid ${it.pushed.pid}${it.pushed.collected ? ", collected ✅" : ""}`);
       }
     }
   }
@@ -4384,10 +4411,113 @@ async function cmdAskPending(
     (count.resolved ? `, ${count.resolved} ✅ already collected` : "") +
     (count.void ? `, ${count.void} voided` : "") +
     (count.explain ? `, ${count.explain} with an ℹ️ explain request` : "") +
+    (count.throttled ? `, ${count.throttled} not checked (rate-limited)` : "") +
+    (opts.push ? `, ${pending.filter((it) => it.pushed).length} pushed to the asker` : "") +
     ` — checked ${rows.length} uncollected question(s) (log: ${sentDbPath()})`,
   );
 }
 
+/** `ask --pending --push --watch`: a push pass every `intervalMs`, and — when
+ *  SLACK_RELAY_URL / SLACK_RELAY_TOKEN are set — one for a single question as
+ *  soon as the relay rings for it (a reaction on it, or a reply in its
+ *  thread), so a pressed pill reaches the asker in seconds. The relay only
+ *  sees the relay app's channels; the interval covers the rest. */
+async function askPushWatch(run: (only?: Set<string>) => Promise<void>, sinceMs: () => number, intervalMs: number): Promise<never> {
+  let busy = false;
+  const queued = new Set<string>();
+  let full = true;
+  const kick = async (): Promise<void> => {
+    if (busy) return;
+    busy = true;
+    try {
+      while (full || queued.size) {
+        const only = full ? undefined : new Set(queued);
+        full = false;
+        queued.clear();
+        try {
+          await run(only);
+        } catch (e: unknown) {
+          console.error(`slack ask --watch: pass failed: ${friendlySlackError(e)}`);
+        }
+      }
+    } finally {
+      busy = false;
+    }
+  };
+  const relayUrl = process.env.SLACK_RELAY_URL;
+  const relayToken = process.env.SLACK_RELAY_TOKEN;
+  if (relayUrl && relayToken) {
+    const subscribe = relaySubscriber(relayUrl, relayToken, undefined, { transport: relayTransport(process.env.SLACK_RELAY_TRANSPORT) });
+    void (async () => {
+      for (let attempt = 0; ; attempt++) {
+        try {
+          await subscribe(undefined, new AbortController().signal, {
+            hello: () => { attempt = 0; },
+            bell: (_seq, b) => {
+              const key = `${b.channel}:${b.thread_ts ?? b.ts}`;
+              const open = uncollectedAsks(sinceMs()) ?? [];
+              if (!open.some((r) => `${r.channel}:${r.ts}` === key)) return;
+              queued.add(key);
+              // A few seconds' grace: a pill press often comes with a reply.
+              setTimeout(() => { void kick(); }, 3000);
+            },
+          });
+        } catch (e: unknown) {
+          console.error(`slack ask --watch: relay: ${e instanceof Error ? e.message : String(e)} — reconnecting`);
+        }
+        await new Promise((r) => setTimeout(r, phiDelay(attempt, 2000, 300_000)));
+      }
+    })();
+  } else {
+    console.error("slack ask --watch: no SLACK_RELAY_URL / SLACK_RELAY_TOKEN — checking every interval only");
+  }
+  for (;;) {
+    full = true;
+    await kick();
+    await new Promise((r) => setTimeout(r, intervalMs));
+  }
+}
+
+/** `--push`: tell each alive asker its answer over ay, then collect a decision
+ *  (✅ in Slack, collected in the log) with the asker's own `--waitFor` line —
+ *  the same code path, so the stamp and the bookkeeping cannot drift. A
+ *  free-text reply is marked delivered instead: it is not a decision, so the
+ *  question stays open, but that reply is not pushed again. An asker that is
+ *  gone gets nothing: the answer stays on the list for a person. */
+function askPushAll(items: AskPendingItem[], selfArgs: string[], problems: string[]): void {
+  const release = acquirePushLock(join(process.env.XDG_STATE_HOME || join(homedir(), ".local", "state"), "slack-term", "ask-push.lock"));
+  if (!release) {
+    problems.push("another `ask --pending --push` pass is running: nothing pushed by this one");
+    return;
+  }
+  try {
+    for (const it of items) {
+      const r = it.row;
+      if (!it.alive || r.pushed_answer === it.answer) continue;
+      const msg = askPushMessage({ link: it.link, answer: it.answer, how: it.how, question: it.question, freeText: it.freeText, collected: !it.freeText });
+      const pid = notifyAgent(r.agent_pid ?? r.pid, msg);
+      if (pid === undefined) {
+        problems.push(`${it.link}: the asker (pid ${r.agent_pid ?? r.pid}) did not take the push over ay — tried again next pass`);
+        continue;
+      }
+      markAskPushed(r.channel, r.ts, it.answer);
+      let collected = false;
+      if (it.freeText) {
+        markAskDelivered(r.channel, r.ts, it.answer, it.cursor ?? "");
+      } else {
+        const after = r.delivered_cursor ? [`--after=${r.delivered_cursor}`] : [];
+        const c = spawnSync(process.execPath, [...process.argv.slice(1, 2), ...selfArgs, "ask", `--waitFor=${it.link}`, "--timeout", "0", ...(r.as_bot ? ["--as-bot"] : []), ...after],
+          { encoding: "utf8", timeout: 120_000, stdio: ["ignore", "pipe", "pipe"] });
+        collected = c.status === 0;
+        if (!collected) problems.push(`${it.link}: pushed, but collecting it exited ${c.status ?? "on a signal"} — collect by hand: ${it.collect}`);
+      }
+      it.pushed = { pid, collected };
+    }
+  } finally {
+    release();
+  }
+}
+
 /** A question about to be voided: where it is, and what it parsed as. */
 interface AskVoidTarget {
   link: string;
@@ -5695,7 +5825,7 @@ async function main(): Promise<void> {
    *  prefix and thread check first). */
   type SendCli = {
     workspace?: string | undefined; target?: string | undefined; message?: string | undefined; code?: string | undefined;
-    "allow-url-adjacent"?: boolean | undefined; "channel-id"?: string | undefined; "user-id"?: string | undefined;
+    "allow-url-adjacent"?: boolean | undefined; "fix-urls"?: boolean | undefined; "channel-id"?: string | undefined; "user-id"?: string | undefined;
     "as-bot"?: boolean | undefined; broadcast?: boolean | undefined; mentions?: boolean | undefined;
     json?: boolean | undefined;
   };
@@ -5704,7 +5834,7 @@ async function main(): Promise<void> {
    *  its subscriptions. */
   type EditCli = {
     workspace?: string | undefined; target?: string | undefined; newText?: string | undefined; code?: string | undefined;
-    force?: boolean | undefined; "allow-url-adjacent"?: boolean | undefined; "channel-id"?: string | undefined;
+    force?: boolean | undefined; "allow-url-adjacent"?: boolean | undefined; "fix-urls"?: boolean | undefined; "channel-id"?: string | undefined;
     mentions?: boolean | undefined; "as-bot"?: boolean | undefined; tick?: { n?: number } | undefined;
   };
   const runEdit = async (argv: EditCli): Promise<void> => {
@@ -5722,6 +5852,7 @@ async function main(): Promise<void> {
         process.exit(1);
       }
     }
+    if (argv["fix-urls"] && argv.newText) argv = { ...argv, newText: fixUrlBoundaries(argv.newText) };
     const args: EditArgs = { target: argv.target!, newText: argv.newText ?? "", allowUrlAdjacent: argv["allow-url-adjacent"] === true };
     if (argv.tick) args.tick = argv.tick;
     if (settings.prefix) args.lanePrefix = settings.prefix;
@@ -5768,6 +5899,7 @@ async function main(): Promise<void> {
       console.error("Error: message is empty");
       process.exit(1);
     }
+    if (argv["fix-urls"]) argv = { ...argv, message: fixUrlBoundaries(argv.message!) };
     const args: SendArgs = { target: argv.target!, message: argv.message!, allowUrlAdjacent: argv["allow-url-adjacent"] === true };
     if (argv.json) args.json = true;
     if (argv.code) args.code = argv.code;
@@ -6416,6 +6548,7 @@ async function main(): Promise<void> {
         .positional("message", { type: "string", demandOption: true })
         .option("code", { type: "string", describe: "Safety hash to confirm send" })
         .option("allow-url-adjacent", { type: "boolean", default: false, describe: "Warn instead of refusing ambiguous bare URL boundaries" })
+        .option("fix-urls", { type: "boolean", default: false, describe: "Wrap ambiguous bare URLs as <url> (cut at the first non-ASCII character / trailing punctuation) before sending" })
         .option("channel-id", { type: "string", describe: "Raw channel ID" })
         .option("user-id", { type: "string", describe: "Raw user ID (opens DM)" })
         .option("as-bot", { type: "boolean", default: false, describe: "Send via the bot token (xoxb / SLACK_BOT_TOKEN) so a DM notifies the recipient and can be two-way" })
@@ -6594,6 +6727,7 @@ async function main(): Promise<void> {
         .option("code", { type: "string", describe: "Safety hash to confirm send" })
         .option("as-bot", { type: "boolean", describe: "Send via the bot token (default: settings' as_bot)" })
         .option("allow-url-adjacent", { type: "boolean", default: false, describe: "Warn instead of refusing ambiguous bare URL boundaries" })
+        .option("fix-urls", { type: "boolean", default: false, describe: "Wrap ambiguous bare URLs as <url> (cut at the first non-ASCII character / trailing punctuation) before sending" })
         .option("broadcast", { type: "boolean", default: false, describe: "Also send to channel" })
         .option("mentions", { type: "boolean", default: true, describe: "Convert @handle tokens to real mentions (--no-mentions for literal text)" }),
       async (argv) => {
@@ -6628,7 +6762,7 @@ async function main(): Promise<void> {
         if (!text.trim()) { console.error("Error: message is empty (only the prefix)"); process.exit(1); }
         await runSend({
           workspace: (argv as W).workspace, target: `${channel}:${threadTs}`, message: text, code: argv.code,
-          "allow-url-adjacent": argv["allow-url-adjacent"], "as-bot": argv["as-bot"] ?? settings.as_bot ?? false,
+          "allow-url-adjacent": argv["allow-url-adjacent"], "fix-urls": argv["fix-urls"], "as-bot": argv["as-bot"] ?? settings.as_bot ?? false,
           broadcast: argv.broadcast, mentions: argv.mentions, json: argv.json,
         });
       },
@@ -6842,7 +6976,10 @@ async function main(): Promise<void> {
         .option("reask", { type: "boolean", default: false, describe: "With ONE --void: post the question again (same channel/thread, same text and choices unless you give new ones as positionals / --body) and void the old one as replaced by it — one gate for both. The way to change the options of a question someone already answered." })
         .option("edit", { type: "string", describe: "Change an open question you posted: --edit=<permalink> [\"new question\" [choices…]] [--body …]. Rebuilt with the ask layout so collect still reads it. Options can change only while nobody has answered (then: --void=<link> --reask). Same --code gate." })
         .option("ls", { type: "boolean", default: false, describe: "List the questions you posted (local sent-log) with their LIVE state: --state open|answered|void|all (default open), --stale 24h, --channel, --json. With --stale it prints the --void command that would retire them." })
-        .option("pending", { type: "boolean", default: false, describe: "List questions THIS machine asked (local sent-log) that somebody answered but nobody collected, grouped by the asking session, with whether that agent is still alive and the --waitFor line that collects each. Read-only: --since 7d, --channel, --json." })
+        .option("pending", { type: "boolean", default: false, describe: "List questions THIS machine asked (local sent-log) that somebody answered but nobody collected, grouped by the asking session, with whether that agent is still alive and the --waitFor line that collects each. Read-only unless --push: --since 7d, --channel, --json. Never waits out a rate limit: a throttled pass says how many it skipped." })
+        .option("push", { type: "boolean", default: false, describe: "With --pending: send each answer to the agent that asked (ay send to the pid in the sent-log, or up to 3 of its parents), then collect a decision (✅, off this list); a free-text reply is marked delivered, the question stays open. Askers that are gone get nothing. Each answer is pushed once." })
+        .option("watch", { type: "boolean", default: false, describe: "With --pending --push: keep running — a pass every --interval, and one for a question as soon as the relay rings for it (SLACK_RELAY_URL / SLACK_RELAY_TOKEN). Prints only what it pushed." })
+        .option("interval", { type: "number", default: 10, describe: "With --watch: minutes between full passes" })
         .option("since", { type: "string", default: "7d", describe: "With --pending: how far back to look (30m, 2h, 7d, or an ISO date)" })
         .option("stale", { type: "string", describe: "With --ls: only open questions older than this (24h, 3d)" })
         .option("state", { type: "string", default: "open", choices: ["open", "answered", "void", "edited", "gone", "unknown", "all"], describe: "With --ls: which state to list" })
@@ -6886,8 +7023,27 @@ async function main(): Promise<void> {
               console.error(`Error: --since needs 30m / 2h / 7d / 1w or an ISO date (got ${stripTerminalControls(String(argv.since))})`);
               process.exit(ASK_EXIT_ERROR);
             }
-            const p: { sinceMs: number; json: boolean; channel?: string } = { sinceMs, json: !!argv.json };
+            if (argv.watch && !argv.push) {
+              console.error("Error: --watch keeps pushing answers — use it with --push.");
+              process.exit(ASK_EXIT_ERROR);
+            }
+            const p: Parameters<typeof cmdAskPending>[1] = { sinceMs, json: !!argv.json, push: !!argv.push };
             if (argv.channel) p.channel = String(argv.channel);
+            // The collect run must read with the same workspace this one does.
+            if (argv.workspace) p.selfArgs = ["-w", String(argv.workspace)];
+            if (argv.watch) {
+              const min = Number(argv.interval);
+              if (!(min > 0)) {
+                console.error("Error: --interval needs a number of minutes above 0.");
+                process.exit(ASK_EXIT_ERROR);
+              }
+              const since = String(argv.since);
+              await askPushWatch(
+                (only) => cmdAskPending(tokens, { ...p, sinceMs: parseSince(since) ?? sinceMs, watch: true, ...(only ? { only } : {}) }),
+                () => parseSince(since) ?? sinceMs,
+                min * 60_000,
+              );
+            }
             try {
               await cmdAskPending(tokens, p);
             } catch (e: unknown) {
@@ -7480,6 +7636,7 @@ async function main(): Promise<void> {
         .option("code", { type: "string", describe: "Safety hash to confirm edit" })
         .option("force", { type: "boolean", default: false, describe: "Apply an edit that BREAKS a `slack ask` / `slack poll` message (new text no longer reads as one, or an answered/void question made open again). Refused by default because collect reads that body back; edits that keep it readable (e.g. writing the ✅ form) need no flag. Prefer `slack ask --edit=` / `--void=`." })
         .option("allow-url-adjacent", { type: "boolean", default: false, describe: "Warn instead of refusing ambiguous bare URL boundaries" })
+        .option("fix-urls", { type: "boolean", default: false, describe: "Wrap ambiguous bare URLs as <url> (cut at the first non-ASCII character / trailing punctuation) before sending" })
         .option("channel-id", { type: "string", describe: "Raw channel ID" })
         .option("mentions", { type: "boolean", default: true, describe: "Convert @handle tokens in the new text to real <@USERID> mentions (on by default). Unresolved handles stay as plain text. Disable with --no-mentions for literal @text." })
         .option("as-bot", { type: "boolean", default: false, describe: "Edit as the bot (xoxb / SLACK_BOT_TOKEN). REQUIRED to correct a message the bot posted: Slack only lets a token edit its OWN messages, so a bot-authored message is uneditable by the user token and chat.update returns cant_update_message. Address it by permalink or --channel-id; an @user target resolves against the BOT's DM list, which is where a bot-authored DM actually lives." }),

package/ts/sentlog.ts

@@ -70,6 +70,10 @@ export interface SentRow {
   delivered_at?: number | null;
   delivered_answer?: string | null;
   delivered_cursor?: string | null;
+  /** `ask --pending --push` handed this answer to the asking agent (over ay):
+   *  the same answer is not pushed twice. */
+  pushed_at?: number | null;
+  pushed_answer?: string | null;
 }
 
 export function attributionEnabled(): boolean {
@@ -239,7 +243,9 @@ CREATE TABLE IF NOT EXISTS sent (
   answer_exit INTEGER,
   delivered_at INTEGER,
   delivered_answer TEXT,
-  delivered_cursor TEXT
+  delivered_cursor TEXT,
+  pushed_at INTEGER,
+  pushed_answer TEXT
 );
 CREATE INDEX IF NOT EXISTS sent_msg ON sent(channel, ts);
 CREATE INDEX IF NOT EXISTS sent_at ON sent(sent_at);
@@ -262,7 +268,7 @@ export function openSentLog(): Db | undefined {
     db.exec(SCHEMA);
     // Columns added after the table first shipped. ALTER fails harmlessly when
     // the column is already there.
-    for (const col of ["delivered_answer TEXT", "delivered_cursor TEXT"]) {
+    for (const col of ["delivered_answer TEXT", "delivered_cursor TEXT", "pushed_at INTEGER", "pushed_answer TEXT"]) {
       try { db.exec(`ALTER TABLE sent ADD COLUMN ${col}`); } catch { /* exists */ }
     }
     return db;
@@ -408,6 +414,14 @@ export function markAskDelivered(channel: string, ts: string, answer: string, cu
   );
 }
 
+/** `ask --pending --push` told the asking agent this answer. */
+export function markAskPushed(channel: string, ts: string, answer: string): void {
+  updateLog(
+    `UPDATE sent SET pushed_at = ?, pushed_answer = ? WHERE kind = 'ask' AND channel = ? AND ts = ?`,
+    Date.now(), answer, channel, ts,
+  );
+}
+
 /** Logged asks nobody has collected yet, oldest first. `undefined` when the
  *  log cannot be opened at all (no SQLite in this runtime). */
 export function uncollectedAsks(sinceMs: number): SentRow[] | undefined {

package/ts/urlGuard.ts

@@ -16,7 +16,15 @@ const URL_TOKEN = /<https?:\/\/[^\s<>|]+(?:\|[^<>\r\n]*)?>|https?:\/\/[^\t\n\v\f
 /** What may sit right before a bare URL and still let it autolink. */
 const SAFE_BEFORE = /[\t\n\v\f\r (*_~]/;
 
-export type UrlBoundaryIssue = { url: string; side: "before" | "after"; char: string };
+/** `trailing`: the URL ends in ASCII punctuation (`)` without a matching `(`, or `.,:;!?`) that Slack may or may
+ *  not take into the link: `(see https://x/a)` would link `https://x/a)`. Only warned, never refused. */
+export type UrlBoundaryIssue = { url: string; side: "before" | "after" | "trailing"; char: string };
+const TRAILING = /[.,:;!?]$/;
+function trailingChar(url: string): string | undefined {
+  if (TRAILING.test(url)) return url.slice(-1);
+  if (url.endsWith(")") && count(url, "(") < count(url, ")")) return ")";
+  return undefined;
+}
 
 /** Code spans and fences are shown verbatim and never autolinked, so a URL in
  *  one is not a boundary problem; neither is a `[label](url)` link, which is
@@ -97,6 +105,11 @@ export function urlBoundaryIssues(text: string, { markdown = true }: { markdown?
     }
     const after = url.match(/[^\x00-\x7f]|>/);
     if (after) issues.push({ url, side: "after", char: after[0] });
+    else {
+      // `(https://x/a)`: the `)` closes the paren before the URL, as GFM and Slack read it
+      const t = trailingChar(url);
+      if (t && !(t === ")" && before === "(")) issues.push({ url, side: "trailing", char: t });
+    }
   }
   return issues;
 }
@@ -107,22 +120,25 @@ export function adjacentUrls(text: string): string[] {
 
 export function guardUrlBoundaries(
   text: string, allow = false, warn: (message: string) => void = console.error,
-  { sides = ["before", "after"], markdown = true }: {
+  { sides = ["before", "after", "trailing"], markdown = true }: {
     sides?: ReadonlyArray<UrlBoundaryIssue["side"]>; markdown?: boolean;
   } = {},
 ): void {
-  const issues = urlBoundaryIssues(text, { markdown }).filter((i) => sides.includes(i.side));
+  const all = urlBoundaryIssues(text, { markdown });
+  const trailing = all.filter((i) => i.side === "trailing" && sides.includes("trailing"));
+  if (trailing.length) warn(`⚠ URL ends in punctuation Slack may take into the link: ${trailing.map((i) => `${JSON.stringify(i.url)} → ${JSON.stringify(fixUrlBoundaries(`x ${i.url}`).slice(2))}`).join(", ")}. If the ${JSON.stringify(trailing[0]!.char)} is not part of the URL, send the suggested form (or --fix-urls).`);
+  const issues = all.filter((i) => i.side !== "trailing" && sides.includes(i.side));
   if (!issues.length) return;
   const described = issues.map((i) => i.side === "before"
     ? `${JSON.stringify(i.url)} (glued to ${JSON.stringify(i.char)} before it — Slack will NOT make it a link)`
-    : `${JSON.stringify(i.url)} (runs into ${JSON.stringify(i.char)} after it)`);
+    : `${JSON.stringify(i.url)} (runs into ${JSON.stringify(i.char)} after it; non-ASCII in a URL is OK only if intended, otherwise send ${JSON.stringify(fixUrlBoundaries(`x ${i.url}`).slice(2))} or put an ASCII space after the URL)`);
   const fixes = issues.some((i) => i.side === "before")
     ? "Put an ASCII space (or a newline) before the URL, " : "";
   const message = `Ambiguous URL boundary: ${described.join(", ")}. ` +
     `${fixes}Put the URL on its own line, or wrap it as <url> / <url|label>. ` +
-    "Use --allow-url-adjacent to warn only.";
+    "Use --fix-urls to wrap them, or --allow-url-adjacent to warn only.";
   if (!allow) throw new Error(message);
-  warn(`Warning: ${message}`);
+  warn(`⚠ Warning: ${message}`);
 }
 
 type RichTextElement =
@@ -167,6 +183,27 @@ export function linkifyRichText(text: string): RichTextElement[] {
   return out;
 }
 
+/** Optional auto-fix (`--fix-urls`): wrap every ambiguous bare URL as `<url>`, cut where Slack's link would end
+ *  (at the first non-ASCII character, before trailing punctuation). Text, code spans and explicit links are kept. */
+export function fixUrlBoundaries(text: string): string {
+  const masked = maskCode(text);
+  let out = "", pos = 0;
+  for (const m of masked.matchAll(URL_TOKEN)) {
+    const at = m.index ?? 0;
+    if (m[0].startsWith("<")) continue;
+    let url = text.slice(at, at + m[0].length);
+    const cut = url.search(/[^\x00-\x7f]|>/);
+    if (cut >= 0) url = url.slice(0, cut);
+    while (trailingChar(url)) url = url.slice(0, -1);
+    const before = at > 0 ? masked[at - 1]! : "";
+    const ambiguous = url.length < m[0].length || (before && !SAFE_BEFORE.test(before));
+    if (!ambiguous || !/^https?:\/\/./i.test(url)) continue;
+    out += text.slice(pos, at) + `<${url}>`;
+    pos = at + url.length;
+  }
+  return out + text.slice(pos);
+}
+
 function count(s: string, ch: string): number {
   return s.split(ch).length - 1;
 }