mcpField
F
fix-review

fix-review

github.com/sickn33测试与质量48.7k186 次安装1 天前更新
代码仓库

关于 fix-review

这是用于审查修复提交的代码审查技能,帮助确认审计发现已解决且未引入回归或安全问题。它指导审查者对照原始发现检查相关代码路径,并记录与确切提交对应的证据。

主要功能

  1. 01对照审计发现检查修复
  2. 02核实根因是否解决
  3. 03检查回归与安全漏洞
  4. 04审查测试覆盖情况
  5. 05记录提交关联证据

适用场景

  1. 01审查安全问题修复提交
  2. 02确认审计问题是否彻底解决
  3. 03查找存在相同根因的相关路径
  4. 04记录未验证的测试与限制

安装

复制下面的命令,在终端中运行。

$npx skills add https://github.com/sickn33/agentic-awesome-skills --skill fix-review

安装后在 Claude Code 中输入/fix-review确认已加载。

详细介绍

修复审查

概述

验证修复提交是否妥善解决了审计发现的问题,且未引入新的错误或安全漏洞。

何时使用此技能

需要验证修复提交是否解决了审计发现的问题且未引入新错误时,请使用此技能。

适用于以下情况:

  • 审查用于解决安全审计发现问题的提交
  • 验证修复未引入新的漏洞
  • 确保代码更改妥善解决了已发现的问题
  • 验证补救措施是否完整且正确

说明

此技能可帮助验证修复提交是否妥善解决了审计发现的问题:

  1. 审查修复提交:分析声称修复了审计发现问题的提交
  2. 验证问题是否解决:确保原始问题得到了妥善解决
  3. 检查回归问题:确认没有引入新的错误或漏洞
  4. 验证修复是否完整:确保已解决发现问题的所有方面

审查流程

审查修复提交时:

  1. 将修复与原始审计发现的问题进行比较
  2. 确认修复针对的是根本原因,而非表面症状
  3. 检查潜在副作用或新问题
  4. 验证测试是否覆盖已修复的场景
  5. 搜索可达的相关路径,查找相同的根本原因;记录已覆盖的内容和仍未验证的内容

证据记录

将审查绑定到确切的完整 head SHA、原始发现问题及受影响范围。记录修复前的触发条件、修复后的行为、实际运行的测试、尚未解决的变体和局限性。无法运行的测试属于未验证,而非通过。仅凭此指令不足以满足仓库 Tessl 或维护者的合并门槛。

最佳实践

  • 结合完整代码库的上下文审查修复
  • 验证针对已修复问题的测试覆盖情况
  • 检查是否存在需要修复的相似模式
  • 确保修复遵循安全最佳实践
  • 记录问题的解决方案

资源

如需了解更多信息,请参阅源代码仓库。

示例

用户请求:

审查用于解决安全审计发现问题的提交。

局限性

  • 仅当任务明确符合上述范围时,才使用此技能。
  • 不要将输出视为针对特定环境的验证、测试或专家审查的替代方案。
  • 如果缺少必要的输入、权限、安全边界或成功标准,请停止并请求澄清。

评论

支持 Markdown