feat(web): render skill calls as inline chips#2572
feat(web): render skill calls as inline chips#2572juliusmarminge merged 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: Approved This PR adds purely presentational rendering logic to display skill tokens as styled chips. The changes are self-contained, thread existing data through components, and have no business logic or security implications. You can customize Macroscope's approvability policy. Learn more. |
6eeb6b1 to
e5fe707
Compare
What Changed
when you send a message that contains a valid $skill snippet, it gets rendered as such
Why
so it looks more cohesive and nice
UI Changes
before


after
Checklist
Note
Render skill
$<name>tokens as inline chips in chat messagesSkillInlineTextcomponent that scans text for$<skillName>tokens and replaces matches with styled skill chips, skippingcodeandaelements in markdown contexts.skillslist fromChatViewthroughMessagesTimelineand into both user and assistant message renderers.SKILL_CHIP_ICON_SVGconstant intocomposerInlineChip.tsso both the composer and the new chip component use the same icon markup.Macroscope summarized e5fe707.
Note
Low Risk
Low risk UI-only change that affects message rendering; main potential issues are incorrect regex matching or performance from per-render token scanning.
Overview
Adds inline rendering of valid
$skilltokens as styled chips across the chat UI.Assistant Markdown rendering (
ChatMarkdown) now accepts askillslist and rewrites text nodes in paragraphs/list items to replace$skillmentions, while skipping links and code. The message timeline passes active provider skills down and wraps user-message text segments similarly, including around inline terminal-context labels.Refactors chip styling assets by moving the skill icon SVG constant to
composerInlineChip.ts, and introduces a sharedSkillInlineText/renderSkillInlineMarkdownChildrenhelper to keep chip rendering consistent.Reviewed by Cursor Bugbot for commit e5fe707. Bugbot is set up for automated code reviews on this repo. Configure here.