Flutter Code Review 是一个针对 Flutter 与 Dart 代码的审查技能,用途是检查 diff、提交(commits)、Pull Request、聚焦的代码范围或已完成的实现,找出具体缺陷和回归风险。其核心定位是:审查是一次有证据支撑的风险评估,不是风格巡览,也不是隐式改写。
用途与审查边界 该技能用于定位对行为、用户、交付或可维护性有实质影响的缺陷与实现残留。使用前需确定请求的审查边界:是某个 diff、提交范围、Pull Request、工作区、已完成的实现,还是某个模块,以及对应的预期行为。随后阅读仓库说明(repository instructions),并检查足够的周边代码、测试、生成边界与配置,以理解被改动的行为。优先审查改动的行,但当证据需要周边上下文时,跟随其数据流与生命周期进行追溯。
在独立审查场景中,除非用户明确授权,否则不编辑文件、不发表评论、不批准、不合并,也不扩展到修复工作;在已授权的实现工作流中,将发现返回给该工作流,使其只修复原始范围内的议题。
按风险审查 审查按风险优先级进行。优先关注:数据丢失、安全暴露、崩溃、错误结果、生命周期中断、并发竞态、不可访问的流程、发布失败、缺失的回归覆盖、被弱化的质量门禁,以及变更引入的残留问题。之后再考虑具有具体成本的架构或可维护性问题。
不报告偏好、假设性的重写、未改动的遗留问题,或当前代码已处理的框架行为。对 API 与包的相关断言,要针对项目的 SDK 与已解析依赖进行验证,在可用时使用 Dart 与 Flutter MCP 语义查找与分析;否则直接检查源码与已解析的包版本。
将兼容性与安全性断言视为举证义务:对比变更前后的输入、输出、默认值、schema、退出行为与受支持的调用方;仅接受旧输入并不能证明向后兼容。对于脱敏、转义、校验或其他安全敏感转换,要测试顺序与边界条件,而非信任 happy path 示例。
条件性加载参考资料 按需读取审查工作流,用于收集变更、追踪影响、校准严重程度与格式化发现。仅针对变更触及的技术领域读取 Flutter 审查清单。仅当其领域被实质改动或某项断言需要更深入验证时,才加载可用的专家内容;一次触及存储的审查并不自动要求所有质量技能。
输出要求 输出以按严重程度排序的发现(findings)为主。每条发现必须指明受影响的文件与精确的行范围,说明失败路径与影响,并提出相称的修正建议。仅在有用时,将开放问题与简短摘要分列为单独部分。
回应前需质疑并去重候选发现:丢弃任何被守卫条件、所有权规则、版本约束、测试或框架保证所推翻的断言。将发现锚定到引入或暴露它的最小改动范围;仅在作为支撑证据时引用未改动代码。
若无 actionable 发现,应直接说明,并指出有意义的验证缺口。绝不为让审查显得全面而编造发现。
审查已完成实现时,最后给出有界的结果:完成前必须解决的实质发现、需上报的外部或范围外事项,以及修正后必须重跑的聚焦检查。
参考来源 该技能参考 Flutter testing、Flutter performance、Flutter architecture、Effective Dart 以及 Dart and Flutter MCP server。