惯性聚合 高效追踪和阅读你感兴趣的博客、新闻、科技资讯
阅读原文 在惯性聚合中打开

推荐订阅源

博客园 - 叶小钗
MyScale Blog
MyScale Blog
博客园 - 【当耐特】
I
InfoQ
腾讯CDC
aimingoo的专栏
aimingoo的专栏
L
LangChain Blog
人人都是产品经理
人人都是产品经理
D
DataBreaches.Net
freeCodeCamp Programming Tutorials: Python, JavaScript, Git & More
Engineering at Meta
Engineering at Meta
A
About on SuperTechFans
Google DeepMind News
Google DeepMind News
Vercel News
Vercel News
C
Check Point Blog
B
Blog RSS Feed
Cyber Security Advisories - MS-ISAC
Cyber Security Advisories - MS-ISAC
美团技术团队
Stack Overflow Blog
Stack Overflow Blog
Y
Y Combinator Blog
D
Docker
MongoDB | Blog
MongoDB | Blog
量子位
博客园_首页

Recent Commits to openclaw:main

test: merge chat side-result checks · openclaw/openclaw@ddd2c2a test: merge cron history checks · openclaw/openclaw@f7eb746 test: merge responsive navigation shell checks · openclaw/openclaw@c2e4b47 docs(changelog): add codex oauth fixes · openclaw/openclaw@628e6cd test: merge navigation routing cases · openclaw/openclaw@5d8cecb Tests: mock channel registry bundled fallback · openclaw/openclaw@2b08233 Secrets: avoid broad web search discovery for single plugin config · openclaw/openclaw@a464f59 test: merge config view browser checks · openclaw/openclaw@20cf511 fix(status): align oauth health with runtime · openclaw/openclaw@eed7116 feat: add macOS screen snapshots for monitor preview (#67954) thanks … · openclaw/openclaw@f377db1 fix: report shared auth scopes in hello-ok (#67810) thanks @BunsDev · openclaw/openclaw@0b6c39b Auto-reply: avoid eager bundled route fallback · openclaw/openclaw@3ea1bf4 Tests: narrow session binding contract setup · openclaw/openclaw@54e4e16 fix(macOS): enable undo/redo in webchat composer text input (#34962) · openclaw/openclaw@00951dc Tests: speed up channel setup promotion · openclaw/openclaw@82b529a Docs: refresh agent instructions · openclaw/openclaw@5775fe2 fix(auth): serialize OAuth refresh across agents to fix #26322 (#67876) · openclaw/openclaw@8e79080 test: allow ollama public surface boundary test · openclaw/openclaw@7d4f1a6 Docs: add test performance guardrails · openclaw/openclaw@89706d3 Tests: restore context-engine usage proof · openclaw/openclaw@e4c4f95 Tests: slim context engine runtime coverage · openclaw/openclaw@74c198f ci: retry failed custom checkouts · openclaw/openclaw@0ee5baf test: trim duplicate provider auth onboarding cases · openclaw/openclaw@1ffc02e matrix: fix sessions_spawn --thread subagent session spawning (#67643) · openclaw/openclaw@1ce2596 test: reduce auth choice fixture churn · openclaw/openclaw@857b9cd test: mock health status config boundaries · openclaw/openclaw@9d5ab4a test: mock onboard config io boundary · openclaw/openclaw@299694d test: mock legacy state plugin boundaries · openclaw/openclaw@2713089 test: mock channel install boundaries · openclaw/openclaw@b945248 test: mock doctor preview channel boundaries · openclaw/openclaw@b1a3ad4
fix(slack): persist inbound delivery dedupe · openclaw/op...
steipete · 2026-05-18 · via Recent Commits to openclaw:main

@@ -0,0 +1,148 @@

1+

import { resolveGlobalDedupeCache } from "openclaw/plugin-sdk/dedupe-runtime";

2+

import { getOptionalSlackRuntime } from "../runtime.js";

3+

import type { SlackMessageEvent } from "../types.js";

4+5+

const TTL_MS = 24 * 60 * 60 * 1000;

6+

const MAX_ENTRIES = 20_000;

7+

const PERSISTENT_MAX_ENTRIES = 20_000;

8+

const PERSISTENT_NAMESPACE = "slack.inbound-deliveries";

9+

const SLACK_INBOUND_DELIVERIES_KEY = Symbol.for("openclaw.slackInboundDeliveries");

10+11+

type SlackInboundDeliveryRecord = {

12+

deliveredAt: number;

13+

};

14+15+

type SlackInboundDeliveryStore = {

16+

register(

17+

key: string,

18+

value: SlackInboundDeliveryRecord,

19+

opts?: { ttlMs?: number },

20+

): Promise<void>;

21+

lookup(key: string): Promise<SlackInboundDeliveryRecord | undefined>;

22+

};

23+24+

const deliveredMessages = resolveGlobalDedupeCache(SLACK_INBOUND_DELIVERIES_KEY, {

25+

ttlMs: TTL_MS,

26+

maxSize: MAX_ENTRIES,

27+

});

28+29+

let persistentStore: SlackInboundDeliveryStore | undefined;

30+

let persistentStoreDisabled = false;

31+32+

function makeKey(accountId: string, channelId: string, ts: string): string {

33+

return `${accountId}:${channelId}:${ts}`;

34+

}

35+36+

function reportPersistentInboundDeliveryError(error: unknown): void {

37+

try {

38+

getOptionalSlackRuntime()

39+

?.logging.getChildLogger({ plugin: "slack", feature: "inbound-delivery-state" })

40+

.warn("Slack persistent inbound delivery state failed", { error: String(error) });

41+

} catch {

42+

// Best effort only: persistent state must never break Slack message handling.

43+

}

44+

}

45+46+

function disablePersistentInboundDelivery(error: unknown): void {

47+

persistentStoreDisabled = true;

48+

persistentStore = undefined;

49+

reportPersistentInboundDeliveryError(error);

50+

}

51+52+

function getPersistentInboundDeliveryStore(): SlackInboundDeliveryStore | undefined {

53+

if (persistentStoreDisabled) {

54+

return undefined;

55+

}

56+

if (persistentStore) {

57+

return persistentStore;

58+

}

59+

const runtime = getOptionalSlackRuntime();

60+

if (!runtime) {

61+

return undefined;

62+

}

63+

try {

64+

persistentStore = runtime.state.openKeyedStore<SlackInboundDeliveryRecord>({

65+

namespace: PERSISTENT_NAMESPACE,

66+

maxEntries: PERSISTENT_MAX_ENTRIES,

67+

defaultTtlMs: TTL_MS,

68+

});

69+

return persistentStore;

70+

} catch (error) {

71+

disablePersistentInboundDelivery(error);

72+

return undefined;

73+

}

74+

}

75+76+

async function lookupPersistentInboundDelivery(key: string): Promise<boolean> {

77+

const store = getPersistentInboundDeliveryStore();

78+

if (!store) {

79+

return false;

80+

}

81+

try {

82+

return Boolean(await store.lookup(key));

83+

} catch (error) {

84+

disablePersistentInboundDelivery(error);

85+

return false;

86+

}

87+

}

88+89+

async function rememberPersistentInboundDelivery(key: string, deliveredAt: number): Promise<void> {

90+

const store = getPersistentInboundDeliveryStore();

91+

if (!store) {

92+

return;

93+

}

94+

try {

95+

await store.register(key, { deliveredAt });

96+

} catch (error) {

97+

disablePersistentInboundDelivery(error);

98+

}

99+

}

100+101+

export async function hasSlackInboundMessageDelivery(params: {

102+

accountId: string;

103+

channelId: string | undefined;

104+

ts: string | undefined;

105+

}): Promise<boolean> {

106+

if (!params.accountId || !params.channelId || !params.ts) {

107+

return false;

108+

}

109+

const key = makeKey(params.accountId, params.channelId, params.ts);

110+

if (deliveredMessages.peek(key)) {

111+

return true;

112+

}

113+

const found = await lookupPersistentInboundDelivery(key);

114+

if (found) {

115+

deliveredMessages.check(key);

116+

}

117+

return found;

118+

}

119+120+

export async function recordSlackInboundMessageDeliveries(params: {

121+

accountId: string;

122+

messages: readonly SlackMessageEvent[];

123+

}): Promise<void> {

124+

if (!params.accountId || params.messages.length === 0) {

125+

return;

126+

}

127+

const deliveredAt = Date.now();

128+

const keys = new Set<string>();

129+

for (const message of params.messages) {

130+

if (!message.channel || !message.ts) {

131+

continue;

132+

}

133+

keys.add(makeKey(params.accountId, message.channel, message.ts));

134+

}

135+

if (keys.size === 0) {

136+

return;

137+

}

138+

for (const key of keys) {

139+

deliveredMessages.check(key, deliveredAt);

140+

}

141+

await Promise.all(Array.from(keys, (key) => rememberPersistentInboundDelivery(key, deliveredAt)));

142+

}

143+144+

export function clearSlackInboundDeliveryStateForTest(): void {

145+

deliveredMessages.clear();

146+

persistentStore = undefined;

147+

persistentStoreDisabled = false;

148+

}