github/openclaw
1
0
Fork 0
mirror of https://github.com/openclaw/openclaw.git synced 2026-05-08 08:41:23 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix(agents): restrict MEDIA: token parsing to line start in tool results (#18510)

Browse Source
This commit is contained in:
yinghaosang
2026-02-17 04:14:43 +08:00
committed by Peter Steinberger
parent 93fbe6482b
commit 0587e4cc73
2 changed files with 105 additions and 11 deletions
Download Patch File Download Diff File Expand all files Collapse all files

28
src/agents/pi-embedded-subscribe.tools.ts
View File

@@ -153,17 +153,23 @@ export function extractToolResultMediaPaths(result: unknown): string[] {
continue;
}
if (entry.type === "text" && typeof entry.text === "string") {
// Reset lastIndex since MEDIA_TOKEN_RE is global.
MEDIA_TOKEN_RE.lastIndex = 0;
let match: RegExpExecArray | null;
while ((match = MEDIA_TOKEN_RE.exec(entry.text)) !== null) {
// Strip surrounding quotes/backticks and whitespace (mirrors cleanCandidate in media/parse).
const p = match[1]
?.replace(/^[`"'[{(]+/, "")
.replace(/[`"'\]})\\,]+$/, "")
.trim();
if (p && p.length <= 4096) {
paths.push(p);
// Only parse lines that start with MEDIA: (after trimming) to avoid
// false-matching placeholders like <media:audio> or mid-line mentions.
// Mirrors the line-start guard in splitMediaFromOutput (media/parse.ts).
for (const line of entry.text.split("\n")) {
if (!line.trimStart().startsWith("MEDIA:")) {
continue;
}
MEDIA_TOKEN_RE.lastIndex = 0;
let match: RegExpExecArray | null;
while ((match = MEDIA_TOKEN_RE.exec(line)) !== null) {
const p = match[1]
?.replace(/^[`"'[{(]+/, "")
.replace(/[`"'\]})\\,]+$/, "")
.trim();
if (p && p.length <= 4096) {
paths.push(p);
}
}
}
}