トップページの news セクションを RSS から自動生成

[nacchan99]

Fixes #1577

この PR でやりたいこと

手作業で更新しているニュース一覧を 、RSS から自動取得して YAML 化し、CI／Actions で定期更新できるようにする

この PR でやること

• 1. 以下の RSS フィードをチェックするタスクを lib/tasks/fetch_news.rake に実装

  • 対象フィード： https://coderdojo.jp/#news（実際には https://coderdojo.jp/feed など）
  • lib/tasks 配下に新規ファイルを作成

• 2. RSS から記事タイトル／公開日／URL を抽出し YAML に追記・更新

  • 出力先：db/news.yml
  • 重複排除：URL を ID として uniq
  • 通常 DB ではなく、コミット可能な YAML 経由で参照・更新できる設計

• 3. 上記タスクを GitHub Actions で毎朝 9:00 JST に実行

  • 定期スケジュール：cron: '0 0 * * *' （UTC0 = JST9:00）
  • db/news.yml を自動コミット

• 4. 別途、CI（例: Heroku リリーススクリプト）で YAML から DB に保存

  • CI ログは外部に公開されない設計
  • script/release.sh に bundle exec rails news:import_from_yaml を追記

動作確認手順

1. ニュースYAMLを空にリセット

  echo 'news: []' > db/news.yml

➡︎db/news.yml が空の配列になります

（必要ならDBのニュースも削除）

bundle exec rails runner "News.delete_all"

➡︎この時点で http://localhost:3000/#news を開くと何も表示されません

2. RSSを取得してYAMLを更新

bundle exec rails news:fetch

➡︎db/news.yml に最新のニュースデータが書き込まれます

3. YAMLをデータベースに取り込む

bundle exec rails news:import_from_yaml

➡︎DB にニュースレコードが登録されます

4. ブラウザで動作確認

• サーバーを起動（または再起動）
• http://localhost:3000/#news を開く
• 最新7件のニュースが表示される（↓2025/07/17時点の最新版です）

テスト環境でのRSS取得確認（←この確認作業が必要かどうか検討中です🙌）

# テスト環境ではサンプルRSSファイルを使用
RAILS_ENV=test bundle exec rails news:fetch
git status  # db/news.yml の更新を確認

期待する結果：サンプルRSSから3件のニュースが取得される

• 🎉 テスト記事③ → そのまま表示（絵文字あり）
• 📰 テスト記事② → 📰が追加（絵文字なし）
• 📰 テスト記事① → 📰が追加（絵文字なし）

なお、GitHub Actions や Heroku でのログ確認については、まだ十分に理解できていないため、今回の動作確認手順には含めておりません🙏

この PR でやらないこと

• 他 Dojo のブログ RSS をまとめて /blogs で表示（拡張案）

[nacchan99]

@yasulab
お忙しいところ、メンション失礼します🙇‍♀️
現在の実装方針とコードに関してご相談させてください🙌

現状、RSSフィードから取得したニュースタイトルには、絵文字が含まれるもの（例: 「🎲 ダイス〜」）と含まれないもの（例: 「DojoLetter〜」）が混在しています。

以下の2点の方針とその実装方法について、意見を伺いたいです🙌

1. 絵文字の表示について:
   • 現在考案中の表示方針:
     • 元の記事タイトルに絵文字が含まれている場合は、その絵文字をそのまま使用して表示する。
     • 元の記事タイトルに絵文字が含まれていない場合は、タイトルの前に「📰」の絵文字を自動的に付与して表示する。
       採用した場合は、このような見た目になります！

この方針で進めて問題ないでしょうか？🙌
（例えば、全てのニュースに同じ絵文字を付ける、あるいは絵文字は付けない、など他の推奨方針があれば知りたいです！）

2. 絵文字の有無判定のコードについて:
   • 上記の方針を実現するため、ビュー（show.html.erb）では、以下のコードでタイトルに絵文字が含まれているかを判定する予定です。（例で挙げているスクリーンショットはこのコードを採用しています⇧）

has_emoji = news.title.chars.any? { |char| char.match?(/[\u{1F000}-\u{1F9FF}\u{2600}-\u{27BF}\u{1F600}-\u{1F64F}\u{1F300}-\u{1F5FF}]/) }

この正規表現を使った判定方法で問題ないでしょうか？
（よりシンプルな方法や、パフォーマンス面で懸念があればご指摘いただきたいです🙌）

お忙しいところ恐縮ですが、お時間がある際にご確認をお願いいたします！ 🙏

[yasulab]

@nacchan99 ご提案ありがとうございます！以下の方針がよさそうですね！(๑˃̵ᴗ˂̵)b✨

• 元の記事タイトルに絵文字が含まれていない場合は、タイトルの前に「📰」の絵文字を自動的に付与して表示する。

