From d6552998e9631c66f4759be90102ea877c74aa42 Mon Sep 17 00:00:00 2001 From: YuzuruS Date: Sat, 28 Feb 2026 20:48:03 +0900 Subject: [PATCH] fix: handle outside-workspace error in media store Address Greptile review: add explicit "outside-workspace" case to toSaveMediaSourceError so it returns "Media path is outside workspace root" instead of the generic "Media path is not safe to read". Co-Authored-By: Claude Opus 4.6 --- src/media/store.ts | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/src/media/store.ts b/src/media/store.ts index 7c2477afa..9bfe481c9 100644 --- a/src/media/store.ts +++ b/src/media/store.ts @@ -241,6 +241,10 @@ function toSaveMediaSourceError(err: SafeOpenError): SaveMediaSourceError { return new SaveMediaSourceError("too-large", "Media exceeds 5MB limit", { cause: err }); case "not-found": return new SaveMediaSourceError("not-found", "Media path does not exist", { cause: err }); + case "outside-workspace": + return new SaveMediaSourceError("invalid-path", "Media path is outside workspace root", { + cause: err, + }); case "invalid-path": default: return new SaveMediaSourceError("invalid-path", "Media path is not safe to read", {