Skip to content

fix(skills): market card sticky desc, nested paths + dt-skill Node 18 / 0.18.6 - #89

Merged
liuxy0551 merged 3 commits into
DTStack:masterfrom
TreeTreeDi:feat_dt-skill_node18_0.18.6
Jul 30, 2026
Merged

fix(skills): market card sticky desc, nested paths + dt-skill Node 18 / 0.18.6#89
liuxy0551 merged 3 commits into
DTStack:masterfrom
TreeTreeDi:feat_dt-skill_node18_0.18.6

Conversation

@TreeTreeDi

@TreeTreeDi TreeTreeDi commented Jul 30, 2026

Copy link
Copy Markdown
Contributor

摘要

  • Skills 注册中心 / Web:市场卡片描述默认从 SKILL.md 提取(与 Web 导入共用解析逻辑);内容重发且未传 --description 时保留原有卡片文案;上传时保留嵌套 filePaths;按 installKey 安装时保留远端 slug;技能详情页 hero 摘要不再人为截断折行。
  • dt-skill CLI:锁定运行时依赖(ora / commander / @clack / p-retry / undici)与 Vitest 3.x,使 CLI 可在 Node >=18.17 上运行与测试(与 Doraemon 服务端对齐);包版本升至 0.18.6

提交

  1. fix(skills): align CLI publish with card summary and nested paths
  2. chore(dt-skill): support Node >=18.17 and bump to 0.18.6

测试计划

  • node --test test/skill-utils-normalize-path.test.js test/skills-registry-contract.test.js(或跑主仓库完整测试)
  • cd dt-skill && npm run test:src(Node 18 与 Node 20)
  • 本地 / 注册中心:发布时不带 --description 应保留市场卡片描述;带 flag 应更新
  • 嵌套技能文件上传后 filePaths 正确
  • 按别名 / installKey 安装能解析到预期 slug
  • 测试环境(可选):本分支部署 http://172.16.100.225:7002/

huaiju added 3 commits July 30, 2026 10:49
Default market description from SKILL.md (shared with Web import), keep
existing card on content re-publish unless --description is set, preserve
nested filePaths on upload, keep remote slug on installKey match, and show
full hero summary without artificial wrap.
Pin CLI deps (ora/commander/clack/p-retry/undici) and vitest 3.x so
dt-skill runs and tests on Node 18 aligned with the Doraemon server.
Fix CI prettier check failure on test/skills-registry-contract.test.js.
@liuxy0551
liuxy0551 merged commit 6ed4811 into DTStack:master Jul 30, 2026
1 check passed
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