关于 flutter-code-review
flutter-code-review 是面向 Flutter 代码变更的审查技能,帮助审查者识别有证据支持、会影响行为或交付的风险。它按风险排序发现,并要求核验、定位和去重问题,而非进行风格巡查或擅自修改代码。
主要功能
- 01界定审查范围并追踪变更影响
- 02按风险优先级审查
- 03按需加载 Flutter 检查清单
- 04核验并去重问题
- 05提供带文件与行号的审查结果
适用场景
- 01审查提交、拉取请求或工作区差异
- 02检查 Flutter 生命周期或存储变更
- 03评估兼容性与安全相关改动
安装
复制下面的命令,在终端中运行。
npx skills add https:/ / github.com/ thienqo/ flutter-skills --skill flutter-code-review安装后在 Claude Code 中输入/flutter-code-review确认已加载。
包含的文件
除 SKILL.md 外,安装时一起下载的参考资料和脚本。
evals/cases.jsonreferences/flutter-review-checklist.mdreview-workflow.md详细介绍
Flutter 代码审查
找出会实质影响行为、用户、交付或可维护性的缺陷和实现残留。审查应基于证据评估风险,而不是罗列风格问题或暗中重写代码。
确定审查范围
- 确定审查对象是请求的差异、提交范围、拉取请求、工作树、已完成的实现还是模块,并明确预期行为。
- 阅读仓库说明,并检查足够多的相关代码、测试、生成边界和配置,以了解变更后的行为。
- 优先审查变更行;如有必要,可根据证据追踪其数据流和生命周期,检查周边上下文。
- 对于独立审查,不要编辑文件、发表评论、批准或合并,也不要擅自扩大到修复工作,除非用户已授权。在获授权的实现工作流中,将发现的问题交给该工作流处理,只修复原始范围内的问题。
按风险审查
优先关注数据丢失、安全暴露、崩溃、结果错误、生命周期异常、并发竞态、无法访问的流程、发布失败、缺少回归覆盖、质量门槛弱化,以及变更引入的残留。之后再考虑会造成实际成本的架构或可维护性问题。
不要报告个人偏好、假设性的重写、未变更的旧问题,或当前代码已处理的框架行为。根据项目使用的 SDK 和已解析依赖项验证 API 与软件包相关判断;如果可用,请使用 Dart 和 Flutter MCP 语义查询与分析,否则直接检查源码和已解析的软件包版本。
将兼容性和安全性声明视为必须举证的事项。将变更后的输入、输出、默认值、架构、退出行为和支持的调用方与基准版本进行比较;仅接受旧输入并不能证明向后兼容。对于脱敏、转义、验证或其他安全敏感转换,应测试处理顺序和边界条件,而不要只依赖正常路径示例。
按需加载参考资料
- 阅读审查工作流,了解如何收集变更、追踪影响、校准严重性并整理审查结果。
- 仅在变更涉及相应技术领域时,阅读Flutter 审查清单。
只有当某个领域发生实质变更,或某项判断需要深入验证时,才加载可用的专家资源。审查涉及存储并不意味着必须加载所有质量类技能。
输出
按严重性排序,并以问题发现开头。每项发现都必须指出受影响文件和精确行范围,说明故障路径及影响,并提出适当的修正建议。仅在有帮助时,才将待解答问题和简短总结分成单独部分。
回复前,质疑并去重候选发现。如果某项判断可被守卫逻辑、所有权规则、版本约束、测试或框架保证推翻,就应舍弃。将发现锚定到引入问题或暴露问题的最小变更范围;仅将未变更代码作为佐证。
如果没有可采取行动的问题,请明确说明,并指出有意义的验证缺口。切勿为了让审查显得全面而捏造问题。
审查已完成的实现时,最后应给出有明确边界的结论:完成前必须解决的实质问题、应上报的外部或范围外事项,以及修正后必须重新运行的针对性检查。