flutter-review-request

请求 Flutter 代码审查

派遣 flutter-craft:flutter-code-reviewer 子代理，在问题扩散前发现它们。

核心原则： 尽早审查，经常审查。

开始时宣布：“我正在使用 flutter-review-request 技能请求代码审查。”

何时请求审查

必须：

• 在子代理驱动开发的每项任务完成后
• 在主要功能完成后
• 合并到 main 之前
• 实施新的 Clean Architecture 层之后

可选但很有价值：

• 遇到瓶颈时（获取新的视角）
• 重构之前（检查基线）
• 修复复杂 Bug 后
• 添加新的状态管理后

如何请求

1. 获取 git SHA

BASE_SHA=$(git rev-parse HEAD~1)  # or origin/main
HEAD_SHA=$(git rev-parse HEAD)

2. 派遣 flutter-code-reviewer 子代理

使用 Task 工具，并指定 flutter-craft:flutter-code-reviewer 类型：

Task tool:
  subagent_type: "flutter-craft:flutter-code-reviewer"
  prompt: |
    ## What Was Implemented
    {WHAT_WAS_IMPLEMENTED}

    ## Requirements/Plan
    {PLAN_OR_REQUIREMENTS}

    ## Git Range to Review
    Base: {BASE_SHA}
    Head: {HEAD_SHA}

    ## Files Changed
    {FILES_CHANGED}

    Please review for:
    - Clean Architecture compliance
    - State management patterns
    - Flutter best practices
    - Test coverage (priority-based)

3. 根据反馈采取行动

问题严重程度	行动
严重	立即修复，然后再继续
重要	在开始下一项任务前修复
次要	记录下来，稍后处理；可以继续

Flutter 专属审查清单

完整清单（Clean Architecture、状态管理、Widget 组合、Dart 风格、基于优先级的测试覆盖，包括 golden 测试）位于审查器本身：agents/flutter-code-reviewer.md。它是唯一的事实来源——不要在此重复；派遣的 Agent 会应用该清单。

示例

[Just completed Task 3: Add AuthBloc]

You: Let me request code review before proceeding.

BASE_SHA=$(git rev-parse HEAD~3)
HEAD_SHA=$(git rev-parse HEAD)

[Dispatch flutter-craft:flutter-code-reviewer subagent]
  WHAT_WAS_IMPLEMENTED: Authentication feature with login/logout
  PLAN_OR_REQUIREMENTS: Task 1-3 from docs/plans/auth-plan.md
  BASE_SHA: a7981ec
  HEAD_SHA: 3df7661
  FILES_CHANGED:
    - lib/features/auth/domain/entities/user.dart
    - lib/features/auth/domain/repositories/auth_repository.dart
    - lib/features/auth/data/repositories/auth_repository_impl.dart
    - lib/features/auth/presentation/bloc/auth_bloc.dart

[Subagent returns]:
  ### Strengths
  - Clean layer separation
  - Proper BLoC state handling
  - Good error states

  ### Issues
  #### Important
  - Missing AuthBloc unit tests (auth_bloc_test.dart)
  - No error boundary in LoginScreen

  #### Minor
  - Consider using freezed for state classes

  ### Assessment
  **Ready to merge: With fixes**
  Fix the Important issues before proceeding.

You: [Add AuthBloc tests, add error boundary]
[Continue to Task 4]

与工作流集成

子代理驱动开发

• 每项任务完成后都要审查
• 在问题累积前发现它们
• 修复后再开始下一项任务

执行计划

• 每批任务（3 项）完成后审查
• 获取反馈、应用反馈，然后继续

临时开发

• 合并前审查
• 遇到瓶颈时审查

警示事项

切勿：

• 因为“这只是简单的 Flutter 代码”而跳过审查
• 忽视严重问题
• 带着尚未修复的重要问题继续
• 跳过对测试反馈的处理

如果审查者有误：

• 提供技术理由进行反驳
• 展示能够证明代码有效的 flutter test 输出
• 请求澄清 Flutter 专属问题

必需的子技能

收到审查反馈后，必须调用： → flutter-craft:flutter-review-receive

继续之前，先系统地处理这些反馈。