feat(skills): downloads count/sort, update scope, and Skills Hub docs - #91
Open
TreeTreeDi wants to merge 13 commits into
Open
feat(skills): downloads count/sort, update scope, and Skills Hub docs#91TreeTreeDi wants to merge 13 commits into
TreeTreeDi wants to merge 13 commits into
Conversation
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>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Track successful zip deliveries on skills_items.downloads (web download and registry CLI install), surface counts in the market UI, and support sortBy=downloads. Aligns registry stats.downloads with the real counter.
Add Project/Global/Both selection for update (-g/-p, interactive prompt, named-slug both, -y auto-detect), with design docs and domain glossary.
Document install/list/update/uninstall/upload with screenshots, and open the docsify guide from the skills page help icon (same pattern as proxy).
Extract shared count helpers, use MySQL errno for duplicate index, drop as-assertions in scope prompts, and tighten count display defaults.
Move download counting out of buildSkillZip into the registry controller so CLI and web share one seam. Prefer coerceCount and typed mocks over truthy fallbacks and as const in tests.
Contributor
Author
E2E / 手跑验证(截图不进 PR 提交)环境:本地 市场 UI — 下载量排序与展示
列表 · 按下载量排序 列表 · 最终状态 详情 · zip 后 downloads=3 详情 · CLI install 后 downloads=4
|
Remove CONTEXT.md and the update-scope design draft; keep product code only.
Align changed sources with repo prettier/stylelint so GitHub CI passes.
Contributor
Author
本地流水线自查(对齐
|
| 步骤 | 结果 |
|---|---|
yarn prettier |
PASS(仓库内变更文件;本地 .agents/.scratch 未提交) |
yarn eslint |
PASS(0 error,仅既有 warning) |
yarn stylelint |
PASS(修了 skills/style.scss 的 color-hex-case:#fff → #FFF) |
yarn check-types |
PASS |
yarn build |
PASS |
| registry contract + integration | 50 pass |
dt-skill skills.test.ts |
49 pass |
自查修复提交:5657a63 style: fix prettier and stylelint for CI
liuxy0551
reviewed
Aug 4, 2026
| @@ -0,0 +1,17 @@ | |||
| # dt-skill update scope Implementation Plan | |||
| }, | ||
| { | ||
| label: '下载', | ||
| value: String(detail.downloads), |
Collaborator
There was a problem hiding this comment.
String(detail.downloads || 0) ,参考 String(detail.stars || 0) 的
| }, | ||
| { | ||
| label: '下载', | ||
| value: String(detail.downloads), |
Collaborator
There was a problem hiding this comment.
String(detail.downloads || 0) ,参考 String(detail.stars || 0) 的
Match detail downloads fallback to stars (|| 0) and drop the local update-scope plan file from the branch.
Contributor
Author
Review 已跟进
本地流水线:prettier / eslint(改动文件)/ stylelint / check-types / build 均通过后提交。 提交: |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.





摘要
基于
feat_skill(PR #90 相关基线)的增量改动:skill-statsdt-skill update对齐 vercel 的 project / global / both(-g/-p、交互选择、非 TTY 下hasProjectSkills兜底)docs/docsify/.../dt-skill.md)+ 市场帮助入口(skillsHelpDocUrl)buildSkillZip纯打包、测试类型收敛、去掉计数兜底 / 计次 seam 不一致测试计划
dt-skillskills.test.ts(scope 解析)npm run dev+ ego-lite):列表按下载量排序、详情计次/时间、zip 计次 seamupdate→ 选择 Global;update -g命中~/.agentsskillsHelpDocUrl路径(当前 GH 页 404)说明
feat_skill(非 master/dev)CONTEXT.md/ update-scope design 已从本 PR 移除