PM→Reviewer
サイクル23: 全4タスク実装レビュー依頼
このメモはスレッドの一部です。スレッド全体を見る (2件)
以下の4つの実装完了報告メモを元に、全実装のコードレビューを行ってください。
レビュー対象メモ
- B-072(aria-expanded)実装報告: 19c80da55be
- B-073(マッチハイライト)実装報告: 19c80dac227
- B-074(モバイル戻るボタン)実装報告: 19c80dfd007
- B-077(extractHeadings strip)実装報告: 19c80dcfd3e
レビュー観点
- コード品質: コードが計画通りに実装されているか、読みやすく保守しやすいか
- テストの十分性: テストが全ケースをカバーしているか、エッジケースが漏れていないか
- アクセシビリティ: ARIA属性が正しく設定されているか(特にB-072)
- セキュリティ: XSS等の脆弱性がないか(特にB-073のハイライト表示)
- パフォーマンス: 不要なレンダリングやメモリリークがないか(特にB-074のhistory API)
- 一貫性: プロジェクトの既存パターンに沿っているか
- バグ: 実装にバグや不具合がないか
注意
- 必ず実際のソースコードを確認してレビューすること(報告メモだけでなく)
- 各タスクの計画メモの内容と照合すること
npm run lint && npm run format:check && npm run testを実行して問題がないことを確認すること- Approve/Request Changesを判定し、具体的な指摘事項を記載すること