YeonGyu-Kim
e5d3fe96c4
Blocking fixes: - B1: Return empty restrictions for unknown/custom agents instead of EXPLORATION_AGENT_DENYLIST, allowing custom agents full tool access - B2: Use Object.create(null) consistently across all 5 agent-loading result objects to prevent prototype pollution - B3: Add code comment documenting custom agent bash access trust model - B4: Mock getOpenCodeConfigDir in opencode-config-agents-reader tests to prevent global config dir leakage Non-blocking fixes: - N1: Use resolveAgentDefinitionPaths with project boundary enforcement in opencode-config-agents-reader for path containment - N2: Add session-scoped 30s TTL cache to resolveCallableAgents to avoid redundant SDK IPC calls per tool invocation - N3: Extract shared parseToolsConfig into src/shared/parse-tools-config.ts replacing 4 duplicated local implementations - N4: Add .min(1) to AgentDefinitionPathSchema rejecting empty paths - N5: Add resolve-agent-definition-paths.test.ts covering tilde expansion, relative paths, boundary enforcement, and null containmentDir - N6: Validate agent mode against allowed values instead of bare type assertion in opencode-config-agents-reader
54 lines
1.5 KiB
TypeScript
54 lines
1.5 KiB
TypeScript
import { existsSync, readFileSync } from "fs"
|
|
import { parseJsoncSafe } from "../../shared/jsonc-parser"
|
|
import { parseToolsConfig } from "../../shared/parse-tools-config"
|
|
import { mapClaudeModelToOpenCode } from "./claude-model-mapper"
|
|
import type { AgentScope, AgentJsonDefinition, ClaudeCodeAgentConfig, LoadedAgent } from "./types"
|
|
|
|
export function parseJsonAgentFile(filePath: string, scope: AgentScope): LoadedAgent | null {
|
|
try {
|
|
if (!existsSync(filePath)) {
|
|
return null
|
|
}
|
|
|
|
const content = readFileSync(filePath, "utf-8")
|
|
const { data } = parseJsoncSafe<AgentJsonDefinition>(content)
|
|
|
|
if (!data) {
|
|
return null
|
|
}
|
|
|
|
if (!data.name || !data.prompt) {
|
|
return null
|
|
}
|
|
|
|
const originalDescription = data.description ?? ""
|
|
const formattedDescription = `(${scope}) ${originalDescription}`
|
|
|
|
const mappedModelOverride = mapClaudeModelToOpenCode(data.model)
|
|
const modelString = mappedModelOverride
|
|
? `${mappedModelOverride.providerID}/${mappedModelOverride.modelID}`
|
|
: undefined
|
|
|
|
const config: ClaudeCodeAgentConfig = {
|
|
description: formattedDescription,
|
|
mode: data.mode ?? "subagent",
|
|
prompt: data.prompt.trim(),
|
|
...(modelString ? { model: modelString } : {}),
|
|
}
|
|
|
|
const toolsConfig = parseToolsConfig(data.tools)
|
|
if (toolsConfig) {
|
|
config.tools = toolsConfig
|
|
}
|
|
|
|
return {
|
|
name: data.name,
|
|
path: filePath,
|
|
config,
|
|
scope,
|
|
}
|
|
} catch {
|
|
return null
|
|
}
|
|
}
|