Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

improvement: add SkillsMP discovery link to Skill Browser dialog#675

Merged
breaking-brake merged 2 commits intomainbreaking-brake/cc-wf-studio:mainfrom
improvement/skill-discovery-linkbreaking-brake/cc-wf-studio:improvement/skill-discovery-linkCopy head branch name to clipboard
Mar 22, 2026
Merged

improvement: add SkillsMP discovery link to Skill Browser dialog#675
breaking-brake merged 2 commits intomainbreaking-brake/cc-wf-studio:mainfrom
improvement/skill-discovery-linkbreaking-brake/cc-wf-studio:improvement/skill-discovery-linkCopy head branch name to clipboard

Conversation

@breaking-brake
Copy link
Copy Markdown
Owner

@breaking-brake breaking-brake commented Mar 22, 2026

Summary

Add an external link to SkillsMP (skillsmp.com) and a "Select Skill" label to the Skill Browser Dialog, matching the MCP server selection pattern for consistency.

What Changed

Before

  • No way to discover new Skills from within the Skill Browser dialog
  • No "Select Skill" label (only dialog title and verbose description)

After

  • "Select Skill" label with "Browse Skills (skillsmp.com)" link, matching MCP dialog layout
  • Dialog description simplified to a single purpose statement

Changes

  • src/webview/src/components/dialogs/SkillBrowserDialog.tsx - Added Select Skill label with SkillsMP link, simplified description
  • src/webview/src/i18n/translation-keys.ts - Added skill.browser.selectSkill and skill.browser.browseSkills keys
  • src/webview/src/i18n/translations/{en,ja,ko,zh-CN,zh-TW}.ts - Added translations, simplified description text

Testing

  • Manual E2E testing completed

🤖 Generated with Claude Code

Summary by CodeRabbit

Release Notes

  • New Features

    • Added a "Browse Skills" discovery link in the Skill Browser Dialog, allowing users to explore available skills through an external marketplace.
  • Localization

    • Updated terminology across all supported languages, replacing "Claude Code Skill" with "Agent Skill" throughout the interface.
    • Added localized labels for skill selection and discovery across English, Japanese, Korean, Simplified Chinese, and Traditional Chinese.

breaking-brake and others added 2 commits March 22, 2026 15:54
- Added external link to skillsmp.com next to "Browse Skills" heading
- Added i18n key skill.browser.browseSkills for all supported languages

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
- Added "Select Skill" label with SkillsMP link matching MCP pattern
- Simplified dialog description to single purpose statement
- Added i18n keys skill.browser.selectSkill and skill.browser.browseSkills

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Mar 22, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: e3330cae-f581-4c14-80dd-6474f4995a6e

📥 Commits

Reviewing files that changed from the base of the PR and between e6f2b6c and 3cf9868.

📒 Files selected for processing (7)
  • src/webview/src/components/dialogs/SkillBrowserDialog.tsx
  • src/webview/src/i18n/translation-keys.ts
  • src/webview/src/i18n/translations/en.ts
  • src/webview/src/i18n/translations/ja.ts
  • src/webview/src/i18n/translations/ko.ts
  • src/webview/src/i18n/translations/zh-CN.ts
  • src/webview/src/i18n/translations/zh-TW.ts

📝 Walkthrough

Walkthrough

This pull request introduces a discovery link in the Skill Browser Dialog that directs users to skillsmp.com, accompanied by support for the new link via localization keys. The implementation includes proper keyboard accessibility and icon styling, with translations added across six languages.

Changes

Cohort / File(s) Summary
UI Discovery Link
src/webview/src/components/dialogs/SkillBrowserDialog.tsx
Added an interactive external link to skillsmp.com with ExternalLink icon, keyboard activation support (Enter/Space), and accessibility attributes (role="button", tabIndex={0}).
Translation Infrastructure
src/webview/src/i18n/translation-keys.ts
Added two new translation key declarations: skill.browser.selectSkill and skill.browser.browseSkills to the WebviewTranslationKeys interface.
Localization Updates
src/webview/src/i18n/translations/en.ts, src/webview/src/i18n/translations/ja.ts, src/webview/src/i18n/translations/ko.ts, src/webview/src/i18n/translations/zh-CN.ts, src/webview/src/i18n/translations/zh-TW.ts
Updated Skill Browser dialog descriptions from "Claude Code Skill" to "Agent Skill" and added translations for the two new UI labels across English, Japanese, Korean, Simplified Chinese, and Traditional Chinese.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Poem

🐰 A link so bright to skills we find,
Where rabbits browse with curious mind,
Five languages whisper, one choice made clear—
Agent Skills await, no need for fear! ✨

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch improvement/skill-discovery-link

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.

@breaking-brake breaking-brake merged commit 7638fdf into main Mar 22, 2026
3 of 4 checks passed
@breaking-brake breaking-brake deleted the improvement/skill-discovery-link branch March 22, 2026 07:08
@breaking-brake breaking-brake mentioned this pull request Mar 23, 2026
braking-brake-semantic-release Bot pushed a commit that referenced this pull request Mar 23, 2026
## [3.30.3](v3.30.2...v3.30.3) (2026-03-23)

### Bug Fixes

* canvas toolbar toggles getting stuck open on rapid hover ([#679](#679)) ([7b42494](7b42494))

### Improvements

* add minimap 3-state display mode with scroll auto-show ([#677](#677)) ([fb26d73](fb26d73))
* add PulseMCP discovery link to MCP server selection ([#674](#674)) ([e6f2b6c](e6f2b6c))
* add SkillsMP discovery link to Skill Browser dialog ([#675](#675)) ([7638fdf](7638fdf))
* add sub-agent discovery link to Sub-Agent Creation dialog ([#676](#676)) ([3523da8](3523da8))
* increase max node limit to 100 with user-configurable setting ([#678](#678)) ([e4318db](e4318db))
@braking-brake-semantic-release
Copy link
Copy Markdown

🎉 This PR is included in version 3.30.3 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

Morty Proxy This is a proxified and sanitized view of the page, visit original site.