commentsテーブルの重複したインデックスを削除

[ryufuta]

Issue

• commentsテーブルに同じインデックスが別名で重複して作成されている #9473

概要

commentsテーブルのuser_idカラムに対してcomment_user_idとindex_comments_on_user_idの2つのインデックスが存在していた。
comment_user_idはマイグレーションファイルを使用せずに追加された可能性が高い。
詳細はIssueを参照。

• 冗長なインデックスを削除する
• マイグレーションファイルとスキーマファイルを整合させる

ためにcomment_user_idのインデックスを削除するマイグレーションファイルを作成した。

変更確認方法

UI、振る舞い、ロジックの変更はない。

Files changedのdb/schema.rbのdiffを見てcomment_user_idのインデックスだけが削除されていることを確認する。

念のため、commentsテーブルについてマイグレーションファイルとスキーマファイルが整合していることを以下の手順で確認する。

1. chore/remove-duplicate-comment-user-indexをローカルに取り込む
2. マイグレーションを最初からやり直す

bin/rails db:migrate:reset

3. db/schema.rbのcommentsテーブルの箇所に変更が生じていないことを確認する

※commentsテーブル以外の不整合は本PRでは解消していないため変更自体は多数生じることに注意。
※確認後は以下の手順でローカルのDBをあるべき状態に戻す。

1. git restore .で変更を戻す
2. bin/rails db:resetを実行する

Summary by CodeRabbit

リリースノート

• その他（Chores）
  • データベーススキーマの最適化を行いました。冗長なインデックスを整理してスキーマを更新し、システムのパフォーマンスと保守性を向上させます。

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

📝 Walkthrough

ウォークスルー

コメントテーブルの comment_user_id という重複インデックスを削除するRailsマイグレーションが追加され、スキーマのActiveRecordバージョンが更新されました。マイグレーションは if_exists: true を使用しています。（50語以内）

変更

コホート / ファイル(複数)	変更サマリー
データベースマイグレーション db/migrate/20260107094500_remove_comment_user_id_index_from_comments.rb	新規マイグレーションクラスを追加。comments.user_id のインデックス comment_user_id を if_exists: true で削除する change メソッドを定義。
スキーマファイル db/schema.rb	ActiveRecordスキーマバージョンを 2026_01_07_000002 から 2026_01_07_094500 に更新。comment_user_id インデックス定義を削除し、index_comments_on_user_id のみを残す変更を反映。

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 分

関連する可能性のある課題

• commentsテーブルに同じインデックスが別名で重複して作成されている #9473: 重複する comments.user_id インデックス comment_user_id を削除する目的と一致しているため関連する可能性あり。

ポエム

🐰 ぴょんと一声、トン

古い索引をふわり取り除き、
データは軽やかに跳ねる、
スキーマは新しい朝を迎え、
ぴょんぴょん進む、コードの野原へ 🌱

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Title check	✅ Passed	タイトルはcommentsテーブルの重複したインデックス削除という主要な変更を正確に説明しており、簡潔で明確である。
Description check	✅ Passed	プルリクエストの説明は必須セクション（Issue、概要、変更確認方法）をすべて含んでおり、変更内容と確認手順が詳細に記載されている。

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

• 📝 Generate docstrings

---

📜 Recent review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 8229440 and 5816fd2.

📒 Files selected for processing (2)

• db/migrate/20260107094500_remove_comment_user_id_index_from_comments.rb
• db/schema.rb

🚧 Files skipped from review as they are similar to previous changes (2)

• db/schema.rb
• db/migrate/20260107094500_remove_comment_user_id_index_from_comments.rb

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)

• GitHub Check: build_and_test

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[ryufuta]

@sharoa119
以前別のPRで躓いていたマイグレーションの不整合の一部を直したものです。
簡単なPRなのでお手隙の際にレビューしていただけるとありがたいです🙏

[sharoa119]

@ryufuta さん
お疲れ様です！
かしこまりました！
しばらくお待ちくださいませ🙂

[sharoa119]

@ryufuta さん
お疲れ様です。
お待たせしてしまいすみません💦
確認させていただき、問題なかったためapproveさせていただきます✨
不整合のご対応をしてくださり、ありがとうございます！

[ryufuta]

@sharoa119
お忙しい中レビューいただきありがとうございました🙏

[ryufuta]

@komagata
レビューをお願いします🙏

[komagata]

確認させていただきました。OKです〜🙆‍♂️