feat: add opencode-skill-loader with 4-source priority system (#331)

* feat: add opencode-skill-loader with 4-source priority system - Create new opencode-skill-loader feature module independent from Claude Code - Support 4 source paths with priority: opencode-project > project > opencode > user - .opencode/skill/ (opencode-project) - .claude/skills/ (project) - ~/.config/opencode/skill/ (opencode) - ~/.claude/skills/ (user) - Support both SKILL.md and {SKILLNAME}.md file patterns - Maintain path awareness for file references (@path syntax) * feat: integrate opencode-skill-loader into main plugin - Import and use new skill loader functions - Load skills from all 4 sources and merge into config.command - Also merge pluginComponents.skills (previously loaded but never used) * feat: add skill discovery to slashcommand tool - Import and use discoverAllSkills from opencode-skill-loader - Display skills alongside commands in tool description and execution - Update formatCommandList to handle combined commands and skills * refactor: remove old claude-code-skill-loader - Delete src/features/claude-code-skill-loader/ directory (was never integrated into main plugin) - Update plugin loader import to use new opencode-skill-loader types * docs: update AGENTS.md for new skill loader - Update structure to show opencode-skill-loader instead of claude-code-skill-loader - Update Skills priority order to include all 4 sources --------- Co-authored-by: sisyphus-dev-ai <sisyphus-dev-ai@users.noreply.github.com>
2025-12-30 10:42:05 +09:00
parent 7324b6c6b5
commit 17e8746eff
9 changed files with 268 additions and 120 deletions
--- a/src/features/opencode-skill-loader/loader.ts
+++ b/src/features/opencode-skill-loader/loader.ts
@@ -0,0 +1,179 @@
+import { existsSync, readdirSync, readFileSync } from "fs"
+import { join, basename, dirname } from "path"
+import { homedir } from "os"
+import { parseFrontmatter } from "../../shared/frontmatter"
+import { sanitizeModelField } from "../../shared/model-sanitizer"
+import { resolveSymlink, isMarkdownFile } from "../../shared/file-utils"
+import { getClaudeConfigDir } from "../../shared"
+import type { CommandDefinition } from "../claude-code-command-loader/types"
+import type { SkillScope, SkillMetadata, LoadedSkill } from "./types"
+
+/**
+ * Load a skill from a markdown file path.
+ * 
+ * @param skillPath - Path to the skill file (SKILL.md or {name}.md)
+ * @param resolvedPath - Directory for file reference resolution (@path references)
+ * @param defaultName - Fallback name if not specified in frontmatter
+ * @param scope - Source scope for priority ordering
+ */
+function loadSkillFromPath(
+  skillPath: string,
+  resolvedPath: string,
+  defaultName: string,
+  scope: SkillScope
+): LoadedSkill | null {
+  try {
+    const content = readFileSync(skillPath, "utf-8")
+    const { data, body } = parseFrontmatter<SkillMetadata>(content)
+
+    const skillName = data.name || defaultName
+    const originalDescription = data.description || ""
+    const isOpencodeSource = scope === "opencode" || scope === "opencode-project"
+    const formattedDescription = `(${scope} - Skill) ${originalDescription}`
+
+    const wrappedTemplate = `<skill-instruction>
+Base directory for this skill: ${resolvedPath}/
+File references (@path) in this skill are relative to this directory.
+
+${body.trim()}
+</skill-instruction>
+
+<user-request>
+$ARGUMENTS
+</user-request>`
+
+    const definition: CommandDefinition = {
+      name: skillName,
+      description: formattedDescription,
+      template: wrappedTemplate,
+      model: sanitizeModelField(data.model, isOpencodeSource ? "opencode" : "claude-code"),
+      agent: data.agent,
+      subtask: data.subtask,
+      argumentHint: data["argument-hint"],
+    }
+
+    return {
+      name: skillName,
+      path: skillPath,
+      resolvedPath,
+      definition,
+      scope,
+    }
+  } catch {
+    return null
+  }
+}
+
+/**
+ * Load skills from a directory, supporting BOTH patterns:
+ * - Directory with SKILL.md: skill-name/SKILL.md
+ * - Directory with {SKILLNAME}.md: skill-name/{SKILLNAME}.md
+ * - Direct markdown file: skill-name.md
+ */
+function loadSkillsFromDir(skillsDir: string, scope: SkillScope): LoadedSkill[] {
+  if (!existsSync(skillsDir)) {
+    return []
+  }
+
+  const entries = readdirSync(skillsDir, { withFileTypes: true })
+  const skills: LoadedSkill[] = []
+
+  for (const entry of entries) {
+    if (entry.name.startsWith(".")) continue
+
+    const entryPath = join(skillsDir, entry.name)
+
+    if (entry.isDirectory() || entry.isSymbolicLink()) {
+      const resolvedPath = resolveSymlink(entryPath)
+      const dirName = entry.name
+
+      const skillMdPath = join(resolvedPath, "SKILL.md")
+      if (existsSync(skillMdPath)) {
+        const skill = loadSkillFromPath(skillMdPath, resolvedPath, dirName, scope)
+        if (skill) skills.push(skill)
+        continue
+      }
+
+      const namedSkillMdPath = join(resolvedPath, `${dirName}.md`)
+      if (existsSync(namedSkillMdPath)) {
+        const skill = loadSkillFromPath(namedSkillMdPath, resolvedPath, dirName, scope)
+        if (skill) skills.push(skill)
+        continue
+      }
+
+      continue
+    }
+
+    if (isMarkdownFile(entry)) {
+      const skillName = basename(entry.name, ".md")
+      const skill = loadSkillFromPath(entryPath, skillsDir, skillName, scope)
+      if (skill) skills.push(skill)
+    }
+  }
+
+  return skills
+}
+
+function skillsToRecord(skills: LoadedSkill[]): Record<string, CommandDefinition> {
+  const result: Record<string, CommandDefinition> = {}
+  for (const skill of skills) {
+    result[skill.name] = skill.definition
+  }
+  return result
+}
+
+/**
+ * Load skills from Claude Code user directory (~/.claude/skills/)
+ */
+export function loadUserSkills(): Record<string, CommandDefinition> {
+  const userSkillsDir = join(getClaudeConfigDir(), "skills")
+  const skills = loadSkillsFromDir(userSkillsDir, "user")
+  return skillsToRecord(skills)
+}
+
+/**
+ * Load skills from Claude Code project directory (.claude/skills/)
+ */
+export function loadProjectSkills(): Record<string, CommandDefinition> {
+  const projectSkillsDir = join(process.cwd(), ".claude", "skills")
+  const skills = loadSkillsFromDir(projectSkillsDir, "project")
+  return skillsToRecord(skills)
+}
+
+/**
+ * Load skills from OpenCode global directory (~/.config/opencode/skill/)
+ */
+export function loadOpencodeGlobalSkills(): Record<string, CommandDefinition> {
+  const opencodeSkillsDir = join(homedir(), ".config", "opencode", "skill")
+  const skills = loadSkillsFromDir(opencodeSkillsDir, "opencode")
+  return skillsToRecord(skills)
+}
+
+/**
+ * Load skills from OpenCode project directory (.opencode/skill/)
+ */
+export function loadOpencodeProjectSkills(): Record<string, CommandDefinition> {
+  const opencodeProjectDir = join(process.cwd(), ".opencode", "skill")
+  const skills = loadSkillsFromDir(opencodeProjectDir, "opencode-project")
+  return skillsToRecord(skills)
+}
+
+/**
+ * Discover all skills from all sources with priority ordering.
+ * Priority order: opencode-project > project > opencode > user
+ * 
+ * @returns Array of LoadedSkill objects for use in slashcommand discovery
+ */
+export function discoverAllSkills(): LoadedSkill[] {
+  const opencodeProjectDir = join(process.cwd(), ".opencode", "skill")
+  const projectDir = join(process.cwd(), ".claude", "skills")
+  const opencodeGlobalDir = join(homedir(), ".config", "opencode", "skill")
+  const userDir = join(getClaudeConfigDir(), "skills")
+
+  const opencodeProjectSkills = loadSkillsFromDir(opencodeProjectDir, "opencode-project")
+  const projectSkills = loadSkillsFromDir(projectDir, "project")
+  const opencodeGlobalSkills = loadSkillsFromDir(opencodeGlobalDir, "opencode")
+  const userSkills = loadSkillsFromDir(userDir, "user")
+
+  return [...opencodeProjectSkills, ...projectSkills, ...opencodeGlobalSkills, ...userSkills]
+}