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

推荐订阅源

C
Check Point Blog
Y
Y Combinator Blog
GbyAI
GbyAI
Microsoft Azure Blog
Microsoft Azure Blog
Martin Fowler
Martin Fowler
博客园_首页
大猫的无限游戏
大猫的无限游戏
美团技术团队
S
SegmentFault 最新的问题
T
The Blog of Author Tim Ferriss
Cyber Security Advisories - MS-ISAC
Cyber Security Advisories - MS-ISAC
MyScale Blog
MyScale Blog
小众软件
小众软件
Vercel News
Vercel News
阮一峰的网络日志
阮一峰的网络日志
N
Netflix TechBlog - Medium
让小产品的独立变现更简单 - ezindie.com
让小产品的独立变现更简单 - ezindie.com
量子位
博客园 - 【当耐特】
J
Java Code Geeks
F
Fortinet All Blogs
宝玉的分享
宝玉的分享
Stack Overflow Blog
Stack Overflow 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: reduce runtime mirror and signal group regressions ·...
steipete · 2026-04-30 · via Recent Commits to openclaw:main
Original file line numberDiff line numberDiff line change

@@ -13,7 +13,7 @@ Docs: https://docs.openclaw.ai

1313

- Plugins/runtime-deps: keep bundled provider policy config loading from staging plugin runtime dependencies, so config reads no longer fail on locked-down `/var/lib/openclaw/plugin-runtime-deps` directories. Fixes #74971. Thanks @eurojojo.

1414

- Memory/runtime-deps: retain the native `node-llama-cpp` runtime only when local memory search is configured, so packaged installs can repair local embeddings without relying on unreachable global npm installs. Fixes #74777. Thanks @LLagoon3.

1515

- Gateway/startup: skip pre-bind web-fetch provider discovery for credential-free `tools.web.fetch` config, so Docker/Kubernetes gateways bind even when optional fetch limits are present. Fixes #74896. Thanks @KoykL.

16-

- Signal: match group allowlists against inbound Signal group ids as well as sender ids, so configured groups no longer drop every message before mention policy runs. Part of #53308. Thanks @minupla and @juan-flores077.

16+

- Signal: match group allowlists against inbound Signal group ids as well as sender ids, and process explicitly configured Signal groups without requiring mentions unless `requireMention` is set. Fixes #53308. Thanks @minupla and @juan-flores077.

1717

- Signal: bound `signal-cli` installer release and archive downloads with explicit timeouts, declared and streamed size checks, and partial-file cleanup. Fixes #54153. Thanks @jinduwang1001-max and @juan-flores077.

1818

- Slack: require bot-authored room messages with `allowBots=true` to come from an explicitly channel-allowlisted bot or from a room where an explicit Slack owner is present, so broad bot relays cannot run unattended. Fixes #59284. Thanks @andrewhong-translucent.

1919

- Signal: derive `getAttachment` HTTP response caps from `channels.signal.mediaMaxMb` with base64 headroom, so inbound photos and videos no longer drop behind the 1 MiB RPC default. Fixes #73564. Thanks @heyhudson.

Original file line numberDiff line numberDiff line change

@@ -194,10 +194,10 @@ DMs:

194194

Groups:

195195
196196

- `channels.signal.groupPolicy = open | allowlist | disabled`.

197-

- `channels.signal.groupAllowFrom` controls which groups or senders can trigger group replies when `allowlist` is set; entries can be Signal group IDs (raw, `group:<id>`, or `signal:group:<id>`), sender phone numbers, or `uuid:<id>` values.

197+

- `channels.signal.groupAllowFrom` controls which groups or senders can trigger group replies when `allowlist` is set; entries can be Signal group IDs (raw, `group:<id>`, or `signal:group:<id>`), sender phone numbers, `uuid:<id>` values, or `*`.

198198

- `channels.signal.groups["<group-id>" | "*"]` can override group behavior with `requireMention`, `tools`, and `toolsBySender`.

199199

- Use `channels.signal.accounts.<id>.groups` for per-account overrides in multi-account setups.

200-

- Allowlisting a Signal group does not disable mention gating. To process every message in an allowlisted group, set `channels.signal.groups["<group-id>"].requireMention=false` or use the `"*"` group default.

200+

- Allowlisting a Signal group through `groupAllowFrom` does not disable mention gating by itself. A specifically configured `channels.signal.groups["<group-id>"]` entry processes every group message unless `requireMention=true` is set.

201201

- Runtime note: if `channels.signal` is completely missing, runtime falls back to `groupPolicy="allowlist"` for group checks (even if `channels.defaults.groupPolicy` is set).

202202
203203

## How it works (behavior)

Original file line numberDiff line numberDiff line change

@@ -134,6 +134,32 @@ describe("signal mention gating", () => {

134134

expect(getCapturedCtx()?.WasMentioned).toBe(false);

135135

});

136136
137+

it("allows explicitly configured Signal groups by group id without a mention", async () => {

138+

const handler = createSignalEventHandler(

139+

createBaseSignalEventHandlerDeps({

140+

cfg: {

141+

messages: {

142+

inbound: { debounceMs: 0 },

143+

groupChat: { mentionPatterns: ["@bot"] },

144+

},

145+

channels: {

146+

signal: {

147+

groupPolicy: "allowlist",

148+

groupAllowFrom: ["group:g1"],

149+

groups: { g1: {} },

150+

},

151+

},

152+

} as unknown as OpenClawConfig,

153+

groupPolicy: "allowlist",

154+

groupAllowFrom: ["group:g1"],

155+

}),

156+

);

157+
158+

await handler(makeGroupEvent({ message: "hello everyone" }));

159+

expect(capturedCtx).toBeTruthy();

160+

expect(getCapturedCtx()?.WasMentioned).toBe(false);

161+

});

162+
137163

it("records pending history for skipped group messages", async () => {

138164

const { handler, groupHistories } = createMentionGatedHistoryHandler();

139165

await handler(makeGroupEvent({ message: "hello from alice" }));

Original file line numberDiff line numberDiff line change

@@ -694,6 +694,7 @@ export function createSignalEventHandler(deps: SignalEventHandlerDeps) {

694694

channel: "signal",

695695

groupId,

696696

accountId: deps.accountId,

697+

configuredGroupDefaultsToNoMention: true,

697698

});

698699

const canDetectMention = mentionRegexes.length > 0;

699700

const mentionDecision = resolveInboundMentionDecision({

Original file line numberDiff line numberDiff line change

@@ -1,6 +1,10 @@

11

import { afterEach, describe, expect, it, vi } from "vitest";

22

import type { OpenClawConfig } from "./config.js";

3-

import { resolveChannelGroupPolicy, resolveToolsBySender } from "./group-policy.js";

3+

import {

4+

resolveChannelGroupPolicy,

5+

resolveChannelGroupRequireMention,

6+

resolveToolsBySender,

7+

} from "./group-policy.js";

48
59

describe("resolveChannelGroupPolicy", () => {

610

it("fails closed when groupPolicy=allowlist and groups are missing", () => {

@@ -129,6 +133,34 @@ describe("resolveChannelGroupPolicy", () => {

129133

expect(policy.allowlistEnabled).toBe(true);

130134

expect(policy.allowed).toBe(false);

131135

});

136+
137+

it("can default explicitly configured groups to no mention for channels that opt in", () => {

138+

const cfg = {

139+

channels: {

140+

whatsapp: {

141+

groups: {

142+

"123@g.us": {},

143+

},

144+

},

145+

},

146+

} as OpenClawConfig;

147+
148+

expect(

149+

resolveChannelGroupRequireMention({

150+

cfg,

151+

channel: "whatsapp",

152+

groupId: "123@g.us",

153+

}),

154+

).toBe(true);

155+

expect(

156+

resolveChannelGroupRequireMention({

157+

cfg,

158+

channel: "whatsapp",

159+

groupId: "123@g.us",

160+

configuredGroupDefaultsToNoMention: true,

161+

}),

162+

).toBe(false);

163+

});

132164

});

133165
134166

describe("resolveToolsBySender", () => {

Original file line numberDiff line numberDiff line change

@@ -372,6 +372,7 @@ export function resolveChannelGroupRequireMention(params: {

372372

accountId?: string | null;

373373

groupIdCaseInsensitive?: boolean;

374374

requireMentionOverride?: boolean;

375+

configuredGroupDefaultsToNoMention?: boolean;

375376

overrideOrder?: "before-config" | "after-config";

376377

}): boolean {

377378

const { requireMentionOverride, overrideOrder = "after-config" } = params;

@@ -392,6 +393,9 @@ export function resolveChannelGroupRequireMention(params: {

392393

if (overrideOrder !== "before-config" && typeof requireMentionOverride === "boolean") {

393394

return requireMentionOverride;

394395

}

396+

if (params.configuredGroupDefaultsToNoMention && groupConfig) {

397+

return false;

398+

}

395399

return true;

396400

}

397401
Original file line numberDiff line numberDiff line change

@@ -152,7 +152,7 @@ function readBundledPluginLocalMemoryEmbeddingRuntimeDeps(value: unknown): Runti

152152

"openclaw.plugin.json runtimeDependencies.localMemoryEmbedding must contain strings",

153153

);

154154

}

155-

return { ...parseInstallableRuntimeDepSpec(spec), pluginIds: [] };

155+

return Object.assign(parseInstallableRuntimeDepSpec(spec), { pluginIds: [] });

156156

});

157157

}

158158
Original file line numberDiff line numberDiff line change

@@ -117,7 +117,6 @@ function isBundledRuntimeMirrorFileAlreadyMaterialized(

117117

sourceStat.ino === targetStat.ino

118118

);

119119

}

120-
121120

function chmodBundledRuntimeMirrorFileReadable(sourcePath: string, targetPath: string): void {

122121

try {

123122

const sourceMode = fs.statSync(sourcePath).mode;

Original file line numberDiff line numberDiff line change

@@ -56,6 +56,29 @@ describe("prepareBundledPluginRuntimeRoot", () => {

5656

expect(fs.readFileSync(target, "utf8")).toBe("export const value = 'old';\n");

5757

});

5858
59+

it("reuses existing hardlinked mirror files without rewriting them", () => {

60+

const root = makeTempRoot();

61+

const source = path.join(root, "source.js");

62+

const target = path.join(root, "mirror", "source.js");

63+

fs.writeFileSync(source, "export const value = 'stable';\n", "utf8");

64+

fs.mkdirSync(path.dirname(target), { recursive: true });

65+

fs.linkSync(source, target);

66+

const initialTargetStat = fs.statSync(target, { bigint: true });

67+
68+

const linkSpy = vi.spyOn(fs, "linkSync");

69+

const copySpy = vi.spyOn(fs, "copyFileSync");

70+

const removeSpy = vi.spyOn(fs, "rmSync");

71+
72+

materializeBundledRuntimeMirrorFile(source, target);

73+
74+

const reusedTargetStat = fs.statSync(target, { bigint: true });

75+

expect(reusedTargetStat.dev).toBe(initialTargetStat.dev);

76+

expect(reusedTargetStat.ino).toBe(initialTargetStat.ino);

77+

expect(linkSpy).not.toHaveBeenCalled();

78+

expect(copySpy).not.toHaveBeenCalled();

79+

expect(removeSpy).not.toHaveBeenCalled();

80+

});

81+
5982

it("materializes root JavaScript chunks in external mirrors", () => {

6083

const packageRoot = makeTempRoot();

6184

const stageDir = makeTempRoot();