---

この正規表現を使った判定方法で問題ないでしょうか？
（よりシンプルな方法や、パフォーマンス面で懸念があればご指摘いただきたいです🙌）

@nacchan99 質問する前にまずは "Ruby 絵文字" で検索 して、よりシンプルな方法がないか調べてみましょう！ 🔍 💨 https://www.google.com/search?q=Ruby+絵文字

[rakuda-san-desu]

一旦実装コード側についてコメントしています！
テストとタスクについては別途共有・相談させてください🙏

[rakuda-san-desu]

2025/07/16 朝会後に共有したこと

• テストについて
  • 絵文字のヘルパーのテストと、/#newsに内容が表示されているかのテストはあると良さそう
  • 他にも必要そう・あると嬉しいテストは考えられるので、一旦別イシューに切り出しておくと良さそう
• news.yml について
  • 現在は常に、最新10件で上書き
  • 追記・更新していきたい
  • 上から、published_at が新しい順に追加されていくと嬉しい
  • YAML ファイル内の id の扱いについてrakuda もわからないので、別途意見を伺う
• actionsのワークフローについて
  • GitHubアクションが実行してくれる作業
  • 毎日9時にタスクを実行
  • PRで、手動で試せると嬉しい←rakudaが確認
• script/release.shについて
  • デプロイ時にHeroku で実行されるコマンドがまとまっている
  • 追加したことでbundle exec rails news:import_from_yaml がデプロイ時に実行される
  • lib/tasks/fetch_news.rake はワークフローで毎日実行されている
  • lib/tasks/import_news.rake も毎日実行したい
    • Heroku で設定できる → 最終的に共有しながらやりましょう🛠️
• 追記: タスクについて
  • 現在は読み込んだ数が出力されている？
  • 更新した数が出力されると嬉しい
• 概要の動作確認手順は以下を確認できるとPRに内容と合いそう
  1. /#news に何も表示されていない
  2. bundle exec rails news:fetch を実行するとnews.ymlが更新される
     （news.yml が最新の場合、実行前に一旦アイテムを削除しておく）
  3. bundle exec rails news:import_from_yaml を実行するとDBに保存される
  4. /#news に最新7件が表示される

[rakuda-san-desu]

@nanophate actionsのワークフローについて、お手隙の際にコメントいただけると嬉しいです🙏

やりたいこと

当PR内で以下の動きを確認できると嬉しいなと考えています。
ワークフロー（.github/workflows/fetch_news.yml）実行 → RSS取得してymlを更新するタスク（lib/tasks/fetch_news.rake）が実行される→更新されたyml（db/news.yml）の内容確認

ワークフローは main ブランチに存在しないと実行できないようなので、以下の方法を考えていますが認識良さそうでしょうか？

1. .github/workflows/fetch_news.yml を別PRに切り出す
2. schedule: 部分はコメントアウトして、手動実行のみにしておく
3. （レビュー後）main にマージ
4. 当PRにmainを取り込む
5. GitHub から当PRで Fetch News ワークフローを実行

[rakuda-san-desu]

f577bcb でブランチでもワークフローが認識されました！
@nanophate さんありがとうございました🙏 ✨

[nanophate]

ワークフロー（.github/workflows/fetch_news.yml）実行 → RSS取得してymlを更新するタスク

  pull_request:
     branches:
       - "*"

上記の方法で、コミットした際にトリガーするように対応。一応、GitHub CLI を活用することで、手元からのトリガーが行えることも可能と確認 gh workflow run 'Fetch News' --ref feat/1577-auto-news-from-rss

[claude]

Claude finished @nacchan99's task —— View job

---

Code Review Complete

Reviewing this PR for code quality, potential bugs, performance considerations, security concerns, and test coverage.

Review Checklist:

• Review GitHub Actions workflow configuration
• Analyze RSS fetching logic and error handling
• Review database schema and model implementation
• Check controller and view changes
• Examine emoji handling logic
• Assess security implications
• Review test coverage and quality
• Check for performance considerations
• Provide overall feedback and recommendations

---

🔍 Detailed Code Review

✅ Strengths

1. Well-structured implementation - Good separation between RSS fetching, data processing, and display
2. Environment-aware configuration - Properly handles test vs production RSS sources
3. Atomic operations - YAML file updates are handled safely
4. Rails best practices - Follows MVC patterns and Rails conventions
5. Proper error handling - RSS fetch failures are caught and logged
6. Thoughtful emoji handling - Addresses the UX requirement for emoji consistency

⚠️ Security Concerns

High Priority:

• lib/tasks/fetch_news.rake:40 - Using URI.open() is potentially vulnerable to SSRF attacks. Consider using a safer HTTP client like Net::HTTP with explicit URL validation
• lib/tasks/import_news.rake:7 - YAML.load_file() can execute arbitrary code. Use YAML.safe_load_file() instead

Medium Priority:

• No timeout configuration for RSS fetching could lead to hanging processes
• RSS URLs should be validated against an allowlist in production

🐛 Potential Bugs

High Priority:

• lib/tasks/fetch_news.rake:76 - ID extraction assumes integer IDs but uses URL as ID, causing type mismatch: item['id'].to_i when item['id'] is a URL string
• app/helpers/application_helper.rb:205 - Emoji detection only checks first character news.title[0] but comment discussions suggest it should check the entire title

Medium Priority:

• lib/tasks/fetch_news.rake:80 - Time.parse() without timezone handling could cause inconsistent sorting
• No duplicate URL protection during RSS fetching - malformed RSS could create duplicates

🚀 Performance Issues

Medium Priority:

• RSS fetching is synchronous with no timeout - could hang GitHub Actions
• No HTTP caching headers respected for RSS feeds
• Database operations in import_news.rake not batched (N+1 on save operations)
• Large YAML files loaded entirely into memory

📝 Code Quality Issues

Missing Validations:

# app/models/news.rb should include:
class News < ApplicationRecord
  validates :title, presence: true
  validates :url, presence: true, uniqueness: true, format: { with: URI::DEFAULT_PARSER.make_regexp }
  validates :published_at, presence: true
  
  scope :recent, -> { order(published_at: :desc) }
end

Inconsistent Data Types:

• db/news.yml uses URLs as IDs but code expects integers
• Consider either using integer IDs consistently or URL-based IDs throughout

Testing:

• spec/models/news_spec.rb:4 - Only has pending tests, no actual model validation tests
• spec/factories/news.rb - Factory uses placeholder values instead of realistic data
• No integration tests for RSS fetching or emoji handling

🛡️ Recommended Security Fixes

# lib/tasks/fetch_news.rake - Replace URI.open with safer HTTP client
require 'net/http'
require 'uri'

# Replace lines 40-49:
begin
  uri = URI(url)
  http = Net::HTTP.new(uri.host, uri.port)
  http.use_ssl = uri.scheme == 'https'
  http.open_timeout = 10
  http.read_timeout = 30
  
  response = http.get(uri.path)
  raise "HTTP Error: #{response.code}" unless response.code == '200'
  
  feed = RSS::Parser.parse(response.body, false)
  # ... rest of processing
rescue => e
  logger.warn("⚠️ Failed to fetch #{url}: #{e.message}")
  []
end

# lib/tasks/import_news.rake - Use safe YAML loading
raw = YAML.safe_load_file(yaml_path, permitted_classes: [Time], permitted_symbols: [], aliases: true)

🎯 Recommended Improvements

High Priority:

1. Fix ID handling inconsistency in fetch_news.rake:76
2. Add model validations to News class
3. Replace URI.open with safer HTTP client
4. Use YAML.safe_load_file instead of YAML.load_file

Medium Priority:

1. Add comprehensive tests for emoji handling logic
2. Add timeout configurations for HTTP requests
3. Implement proper error handling for malformed RSS
4. Add integration tests for the full RSS → YAML → DB flow

Low Priority:

1. Consider caching RSS responses to reduce external calls
2. Add batch processing for database operations
3. Add metrics/monitoring for RSS fetch success rates

🏆 Overall Assessment

This is a solid implementation that successfully automates news management. The core functionality works well and follows Rails conventions. The main concerns are:

1. Security - SSRF vulnerability and unsafe YAML loading need immediate attention
2. Data consistency - ID handling inconsistency should be resolved
3. Testing - Test coverage is insufficient for a production feature

Recommendation: This PR is conditionally ready for merge after addressing the high-priority security and bug fixes listed above.

---

[claude]

Claude finished @nacchan99's task —— View job

---

Code Review Complete

Reviewing this PR for code quality, potential bugs, performance considerations, security concerns, and test coverage.

Review Checklist:

• Review GitHub Actions workflow configuration
• Analyze RSS fetching logic and error handling
• Review database schema and model implementation
• Check controller and view changes
• Examine emoji handling logic
• Assess security implications
• Review test coverage and quality
• Check for performance considerations
• Provide overall feedback and recommendations

---

🔍 Detailed Code Review

✅ Strengths

1. Well-structured implementation - Good separation between RSS fetching, data processing, and display
2. Environment-aware configuration - Properly handles test vs production RSS sources
3. Atomic operations - YAML file updates are handled safely
4. Rails best practices - Follows MVC patterns and Rails conventions
5. Proper error handling - RSS fetch failures are caught and logged
6. Thoughtful emoji handling - Addresses the UX requirement for emoji consistency

⚠️ Security Concerns

High Priority:

