@sentry/junior 0.5.0 → 0.6.0

package/dist/{chunk-3ENACZ27.js → chunk-QNOV65P4.js}

@@ -1,7 +1,12 @@
+import {
+  parseSkillFile,
+  stripFrontmatter
+} from "./chunk-KT5HARSN.js";
 import {
   CredentialUnavailableError,
   SANDBOX_SKILLS_ROOT,
   SANDBOX_WORKSPACE_ROOT,
+  aboutPathCandidates,
   botConfig,
   claimQueueIngressDedup,
   createPluginBroker,
@@ -26,7 +31,7 @@ import {
   skillRoots,
   soulPathCandidates,
   upsertAgentTurnSessionCheckpoint
-} from "./chunk-BKYYVLVN.js";
+} from "./chunk-5UJSQX4R.js";
 import {
   logError,
   logException,
@@ -281,6 +286,11 @@ var ACK_REGEXES = [
 ];
 var QUESTION_PREFIX_RE = /^(what|why|how|when|where|which|who|can|could|would|should|do|does|did|is|are|was|were|will)\b/i;
 var FOLLOW_UP_REF_RE = /\b(you|your|that|this|it|above|previous|earlier|last|just\s+said)\b/i;
+var LEADING_SLACK_MENTION_RE = /^\s*<@([A-Z0-9]+)(?:\|([^>]+))?>[\s,:-]*/i;
+var LEADING_NAMED_MENTION_RE = /^\s*@([a-z0-9._-]+)\b[\s,:-]*/i;
+function escapeRegExp(value) {
+  return value.replace(/[.*+?^${}()|[\]\\]/g, "\\$&");
+}
 function tokenizeForOverlap(value) {
   return value.toLowerCase().split(/[^a-z0-9]+/).filter((token) => token.length >= 4);
 }
@@ -329,6 +339,54 @@ function isLikelyAssistantDirectedFollowUp(text, conversationContext) {
   }
   return false;
 }
