@syengup/friday-channel-next 0.1.30 → 0.1.37

package/index.ts

@@ -1,7 +1,10 @@
-import type { ChannelPlugin } from "openclaw/plugin-sdk/core";
 import { defineChannelPluginEntry } from "openclaw/plugin-sdk/core";
 import type { OpenClawPluginApi } from "openclaw/plugin-sdk/plugin-entry";
-import type { PluginHookBeforeToolCallEvent, PluginHookAfterToolCallEvent, PluginHookToolContext } from "openclaw/plugin-sdk/plugins/types";
+import type {
+  PluginHookBeforeToolCallEvent,
+  PluginHookAfterToolCallEvent,
+  PluginHookToolContext,
+} from "openclaw/plugin-sdk/plugins/types";
 import { fridayNextChannelPlugin } from "./src/channel.js";
 import { setFridayNextRuntime } from "./src/runtime.js";
 import { resolveFridayNextConfig } from "./src/config.js";
@@ -44,7 +47,7 @@ function deviceIdFromToolContext(ctx: PluginHookToolContext): string | null {
   const sk =
     typeof ctx.sessionKey === "string" && ctx.sessionKey.trim()
       ? ctx.sessionKey.trim()
-      : (ctx.runId ? getOpenClawAgentRunContext(ctx.runId)?.sessionKey?.trim() : undefined) ?? "";
+      : ((ctx.runId ? getOpenClawAgentRunContext(ctx.runId)?.sessionKey?.trim() : undefined) ?? "");
   if (sk) {
     const d = resolveFridayDeviceIdForSessionKey(sk);
     if (d) return d;
@@ -83,7 +86,7 @@ export default defineChannelPluginEntry({
   id: "friday-next",
   name: "Friday Next",
   description: "Friday Next Apple 应用通道",
-  plugin: fridayNextChannelPlugin as ChannelPlugin,
+  plugin: fridayNextChannelPlugin,
   setRuntime: setFridayNextRuntime,
   registerFull: (api: OpenClawPluginApi) => {
     setFridayAgentForwardRuntime(api);
@@ -93,7 +96,9 @@ export default defineChannelPluginEntry({
       lastApiRoutesRegistered = new WeakRef(api);
       registerFridayNextHttpRoutes(api);
     } else {
-      const cfg = resolveFridayNextConfig(getHostOpenClawConfigSnapshot(getFridayNextRuntime().config));
+      const cfg = resolveFridayNextConfig(
+        getHostOpenClawConfigSnapshot(getFridayNextRuntime().config),
+      );
       sseEmitter.setBacklogLimit(cfg.sseBacklogPerDevice);
     }
 
@@ -147,36 +152,39 @@ export default defineChannelPluginEntry({
       };
     });
 
-    api.on("before_tool_call", (event: PluginHookBeforeToolCallEvent, ctx: PluginHookToolContext) => {
-      if (!shouldForwardToolEventToFriday(ctx)) return;
-      const deviceId = deviceIdFromToolContext(ctx);
-      const runId = ctx.runId ?? "(unknown)";
-
-      const logLine = (detail: string) => {
-        hookLogger.debug(
-          `[TOOL_CALL] toolName=${event.toolName} runId=${runId} deviceId=${deviceId ?? "(unknown)"} detail=${detail}`,
-        );
-      };
-
-      if (!deviceId) {
-        logLine("SKIP_no_deviceId");
-        return;
-      }
-
-      logLine("START");
-      sseEmitter.broadcastToolEvent(deviceId.toUpperCase(), runId, {
-        type: "tool-hook",
-        data: {
-          when: "before",
-          runId,
-          deviceId: deviceId.toUpperCase(),
-          sessionKey: ctx.sessionKey,
-          toolName: event.toolName,
-          params: event.params,
-          ts: Date.now(),
-        },
-      });
-    });
+    api.on(
+      "before_tool_call",
+      (event: PluginHookBeforeToolCallEvent, ctx: PluginHookToolContext) => {
+        if (!shouldForwardToolEventToFriday(ctx)) return;
+        const deviceId = deviceIdFromToolContext(ctx);
+        const runId = ctx.runId ?? "(unknown)";
+
+        const logLine = (detail: string) => {
+          hookLogger.debug(
+            `[TOOL_CALL] toolName=${event.toolName} runId=${runId} deviceId=${deviceId ?? "(unknown)"} detail=${detail}`,
+          );
+        };
+
+        if (!deviceId) {
+          logLine("SKIP_no_deviceId");
+          return;
+        }
+
+        logLine("START");
+        sseEmitter.broadcastToolEvent(deviceId.toUpperCase(), runId, {
+          type: "tool-hook",
+          data: {
+            when: "before",
+            runId,
+            deviceId: deviceId.toUpperCase(),
+            sessionKey: ctx.sessionKey,
+            toolName: event.toolName,
+            params: event.params,
+            ts: Date.now(),
+          },
+        });
+      },
+    );
 
     api.on("after_tool_call", (event: PluginHookAfterToolCallEvent, ctx: PluginHookToolContext) => {
       if (!shouldForwardToolEventToFriday(ctx)) return;

package/install.js

@@ -14,11 +14,7 @@ function realHome() {
     const h = execSync(`sh -c 'echo ~${sudoUser}'`, { encoding: "utf8" }).trim();
     if (h && !h.startsWith("~") && existsSync(h)) return h;
   } catch {}
-  for (const g of [
-    `/home/${sudoUser}`,
-    `/Users/${sudoUser}`,
-    `C:\\Users\\${sudoUser}`,
-  ]) {
+  for (const g of [`/home/${sudoUser}`, `/Users/${sudoUser}`, `C:\\Users\\${sudoUser}`]) {
     if (existsSync(g)) return g;
   }
   return current;
@@ -30,13 +26,23 @@ const OPENCLAW_CONFIG = join(USER_HOME, ".openclaw", "openclaw.json");
 const G = (s) => `\x1b[32m${s}\x1b[0m`;
 const Y = (s) => `\x1b[33m${s}\x1b[0m`;
 const R = (s) => `\x1b[31m${s}\x1b[0m`;
-function log(msg) { console.log(`  ${msg}`); }
-function warn(msg) { console.log(`  ${Y("!")} ${msg}`); }
-function err(msg) { console.error(`  ${R("X")} ${msg}`); }
+function log(msg) {
+  console.log(`  ${msg}`);
+}
+function warn(msg) {
+  console.log(`  ${Y("!")} ${msg}`);
+}
+function err(msg) {
+  console.error(`  ${R("X")} ${msg}`);
+}
 
 function has(cmd) {
-  try { execSync(`${cmd} --version`, { stdio: "ignore" }); return true; }
-  catch { return false; }
+  try {
+    execSync(`${cmd} --version`, { stdio: "ignore" });
+    return true;
+  } catch {
+    return false;
+  }
 }
 
 let openclawCmd = "openclaw";
@@ -79,7 +85,10 @@ if (!hasOpenclaw()) {
       let tooOld = false;
       for (let i = 0; i < 3; i++) {
         if (cur[i] > MIN_OPENCLAW[i]) break;
-        if (cur[i] < MIN_OPENCLAW[i]) { tooOld = true; break; }
+        if (cur[i] < MIN_OPENCLAW[i]) {
+          tooOld = true;
+          break;
+        }
       }
       if (tooOld) {
         err(`OpenClaw version ${m[0]} is too old.`);
@@ -100,7 +109,7 @@ log("Installing Friday Next channel plugin...");
 try {
   const out = execSync(
     `${openclawCmd} plugins install @syengup/friday-channel-next@latest --force`,
-    { encoding: "utf8", stdio: "pipe", timeout: 120000 }
+    { encoding: "utf8", stdio: "pipe", timeout: 120000 },
   );
   if (out.trim()) console.log(out.trim());
   log("Plugin registered with install record — auto-upgrade enabled.");
@@ -108,8 +117,12 @@ try {
   // Remove old manual install to avoid "duplicate plugin id" warning.
   const legacyDir = join(USER_HOME, ".openclaw", "extensions", "friday-channel-next");
   if (existsSync(legacyDir)) {
-    try { rmSync(legacyDir, { recursive: true, force: true }); log("Removed legacy manual install."); }
-    catch { /* non-critical */ }
+    try {
+      rmSync(legacyDir, { recursive: true, force: true });
+      log("Removed legacy manual install.");
+    } catch {
+      /* non-critical */
+    }
   }
 } catch (e) {
   const msg = (e.stderr || e.stdout || e.message || "").toString();
@@ -150,7 +163,10 @@ function setConfig(path, value) {
 }
 
 function ensureArrayContains(arr, item) {
-  if (!arr.includes(item)) { arr.push(item); configChanged = true; }
+  if (!arr.includes(item)) {
+    arr.push(item);
+    configChanged = true;
+  }
 }
 
 // Plugins
@@ -161,30 +177,58 @@ ensureArrayContains(config.plugins.allow, "canvas");
 
 if (!config.plugins.entries) config.plugins.entries = {};
 for (const id of ["friday-next", "canvas"]) {
-  if (!config.plugins.entries[id]) { config.plugins.entries[id] = { enabled: true }; configChanged = true; }
-  else if (!config.plugins.entries[id].enabled) { config.plugins.entries[id].enabled = true; configChanged = true; }
+  if (!config.plugins.entries[id]) {
+    config.plugins.entries[id] = { enabled: true };
+    configChanged = true;
+  } else if (!config.plugins.entries[id].enabled) {
+    config.plugins.entries[id].enabled = true;
+    configChanged = true;
+  }
 }
 
 // llm_output hook requires allowConversationAccess for non-bundled plugins.
-if (!config.plugins.entries["friday-next"].hooks) { config.plugins.entries["friday-next"].hooks = {}; configChanged = true; }
-if (!config.plugins.entries["friday-next"].hooks.allowConversationAccess) { config.plugins.entries["friday-next"].hooks.allowConversationAccess = true; configChanged = true; }
+if (!config.plugins.entries["friday-next"].hooks) {
+  config.plugins.entries["friday-next"].hooks = {};
+  configChanged = true;
+}
+if (!config.plugins.entries["friday-next"].hooks.allowConversationAccess) {
+  config.plugins.entries["friday-next"].hooks.allowConversationAccess = true;
+  configChanged = true;
+}
 
 // Channel
 if (!config.channels) config.channels = {};
-if (!config.channels["friday-next"]) { config.channels["friday-next"] = { enabled: true, transport: "http+sse" }; configChanged = true; }
-else {
-  if (!config.channels["friday-next"].enabled) { config.channels["friday-next"].enabled = true; configChanged = true; }
-  if (!config.channels["friday-next"].transport) { config.channels["friday-next"].transport = "http+sse"; configChanged = true; }
+if (!config.channels["friday-next"]) {
+  config.channels["friday-next"] = { enabled: true, transport: "http+sse" };
+  configChanged = true;
+} else {
+  if (!config.channels["friday-next"].enabled) {
+    config.channels["friday-next"].enabled = true;
+    configChanged = true;
+  }
+  if (!config.channels["friday-next"].transport) {
+    config.channels["friday-next"].transport = "http+sse";
+    configChanged = true;
+  }
 }
 
 // Gateway bind + nodes
 if (!config.gateway) config.gateway = {};
-if (config.gateway.bind !== "lan") { config.gateway.bind = "lan"; configChanged = true; }
+if (config.gateway.bind !== "lan") {
+  config.gateway.bind = "lan";
+  configChanged = true;
+}
 if (!config.gateway.nodes) config.gateway.nodes = {};
 if (!Array.isArray(config.gateway.nodes.allowCommands)) config.gateway.nodes.allowCommands = [];
 for (const cmd of [
-  "canvas.navigate", "canvas.present", "canvas.hide", "canvas.eval",
-  "canvas.snapshot", "canvas.a2ui.push", "canvas.a2ui.reset", "canvas.a2ui.pushJSONL",
+  "canvas.navigate",
+  "canvas.present",
+  "canvas.hide",
+  "canvas.eval",
+  "canvas.snapshot",
+  "canvas.a2ui.push",
+  "canvas.a2ui.reset",
+  "canvas.a2ui.pushJSONL",
 ]) {
   ensureArrayContains(config.gateway.nodes.allowCommands, cmd);
 }
@@ -193,7 +237,11 @@ for (const cmd of [
 if (!config.agents) config.agents = {};
 if (!Array.isArray(config.agents.list)) config.agents.list = [];
 let mainAgent = config.agents.list.find((a) => a.id === "main");
-if (!mainAgent) { mainAgent = { id: "main" }; config.agents.list.push(mainAgent); configChanged = true; }
+if (!mainAgent) {
+  mainAgent = { id: "main" };
+  config.agents.list.push(mainAgent);
+  configChanged = true;
+}
 if (!mainAgent.tools) mainAgent.tools = {};
 if (!Array.isArray(mainAgent.tools.alsoAllow)) mainAgent.tools.alsoAllow = [];
 for (const tool of ["canvas", "nodes"]) {
@@ -202,7 +250,10 @@ for (const tool of ["canvas", "nodes"]) {
 if (Array.isArray(mainAgent.tools.deny)) {
   for (const tool of ["canvas", "nodes"]) {
     const idx = mainAgent.tools.deny.indexOf(tool);
-    if (idx !== -1) { mainAgent.tools.deny.splice(idx, 1); configChanged = true; }
+    if (idx !== -1) {
+      mainAgent.tools.deny.splice(idx, 1);
+      configChanged = true;
+    }
   }
 }
 
@@ -224,7 +275,11 @@ log("Restarting OpenClaw gateway... (this can take 20-30s)");
 try {
   // A full gateway restart commonly takes 20s+ on a fresh boot; give it plenty of room
   // so we don't kill it mid-restart and report a false failure.
-  const out = execSync(`${openclawCmd} gateway restart`, { encoding: "utf8", stdio: "pipe", timeout: 90000 });
+  const out = execSync(`${openclawCmd} gateway restart`, {
+    encoding: "utf8",
+    stdio: "pipe",
+    timeout: 90000,
+  });
   if (out.trim()) console.log(out.trim());
 } catch (e) {
   if (e.stdout?.trim()) console.log(e.stdout.trim());
@@ -250,15 +305,18 @@ function getLanIp() {
   return "127.0.0.1";
 }
 
-try { config = JSON.parse(readFileSync(OPENCLAW_CONFIG, "utf8")); } catch { config = {}; }
+try {
+  config = JSON.parse(readFileSync(OPENCLAW_CONFIG, "utf8"));
+} catch {
+  config = {};
+}
 
 const gatewayPort = config.gateway?.port || 18789;
 const gatewayToken = config.gateway?.auth?.token || "(not set)";
 const bindMode = config.gateway?.bind || "localhost";
 
-const gatewayUrl = bindMode === "lan"
-  ? `http://${getLanIp()}:${gatewayPort}`
-  : `http://127.0.0.1:${gatewayPort}`;
+const gatewayUrl =
+  bindMode === "lan" ? `http://${getLanIp()}:${gatewayPort}` : `http://127.0.0.1:${gatewayPort}`;
 
 // Always verify against loopback: the gateway binds 0.0.0.0 so it's reachable here,
 // and this avoids false negatives from LAN/NAT routing of the advertised IP.
@@ -272,19 +330,38 @@ async function verifyGateway(url, token, retries = 30) {
     try {
       const res = await new Promise((resolve, reject) => {
         const req = http.request(
-          { hostname, port, path: "/friday-next/status", method: "GET",
-            headers: { authorization: `Bearer ${token}` }, timeout: 5000 },
-          (res) => { let body = ""; res.on("data", (c) => body += c); res.on("end", () => resolve({ status: res.statusCode, body })); },
+          {
+            hostname,
+            port,
+            path: "/friday-next/status",
+            method: "GET",
+            headers: { authorization: `Bearer ${token}` },
+            timeout: 5000,
+          },
+          (res) => {
+            let body = "";
+            res.on("data", (c) => (body += c));
+            res.on("end", () => resolve({ status: res.statusCode, body }));
+          },
         );
         req.on("error", reject);
-        req.on("timeout", () => { req.destroy(); reject(new Error("timeout")); });
+        req.on("timeout", () => {
+          req.destroy();
+          reject(new Error("timeout"));
+        });
         req.end();
       });
       if (res.status === 200) {
         try {
           const data = JSON.parse(res.body);
           if (data.ok) {
-            log("Gateway verified OK (friday-next " + data.version + ", " + data.connections + " connections).");
+            log(
+              "Gateway verified OK (friday-next " +
+                data.version +
+                ", " +
+                data.connections +
+                " connections).",
+            );
             return true;
           }
           warn("Plugin responded but ok=false — " + JSON.stringify(data));
@@ -294,8 +371,14 @@ async function verifyGateway(url, token, retries = 30) {
           continue;
         }
       }
-      if (res.status === 401) { warn("Auth token mismatch — check gateway.auth.token."); return false; }
-      if (res.status === 404) { warn("Route not found — plugin may not be loaded."); return false; }
+      if (res.status === 401) {
+        warn("Auth token mismatch — check gateway.auth.token.");
+        return false;
+      }
+      if (res.status === 404) {
+        warn("Route not found — plugin may not be loaded.");
+        return false;
+      }
       if (i < retries) warn(`Gateway responded ${res.status}, retrying (${i}/${retries})...`);
     } catch {
       if (i < retries) warn(`Gateway not reachable, retrying (${i}/${retries})...`);
@@ -404,14 +487,19 @@ async function detectPublicIp() {
       const ipStr = await new Promise((resolve, reject) => {
         const req = http.get(url, { timeout: 3000 }, (res) => {
           let body = "";
-          res.on("data", (c) => body += c);
+          res.on("data", (c) => (body += c));
           res.on("end", () => resolve(body.trim()));
         });
         req.on("error", reject);
-        req.on("timeout", () => { req.destroy(); reject(new Error("timeout")); });
+        req.on("timeout", () => {
+          req.destroy();
+          reject(new Error("timeout"));
+        });
       });
       if (/^\d{1,3}(\.\d{1,3}){3}$/.test(ipStr)) return ipStr;
-    } catch { /* try next */ }
+    } catch {
+      /* try next */
+    }
   }
   return null;
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@syengup/friday-channel-next",
-  "version": "0.1.30",
+  "version": "0.1.37",
   "description": "OpenClaw Friday Next Apple channel plugin",
   "license": "MIT",
   "type": "module",
@@ -14,6 +14,10 @@
   ],
   "scripts": {
     "build": "tsc -p tsconfig.json",
+    "lint": "eslint .",
+    "lint:fix": "eslint . --fix",
+    "format": "prettier --write .",
+    "format:check": "prettier --check .",
     "prepublishOnly": "pnpm build && rm -rf dist/attachments",
     "test": "npm run test:unit && npm run test:e2e",
     "test:unit": "vitest run",
@@ -58,12 +62,17 @@
     "qrcode-terminal": "^0.12.0"
   },
   "devDependencies": {
+    "@eslint/js": "^10.0.1",
     "@types/node": "^25.6.0",
     "chalk": "^5.6.2",
+    "eslint": "^10.5.0",
+    "eslint-config-prettier": "^10.1.8",
     "jiti": "^2.6.1",
     "json5": "^2.2.3",
+    "prettier": "^3.8.4",
     "tslog": "^4.10.2",
     "typescript": "^6.0.3",
+    "typescript-eslint": "^8.61.1",
     "vitest": "^4.1.5",
     "zod": "^4.3.6"
   }

package/src/agent/abort-run.ts

@@ -1,10 +1,25 @@
-export async function abortRun(runId: string): Promise<void> {
-  if (process.env.VITEST !== "true") {
-    try {
-      const { abortAgentHarnessRun } = await import("openclaw/plugin-sdk/agent-harness");
-      abortAgentHarnessRun(runId);
-    } catch {
-      // optional at runtime
-    }
+export type AbortRunResult = { aborted: boolean; drained: boolean };
+
+/**
+ * Abort the active run for a channel `sessionKey`.
+ *
+ * A session has at most one active run at a time, and the SDK keys active runs by
+ * their internal `sessionId` (not the channel runId). So resolve sessionKey → sessionId
+ * first, then abort-and-drain so the caller learns whether the run actually settled.
+ */
+export async function abortRunForSessionKey(sessionKey: string): Promise<AbortRunResult> {
+  if (process.env.VITEST === "true") return { aborted: false, drained: false };
+  const key = sessionKey.trim();
+  if (!key) return { aborted: false, drained: false };
+  try {
+    const { resolveActiveEmbeddedRunSessionId, abortAndDrainAgentHarnessRun } =
+      await import("openclaw/plugin-sdk/agent-harness");
+    const sessionId = resolveActiveEmbeddedRunSessionId(key);
+    if (!sessionId) return { aborted: false, drained: false };
+    const result = await abortAndDrainAgentHarnessRun({ sessionId, sessionKey: key });
+    return { aborted: result.aborted, drained: result.drained };
+  } catch {
+    // optional at runtime
+    return { aborted: false, drained: false };
   }
 }

package/src/agent/dispatch-bridge.ts

@@ -1,4 +1,5 @@
-type DispatchFn = (args: unknown) => Promise<unknown> | unknown;
+// Returns a value or a thenable; `unknown` covers both (callers `await` the result).
+type DispatchFn = (args: unknown) => unknown;
 
 let overrideDispatch: DispatchFn | null = null;
 

package/src/agent/media-bridge.test.ts

@@ -0,0 +1,71 @@
+import { describe, it, expect, vi, beforeEach } from "vitest";
+
+const saveMediaBuffer = vi.fn();
+vi.mock("openclaw/plugin-sdk/media-store", () => ({
+  saveMediaBuffer: (...args: unknown[]) => saveMediaBuffer(...args),
+}));
+
+const IMAGE_MAX = 6 * 1024 * 1024;
+const DOCUMENT_MAX = 100 * 1024 * 1024;
+vi.mock("openclaw/plugin-sdk/media-runtime", () => ({
+  mediaKindFromMime: (mime?: string) => (mime?.startsWith("image/") ? "image" : "document"),
+  maxBytesForKind: (kind: string) => (kind === "image" ? IMAGE_MAX : DOCUMENT_MAX),
+}));
+
+import { saveInboundMediaBuffer } from "./media-bridge.js";
+
+describe("saveInboundMediaBuffer", () => {
+  beforeEach(() => {
+    saveMediaBuffer.mockReset();
+  });
+
+  it("forwards the original filename so core preserves name+extension", async () => {
+    // Without the filename, core's media-store saves inbound media as a bare uuid
+    // (no extension) and the agent sees `[media attached: file://.../inbound/<uuid>]`
+    // with zero file-format signal. Passing originalFilename (5th arg) restores it.
+    saveMediaBuffer.mockResolvedValue({
+      id: "report---uuid.pdf",
+      path: "/m/inbound/report---uuid.pdf",
+    });
+
+    const out = await saveInboundMediaBuffer(Buffer.from("x"), "application/pdf", "report.pdf");
+
+    expect(saveMediaBuffer).toHaveBeenCalledWith(
+      expect.any(Buffer),
+      "application/pdf",
+      "inbound",
+      DOCUMENT_MAX,
+      "report.pdf",
+    );
+    expect(out.path).toContain(".pdf");
+  });
+
+  it("uses openclaw's per-kind byte cap instead of the 5MB save default", async () => {
+    saveMediaBuffer.mockResolvedValue({ id: "uuid.png", path: "/m/inbound/uuid.png" });
+
+    await saveInboundMediaBuffer(Buffer.from("x"), "image/png", "photo.png");
+
+    expect(saveMediaBuffer).toHaveBeenCalledWith(
+      expect.any(Buffer),
+      "image/png",
+      "inbound",
+      IMAGE_MAX,
+      "photo.png",
+    );
+  });
+
+  it("works without a filename (still applies the per-kind cap)", async () => {
+    saveMediaBuffer.mockResolvedValue({ id: "uuid", path: "/m/inbound/uuid" });
+
+    const out = await saveInboundMediaBuffer(Buffer.from("x"), "image/png");
+
+    expect(saveMediaBuffer).toHaveBeenCalledWith(
+      expect.any(Buffer),
+      "image/png",
+      "inbound",
+      IMAGE_MAX,
+      undefined,
+    );
+    expect(out.id).toBe("uuid");
+  });
+});

package/src/agent/media-bridge.ts

@@ -3,13 +3,42 @@ import os from "node:os";
 import path from "node:path";
 import crypto from "node:crypto";
 
+/**
+ * openclaw's own per-kind byte cap for a mime type (image 6MB / audio·video 16MB /
+ * document 100MB). Unknown mimes fall back to the most permissive ("document") cap.
+ * Returns undefined if the media runtime isn't importable (tests / stripped runtime),
+ * letting `saveMediaBuffer` apply its built-in default.
+ */
+export async function resolveMediaMaxBytes(mimeType: string): Promise<number | undefined> {
+  try {
+    const { maxBytesForKind, mediaKindFromMime } =
+      await import("openclaw/plugin-sdk/media-runtime");
+    return maxBytesForKind(mediaKindFromMime(mimeType) ?? "document");
+  } catch {
+    return undefined;
+  }
+}
+
 export async function saveInboundMediaBuffer(
   buffer: Buffer,
   mimeType: string,
+  originalFilename?: string,
 ): Promise<{ id: string; path: string }> {
   try {
     const sdk = await import("openclaw/plugin-sdk/media-store");
-    const saved = await sdk.saveMediaBuffer(buffer, mimeType, "inbound");
+    // Accept whatever openclaw itself supports for this media kind instead of
+    // saveMediaBuffer's conservative 5MB default.
+    const maxBytes = await resolveMediaMaxBytes(mimeType);
+    // Pass the original filename (5th arg) so core's media-store preserves the
+    // name+extension instead of saving a bare uuid. Otherwise the agent receives
+    // `[media attached: file://.../inbound/<uuid>]` with no file-format signal.
+    const saved = await sdk.saveMediaBuffer(
+      buffer,
+      mimeType,
+      "inbound",
+      maxBytes,
+      originalFilename,
+    );
     if (saved?.id && saved?.path) return { id: saved.id, path: saved.path };
   } catch {
     // fallback for tests or stripped runtime