fix(qqbot): preserve host-read media roots

sliverp · zhangguiping-xydt · commit 257d0cbd9b14 · 2026-07-01T22:11:41.000+08:00
diff --git a/extensions/qqbot/src/engine/messaging/outbound-media-send.test.ts b/extensions/qqbot/src/engine/messaging/outbound-media-send.test.ts
@@ -224,7 +224,10 @@ describe("trySendViaHostRead error handling", () => {
     expect(mockedLoadOutboundMediaFromUrl).toHaveBeenCalledWith(
       "/tmp/workspace/report.docx",
       expect.objectContaining({
-        mediaAccess: expect.objectContaining({ workspaceDir: "/tmp/workspace" }),
+        mediaAccess: expect.objectContaining({
+          localRoots: ["/tmp/openclaw-sandbox", "/tmp/workspace"],
+          workspaceDir: "/tmp/workspace",
+        }),
         workspaceDir: "/tmp/workspace",
       }),
     );
@@ -257,10 +260,9 @@ describe("trySendViaHostRead error handling", () => {
       "/workspace/attachments/report.docx",
       expect.objectContaining({
         mediaAccess: expect.objectContaining({
-          localRoots: ["/workspace/attachments"],
+          localRoots: ["/workspace/attachments", "/tmp/agent-workspace"],
           workspaceDir: "/tmp/agent-workspace",
         }),
-        mediaLocalRoots: ["/workspace/attachments"],
         workspaceDir: "/tmp/agent-workspace",
       }),
     );
@@ -280,7 +282,13 @@ describe("trySendViaHostRead error handling", () => {
     expect(result).toMatchObject({ channel: "qqbot", messageId: "voice-1" });
     expect(mockedLoadOutboundMediaFromUrl).toHaveBeenCalledWith(
       "/tmp/workspace/clip.mp3",
-      expect.objectContaining({ maxBytes: expect.any(Number) }),
+      expect.objectContaining({
+        maxBytes: expect.any(Number),
+        mediaAccess: expect.objectContaining({
+          localRoots: ["/tmp/openclaw-sandbox", "/tmp/workspace"],
+          workspaceDir: "/tmp/workspace",
+        }),
+      }),
     );
     expect(audioPortMock.waitForFile).toHaveBeenCalledWith(expect.stringMatching(/clip-.*\.mp3$/));
     expect(mockedSenderSendMedia).toHaveBeenCalledWith(
diff --git a/extensions/qqbot/src/engine/messaging/outbound-media-send.ts b/extensions/qqbot/src/engine/messaging/outbound-media-send.ts
@@ -210,6 +210,19 @@ function resolveOutboundMediaReadFile(ctx: OutboundMediaAccessContext) {
   return ctx.mediaAccess?.readFile ?? ctx.mediaReadFile;
 }
 
+function resolveHostReadMediaAccess(
+  ctx: OutboundMediaAccessContext,
+): OutboundMediaAccessContext["mediaAccess"] | undefined {
+  const mediaLocalRoots = resolveOutboundMediaLocalRoots(ctx);
+  if (!ctx.mediaAccess && !mediaLocalRoots) {
+    return undefined;
+  }
+  return {
+    ...(ctx.mediaAccess ?? {}),
+    ...(mediaLocalRoots ? { localRoots: mediaLocalRoots } : {}),
+  };
+}
+
 function mediaFileTypeForKind(mediaKind: QQBotMediaKind): MediaFileType {
   switch (mediaKind) {
     case "image":
@@ -265,12 +278,12 @@ async function stageHostReadVoice(
     return null;
   }
   const hostReadMediaPath = resolveHostReadMediaPath(ctx, mediaPath);
+  const mediaAccess = resolveHostReadMediaAccess(ctx);
   const loaded = await loadOutboundMediaFromUrl(hostReadMediaPath, {
     maxBytes: getMaxUploadSize(MediaFileType.VOICE),
-    mediaAccess: ctx.mediaAccess,
-    mediaLocalRoots: ctx.mediaLocalRoots,
+    mediaAccess,
     mediaReadFile,
-    workspaceDir: ctx.mediaAccess?.workspaceDir,
+    workspaceDir: mediaAccess?.workspaceDir,
   });
   const stagedDir = getQQBotMediaDir("host-read", "voice");
   await mkdir(stagedDir, { recursive: true });
@@ -292,13 +305,13 @@ async function trySendViaHostRead(
     return null;
   }
   const hostReadMediaPath = resolveHostReadMediaPath(ctx, mediaPath);
+  const mediaAccess = resolveHostReadMediaAccess(ctx);
   try {
     const loaded = await loadOutboundMediaFromUrl(hostReadMediaPath, {
       maxBytes: getMaxUploadSize(mediaFileTypeForKind(mediaKind)),
-      mediaAccess: ctx.mediaAccess,
-      mediaLocalRoots: ctx.mediaLocalRoots,
+      mediaAccess,
       mediaReadFile,
-      workspaceDir: ctx.mediaAccess?.workspaceDir,
+      workspaceDir: mediaAccess?.workspaceDir,
     });
     const kind = senderKindForLoadedMedia(mediaKind, loaded.kind);
     const creds = accountToCreds(ctx.account);
