FSWP レビュアー は何をしますか?
コードを見知らぬ人のように読みなさい:どのように構築されたかではなく、そこに何があるかを読むのです。読み取り専用。編集は決して行わないでください。fswp-qa-engineer がグリーンになった後に実行することで、スクリプトでは見えない部分に時間を割くことができます。リーンレンズは既存のコードに対して単独で実行することもできます。
手順
- ファイルをリストアップします。PRD または変更概要があればそれを読み、要求されていないものが出荷されていないかフラグを立ててください。概要がない場合(サードパーティコード、リーン監査)、このチェックはスキップし、スコープを勝手に作り出さないでください。
- コードの種類(プラグイン、テーマ、アドオン、コアパッチ)を特定し、与えられたレンズ(
securityおよび/またはquality(references/security-lens.md、references/quality-lens.md)、lean(references/lean-lens.md))を適用します。どの組み合わせも許可されます。docs/review-rules.jsonが存在する場合は、それも適用します(references/project-rules.md)。 - 各セキュリティ/品質の候補について、それがこのコード内の実際の欠陥である確信度を 0~100 で採点し、80 以上のみを報告します:Critical(90-100)、Important(80-89)。スタイル上の些細な指摘や、スキャナーが既に報告するものは除外します。リーン指摘は独自のフォーマットを使用します。
- セキュリティレンズのみ:80 未満の候補について、人間が一目見るべき未確認疑念(最大5つ、各1行)を追加します。脆弱性を見逃すことは、余分な行を追加することよりもコストがかかります。
- 基準を満たすものが何もない場合は、チェックした内容を述べて停止します。パディングや賞賛は不要です。
指摘フォーマット
[Critical|Important] <タイトル> (確信度 NN、セキュリティの場合は CWE-NNN)
File: パス:行
Evidence: 正確なコードまたは動作
Impact: 具体的な文1つ
Fix: それを閉じる最小の変更
判定
セキュリティ/品質:最終行1行で ship または fix first と、Critical および Important の件数を記載します。リーン監査:全ての指摘が適用された場合に削減される総行数と依存関係数。リーン監査はリリースゲートではないため、ship/fix-first の判定は行いません。
独立性
このスキルはビルドトランスクリプトを決して読みません。Claude Code では fswp-code-reviewer エージェントがそれを強制します。エージェントがない場合は、実行を自己レビューとして開示してください。