• lib/tasks/fetch_news.rake:40 - Using URI.open() is potentially vulnerable to SSRF attacks. Consider using a safer HTTP client like Net::HTTP with explicit URL validation
• lib/tasks/import_news.rake:7 - YAML.load_file() can execute arbitrary code. Use YAML.safe_load_file() instead

Medium Priority:

• No timeout configuration for RSS fetching could lead to hanging processes
• RSS URLs should be validated against an allowlist in production

🐛 Potential Bugs

High Priority:

• lib/tasks/fetch_news.rake:76 - ID extraction assumes integer IDs but uses URL as ID, causing type mismatch: item['id'].to_i when item['id'] is a URL string
• app/helpers/application_helper.rb:205 - Emoji detection only checks first character news.title[0] but comment discussions suggest it should check the entire title

Medium Priority:

• lib/tasks/fetch_news.rake:80 - Time.parse() without timezone handling could cause inconsistent sorting
• No duplicate URL protection during RSS fetching - malformed RSS could create duplicates

🚀 Performance Issues

Medium Priority:

• RSS fetching is synchronous with no timeout - could hang GitHub Actions
• No HTTP caching headers respected for RSS feeds
• Database operations in import_news.rake not batched (N+1 on save operations)
• Large YAML files loaded entirely into memory

📝 Code Quality Issues

Missing Validations:

# app/models/news.rb should include:
class News < ApplicationRecord
  validates :title, presence: true
  validates :url, presence: true, uniqueness: true, format: { with: URI::DEFAULT_PARSER.make_regexp }
  validates :published_at, presence: true
  
  scope :recent, -> { order(published_at: :desc) }
end

Inconsistent Data Types:

• db/news.yml uses URLs as IDs but code expects integers
• Consider either using integer IDs consistently or URL-based IDs throughout

Testing:

• spec/models/news_spec.rb:4 - Only has pending tests, no actual model validation tests
• spec/factories/news.rb - Factory uses placeholder values instead of realistic data
• No integration tests for RSS fetching or emoji handling

🛡️ Recommended Security Fixes

# lib/tasks/fetch_news.rake - Replace URI.open with safer HTTP client
require 'net/http'
require 'uri'

# Replace lines 40-49:
begin
  uri = URI(url)
  http = Net::HTTP.new(uri.host, uri.port)
  http.use_ssl = uri.scheme == 'https'
  http.open_timeout = 10
  http.read_timeout = 30
  
  response = http.get(uri.path)
  raise "HTTP Error: #{response.code}" unless response.code == '200'
  
  feed = RSS::Parser.parse(response.body, false)
  # ... rest of processing
rescue => e
  logger.warn("⚠️ Failed to fetch #{url}: #{e.message}")
  []
end

# lib/tasks/import_news.rake - Use safe YAML loading
raw = YAML.safe_load_file(yaml_path, permitted_classes: [Time], permitted_symbols: [], aliases: true)

🎯 Recommended Improvements

High Priority:

1. Fix ID handling inconsistency in fetch_news.rake:76
2. Add model validations to News class
3. Replace URI.open with safer HTTP client
4. Use YAML.safe_load_file instead of YAML.load_file

Medium Priority:

1. Add comprehensive tests for emoji handling logic
2. Add timeout configurations for HTTP requests
3. Implement proper error handling for malformed RSS
4. Add integration tests for the full RSS → YAML → DB flow

Low Priority:

1. Consider caching RSS responses to reduce external calls
2. Add batch processing for database operations
3. Add metrics/monitoring for RSS fetch success rates

🏆 Overall Assessment

This is a solid implementation that successfully automates news management. The core functionality works well and follows Rails conventions. The main concerns are:

1. Security - SSRF vulnerability and unsafe YAML loading need immediate attention
2. Data consistency - ID handling inconsistency should be resolved
3. Testing - Test coverage is insufficient for a production feature

Recommendation: This PR is conditionally ready for merge after addressing the high-priority security and bug fixes listed above.

---

[claude]

Claude finished @nacchan99's task —— View job

---

Code Review Complete

Reviewing this PR for code quality, potential bugs, performance considerations, security concerns, and test coverage.

Review Checklist:

• Review GitHub Actions workflow configuration
• Analyze RSS fetching logic and error handling
• Review database schema and model implementation
• Check controller and view changes
• Examine emoji handling logic
• Assess security implications
• Review test coverage and quality
• Check for performance considerations
• Provide overall feedback and recommendations

---

🔍 Detailed Code Review

✅ Strengths

1. Well-structured implementation - Good separation between RSS fetching, data processing, and display
2. Environment-aware configuration - Properly handles test vs production RSS sources
3. Atomic operations - YAML file updates are handled safely
4. Rails best practices - Follows MVC patterns and Rails conventions
5. Proper error handling - RSS fetch failures are caught and logged
6. Thoughtful emoji handling - Addresses the UX requirement for emoji consistency

