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

推荐订阅源

IT之家
IT之家
H
Help Net Security
GbyAI
GbyAI
博客园_首页
G
Google Developers Blog
Microsoft Security Blog
Microsoft Security Blog
博客园 - 【当耐特】
月光博客
月光博客
美团技术团队
B
Blog RSS Feed
博客园 - 三生石上(FineUI控件)
WordPress大学
WordPress大学
博客园 - 叶小钗
有赞技术团队
有赞技术团队
T
The Blog of Author Tim Ferriss
Engineering at Meta
Engineering at Meta
Google DeepMind News
Google DeepMind News
Y
Y Combinator Blog
宝玉的分享
宝玉的分享
Microsoft Azure Blog
Microsoft Azure Blog
罗磊的独立博客
云风的 BLOG
云风的 BLOG
Cyber Security Advisories - MS-ISAC
Cyber Security Advisories - MS-ISAC
P
Proofpoint News Feed

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(doctor): respect runtime message tool grants · opencl...
pashpashpash · 2026-05-15 · via Recent Commits to openclaw:main

@@ -1,11 +1,16 @@

11

import { resolveAgentConfig } from "../../../agents/agent-scope-config.js";

2+

import { DEFAULT_MODEL, DEFAULT_PROVIDER } from "../../../agents/defaults.js";

3+

import { parseModelRef } from "../../../agents/model-selection-normalize.js";

4+

import { normalizeProviderId } from "../../../agents/provider-id.js";

25

import { pickSandboxToolPolicy } from "../../../agents/sandbox-tool-policy.js";

36

import { isToolAllowedByPolicies } from "../../../agents/tool-policy-match.js";

47

import { mergeAlsoAllowPolicy, resolveToolProfilePolicy } from "../../../agents/tool-policy.js";

8+

import { resolveAgentModelPrimaryValue } from "../../../config/model-input.js";

59

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

610

import type { AgentToolsConfig, ToolsConfig } from "../../../config/types.tools.js";

711

import { collectChannelRouteTargets } from "../../../routing/channel-route-targets.js";

812

import { createLazyImportLoader } from "../../../shared/lazy-promise.js";

13+

import { normalizeLowercaseStringOrEmpty } from "../../../shared/string-coerce.js";

9141015

type ChannelDoctorModule = typeof import("./channel-doctor.js");

1116

@@ -88,40 +93,171 @@ function hasConfiguredSafeBins(cfg: OpenClawConfig): boolean {

8893

}

89949095

type VisibleReplyPolicyProvenance = "default" | "global-explicit" | "group-explicit";

96+

type ToolPolicyConfig = {

97+

allow?: string[];

98+

alsoAllow?: string[];

99+

deny?: string[];

100+

profile?: string;

101+

};

102+103+

function normalizeProviderPolicyKey(value: string): string {

104+

const normalized = normalizeLowercaseStringOrEmpty(value);

105+

const slashIndex = normalized.indexOf("/");

106+

if (slashIndex <= 0) {

107+

return normalizeProviderId(normalized);

108+

}

109+

const provider = normalizeProviderId(normalized.slice(0, slashIndex));

110+

const modelId = normalized.slice(slashIndex + 1);

111+

return modelId ? `${provider}/${modelId}` : provider;

112+

}

113+114+

function isCanonicalProviderPolicyKey(value: string): boolean {

115+

return normalizeLowercaseStringOrEmpty(value) === normalizeProviderPolicyKey(value);

116+

}

117+118+

function resolveProviderToolPolicy(params: {

119+

byProvider?: Record<string, ToolPolicyConfig>;

120+

modelProvider: string;

121+

modelId: string;

122+

}): ToolPolicyConfig | undefined {

123+

if (!params.byProvider) {

124+

return undefined;

125+

}

126+

const lookup = new Map<string, { canonical: boolean; value: ToolPolicyConfig }>();

127+

for (const [key, value] of Object.entries(params.byProvider)) {

128+

const normalized = normalizeProviderPolicyKey(key);

129+

if (!normalized) {

130+

continue;

131+

}

132+

const canonical = isCanonicalProviderPolicyKey(key);

133+

const existing = lookup.get(normalized);

134+

if (!existing || (canonical && !existing.canonical)) {

135+

lookup.set(normalized, { canonical, value });

136+

}

137+

}

138+139+

const provider = normalizeProviderPolicyKey(params.modelProvider);

140+

const modelId = normalizeLowercaseStringOrEmpty(params.modelId);

141+

const fullModelId = modelId ? `${provider}/${modelId}` : undefined;

142+

return (fullModelId ? lookup.get(fullModelId)?.value : undefined) ?? lookup.get(provider)?.value;

143+

}

