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

推荐订阅源

博客园 - 叶小钗
爱范儿
爱范儿
WordPress大学
WordPress大学
Last Week in AI
Last Week in AI
博客园 - 聂微东
雷峰网
雷峰网
奇客Solidot–传递最新科技情报
奇客Solidot–传递最新科技情报
博客园 - 三生石上(FineUI控件)
T
Tailwind CSS Blog
博客园 - Franky
酷 壳 – CoolShell
酷 壳 – CoolShell
博客园_首页
OSCHINA 社区最新新闻
OSCHINA 社区最新新闻
博客园 - 司徒正美
月光博客
月光博客
大猫的无限游戏
大猫的无限游戏
钛媒体:引领未来商业与生活新知
钛媒体:引领未来商业与生活新知
freeCodeCamp Programming Tutorials: Python, JavaScript, Git & More
The Cloudflare Blog
让小产品的独立变现更简单 - ezindie.com
让小产品的独立变现更简单 - ezindie.com
人人都是产品经理
人人都是产品经理
宝玉的分享
宝玉的分享
罗磊的独立博客
Jina AI
Jina AI

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
revert: 60bec8c duplicate tool display guard · openclaw/o...
joshavant · 2026-05-27 · via Recent Commits to openclaw:main

@@ -1,12 +1,10 @@

11

import fs from "node:fs";

22

import path from "node:path";

33

import { fileURLToPath } from "node:url";

4-

import ts from "typescript";

54

import { TOOL_DISPLAY_CONFIG, type ToolDisplayConfig } from "../src/agents/tool-display-config.js";

6576

const scriptDir = path.dirname(fileURLToPath(import.meta.url));

87

const repoRoot = path.resolve(scriptDir, "..");

9-

const configPath = path.join(repoRoot, "src/agents/tool-display-config.ts");

108

const outputPath = path.join(

119

repoRoot,

1210

"apps/shared/OpenClawKit/Sources/OpenClawKit/Resources/tool-display.json",

@@ -18,63 +16,49 @@ const toolSources = [

1816

path.join(repoRoot, "src/auto-reply/reply/acp-projector.ts"),

1917

];

201821-

type DuplicateToolKey = {

22-

name: string;

23-

lines: number[];

24-

};

19+

const args = new Set(process.argv.slice(2));

20+

const shouldCheck = args.has("--check");

21+

const shouldWrite = args.has("--write");

252226-

export function main(argv = process.argv.slice(2)): number {

27-

const args = new Set(argv);

28-

const shouldCheck = args.has("--check");

29-

const shouldWrite = args.has("--write");

30-31-

if (!shouldCheck && !shouldWrite) {

32-

console.error("Usage: node --import tsx scripts/tool-display.ts --check|--write");

33-

return 1;

34-

}

35-36-

const duplicateErrors = collectToolDisplayDuplicateErrors({ includeSnapshot: shouldCheck });

37-

if (duplicateErrors.length > 0) {

38-

console.error(duplicateErrors.join("\n"));

39-

return 1;

40-

}

41-42-

const expected = serializeToolDisplayConfig();

43-

ensureCoreToolCoverage();

23+

if (!shouldCheck && !shouldWrite) {

24+

console.error("Usage: node --import tsx scripts/tool-display.ts --check|--write");

25+

process.exit(1);

26+

}

442745-

if (shouldWrite) {

46-

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

47-

fs.writeFileSync(outputPath, expected);

48-

process.stdout.write(`wrote ${path.relative(repoRoot, outputPath)}\n`);

49-

return 0;

50-

}

28+

const expected = serializeToolDisplayConfig();

29+

ensureCoreToolCoverage();

513052-

if (!fs.existsSync(path.dirname(outputPath))) {

53-

process.stdout.write(

54-

`skip tool-display snapshot check; missing ${path.relative(repoRoot, path.dirname(outputPath))}\n`,

55-

);

56-

return 0;

57-

}

31+

if (shouldWrite) {

32+

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

33+

fs.writeFileSync(outputPath, expected);

34+

process.stdout.write(`wrote ${path.relative(repoRoot, outputPath)}\n`);

35+

process.exit(0);

36+

}

583759-

if (!fs.existsSync(outputPath)) {

60-

console.error(

61-

`missing generated snapshot: ${path.relative(repoRoot, outputPath)}\nrun: pnpm tool-display:write`,

62-

);

63-

return 1;

64-

}

38+

if (!fs.existsSync(path.dirname(outputPath))) {

39+

process.stdout.write(

40+

`skip tool-display snapshot check; missing ${path.relative(repoRoot, path.dirname(outputPath))}\n`,

41+

);

42+

process.exit(0);

43+

}

654466-

const actual = fs.readFileSync(outputPath, "utf8");

67-

if (actual !== expected) {

68-

console.error(

69-

`tool-display snapshot is stale: ${path.relative(repoRoot, outputPath)}\nrun: pnpm tool-display:write`,

70-

);

71-

return 1;

72-

}

45+

if (!fs.existsSync(outputPath)) {

46+

console.error(

47+

`missing generated snapshot: ${path.relative(repoRoot, outputPath)}\nrun: pnpm tool-display:write`,

48+

);

49+

process.exit(1);

50+

}

735174-

process.stdout.write("tool-display snapshot is up to date\n");

75-

return 0;

52+

const actual = fs.readFileSync(outputPath, "utf8");

53+

if (actual !== expected) {

54+

console.error(

55+

`tool-display snapshot is stale: ${path.relative(repoRoot, outputPath)}\nrun: pnpm tool-display:write`,

56+

);

57+

process.exit(1);

7658

}

775960+

process.stdout.write("tool-display snapshot is up to date\n");

61+7862

function ensureCoreToolCoverage() {

7963

const toolNames = new Set<string>();

8064

for (const sourcePath of toolSources) {

@@ -108,131 +92,3 @@ function collectToolNamesFromFile(sourcePath: string, names: Set<string>) {

10892

function serializeToolDisplayConfig(config: ToolDisplayConfig = TOOL_DISPLAY_CONFIG): string {

10993

return `${JSON.stringify(config, null, 2)}\n`;

11094

}

111-112-

function collectToolDisplayDuplicateErrors(options: { includeSnapshot: boolean }): string[] {

113-

const duplicateErrors: string[] = [];

114-

const configSource = fs.readFileSync(configPath, "utf8");

115-

const configDuplicates = collectToolDisplayConfigDuplicateKeys(configSource, configPath);

116-

if (configDuplicates.length > 0) {

117-

duplicateErrors.push(

118-

formatDuplicateToolKeyError(path.relative(repoRoot, configPath), configDuplicates),

119-

);

120-

}

121-122-

if (options.includeSnapshot && fs.existsSync(outputPath)) {

123-

const snapshotSource = fs.readFileSync(outputPath, "utf8");

124-

const snapshotDuplicates = collectToolDisplaySnapshotDuplicateKeys(snapshotSource, outputPath);

125-

if (snapshotDuplicates.length > 0) {

126-

duplicateErrors.push(

127-

formatDuplicateToolKeyError(path.relative(repoRoot, outputPath), snapshotDuplicates),

128-

);

129-

}

130-

}

131-

return duplicateErrors;

132-

}

133-134-

export function collectToolDisplayConfigDuplicateKeys(

135-

source: string,

136-

sourcePath = "src/agents/tool-display-config.ts",

137-

): DuplicateToolKey[] {

138-

const sourceFile = ts.createSourceFile(sourcePath, source, ts.ScriptTarget.Latest, true);

139-

let toolsObject: ts.ObjectLiteralExpression | undefined;

140-

visitToolDisplayConfig(sourceFile, (configObject) => {

141-

toolsObject = findObjectProperty(configObject, "tools");

142-

});

143-

return toolsObject ? collectDuplicatePropertyKeys(toolsObject, sourceFile) : [];

144-

}

145-146-

export function collectToolDisplaySnapshotDuplicateKeys(

147-

source: string,

148-

sourcePath = "tool-display.json",

149-

): DuplicateToolKey[] {

150-

const sourceFile = ts.parseJsonText(sourcePath, source);

151-

const statement = sourceFile.statements[0];

152-

if (!statement || !ts.isExpressionStatement(statement)) {

153-

return [];

154-

}

155-

const root = statement.expression;

156-

if (!ts.isObjectLiteralExpression(root)) {

157-

return [];

158-

}

159-

const toolsObject = findObjectProperty(root, "tools");

160-

return toolsObject ? collectDuplicatePropertyKeys(toolsObject, sourceFile) : [];

161-

}

162-163-

export function formatDuplicateToolKeyError(

164-

relativePath: string,

165-

duplicates: DuplicateToolKey[],

166-

): string {

167-

const formatted = duplicates

168-

.map((duplicate) => `${duplicate.name} at lines ${duplicate.lines.join(", ")}`)

169-

.join("; ");

170-

return `tool-display metadata has duplicate tool ids in ${relativePath}: ${formatted}`;

171-

}

172-173-

function visitToolDisplayConfig(

174-

node: ts.Node,

175-

onConfig: (configObject: ts.ObjectLiteralExpression) => void,

176-

) {

177-

if (

178-

ts.isVariableDeclaration(node) &&

179-

ts.isIdentifier(node.name) &&

180-

node.name.text === "TOOL_DISPLAY_CONFIG" &&

181-

node.initializer &&

182-

ts.isObjectLiteralExpression(node.initializer)

183-

) {

184-

onConfig(node.initializer);

185-

return;

186-

}

187-

ts.forEachChild(node, (child) => visitToolDisplayConfig(child, onConfig));

188-

}

189-190-

function findObjectProperty(

191-

object: ts.ObjectLiteralExpression,

192-

propertyName: string,

193-

): ts.ObjectLiteralExpression | undefined {

194-

for (const property of object.properties) {

195-

if (

196-

ts.isPropertyAssignment(property) &&

197-

getPropertyNameText(property.name) === propertyName &&

198-

ts.isObjectLiteralExpression(property.initializer)

199-

) {

200-

return property.initializer;

201-

}

202-

}

203-

return undefined;

204-

}

205-206-

function collectDuplicatePropertyKeys(

207-

object: ts.ObjectLiteralExpression,

208-

sourceFile: ts.SourceFile,

209-

): DuplicateToolKey[] {

210-

const keyLines = new Map<string, number[]>();

211-

for (const property of object.properties) {

212-

if (!ts.isPropertyAssignment(property)) {

213-

continue;

214-

}

215-

const name = getPropertyNameText(property.name);

216-

if (!name) {

217-

continue;

218-

}

219-

const line =

220-

sourceFile.getLineAndCharacterOfPosition(property.name.getStart(sourceFile)).line + 1;

221-

keyLines.set(name, [...(keyLines.get(name) ?? []), line]);

222-

}

223-

return [...keyLines.entries()]

224-

.filter(([, lines]) => lines.length > 1)

225-

.map(([name, lines]) => ({ name, lines }))

226-

.toSorted((left, right) => left.name.localeCompare(right.name));

227-

}

228-229-

function getPropertyNameText(name: ts.PropertyName): string | undefined {

230-

if (ts.isIdentifier(name) || ts.isStringLiteral(name) || ts.isNumericLiteral(name)) {

231-

return name.text;

232-

}

233-

return undefined;

234-

}

235-236-

if (process.argv[1] && path.resolve(process.argv[1]) === fileURLToPath(import.meta.url)) {

237-

process.exitCode = main();

238-

}