feat: add Mistral Vibe support#1125
Conversation
|
Note Reviews pausedIt looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the Use the following commands to manage reviews:
Use the checkboxes below for quick actions:
📝 WalkthroughWalkthroughThis PR adds Mistral Vibe ( ChangesMistral Vibe Tool Support
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related PRs
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ 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.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@openspec/changes/archive/2025-05-25-add-mistral-vibe-support/.openspec.yaml`:
- Line 2: The archive directory name "2025-05-25-add-mistral-vibe-support" does
not match the metadata key created: 2026-05-25; either rename the archive
directory to "2026-05-25-add-mistral-vibe-support" to match the created
metadata, or update the created field value to 2025-05-25 if the older date was
intentional—ensure the directory base name and the created metadata key are
identical.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: 4bc1f5e2-4391-47f5-8824-f3bf06610253
📒 Files selected for processing (8)
docs/supported-tools.mdopenspec/changes/archive/2025-05-25-add-mistral-vibe-support/.openspec.yamlopenspec/changes/archive/2025-05-25-add-mistral-vibe-support/design.mdopenspec/changes/archive/2025-05-25-add-mistral-vibe-support/proposal.mdopenspec/changes/archive/2025-05-25-add-mistral-vibe-support/specs/ai-tool-paths/spec.mdopenspec/changes/archive/2025-05-25-add-mistral-vibe-support/specs/vibe-tool-config/spec.mdopenspec/changes/archive/2025-05-25-add-mistral-vibe-support/tasks.mdsrc/core/config.ts
alfred-openspec
left a comment
There was a problem hiding this comment.
Vibe support should land, but this PR is too broad and duplicates #1114. I would keep this to the minimal AI_TOOLS plus docs change, verify the real Vibe skill path semantics from #1114's ~/.vibe/skills / skill_paths note, add a small detection/init regression test if using detectionPaths, and drop the archived change plus new spec corpus from this PR.
f77f63c to
b122191
Compare
|
Per Mistral Vibe documentation, Vibe checks for skills in |
2c35246 to
f1d9685
Compare
alfred-openspec
left a comment
There was a problem hiding this comment.
Thanks for narrowing this down, this is now the right implementation shape to keep over #1114. Before merge, please add one focused regression test that proves Vibe initializes and detects skills under .vibe/skills so the path semantics do not drift.
|
A focused regression test has been added to prevent path semantics drift for Mistral Vibe skill detection. The test verifies that Vibe is correctly detected when the |
|
@tbetous I don't have the permissions to make commits on your fork. But merging the latest main into your branhc should fix the CI error. Can be merged after that! |
Generated by Mistral Vibe. Co-Authored-By: Mistral Vibe <vibe@mistral.ai>
Generated by Mistral Vibe. Co-Authored-By: Mistral Vibe <vibe@mistral.ai>
Generated by Mistral Vibe. Co-Authored-By: Mistral Vibe <vibe@mistral.ai>
Generated by Mistral Vibe. Co-Authored-By: Mistral Vibe <vibe@mistral.ai>
Generated by Mistral Vibe. Co-Authored-By: Mistral Vibe <vibe@mistral.ai>
- Remove detectionPaths from Mistral Vibe to prevent double-nested skills dir - Fix lingma alphabetical position in tool IDs list Generated by Mistral Vibe. Co-Authored-By: Mistral Vibe <vibe@mistral.ai>
Remove archive directory per PR review feedback to keep PR minimal Generated by Mistral Vibe. Co-Authored-By: Mistral Vibe <vibe@mistral.ai>
Remove new spec corpus (vibe-tool-config + Mistral Vibe scenario in ai-tool-paths) Generated by Mistral Vibe. Co-Authored-By: Mistral Vibe <vibe@mistral.ai>
Add focused regression test that proves Vibe initializes and detects skills under .vibe/skills so the path semantics do not drift. Generated by Mistral Vibe. Co-Authored-By: Mistral Vibe <vibe@mistral.ai>
Head branch was pushed to by a user without write access
1cf4fdd to
96fc047
Compare
|
@TabishB Thanks for your message. My branch is now up to date with latest main. |
|
Ah ok this does need an updated test for the CI to pass, the small fix as specified by codex: |
|
@tbetous, I might just branch off this and commit the above fix as I want to get it in the next release (You would still be credited for the contribution!) |
|
@TabishB Ok no problem! Let's do this. Sorry for not having fix earlier.👍 Do you want me to close this PR? |
Pull request was closed
Add Mistral Vibe as a supported OpenSpec tool.
This enables users to set up Mistral Vibe through standard OpenSpec workflows:
openspec init --tools vibecreates Vibe skill files automaticallyopenspec updaterefreshes Vibe skills when OpenSpec version changesopenspec statusdetects Vibe skill configurationChanges:
Fixes #417
Summary by CodeRabbit
New Features
Documentation
Tests