Skip to content

fix: fallback to an existing bundled skills root#55

Merged
dbpolito merged 2 commits intomainfrom
fix/bundled-skills-root-fallback
Mar 25, 2026
Merged

fix: fallback to an existing bundled skills root#55
dbpolito merged 2 commits intomainfrom
fix/bundled-skills-root-fallback

Conversation

@dbpolito
Copy link
Copy Markdown
Contributor

Summary

  • prefer a bundled skills root that already contains skill directories, while still falling back to an existing configured path
  • remove the draft kompass-workflows skill from base and generated config so bundled skill registration matches the current supported set
  • update bundled skill documentation to reflect the current draft-only workflow guidance and fallback behavior

- prefer a bundled skills root that contains skill directories while still falling back to an existing path
- stop enabling the draft kompass-workflows skill in base config and generated artifacts
- update bundled skill docs and readmes to reflect the current draft-only status
Copy link
Copy Markdown

@kompassdev kompassdev bot left a comment

Choose a reason for hiding this comment

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

★★★★☆

The code changes are correct and well-implemented. The fallback logic for bundled skills root is sensible and handles edge cases properly. Found one documentation inconsistency that should be addressed.

@dbpolito dbpolito merged commit 482cb90 into main Mar 25, 2026
1 check passed
@dbpolito dbpolito deleted the fix/bundled-skills-root-fallback branch March 25, 2026 02:52
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