+function containsAssistantInvocation(text, botUserName) {
+  const escapedUserName = escapeRegExp(botUserName);
+  const plainNameMentionRe = new RegExp(`(^|\\s)@${escapedUserName}\\b`, "i");
+  const labeledEntityMentionRe = new RegExp(
+    `<@[^>|]+\\|${escapedUserName}>`,
+    "i"
+  );
+  return plainNameMentionRe.test(text) || labeledEntityMentionRe.test(text);
+}
+function detectLeadingOtherPartyAddress(rawText, text, botUserName) {
+  if (containsAssistantInvocation(rawText, botUserName) || containsAssistantInvocation(text, botUserName)) {
+    return void 0;
+  }
+  const leadingSlackMention = rawText.match(LEADING_SLACK_MENTION_RE);
+  if (leadingSlackMention) {
+    const label = leadingSlackMention[2]?.trim();
+    return label ? `slack_mention:${label}` : "slack_mention";
+  }
+  const leadingNamedMention = text.match(LEADING_NAMED_MENTION_RE);
+  if (!leadingNamedMention) {
+    return void 0;
+  }
+  const directedName = leadingNamedMention[1]?.trim();
+  if (!directedName || directedName.toLowerCase() === botUserName.toLowerCase()) {
+    return void 0;
+  }
+  return `named_mention:${directedName}`;
+}
+function getSubscribedReplyPreflightDecision(args) {
+  const text = args.text.trim();
+  const rawText = args.rawText.trim();
+  if (args.isExplicitMention) {
+    return { shouldReply: true, reason: "explicit_mention" /* ExplicitMention */ };
+  }
+  const leadingOtherPartyAddress = detectLeadingOtherPartyAddress(
+    rawText,
+    text,
+    args.botUserName
+  );
+  if (!leadingOtherPartyAddress) {
+    return void 0;
+  }
+  return {
+    shouldReply: false,
+    reason: "directed_to_other_party" /* DirectedToOtherParty */,
+    reasonDetail: leadingOtherPartyAddress
+  };
+}
 function buildRouterSystemPrompt(botUserName, conversationContext) {
   return [
     "You are a message router for a Slack assistant named Junior in a subscribed Slack thread.",
@@ -352,8 +410,14 @@ function buildRouterSystemPrompt(botUserName, conversationContext) {
 async function decideSubscribedThreadReply(args) {
   const text = args.input.text.trim();
   const rawText = args.input.rawText.trim();
-  if (args.input.isExplicitMention) {
-    return { shouldReply: true, reason: "explicit_mention" /* ExplicitMention */ };
+  const preflightDecision = getSubscribedReplyPreflightDecision({
+    botUserName: args.botUserName,
+    rawText,
+    text,
+    isExplicitMention: args.input.isExplicitMention
+  });
+  if (preflightDecision) {
+    return preflightDecision;
   }
   if (!text && !args.input.hasAttachments) {
     return { shouldReply: false, reason: "empty_message" /* EmptyMessage */ };
@@ -365,7 +429,10 @@ async function decideSubscribedThreadReply(args) {
     return { shouldReply: false, reason: "acknowledgment" /* Acknowledgment */ };
   }
   if (isLikelyAssistantDirectedFollowUp(text, args.input.conversationContext)) {
-    return { shouldReply: true, reason: "follow_up_question" /* FollowUpQuestion */ };
+    return {
+      shouldReply: true,
+      reason: "follow_up_question" /* FollowUpQuestion */
+    };
   }
   try {
     const result = await args.completeObject({
@@ -373,7 +440,10 @@ async function decideSubscribedThreadReply(args) {
       schema: replyDecisionSchema,
       maxTokens: 120,
       temperature: 0,
-      system: buildRouterSystemPrompt(args.botUserName, args.input.conversationContext),
+      system: buildRouterSystemPrompt(
+        args.botUserName,
+        args.input.conversationContext
+      ),
       prompt: rawText,
       metadata: {
         modelId: args.modelId,
@@ -694,20 +764,20 @@ var slackOutputPolicy = {
 
 // src/chat/prompt.ts
 var DEFAULT_SOUL = "You are Junior, a practical and concise assistant.";
-function loadSoul() {
+function loadOptionalMarkdownFile(candidates, fileName) {
   const attempted = [];
-  for (const resolved of soulPathCandidates()) {
+  for (const resolved of candidates) {
     attempted.push(resolved);
     try {
       const raw = fs.readFileSync(resolved, "utf8").trim();
       if (raw.length > 0) {
         logInfo(
-          "soul_loaded",
+          `${fileName.toLowerCase()}_loaded`,
           {},
           {
             "file.path": resolved
           },
-          "Loaded SOUL.md"
+          `Loaded ${fileName}`
         );
         return raw;
       }
@@ -715,16 +785,26 @@ function loadSoul() {
       continue;
     }
   }
+  return null;
+}
+function loadSoul() {
+  const soul = loadOptionalMarkdownFile(soulPathCandidates(), "SOUL.md");
+  if (soul) {
+    return soul;
+  }
   logWarn(
     "soul_load_fallback",
     {},
     {
-      "file.candidates": attempted
+      "file.candidates": soulPathCandidates()
     },
     "SOUL.md not found; using built-in default personality"
   );
   return DEFAULT_SOUL;
 }
+function loadAbout() {
+  return loadOptionalMarkdownFile(aboutPathCandidates(), "ABOUT.md");
+}
 var JUNIOR_PERSONALITY = (() => {
   try {
     return loadSoul();
@@ -740,6 +820,21 @@ var JUNIOR_PERSONALITY = (() => {
     return DEFAULT_SOUL;
   }
 })();
+var JUNIOR_ABOUT = (() => {
+  try {
+    return loadAbout();
+  } catch (error) {
+    logWarn(
+      "about_load_failed",
+      {},
+      {
+        "error.message": error instanceof Error ? error.message : String(error)
+      },
+      "Failed to load ABOUT.md; omitting about prompt context"
+    );
+    return null;
+  }
+})();
 function workspaceSkillDir(skillName) {
   return sandboxSkillDir(skillName);
 }
@@ -912,6 +1007,16 @@ function buildSystemPrompt(params) {
         JUNIOR_PERSONALITY.trim()
       ].join("\n")
     ),
+    ...JUNIOR_ABOUT ? [
+      renderTag(
+        "about",
+        [
+          "Use this as the assistant's product/domain description when relevant.",
+          "",
+          JUNIOR_ABOUT.trim()
+        ].join("\n")
+      )
+    ] : [],
     renderTag(
       "identity-context",
       [
@@ -2358,127 +2463,6 @@ import path2 from "path";
 // src/chat/skills.ts
 import fs2 from "fs/promises";
 import path from "path";
-
-// src/chat/skill-frontmatter.ts
-import { parse as parseYaml } from "yaml";
-var FRONTMATTER_RE = /^---\r?\n([\s\S]*?)\r?\n---\r?\n?/;
-var SKILL_NAME_RE = /^[a-z0-9-]+$/;
-var CAPABILITY_TOKEN_RE = /^[a-z0-9]+(?:\.[a-z0-9-]+)+$/;
-var MAX_NAME_LENGTH = 64;
-var MAX_DESCRIPTION_LENGTH = 1024;
-var MAX_COMPATIBILITY_LENGTH = 500;
-function hasAngleBrackets(value) {
-  return value.includes("<") || value.includes(">");
-}
-function validateSkillName(name) {
-  if (!name) return "name must not be empty";
-  if (name.length > MAX_NAME_LENGTH) return `name must be <= ${MAX_NAME_LENGTH} characters`;
-  if (!SKILL_NAME_RE.test(name)) return "name must contain only lowercase letters, digits, and hyphens";
-  if (name.startsWith("-") || name.endsWith("-")) return "name must not start or end with a hyphen";
-  if (name.includes("--")) return "name must not contain consecutive hyphens";
-  return null;
-}
-function stripFrontmatter(raw) {
-  return raw.replace(FRONTMATTER_RE, "").trim();
-}
-function parseAndValidateSkillFrontmatter(raw, expectedName) {
-  const match = FRONTMATTER_RE.exec(raw);
-  if (!match) {
-    return { ok: false, error: "Missing YAML frontmatter at start of file" };
-  }
-  let parsed;
-  try {
-    parsed = parseYaml(match[1]);
-  } catch (error) {
-    return {
-      ok: false,
-      error: `Invalid YAML frontmatter: ${error instanceof Error ? error.message : String(error)}`
-    };
-  }
-  if (!parsed || typeof parsed !== "object" || Array.isArray(parsed)) {
-    return { ok: false, error: "Frontmatter must be a YAML object" };
-  }
-  const frontmatter = parsed;
-  const name = frontmatter.name;
-  const description = frontmatter.description;
-  if (typeof name !== "string") {
-    return { ok: false, error: 'Frontmatter field "name" must be a string' };
-  }
-  const nameError = validateSkillName(name);
-  if (nameError) {
-    return { ok: false, error: nameError };
-  }
-  if (expectedName && name !== expectedName) {
-    return { ok: false, error: `name "${name}" must match directory "${expectedName}"` };
-  }
-  if (typeof description !== "string") {
-    return { ok: false, error: 'Frontmatter field "description" must be a string' };
-  }
-  if (!description.trim()) {
-    return { ok: false, error: "description must not be empty" };
-  }
-  if (description.length > MAX_DESCRIPTION_LENGTH) {
-    return { ok: false, error: `description must be <= ${MAX_DESCRIPTION_LENGTH} characters` };
-  }
-  if (hasAngleBrackets(description)) {
-    return { ok: false, error: 'description must not contain "<" or ">"' };
-  }
-  if ("metadata" in frontmatter && (typeof frontmatter.metadata !== "object" || !frontmatter.metadata || Array.isArray(frontmatter.metadata))) {
-    return { ok: false, error: 'Frontmatter field "metadata" must be an object when present' };
-  }
-  if ("compatibility" in frontmatter) {
-    if (typeof frontmatter.compatibility !== "string") {
-      return { ok: false, error: 'Frontmatter field "compatibility" must be a string when present' };
-    }
-    if (frontmatter.compatibility.length > MAX_COMPATIBILITY_LENGTH) {
-      return { ok: false, error: `compatibility must be <= ${MAX_COMPATIBILITY_LENGTH} characters` };
-    }
-  }
-  if ("license" in frontmatter && typeof frontmatter.license !== "string") {
-    return { ok: false, error: 'Frontmatter field "license" must be a string when present' };
-  }
-  if ("allowed-tools" in frontmatter && typeof frontmatter["allowed-tools"] !== "string") {
-    return { ok: false, error: 'Frontmatter field "allowed-tools" must be a string when present' };
-  }
-  if ("requires-capabilities" in frontmatter) {
-    if (typeof frontmatter["requires-capabilities"] !== "string") {
-      return { ok: false, error: 'Frontmatter field "requires-capabilities" must be a string when present' };
-    }
-    const tokens = frontmatter["requires-capabilities"].split(/\s+/).map((token) => token.trim()).filter((token) => token.length > 0);
-    for (const token of tokens) {
-      if (!CAPABILITY_TOKEN_RE.test(token)) {
-        return {
-          ok: false,
-          error: `requires-capabilities token "${token}" is invalid; expected dotted lowercase tokens (for example "github.issues.write")`
-        };
-      }
-    }
-  }
-  if ("uses-config" in frontmatter) {
-    if (typeof frontmatter["uses-config"] !== "string") {
-      return { ok: false, error: 'Frontmatter field "uses-config" must be a string when present' };
-    }
-    const tokens = frontmatter["uses-config"].split(/\s+/).map((token) => token.trim()).filter((token) => token.length > 0);
-    for (const token of tokens) {
-      if (!CAPABILITY_TOKEN_RE.test(token)) {
-        return {
-          ok: false,
-          error: `uses-config token "${token}" is invalid; expected dotted lowercase tokens (for example "github.repo")`
-        };
-      }
-    }
-  }
-  return {
-    ok: true,
-    frontmatter: {
-      ...frontmatter,
-      name,
-      description
-    }
-  };
-}
-
-// src/chat/skills.ts
 var SKILL_CACHE_TTL_MS = 5e3;
 var skillCache = null;
 function resolveSkillRoots(options) {
@@ -2488,7 +2472,12 @@ function resolveSkillRoots(options) {
   const pluginRoots = getPluginSkillRoots();
   const seen = /* @__PURE__ */ new Set();
   const resolved = [];
-  for (const root of [...additionalRoots, ...envRoots, ...defaults, ...pluginRoots]) {
+  for (const root of [
+    ...additionalRoots,
+    ...envRoots,
+    ...defaults,
+    ...pluginRoots
+  ]) {
     const normalized = path.resolve(root);
     if (seen.has(normalized)) {
       continue;
@@ -2498,22 +2487,6 @@ function resolveSkillRoots(options) {
   }
   return resolved;
 }
-function parseAllowedTools(value) {
-  return parseTokenList(value);
-}
-function parseRequiresCapabilities(value) {
-  return parseTokenList(value);
-}
-function parseUsesConfig(value) {
-  return parseTokenList(value);
-}
-function parseTokenList(value) {
-  if (typeof value !== "string") {
-    return void 0;
-  }
-  const parsed = value.split(/\s+/).map((token) => token.trim()).filter((token) => token.length > 0);
-  return parsed.length > 0 ? parsed : void 0;
-}
 function validateSkillMetadata(input) {
   const unknownCapabilities = (input.requiresCapabilities ?? []).filter(
     (capability) => !isKnownCapability(capability)
@@ -2521,7 +2494,9 @@ function validateSkillMetadata(input) {
   if (unknownCapabilities.length > 0) {
     return `Unknown requires-capabilities values: ${unknownCapabilities.join(", ")}`;
   }
-  const unknownConfigKeys = (input.usesConfig ?? []).filter((configKey) => !isKnownConfigKey(configKey));
+  const unknownConfigKeys = (input.usesConfig ?? []).filter(
+    (configKey) => !isKnownConfigKey(configKey)
+  );
   if (unknownConfigKeys.length > 0) {
     return `Unknown uses-config values: ${unknownConfigKeys.join(", ")}`;
   }
@@ -2531,24 +2506,40 @@ async function readSkillDirectory(skillDir) {
   const skillFile = path.join(skillDir, "SKILL.md");
   try {
     const raw = await fs2.readFile(skillFile, "utf8");
-    const parsed = parseAndValidateSkillFrontmatter(raw, path.basename(skillDir));
+    const parsed = parseSkillFile(raw, path.basename(skillDir));
     if (!parsed.ok) {
-      logWarn("skill_frontmatter_invalid", {}, {
-        "file.path": skillDir,
-        "error.message": parsed.error
-      }, "Invalid skill frontmatter");
+      logWarn(
+        "skill_frontmatter_invalid",
+        {},
+        {
+          "file.path": skillDir,
+          "error.message": parsed.error
+        },
+        "Invalid skill frontmatter"
+      );
       return null;
     }
-    const { name, description } = parsed.frontmatter;
-    const allowedTools = parseAllowedTools(parsed.frontmatter["allowed-tools"]);
-    const requiresCapabilities = parseRequiresCapabilities(parsed.frontmatter["requires-capabilities"]);
-    const usesConfig = parseUsesConfig(parsed.frontmatter["uses-config"]);
-    const metadataError = validateSkillMetadata({ requiresCapabilities, usesConfig });
+    const {
+      name,
+      description,
+      allowedTools,
+      requiresCapabilities,
+      usesConfig
+    } = parsed.skill;
+    const metadataError = validateSkillMetadata({
+      requiresCapabilities,
+      usesConfig
+    });
     if (metadataError) {
-      logWarn("skill_frontmatter_invalid", {}, {
-        "file.path": skillDir,
-        "error.message": metadataError
-      }, "Invalid skill frontmatter");
+      logWarn(
+        "skill_frontmatter_invalid",
+        {},
+        {
+          "file.path": skillDir,
+          "error.message": metadataError
+        },
+        "Invalid skill frontmatter"
+      );
       return null;
     }
     return {
@@ -2560,10 +2551,15 @@ async function readSkillDirectory(skillDir) {
       usesConfig
     };
   } catch (error) {
-    logWarn("skill_directory_read_failed", {}, {
-      "file.path": skillDir,
-      "error.message": error instanceof Error ? error.message : String(error)
-    }, "Failed to read skill directory");
+    logWarn(
+      "skill_directory_read_failed",
+      {},
+      {
+        "file.path": skillDir,
+        "error.message": error instanceof Error ? error.message : String(error)
+      },
+      "Failed to read skill directory"
+    );
     return null;
   }
 }
@@ -2578,7 +2574,9 @@ async function discoverSkills(options) {
   for (const root of roots) {
     try {
       const entries = await fs2.readdir(root, { withFileTypes: true });
-      for (const entry of entries.sort((a, b) => a.name.localeCompare(b.name))) {
+      for (const entry of entries.sort(
+        (a, b) => a.name.localeCompare(b.name)
+      )) {
         if (!entry.isDirectory()) {
           continue;
         }
@@ -2589,10 +2587,15 @@ async function discoverSkills(options) {
         }
       }
     } catch (error) {
-      logWarn("skill_root_read_failed", {}, {
-        "file.directory": root,
-        "error.message": error instanceof Error ? error.message : String(error)
-      }, "Failed to read skill root");
+      logWarn(
+        "skill_root_read_failed",
+        {},
+        {
+          "file.directory": root,
+          "error.message": error instanceof Error ? error.message : String(error)
+        },
+        "Failed to read skill root"
+      );
     }
   }
   const sorted = discovered.sort((a, b) => a.name.localeCompare(b.name));
@@ -2605,7 +2608,9 @@ async function discoverSkills(options) {
 }
 function parseSkillInvocation(messageText, availableSkills) {
   const trimmed = messageText.trim();
-  const match = /(?:^|\s)\/([a-z0-9]+(?:-[a-z0-9]+)*)(?:\s+([\s\S]*))?/i.exec(trimmed);
+  const match = /(?:^|\s)\/([a-z0-9]+(?:-[a-z0-9]+)*)(?:\s+([\s\S]*))?/i.exec(
+    trimmed
+  );
   if (!match) {
     return null;
   }
@@ -2630,9 +2635,13 @@ async function loadSkillsByName(skillNames, available) {
     }
     const skillFile = path.join(meta.skillPath, "SKILL.md");
     const raw = await fs2.readFile(skillFile, "utf8");
+    const parsed = parseSkillFile(raw, meta.name);
+    if (!parsed.ok) {
+      throw new Error(`Invalid skill file in ${skillFile}: ${parsed.error}`);
+    }
     skills.push({
       ...meta,
-      body: stripFrontmatter(raw)
+      body: parsed.skill.body
     });
   }
   return skills;
@@ -3337,19 +3346,11 @@ function toLoadedSkill(result) {
     body: result.instructions
   };
 }
-function stripFrontmatter2(raw) {
-  if (!raw.startsWith("---")) {
-    return raw;
-  }
-  const match = /^---\n[\s\S]*?\n---\n?/.exec(raw);
-  if (!match) {
-    return raw;
-  }
-  return raw.slice(match[0].length);
-}
 async function loadSkillFromSandbox(sandbox, availableSkills, skillName) {
   const requested = skillName.trim().toLowerCase();
-  const skill = availableSkills.find((entry) => entry.name.toLowerCase() === requested);
+  const skill = availableSkills.find(
+    (entry) => entry.name.toLowerCase() === requested
+  );
   if (!skill) {
     return {
       ok: false,
@@ -3369,7 +3370,7 @@ async function loadSkillFromSandbox(sandbox, availableSkills, skillName) {
     description: skill.description,
     skill_dir: skillDir,
     location: skillFilePath,
-    instructions: stripFrontmatter2(file.toString("utf8"))
+    instructions: stripFrontmatter(file.toString("utf8"))
   };
 }
 function createLoadSkillTool(sandbox, availableSkills, options) {
@@ -3382,7 +3383,11 @@ function createLoadSkillTool(sandbox, availableSkills, options) {
       })
     }),
     execute: async ({ skill_name }) => {
-      const result = await loadSkillFromSandbox(sandbox, availableSkills, skill_name);
+      const result = await loadSkillFromSandbox(
+        sandbox,
+        availableSkills,
+        skill_name
+      );
       const loadedSkill = toLoadedSkill(result);
       if (loadedSkill) {
         await options?.onSkillLoaded?.(loadedSkill);
@@ -5540,14 +5545,14 @@ function createSandboxExecutor(options) {
                   } : {}
                 });
                 if (!snapshot.snapshotId) {
-                  await emitSandboxStatus("Starting sandbox...");
+                  await emitSandboxStatus("Booting up...");
                   return await Sandbox.create({
                     timeout: timeoutMs,
                     runtime
                   });
                 }
                 try {
-                  await emitSandboxStatus("Starting sandbox from snapshot...");
+                  await emitSandboxStatus("Booting up...");
                   return await Sandbox.create({
                     timeout: timeoutMs,
                     source: {
@@ -5573,7 +5578,7 @@ function createSandboxExecutor(options) {
                     throw error;
                   }
                   await emitSandboxStatus(
-                    "Retrying sandbox startup with a fresh snapshot..."
+                    "Booting up..."
                   );
                   return await Sandbox.create({
                     timeout: timeoutMs,
@@ -6185,9 +6190,9 @@ function formatToolStatus(toolName) {
   const known = {
     loadSkill: "Loading skill instructions",
     systemTime: "Reading current system time",
-    bash: "Running shell command in sandbox",
-    readFile: "Reading file in sandbox",
-    writeFile: "Writing file in sandbox",
+    bash: "Working in the shell",
+    readFile: "Reading a file",
+    writeFile: "Updating a file",
     webSearch: "Searching public sources",
     webFetch: "Reading source pages",
     slackChannelPostMessage: "Posting message to channel",
@@ -6208,16 +6213,23 @@ function formatToolStatus(toolName) {
 }
 function formatToolStatusWithInput(toolName, input) {
   const obj = input && typeof input === "object" ? input : void 0;
+  const command = obj ? compactStatusText(obj.command, 70) : void 0;
   const path6 = obj ? compactStatusPath(obj.path) : void 0;
   const filename = obj ? compactStatusFilename(obj.path) : void 0;
   const query = obj ? compactStatusText(obj.query, 70) : void 0;
   const domain = obj ? extractStatusUrlDomain(obj.url) : void 0;
   const skillName = obj ? compactStatusText(obj.skill_name ?? obj.skillName, 40) : void 0;
+  if (command && toolName === "bash") {
+    return `Running ${command}`;
+  }
   if (filename && toolName === "readFile") {
     return `Reading file ${filename}`;
   }
+  if (filename && toolName === "writeFile") {
+    return `Updating file ${filename}`;
+  }
   if (path6 && toolName === "writeFile") {
-    return `Writing file ${path6}`;
+    return `Updating file ${path6}`;
   }
   if (skillName && toolName === "loadSkill") {
     return `Loading skill ${skillName}`;
@@ -6234,7 +6246,7 @@ function formatToolResultStatus(toolName) {
   const known = {
     loadSkill: "Integrating loaded skill guidance",
     systemTime: "Applying current time context",
-    bash: "Analyzing command output",
+    bash: "Reviewing command results",
     readFile: "Analyzing file contents",
     writeFile: "Saving file update",
     webSearch: "Reviewing search results",
@@ -6257,16 +6269,23 @@ function formatToolResultStatus(toolName) {
 }
 function formatToolResultStatusWithInput(toolName, input) {
   const obj = input && typeof input === "object" ? input : void 0;
+  const command = obj ? compactStatusText(obj.command, 70) : void 0;
   const path6 = obj ? compactStatusPath(obj.path) : void 0;
   const filename = obj ? compactStatusFilename(obj.path) : void 0;
   const query = obj ? compactStatusText(obj.query, 70) : void 0;
   const domain = obj ? extractStatusUrlDomain(obj.url) : void 0;
   const skillName = obj ? compactStatusText(obj.skill_name ?? obj.skillName, 40) : void 0;
+  if (command && toolName === "bash") {
+    return `Reviewed results from ${command}`;
+  }
   if (filename && toolName === "readFile") {
     return `Reviewed file ${filename}`;
   }
+  if (filename && toolName === "writeFile") {
+    return `Updated file ${filename}`;
+  }
   if (path6 && toolName === "writeFile") {
-    return `Saved file ${path6}`;
+    return `Updated file ${path6}`;
   }
   if (skillName && toolName === "loadSkill") {
     return `Loaded skill ${skillName}`;
@@ -7408,7 +7427,7 @@ function resolveSlackChannelIdFromMessage(message) {
 }
 
 // src/chat/runtime/thread-context.ts
-function escapeRegExp(value) {
+function escapeRegExp2(value) {
   return value.replace(/[.*+?^${}()|[\]\\]/g, "\\$&");
 }
 function stripLeadingBotMention(text, options = {}) {
@@ -7417,10 +7436,10 @@ function stripLeadingBotMention(text, options = {}) {
   if (options.stripLeadingSlackMentionToken) {
     next = next.replace(/^\s*<@[^>]+>[\s,:-]*/, "").trim();
   }
-  const mentionByNameRe = new RegExp(`^\\s*@${escapeRegExp(botConfig.userName)}\\b[\\s,:-]*`, "i");
+  const mentionByNameRe = new RegExp(`^\\s*@${escapeRegExp2(botConfig.userName)}\\b[\\s,:-]*`, "i");
   next = next.replace(mentionByNameRe, "").trim();
   const mentionByLabeledEntityRe = new RegExp(
-    `^\\s*<@[^>|]+\\|${escapeRegExp(botConfig.userName)}>[\\s,:-]*`,
+    `^\\s*<@[^>|]+\\|${escapeRegExp2(botConfig.userName)}>[\\s,:-]*`,
     "i"
   );
   next = next.replace(mentionByLabeledEntityRe, "").trim();
@@ -8019,18 +8038,13 @@ function createAppSlackRuntime(deps) {
           requesterUserName: message.author.userName,
           runId
         });
-        await deps.withSpan(
-          "chat.turn",
-          "chat.turn",
-          context,
-          async () => {
-            await thread.subscribe();
-            await deps.replyToThread(thread, message, {
-              explicitMention: true,
-              beforeFirstResponsePost: hooks?.beforeFirstResponsePost
-            });
-          }
-        );
+        await deps.withSpan("chat.turn", "chat.turn", context, async () => {
+          await thread.subscribe();
+          await deps.replyToThread(thread, message, {
+            explicitMention: true,
+            beforeFirstResponsePost: hooks?.beforeFirstResponsePost
+          });
+        });
       } catch (error) {
         const errorContext = logContext({
           threadId: deps.getThreadId(thread, message),
@@ -8083,6 +8097,45 @@ function createAppSlackRuntime(deps) {
           channelId,
           runId
         };
+        const preflightDecision = hooks?.preApprovedReply ? void 0 : getSubscribedReplyPreflightDecision({
+          botUserName: deps.assistantUserName,
+          rawText: rawUserText,
+          text: userText,
+          isExplicitMention: Boolean(message.isMention)
+        });
+        if (preflightDecision && !preflightDecision.shouldReply) {
+          const completedAtMs = deps.now();
+          const reason = preflightDecision.reasonDetail ? `${preflightDecision.reason}:${preflightDecision.reasonDetail}` : preflightDecision.reason;
+          deps.logWarn(
+            "subscribed_message_reply_skipped",
+            logContext({
+              threadId,
+              requesterId: message.author.userId,
+              requesterUserName: message.author.userName,
+              channelId,
+              runId
+            }),
+            {
+              "app.decision.reason": reason
+            },
+            "Skipping subscribed message reply"
+          );
+          await deps.onSubscribedMessageSkipped({
+            thread,
+            message,
+            decision: { shouldReply: false, reason },
+            completedAtMs,
+            preparedState: void 0
+          });
+          await deps.recordSkippedSubscribedMessage({
+            thread,
+            message,
+            decision: { shouldReply: false, reason },
+            completedAtMs,
+            userText
+          });
+          return;
+        }
         const preparedState = await deps.prepareTurnState({
           thread,
           message,
@@ -10230,11 +10283,58 @@ var appSlackRuntime = createAppSlackRuntime({
   },
   getPreparedConversationContext: (preparedState) => preparedState.routingContext ?? preparedState.conversationContext,
   shouldReplyInSubscribedThread,
-  onSubscribedMessageSkipped: async ({ thread, preparedState, decision, completedAtMs }) => {
-    markConversationMessage(preparedState.conversation, preparedState.userMessageId, {
-      replied: false,
-      skippedReason: decision.reason
+  recordSkippedSubscribedMessage: async ({
+    thread,
+    message,
+    decision,
+    completedAtMs,
+    userText
+  }) => {
+    const conversation = coerceThreadConversationState(await thread.state);
+    const normalizedUserText = normalizeConversationText(userText) || "[non-text message]";
+    upsertConversationMessage(conversation, {
+      id: message.id,
+      role: "user",
+      text: normalizedUserText,
+      createdAtMs: message.metadata.dateSent.getTime(),
+      author: {
+        userId: message.author.userId,
+        userName: message.author.userName,
+        fullName: message.author.fullName,
+        isBot: typeof message.author.isBot === "boolean" ? message.author.isBot : void 0
+      },
+      meta: {
+        explicitMention: Boolean(message.isMention),
+        slackTs: message.id,
+        replied: false,
+        skippedReason: decision.reason,
+        imagesHydrated: true
+      }
+    });
+    conversation.processing.activeTurnId = void 0;
+    conversation.processing.lastCompletedAtMs = completedAtMs;
+    updateConversationStats(conversation);
+    await persistThreadState(thread, {
+      conversation
     });
+  },
+  onSubscribedMessageSkipped: async ({
+    thread,
+    preparedState,
+    decision,
+    completedAtMs
+  }) => {
+    if (!preparedState) {
+      return;
+    }
+    markConversationMessage(
+      preparedState.conversation,
+      preparedState.userMessageId,
+      {
+        replied: false,
+        skippedReason: decision.reason
+      }
+    );
     preparedState.conversation.processing.activeTurnId = void 0;
     preparedState.conversation.processing.lastCompletedAtMs = completedAtMs;
     updateConversationStats(preparedState.conversation);
@@ -10243,7 +10343,12 @@ var appSlackRuntime = createAppSlackRuntime({
     });
   },
   replyToThread,
-  initializeAssistantThread: async ({ threadId, channelId, threadTs, sourceChannelId }) => {
+  initializeAssistantThread: async ({
+    threadId,
+    channelId,
+    threadTs,
+    sourceChannelId
+  }) => {
     await initializeAssistantThread({
       threadId,
       channelId,