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.
| skill.description ?? | ||
| (skill.scope ? `${skill.scope} skill` : "Run provider skill"), | ||
| }), | ||
| ); |
There was a problem hiding this comment.
Duplicated skill-to-menu-item mapping logic
Low Severity
The skill-to-ComposerCommandItem mapping (building the object with id, type, provider, skill, label, description) is duplicated verbatim between the slash-command branch and the skill trigger branch within the same useMemo. If the item shape or description fallback logic ever needs updating, both sites must be changed in lockstep, risking divergence.
Additional Locations (1)
Reviewed by Cursor Bugbot for commit 2f1ba85. Configure here.
ApprovabilityVerdict: Approved Small UI enhancement that surfaces existing provider skills in the slash command menu. Changes are self-contained with test coverage, and the only review comment is a low-severity code duplication note (not a bug). You can customize Macroscope's approvability policy. Learn more. |


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 composition change limited to chat composer command list/grouping, plus a browser test; no auth, data, or backend behavior changes.
Overview
The composer
/command menu now includes provider-detected skills alongside built-in and provider slash commands, and will filter them using the same query text.The command menu grouping logic is updated to show a separate Skills section when the slash menu is grouped, and a new browser test asserts that selecting a skill from
/inserts the existing$skilltoken format.Reviewed by Cursor Bugbot for commit 2f1ba85. Bugbot is set up for automated code reviews on this repo. Configure here.
Note
Show provider skills in the slash-command menu of the chat composer
/, for both empty and non-empty query strings.$skill-name.Macroscope summarized 2f1ba85.