fix(web): show provider skills in slash menu#2650
Conversation
Include provider skills alongside slash commands when users open the composer / menu, and keep selection behavior inserting the existing $skill token format.
|
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)
Tip 💬 Introducing Slack Agent: The best way for teams to turn conversations into code.Slack Agent is built on CodeRabbit's deep understanding of your code, so your team can collaborate across the entire SDLC without losing context.
Built for teams:
One agent for your entire SDLC. Right inside Slack. 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.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 2f1ba85. Configure here.
ApprovabilityVerdict: Approved This is a low-risk UI enhancement that surfaces existing provider skills in the slash command menu. The changes are self-contained to the chat composer, include a helper function extraction (refactor), and add test coverage for the new behavior. You can customize Macroscope's approvability policy. Learn more. |
Extract provider skill menu item formatting so slash-command and skill triggers share the same ComposerCommandItem shape.
Dismissing prior approval to re-evaluate 847b12a

Summary
Scope
This partially addresses #2637 by surfacing detected Codex/provider skills from the / menu. It does not add Codex CLI-only slash commands such as /help or /status, which do not appear to be exposed through the current provider status shape.
Verification
px --yes bun@1.3.11 run fmt
px --yes bun@1.3.11 run --bun lint (passes with pre-existing warnings)
px --yes bun@1.3.11 run --bun typecheck
px --yes bun@1.3.11 run test:browser -- src/components/ChatView.browser.tsx from �pps/web (76 passed)
px --yes bun@1.3.11 run --bun test -- src/components/chat/MessagesTimeline.test.tsx from �pps/web (6 passed)
Full-suite note
px --yes bun@1.3.11 run --bun test hit unrelated 5s timeout flakes locally under load. First run timed out in @t3tools/oxlint-plugin-t3code; second run timed out in effect-acp; --concurrency=1 later timed out in MessagesTimeline.test.tsx, which passed in isolation.
Note
Low Risk
Low risk UI/menu behavior change limited to chat composer command suggestions and grouping; main risk is minor UX regressions in command ordering/filtering.
Overview
The chat composer
/menu now also surfaces provider skills (in addition to built-in and provider slash commands), using provider-skill search and keeping insertion behavior as the existing$skill-nametoken.The command menu grouping logic is updated to include a Skills section when opened via
/, and a browser test is added to verify selecting a skill from the slash menu inserts the expected token.Reviewed by Cursor Bugbot for commit 847b12a. Bugbot is set up for automated code reviews on this repo. Configure here.
Note
Show provider skills in the slash-command menu of ChatComposer
/) menu in ChatComposer.tsx, both with and without a search query, alongside existing built-in and provider slash-command items.toSkillComposerCommandItemhelper to build standardizedComposerCommandItemobjects for provider skills./surfaces a provider skill and inserts the correct token into the composer.Macroscope summarized 847b12a.