⚠️ Security Concerns

High Priority:

• lib/tasks/fetch_news.rake:40 - Using URI.open() is potentially vulnerable to SSRF attacks. Consider using a safer HTTP client like Net::HTTP with explicit URL validation
• lib/tasks/import_news.rake:7 - YAML.load_file() can execute arbitrary code. Use YAML.safe_load_file() instead

Medium Priority:

• No timeout configuration for RSS fetching could lead to hanging processes
• RSS URLs should be validated against an allowlist in production

🐛 Potential Bugs

High Priority:

• lib/tasks/fetch_news.rake:76 - ID extraction assumes integer IDs but uses URL as ID, causing type mismatch: item['id'].to_i when item['id'] is a URL string
• app/helpers/application_helper.rb:207 - Emoji detection only checks first character news.title[0] but comment discussions suggest it should check the entire title

Medium Priority:

• lib/tasks/fetch_news.rake:80 - Time.parse() without timezone handling could cause inconsistent sorting
• No duplicate URL protection during RSS fetching - malformed RSS could create duplicates

🚀 Performance Issues

Medium Priority:

• RSS fetching is synchronous with no timeout - could hang GitHub Actions
• No HTTP caching headers respected for RSS feeds
• Database operations in import_news.rake not batched (N+1 on save operations)
• Large YAML files loaded entirely into memory

📝 Code Quality Issues

Missing Validations:

# app/models/news.rb should include:
class News < ApplicationRecord
  validates :title, presence: true
  validates :url, presence: true, uniqueness: true, format: { with: URI::DEFAULT_PARSER.make_regexp }
  validates :published_at, presence: true
  
  scope :recent, -> { order(published_at: :desc) }
end

Inconsistent Data Types:

• db/news.yml uses URLs as IDs but code expects integers
• Consider either using integer IDs consistently or URL-based IDs throughout

Testing:

• spec/models/news_spec.rb:4 - Only has pending tests, no actual model validation tests
• spec/factories/news.rb - Factory uses placeholder values instead of realistic data
• No integration tests for RSS fetching or emoji handling

🛡️ Recommended Security Fixes

# lib/tasks/fetch_news.rake - Replace URI.open with safer HTTP client
require 'net/http'
require 'uri'

# Replace lines 40-49:
begin
  uri = URI(url)
  http = Net::HTTP.new(uri.host, uri.port)
  http.use_ssl = uri.scheme == 'https'
  http.open_timeout = 10
  http.read_timeout = 30
  
  response = http.get(uri.path)
  raise "HTTP Error: #{response.code}" unless response.code == '200'
  
  feed = RSS::Parser.parse(response.body, false)
  # ... rest of processing
rescue => e
  logger.warn("⚠️ Failed to fetch #{url}: #{e.message}")
  []
end

# lib/tasks/import_news.rake - Use safe YAML loading
raw = YAML.safe_load_file(yaml_path, permitted_classes: [Time], permitted_symbols: [], aliases: true)

🎯 Recommended Improvements

High Priority:

1. Fix ID handling inconsistency in fetch_news.rake:76
2. Add model validations to News class
3. Replace URI.open with safer HTTP client
4. Use YAML.safe_load_file instead of YAML.load_file

Medium Priority:

1. Add comprehensive tests for emoji handling logic
2. Add timeout configurations for HTTP requests
3. Implement proper error handling for malformed RSS
4. Add integration tests for the full RSS → YAML → DB flow

Low Priority:

1. Consider caching RSS responses to reduce external calls
2. Add batch processing for database operations
3. Add metrics/monitoring for RSS fetch success rates

🏆 Overall Assessment

This is a solid implementation that successfully automates news management. The core functionality works well and follows Rails conventions. The main concerns are:

1. Security - SSRF vulnerability and unsafe YAML loading need immediate attention
2. Data consistency - ID handling inconsistency should be resolved
3. Testing - Test coverage is insufficient for a production feature

Recommendation: This PR is conditionally ready for merge after addressing the high-priority security and bug fixes listed above.

---

[nacchan99]

CIテストのエラー解消のため、mainブランチをリベースして最新を取り込みました🙌

[claude]

Claude finished @nacchan99's task —— View job

---

I'll analyze this and get back to you.

[nacchan99]

@claude Can you suggest additional tests we should add for this PR, in Japanese?

[claude]

Claude finished @nacchan99's task —— View job

---

追加テストの提案分析

Todo List:

• 既存のテストカバレッジを分析
• 実装されたRSS機能を確認
• テストの隙間を特定
• 追加すべきテストを日本語で提案
• 分析完了

---

📋 現在のテストカバレッジ状況

