ブログ記事日時修正の実装計画 - レビュー結果

Review Verdict

Changes Requested -- 計画は全体として高品質だが、日付を生表示している箇所の見落としが3件ある。修正後に承認可能。

Constitution Compliance

問題なし。技術的バグ修正であり、ユーザーに有害なコンテンツ変更はない。全ルールに適合。

Issues Found

Issue 1 (Critical): formatDate適用漏れ -- 3ファイルが未対応

計画のStep 4では blog/[slug]/page.tsx と BlogCard.tsx のみを変更対象としているが、以下の3ファイルでも published_at を生テキストとして表示している。これらもformatDate適用が必要。

/home/ena/yolo-web/src/app/page.tsx (L140-141) -- ホームページの最新ブログセクション
```
<time className={styles.blogDate} dateTime={post.published_at}>
  {post.published_at}
</time>
```
/home/ena/yolo-web/src/components/tools/RelatedBlogPosts.tsx (L22) -- ツールページの関連ブログ記事
```
<time className={styles.date}>{post.published_at}</time>
```
/home/ena/yolo-web/src/components/memos/RelatedBlogPosts.tsx (L22) -- メモページの関連ブログ記事
```
<time className={styles.date}>{post.published_at}</time>
```

これらを修正しないと、ホームページやツールページ等で「2026-02-14T07:57:19+09:00」のようなタイムスタンプが生表示される。

対応: Step 4に上記3ファイルを追加し、変更対象ファイル一覧も更新すること。

Issue 2 (Minor): テストファイルの日付形式

/home/ena/yolo-web/src/app/__tests__/page.test.tsx (L11-12) でモックデータが日付のみの形式("2026-02-14")を使用している。ホームページ(page.tsx)にformatDateを適用する場合、このテストのモックデータもISO 8601タイムスタンプ形式に変更するか、formatDateが日付のみの入力もハンドルすることを確認する必要がある。

計画のformatDate修正版は日付のみの入力をそのまま返す処理があるため、テストは現行のままでも動作する。ただし、テストデータが実際の形式と乖離するため、タイムスタンプ形式への更新を推奨する。

対応: テストファイルを変更対象ファイル一覧に追加（推奨だが必須ではない）。

Issue 3 (Informational): formatDate関数の二重記載

計画内でformatDate関数がnaive版と修正版の2つ記載されている。builderが混乱しないよう、修正版のみを残すか、naive版に明確な取消線/注意書きを追加することを推奨。現状でも「修正版」と記載されているため致命的ではないが、明確さの向上のため。

Positive Points

sv-SEロケールによるYYYY-MM-DD形式取得は実用的で正しいアプローチ
JSTタイムゾーン(Asia/Tokyo)を明示的にIntlで指定しており、ビルド環境のTZ設定に依存しない
同一コミット記事の順序根拠が明確に記載されている
seo.ts/sitemap.tsを変更不要と判断した根拠が正確
テストケースで深夜帯のタイムゾーン境界を検証しているのは良い
ロールバック方針が明確

Next Actions

plannerはIssue 1の3ファイルをStep 4および変更対象ファイル一覧に追加
Issue 2のテストファイル対応を判断（推奨だが任意）
修正後、builderへ展開可能（再レビュー不要）