Skip to content

feat(skills): Skills 市场 UI 优化与搜索防抖 - #90

Open
liuxy0551 wants to merge 5 commits into
masterfrom
feat_skill
Open

feat(skills): Skills 市场 UI 优化与搜索防抖#90
liuxy0551 wants to merge 5 commits into
masterfrom
feat_skill

Conversation

@liuxy0551

Copy link
Copy Markdown
Collaborator

Bug 修复

  • 修复上传 skill 文件数量超过 10 个时报错 Reach files limit,multipart files 限制从 10 提升到 50
  • 修复搜索防抖失效导致重复请求的问题
  • 修复搜索防抖后分页、排序、分类筛选不触发请求的问题

功能优化

  • 搜索框增加 300ms 防抖,减少无效 API 请求
  • 详情页"下载 .zip"按钮右侧增加复制链接按钮(hover 提示"复制链接")
  • 所有命令展示统一为终端风格(BASH 标签 + $ 提示符)
  • 简化 Human 安装区域为单个命令,移除"先安装/再安装"步骤
  • 移除"安装方式"区域的 SOON 徽章
  • 最近更新时间格式从 YYYY/M/D 改为 YYYY-MM-DD HH:mm:ss

代码清理

  • 删除未使用的 renderInlineCommand 函数
  • 删除孤儿 CSS 选择器 .human-command-card、.human-command-title、.install-soon-badge

改动文件

  • config/config.default.js — multipart files 配置
  • app/web/pages/skills/index.tsx — 搜索防抖 + 分页修复
  • app/web/pages/skills/detail/components/SkillInstallPanel.tsx — 安装面板 UI
  • app/web/pages/skills/detail/style.scss — 样式清理
  • app/web/pages/skills/detail/utils/skillDetailUtils.ts — 时间格式化

liuxy0551 and others added 4 commits August 4, 2026 10:32
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
- Add copy link button next to download .zip button with tooltip
- Use terminal style (BASH) for all command displays
- Simplify human install section to single command
- Remove SOON badge from install section

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
- Add lodash debounce (300ms) to search input to prevent excessive API calls
- Fix pagination/sort/category not fetching by calling fetchSkills in updateQueryAndFetch
- Remove dead code renderInlineCommand function
- Remove orphaned CSS selectors .human-command-card and .human-command-title

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@TreeTreeDi

Copy link
Copy Markdown
Contributor

+1

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants