惯性聚合 高效追踪和阅读你感兴趣的博客、新闻、科技资讯
阅读原文 在惯性聚合中打开

推荐订阅源

人人都是产品经理
人人都是产品经理
博客园_首页
IT之家
IT之家
让小产品的独立变现更简单 - ezindie.com
让小产品的独立变现更简单 - ezindie.com
Vercel News
Vercel News
美团技术团队
D
Docker
WordPress大学
WordPress大学
T
Tailwind CSS Blog
酷 壳 – CoolShell
酷 壳 – CoolShell
The Cloudflare Blog
Y
Y Combinator Blog
F
Fortinet All Blogs
freeCodeCamp Programming Tutorials: Python, JavaScript, Git & More
G
Google Developers Blog
爱范儿
爱范儿
奇客Solidot–传递最新科技情报
奇客Solidot–传递最新科技情报
月光博客
月光博客
MongoDB | Blog
MongoDB | Blog
S
SegmentFault 最新的问题
GbyAI
GbyAI
Hugging Face - Blog
Hugging Face - Blog
Microsoft Azure Blog
Microsoft Azure Blog
A
About on SuperTechFans

Recent Commits to openclaw:main

test: merge chat side-result checks · openclaw/openclaw@ddd2c2a test: merge cron history checks · openclaw/openclaw@f7eb746 test: merge responsive navigation shell checks · openclaw/openclaw@c2e4b47 docs(changelog): add codex oauth fixes · openclaw/openclaw@628e6cd test: merge navigation routing cases · openclaw/openclaw@5d8cecb Tests: mock channel registry bundled fallback · openclaw/openclaw@2b08233 Secrets: avoid broad web search discovery for single plugin config · openclaw/openclaw@a464f59 test: merge config view browser checks · openclaw/openclaw@20cf511 fix(status): align oauth health with runtime · openclaw/openclaw@eed7116 feat: add macOS screen snapshots for monitor preview (#67954) thanks … · openclaw/openclaw@f377db1 fix: report shared auth scopes in hello-ok (#67810) thanks @BunsDev · openclaw/openclaw@0b6c39b Auto-reply: avoid eager bundled route fallback · openclaw/openclaw@3ea1bf4 Tests: narrow session binding contract setup · openclaw/openclaw@54e4e16 fix(macOS): enable undo/redo in webchat composer text input (#34962) · openclaw/openclaw@00951dc Tests: speed up channel setup promotion · openclaw/openclaw@82b529a Docs: refresh agent instructions · openclaw/openclaw@5775fe2 fix(auth): serialize OAuth refresh across agents to fix #26322 (#67876) · openclaw/openclaw@8e79080 test: allow ollama public surface boundary test · openclaw/openclaw@7d4f1a6 Docs: add test performance guardrails · openclaw/openclaw@89706d3 Tests: restore context-engine usage proof · openclaw/openclaw@e4c4f95 Tests: slim context engine runtime coverage · openclaw/openclaw@74c198f ci: retry failed custom checkouts · openclaw/openclaw@0ee5baf test: trim duplicate provider auth onboarding cases · openclaw/openclaw@1ffc02e matrix: fix sessions_spawn --thread subagent session spawning (#67643) · openclaw/openclaw@1ce2596 test: reduce auth choice fixture churn · openclaw/openclaw@857b9cd test: mock health status config boundaries · openclaw/openclaw@9d5ab4a test: mock onboard config io boundary · openclaw/openclaw@299694d test: mock legacy state plugin boundaries · openclaw/openclaw@2713089 test: mock channel install boundaries · openclaw/openclaw@b945248 test: mock doctor preview channel boundaries · openclaw/openclaw@b1a3ad4
fix(agents): avoid duplicate generated media attachments ...
steipete · 2026-05-05 · via Recent Commits to openclaw:main

@@ -424,6 +424,83 @@ describe("subscribeEmbeddedPiSession", () => {

424424

);

425425

});

426426427+

