feat(vibe): migrate to SkillsIntegration from the old prompts-based MarkdownIntegration#2336
Conversation
…ontmatter Switches VibeIntegration from the old prompts-based MarkdownIntegration to SkillsIntegration, adopting the .vibe/skills/speckit-<name>/SKILL.md layout required by Mistral Vibe v2.0.0+. Post-processes each generated SKILL.md to inject `user-invocable: true` so skills are directly callable by users, not just by other agents.
There was a problem hiding this comment.
Pull request overview
Migrates the Mistral Vibe integration from the legacy prompts-based Markdown command layout to the skills-based speckit-<name>/SKILL.md layout required by newer Vibe versions, and post-processes generated skills to add Vibe-specific frontmatter.
Changes:
- Switch
VibeIntegrationto subclassSkillsIntegration, updating config/registrar settings to.vibe/skills/speckit-<name>/SKILL.md. - Add Vibe-specific
post_process_skill_content()and asetup()post-pass to injectuser-invocable: true. - Update the Vibe integration test mixin from Markdown-based tests to skills-based tests.
Show a summary per file
| File | Description |
|---|---|
src/specify_cli/integrations/vibe/__init__.py |
Converts Vibe integration to skills layout and injects user-invocable into generated SKILL.md files. |
tests/integrations/test_integration_vibe.py |
Updates integration test harness to validate Vibe as a SkillsIntegration. |
Copilot's findings
Tip
Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
- Files reviewed: 2/2 changed files
- Comments generated: 2
mnriem
left a comment
There was a problem hiding this comment.
Please address Copilot feedback. If it is truly shareable code between all integrations using skills please pull it up into the SkillIngration
|
Done |
There was a problem hiding this comment.
Copilot's findings
Tip
Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
- Files reviewed: 2/2 changed files
- Comments generated: 1
|
@Fango2007 Please address Copilot feedback |
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
Copilot's findings
Tip
Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
- Files reviewed: 2/2 changed files
- Comments generated: 0 new
|
Thank you! |
|
Thank you. |
PR purpose : Updating Mistral Vibe integration
Since v2.0.0, Mistral Vibe supports custom slash commands through skills system https://github.com/mistralai/mistral-vibe#custom-slash-commands-via-skills
Summary of changes
Switches VibeIntegration from the old prompts-based MarkdownIntegration to SkillsIntegration, adopting the .vibe/skills/speckit-/SKILL.md layout required by Mistral Vibe v2.0.0+.
Post-processes each generated SKILL.md to inject
user-invocable: trueso skills are directly callable by users, not just by other agents.Testing
uv run specify --helpuv sync && uv run pytestAI Disclosure
This PR was drafted with the help of Claude. The code was then manually refined, largely by aligning it with the Claude integration. It also helped me understand the underlying logic and validate the implementation.