Agent Skills › 7-e1even/learn-agent

7-e1even/learn-agent

GitHub

提供代码审查清单，指导先读需求再审diff。覆盖正确性、边界条件、错误处理及测试盲区四大维度，强调并发与资源释放。要求按严重度输出具体行号结论，避免主观评价，确保审查全面且可执行。

2 个 Skill 150

GitHub

安装全部 Skills

npx skills add 7-e1even/learn-agent --all -g -y

更多选项

预览集合内 Skills

npx skills add 7-e1even/learn-agent --list

集合内 Skills (2)

code-review-checklist

用户请求审查代码改动或PR 用户提供diff内容要求评审

s10_prompt_assembly/skills/code-review-checklist/SKILL.md

查看 Skill 详情 ›

npx skills add 7-e1even/learn-agent --skill code-review-checklist -g -y

SKILL.md

Frontmatter

{
    "name": "code-review-checklist",
    "description": "审查代码改动（review diff \/ PR）时使用——按固定清单过正确性、边界、错误处理和测试盲区，避免只看顺眼不顺眼。"
}

代码审查清单

审查的第一原则：先读需求再读 diff。不知道这次改动想达成什么，就只能审出格式问题。

正确性（最高优先级）

改动是否真的解决了它声称要解决的问题？构造一个具体输入在脑中跑一遍。
有没有"顺手"改了不相关的行为？每一行改动都应该能追溯到本次目标。
并发/重入：这段代码被同时调用两次会怎样？

边界

空集合、空字符串、null/undefined、0、负数、超长输入——逐个问"这里会怎样"。
循环的第一次和最后一次迭代是否和中间行为一致？
时区、编码（UTF-8 BOM、CRLF）、路径分隔符这类"在我机器上没问题"的经典来源。

错误处理

失败路径是吞掉、抛出还是返回错误值？和周围代码的约定一致吗？
报错文案是否包含足够上下文（哪个文件、哪个参数、期望什么）让人照做就能修？
资源（文件句柄、子进程、定时器）在错误路径上是否也被释放？

测试盲区

新增分支有没有对应测试？没有的话，是"难测"还是"忘了"？
测试断言的是行为还是实现细节？断实现细节的测试会在无害重构时误报。

输出格式

按严重度分组给结论：必须改（正确性/安全）→ 建议改（可维护性）→ 可选（风格）。每条指出具体行号和理由，不说"感觉不太好"这种无法执行的话。

git-commit-convention

指导如何编写符合 Conventional Commits 规范的 Git 提交信息，涵盖格式、type 选择、footer 规则及拆分原则，并提供提交前自查清单。

用户要求生成 git commit message 用户询问如何规范提交代码用户需要整理或重写提交历史

s10_prompt_assembly/skills/git-commit-convention/SKILL.md

查看 Skill 详情 ›

npx skills add 7-e1even/learn-agent --skill git-commit-convention -g -y

SKILL.md

Frontmatter

{
    "name": "git-commit-convention",
    "description": "写 git 提交信息、整理提交历史时使用——Conventional Commits 的格式、类型选择和拆分原则。"
}

Git 提交规范

格式

<type>(<scope>): <subject>

<body>

<footer>

type 必选，scope 可选（改动涉及的模块/目录名），subject 必选。
subject 用祈使句（"add" 不是 "added"），不加句号，不超过 50 字符。
body 讲为什么改，不复述 diff 里能看到的"改了什么"。
一次提交只做一件事：重构和功能修改不混在同一个提交里。

type 怎么选

type	用于
feat	新功能（用户可感知的行为变化）
fix	修 bug（引用 issue 编号写进 footer）
refactor	不改行为的结构调整
test	只增改测试
docs	只改文档/注释
chore	构建、依赖、CI 等工程杂务

拿不准 feat 还是 fix：问"用户之前的预期是什么"——之前就该这样但没做到 = fix；之前没有这个预期 = feat。

footer

不兼容变更：以 BREAKING CHANGE: 开头，说明迁移方法。
关联 issue：Closes #123。

提交前自查

git diff --staged 过一遍：有没有混进无关文件（调试代码、锁文件意外变更）？
这个提交单独 revert 时，代码库还能编译/测试通过吗？不能 → 拆分方式有问题。
subject 能让三个月后的自己一眼看懂吗？

7-e1even/learn-agent

7-e1even/learn-agent

安装全部 Skills

集合内 Skills (2)

code-review-checklist

代码审查清单

正确性（最高优先级）

边界

错误处理

测试盲区

输出格式

git-commit-convention

Git 提交规范

格式

type 怎么选

footer

提交前自查

文库