it("does not attach generated image media to an early streamed chunk before explicit MEDIA", async () => {

428+

const onToolResult = vi.fn();

429+

const onBlockReply = vi.fn();

430+

const { emit } = createSubscribedHarness({

431+

runId: "run",

432+

onToolResult,

433+

onBlockReply,

434+

verboseLevel: "full",

435+

blockReplyBreak: "text_end",

436+

blockReplyChunking: { minChars: 5, maxChars: 200, breakPreference: "newline" },

437+

builtinToolNames: new Set(["image_generate"]),

438+

});

439+440+

emitToolRun({

441+

emit,

442+

toolName: "image_generate",

443+

toolCallId: "tool-1",

444+

isError: false,

445+

result: {

446+

content: [

447+

{

448+

type: "text",

449+

text: "Generated 1 image with google/gemini-3.1-flash-image-preview.\nMEDIA:/tmp/generated.png",

450+

},

451+

],

452+

details: {

453+

media: {

454+

mediaUrls: ["/tmp/generated.png"],

455+

},

456+

},

457+

},

458+

});

459+460+

await vi.waitFor(() => {

461+

expect(onToolResult).toHaveBeenCalled();

462+

});

463+464+

emit({ type: "message_start", message: { role: "assistant" } });

465+

emitAssistantTextDelta(emit, "Generated 1 image.\n");

466+467+

expect(onBlockReply).toHaveBeenCalledWith(

468+

expect.objectContaining({

469+

text: "Generated 1 image.",

470+

}),

471+

);

472+

expect(onBlockReply.mock.calls.some(([payload]) => payload.mediaUrls?.length)).toBe(false);

473+474+

emitAssistantTextDelta(emit, "MEDIA:/tmp/generated.png");

475+

emit({

476+

type: "message_update",

477+

message: { role: "assistant" },

478+

assistantMessageEvent: {

479+

type: "text_end",

480+

content: "Generated 1 image.\nMEDIA:/tmp/generated.png",

481+

},

482+

});

483+

emit({

484+

type: "message_end",

485+

message: {

486+

role: "assistant",

487+

content: [

488+

{

489+

type: "text",

490+

text: "Generated 1 image.\nMEDIA:/tmp/generated.png",

491+

},

492+

],

493+

},

494+

});

495+

emit({ type: "agent_end" });

496+

await flushBlockReplyCallbacks();

497+498+

const mediaPayloads = onBlockReply.mock.calls

499+

.map(([payload]) => payload)

500+

.filter((payload) => payload.mediaUrls?.includes("/tmp/generated.png"));

501+

expect(mediaPayloads).toHaveLength(1);

502+

});

503+427504

it("attaches media from internal completion events even when assistant omits MEDIA lines", async () => {

428505

const onBlockReply = vi.fn();

429506

const { emit } = createSubscribedHarness({

@@ -469,6 +546,104 @@ describe("subscribeEmbeddedPiSession", () => {

469546

);

470547

});

471548549+

it.each([

550+

{

551+

label: "music",

552+

source: "music_generation" as const,

553+

childSessionKey: "music_generate:task-123",

554+

announceType: "music generation task",

555+

taskLabel: "launch anthem",

556+

result: "Generated 1 track.\nMEDIA:/tmp/launch-anthem.mp3",

557+

mediaUrl: "/tmp/launch-anthem.mp3",

558+

firstChunk: "Generated 1 track.\n",

559+

finalText: "Generated 1 track.\nMEDIA:/tmp/launch-anthem.mp3",

560+

},

561+

{

562+

label: "video",

563+

source: "video_generation" as const,

564+

childSessionKey: "video_generate:task-123",

565+

announceType: "video generation task",

566+

taskLabel: "launch reel",

567+

result: "Generated 1 video.\nMEDIA:/tmp/launch-reel.mp4",

568+

mediaUrl: "/tmp/launch-reel.mp4",

569+

firstChunk: "Generated 1 video.\n",

570+

finalText: "Generated 1 video.\nMEDIA:/tmp/launch-reel.mp4",

571+

},

572+

])(

573+

"does not attach $label internal completion media to an early streamed chunk before explicit MEDIA",

574+

async ({

575+

source,

576+

childSessionKey,

577+

announceType,

578+

taskLabel,

579+

result,

580+

mediaUrl,

581+

firstChunk,

582+

finalText,

583+

}) => {

584+

const onBlockReply = vi.fn();

585+

const { emit } = createSubscribedHarness({

586+

runId: "run",

587+

onBlockReply,

588+

blockReplyBreak: "text_end",

589+

blockReplyChunking: { minChars: 5, maxChars: 200, breakPreference: "newline" },

590+

internalEvents: [

591+

{

592+

type: "task_completion",

593+

source,

594+

childSessionKey,

595+

announceType,

596+

taskLabel,

597+

status: "ok",

598+

statusLabel: "completed successfully",

599+

result,

600+

mediaUrls: [mediaUrl],

601+

replyInstruction: "Reply normally.",

602+

},

603+

],

604+

});

605+606+

emit({ type: "message_start", message: { role: "assistant" } });

607+

emitAssistantTextDelta(emit, firstChunk);

608+609+

expect(onBlockReply).toHaveBeenCalledWith(

610+

expect.objectContaining({

611+

text: firstChunk.trim(),

612+

}),

613+

);

614+

expect(onBlockReply.mock.calls.some(([payload]) => payload.mediaUrls?.length)).toBe(false);

615+616+

emitAssistantTextDelta(emit, `MEDIA:${mediaUrl}`);

617+

emit({

618+

type: "message_update",

619+

message: { role: "assistant" },

620+

assistantMessageEvent: {

621+

type: "text_end",

622+

content: finalText,

623+

},

624+

});

625+

emit({

626+

type: "message_end",

627+

message: {

628+

role: "assistant",

629+

content: [

630+

{

631+

type: "text",

632+

text: finalText,

633+

},

634+

],

635+

},

636+

});

637+

emit({ type: "agent_end" });

638+

await flushBlockReplyCallbacks();

639+640+

const mediaPayloads = onBlockReply.mock.calls

641+

.map(([payload]) => payload)

642+

.filter((payload) => payload.mediaUrls?.includes(mediaUrl));

643+

expect(mediaPayloads).toHaveLength(1);

644+

},

645+

);

646+472647

it("keeps orphaned tool media available for non-block final payload assembly", () => {

473648

const { emit, subscription } = createSubscribedSessionHarness({

474649

runId: "run",