fix: normalizeMediaSource should handle uppercase FILE:// URI schemes

xydt-lcy · claude · xydt-lcy · commit 339b893a378e · 2026-07-10T15:08:10.000+08:00
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>
diff --git a/src/media/parse.test.ts b/src/media/parse.test.ts
@@ -66,6 +66,8 @@ 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"],
   ] 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
@@ -35,7 +35,7 @@ export type SplitMediaFromOutputOptions = {
 
 /** 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:\/\//i, "");
 }
 
 const TRAILING_SERIALIZED_JSON_AFTER_EXT_RE = /^(.*\.\w{1,10})\\?"(?=[\]},:]|$).*/s;

Original file line number	Diff line number	Diff line change
`@@ -35,7 +35,7 @@ export type SplitMediaFromOutputOptions = {`
`35`	`35`
`36`	`36`	`/** Converts file URLs into plain local paths before downstream media validation. */`
`37`	`37`	`export function normalizeMediaSource(src: string): string {`
`38`		`- return src.startsWith("file://") ? src.replace("file://", "") : src;`
	`38`	`+ return src.replace(/^file:\/\//i, "");`
`39`	`39`	`}`
`40`	`40`
`41`	`41`	`const TRAILING_SERIALIZED_JSON_AFTER_EXT_RE = /^(.\.\w{1,10})\\?"(?=[\]},:]\|$)./s;`