mcpField
F
flutter-code-review

flutter-code-review

github.com/thienqo测试与质量1368 次安装今天更新
代码仓库

关于 flutter-code-review

flutter-code-review 是面向 Flutter 代码变更的审查技能,帮助审查者识别有证据支持、会影响行为或交付的风险。它按风险排序发现,并要求核验、定位和去重问题,而非进行风格巡查或擅自修改代码。

主要功能

  1. 01界定审查范围并追踪变更影响
  2. 02按风险优先级审查
  3. 03按需加载 Flutter 检查清单
  4. 04核验并去重问题
  5. 05提供带文件与行号的审查结果

适用场景

  1. 01审查提交、拉取请求或工作区差异
  2. 02检查 Flutter 生命周期或存储变更
  3. 03评估兼容性与安全相关改动

安装

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

$npx skills add https://github.com/thienqo/flutter-skills --skill flutter-code-review

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

包含的文件

除 SKILL.md 外,安装时一起下载的参考资料和脚本。

evals/
cases.json
references/
flutter-review-checklist.md
review-workflow.md

详细介绍

Flutter 代码审查

找出会实质影响行为、用户、交付或可维护性的缺陷和实现残留。审查应基于证据评估风险,而不是罗列风格问题或暗中重写代码。

确定审查范围

  1. 确定审查对象是请求的差异、提交范围、拉取请求、工作树、已完成的实现还是模块,并明确预期行为。
  2. 阅读仓库说明,并检查足够多的相关代码、测试、生成边界和配置,以了解变更后的行为。
  3. 优先审查变更行;如有必要,可根据证据追踪其数据流和生命周期,检查周边上下文。
  4. 对于独立审查,不要编辑文件、发表评论、批准或合并,也不要擅自扩大到修复工作,除非用户已授权。在获授权的实现工作流中,将发现的问题交给该工作流处理,只修复原始范围内的问题。

按风险审查

优先关注数据丢失、安全暴露、崩溃、结果错误、生命周期异常、并发竞态、无法访问的流程、发布失败、缺少回归覆盖、质量门槛弱化,以及变更引入的残留。之后再考虑会造成实际成本的架构或可维护性问题。

不要报告个人偏好、假设性的重写、未变更的旧问题,或当前代码已处理的框架行为。根据项目使用的 SDK 和已解析依赖项验证 API 与软件包相关判断;如果可用,请使用 Dart 和 Flutter MCP 语义查询与分析,否则直接检查源码和已解析的软件包版本。

将兼容性和安全性声明视为必须举证的事项。将变更后的输入、输出、默认值、架构、退出行为和支持的调用方与基准版本进行比较;仅接受旧输入并不能证明向后兼容。对于脱敏、转义、验证或其他安全敏感转换,应测试处理顺序和边界条件,而不要只依赖正常路径示例。

按需加载参考资料

  • 阅读审查工作流,了解如何收集变更、追踪影响、校准严重性并整理审查结果。
  • 仅在变更涉及相应技术领域时,阅读Flutter 审查清单。

只有当某个领域发生实质变更,或某项判断需要深入验证时,才加载可用的专家资源。审查涉及存储并不意味着必须加载所有质量类技能。

输出

按严重性排序,并以问题发现开头。每项发现都必须指出受影响文件和精确行范围,说明故障路径及影响,并提出适当的修正建议。仅在有帮助时,才将待解答问题和简短总结分成单独部分。

回复前,质疑并去重候选发现。如果某项判断可被守卫逻辑、所有权规则、版本约束、测试或框架保证推翻,就应舍弃。将发现锚定到引入问题或暴露问题的最小变更范围;仅将未变更代码作为佐证。

如果没有可采取行动的问题,请明确说明,并指出有意义的验证缺口。切勿为了让审查显得全面而捏造问题。

审查已完成的实现时,最后应给出有明确边界的结论:完成前必须解决的实质问题、应上报的外部或范围外事项,以及修正后必须重新运行的针对性检查。

来源

评论

支持 Markdown