fix(media): normalizeMediaSource should handle uppercase FILE:// URI schemes (#103490)

xydt-lcy · claude · steipete · web-flow · commit e0df20b0b39e · 2026-07-10T16:43:00.000+01:00
* fix: normalizeMediaSource should handle uppercase FILE:// URI schemes URI schemes are case-insensitive per RFC 3986. normalizeMediaSource only matched lowercase 'file://', so MEDIA:FILE:///tmp/generated.png was left in assistant text instead of being extracted as a local media attachment. Replaced startsWith + replace with a case-insensitive regex to handle FILE://, file://, and any mixed-case variant. Fixes #103473 Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> * fix(media): handle spaced uppercase file URLs --------- Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com> Co-authored-by: Peter Steinberger <steipete@gmail.com>
diff --git a/src/media/parse.test.ts b/src/media/parse.test.ts
@@ -66,6 +66,10 @@ describe("splitMediaFromOutput", () => {
       String.raw`MEDIA:/path/to/image.png\"}],\"details\":{\"provider\":\"openai\"}`,
     ],
     ["/tmp/render,final.png", "MEDIA:/tmp/render,final.png"],
+    ["/tmp/generated.png", "MEDIA:FILE:///tmp/generated.png"],
+    ["/tmp/generated.png", "MEDIA:file:///tmp/generated.png"],
+    ["/Users/pete/My File.png", "MEDIA:FILE:///Users/pete/My File.png"],
+    ["/Users/pete/My File.png", "MEDIA:file:///Users/pete/My File.png"],
   ] as const)("accepts supported media path variant: %s", (expectedPath, input) => {
     expectAcceptedMediaPathCase(expectedPath, input);
   });
diff --git a/src/media/parse.ts b/src/media/parse.ts
@@ -33,9 +33,11 @@ export type SplitMediaFromOutputOptions = {
   extractMediaDirectives?: boolean;
 };
 
+const FILE_URL_PREFIX_RE = /^file:\/\//i;
+
 /** Converts file URLs into plain local paths before downstream media validation. */
 export function normalizeMediaSource(src: string): string {
-  return src.startsWith("file://") ? src.replace("file://", "") : src;
+  return src.replace(FILE_URL_PREFIX_RE, "");
 }
 
 const TRAILING_SERIALIZED_JSON_AFTER_EXT_RE = /^(.*\.\w{1,10})\\?"(?=[\]},:]|$).*/s;
@@ -603,7 +605,7 @@ export function splitMediaFromOutput(
 
       const trimmedPayload = payloadValue.trim();
       const looksLikeLocalPath =
-        looksLikeLocalFilePath(trimmedPayload) || trimmedPayload.startsWith("file://");
+        looksLikeLocalFilePath(trimmedPayload) || FILE_URL_PREFIX_RE.test(trimmedPayload);
       if (
         !unwrapped &&
         validCount === 1 &&
