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

推荐订阅源

雷峰网
雷峰网
B
Blog
博客园_首页
云风的 BLOG
云风的 BLOG
S
SegmentFault 最新的问题
罗磊的独立博客
Jina AI
Jina AI
C
Check Point Blog
Martin Fowler
Martin Fowler
J
Java Code Geeks
博客园 - 司徒正美
美团技术团队
MongoDB | Blog
MongoDB | Blog
钛媒体:引领未来商业与生活新知
钛媒体:引领未来商业与生活新知
奇客Solidot–传递最新科技情报
奇客Solidot–传递最新科技情报
大猫的无限游戏
大猫的无限游戏
有赞技术团队
有赞技术团队
U
Unit 42
Hugging Face - Blog
Hugging Face - Blog
WordPress大学
WordPress大学
让小产品的独立变现更简单 - ezindie.com
让小产品的独立变现更简单 - ezindie.com
博客园 - 叶小钗
博客园 - 三生石上(FineUI控件)
小众软件
小众软件

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
chore(deadcode): inline message provider tool filtering ·...
vincentkoc · 2026-06-21 · via Recent Commits to openclaw:main
Original file line numberDiff line numberDiff line change

@@ -4,21 +4,42 @@

44

* unsafe or redundant for the active channel.

55

*/

66

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

7-

import { filterToolNamesByMessageProvider } from "./agent-tools.message-provider-policy.js";

7+

import { filterToolsByMessageProvider } from "./agent-tools.message-provider-policy.js";

88
9-

const DEFAULT_TOOL_NAMES = ["read", "write", "tts", "web_search"];

9+

const DEFAULT_TOOLS = [

10+

{ name: "read" },

11+

{ name: "write" },

12+

{ name: "tts" },

13+

{ name: "web_search" },

14+

];

15+
16+

function toolNames(tools: readonly { name: string }[]): Set<string> {

17+

return new Set(tools.map((tool) => tool.name));

18+

}

1019
1120

describe("createOpenClawCodingTools message provider policy", () => {

1221

it.each(["voice", "VOICE", " Voice ", "discord-voice", "DISCORD-VOICE", " Discord-Voice "])(

1322

"does not expose tts tool for normalized voice provider: %s",

1423

(messageProvider) => {

15-

const names = new Set(filterToolNamesByMessageProvider(DEFAULT_TOOL_NAMES, messageProvider));

24+

const names = toolNames(filterToolsByMessageProvider(DEFAULT_TOOLS, messageProvider));

1625

expect(names.has("tts")).toBe(false);

1726

},

1827

);

1928
2029

it("keeps tts tool for non-voice providers", () => {

21-

const names = new Set(filterToolNamesByMessageProvider(DEFAULT_TOOL_NAMES, "guildchat"));

30+

const names = toolNames(filterToolsByMessageProvider(DEFAULT_TOOLS, "guildchat"));

2231

expect(names.has("tts")).toBe(true);

2332

});

33+
34+

it("preserves duplicate tool entries while filtering", () => {

35+

const tools = [

36+

{ name: "read", id: 1 },

37+

{ name: "tts", id: 2 },

38+

{ name: "read", id: 3 },

39+

];

40+

expect(filterToolsByMessageProvider(tools, "voice")).toStrictEqual([

41+

{ name: "read", id: 1 },

42+

{ name: "read", id: 3 },

43+

]);

44+

});

2445

});

Original file line numberDiff line numberDiff line change

@@ -14,49 +14,24 @@ const TOOL_ALLOW_BY_MESSAGE_PROVIDER: Readonly<Record<string, readonly string[]>

1414

node: ["canvas", "image", "pdf", "tts", "web_fetch", "web_search"],

1515

};

1616
17-

/** Filters tool names by the active message-provider allow/deny policy. */

18-

export function filterToolNamesByMessageProvider(

19-

toolNames: readonly string[],

17+

/** Applies message-provider filtering while preserving duplicate tool entries. */

18+

export function filterToolsByMessageProvider<TTool extends { name: string }>(

19+

tools: readonly TTool[],

2020

messageProvider?: string,

21-

): string[] {

21+

): TTool[] {

2222

const normalizedProvider = normalizeOptionalLowercaseString(messageProvider);

2323

if (!normalizedProvider) {

24-

return [...toolNames];

24+

return [...tools];

2525

}

2626

const allowedTools = TOOL_ALLOW_BY_MESSAGE_PROVIDER[normalizedProvider];

2727

if (allowedTools && allowedTools.length > 0) {

2828

const allowedSet = new Set(allowedTools);

29-

return toolNames.filter((toolName) => allowedSet.has(toolName));

29+

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

3030

}

3131

const deniedTools = TOOL_DENY_BY_MESSAGE_PROVIDER[normalizedProvider];

3232

if (!deniedTools || deniedTools.length === 0) {

33-

return [...toolNames];

33+

return [...tools];

3434

}

3535

const deniedSet = new Set(deniedTools);

36-

return toolNames.filter((toolName) => !deniedSet.has(toolName));

37-

}

38-
39-

/** Applies message-provider filtering while preserving duplicate tool entries. */

40-

export function filterToolsByMessageProvider<TTool extends { name: string }>(

41-

tools: readonly TTool[],

42-

messageProvider?: string,

43-

): TTool[] {

44-

const filteredToolNames = filterToolNamesByMessageProvider(

45-

tools.map((tool) => tool.name),

46-

messageProvider,

47-

);

48-

const remainingCounts = new Map<string, number>();

49-

for (const toolName of filteredToolNames) {

50-

remainingCounts.set(toolName, (remainingCounts.get(toolName) ?? 0) + 1);

51-

}

52-

return tools.filter((tool) => {

53-

// Counted matching preserves the original order and duplicate instances

54-

// after name-level policy filtering.

55-

const remaining = remainingCounts.get(tool.name) ?? 0;

56-

if (remaining <= 0) {

57-

return false;

58-

}

59-

remainingCounts.set(tool.name, remaining - 1);

60-

return true;

61-

});

36+

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

6237

}