Skip to content

fix(web): show provider skills in slash menu#2650

Open
delusionofgrandeur wants to merge 1 commit into
pingdotgg:mainfrom
delusionofgrandeur:fix/slash-menu-provider-skills
Open

fix(web): show provider skills in slash menu#2650
delusionofgrandeur wants to merge 1 commit into
pingdotgg:mainfrom
delusionofgrandeur:fix/slash-menu-provider-skills

Conversation

@delusionofgrandeur
Copy link
Copy Markdown

@delusionofgrandeur delusionofgrandeur commented May 11, 2026

Summary

  • include provider skills when the composer slash-command menu opens
  • keep selection behavior inserting the existing $skill token format
  • add browser coverage for selecting a Codex skill from /

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 $skill token 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

  • ChatComposer.tsx now queries provider skills alongside built-in and provider slash commands when the trigger is /, for both empty and non-empty query strings.
  • ComposerCommandMenu.tsx adds a 'Skills' group in the command menu when skill items are present.
  • Skills are matched via their own search path rather than the fuzzy slash-command search, and inserted into the composer as $skill-name .

Macroscope summarized 2f1ba85.

Include provider skills alongside slash commands when users open the composer / menu, and keep selection behavior inserting the existing $skill token format.
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 11, 2026

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 348ddd05-4a5f-4a8c-9618-74328faf95db

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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.

  • Generate code and open pull requests
  • Plan features and break down work
  • Investigate incidents and troubleshoot customer tickets together
  • Automate recurring tasks and respond to alerts with triggers
  • Summarize progress and report instantly

Built for teams:

  • Shared memory across your entire org—no repeating context
  • Per-thread sandboxes to safely plan and execute work
  • Governance built-in—scoped access, auditability, and budget controls

One agent for your entire SDLC. Right inside Slack.

👉 Get started


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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions github-actions Bot added vouch:unvouched PR author is not yet trusted in the VOUCHED list. size:S 10-29 changed lines (additions + deletions). labels May 11, 2026
Copy link
Copy Markdown
Contributor

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ 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"),
}),
);
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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)
Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit 2f1ba85. Configure here.

@macroscopeapp
Copy link
Copy Markdown
Contributor

macroscopeapp Bot commented May 11, 2026

Approvability

Verdict: 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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:S 10-29 changed lines (additions + deletions). vouch:unvouched PR author is not yet trusted in the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant