luna/oh-my-opencode
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

fix(tools/skill): make factory pure and stop defeating skill-loader cache

Browse Source
This commit is contained in:
YeonGyu-Kim
2026-04-18 14:15:14 +09:00
parent 40bd3e02d2
commit 1be1cd6e53
1 changed files with 1 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/tools/skill/tools.ts
+1 -4
View File
@@ -4,7 +4,7 @@ import type { ToolContext } from "@opencode-ai/plugin/tool"
import { TOOL_DESCRIPTION_PREFIX } from "./constants" import { TOOL_DESCRIPTION_PREFIX } from "./constants"
import type { SkillArgs, SkillLoadOptions } from "./types" import type { SkillArgs, SkillLoadOptions } from "./types"
import type { LoadedSkill } from "../../features/opencode-skill-loader" import type { LoadedSkill } from "../../features/opencode-skill-loader"
import { getAllSkills, clearSkillCache } from "../../features/opencode-skill-loader/skill-content" import { getAllSkills } from "../../features/opencode-skill-loader/skill-content"
import { injectGitMasterConfig } from "../../features/opencode-skill-loader/skill-content" import { injectGitMasterConfig } from "../../features/opencode-skill-loader/skill-content"
import { discoverCommandsSync } from "../slashcommand/command-discovery" import { discoverCommandsSync } from "../slashcommand/command-discovery"
import type { CommandInfo } from "../slashcommand/types" import type { CommandInfo } from "../slashcommand/types"
@@ -28,7 +28,6 @@ export function createSkillTool(options: SkillLoadOptions = {}): ToolDefinition
let cachedDescription: string | null = null let cachedDescription: string | null = null
const getSkills = async (): Promise<LoadedSkill[]> => { const getSkills = async (): Promise<LoadedSkill[]> => {
clearSkillCache()
const discovered = await getAllSkills({ const discovered = await getAllSkills({
disabledSkills: options?.disabledSkills, disabledSkills: options?.disabledSkills,
browserProvider: options?.browserProvider, browserProvider: options?.browserProvider,
@@ -92,8 +91,6 @@ export function createSkillTool(options: SkillLoadOptions = {}): ToolDefinition
} }
} else if (options.commands !== undefined) { } else if (options.commands !== undefined) {
cachedDescription = formatCombinedDescription([], options.commands) cachedDescription = formatCombinedDescription([], options.commands)
} else {
void buildDescription()
} }
return tool({ return tool({