fix(reply): normalize Windows media paths for dedupe

aniruddhaadak80 · aniruddhaadak80 · commit 4e0ca4c7108f · 2026-03-12T21:53:47.000+05:30
diff --git a/src/auto-reply/reply/reply-payloads.test.ts b/src/auto-reply/reply/reply-payloads.test.ts
@@ -77,6 +77,14 @@ describe("filterMessagingToolMediaDuplicates", () => {
     });
     expect(result).toEqual([{ text: "hello", mediaUrl: undefined, mediaUrls: undefined }]);
   });
+
+  it("dedupes Windows drive-letter file URLs against local paths", () => {
+    const result = filterMessagingToolMediaDuplicates({
+      payloads: [{ text: "hello", mediaUrl: "file:///C:/tmp/photo one.jpg" }],
+      sentMediaUrls: ["C:\\tmp\\photo one.jpg", "C:/tmp/photo one.jpg"],
+    });
+    expect(result).toEqual([{ text: "hello", mediaUrl: undefined, mediaUrls: undefined }]);
+  });
 });
 
 describe("shouldSuppressMessagingToolReplies", () => {
diff --git a/src/auto-reply/reply/reply-payloads.ts b/src/auto-reply/reply/reply-payloads.ts
@@ -10,6 +10,8 @@ import type { ReplyPayload } from "../types.js";
 import { extractReplyToTag } from "./reply-tags.js";
 import { createReplyToModeFilterForChannel } from "./reply-threading.js";
 
+const WINDOWS_ABSOLUTE_PATH_RE = /^[a-zA-Z]:[\\/]/;
+
 function resolveReplyThreadingForPayload(params: {
   payload: ReplyPayload;
   implicitReplyToId?: string;
@@ -112,12 +114,16 @@ export function filterMessagingToolMediaDuplicates(params: {
       return "";
     }
     if (!trimmed.toLowerCase().startsWith("file://")) {
-      return trimmed;
+      return WINDOWS_ABSOLUTE_PATH_RE.test(trimmed) ? trimmed.replaceAll("/", "\\") : trimmed;
     }
     try {
       const parsed = new URL(trimmed);
       if (parsed.protocol === "file:") {
-        return decodeURIComponent(parsed.pathname || "");
+        const decodedPath = decodeURIComponent(parsed.pathname || "");
+        if (/^\/[a-zA-Z]:\//.test(decodedPath)) {
+          return decodedPath.slice(1).replaceAll("/", "\\");
+        }
+        return decodedPath;
       }
     } catch {
       // Keep fallback below for non-URL-like inputs.
