Skip to content

feat: add opencode skills support to Makefile#62

Merged
shunkakinoki merged 1 commit intomainfrom
feat/add-opencode-skills-support
Jan 21, 2026
Merged

feat: add opencode skills support to Makefile#62
shunkakinoki merged 1 commit intomainfrom
feat/add-opencode-skills-support

Conversation

@shunkakinoki
Copy link
Owner

@shunkakinoki shunkakinoki commented Jan 21, 2026

Changes

  • Added ~/.config/opencode/skills to SKILLS_TARGET_DIRS in Makefile

Technical 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

  • Verified Makefile syntax
  • Confirmed new directory path follows existing pattern

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.

Copilot AI review requested due to automatic review settings January 21, 2026 22:50
@shunkakinoki shunkakinoki enabled auto-merge (squash) January 21, 2026 22:50
@coderabbitai
Copy link

coderabbitai bot commented Jan 21, 2026

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Summary by CodeRabbit

  • Chores
    • Updated build configuration to support synchronizing skills to an additional configuration directory in the user's home folder.

✏️ Tip: You can customize this high-level summary in your review settings.

Walkthrough

The Makefile is updated to extend the SKILLS_TARGET_DIRS variable by appending a new destination path at $(HOME)/.config/opencode/skills, enabling skills to be synchronized to an additional configuration directory.

Changes

Cohort / File(s) Change Summary
Makefile Configuration
Makefile
Extended SKILLS_TARGET_DIRS variable to include $(HOME)/.config/opencode/skills as a new synchronization destination

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Suggested labels

enhancement

Poem

🐰 A path to the config we add with care,
Where skills shall flourish and be shared,
One line changed, the destination clear,
The rabbit hops—progress is here! 🌿

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch feat/add-opencode-skills-support

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@shunkakinoki shunkakinoki merged commit 46bfb64 into main Jan 21, 2026
7 of 8 checks passed
@shunkakinoki shunkakinoki deleted the feat/add-opencode-skills-support branch January 21, 2026 22:51
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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/skills to 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
Copy link

Copilot AI Jan 21, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

Copilot uses AI. Check for mistakes.
Copy link

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 1 file

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.

1 participant