fix(slack): guard against undefined text in includes calls during mention handling

src/auto-reply/reply/mentions.test.ts

@@ -0,0 +1,20 @@
+import { describe, expect, it } from "vitest";
+import { stripStructuralPrefixes } from "./mentions.js";
+
+describe("stripStructuralPrefixes", () => {
+  it("returns empty string for undefined input at runtime", () => {
+    expect(stripStructuralPrefixes(undefined as unknown as string)).toBe("");
+  });
+
+  it("returns empty string for empty input", () => {
+    expect(stripStructuralPrefixes("")).toBe("");
+  });
+
+  it("strips sender prefix labels", () => {
+    expect(stripStructuralPrefixes("John: hello")).toBe("hello");
+  });
+
+  it("passes through plain text", () => {
+    expect(stripStructuralPrefixes("just a message")).toBe("just a message");
+  });
+});

src/auto-reply/reply/mentions.ts

@@ -111,6 +111,9 @@ export function matchesMentionWithExplicit(params: {
 }
 
 export function stripStructuralPrefixes(text: string): string {
+  if (!text) {
+    return "";
+  }
   // Ignore wrapper labels, timestamps, and sender prefixes so directive-only
   // detection still works in group batches that include history/context.
   const afterMarker = text.includes(CURRENT_MESSAGE_MARKER)

src/slack/format.test.ts

@@ -57,6 +57,10 @@ describe("markdownToSlackMrkdwn", () => {
       "*Important:* Check the _docs_ at <https://example.com|link>\n\n• first\n• second",
     );
   });
+
+  it("does not throw when input is undefined at runtime", () => {
+    expect(markdownToSlackMrkdwn(undefined as unknown as string)).toBe("");
+  });
 });
 
 describe("escapeSlackMrkdwn", () => {

src/slack/format.ts

@@ -28,6 +28,9 @@ function isAllowedSlackAngleToken(token: string): boolean {
 }
 
 function escapeSlackMrkdwnContent(text: string): string {
+  if (!text) {
+    return "";
+  }
   if (!text.includes("&") && !text.includes("<") && !text.includes(">")) {
     return text;
   }
@@ -53,6 +56,9 @@ function escapeSlackMrkdwnContent(text: string): string {
 }
 
 function escapeSlackMrkdwnText(text: string): string {
+  if (!text) {
+    return "";
+  }
   if (!text.includes("&") && !text.includes("<") && !text.includes(">")) {
     return text;
   }