Skip to content

feat: sync skills directory by default#56

Merged
iHildy merged 5 commits intomainfrom
ian/sync-opencode-skills
Apr 13, 2026
Merged

feat: sync skills directory by default#56
iHildy merged 5 commits intomainfrom
ian/sync-opencode-skills

Conversation

@iHildy
Copy link
Copy Markdown
Owner

@iHildy iHildy commented Apr 13, 2026

Summary

  • sync ~/.config/opencode/skills/ by default alongside other config directories
  • filter skills from extraConfigPaths so there is a single canonical sync location
  • document the default behavior and add focused path-plan coverage

Testing

  • bun run check
  • bun run test

Fixes #40

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request adds the skills directory to the default configuration directories for synchronization, updating the documentation, tests, and path resolution logic. The changes ensure that the skills folder is synced by default and filtered out if manually specified in extraConfigPaths. Feedback was provided to generalize the path filtering logic in src/sync/paths.ts to exclude any default sync item from the extra configuration paths, which would improve maintainability by avoiding hardcoded values.

@jules-relay
Copy link
Copy Markdown

jules-relay bot commented Apr 13, 2026

🤖 Review Jules Relay

I found 1 Gemini suggestion so far.

Type /relay batch to send all suggestions to Jules.

@iHildy iHildy merged commit a627673 into main Apr 13, 2026
3 checks passed
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.

Feature Request: Add support for syncing skills directory

1 participant