diff --git a/CHANGELOG.md b/CHANGELOG.md index 18f8aa8301d..47fda0a2858 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -26,6 +26,7 @@ Docs: https://docs.openclaw.ai ### Fixes +- Slack/local file upload allowlist parity: propagate `mediaLocalRoots` through the Slack send action pipeline so workspace-rooted attachments pass `assertLocalMediaAllowed` checks while non-allowlisted paths remain blocked. (synthesis: #36656; overlap considered from #36516, #36496, #36493, #36484, #32648, #30888) Thanks @2233admin. - Agents/compaction safeguard pre-check: skip embedded compaction before entering the Pi SDK when a session has no real conversation messages, avoiding unnecessary LLM API calls on idle sessions. (#36451) thanks @Sid-Qin. - Config/schema cache key stability: build merged schema cache keys with incremental hashing to avoid large single-string serialization and prevent `RangeError: Invalid string length` on high-cardinality plugin/channel metadata. (#36603) Thanks @powermaster888. - iMessage/cron completion announces: strip leaked inline reply tags (for example `[[reply_to:6100]]`) from user-visible completion text so announcement deliveries do not expose threading metadata. (#24600) Thanks @vincentkoc. diff --git a/src/agents/tools/slack-actions.ts b/src/agents/tools/slack-actions.ts index 20a491c350d..1cb233f06a7 100644 --- a/src/agents/tools/slack-actions.ts +++ b/src/agents/tools/slack-actions.ts @@ -50,6 +50,8 @@ export type SlackActionContext = { replyToMode?: "off" | "first" | "all"; /** Mutable ref to track if a reply was sent (for "first" mode). */ hasRepliedRef?: { value: boolean }; + /** Allowed local media directories for file uploads. */ + mediaLocalRoots?: readonly string[]; }; /** @@ -209,6 +211,7 @@ export async function handleSlackAction( const result = await sendSlackMessage(to, content ?? "", { ...writeOpts, mediaUrl: mediaUrl ?? undefined, + mediaLocalRoots: context?.mediaLocalRoots, threadTs: threadTs ?? undefined, blocks, }); diff --git a/src/channels/plugins/slack.actions.ts b/src/channels/plugins/slack.actions.ts index abd4e75d45c..e30e57c9d05 100644 --- a/src/channels/plugins/slack.actions.ts +++ b/src/channels/plugins/slack.actions.ts @@ -15,7 +15,10 @@ export function createSlackActions(providerId: string): ChannelMessageActionAdap normalizeChannelId: resolveSlackChannelId, includeReadThreadId: true, invoke: async (action, cfg, toolContext) => - await handleSlackAction(action, cfg, toolContext as SlackActionContext | undefined), + await handleSlackAction(action, cfg, { + ...(toolContext as SlackActionContext | undefined), + mediaLocalRoots: ctx.mediaLocalRoots, + }), }); }, }; diff --git a/src/slack/actions.ts b/src/slack/actions.ts index d2e57959b0e..2ae36e6b0d4 100644 --- a/src/slack/actions.ts +++ b/src/slack/actions.ts @@ -159,6 +159,7 @@ export async function sendSlackMessage( content: string, opts: SlackActionClientOpts & { mediaUrl?: string; + mediaLocalRoots?: readonly string[]; threadTs?: string; blocks?: (Block | KnownBlock)[]; } = {}, @@ -167,6 +168,7 @@ export async function sendSlackMessage( accountId: opts.accountId, token: opts.token, mediaUrl: opts.mediaUrl, + mediaLocalRoots: opts.mediaLocalRoots, client: opts.client, threadTs: opts.threadTs, blocks: opts.blocks,