fix(hooks): pass media metadata to received hook

WarrenJones · WarrenJones · web-flow · commit 65e2120f8c0d · 2026-05-28T01:06:00.000+01:00
Forward canonical inbound media metadata to plugin message_received hooks so plugins can inspect the same mediaPath, mediaUrl, mediaType, mediaPaths, mediaUrls, and mediaTypes fields already available to inbound_claim. Verification: - node scripts/run-vitest.mjs src/hooks/message-hook-mappers.test.ts - /Users/steipete/Projects/agent-scripts/skills/autoreview/scripts/autoreview --mode branch --base origin/main Refs: #87297 Co-authored-by: WarrenJones <8704779+WarrenJones@users.noreply.github.com>
diff --git a/src/hooks/message-hook-mappers.test.ts b/src/hooks/message-hook-mappers.test.ts
@@ -41,6 +41,7 @@ function makeInboundCtx(overrides: Partial<FinalizedMsgContext> = {}): Finalized
     SenderE164: "+15551234567",
     MessageThreadId: 42,
     MediaPath: "/tmp/audio.ogg",
+    MediaUrl: "https://cdn.example.com/audio.ogg",
     MediaType: "audio/ogg",
     GroupSubject: "ops",
     GroupChannel: "ops-room",
@@ -198,7 +199,14 @@ describe("message hook mappers", () => {
       parentSpanId: "3333333333333333",
     };
     const canonical = {
-      ...deriveInboundMessageHookContext(makeInboundCtx({ TopicName: "Deployments" })),
+      ...deriveInboundMessageHookContext(
+        makeInboundCtx({
+          TopicName: "Deployments",
+          MediaPaths: ["/tmp/audio.ogg", "/tmp/photo.jpg"],
+          MediaUrls: ["https://cdn.example.com/audio.ogg", "https://cdn.example.com/photo.jpg"],
+          MediaTypes: ["audio/ogg", "image/jpeg"],
+        }),
+      ),
       runId: "run-1",
       trace,
       callDepth: 2,
@@ -245,6 +253,15 @@ describe("message hook mappers", () => {
     expect(receivedMetadata?.senderName).toBe("User One");
     expect(receivedMetadata?.threadId).toBe(42);
     expect(receivedMetadata?.topicName).toBe("Deployments");
+    expect(receivedMetadata?.mediaPath).toBe("/tmp/audio.ogg");
+    expect(receivedMetadata?.mediaUrl).toBe("https://cdn.example.com/audio.ogg");
+    expect(receivedMetadata?.mediaType).toBe("audio/ogg");
+    expect(receivedMetadata?.mediaPaths).toEqual(["/tmp/audio.ogg", "/tmp/photo.jpg"]);
+    expect(receivedMetadata?.mediaUrls).toEqual([
+      "https://cdn.example.com/audio.ogg",
+      "https://cdn.example.com/photo.jpg",
+    ]);
+    expect(receivedMetadata?.mediaTypes).toEqual(["audio/ogg", "image/jpeg"]);
     const internalReceived = toInternalMessageReceivedContext(canonical);
     const { metadata: internalMetadata, ...internalReceivedBase } = internalReceived;
     expect(internalReceivedBase).toEqual({
diff --git a/src/hooks/message-hook-mappers.ts b/src/hooks/message-hook-mappers.ts
@@ -381,6 +381,12 @@ export function toPluginMessageReceivedEvent(
       senderName: canonical.senderName,
       senderUsername: canonical.senderUsername,
       senderE164: canonical.senderE164,
+      mediaPath: canonical.mediaPath,
+      mediaUrl: canonical.mediaUrl,
+      mediaType: canonical.mediaType,
+      mediaPaths: canonical.mediaPaths,
+      mediaUrls: canonical.mediaUrls,
+      mediaTypes: canonical.mediaTypes,
       guildId: canonical.guildId,
       channelName: canonical.channelName,
       topicName: canonical.topicName,
