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

推荐订阅源

aimingoo的专栏
aimingoo的专栏
G
Google Developers Blog
B
Blog RSS Feed
A
About on SuperTechFans
奇客Solidot–传递最新科技情报
奇客Solidot–传递最新科技情报
V
V2EX
Stack Overflow Blog
Stack Overflow Blog
C
Check Point Blog
H
Hackread – Cybersecurity News, Data Breaches, AI and More
Engineering at Meta
Engineering at Meta
Cyber Security Advisories - MS-ISAC
Cyber Security Advisories - MS-ISAC
博客园 - 司徒正美
D
Docker
F
Fortinet All Blogs
Hugging Face - Blog
Hugging Face - Blog
Last Week in AI
Last Week in AI
H
Help Net Security
WordPress大学
WordPress大学
MyScale Blog
MyScale Blog
博客园 - Franky
人人都是产品经理
人人都是产品经理
钛媒体:引领未来商业与生活新知
钛媒体:引领未来商业与生活新知
Blog — PlanetScale
Blog — PlanetScale
L
LangChain 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(agents): fail empty explicit tool allowlists · opencl...
steipete · 2026-04-25 · via Recent Commits to openclaw:main

@@ -108,6 +108,11 @@ import {

108108

toClientToolDefinitions,

109109

} from "../../pi-tool-definition-adapter.js";

110110

import { createOpenClawCodingTools, resolveToolLoopDetectionConfig } from "../../pi-tools.js";

111+

import {

112+

resolveEffectiveToolPolicy,

113+

resolveGroupToolPolicy,

114+

resolveSubagentToolPolicyForSession,

115+

} from "../../pi-tools.policy.js";

111116

import { wrapStreamFnTextTransforms } from "../../plugin-text-transforms.js";

112117

import { describeProviderRequestRoutingSummary } from "../../provider-attribution.js";

113118

import { registerProviderStreamForModel } from "../../provider-stream.js";

@@ -126,10 +131,18 @@ import {

126131

applySkillEnvOverridesFromSnapshot,

127132

resolveSkillsPromptForRun,

128133

} from "../../skills.js";

134+

import {

135+

isSubagentEnvelopeSession,

136+

resolveSubagentCapabilityStore,

137+

} from "../../subagent-capabilities.js";

129138

import { resolveSystemPromptOverride } from "../../system-prompt-override.js";

130139

import { buildSystemPromptParams } from "../../system-prompt-params.js";

131140

import { buildSystemPromptReport } from "../../system-prompt-report.js";

132141

import { resolveAgentTimeoutMs } from "../../timeout.js";

142+

import {

143+

buildEmptyExplicitToolAllowlistError,

144+

collectExplicitToolAllowlistSources,

145+

} from "../../tool-allowlist-guard.js";

133146

import { UNKNOWN_TOOL_THRESHOLD } from "../../tool-loop-detection.js";

134147