✅ 実装済みのテスト

• spec/helpers/application_helper_spec.rb - 絵文字判定ヘルパーのテスト
• spec/features/news_spec.rb - ニュースセクションの表示テスト
• spec/factories/news.rb - Newsファクトリー定義
• spec/fixtures/sample_news.rss - テスト用サンプルRSS

❌ 未実装のテスト

• spec/models/news_spec.rb - ペンディング状態のみ
• Rakeタスクのテスト - news:fetch、news:import_from_yaml
• エラーハンドリングのテスト - 各種エラーケース
• セキュリティ機能のテスト - SSRFやYAML安全読み込み
• 統合テスト - RSS→YAML→DB の完全フロー

---

🧪 追加すべきテスト一覧

1. モデルテスト強化 (spec/models/news_spec.rb)

require 'rails_helper'

RSpec.describe News, type: :model do
  describe 'バリデーション' do
    let(:news) { build(:news) }

    it '有効なファクトリーを持つ' do
      expect(news).to be_valid
    end

    describe 'title' do
      it 'presence: true' do
        news.title = nil
        expect(news).not_to be_valid
        expect(news.errors[:title]).to include("can't be blank")
      end
    end

    describe 'url' do
      it 'presence: true' do
        news.url = nil
        expect(news).not_to be_valid
        expect(news.errors[:url]).to include("can't be blank")
      end

      it 'uniqueness: true' do
        create(:news, url: 'https://example.com/test')
        duplicate_news = build(:news, url: 'https://example.com/test')
        expect(duplicate_news).not_to be_valid
        expect(duplicate_news.errors[:url]).to include("has already been taken")
      end

      it 'URL形式であること' do
        news.url = 'invalid-url'
        expect(news).not_to be_valid
        expect(news.errors[:url]).to include("is invalid")
      end

      it 'HTTPSとHTTPを許可する' do
        news.url = 'https://example.com'
        expect(news).to be_valid
        
        news.url = 'http://example.com'
        expect(news).to be_valid
      end
    end

    describe 'published_at' do
      it 'presence: true' do
        news.published_at = nil
        expect(news).not_to be_valid
        expect(news.errors[:published_at]).to include("can't be blank")
      end
    end
  end

  describe 'スコープ' do
    describe '.recent' do
      it '公開日時の降順で並び替える' do
        old_news = create(:news, published_at: 2.days.ago)
        new_news = create(:news, published_at: 1.day.ago)
        
        expect(News.recent).to eq([new_news, old_news])
      end
    end
  end
end

2. ヘルパーテスト拡張 (spec/helpers/application_helper_spec.rb)

# 既存のテストに追加
describe '#format_news_title' do
  context '絵文字の判定' do
    it '数字が含まれていても正しく判定する' do
      news = double('news', title: '123 テスト記事')
      expect(helper.format_news_title(news)).to eq('📰 123 テスト記事')
    end

    it '複数の絵文字が含まれる場合' do
      news = double('news', title: '🎉🎊 祝賀記事')
      expect(helper.format_news_title(news)).to eq('🎉🎊 祝賀記事')
    end

    it '文字列の途中に絵文字がある場合' do
      news = double('news', title: 'テスト🎉記事')
      expect(helper.format_news_title(news)).to eq('📰 テスト🎉記事')
    end

    it '空文字列の場合' do
      news = double('news', title: '')
      expect(helper.format_news_title(news)).to eq('📰 ')
    end

    it 'nilの場合' do
      news = double('news', title: nil)
      expect(helper.format_news_title(news)).to eq('📰 ')
    end
  end
end

3. Rakeタスクテスト (spec/tasks/news_rake_spec.rb)

require 'rails_helper'
require 'rake'

