Skip to content

Conversation

@pascalandy
Copy link

@pascalandy pascalandy commented Dec 23, 2025

Summary

  • Changed skill discovery path from .opencode/skill/ to .opencode/skills/ (plural).
  • Added support for discovering skills from .claude/skills/ in both project and home directories.
  • Added validation to guide users to use the plural skills/ directory.
  • Updated documentation and tests to reflect these changes.

Context

In version 1.0.192, skills located in .opencode/skill/ (as previously documented) were failing to load for users with large skill libraries (~25 skills).

Solution: Renaming the parent directory to skills/ resolved the issue.

While the Agent Skills spec defines the SKILL.md file, it does not strictly define the parent directory name. This PR standardizes on skills/ to align with Claude-compatible conventions and ensures OpenCode looks in both .opencode/ and .claude/ locations. This fixes the documentation mismatch and the loading issue.

tree

per example, this works as expected as tested in the current version of oc (1.0.192)

.opencode
├── agent
├── command
└── skills
    ├── changelog
    │   ├── references
    │   │   └── changelog_sop.md
    │   └── SKILL.md
    ├── cli-gh
    │   ├── references
    │   │   └── help.md
    │   └── SKILL.md
    └── cli-chezmoi
        ├── references
        │   └── cli_reference.md
        ├── scripts
        │   └── apply_chezmoi.sh
        ├── assets
        └── SKILL.md

Session

https://opncd.ai/share/1XhiOijp

Crap, this share seems broken 😥

@pascalandy pascalandy changed the title feat(skill): change skill path to skills/ and add .claude support feat(skill): change skill path to skills/ Dec 23, 2025
@rekram1-node
Copy link
Collaborator

opencode uses singular for everything else so while we may pick up skills in addition to skill we don't want to disallow skill and we still should read skill.

We can load the claude skills but we need to fix the validation to be less strict

@rekram1-node
Copy link
Collaborator

for now let's not read "skills/", we will pick it up if we allow pluralization of all the other dirs too

@pascalandy
Copy link
Author

pascalandy commented Dec 23, 2025

So meanwhile, I was about to report back..
and I guess the confirmation that a skill triggered is this:

image

It's a bit hard to test as this is not deterministic

I'll use skill then :-p

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants