Agent Skills
返回列表
代码实现自检

代码实现自检

开发编程 更新于 2026.08.30

将以下提示词粘贴到你的 AI 对话框中：

请根据 https://skillhub.cn/install/skillhub.md，安装 @user_4bc11d61/implementation-self-review。

技能介绍

要解决什么问题

首轮实现往往已经能跑通，但交付前仍常见：文件职责变糊、抽象和配置越堆越多、内部到处做防御性分支、错误语义和日志对不上真实失败、文档还写着「待确认」或旧契约。用户若只点出一个点，同类问题可能在别处重复出现；若把这类讨论当成完整 PR Review 或安全审计，范围也会跑偏。

技能如何工作

本技能定位为紧凑的工程自检，在已有 diff 或实现摘要的前提下，按固定流程走一遍：先界定变更行为与触及文件，再对照仓库里当前生效的协作与架构规则，最后用一组 Gates 逐项提问并先报具体发现。

核心门禁包括：

  • Placement：代码是否让单文件职责不清，或拆文件反而难追踪、难删除
  • Necessity：新类型、字段、抽象是否为当前契约所需，还是未来占位
  • Boundary And Defense：防御是否放在真实入口边界，而非掩盖调用方契约
  • Naming / Error Semantics / Observability：命名是否叠词、错误类别是否匹配失败、边界日志是否带关联上下文且脱敏
  • Configuration And Secret / State And Concurrency / Test Intent：配置旋钮是否必要、状态与幂等是否说清楚、测试是否描述行为而非函数名
  • Code And Documentation Alignment：README.md、design.md、implementation-plan.md 等现状文档是否与实现一致

若用户因反馈触发，还会做 Same-Class Scan：在改代码前扫描同类冗余字段、过度防御、命名漂移、文档/测试不一致。输出顺序为：按严重程度的发现 → 同类扫描 → 建议修复或简化 → 已核对规范 → 已跑/待跑的验证。

适用边界

  • 不要用：用户要完整 PR/代码评审、安全审计/威胁建模、尚无实现可审的架构或需求讨论
  • 修复范围：仅当用户明确要求改实现，或当前任务本身就是实现任务时才动手改代码
  • 无 diff 时：只基于可见摘要评审，避免臆断具体文件
  • 反复修改：同一关切点已修订 5 次及以上时，技能会建议暂停实现，先对齐责任边界与验收标准

使用场景

  • 功能首版已实现、准备合入或交付前，手头有 diff 或变更摘要，想按 Placement、Necessity 等门禁过一遍而非开完整 PR Review。
  • 用户或同事质疑文件职责、拆分方式、命名冗长或内部防御分支过多，需要对照变更做结构化自检并做同类问题扫描。
  • 本次改动涉及默认行为、配置语义或迭代文档（如 design.md、implementation-plan），要在交付前核对现状文档是否与实现一致。
  • 同一责任边界或契约措辞已反复修改多轮，希望在继续改代码前判断是否应暂停并对齐验收与迭代范围。

适合人员

  • 负责小步提交、在合入前习惯自己做一轮紧凑工程自检的后端或全栈工程师
  • 被反馈「抽象过多」「配置旋钮没必要」或职责边界不清，需要用 Necessity、Placement 门禁收敛改动的维护者
  • 仓库用 README、design、implementation-plan 管理迭代，发版前需扫「待确认」「后续实现」等陈旧表述的模块负责人
  • 用户追问错误分类、静默 fallback、幂等与并发不变量，希望按 Error Semantics、State Gate 产出可执行结论的开发者