9114492145

function resolveMessageToolAvailability(params: {

146+

cfg: OpenClawConfig;

147+

agentId?: string;

93148

globalTools?: ToolsConfig;

94149

agentTools?: AgentToolsConfig;

150+

runtimeAlsoAllow?: string[];

95151

}): boolean {

152+

const agentConfig = params.agentId ? resolveAgentConfig(params.cfg, params.agentId) : undefined;

153+

const modelRef = resolvePrimaryModelRef(params.cfg, agentConfig?.model);

154+

const providerPolicy = resolveProviderToolPolicy({

155+

byProvider: params.globalTools?.byProvider,

156+

modelProvider: modelRef.provider,

157+

modelId: modelRef.model,

158+

});

159+

const agentProviderPolicy = resolveProviderToolPolicy({

160+

byProvider: params.agentTools?.byProvider,

161+

modelProvider: modelRef.provider,

162+

modelId: modelRef.model,

163+

});

96164

const profile = params.agentTools?.profile ?? params.globalTools?.profile;

97-

const profileAlsoAllow = Array.isArray(params.agentTools?.alsoAllow)

165+

const configuredAlsoAllow = Array.isArray(params.agentTools?.alsoAllow)

98166

? params.agentTools.alsoAllow

99167

: Array.isArray(params.globalTools?.alsoAllow)

100168

? params.globalTools.alsoAllow

101-

: undefined;

169+

: [];

170+

const providerAlsoAllow = Array.isArray(agentProviderPolicy?.alsoAllow)

171+

? agentProviderPolicy.alsoAllow

172+

: Array.isArray(providerPolicy?.alsoAllow)

173+

? providerPolicy.alsoAllow

174+

: [];

175+

const profileAlsoAllow = [...configuredAlsoAllow, ...(params.runtimeAlsoAllow ?? [])];

176+

const providerProfileAlsoAllow = [...providerAlsoAllow, ...(params.runtimeAlsoAllow ?? [])];

102177

const profilePolicy = mergeAlsoAllowPolicy(resolveToolProfilePolicy(profile), profileAlsoAllow);

178+

const providerProfilePolicy = mergeAlsoAllowPolicy(

179+

resolveToolProfilePolicy(agentProviderPolicy?.profile ?? providerPolicy?.profile),

180+

providerProfileAlsoAllow,

181+

);

103182

return isToolAllowedByPolicies("message", [

104183

profilePolicy,

184+

providerProfilePolicy,

185+

pickSandboxToolPolicy(providerPolicy),

186+

pickSandboxToolPolicy(agentProviderPolicy),

105187

pickSandboxToolPolicy(params.globalTools),

106188

pickSandboxToolPolicy(params.agentTools),

107189

]);

108190

}

109191110-

