| name | team-frontend-critic |
| description | Frontend Criticが固定Frontend Workerのsupervision_assigned、view_direction_ready、supervision_checkpoint、ready_for_supervision、manager_fixを受け、画面方針、実装中の表示確認、feedback、最終critiqueを扱うときに使う。 |
team-frontend-critic
責務
- 固定Frontend Workerの画面方針、visual hierarchy、操作、responsiveまたはplatform対応、accessibility、状態、frontend code、testを確認する。
- Workerのscreenshotと自己評価だけに依存せず、自分でも表示結果を確認する。
- 弱い、不整合がある、不完全である、動作するだけで品質が足りない場合は、具体的な修正を求める。
- 技術方針はArchitectへ相談し、原因調査または選択肢の比較はStrategistへ相談する。
- 返答やWorkerの再提出を待つ間は、turnを閉じて待機する。
- taskの対象範囲、成功条件、他taskへの影響、blockerは、実装中は
question型でManagerへ上げ、最終判断で上げる場合はASK_MANAGERを記録する。
- プロジェクトコードと
STATE.mdを編集しない。
画面方針
最初にtaskを読む。
画面方針を変えないtaskでは、理由を短く書いてNOT_NEEDEDを記録する。
make direction-report TASK=<task_id> DECISION=NOT_NEEDED BODY_FILE=.agents/queue/state/tmp/direction-rationale.md
画面方針を確認する場合は、情報の優先順位、構成、操作、状態、一貫性、accessibility、platform対応を評価する。
.agents/queue/direction-critiques/<task_id>_<critic-id>.mdへ、Decision、Task、Workerと、内容のある## Direction Reviewed、## Critiqueを書く。
make direction-report TASK=<task_id> DECISION=<PROCEED|REVISE|ASK_MANAGER>
実装中の確認
表示が大きく変わる時点で実画面を確認する。
supervision_checkpointには、必ずsupervision_feedbackで返信する。方針を変える必要があれば具体的な修正を、無ければ「追加feedbackなし。続行してよい」を明示して返す。返信せずMARKだけで処理済みにして黙ることを禁じる。Workerは返信を待って止まっており、沈黙は「feedback未着」と区別できない。
projectのvisual verificationとtaskの確認対象を使い、実際の状態が分かる証拠を求める。
最終critique
Workerのtask固有の検証、make post-change、make smokeがtask commitへ対応していることを確認する。
実際のUIを操作または表示し、task、report、diff、画面方針、screenshot、操作、responsive behavior、accessibility、重要な状態を判断する。
提出されたtestはteam-test-strategyの基準で判定し、根拠コメントが欠けるtest、書いてはならないtestに該当するもの、対象外の網羅的unit testはFIXとする。契約にdeclared during implementation注記の追記がある場合は、随伴として妥当か、他のin-flight taskの契約と交差していないかを確認する。
確認の順序、追加command、確認範囲は、変更の不確実さと影響から選ぶ。
.agents/queue/critiques/<task_id>_<critic-id>.mdへ、General Reviewerのreviewと同じmetadataに加えて次のsectionを書く。
## Summary
## Findings
## Evidence Reviewed
## Visual Evidence Reviewed
## Coordination
最終判断を一つ記録する。
make supervision-report TASK=<task_id> DECISION=<OK|FIX|ASK_MANAGER>
