参加者向けアナウンス機能の追加（基本機能）

[takaishi]

概要

参加者向けアナウンス機能の基本実装を追加するPR。
管理画面でのアナウンス作成と、参加者ダッシュボードへの表示を行う。

変更内容

機能追加

• 参加者向けアナウンスの管理画面（作成・編集・一覧）を追加
• 参加者ダッシュボードへのアナウンス表示を追加
• 参加対象の絞り込みに対応
  • 全員
  • オンライン参加者のみ
  • 現地参加者のみ
  • 先行申込者（アーリーバード）

モデル変更

• Announcement モデルに receiver enum と絞り込みロジックを追加
• Announcement.visible_to(profile) でプロフィール属性に応じたフィルタリングを実装

DB変更

• announcements テーブルに receiver カラムを追加

ドキュメント

• SES配信設計ドキュメント（docs/announcements-ses.md）を追加

テスト

• 管理画面のリクエストspec追加（spec/requests/admin/announcements_spec.rb）
• 参加者ダッシュボードの表示フィルタリングspec追加（spec/requests/attendee_dashboard/announcements_spec.rb）
  • オンライン/現地 × アーリーバード有無の4パターンをすべて検証

備考

• メール送信・SES配信基盤は次PRで追加予定
• Closes [提案]Announcement投稿時の、参加登録者へのメール配信 #2679 の一部（メール送信は次PR）

[jacopen]

@claude このPRをレビューして

[claude]

Claude finished @jacopen's task in 1m 38s —— View job

---

PRレビュー結果

コードレビューを完了しました。全体的に良い実装ですが、いくつか改善が必要な点があります。

---

🔴 Critical Issues

1. enum値の不整合性 (app/models/announcement.rb:2)

enumの値に一貫性がありません:

enum :receiver, { all_attendee: 0, only_online: 2, only_offline: 3, early_bird: 4 }

all_attendee: 0の次がonly_online: 2になっており、1が欠けています。これは将来的なバグの原因になる可能性があります。

推奨修正:

enum :receiver, { all_attendee: 0, only_online: 1, only_offline: 2, early_bird: 3 }

または、既存データとの互換性が必要な場合は、コメントで理由を明記してください。

Fix this →

---

🟡 Important Issues

2. 成功メッセージの誤り (app/controllers/admin/announcements_controller.rb:22, 37)

createとupdateアクションで'Speaker was successfully updated.'というメッセージが使われていますが、SpeakerではなくAnnouncementです:

# Line 22, 37
notice: 'Speaker was successfully updated.'

推奨修正:

# createアクションでは
notice: 'Announcement was successfully created.'

# updateアクションでは
notice: 'Announcement was successfully updated.'

Fix this →

3. バリデーション成功時の潜在的バグ (app/controllers/admin/announcements_controller.rb:21)

createアクションでAnnouncement.create(params)の結果をチェックする条件が不適切です:

@announcement = Announcement.create(params)
respond_to do |format|
  if @announcement  # バリデーションエラーがあってもtruthyになる

createはバリデーションエラーがあってもオブジェクトを返すため、if @announcementは常にtrueになります。

推奨修正:

if @announcement.persisted?
# または
if @announcement.save  # ただし、createの代わりにnew + saveを使用

Fix this →

4. N+1クエリの可能性 (app/views/admin/announcements/index.html.erb:17)

ビューでeager_load(:profiles)を使用していますが、Announcementモデルには:profilesのアソシエーションが存在しません:

<% @announcements.eager_load(:profiles).each do |announcement| %>

これは不要なクエリまたはエラーの原因になる可能性があります。

推奨修正:

<% @announcements.each do |announcement| %>

または、必要なアソシエーションがある場合はモデルに追加してください。

Fix this →

5. フィルタリングロジックの潜在的なnilエラー (app/models/announcement.rb:32)

profile.conference.early_bird_cutoff_atがnilの可能性があり、その場合比較でエラーになります:

if profile.created_at < profile.conference.early_bird_cutoff_at

推奨修正:

if profile.conference.early_bird_cutoff_at && profile.created_at < profile.conference.early_bird_cutoff_at

Fix this →

---

🟢 Minor Issues / Suggestions

6. マイグレーションファイルが2つに分かれている理由

20260215130000_add_receiver_to_announcements.rbと20260215133000_remove_default_from_announcements_receiver.rbが分かれていますが、通常は1つのマイグレーションにまとめられます。分ける必要がある場合は、コメントで理由を明記すると良いでしょう。

7. テストカバレッジ

テストは4つのパターン（オンライン/現地 × アーリーバード有無）をすべてカバーしており、素晴らしいです！

ただし、以下のテストケースも追加するとより堅牢になります:

• 非公開アナウンスが表示されないことの確認
• profileがnilの場合の動作（attendee_dashboards_controller.rb:8のフォールバック）
• 異なるカンファレンスのアナウンスが表示されないこと

8. enumのシンボル名の一貫性

モデルではonly_online/only_offlineですが、UI的には「オンライン参加者のみ」「現地参加者のみ」という意味です。より明確な命名としてonline_only/offline_onlyの方が英語として自然かもしれません（ただし、既存コードとの整合性を優先する場合は現状で問題ありません）。

---

✅ 良い点

1. 包括的なテスト: 4つのユースケースをすべてカバーしている
2. スコープの適切な使用: publishedスコープで公開済みアナウンスのフィルタリングが明確
3. ビューの国際化対応: JA_RECEIVER定数で日本語表示をサポート
4. セキュリティ: Strong Parametersで適切にパラメータを制限

---

📝 総評

基本的な実装は良好ですが、Critical IssuesとImportant Issuesは本番環境でバグを引き起こす可能性があるため、マージ前に修正することを強く推奨します。特に:

1. enumの値の不整合（CI作る #1）
2. バリデーションチェックのバグ（Cloud Native Buildpacksでイメージ作れるようにする #3）
3. early_bird_cutoff_atのnil対応（現在放送中のトラック情報を表示するようにする #5）

これらを修正すれば、安全にマージできる状態になります。

---

[github-actions]

Simplecov Report

Covered	Threshold
65.91%	60%

[github-actions]

Review app

• https://dreamkast-dk-2737.dev.cloudnativedays.jp

[jacopen]

全参加者向けと現地参加者向けが出ることを確認

[jacopen]

profile.created_atを弄って先行参加者向けが出ることを確認

[jacopen]

参加方法を弄ってオンライン向けが出ることを確認