工程纪律技能包 ben7am1n/dsh-review-skills

dsh-review-skills

面向 DeepSeek Harness 的工程纪律技能包 —— 以捆绑技能提供者的形式交付五套久经考验的流程：代码审查、简化、先计划后执行、测试先行和冲突解决。

概述

dsh-review-skills 将资深工程师凭习惯运用的纪律打包成模型可见的技能。安装后，harness 会通过标准的 ctx.skills 注册表发现五个 SKILL.md 包，并在任务匹配时调用它们。

技能

| 技能 | 使用时机 |
| code-review | 合并前：审查 diff 的正确性、测试覆盖、边界情况、不必要的复杂度和安全性。产出按严重程度排序的发现。 |
| simplify | 变更落地后：找出并移除意外引入的复杂度——更清晰的名字、更少的层次、更少的代码做同样的事。 |
| plan-then-execute | 实现任何非平凡任务之前：写一份计划，获得认可，然后执行。 |
| test-first | TDD 纪律：红 → 绿 → 重构，测试诚实，不造假绿。 |
| resolve-conflict | 当合并或变基遇到冲突时：理解双方，审慎解决，验证。 |

适合谁？

- 想要资深工程师检查清单但不想雇一个的独立开发者。
- 希望跨会话保持一致的审查和规划标准的团队。
- 任何基于 dsh 技能系统构建、想要一份可复制的参考级技能包的人。

兼容性

- 针对 @deepseek-ai/dsh@0.1.0-rc.6 / @deepseek-ai/cordis@^4.0.1 / @deepseek-ai/dsh-skill@^0.1.0-rc.6 构建并验证。
- 最后验证： 2026-08-14。
- 需要挂载 ctx.skills 服务的 dsh 组合（dsh-base 包自带）。
- dsh 处于开发者预览阶段；技能 frontmatter 契约（name / description / whenToUse / 调用标志）可能演变。随附的单元测试固定了本包所针对的格式。

安装 / 卸载

安装到 dsh profile（本地检出）：

cd /path/to/deepseek-harness
pnpm dsh plugin --profile web add /path/to/dsh-review-skills

从 GitHub 安装（源码安装——pnpm 会运行 prepare 脚本，因此需允许一次）：

pnpm dsh plugin --profile web add github:/dsh-review-skills

从 npm 安装（发布后）：

pnpm dsh plugin --profile web add dsh-review-skills

卸载：

pnpm dsh plugin --profile web remove dsh-review-skills

快速开始

安装该包，重启 dsh，然后直接提问：

Use the code-review skill on the current diff before we merge.

或在上下文中按名称提及某个技能：

Plan-then-execute: refactor the auth module, then implement.

harness 会通过 ctx.skills 解析技能，并在模型调用时注入其流程。

配置无。该插件注册其提供程序时无需任何配置；技能及其路由元数据位于 skills/*/SKILL.md 的 frontmatter 中（name、description、whenToUse、disable-model-invocation、user-invocable）。

权限与数据

- 只读： 该插件读取其自身打包的 skills/ 目录；它不写入任何内容，也不触碰任何用户文件。
- 无网络： 全部五个技能均为打包在包内的静态 Markdown。
- 模型可见内容： 根据 harness 技能目录策略，技能正文仅在技能被调用时注入模型上下文。

故障排除

| 症状 | 原因 | 修复方法 |
| 技能无法被发现 | 组合中未挂载 ctx.skills 服务 | 验证 profile 是否加载了 dsh-base（其附带技能服务） |
| 启动时出现 missing YAML frontmatter | 某个打包的 SKILL.md 丢失了其 --- 块 | 重新构建/重新安装；测试（pnpm test）会在发布前捕获此问题 |
| 技能未应用于对话 | 模型判定其不相关 | 明确要求：“use the code-review skill” |
| 技能内容陈旧 | 编辑 skills/ 后仍使用旧的 lib/ | 重新构建（pnpm run build）并重新安装；技能正文从磁盘惰性加载 |

开发

pnpm install
pnpm run build # tsc → lib/
pnpm run test # vitest: frontmatter contract + 5 bundled skills present

结构：

- skills//SKILL.md — 每个目录一个技能；frontmatter 驱动路由，正文承载流程。
- src/skills.ts — frontmatter 解析器及打包的提供程序（惰性加载正文）。
- src/index.ts — 插件入口，在 ctx.skills 上注册提供程序。
- tests/ — 固定 frontmatter 契约，并断言全部五个包均可解析。

设计说明：

- 遵循 dsh 仓库中现有的技能文件约定（@deepseek-ai/dsh-skill 类型：SkillCandidate、SkillDefinition、SkillProvider）——此包沿用了 dsh-skill-badge 的先例。
- 保持每个技能正文聚焦：何时使用、流程步骤、陷阱、验证。试图涵盖一切的技能什么都涵盖不了。

许可证与安全

MIT。请通过仓库的安全公告私下报告安全问题。此包仅附带静态 Markdown；它不执行自身的任何代码。