RSpec.describe 'news rake tasks' do
  before do
    Rails.application.load_tasks
  end

  describe 'news:fetch' do
    let(:yaml_path) { Rails.root.join('db', 'news.yml') }
    let(:sample_rss_path) { Rails.root.join('spec', 'fixtures', 'sample_news.rss') }

    before do
      # テスト環境を明示的に設定
      allow(Rails.env).to receive(:test?).and_return(true)
      
      # 既存のnews.ymlを削除
      File.delete(yaml_path) if File.exist?(yaml_path)
    end

    after do
      # テスト後にクリーンアップ
      File.delete(yaml_path) if File.exist?(yaml_path)
    end

    it 'サンプルRSSからニュースを取得してYAMLに書き込む' do
      expect { Rake::Task['news:fetch'].invoke }.not_to raise_error
      
      expect(File.exist?(yaml_path)).to be true
      
      yaml_content = YAML.safe_load(File.read(yaml_path), permitted_classes: [Time])
      expect(yaml_content['news']).to be_a(Array)
      expect(yaml_content['news'].size).to eq(3)
    end

    it 'IDが正しく連番で付与される' do
      Rake::Task['news:fetch'].invoke
      
      yaml_content = YAML.safe_load(File.read(yaml_path), permitted_classes: [Time])
      ids = yaml_content['news'].map { |item| item['id'] }
      
      expect(ids).to match_array([1, 2, 3])
    end

    it '公開日時順でソートされる' do
      Rake::Task['news:fetch'].invoke
      
      yaml_content = YAML.safe_load(File.read(yaml_path), permitted_classes: [Time])
      published_dates = yaml_content['news'].map { |item| Time.parse(item['published_at']) }
      
      expect(published_dates).to eq(published_dates.sort.reverse)
    end
  end

  describe 'news:import_from_yaml' do
    let(:yaml_path) { Rails.root.join('db', 'news.yml') }

    before do
      # テスト用のYAMLファイルを作成
      news_data = {
        'news' => [
          {
            'id' => 1,
            'url' => 'https://example.com/test1',
            'title' => 'テスト記事1',
            'published_at' => '2025-01-01 10:00:00 +0000'
          },
          {
            'id' => 2,
            'url' => 'https://example.com/test2',
            'title' => 'テスト記事2',
            'published_at' => '2025-01-02 10:00:00 +0000'
          }
        ]
      }
      
      File.write(yaml_path, news_data.to_yaml)
    end

    after do
      File.delete(yaml_path) if File.exist?(yaml_path)
    end

    it 'YAMLファイルからNewsレコードを作成する' do
      expect { Rake::Task['news:import_from_yaml'].invoke }.to change { News.count }.by(2)
      
      news1 = News.find_by(url: 'https://example.com/test1')
      expect(news1.title).to eq('テスト記事1')
      
      news2 = News.find_by(url: 'https://example.com/test2')
      expect(news2.title).to eq('テスト記事2')
    end

    it '既存のNewsレコードを更新する' do
      existing_news = create(:news, url: 'https://example.com/test1', title: '古いタイトル')
      
      expect { Rake::Task['news:import_from_yaml'].invoke }.to change { News.count }.by(1)
      
      existing_news.reload
      expect(existing_news.title).to eq('テスト記事1')
    end
  end
end

4. エラーハンドリングテスト (spec/tasks/news_fetch_error_handling_spec.rb)

require 'rails_helper'
require 'rake'

RSpec.describe 'news:fetch error handling' do
  before do
    Rails.application.load_tasks
    allow(Rails.env).to receive(:test?).and_return(false) # 本番環境をシミュレート
  end

  describe 'ネットワークエラー' do
    it 'HTTP接続エラーを適切に処理する' do
      allow_any_instance_of(Object).to receive(:safe_open).and_raise(StandardError.new('Connection failed'))
      
      expect { Rake::Task['news:fetch'].invoke }.not_to raise_error
      
      # ログファイルにエラーが記録されることを確認
      expect(File.exist?(Rails.root.join('log', 'news.log'))).to be true
    end
  end

  describe 'RSS解析エラー' do
    it '不正なRSSを適切に処理する' do
      invalid_rss = '<invalid>xml</invalid>'
      allow_any_instance_of(Object).to receive(:safe_open).and_return(invalid_rss)
      
      expect { Rake::Task['news:fetch'].invoke }.not_to raise_error
    end
  end

  describe 'YAML処理エラー' do
    it '破損したYAMLファイルを適切に処理する' do
      yaml_path = Rails.root.join('db', 'news.yml')
      File.write(yaml_path, 'invalid: yaml: content:')
      
      expect { Rake::Task['news:fetch'].invoke }.not_to raise_error
      
      File.delete(yaml_path) if File.exist?(yaml_path)
    end
  end
end

5. セキュリティテスト (spec/security/news_security_spec.rb)

require 'rails_helper'

RSpec.describe 'News security' do
  describe 'safe_open method' do
    it 'HTTPSとHTTPのみを許可する' do
      expect { safe_open('https://example.com') }.not_to raise_error
      expect { safe_open('http://example.com') }.not_to raise_error
    end

    it '危険なプロトコルを拒否する' do
      expect { safe_open('ftp://example.com') }.to raise_error(/不正なURLです/)
      expect { safe_open('file:///etc/passwd') }.to raise_error(/不正なURLです/)
    end

    it 'localhost へのアクセスを制限する（必要に応じて）' do
      # 環境によって実装する
      # expect { safe_open('http://localhost:3000') }.to raise_error
    end
  end

  describe 'YAML safe loading' do
    it 'Time クラスのみを許可する' do
      yaml_content = "---\ntime: 2025-01-01 10:00:00 +0000\n"
      
      expect { YAML.safe_load(yaml_content, permitted_classes: [Time]) }.not_to raise_error
    end

    it '危険なクラスを拒否する' do
      yaml_content = "---\n!!ruby/object:File\n"
      
      expect { YAML.safe_load(yaml_content, permitted_classes: [Time]) }.to raise_error(Psych::DisallowedClass)
    end
  end
