Skip to content

chore: rename skill directory to sth#51

Merged
Svtter merged 2 commits into
mainfrom
chore/rename-skill-to-sth
May 14, 2026
Merged

chore: rename skill directory to sth#51
Svtter merged 2 commits into
mainfrom
chore/rename-skill-to-sth

Conversation

@Svtter
Copy link
Copy Markdown
Contributor

@Svtter Svtter commented May 14, 2026

Summary

  • Rename skills/static-html-preview/skills/sth/ to align directory name with skill name
  • Ensures npx skills add sun-praise/static-html discovers the skill correctly

Test plan

  • Run npx skills add sun-praise/static-html --list to verify skill discovery

🤖 Generated with Claude Code

Align directory name with the skill name for npx skills compatibility.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
@github-actions
Copy link
Copy Markdown

有条件合并

总结

该 PR 将 skills/static-html-preview/ 重命名为 skills/sth/,4 个文件均为纯重命名操作(R100),内容无变更。重命名本身干净正确,与 skill 元数据中的 name: sth 保持一致。

阻塞项

  1. README.md 和 README.cn.md 中安装路径尚未更新 — 两个文件的安装说明仍引用 --path skills/static-html-preview(第 142 行),但该目录已不存在。用户按文档操作会失败。需将两处 static-html-preview 改为 sth

建议项

  1. openspec/changes/ 目录下的历史文档(如 add-localhost-access/tasks.md 等)仍引用旧路径,属于归档记录,可不必修改。但如果项目规范要求统一,也可一并更新。

New%20session%20-%202026-05-14T14%3A07%3A25.729Z
opencode session  |  github run

@github-actions
Copy link
Copy Markdown

发现遗漏

摘要

PR 正确地将 skills/static-html-preview/ 目录重命名为 skills/sth/,4 个文件已重命名且旧目录已删除,skills 内部没有遗留的旧路径引用。但是,仓库的两个 README 文件仍引用旧路径 skills/static-html-preview,这破坏了文档中的安装指引。


MEDIUM

README 文件中的旧路径未更新

  • 描述: README.md:135,142README.cn.md:135,142 仍引用已经不复存在的 skills/static-html-preview 路径。
  • 影响: 用户按照 README 的指引使用 npx skills add sun-praise/static-html --path skills/static-html-preview 将会失败,因为该目录已不存在。这与 PR 本身的目标("确保技能能被正确发现")相矛盾。
  • 建议位置:
    • README.md 第 135、142 行:将 skills/static-html-preview 改为 skills/sth
    • README.cn.md 第 135、142 行:将 skills/static-html-preview 改为 skills/sth

New%20session%20-%202026-05-14T14%3A07%3A32.106Z
opencode session  |  github run

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
@github-actions
Copy link
Copy Markdown

总结

该 PR 将 skills/static-html-preview/ 重命名为 skills/sth/,SKILL.md 中 name: sth 与目录名一致。同时更新了两份 README 的安装说明,将旧版基于 --path 的脚本安装方式改为 npx skills add sun-praise/static-html,不再引用旧路径 static-html-preview。旧目录已删除,skills/ 下仅有 sth/

之前 review 中提到的"README 仍引用旧路径"问题已被本次提交修复(第 2 个 commit 68dcc45),当前代码已无残留的 static-html-preview 引用(除 openspec/changes/ 下的归档历史记录外,属于正常保留的变更日志)。

阻塞项:无

建议项:无

New%20session%20-%202026-05-14T14%3A09%3A36.184Z
opencode session  |  github run

@github-actions
Copy link
Copy Markdown

无遗漏

分析

PR 要求(取自标题和正文):

  1. skills/static-html-preview/ 目录重命名为 skills/sth/
  2. 确保 npx skills add sun-praise/static-html 能正确发现技能

实现检查:

  • skills/sth/SKILL.md — ✅ 重命名完成,frontmatter 中 name: sth 与新目录名一致
  • skills/sth/scripts/bootstrap-repo.sh — ✅ 重命名完成
  • skills/sth/scripts/send-file.sh — ✅ 重命名完成
  • skills/sth/scripts/start-server.sh — ✅ 重命名完成
  • 旧目录 skills/static-html-preview/ 已删除 — ✅ 无残留
  • README.md — ✅ 更新为 skills/sth 路径,安装命令改为 npx skills add sun-praise/static-html
  • README.cn.md — ✅ 同上中文版本
  • 配置文件(json/yaml/yml/toml)中无旧路径引用 — ✅
  • openspec/changes/ 下的历史文档引用的是旧路径,属归档记录,不影响功能,无需修改

PR 范围内的所有要求均已完整实现,无遗漏。

New%20session%20-%202026-05-14T14%3A09%3A39.745Z
opencode session  |  github run

@Svtter Svtter merged commit 425bc6a into main May 14, 2026
3 of 4 checks 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.

1 participant