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

fix: add refresh button to Skill Browser dialog#163

Merged
breaking-brake merged 1 commit intomainbreaking-brake/cc-wf-studio:mainfrom
fix/skill-browser-refresh-buttonbreaking-brake/cc-wf-studio:fix/skill-browser-refresh-buttonCopy head branch name to clipboard
Nov 27, 2025
Merged

fix: add refresh button to Skill Browser dialog#163
breaking-brake merged 1 commit intomainbreaking-brake/cc-wf-studio:mainfrom
fix/skill-browser-refresh-buttonbreaking-brake/cc-wf-studio:fix/skill-browser-refresh-buttonCopy head branch name to clipboard

Conversation

@breaking-brake
Copy link
Copy Markdown
Owner

Problem

Current Behavior

  1. User opens Skill Browser dialog
  2. ❌ Adding a new skill while dialog is open does not update the list
  3. User must close and reopen the dialog to refresh the skill list

Expected Behavior

  1. User opens Skill Browser dialog
  2. ✅ Clicking the "Refresh" button updates the skill list to the latest state

Solution

Add a refresh button similar to MCP server selection, allowing users to update the skill list without closing the dialog.

Changes

File: src/webview/src/components/dialogs/SkillBrowserDialog.tsx

  • Added refreshing state
  • Added handleRefresh() function
  • Placed refresh button UI below tabs

Files: Translation files (6 files)

  • Added skill.action.refresh key
  • Added skill.refreshing key
  • Added skill.error.refreshFailed key

Impact

  • Improved UX for adding Skill nodes
  • No impact on existing functionality (new feature only)
  • Supported languages: en, ja, ko, zh-CN, zh-TW

Testing

  • Manual E2E testing: Open dialog → Refresh button is displayed
  • Manual E2E testing: Click refresh button → Loading state is shown
  • Manual E2E testing: Add skill then refresh → New skill appears in list
  • Code quality checks passed (format, lint, check, build)

🤖 Generated with Claude Code

- Added refresh button to reload skill list without closing dialog
- Added refreshing state management
- Added i18n support (en, ja, ko, zh-CN, zh-TW)

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
@breaking-brake breaking-brake merged commit 41ac722 into main Nov 27, 2025
3 checks passed
@breaking-brake breaking-brake mentioned this pull request Nov 27, 2025
github-actions Bot added a commit that referenced this pull request Nov 27, 2025
## [2.14.1](v2.14.0...v2.14.1) (2025-11-27)

### Bug Fixes

* add refresh button to Skill Browser dialog ([#163](#163)) ([41ac722](41ac722))
@github-actions
Copy link
Copy Markdown
Contributor

🎉 This PR is included in version 2.14.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@breaking-brake breaking-brake mentioned this pull request Nov 29, 2025
@breaking-brake breaking-brake deleted the fix/skill-browser-refresh-button branch December 3, 2025 11:14
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.