Add provider skill discovery and composer picker#2564
Add provider skill discovery and composer picker#2564lintowe wants to merge 5 commits intopingdotgg:mainfrom
Conversation
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ 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 |
ApprovabilityVerdict: Needs human review This PR introduces a new skill discovery system that scans for SKILL.md files across project and user directories, integrates discovered skills into three providers (Claude, Codex, OpenCode), and surfaces them in the composer UI. The scope—new modules, new workflows, contract changes, and cross-provider integration—warrants human review. You can customize Macroscope's approvability policy. Learn more. |
|
how is this different from what we have already? |
I reinstalled the current regular build to see if there's any way to do skills, either via doing # or @ or / in the chat space but I'm not able to trigger any of the skills that I've personally installed for Codex/Claude Code. Either the implementation might not be fully there or I'm completely missing something in the application. I made the PR due to that, and if it's already implemented then I apologize in advance |
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 2 potential issues.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit d9d92f5. Configure here.

Summary
What Changed
/to find matching provider skills alongside commandsScreenshots
Testing
bun fmtbun lintbun typecheckbun run test src/provider/Layers/CodexProvider.test.tsfromapps/serverNote
Medium Risk
Introduces new filesystem-based skill discovery and threads the resulting
skills/slashCommandsinto provider snapshots and the web composer, which could affect provider status payloads and command menu behavior across platforms. Risk is moderated by defensive parsing, deduping, and added tests, but path/home/cwd handling could still surface edge cases.Overview
Adds provider skill discovery by scanning
SKILL.mdfiles in project/user roots and parsing lightweight frontmatter, then attaches these discoveredskillsto provider snapshots.ClaudeandOpenCodeprovider status checks now include discovered skills, and Claude additionally merges skill names into itsslashCommandslist (without overriding native commands).Codexnow filters app/plugin-backed skills fromskills/listresults and fails fast with a clear error when thecodexbinary is not on PATH.Updates the web composer command palette to show provider skills alongside slash commands (including when typing
/), groups them under a separate section, and inserts the correct invocation prefix via a newinvocationPrefixfield added to theServerProviderSkillcontract (backwards-compatible decoding covered by tests).Reviewed by Cursor Bugbot for commit 83c436f. Bugbot is set up for automated code reviews on this repo. Configure here.
Note
Add provider skill discovery and composer picker for Claude and OpenCode providers
SkillDiscovery.tsto parseSKILL.mdfiles from project and user directories into typedServerProviderSkillobjects, with deduplication and error resilience./and$invocation prefixes respectively) into the provider snapshot.ServerProviderSkillschema with an optionalinvocationPrefixfield constrained to'$'or'/'./command menu in the composer now surfaces provider skills alongside slash commands, grouped under a 'Skills' section; skill selection inserts the correct provider-specific prefix.codexbinary is not on PATH.Macroscope summarized 83c436f.