Skip to content

Feature/skills mcp UI standardization - #427

Merged
Loosand merged 13 commits into
Stack-Cairn:mainfrom
Loosand:feature/skills-mcp-ui-standardization
Aug 12, 2026
Merged

Feature/skills mcp UI standardization#427
Loosand merged 13 commits into
Stack-Cairn:mainfrom
Loosand:feature/skills-mcp-ui-standardization

Conversation

@Loosand

@Loosand Loosand commented Aug 11, 2026

Copy link
Copy Markdown
Collaborator

Linked issue

Summary

统一 Skills Hub、MCP Hub 和工作区资源设置的界面与交互体验。

  • 使用共享 Tabs、Sheet、Badge、Checkbox 等标准 UI 组件;
  • 重构 Skills 与 MCP 页面,将大型单文件拆分为职责明确的组件;
  • 统一页面间距、卡片、分类导航、搜索框、开关和状态反馈;
  • 增加模糊搜索与关键词高亮;
  • 优化技能商店目录和详情缓存,避免重复加载;
  • MCP Store 使用每页 24 项的自动触底加载,最多一行三张卡片;
  • 改进重新扫描、本地导入、批量选择、启用和删除等交互;
  • 统一 Markdown 预览与复制反馈。

Change scope

  • Modules: agent-ui / agent-gui / agent-gateway/web
  • Key paths:
    • crates/agent-ui/src/pages/skills-hub/
    • crates/agent-ui/src/pages/mcp-hub/
    • crates/agent-ui/src/components/resources/
    • crates/agent-ui/src/components/ui/
    • crates/agent-ui/src/components/markdown/
    • crates/agent-ui/src/components/chat/WorkspaceResourceSettingsDrawer.tsx
    • crates/agent-ui/src/lib/shared/fuzzySearch.ts
    • crates/agent-gui/test/skills/
    • crates/agent-gui/test/settings/
    • crates/agent-gui/test/tools/mcp-registry.test.mjs
    • crates/agent-gui/src/i18n/config.ts
    • crates/agent-gateway/web/src/i18n/config.ts

Verification

  • pnpm --filter liveagent exec tsc --noEmit: passed
  • pnpm --filter @liveagent/gateway-webui exec tsc --noEmit: passed
  • Targeted GUI regression tests: 85/85 passed
  • pnpm test:webui: 540/540 passed
  • pnpm test:gui: 1548/1548 passed
  • pnpm build:gui: passed
  • pnpm build:webui: passed
  • pnpm lint:gui: passed
  • pnpm lint:webui: passed
  • Targeted Biome checks for the 45 affected shared UI files: passed
  • pnpm check:ui-boundaries: passed
  • git diff --check upstream/main: passed
  • pnpm lint:ui: still reports 3 pre-existing import-order errors in unchanged files; the clean upstream/main@24e268f9 baseline independently reproduces 6 import-order errors.

Pre-submit checklist

  • The change is focused, with no unrelated modifications.
  • No secrets, tokens, or personal data included.
  • Required screenshots or recording have been uploaded.

Screenshots / preview

截屏2026-08-11 23 27 47 截屏2026-08-11 23 17 52 截屏2026-08-11 23 17 59 截屏2026-08-11 23 16 21 截屏2026-08-11 23 16 32 截屏2026-08-11 23 16 52 截屏2026-08-11 23 27 56 截屏2026-08-11 23 28 02 截屏2026-08-11 23 28 16 截屏2026-08-11 23 17 11 截屏2026-08-11 23 17 25 截屏2026-08-11 23 17 35

Loosand and others added 11 commits August 8, 2026 15:41
…component

- Introduced a new Switch component for better UI consistency.
- Implemented a search input in the SettingsPage for filtering navigation items.
- Updated translations for search placeholders and no results messages in both English and Chinese.
- Enhanced the layout and styling of the SettingsPage for improved user experience.
feat(settings): add customizable search functionality and new switch …
# Conflicts:
#	crates/agent-gateway/web/src/pages/SettingsPage.tsx
#	crates/agent-gui/src/pages/SettingsPage.tsx
#	crates/agent-gui/src/pages/settings/SystemSettingsForm.tsx
#	crates/agent-gui/src/pages/settings/shared.tsx
#	crates/agent-ui/src/components/ui/switch.tsx
#	crates/agent-ui/src/pages/settings/SystemSettingsForm.tsx
- Introduced `SettingsGroup`, `SettingsRow`, and `SettingsChoiceRow` components for better structure and reusability in the System Settings UI.
- Replaced existing button implementations with `SettingsChoiceRow` for execution mode selection.
- Updated the layout of the close window behavior and tray settings sections to utilize the new components, improving readability and maintainability.
- Added a new `.document-markdown` class in CSS for improved typography in long-form files and documentation previews.
- Updated translation strings in both `i18n/config.ts` files to provide clearer context for skills-related actions, including bulk selection and scanning feedback.
- Introduced new tests for bulk mode interactions, skill card actions, and scan feedback to ensure proper functionality and user experience.
- Refactored components to utilize shared UI elements, enhancing consistency across the application.
- Introduced a new test suite for MCP Hub tabs, ensuring consistent use of UI components and navigation structure.
- Added tests for the Skills Hub search input to verify focus behavior and styling.
- Implemented tests for the MCP import view, including rescan functionality and UI feedback.
- Refactored existing components to utilize shared UI elements, enhancing consistency across the application.
@StackCairn
StackCairn marked this pull request as draft August 11, 2026 15:38
@github-actions

Copy link
Copy Markdown
Contributor

PR governance checks failed — this PR has been converted to draft.

  • No linked issue: the PR body must contain Closes #123 / Fixes #123 / Resolves #123. This project requires an issue before a PR — see the contribution guidelines.

Fix the items above, then click Ready for review to re-run the checks.

- Added a new import for the CopyButton in the Markdown module test to improve component coverage.
- Updated the MCP Hub tabs test to reflect changes in grid column styling for better layout consistency.
- Refactored the McpServerCard component to utilize the openUrl function for external links, enhancing user experience.
- Cleaned up imports in McpServersForm for better readability and organization.
@su-fen su-fen added the governance-exempt Skip PR governance checks label Aug 12, 2026
…ui-standardization

# Conflicts:
#	crates/agent-ui/src/components/chat/WorkspaceResourceSettingsDrawer.tsx
#	crates/agent-ui/src/components/resources/ResourceActivationSwitch.tsx
@Loosand
Loosand force-pushed the feature/skills-mcp-ui-standardization branch from 4a41890 to 89e5bfb Compare August 12, 2026 12:43
@Loosand
Loosand marked this pull request as ready for review August 12, 2026 12:50
@Loosand
Loosand merged commit b924962 into Stack-Cairn:main Aug 12, 2026
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

governance-exempt Skip PR governance checks

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants