fix: clarify generated media reply prompts (#88458) · openclaw/openclaw@8a46790

Recent Commits to openclaw:main

steipete · 2026-05-31 · via Recent Commits to openclaw:main

Original file line number	Diff line number	Diff line change
`@@ -991,7 +991,9 @@ describe("buildAgentSystemPrompt", () => {`
`991`	`991`	`});`
`992`	`992`
`993`	`993`	expect(prompt).toContain("use `message(action=send)` for visible source-channel output");
`994`		`-expect(prompt).toContain("Attach media with message-tool attachment fields");`
	`994`	`+expect(prompt).toContain(`
	`995`	+"Tool/generated media paths are attachments, not prose; send one with `media`, multiple with `attachments: [{media: ...}]`.",
	`996`	`+);`
`995`	`997`	expect(prompt).not.toContain("Attach media: `MEDIA:<path-or-url>`");
`996`	`998`	`expect(prompt).toContain("The target defaults to the current source channel");`
`997`	`999`	`expect(prompt).toContain("do not repeat that visible content in your final answer");`
`@@ -1003,6 +1005,23 @@ describe("buildAgentSystemPrompt", () => {`
`1003`	`1005`	expect(prompt).not.toContain("For `action=send`, include `target` and `message`.");
`1004`	`1006`	`});`
`1005`	`1007`
	`1008`	`+it("tells automatic source delivery to expose generated media as MEDIA directives", () => {`
	`1009`	`+const prompt = buildAgentSystemPrompt({`
	`1010`	`+workspaceDir: "/tmp/openclaw",`
	`1011`	`+toolNames: ["message"],`
	`1012`	`+runtimeInfo: {`
	`1013`	`+channel: "telegram",`
	`1014`	`+},`
	`1015`	`+});`
	`1016`	`+`
	`1017`	`+expect(prompt).toContain(`
	`1018`	+"Attach media in the final visible reply with `MEDIA:<path-or-url>` on its own line.",
	`1019`	`+);`
	`1020`	`+expect(prompt).toContain(`
	`1021`	+"Tool/generated media paths are attachments, not prose; emit each as its own `MEDIA:<path-or-url>` line.",
	`1022`	`+);`
	`1023`	`+});`
	`1024`	`+`
`1006`	`1025`	`it("suppresses plain chat approval commands when inline approval UI is available", () => {`
`1007`	`1026`	`const prompt = buildAgentSystemPrompt({`
`1008`	`1027`	`workspaceDir: "/tmp/openclaw",`

Original file line number	Diff line number	Diff line change
`@@ -396,7 +396,8 @@ function buildAssistantOutputDirectivesSection(params: {`
`396`	`396`	`return [`
`397`	`397`	`"## Assistant Output Directives",`
`398`	`398`	"- Visible source-channel output is delivered through `message(action=send)`.",
`399`		-"- Attach media with message-tool attachment fields such as `media`, `mediaUrl`, `path`, or `filePath`; do not write attachment commands in text.",
	`399`	+"- Tool/generated media paths are attachments, not prose; send one with `media`, multiple with `attachments: [{media: ...}]`.",
	`400`	+"- Do not use legacy `MEDIA:` directives for source-channel delivery.",
`400`	`401`	"- Voice-note audio hint: use message-tool `asVoice` when sending audio as a voice note.",
`401`	`402`	"- Native quote/reply: use message-tool `replyTo` when an explicit reply target is needed.",
`402`	`403`	`"",`
`@@ -405,6 +406,7 @@ function buildAssistantOutputDirectivesSection(params: {`
`405`	`406`	`return [`
`406`	`407`	`"## Assistant Output Directives",`
`407`	`408`	"- Attach media in the final visible reply with `MEDIA:<path-or-url>` on its own line.",
	`409`	+"- Tool/generated media paths are attachments, not prose; emit each as its own `MEDIA:<path-or-url>` line.",
`408`	`410`	" The MEDIA directive must start the line as plain text, outside code fences and without Markdown wrappers. Do not write `MEDIA:...`, `` `MEDIA:...` ``, or inline prose like `Here is the file: MEDIA:...`.",
`409`	`411`	"- Voice-note audio hint: `[[audio_as_voice]]` when audio is attached.",
`410`	`412`	"- Native quote/reply: first token `[[reply_to_current]]`; use `[[reply_to:<id>]]` only with an explicit id.",

Original file line number	Diff line number	Diff line change
`@@ -1075,6 +1075,25 @@ describe("message tool explicit target guard", () => {`
`1075`	`1075`	`});`
`1076`	`1076`
`1077`	`1077`	`describe("message tool path passthrough", () => {`
	`1078`	`+it("advertises canonical media params without compat aliases", () => {`
	`1079`	`+const properties = getToolProperties(createMessageTool());`
	`1080`	`+const attachments = properties.attachments as`
	`1081`	`+\| { items?: { properties?: Record<string, unknown> } }`
	`1082`	`+\| undefined;`
	`1083`	`+const attachmentProperties = attachments?.items?.properties ?? {};`
	`1084`	`+`
	`1085`	`+expect(properties).toHaveProperty("media");`
	`1086`	`+expect(properties).not.toHaveProperty("mediaUrl");`
	`1087`	`+expect(properties).not.toHaveProperty("mediaUrls");`
	`1088`	`+expect(properties).not.toHaveProperty("path");`
	`1089`	`+expect(properties).not.toHaveProperty("filePath");`
	`1090`	`+expect(properties).not.toHaveProperty("fileUrl");`
	`1091`	`+expect(attachmentProperties).toHaveProperty("media");`
	`1092`	`+for (const name of ["mediaUrl", "path", "filePath", "fileUrl", "url"]) {`
	`1093`	`+expect(attachmentProperties).not.toHaveProperty(name);`
	`1094`	`+}`
	`1095`	`+});`
	`1096`	`+`
`1078`	`1097`	`it.each([`
`1079`	`1098`	`{ field: "path", value: "~/Downloads/voice.ogg" },`
`1080`	`1099`	`{ field: "filePath", value: "./tmp/note.m4a" },`

Original file line number	Diff line number	Diff line change
`@@ -190,16 +190,6 @@ function buildSendSchema(options: {`
`190`	`190`	`description: "Media URL/path. data: use buffer.",`
`191`	`191`	`}),`
`192`	`192`	`),`
`193`		`-mediaUrl: Type.Optional(`
`194`		`-Type.String({`
`195`		`-description: "Alias for media.",`
`196`		`-}),`
`197`		`-),`
`198`		`-mediaUrls: Type.Optional(`
`199`		`-Type.Array(Type.String(), {`
`200`		`-description: "Multiple media URLs/paths.",`
`201`		`-}),`
`202`		`-),`
`203`	`193`	`filename: Type.Optional(Type.String()),`
`204`	`194`	`buffer: Type.Optional(`
`205`	`195`	`Type.String({`
`@@ -209,24 +199,16 @@ function buildSendSchema(options: {`
`209`	`199`	`contentType: Type.Optional(Type.String()),`
`210`	`200`	`mimeType: Type.Optional(Type.String()),`
`211`	`201`	`caption: Type.Optional(Type.String()),`
`212`		`-path: Type.Optional(Type.String()),`
`213`		`-filePath: Type.Optional(Type.String()),`
`214`	`202`	`attachments: Type.Optional(`
`215`	`203`	`Type.Array(`
`216`	`204`	`Type.Object({`
`217`	`205`	`type: Type.Optional(stringEnum(["image", "audio", "video", "file"])),`
`218`	`206`	`media: Type.Optional(Type.String()),`
`219`		`-mediaUrl: Type.Optional(Type.String()),`
`220`		`-path: Type.Optional(Type.String()),`
`221`		`-filePath: Type.Optional(Type.String()),`
`222`		`-fileUrl: Type.Optional(Type.String()),`
`223`		`-url: Type.Optional(Type.String()),`
`224`	`207`	`name: Type.Optional(Type.String()),`
`225`	`208`	`mimeType: Type.Optional(Type.String()),`
`226`	`209`	`}),`
`227`	`210`	`{`
`228`		`-description:`
`229`		`-"Structured attachments; each needs media/mediaUrl/path/filePath/fileUrl/url.",`
	`211`	`+description: "Structured attachments; each entry uses media.",`
`230`	`212`	`},`
`231`	`213`	`),`
`232`	`214`	`),`

Original file line number	Diff line number	Diff line change
`@@ -658,36 +658,21 @@`
`658`	`658`	`"type": "boolean"`
`659`	`659`	`},`
`660`	`660`	`"attachments": {`
`661`		`-"description": "Structured attachments; each needs media/mediaUrl/path/filePath/fileUrl/url.",`
	`661`	`+"description": "Structured attachments; each entry uses media.",`
`662`	`662`	`"items": {`
`663`	`663`	`"properties": {`
`664`		`-"filePath": {`
`665`		`-"type": "string"`
`666`		`- },`
`667`		`-"fileUrl": {`
`668`		`-"type": "string"`
`669`		`- },`
`670`	`664`	`"media": {`
`671`	`665`	`"type": "string"`
`672`	`666`	`},`
`673`		`-"mediaUrl": {`
`674`		`-"type": "string"`
`675`		`- },`
`676`	`667`	`"mimeType": {`
`677`	`668`	`"type": "string"`
`678`	`669`	`},`
`679`	`670`	`"name": {`
`680`	`671`	`"type": "string"`
`681`	`672`	`},`
`682`		`-"path": {`
`683`		`-"type": "string"`
`684`		`- },`
`685`	`673`	`"type": {`
`686`	`674`	`"enum": ["image", "audio", "video", "file"],`
`687`	`675`	`"type": "string"`
`688`		`- },`
`689`		`-"url": {`
`690`		`-"type": "string"`
`691`	`676`	`}`
`692`	`677`	`},`
`693`	`678`	`"type": "object"`
`@@ -721,9 +706,6 @@`
`721`	`706`	`"filename": {`
`722`	`707`	`"type": "string"`
`723`	`708`	`},`
`724`		`-"filePath": {`
`725`		`-"type": "string"`
`726`		`- },`
`727`	`709`	`"forceDocument": {`
`728`	`710`	`"description": "Send image/GIF/video as document; avoids compression.",`
`729`	`711`	`"type": "boolean"`
`@@ -741,26 +723,12 @@`
`741`	`723`	`"description": "Media URL/path. data: use buffer.",`
`742`	`724`	`"type": "string"`
`743`	`725`	`},`
`744`		`-"mediaUrl": {`
`745`		`-"description": "Alias for media.",`
`746`		`-"type": "string"`
`747`		`- },`
`748`		`-"mediaUrls": {`
`749`		`-"description": "Multiple media URLs/paths.",`
`750`		`-"items": {`
`751`		`-"type": "string"`
`752`		`- },`
`753`		`-"type": "array"`
`754`		`- },`
`755`	`726`	`"message": {`
`756`	`727`	`"type": "string"`
`757`	`728`	`},`
`758`	`729`	`"mimeType": {`
`759`	`730`	`"type": "string"`
`760`	`731`	`},`
`761`		`-"path": {`
`762`		`-"type": "string"`
`763`		`- },`
`764`	`732`	`"quoteText": {`
`765`	`733`	`"description": "Telegram reply quote text.",`
`766`	`734`	`"type": "string"`

Original file line number	Diff line number	Diff line change
`@@ -658,36 +658,21 @@`
`658`	`658`	`"type": "boolean"`
`659`	`659`	`},`
`660`	`660`	`"attachments": {`
`661`		`-"description": "Structured attachments; each needs media/mediaUrl/path/filePath/fileUrl/url.",`
	`661`	`+"description": "Structured attachments; each entry uses media.",`
`662`	`662`	`"items": {`
`663`	`663`	`"properties": {`
`664`		`-"filePath": {`
`665`		`-"type": "string"`
`666`		`- },`
`667`		`-"fileUrl": {`
`668`		`-"type": "string"`
`669`		`- },`
`670`	`664`	`"media": {`
`671`	`665`	`"type": "string"`
`672`	`666`	`},`
`673`		`-"mediaUrl": {`
`674`		`-"type": "string"`
`675`		`- },`
`676`	`667`	`"mimeType": {`
`677`	`668`	`"type": "string"`
`678`	`669`	`},`
`679`	`670`	`"name": {`
`680`	`671`	`"type": "string"`
`681`	`672`	`},`
`682`		`-"path": {`
`683`		`-"type": "string"`
`684`		`- },`
`685`	`673`	`"type": {`
`686`	`674`	`"enum": ["image", "audio", "video", "file"],`
`687`	`675`	`"type": "string"`
`688`		`- },`
`689`		`-"url": {`
`690`		`-"type": "string"`
`691`	`676`	`}`
`692`	`677`	`},`
`693`	`678`	`"type": "object"`
`@@ -721,9 +706,6 @@`
`721`	`706`	`"filename": {`
`722`	`707`	`"type": "string"`
`723`	`708`	`},`
`724`		`-"filePath": {`
`725`		`-"type": "string"`
`726`		`- },`
`727`	`709`	`"forceDocument": {`
`728`	`710`	`"description": "Send image/GIF/video as document; avoids compression.",`
`729`	`711`	`"type": "boolean"`
`@@ -741,26 +723,12 @@`
`741`	`723`	`"description": "Media URL/path. data: use buffer.",`
`742`	`724`	`"type": "string"`
`743`	`725`	`},`
`744`		`-"mediaUrl": {`
`745`		`-"description": "Alias for media.",`
`746`		`-"type": "string"`
`747`		`- },`
`748`		`-"mediaUrls": {`
`749`		`-"description": "Multiple media URLs/paths.",`
`750`		`-"items": {`
`751`		`-"type": "string"`
`752`		`- },`
`753`		`-"type": "array"`
`754`		`- },`
`755`	`726`	`"message": {`
`756`	`727`	`"type": "string"`
`757`	`728`	`},`
`758`	`729`	`"mimeType": {`
`759`	`730`	`"type": "string"`
`760`	`731`	`},`
`761`		`-"path": {`
`762`		`-"type": "string"`
`763`		`- },`
`764`	`732`	`"quoteText": {`
`765`	`733`	`"description": "Telegram reply quote text.",`
`766`	`734`	`"type": "string"`

Original file line number	Diff line number	Diff line change
`@@ -658,36 +658,21 @@`
`658`	`658`	`"type": "boolean"`
`659`	`659`	`},`
`660`	`660`	`"attachments": {`
`661`		`-"description": "Structured attachments; each needs media/mediaUrl/path/filePath/fileUrl/url.",`
	`661`	`+"description": "Structured attachments; each entry uses media.",`
`662`	`662`	`"items": {`
`663`	`663`	`"properties": {`
`664`		`-"filePath": {`
`665`		`-"type": "string"`
`666`		`- },`
`667`		`-"fileUrl": {`
`668`		`-"type": "string"`
`669`		`- },`
`670`	`664`	`"media": {`
`671`	`665`	`"type": "string"`
`672`	`666`	`},`
`673`		`-"mediaUrl": {`
`674`		`-"type": "string"`
`675`		`- },`
`676`	`667`	`"mimeType": {`
`677`	`668`	`"type": "string"`
`678`	`669`	`},`
`679`	`670`	`"name": {`
`680`	`671`	`"type": "string"`
`681`	`672`	`},`
`682`		`-"path": {`
`683`		`-"type": "string"`
`684`		`- },`
`685`	`673`	`"type": {`
`686`	`674`	`"enum": ["image", "audio", "video", "file"],`
`687`	`675`	`"type": "string"`
`688`		`- },`
`689`		`-"url": {`
`690`		`-"type": "string"`
`691`	`676`	`}`
`692`	`677`	`},`
`693`	`678`	`"type": "object"`
`@@ -721,9 +706,6 @@`
`721`	`706`	`"filename": {`
`722`	`707`	`"type": "string"`
`723`	`708`	`},`
`724`		`-"filePath": {`
`725`		`-"type": "string"`
`726`		`- },`
`727`	`709`	`"forceDocument": {`
`728`	`710`	`"description": "Send image/GIF/video as document; avoids compression.",`
`729`	`711`	`"type": "boolean"`
`@@ -741,26 +723,12 @@`
`741`	`723`	`"description": "Media URL/path. data: use buffer.",`
`742`	`724`	`"type": "string"`
`743`	`725`	`},`
`744`		`-"mediaUrl": {`
`745`		`-"description": "Alias for media.",`
`746`		`-"type": "string"`
`747`		`- },`
`748`		`-"mediaUrls": {`
`749`		`-"description": "Multiple media URLs/paths.",`
`750`		`-"items": {`
`751`		`-"type": "string"`
`752`		`- },`
`753`		`-"type": "array"`
`754`		`- },`
`755`	`726`	`"message": {`
`756`	`727`	`"type": "string"`
`757`	`728`	`},`
`758`	`729`	`"mimeType": {`
`759`	`730`	`"type": "string"`
`760`	`731`	`},`
`761`		`-"path": {`
`762`		`-"type": "string"`
`763`		`- },`
`764`	`732`	`"quoteText": {`
`765`	`733`	`"description": "Telegram reply quote text.",`
`766`	`734`	`"type": "string"`

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

推荐订阅源

Recent Commits to openclaw:main