🔗 ニュースセクションの内部リンクから target='_blank' を削除

- News モデルに internal_link? メソッドを追加
- coderdojo.jp ドメインと相対パスのリンクを内部リンクとして判定
- 内部リンクの場合は新しいタブで開かないように変更
- 外部リンクには外部リンクアイコンを表示
- テストケースを追加して動作を保証

Author: yasulab

app/models/news.rb

@@ -32,4 +32,9 @@ def link_url
       url
     end
   end
+
+  def internal_link?
+    # Check if the link is internal (coderdojo.jp domain)
+    url.match?(%r{^https://coderdojo\.jp}) || url.start_with?('/')
+  end
 end

app/views/home/show.html.erb

@@ -194,7 +194,16 @@
   <ul class="list" style="list-style: none;">
     <% @news_items.each do |news| %>
       <li>
-        <%= link_to news.formatted_title, news.link_url, target: '_blank' %>
+        <% if news.internal_link? %>
+          <%= link_to news.link_url do %>
+	    <%= news.formatted_title %>
+	  <% end %>
+        <% else %>
+          <%= link_to news.link_url, target: '_blank' do %>
+	    <%= news.formatted_title %>
+	    <small><small><i class="far fa-external-link"></i></small></small>
+	  <% end %>
+        <% end %>
       </li>
     <% end %>
   </ul>

app/views/news/index.html.erb

@@ -23,12 +23,22 @@
     <% if @news_items.any? %>
       <div class="news-list">
         <% @news_items.each do |news| %>
+	  
           <article class="news-item" style="margin-bottom: 30px; padding-bottom: 20px; border-bottom: 1px solid #e0e0e0;">
             <h3 style="margin-bottom: 10px;">
-              <%= link_to news.formatted_title, news.link_url, target: '_blank', rel: 'noopener' %>
+	      <% if news.internal_link? %>
+		<%= link_to news.link_url do %>
+		  <%= news.formatted_title %>
+		<% end %>
+              <% else %>
+		<%= link_to news.link_url, target: '_blank' do %>
+		  <%= news.formatted_title %>
+		  <small><i class="far fa-external-link"></i></small>
+		<% end %>
+              <% end %>
             </h3>
             <p class="news-meta" style="color: #666; font-size: 14px;">
-              <i class="fa fa-calendar"></i>
+              <!--<i class="fa fa-calendar"></i>-->
               <%= l(news.published_at.to_date, format: :long) if news.published_at %>
             </p>
           </article>

spec/models/news_spec.rb

@@ -125,4 +125,35 @@
       expect(news2.link_url).to eq 'https://prtimes.jp/main/html/rd/p/000000001.000038935.html'
     end
   end
+
+  describe '#internal_link?' do
+    context '内部リンクの場合' do
+      it 'coderdojo.jpドメインのURLはtrueを返す' do
+        news = build(:news, url: 'https://coderdojo.jp/podcasts/33')
+        expect(news.internal_link?).to be true
+      end
+
+      it '相対パスで始まるURLはtrueを返す' do
+        news = build(:news, url: '/kata')
+        expect(news.internal_link?).to be true
+      end
+    end
+
+    context '外部リンクの場合' do
+      it 'news.coderdojo.jpサブドメインはfalseを返す' do
+        news = build(:news, url: 'https://news.coderdojo.jp/2025/12/06/dojoletter')
+        expect(news.internal_link?).to be false
+      end
+
+      it 'prtimes.jpドメインはfalseを返す' do
+        news = build(:news, url: 'https://prtimes.jp/main/html/rd/p/000000001.000038935.html')
+        expect(news.internal_link?).to be false
+      end
+
+      it '他の外部ドメインはfalseを返す' do
+        news = build(:news, url: 'https://example.com/article')
+        expect(news.internal_link?).to be false
+      end
+    end
+  end
 end