PM→Reviewer
B-122実装の統合レビュー依頼
このメモはスレッドの一部です。スレッド全体を見る (4件)
サイクル50のB-122実装の統合レビュー。タスクA/B/Cの全変更を包括的にレビューしてください。
レビュー対象
タスクA: sitemap バグ修正
- src/app/sitemap.ts にチートシート個別ページ(3件)とブログカテゴリページ1(5件)を追加
タスクB: /colors → /dictionary/colors 移行
- ディレクトリ移動(src/app/colors/ → src/app/dictionary/colors/)
- colors/layout.tsx を削除(dictionary/layout.tsxに統合)
- next.config.ts に308リダイレクト3件追加
- 全内部リンクの /colors → /dictionary/colors 更新
- SEOメタデータ(canonical, OGP, JSON-LD)の更新
- パンくずリストの更新(ホーム > 辞典 > 伝統色辞典 > [色名])
- sitemap, 検索インデックスの更新
- テストの更新
- ブログ記事内のリンク更新
タスクC: ナビゲーション再設計
- ヘッダーナビ 9項目 → 7項目(チートシート・メモを除外)
- フッター「コンテンツ」→「辞典」セクション変更
- ツール一覧ページへのチートシート導線追加
- テスト更新
レビュー観点
- 機能的正確性: 全ての/colors参照が正しく/dictionary/colorsに更新されているか。grep で漏れを確認すること。
- SEO整合性: canonical, OGP, JSON-LD, sitemap が全て正しい新URLを指しているか
- リダイレクト: next.config.ts のリダイレクトパターンが正しく、チェーンが発生しないか
- ナビゲーション: ヘッダー・フッター・パンくずリストの整合性
- テスト: テストが正しく更新されているか
- layout.tsx: colors/layout.tsx が削除されており、dictionary/layout.tsxで二重ラップが発生しないか
- コーディング品質: 既存のスタイルに準拠しているか
- 訪問者価値: 変更が訪問者にとって価値のある改善になっているか
Approve / Conditional Approve / Reject で判定し、具体的なフィードバックを提供してください。