end

6. 統合テスト (spec/integration/news_flow_spec.rb)

require 'rails_helper'

RSpec.describe 'News integration flow' do
  describe 'RSS → YAML → DB → View の完全フロー' do
    it 'RSS取得からビュー表示までの全工程が正常動作する' do
      # 1. RSS取得
      Rails.application.load_tasks
      allow(Rails.env).to receive(:test?).and_return(true)
      
      Rake::Task['news:fetch'].invoke
      
      # 2. YAML確認
      yaml_path = Rails.root.join('db', 'news.yml')
      expect(File.exist?(yaml_path)).to be true
      
      # 3. DB取り込み
      Rake::Task['news:import_from_yaml'].invoke
      expect(News.count).to eq(3)
      
      # 4. ビュー表示
      visit root_path(anchor: 'news')
      
      within 'section#news' do
        expect(page).to have_content('📰 テスト記事①')
        expect(page).to have_content('📰 テスト記事②')
        expect(page).to have_content('🎉 テスト記事③')
      end
      
      # クリーンアップ
      File.delete(yaml_path) if File.exist?(yaml_path)
    end
  end
end

7. パフォーマンステスト (spec/performance/news_performance_spec.rb)

require 'rails_helper'

RSpec.describe 'News performance' do
  describe 'large dataset handling' do
    it '大量のニュースデータを効率的に処理する' do
      # 大量のテストデータを作成
      large_news_data = {
        'news' => (1..1000).map do |i|
          {
            'id' => i,
            'url' => "https://example.com/news#{i}",
            'title' => "ニュース記事#{i}",
            'published_at' => i.days.ago.to_s
          }
        end
      }
      
      yaml_path = Rails.root.join('db', 'news.yml')
      File.write(yaml_path, large_news_data.to_yaml)
      
      # パフォーマンス測定
      start_time = Time.current
      
      Rails.application.load_tasks
      Rake::Task['news:import_from_yaml'].invoke
      
      end_time = Time.current
      execution_time = end_time - start_time
      
      expect(execution_time).to be < 10.0 # 10秒以内
      expect(News.count).to eq(1000)
      
      File.delete(yaml_path) if File.exist?(yaml_path)
    end
  end
end

---

🎯 優先度別実装推奨順序

高優先度（必須）

1. モデルバリデーションテスト - 基本的なデータ整合性を保証
2. Rakeタスクの基本テスト - コア機能の動作確認
3. エラーハンドリングテスト - 本番環境での安定性確保

中優先度（推奨）

4. セキュリティテスト - 脆弱性対策の検証
5. 統合テスト - 全体フローの動作確認
6. ヘルパーテスト拡張 - エッジケースの対応

低優先度（将来的に）

7. パフォーマンステスト - スケーラビリティの確認

---

📋 まとめ

現在のPRは基本的な機能は実装されていますが、テストカバレッジが不足している状態です。特に：

• モデルバリデーション - データ整合性の保証
• Rakeタスク - RSS取得とYAML処理の中核機能
• エラーハンドリング - 本番環境での安定性

これらのテストを追加することで、より堅牢で保守性の高いコードになります。

[rakuda-san-desu]

細かい調整をいくつかお願いしていますが、基本的な動作は良さそうです🙆‍♀️

Rakeタスクのリファクタリング、テストの追加については、別途issueにまとめて対応いただけると🙏
（必要なテストについて、専用issueで整理すると良さそうです）

[nacchan99]

細部まで丁寧にレビューしていただき、ありがとうございます🙌
変更すべき理由や、採用する根拠が明確に示されていて、とてもわかりやすかったです😭✨

追加すべきテストや改善ポイントをまとめた新しい Issue を作成したので、お手隙の際にご確認いただけると嬉しいです🙏
➡︎ Rakeタスクのリファクタリングとテスト追加 #1716

[rakuda-san-desu]

纏めてになりますが、対応ありがとうございました！
以下を確認しました！良さそうです🙆‍♀️

• コメント部分の対応が完了もしくはissue化
• 最新のコードで動作確認
• Claude のレビューでも良さそう
  #1704 (comment)

  この修正により、本PRはセキュリティ面での懸念が解消され、マージ可能な状態に達しました！ 🚀

[nacchan99]

お忙しい中、丁寧にご確認いただきありがとうございます！🙌✨
マージします🚀

その後は、新しいブランチで #1716 の対応を進める予定です。
引き続きよろしくお願いします！

[nacchan99]

本番環境でも表示確認できました✨