feat: add opencode skills support to Makefile#62
Conversation
|
Caution Review failedThe pull request is closed. 📝 WalkthroughSummary by CodeRabbit
✏️ Tip: You can customize this high-level summary in your review settings. WalkthroughThe Makefile is updated to extend the Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Suggested labels
Poem
✨ Finishing touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Pull request overview
This PR extends the Makefile to include opencode editor's skills directory in the automated skill installation and synchronization process. The change adds opencode to the list of supported editors for skills management.
Changes:
- Added
$(HOME)/.config/opencode/skillsto SKILLS_TARGET_DIRS variable
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| SKILLS_SRC_DIR := $(dir $(lastword $(MAKEFILE_LIST)))skills | ||
| SKILLS_RULER_DIR := $(dir $(lastword $(MAKEFILE_LIST))).ruler/skills | ||
| SKILLS_TARGET_DIRS := $(HOME)/.claude/skills $(HOME)/.cursor/skills $(HOME)/.codex/skills $(HOME)/.roo/skills $(HOME)/.gemini/skills $(HOME)/.agents/skills $(HOME)/.vibe/skills | ||
| SKILLS_TARGET_DIRS := $(HOME)/.claude/skills $(HOME)/.cursor/skills $(HOME)/.codex/skills $(HOME)/.roo/skills $(HOME)/.gemini/skills $(HOME)/.agents/skills $(HOME)/.vibe/skills $(HOME)/.config/opencode/skills |
There was a problem hiding this comment.
There's an inconsistency in the opencode path configuration. Line 8 uses $(HOME)/.config/opencode/command (singular "command") while this change adds $(HOME)/.config/opencode/skills (plural "skills"). The directory name should be consistent - either both should be singular or both should be plural. Please verify the correct directory structure for opencode and ensure consistency across COMMANDS_TARGET_DIRS and SKILLS_TARGET_DIRS.
Changes
~/.config/opencode/skillsto SKILLS_TARGET_DIRS in MakefileTechnical Details
This extends the skill installation targets to include the opencode editor's skills directory, ensuring skills are automatically copied to opencode's configuration location.
Testing
Generated with opencode by claude-sonnet
Summary by cubic
Added opencode skills support by adding ~/.config/opencode/skills to SKILLS_TARGET_DIRS. Make targets now copy skills into opencode’s config alongside other editors.
Written for commit 9c00f82. Summary will update on new commits.