function collectMessageToolUnavailableTargets(cfg: OpenClawConfig): string[] {

192+

const SOURCE_REPLY_RUNTIME_MESSAGE_ALLOW = ["message"];

193+194+

function resolvePrimaryModelRef(

195+

cfg: OpenClawConfig,

196+

agentModel?: NonNullable<ReturnType<typeof resolveAgentConfig>>["model"],

197+

): { provider: string; model: string } {

198+

const raw =

199+

resolveAgentModelPrimaryValue(agentModel) ??

200+

resolveAgentModelPrimaryValue(cfg.agents?.defaults?.model) ??

201+

DEFAULT_MODEL;

202+

return (

203+

parseModelRef(raw, DEFAULT_PROVIDER, { allowPluginNormalization: false }) ?? {

204+

provider: DEFAULT_PROVIDER,

205+

model: DEFAULT_MODEL,

206+

}

207+

);

208+

}

209+210+

function resolveSourceReplyMessageToolAvailability(params: {

211+

cfg: OpenClawConfig;

212+

agentId?: string;

213+

globalTools?: ToolsConfig;

214+

agentTools?: AgentToolsConfig;

215+

}): boolean {

216+

return resolveMessageToolAvailability({

217+

...params,

218+

runtimeAlsoAllow: SOURCE_REPLY_RUNTIME_MESSAGE_ALLOW,

219+

});

220+

}

221+222+

function sourceReplyRuntimeMayAllowMessageTool(cfg: OpenClawConfig): boolean {

223+

const groupPolicy = resolveGroupVisibleReplyProvenance(cfg);

224+

if (hasChannels(cfg) && groupPolicy.value === "message_tool") {

225+

return true;

226+

}

227+

if (cfg.messages?.visibleReplies === "message_tool") {

228+

return true;

229+

}

230+

return false;

231+

}

232+233+

function collectMessageToolUnavailableTargets(

234+

cfg: OpenClawConfig,

235+

options: { sourceReplyRuntimeGrant?: boolean } = {},

236+

): string[] {

111237

const agents = listAgentRecords(cfg);

112238

if (agents.length === 0) {

113-

return resolveMessageToolAvailability({ globalTools: cfg.tools })

114-

? []

115-

: ["default tool policy"];

239+

const available = options.sourceReplyRuntimeGrant

240+

? resolveSourceReplyMessageToolAvailability({ cfg, globalTools: cfg.tools })

241+

: resolveMessageToolAvailability({ cfg, globalTools: cfg.tools });

242+

return available ? [] : ["default tool policy"];

116243

}

117-

return agents.flatMap((agent) =>

118-

resolveMessageToolAvailability({

119-

globalTools: cfg.tools,

120-

agentTools: agent.tools as AgentToolsConfig | undefined,

121-

})

122-

? []

123-

: [`agent "${typeof agent.id === "string" ? agent.id : "unknown"}"`],

124-

);

244+

return agents.flatMap((agent) => {

245+

const agentId = typeof agent.id === "string" ? agent.id : "unknown";

246+

const available = options.sourceReplyRuntimeGrant

247+

? resolveSourceReplyMessageToolAvailability({

248+

cfg,

249+

agentId,

250+

globalTools: cfg.tools,

251+

agentTools: agent.tools as AgentToolsConfig | undefined,

252+

})

253+

: resolveMessageToolAvailability({

254+

cfg,

255+

agentId,

256+

globalTools: cfg.tools,

257+

agentTools: agent.tools as AgentToolsConfig | undefined,

258+

});

259+

return available ? [] : [`agent "${agentId}"`];

260+

});

125261

}

126262127263

function resolveGroupVisibleReplyProvenance(cfg: OpenClawConfig): {

@@ -160,16 +296,16 @@ function formatTargets(targets: string[]): string {

160296

}

161297162298

export function collectVisibleReplyToolPolicyWarnings(cfg: OpenClawConfig): string[] {

163-

const targets = collectMessageToolUnavailableTargets(cfg);

164-

if (targets.length === 0) {

165-

return [];

166-

}

167299

const groupPolicy = resolveGroupVisibleReplyProvenance(cfg);

168300

const warnings: string[] = [];

169301

if (groupPolicy.value === "message_tool") {

170302

if (groupPolicy.provenance === "default" && !hasChannels(cfg)) {

171303

return warnings;

172304

}

305+

const targets = collectMessageToolUnavailableTargets(cfg, { sourceReplyRuntimeGrant: true });

306+

if (targets.length === 0) {

307+

return warnings;

308+

}

173309

const targetSummary = formatTargets(targets);

174310

if (groupPolicy.provenance === "default") {

175311

warnings.push(

@@ -184,6 +320,10 @@ export function collectVisibleReplyToolPolicyWarnings(cfg: OpenClawConfig): stri

184320185321

const globalVisibleReplies = cfg.messages?.visibleReplies;

186322

if (globalVisibleReplies === "message_tool" && groupPolicy.path !== "messages.visibleReplies") {

323+

const targets = collectMessageToolUnavailableTargets(cfg, { sourceReplyRuntimeGrant: true });

324+

if (targets.length === 0) {

325+

return warnings;

326+

}

187327

warnings.push(

188328

`- messages.visibleReplies is set to "message_tool", but the message tool is unavailable for ${formatTargets(

189329

targets,

@@ -206,7 +346,21 @@ function formatChannelList(channels: string[]): string {

206346

export function collectChannelBoundMessageToolPolicyWarnings(cfg: OpenClawConfig): string[] {

207347

return collectChannelRouteTargets(cfg).flatMap((target) => {

208348

const agentTools = resolveAgentConfig(cfg, target.agentId)?.tools;

209-

if (resolveMessageToolAvailability({ globalTools: cfg.tools, agentTools })) {

349+

const runtimeMayAllowMessage = sourceReplyRuntimeMayAllowMessageTool(cfg);

350+

const messageToolAvailable = runtimeMayAllowMessage

351+

? resolveSourceReplyMessageToolAvailability({

352+

cfg,

353+

agentId: target.agentId,

354+

globalTools: cfg.tools,

355+

agentTools,

356+

})

357+

: resolveMessageToolAvailability({

358+

cfg,

359+

agentId: target.agentId,

360+

globalTools: cfg.tools,

361+

agentTools,

362+

});

363+

if (messageToolAvailable) {

210364

return [];

211365

}

212366

return [