feat(plugin-skills): integrate plugin skills into skill system

[devkade]

Summary

• Solve Plugin skills not visible in skill/slashcommand tools despite loading successfully #537, Feature Request: Support Claude Code plugins by auto-loading content from .claude/plugin. #117
• Add plugin scope to skill system, enabling external plugins to register skills
• Plugin skills can be resolved via sisyphus_task and agent skill injection
• Plugin skills take highest priority (scope priority 7) over all other skill sources

Changes

Types - Add plugin scope

Files: src/features/opencode-skill-loader/types.ts, src/tools/slashcommand/types.ts

Added plugin to SkillScope and CommandScope union types.

- export type SkillScope = "builtin" | "config" | "user" | "project" | "opencode" | "opencode-project"
+ export type SkillScope = "builtin" | "config" | "user" | "project" | "opencode" | "opencode-project" | "plugin"

Skill Resolution - Support plugin skills lookup

File: src/features/opencode-skill-loader/skill-content.ts

resolveSkillContent and resolveMultipleSkills now accept pluginSkills option to resolve plugin-provided skills.

export interface SkillResolutionOptions {
  gitMasterConfig?: GitMasterConfig
  pluginSkills?: Map<string, LoadedSkill>  // NEW
}

Skill Merging - Plugin skills priority

File: src/features/opencode-skill-loader/merger.ts

mergeSkills integrates plugin skills with proper priority ordering. Plugin scope has highest priority (7).

const SCOPE_PRIORITY: Record<SkillScope, number> = {
  builtin: 1,
  config: 2,
  user: 3,
  opencode: 4,
  project: 5,
  "opencode-project": 6,
  plugin: 7,  // NEW - highest priority
}

Agent Utils - Pass plugin skills for skill injection

File: src/agents/utils.ts

buildAgent and createBuiltinAgents now accept and pass pluginSkills parameter for agent skill injection.

Sisyphus Task - Plugin skills in task delegation

File: src/tools/sisyphus-task/tools.ts

Plugin skills available in task delegation. Error messages now include plugin skill names in available list.

const builtinNames = createBuiltinSkills().map(s => s.name)
const pluginNames = pluginSkills ? Array.from(pluginSkills.keys()) : []
const available = [...builtinNames, ...pluginNames].join(", ")

Plugin Loaders - Load plugin skills

Files: src/index.ts, src/plugin-handlers/config-handler.ts

Plugin skills loaded via discoverInstalledPlugins and loadPluginSkillsAsCommands, then passed to relevant functions.

Testing

• ✅ bun run typecheck passes
• Added 8 new test cases for plugin skill resolution in skill-content.test.ts
  • Plugin skill resolution with options
  • Plugin skill not found without options
  • Plugin skill priority over builtin (conflict)
  • Mixed builtin and plugin skills resolution
  • Multiple plugin skills resolution

---

Summary by cubic

Integrates plugin-provided skills into the skill system so plugins can register skills that resolve across agents and tasks and override other sources.

• New Features
  • Added plugin scope to SkillScope and CommandScope.
  • Plugin skills have highest priority (7) in merge order.
  • resolveSkillContent/resolveMultipleSkills accept pluginSkills and check them first.
  • mergeSkills now accepts pluginSkills and merges them with priority.
  • buildAgent/createBuiltinAgents accept pluginSkills for skill injection.
  • Sisyphus task supports plugin skills and shows them in “available” errors.
  • Plugin skills are discovered and loaded, then passed as LoadedSkill map/array where needed.
  • Added tests covering resolution, priority, and mixed builtin+plugin cases.

^{Written for commit 46d7a2e. Summary will update on new commits.}

[github-actions]

All contributors have signed the CLA. Thank you! ✅
_{Posted by the CLA Assistant Lite bot.}

[devkade]

I have read the CLA Document and I hereby sign the CLA

[chatgpt-codex-connector]

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 6a6d1abdbb

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

[cubic-dev-ai]

1 issue found across 9 files

Confidence score: 3/5

• Merging plugin metadata after applying the disable list in src/features/opencode-skill-loader/merger.ts can re-enable previously blocked skills, so user-facing configurations may silently regress.
• The issue is well understood and should be straightforward to fix, but until the merge order changes, the configuration safety net isn’t reliable.
• Pay close attention to src/features/opencode-skill-loader/merger.ts – plugin merge order currently overrides the disable list.

Prompt for AI agents (all issues)

Check if these issues are valid — if so, understand the root cause of each and fix them.


<file name="src/features/opencode-skill-loader/merger.ts">

<violation number="1" location="src/features/opencode-skill-loader/merger.ts:259">
P2: Disabled skills can be reintroduced because plugins are merged after the disable list is applied</violation>
</file>

---

Since this is your first cubic review, here's how it works:

• cubic automatically reviews your code and comments on bugs and improvements
• Teach cubic by replying to its comments. cubic learns from your replies and gets better over time
• Ask questions if you need clarification on any suggestion

_{Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.}

[cubic-dev-ai]

1 issue found across 1 file (changes from recent commits).

Prompt for AI agents (all issues)

Check if these issues are valid — if so, understand the root cause of each and fix them.


<file name="src/features/opencode-skill-loader/merger.ts">

<violation number="1" location="src/features/opencode-skill-loader/merger.ts:264">
P2: Per-entry disables (`entries[name]=false`/`entry.disable`) are applied before plugin skills are merged, so a plugin skill with the same name can be re-added and bypass the disable.</violation>
</file>

_{Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.}