fix(slack): keep downloaded files out of reply media (#86...

Recent Commits to openclaw:main

neeravmakwan · 2026-05-25 · via Recent Commits to openclaw:main

Original file line number	Diff line number	Diff line change
`@@ -448,6 +448,7 @@ describe("handleSlackAction", () => {`
`448`	`448`	`});`
`449`	`449`	`expect(details.media).toEqual({`
`450`	`450`	`mediaUrl: "/tmp/openclaw-media/report.pdf",`
	`451`	`+outbound: false,`
`451`	`452`	`contentType: "application/pdf",`
`452`	`453`	`});`
`453`	`454`	`});`

Original file line number	Diff line number	Diff line change
`@@ -492,6 +492,7 @@ export async function handleSlackAction(`
`492`	`492`	`placeholder: downloaded.placeholder,`
`493`	`493`	`media: {`
`494`	`494`	`mediaUrl: downloaded.path,`
	`495`	`+outbound: false,`
`495`	`496`	`...(downloaded.contentType ? { contentType: downloaded.contentType } : {}),`
`496`	`497`	`},`
`497`	`498`	`});`
`@@ -504,6 +505,7 @@ export async function handleSlackAction(`
`504`	`505`	`fileId,`
`505`	`506`	`path: downloaded.path,`
`506`	`507`	`...(downloaded.contentType ? { contentType: downloaded.contentType } : {}),`
	`508`	`+media: { outbound: false },`
`507`	`509`	`},`
`508`	`510`	`});`
`509`	`511`	`}`

Original file line number	Diff line number	Diff line change
`@@ -664,6 +664,61 @@ describe("handleToolExecutionEnd media emission", () => {`
`664`	`664`	`expect(ctx.state.pendingToolTrustedLocalMedia).toBe(true);`
`665`	`665`	`});`
`666`	`666`
	`667`	`+it("does NOT queue structured media marked as non-outbound", async () => {`
	`668`	`+const ctx = createMockContext({`
	`669`	`+shouldEmitToolOutput: false,`
	`670`	`+onToolResult: vi.fn(),`
	`671`	`+builtinToolNames: new Set(["message"]),`
	`672`	`+});`
	`673`	`+`
	`674`	`+await handleToolExecutionEnd(ctx, {`
	`675`	`+type: "tool_execution_end",`
	`676`	`+toolName: "message",`
	`677`	`+toolCallId: "tc-1",`
	`678`	`+isError: false,`
	`679`	`+result: {`
	`680`	`+content: [{ type: "text", text: "Downloaded Slack file to /tmp/report.pdf" }],`
	`681`	`+details: {`
	`682`	`+media: {`
	`683`	`+mediaUrl: "/tmp/report.pdf",`
	`684`	`+outbound: false,`
	`685`	`+},`
	`686`	`+},`
	`687`	`+},`
	`688`	`+});`
	`689`	`+`
	`690`	`+expect(ctx.state.pendingToolMediaUrls).toStrictEqual([]);`
	`691`	`+});`
	`692`	`+`
	`693`	`+it("does NOT queue image fallback paths when media is marked as non-outbound", async () => {`
	`694`	`+const ctx = createMockContext({`
	`695`	`+shouldEmitToolOutput: false,`
	`696`	`+onToolResult: vi.fn(),`
	`697`	`+builtinToolNames: new Set(["message"]),`
	`698`	`+});`
	`699`	`+`
	`700`	`+await handleToolExecutionEnd(ctx, {`
	`701`	`+type: "tool_execution_end",`
	`702`	`+toolName: "message",`
	`703`	`+toolCallId: "tc-1",`
	`704`	`+isError: false,`
	`705`	`+result: {`
	`706`	`+content: [`
	`707`	`+{ type: "text", text: "Downloaded Slack image" },`
	`708`	`+{ type: "image", data: "base64", mimeType: "image/png" },`
	`709`	`+],`
	`710`	`+details: {`
	`711`	`+path: "/tmp/slack-image.png",`
	`712`	`+media: {`
	`713`	`+outbound: false,`
	`714`	`+},`
	`715`	`+},`
	`716`	`+},`
	`717`	`+});`
	`718`	`+`
	`719`	`+expect(ctx.state.pendingToolMediaUrls).toStrictEqual([]);`
	`720`	`+});`
	`721`	`+`
`667`	`722`	`it("queues trusted TTS local media when the exact built-in name is absent", async () => {`
`668`	`723`	`const ctx = createMockContext({`
`669`	`724`	`shouldEmitToolOutput: false,`

Original file line number	Diff line number	Diff line change
`@@ -463,6 +463,10 @@ function collectStructuredMediaUrls(media: Record<string, unknown>): string[] {`
`463`	`463`	`return Array.from(new Set(urls));`
`464`	`464`	`}`
`465`	`465`
	`466`	`+function isNonOutboundToolResultMedia(media: Record<string, unknown>): boolean {`
	`467`	`+return media.outbound === false;`
	`468`	`+}`
	`469`	`+`
`466`	`470`	`function extractTextContentMediaArtifact(content: unknown[]): {`
`467`	`471`	`mediaUrls: string[];`
`468`	`472`	`audioAsVoice?: boolean;`
`@@ -510,6 +514,9 @@ export function extractToolResultMediaArtifact(`
`510`	`514`	`const record = result as Record<string, unknown>;`
`511`	`515`	`const detailsMedia = readToolResultDetailsMedia(record);`
`512`	`516`	`if (detailsMedia) {`
	`517`	`+if (isNonOutboundToolResultMedia(detailsMedia)) {`
	`518`	`+return undefined;`
	`519`	`+}`
`513`	`520`	`const mediaUrls = collectStructuredMediaUrls(detailsMedia);`
`514`	`521`	`if (mediaUrls.length > 0) {`
`515`	`522`	`return {`

此内容由惯性聚合(RSS阅读器)自动聚合整理，仅供阅读参考。原文来自 — 版权归原作者所有。

推荐订阅源

Recent Commits to openclaw:main