import {

135148

resolveTranscriptPolicy,

@@ -451,6 +464,79 @@ export function applyEmbeddedAttemptToolsAllow<T extends { name: string }>(

451464

return tools.filter((tool) => allowSet.has(tool.name));

452465

}

453466467+

function collectAttemptExplicitToolAllowlistSources(params: {

468+

config?: EmbeddedRunAttemptParams["config"];

469+

sessionKey?: string;

470+

sandboxSessionKey?: string;

471+

agentId?: string;

472+

modelProvider?: string;

473+

modelId?: string;

474+

messageProvider?: string;

475+

agentAccountId?: string | null;

476+

groupId?: string | null;

477+

groupChannel?: string | null;

478+

groupSpace?: string | null;

479+

spawnedBy?: string | null;

480+

senderId?: string | null;

481+

senderName?: string | null;

482+

senderUsername?: string | null;

483+

senderE164?: string | null;

484+

sandboxToolPolicy?: { allow?: string[]; deny?: string[] };

485+

toolsAllow?: string[];

486+

}) {

487+

const { agentId, globalPolicy, globalProviderPolicy, agentPolicy, agentProviderPolicy } =

488+

resolveEffectiveToolPolicy({

489+

config: params.config,

490+

sessionKey: params.sessionKey,

491+

agentId: params.agentId,

492+

modelProvider: params.modelProvider,

493+

modelId: params.modelId,

494+

});

495+

const groupPolicy = resolveGroupToolPolicy({

496+

config: params.config,

497+

sessionKey: params.sessionKey,

498+

spawnedBy: params.spawnedBy,

499+

messageProvider: params.messageProvider,

500+

groupId: params.groupId,

501+

groupChannel: params.groupChannel,

502+

groupSpace: params.groupSpace,

503+

accountId: params.agentAccountId,

504+

senderId: params.senderId,

505+

senderName: params.senderName,

506+

senderUsername: params.senderUsername,

507+

senderE164: params.senderE164,

508+

});

509+

const subagentStore = resolveSubagentCapabilityStore(params.sandboxSessionKey, {

510+

cfg: params.config,

511+

});

512+

const subagentPolicy =

513+

params.sandboxSessionKey &&

514+

isSubagentEnvelopeSession(params.sandboxSessionKey, {

515+

cfg: params.config,

516+

store: subagentStore,

517+

})

518+

? resolveSubagentToolPolicyForSession(params.config, params.sandboxSessionKey, {

519+

store: subagentStore,

520+

})

521+

: undefined;

522+

return collectExplicitToolAllowlistSources([

523+

{ label: "tools.allow", allow: globalPolicy?.allow },

524+

{ label: "tools.byProvider.allow", allow: globalProviderPolicy?.allow },

525+

{

526+

label: agentId ? `agents.${agentId}.tools.allow` : "agent tools.allow",

527+

allow: agentPolicy?.allow,

528+

},

529+

{

530+

label: agentId ? `agents.${agentId}.tools.byProvider.allow` : "agent tools.byProvider.allow",

531+

allow: agentProviderPolicy?.allow,

532+

},

533+

{ label: "group tools.allow", allow: groupPolicy?.allow },

534+

{ label: "sandbox tools.allow", allow: params.sandboxToolPolicy?.allow },

535+

{ label: "subagent tools.allow", allow: subagentPolicy?.allow },

536+

{ label: "runtime toolsAllow", allow: params.toolsAllow },

537+

]);

538+

}

539+454540

export async function runEmbeddedAttempt(

455541

params: EmbeddedRunAttemptParams,

456542

): Promise<EmbeddedRunAttemptResult> {

@@ -802,6 +888,32 @@ export async function runEmbeddedAttempt(

802888

tools: effectiveTools,

803889

clientTools,

804890

});

891+

const explicitToolAllowlistSources = collectAttemptExplicitToolAllowlistSources({

892+

config: params.config,

893+

sessionKey: params.sessionKey,

894+

sandboxSessionKey,

895+

agentId: sessionAgentId,

896+

modelProvider: params.provider,

897+

modelId: params.modelId,

898+

messageProvider: params.messageChannel ?? params.messageProvider,

899+

agentAccountId: params.agentAccountId,

900+

groupId: params.groupId,

901+

groupChannel: params.groupChannel,

902+

groupSpace: params.groupSpace,

903+

spawnedBy: params.spawnedBy,

904+

senderId: params.senderId,

905+

senderName: params.senderName,

906+

senderUsername: params.senderUsername,

907+

senderE164: params.senderE164,

908+

sandboxToolPolicy: sandbox?.tools,

909+

toolsAllow: params.toolsAllow,

910+

});

911+

const emptyExplicitToolAllowlistError = buildEmptyExplicitToolAllowlistError({

912+

sources: explicitToolAllowlistSources,

913+

callableToolNames: effectiveTools.map((tool) => tool.name),

914+

toolsEnabled,

915+

disableTools: params.disableTools,

916+

});

805917

if (params.runtimePlan) {

806918

params.runtimePlan.tools.logDiagnostics(effectiveTools, runtimePlanModelContext);

807919

} else {

@@ -2031,6 +2143,12 @@ export async function runEmbeddedAttempt(

20312143

let skipPromptSubmission = false;

20322144

try {

20332145

const promptStartedAt = Date.now();

2146+

if (emptyExplicitToolAllowlistError) {

2147+

promptError = emptyExplicitToolAllowlistError;

2148+

promptErrorSource = "precheck";

2149+

skipPromptSubmission = true;

2150+

log.warn(`[tools] ${emptyExplicitToolAllowlistError.message}`);

2151+

}

2034215220352153

// Run before_prompt_build hooks to allow plugins to inject prompt context.

20362154

// Legacy compatibility: before_agent_start is also checked for context fields.