From a02bcb362030262d8cd78b621ecd519fb6987cac Mon Sep 17 00:00:00 2001 From: Yaroslav Boiko Date: Mon, 16 Feb 2026 22:48:58 +0100 Subject: [PATCH] fix(test): add missing media dedup state fields to mock contexts Pre-existing test mocks lacked pendingMessagingMediaUrls and messagingToolSentMediaUrls fields added by the media dedup feature, causing runtime errors in handleToolExecutionEnd. Co-Authored-By: Claude Opus 4.6 --- ...nner.run-embedded-pi-agent.auth-profile-rotation.e2e.test.ts | 1 + src/agents/pi-embedded-subscribe.handlers.tools.media.test.ts | 2 ++ src/agents/pi-embedded-subscribe.handlers.tools.test.ts | 1 - src/plugins/wired-hooks-after-tool-call.e2e.test.ts | 2 ++ 4 files changed, 5 insertions(+), 1 deletion(-) diff --git a/src/agents/pi-embedded-runner.run-embedded-pi-agent.auth-profile-rotation.e2e.test.ts b/src/agents/pi-embedded-runner.run-embedded-pi-agent.auth-profile-rotation.e2e.test.ts index 1311f9d2b0..bc13d4fd36 100644 --- a/src/agents/pi-embedded-runner.run-embedded-pi-agent.auth-profile-rotation.e2e.test.ts +++ b/src/agents/pi-embedded-runner.run-embedded-pi-agent.auth-profile-rotation.e2e.test.ts @@ -57,6 +57,7 @@ const makeAttempt = (overrides: Partial): EmbeddedRunA lastAssistant: undefined, didSendViaMessagingTool: false, messagingToolSentTexts: [], + messagingToolSentMediaUrls: [], messagingToolSentTargets: [], cloudCodeAssistFormatError: false, ...overrides, diff --git a/src/agents/pi-embedded-subscribe.handlers.tools.media.test.ts b/src/agents/pi-embedded-subscribe.handlers.tools.media.test.ts index 053f13f179..5d1c04cd8d 100644 --- a/src/agents/pi-embedded-subscribe.handlers.tools.media.test.ts +++ b/src/agents/pi-embedded-subscribe.handlers.tools.media.test.ts @@ -23,8 +23,10 @@ function createMockContext(overrides?: { toolSummaryById: new Set(), pendingMessagingTexts: new Map(), pendingMessagingTargets: new Map(), + pendingMessagingMediaUrls: new Map(), messagingToolSentTexts: [], messagingToolSentTextsNormalized: [], + messagingToolSentMediaUrls: [], messagingToolSentTargets: [], }, log: { debug: vi.fn(), warn: vi.fn() }, diff --git a/src/agents/pi-embedded-subscribe.handlers.tools.test.ts b/src/agents/pi-embedded-subscribe.handlers.tools.test.ts index 8fbd298347..bf710df713 100644 --- a/src/agents/pi-embedded-subscribe.handlers.tools.test.ts +++ b/src/agents/pi-embedded-subscribe.handlers.tools.test.ts @@ -264,6 +264,5 @@ describe("messaging tool media URL tracking", () => { expect(ctx.state.messagingToolSentMediaUrls).toHaveLength(0); expect(ctx.state.pendingMessagingMediaUrls.has("tool-m3")).toBe(false); ->>>>>>> 018297172 (test(media-dedup): add missing coverage for Discord media dedup wiring) }); }); diff --git a/src/plugins/wired-hooks-after-tool-call.e2e.test.ts b/src/plugins/wired-hooks-after-tool-call.e2e.test.ts index 91cfa51014..868f539ecd 100644 --- a/src/plugins/wired-hooks-after-tool-call.e2e.test.ts +++ b/src/plugins/wired-hooks-after-tool-call.e2e.test.ts @@ -41,8 +41,10 @@ function createToolHandlerCtx(params: { lastToolError: undefined, pendingMessagingTexts: new Map(), pendingMessagingTargets: new Map(), + pendingMessagingMediaUrls: new Map(), messagingToolSentTexts: [] as string[], messagingToolSentTextsNormalized: [] as string[], + messagingToolSentMediaUrls: [] as string[], messagingToolSentTargets: [] as unknown[], blockBuffer: "", },