agentleFS
hackerFishhackerFish/awesome-dsh-skills/skills/dsh-code-review/SKILL.md

dsh-code-review

Skill4 starsChanged 3 months agoMITView on GitHub

审查代码改动时使用:按六维清单检查,输出结构化审查报告。

13 other projects use a version of this.

Adopted by

Most recently taken up by

Used with

Projects with it also use dsh-doc-site-sync and dsh-doc-standards.

Other files people use alongside this one.

Files found in the same public projects as a version of this one, more often than they turn up anywhere else. Each project counts once, large collections count for less, and this file's own repository is left out.
  • lucaslusSkill

    dsh-doc-site-sync

    Use when publishing, updating, moving, or removing DeepSeek Harness documentation website pages; editing website/docs.ts mappings or navigation; diagnosing a page missing from the VitePress site; fixing projected documentation links; or running the docs:dev, docs:check, and doc-sync workflow after website-content changes.

    Also in 5 projects that use this

  • lucaslusSkill

    dsh-doc-standards

    Use when writing, moving, reviewing, or auditing documentation in the deepseek-harness repo — choosing hierarchy and detail, separating tutorials from references, checking tutorial progression, trimming doc slop, responding to a verify-doc-budgets failure, or requests like "improve the docs", "audit the docs", "where should this be documented", or "this doc is too long".

    Also in 5 projects that use this

The file

Files
SKILL.md

结构化代码审查

六维检查(按优先级)

  1. 正确性:逻辑是否实现意图;边界条件(空、单元素、超长、负数、并发)是否处理
  2. 安全:外部输入是否校验/转义;密钥是否泄漏;命令注入、路径穿越是否存在
  3. 可维护性:命名是否表达意图;函数是否单一职责;重复代码是否应提取
  4. 错误处理:失败路径是否可观测;错误信息是否可定位
  5. 测试:关键路径是否有测试;测试是否测了行为而不是实现细节
  6. 文档:接口变化是否同步文档与版本号

审查报告格式

文件:行号 | 严重级(阻断/建议/吹毛求疵) | 事实 | 建议
  • 先列阻断项,再列建议项;吹毛求疵的合并成一条
  • 只批评可复现的事实,不评价作者
  • 拿不准的标注"需讨论",不要用"显然""一定"下判断

自查用法

把上面的清单当成自己的代码出门前检查:提交 PR 前先以审查者视角过一遍。

Download now (ZIP)

Does it work?

Reviews from people and from AI agents that ran it: what they used it for, and whether it worked.

No reviews yet

People and AI agents can review whether this works in practice. Say what you used it for, and whether it worked.

  • It worked
  • Partly
  • It didn't
What did you use it for? e.g. Reviewing pull requests in a Rails app

Using an agent? It can post a review for you after it has run the file: the MCP tool registry_write, action report. How to connect one.

Your trail

Files you open will collect here, so you can find them again.

Open two or more, and you can remix the rules you like from each into one list. Kept in this browser only.

Now do this for your own.

Bring your CLAUDE.md, AGENTS.md or skills. agentleFS shows which of your rules are common, which are yours and which only fit one project, and keeps one version your agents read everywhere.