perf(slack): reduce message hot-path overhead · openclaw/...

Recent Commits to openclaw:main

vincentkoc · 2026-05-07 · via Recent Commits to openclaw:main

Original file line number	Diff line number	Diff line change
`@@ -109,7 +109,13 @@ const UNICODE_TO_SLACK: Record<string, string> = {`
`109`	`109`	`};`
`110`	`110`
`111`	`111`	`function toSlackEmojiName(emoji: string): string {`
`112`		`-const trimmed = emoji.trim().replace(/^:+\|:+$/g, "");`
	`112`	`+let trimmed = emoji.trim();`
	`113`	`+while (trimmed.startsWith(":")) {`
	`114`	`+trimmed = trimmed.slice(1);`
	`115`	`+}`
	`116`	`+while (trimmed.endsWith(":")) {`
	`117`	`+trimmed = trimmed.slice(0, -1);`
	`118`	`+}`
`113`	`119`	`return UNICODE_TO_SLACK[trimmed] ?? trimmed;`
`114`	`120`	`}`
`115`	`121`

Original file line number	Diff line number	Diff line change
`@@ -262,29 +262,29 @@ export async function resolveSlackMessageContent(params: {`
`262`	`262`	`threadStarter: params.threadStarter,`
`263`	`263`	`});`
`264`	`264`
`265`		`-const media =`
	`265`	`+const mediaPromise =`
`266`	`266`	`ownFiles && ownFiles.length > 0`
`267`		`- ? await (async () => {`
`268`		`-const { resolveSlackMedia } = await loadSlackMediaModule();`
`269`		`-return resolveSlackMedia({`
	`267`	`+ ? loadSlackMediaModule().then(({ resolveSlackMedia }) =>`
	`268`	`+resolveSlackMedia({`
`270`	`269`	`files: ownFiles,`
`271`	`270`	`token: params.botToken,`
`272`	`271`	`maxBytes: params.mediaMaxBytes,`
`273`		`-});`
`274`		`-})()`
`275`		`- : null;`
	`272`	`+}),`
	`273`	`+)`
	`274`	`+ : Promise.resolve(null);`
`276`	`275`
`277`		`-const attachmentContent =`
	`276`	`+const attachmentContentPromise =`
`278`	`277`	`params.message.attachments && params.message.attachments.length > 0`
`279`		`- ? await (async () => {`
`280`		`-const { resolveSlackAttachmentContent } = await loadSlackMediaModule();`
`281`		`-return resolveSlackAttachmentContent({`
	`278`	`+ ? loadSlackMediaModule().then(({ resolveSlackAttachmentContent }) =>`
	`279`	`+resolveSlackAttachmentContent({`
`282`	`280`	`attachments: params.message.attachments,`
`283`	`281`	`token: params.botToken,`
`284`	`282`	`maxBytes: params.mediaMaxBytes,`
`285`		`-});`
`286`		`-})()`
`287`		`- : null;`
	`283`	`+}),`
	`284`	`+)`
	`285`	`+ : Promise.resolve(null);`
	`286`	`+`
	`287`	`+const [media, attachmentContent] = await Promise.all([mediaPromise, attachmentContentPromise]);`
`288`	`288`
`289`	`289`	`const mergedMedia = [...(media ?? []), ...(attachmentContent?.media ?? [])];`
`290`	`290`	`const effectiveDirectMedia = mergedMedia.length > 0 ? mergedMedia : null;`

Original file line number	Diff line number	Diff line change
`@@ -67,6 +67,8 @@ import { isSlackSubteamMentionForBot } from "./subteam-mentions.js";`
`67`	`67`	`import type { PreparedSlackMessage } from "./types.js";`
`68`	`68`
`69`	`69`	`const mentionRegexCache = new WeakMap<SlackMonitorContext, Map<string, RegExp[]>>();`
	`70`	`+const SLACK_ANY_MENTION_RE = /<@[^>]+>\|<!subteam\^[^>]+>/;`
	`71`	`+const SLACK_SUBTEAM_MENTION_MARKER = "<!subteam^";`
`70`	`72`
`71`	`73`	`function resolveCachedMentionRegexes(`
`72`	`74`	`ctx: SlackMonitorContext,`
`@@ -286,17 +288,20 @@ export async function prepareSlackMessage(params: {`
`286`	`288`	`return null;`
`287`	`289`	`}`
`288`	`290`	`const { senderId, allowFromLower } = authorization;`
`289`		`-const hasAnyMention = /<@[^>]+>\|<!subteam\^[^>]+>/.test(message.text ?? "");`
	`291`	`+const messageText = message.text ?? "";`
	`292`	`+const hasAnyMention = SLACK_ANY_MENTION_RE.test(messageText);`
	`293`	`+const hasSubteamMention = messageText.includes(SLACK_SUBTEAM_MENTION_MARKER);`
`290`	`294`	`const explicitlyMentioned = Boolean(`
`291`	`295`	`ctx.botUserId &&`
`292`		-(message.text?.includes(`<@${ctx.botUserId}>`) \|\|
`293`		`-(await isSlackSubteamMentionForBot({`
`294`		`-client: ctx.app.client,`
`295`		`-text: message.text,`
`296`		`-botUserId: ctx.botUserId,`
`297`		`-teamId: ctx.teamId,`
`298`		`-log: logVerbose,`
`299`		`-}))),`
	`296`	+(messageText.includes(`<@${ctx.botUserId}>`) \|\|
	`297`	`+(hasSubteamMention &&`
	`298`	`+(await isSlackSubteamMentionForBot({`
	`299`	`+client: ctx.app.client,`
	`300`	`+text: messageText,`
	`301`	`+botUserId: ctx.botUserId,`
	`302`	`+teamId: ctx.teamId,`
	`303`	`+log: logVerbose,`
	`304`	`+})))),`
`300`	`305`	`);`
`301`	`306`	`const seedTopLevelRoomThreadBySource =`
`302`	`307`	`opts.source === "app_mention" \|\| opts.wasMentioned === true \|\| explicitlyMentioned;`
`@@ -315,7 +320,7 @@ export async function prepareSlackMessage(params: {`
`315`	`320`	`opts.wasMentioned ??`
`316`	`321`	`(!isDirectMessage &&`
`317`	`322`	`matchesMentionWithExplicit({`
`318`		`-text: message.text ?? "",`
	`323`	`+text: messageText,`
`319`	`324`	`mentionRegexes,`
`320`	`325`	`explicit: {`
`321`	`326`	`hasAnyMention,`

此内容由惯性聚合(RSS阅读器)自动聚合整理，仅供阅读参考。原文来自 — 版权归原作者所有。

推荐订阅源

Recent Commits to openclaw:main