claude-code-best/src/components/agents/validateAgent.ts
claude-code-best 2fb1c9dcd8
feat: 工具层及 mcp 大重构 (#252)
* feat: 第一版大重构

* fix: 修复类型问题

* chore: 更新版本到 1.3.2

* Add brave as alternative WebSearchTool

* fix: 修正顺序

* fix: 修复对穷鬼模式的 auto dream 和 session memory 越过

* feat: 穷鬼模式去除 session-summary

* feat: 创建 builtin-tools 包,搬运所有工具实现

将 src/tools/ 下的全部 60 个工具目录迁移至 packages/builtin-tools/src/tools/,
内部导入路径已更新为 src/ alias 模式。

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

* refactor: 更新 src/ 中所有工具引用至 builtin-tools 包,删除 src/tools/

- src/tools.ts 及 178 个 src/ 文件的 import 路径从 ./tools/ 改为 builtin-tools/tools/
- 删除 src/tools/ 整个目录(已迁移至 packages/builtin-tools/)

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

* chore: 添加 builtin-tools 路径别名至 tsconfig,更新 bun.lock

- tsconfig.json 新增 builtin-tools/* 和 builtin-tools 路径映射
- 新增 packages/builtin-tools/src 至 include

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

* refactor: 为 builtin-tools、mcp-client、agent-tools 添加 @claude-code-best 作用域前缀

所有包名及 import 路径统一添加 @claude-code-best/ 前缀:
- builtin-tools → @claude-code-best/builtin-tools
- mcp-client → @claude-code-best/mcp-client
- agent-tools → @claude-code-best/agent-tools

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

* fix: 修复 node 环境没有 bun 的问题

---------

Co-authored-by: Eric-Guo <eric.guocz@gmail.com>
Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
2026-04-13 09:52:05 +08:00

110 lines
3.1 KiB
TypeScript

import type { Tools } from '../../Tool.js'
import { resolveAgentTools } from '@claude-code-best/builtin-tools/tools/AgentTool/agentToolUtils.js'
import type {
AgentDefinition,
CustomAgentDefinition,
} from '@claude-code-best/builtin-tools/tools/AgentTool/loadAgentsDir.js'
import { getAgentSourceDisplayName } from './utils.js'
export type AgentValidationResult = {
isValid: boolean
errors: string[]
warnings: string[]
}
export function validateAgentType(agentType: string): string | null {
if (!agentType) {
return 'Agent type is required'
}
if (!/^[a-zA-Z0-9][a-zA-Z0-9-]*[a-zA-Z0-9]$/.test(agentType)) {
return 'Agent type must start and end with alphanumeric characters and contain only letters, numbers, and hyphens'
}
if (agentType.length < 3) {
return 'Agent type must be at least 3 characters long'
}
if (agentType.length > 50) {
return 'Agent type must be less than 50 characters'
}
return null
}
export function validateAgent(
agent: Omit<CustomAgentDefinition, 'location'>,
availableTools: Tools,
existingAgents: AgentDefinition[],
): AgentValidationResult {
const errors: string[] = []
const warnings: string[] = []
// Validate agent type
if (!agent.agentType) {
errors.push('Agent type is required')
} else {
const typeError = validateAgentType(agent.agentType)
if (typeError) {
errors.push(typeError)
}
// Check for duplicates (excluding self for editing)
const duplicate = existingAgents.find(
a => a.agentType === agent.agentType && a.source !== agent.source,
)
if (duplicate) {
errors.push(
`Agent type "${agent.agentType}" already exists in ${getAgentSourceDisplayName(duplicate.source)}`,
)
}
}
// Validate description
if (!agent.whenToUse) {
errors.push('Description (description) is required')
} else if (agent.whenToUse.length < 10) {
warnings.push(
'Description should be more descriptive (at least 10 characters)',
)
} else if (agent.whenToUse.length > 5000) {
warnings.push('Description is very long (over 5000 characters)')
}
// Validate tools
if (agent.tools !== undefined && !Array.isArray(agent.tools)) {
errors.push('Tools must be an array')
} else {
if (agent.tools === undefined) {
warnings.push('Agent has access to all tools')
} else if (agent.tools.length === 0) {
warnings.push(
'No tools selected - agent will have very limited capabilities',
)
}
// Check for invalid tools
const resolvedTools = resolveAgentTools(agent, availableTools, false)
if (resolvedTools.invalidTools.length > 0) {
errors.push(`Invalid tools: ${resolvedTools.invalidTools.join(', ')}`)
}
}
// Validate system prompt
const systemPrompt = agent.getSystemPrompt()
if (!systemPrompt) {
errors.push('System prompt is required')
} else if (systemPrompt.length < 20) {
errors.push('System prompt is too short (minimum 20 characters)')
} else if (systemPrompt.length > 10000) {
warnings.push('System prompt is very long (over 10,000 characters)')
}
return {
isValid: errors.length === 0,
errors,
warnings,
}
}