トップページの news セクションの RSS を複数登録できるようにし、PR TIMES の RSS にも対応する

[yasulab]

対応したい PR TIMES のアカウント (CoderDojo Japan)

• Index: https://prtimes.jp/main/html/searchrlp/company_id/38935
• RSS: https://prtimes.jp/companyrdf.php?company_id=38935

関連 Issue/PR

• トップページの news セクションを RSS から自動生成したい #1577
• トップページの news セクションを RSS から自動生成 #1704
• Rakeタスクのリファクタリングをしたい #1716
• news セクションの自動生成に関するテストを追加する #1718
• #1704 の修正（checkout@v4 に更新 & rel 属性削除） #1720
• Rakeタスクのリファクタリング #1724
• Cross-Site Scripting警告の解消 #1740
• Fix: Failed to update news section automatically #1748

対応 PR

• refactor: News rake タスクの統合と一貫性向上 #1759

完了条件

対応が完了すると、以下の news セクションに、

PR TIMES の最新のプレスリリースも表示されるようになる。

https://prtimes.jp/main/html/rd/p/000000016.000038935.html

[yasulab]

@claude Read this description and suggest steps to implement this feature?

[claude]

Claude finished @yasulab's task —— View job

---

タスクリスト

• 現在のニュースセクションの実装を確認
• RSS フィード処理のコードを分析
• 複数 RSS 対応の実装方法を提案
• PR TIMES RSS の統合方法を提案

---

現在の実装分析と実装提案

現在のシステムを分析した結果、複数RSSフィードへの対応と PR TIMES RSS の統合に必要な手順を以下にまとめました。

💡 提案 (より良い実装方法)

1. 複数RSS対応の実装手順

現在の実装構造

• RSS取得処理: lib/tasks/fetch_news.rake で実装
• 単一RSS設定: ハードコードされた https://news.coderdojo.jp/feed/ のみ
• データ保存: db/news.yml ファイル経由でDBへインポート

実装手順

Step 1: RSS設定の外部化 (config/rss_feeds.yml)

production:
  feeds:
    - name: "CoderDojo News"
      url: "https://news.coderdojo.jp/feed/"
      enabled: true
    - name: "PR TIMES - CoderDojo Japan"
      url: "https://prtimes.jp/companyrdf.php?company_id=38935"
      enabled: true
      source_type: "prtimes"  # 特殊処理が必要な場合のフラグ

development:
  feeds:
    - name: "Sample News"
      url: "spec/fixtures/sample_news.rss"
      enabled: true

Step 2: fetch_news.rake の修正 (62-66行目付近)

# 現在のコード（単一RSS）
feed_urls = if Rails.env.test? || Rails.env.staging?
              [Rails.root.join('spec', 'fixtures', 'sample_news.rss').to_s]
            else
              ['https://news.coderdojo.jp/feed/']
            end

# 修正後（複数RSS対応）
config_path = Rails.root.join('config', 'rss_feeds.yml')
rss_config = YAML.safe_load(File.read(config_path))[Rails.env]
feed_urls = rss_config['feeds']
              .select { |feed| feed['enabled'] }
              .map { |feed| feed['url'] }

Step 3: PR TIMES RSS の特殊処理 (fetch_rss_items メソッドの拡張)

def fetch_rss_items(feed_config, logger)
  url = feed_config['url']
  logger.info("Fetching RSS → #{url}")
  
  begin
    rss = safe_open(url)
    feed = RSS::Parser.parse(rss, false)
    
    items = feed.items.map do |item|
      if feed_config['source_type'] == 'prtimes'
        # PR TIMES特有のフィールドマッピング
        {
          'url'          => item.link,
          'title'        => "📢 #{item.title}", # PR識別用の絵文字追加
          'published_at' => item.dc_date&.to_s || item.pubDate&.to_s,
          'source'       => 'PR TIMES'
        }
      else
        item_to_hash(item)
      end
    end
    
    items
  rescue => e
    logger.warn("⚠️ Failed to fetch #{url}: #{e.message}")
    []
  end
end

2. データベーススキーマの拡張提案

PR TIMES などの異なるソースを識別できるよう、news テーブルに source カラムを追加：

# db/migrate/xxx_add_source_to_news.rb
class AddSourceToNews < ActiveRecord::Migration[7.0]
  def change
    add_column :news, :source, :string, default: 'CoderDojo News'
    add_index :news, :source
  end
end

3. ビューでの表示改善

app/views/home/show.html.erb (185行目付近) で、ソース別のスタイリング：

<% @news_items.each do |news| %>
  <a href="<%= news.url %>" class="news-item <%= news.source&.downcase&.tr(' ', '-') %>" target="_blank">
    <%= format_news_title(news) %>
    <% if news.source == 'PR TIMES' %>
      <span class="badge badge-pr">PR</span>
    <% end %>
  </a>
<% end %>

4. GitHub Actions の更新

.github/workflows/daily.yml は変更不要（既存の news:fetch タスクが自動的に複数RSSを処理）

5. テストの追加

spec/lib/tasks/fetch_news_spec.rb に複数RSS処理のテスト追加：

RSpec.describe 'news:fetch' do
  it '複数のRSSフィードを処理できる' do
    # PR TIMES のモックRSSを用意
    # 両方のフィードからデータが取得されることを確認
  end
  
  it 'PR TIMES のRSSフォーマットを正しくパース' do
    # dc:date フィールドなど特殊なフィールドの処理を確認
  end
end

⚠️ 改善推奨 (ベストプラクティス違反)

現在の実装の改善点

1. ハードコードされたRSS URL

   • 問題: fetch_news.rake にURLが直接記載
   • 改善: 設定ファイル化で管理しやすくする

2. 重複記事の処理

   • 現状: URL のみで重複判定
   • 懸念: 複数RSSソースで同じニュースが配信される場合の対応
   • 改善: ソース名も含めた複合キーでの管理

3. エラー処理の強化

   • 現状: 個別のRSS取得エラーは警告のみ
   • 改善: 部分的な成功/失敗をより詳細にレポート

✅ 良い実装 (参考になる箇所)

現在の実装の優れた点

1. 安全なURL処理 (safe_open 関数)

   • HTTP/HTTPSのみ許可
   • SSRFやオープンリダイレクトの防止

2. 増分ID管理

   • 既存の最大IDから連番を生成
   • 重複を避ける仕組み

3. 自動化された更新フロー

   • GitHub Actions による日次更新
   • 変更検知による自動デプロイ

4. ロギング体制

   • ファイルとコンソール両方への出力
   • デバッグしやすい構造

---

実装の優先順位

1. Phase 1 (最小実装)

   • fetch_news.rake の62行目に PR TIMES の URL を追加
   • 基本的な動作確認

2. Phase 2 (設定の外部化)

   • config/rss_feeds.yml による設定管理
   • 環境別の設定対応

3. Phase 3 (表示の改善)

   • ソース別のバッジ表示
   • PR TIMES 記事の視覚的識別

この段階的アプローチにより、リスクを最小限に抑えながら機能を追加できます。

[yasulab]

対応 PR

• refactor: News rake タスクの統合と一貫性向上 #1759

This was already fixed in the PR above.