fix(skills): Fix skill tagging inconsistency between $ and / across providers#2470
fix(skills): Fix skill tagging inconsistency between $ and / across providers#2470AntonVishal wants to merge 1 commit 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 enables new user-facing functionality (the $ skill picker for Claude) by populating skills data that wasn't previously present. While the code is straightforward, enabling new capabilities warrants human review. You can customize Macroscope's approvability policy. Learn more. |
What Changed
Fixed inconsistent skill tagging between $ and /
Enabled $ parsing for Claude (was not working)
Aligned behavior so multiple skill tags work consistently across providers
Checklist
Note
Fix skill tagging inconsistency by deriving Claude provider skills from slash commands
claudeSlashCommandsAsProviderSkillshelper inclaudeSlashCommandsAsProviderSkills.tsthat maps slash commands toServerProviderSkillobjects with stable, URL-encoded paths (claude-agent:///slash-command/<name>).checkClaudeProviderStatusinClaudeProvider.tsto pass the derived skills array tobuildServerProvider, replacing prior inconsistent tagging behavior.enabled=truewithscope="claude-agent", anddescription/shortDescriptionare populated from slash command fields when present.Macroscope summarized 633533f.