ポッドキャストのタイムスタンプ自動リンク機能を修正

- hh:mm:ss形式（00:00:00）のタイムスタンプに対応
- タイムスタンプをYouTubeの時間形式（?t=1h23m45s）に正確に変換
- mm:ss形式も引き続きサポート
- テストケースを追加して動作を保証
- エピソード32のタイムスタンプ形式を統一し、欠けていたURLを追加

Author: yasulab

app/controllers/podcasts_controller.rb

@@ -48,18 +48,24 @@ def convert_shownote(content)
     return content unless content.match?(Podcast::TIMESTAMP_REGEX)
     youtube_id = @episode.content.match(Podcast::YOUTUBE_ID_REGEX)[1]
     content.gsub!(Podcast::TIMESTAMP_REGEX) do
-        t = $1
-        t = (t.size ==  '0:00'.size) ?   '0' + t : t
-        t = (t.size == '00:00'.size) ? '00:' + t : t
-
-        #t = Time.parse(t).seconds_since_midnight.to_i
-        # This format should be faster than above:
-        #   ?t=01:23:45
-        #   ?t=01h23m45s
-        t[2] = 'h'
-        t[5] = 'm'
-        t   << 's'
-        "- [#{$1}](https://youtu.be/#{youtube_id}?t=#{t}) &nbsp; "
+        original_t = $1
+        parts = original_t.split(':')
+        
+        # タイムスタンプをh:m:s形式に変換
+        if parts.size == 3
+          # 00:00:00 形式
+          h, m, s = parts
+          t = "#{h}h#{m}m#{s}s"
+        elsif parts.size == 2
+          # 00:00 形式
+          m, s = parts
+          t = "#{m}m#{s}s"
+        else
+          # それ以外（通常は来ないはず）
+          t = original_t
+        end
+        
+        "- [#{original_t}](https://youtu.be/#{youtube_id}?t=#{t}) &nbsp; "
     end
   end
 end

app/models/podcast.rb

@@ -2,7 +2,7 @@ class Podcast < ApplicationRecord
   self.table_name = 'podcasts'
   DIR_PATH        = 'public/podcasts'
   WDAY2JAPANESE   = %w(日 月 火 水 木 金 土)
-  TIMESTAMP_REGEX  = /-\s((\d:)?\d{1,}:\d{2})/
+  TIMESTAMP_REGEX  = /-\s((\d{1,2}:)?\d{1,2}:\d{2})/
   YOUTUBE_ID_REGEX = /watch\?v=((\w)*)/
 
   validates :title,          presence: true

public/podcasts/32.md

@@ -32,46 +32,46 @@
 
 ## Shownote
 
-- 00:00 米国系 IT 企業から CoderDojo へ、233台のノートPC寄贈 
-- 00:24 配送作業の裏話と、こだわりポイントの紹介
-- 01:06 CoderDojo Japan からのお知らせ記事の紹介
-- 03:01 過去の寄贈（128台、55台）から233台への規模拡大
-- 03:40 配送作業のノウハウ蓄積と、数学的な配分問題
-- 04:09 寄贈後のトラブル確認と、英語キーボード配列
-- 07:20 制約充足問題として配分を数学的に解いてみる
-- 10:20 ダンボール箱の最適化（80サイズで2台ずつ梱包）
-- 13:03 偶数台数での配送と、発注する箱数の計算
-- 14:08 搬入から搬出までの作業時間の制約
-- 16:04 １道場あたりの台数制限（税務上の制約）
-- 17:10 土日の30時間で全作業を完了させる計画
-- 17:24 配分の公平性をジニ係数で数値化
-- 19:30 AI を活用したマッチング（希望機種の振り分け）
-- 21:00 マッチ度をなるべく上げていく試み
-- 23:52 AI との対話で制約充足問題を解決
-- 25:01 AI と遊んでみる回の動画 
-- 28:27 梱包作業の実際の様子（開封から箱詰めまで）
-- 29:02 PC が子供たちに手に届くまで、多くの人が関わっている話
-- 31:05 数値化だけでは全てを把握できない話（機種グレードも考慮した例）
-- 33:54 ACアダプターなどの対応確認、声かけ確認
-- 35:37 指差し確認で間違いを防いで作業する
-- 38:40 番号で管理し、最終確認も実施する
-- 41:08 233台、すべて初期化していただいた話
-- 42:02 梱包の工夫（ACアダプターを上に置かない等）
-- 44:13 作業完了後のふりかえり
-- 44:55 今後の寄贈プロジェクトへの展望
-- 46:02 日本国内での英語版CoderDojoの話
-- 46:35 統計情報の英語対応の話 https://coderdojo.jp/stats/english
-- 47:00 CASE Shinjuku の日常にも、国際化の流れ
-- 49:10 CoderDojo 立ち上げ相談（ロボット工場のある町）
-- 54:07 CASE Shinjuku の紹介（1時間500円から利用可能）
-- 56:00 多様な利用者層（絵を描く人、DIY、音楽活動）
-- 57:52 CoderDojo 青山の「青学つくまなラボ」の話
-- 58:00 山崎さんの歌をYouTube配信で使用した話
-- 59:14 CASE Shinjuku 利用者と CoderDojo の繋がり
-- 1:00:57 CASE Shinjuku の英語アクセスページ
-- 1:02:01 CASE Shinjuku の英語インタビュー記事
-- 1:04:03 アクセス記事を更新する企画
-- 1:07:13 12年前のプレオープン、何もない空間で初めて話した話
-- 1:08:35 「私より後に生まれてきた人のために」
-- 1:09:22 Pay it forward（恩送り）の考え方
-- 1:10:41 エンディング（熱中症に気をつけて）
+- 00:00:00 米国系 IT 企業から CoderDojo へ、233台のノートPC寄贈 
+- 00:00:24 配送作業の裏話と、こだわりポイントの紹介
+- 00:01:06 CoderDojo Japan からのお知らせ記事の紹介
+- 00:03:01 過去の寄贈（128台、55台）から233台への規模拡大
+- 00:03:40 配送作業のノウハウ蓄積と、数学的な配分問題
+- 00:04:09 寄贈後のトラブル確認と、英語キーボード配列
+- 00:07:20 制約充足問題として配分を数学的に解いてみる
+- 00:10:20 ダンボール箱の最適化（80サイズで2台ずつ梱包）
+- 00:13:03 偶数台数での配送と、発注する箱数の計算
+- 00:14:08 搬入から搬出までの作業時間の制約
+- 00:16:04 １道場あたりの台数制限（税務上の制約）
+- 00:17:10 土日の30時間で全作業を完了させる計画
+- 00:17:24 配分の公平性をジニ係数で数値化
+- 00:19:30 AI を活用したマッチング（希望機種の振り分け）
+- 00:21:00 マッチ度をなるべく上げていく試み
+- 00:23:52 AI との対話で制約充足問題を解決
+- 00:25:01 AI と遊んでみる回の動画 https://youtu.be/BYpa1CcYtss?t=1425
+- 00:28:27 梱包作業の実際の様子（開封から箱詰めまで）
+- 00:29:02 PC が子供たちに手に届くまで、多くの人が関わっている話
+- 00:31:05 数値化だけでは全てを把握できない話（機種グレードも考慮した例）
+- 00:33:54 ACアダプターなどの対応確認、声かけ確認
+- 00:35:37 指差し確認で間違いを防いで作業する
+- 00:38:40 番号で管理し、最終確認も実施する
+- 00:41:08 233台、すべて初期化していただいた話
+- 00:42:02 梱包の工夫（ACアダプターを上に置かない等）
+- 00:44:13 作業完了後のふりかえり
+- 00:44:55 今後の寄贈プロジェクトへの展望
+- 00:46:02 日本国内での英語版CoderDojoの話
+- 00:46:35 統計情報の英語対応の話 https://coderdojo.jp/stats/english
+- 00:47:00 CASE Shinjuku の日常にも、国際化の流れ
+- 00:49:10 CoderDojo 立ち上げ相談（ロボット工場のある町）
+- 00:54:07 CASE Shinjuku の紹介（1時間500円から利用可能）
+- 00:56:00 多様な利用者層（絵を描く人、DIY、音楽活動）
+- 00:57:52 CoderDojo 青山の「青学つくまなラボ」の話
+- 00:58:00 山崎さんの歌をYouTube配信で使用した話 https://youtu.be/ZJdlsgMzWrU
+- 00:59:14 CASE Shinjuku 利用者と CoderDojo の繋がり
+- 01:00:57 CASE Shinjuku の英語アクセスページ https://case-shinjuku.com/english
+- 01:02:01 CASE Shinjuku の英語インタビュー記事
+- 01:04:03 アクセス記事を更新する企画
+- 01:07:13 12年前のプレオープン、何もない空間で初めて話した話
+- 01:08:35 「私より後に生まれてきた人のために」
+- 01:09:22 Pay it forward（恩送り）の考え方
+- 01:10:41 エンディング（熱中症に気をつけて）

spec/features/podcasts_spec.rb

@@ -36,5 +36,67 @@
       click_link target, match: :first
       expect(page).to have_http_status(:success)
     end
+
+    scenario 'Show note timestamps are converted to YouTube links' do
+      @podcast = create(:podcast)
+      allow(@podcast).to receive(:exist?) { true }
+      allow(@podcast).to receive(:content) { 
+        <<~CONTENT
+          タイトル
+          収録日: 2019/05/10
+          
+          YouTubeリンク: https://www.youtube.com/watch?v=Dd9IYiF0R6E
+          
+          ## Shownote
+          
+          - 00:00:00 米国系 IT 企業から CoderDojo へ、233台のノートPC寄贈
+          - 00:25:01 AI と遊んでみる回の動画 https://youtu.be/BYpa1CcYtss?t=1425
+          - 00:59:14 CASE Shinjuku 利用者と CoderDojo の繋がり
+          - 01:00:57 CASE Shinjuku の英語アクセスページ https://case-shinjuku.com/english
+        CONTENT
+      }
+      allow(Podcast).to  receive(:find_by).with(id: @podcast.id.to_s) { @podcast }
+
+      visit  "/podcasts/#{@podcast.id}"
+      expect(page).to have_http_status(:success)
+      
+      # タイムスタンプがYouTubeリンクに変換されているか確認
+      expect(page).to have_link '00:00:00', href: 'https://youtu.be/Dd9IYiF0R6E?t=00h00m00s'
+      expect(page).to have_link '00:25:01', href: 'https://youtu.be/Dd9IYiF0R6E?t=00h25m01s'
+      expect(page).to have_link '00:59:14', href: 'https://youtu.be/Dd9IYiF0R6E?t=00h59m14s'
+      expect(page).to have_link '01:00:57', href: 'https://youtu.be/Dd9IYiF0R6E?t=01h00m57s'
+      
+      # 既存のURL付きタイムスタンプはそのまま表示されること
+      expect(page).to have_content 'AI と遊んでみる回の動画 https://youtu.be/BYpa1CcYtss?t=1425'
+      expect(page).to have_content 'CASE Shinjuku の英語アクセスページ https://case-shinjuku.com/english'
+    end
+
+    scenario 'Show note timestamps with mm:ss format are converted to YouTube links' do
+      @podcast = create(:podcast)
+      allow(@podcast).to receive(:exist?) { true }
+      allow(@podcast).to receive(:content) { 
+        <<~CONTENT
+          タイトル
+          収録日: 2019/05/10
+          
+          YouTubeリンク: https://www.youtube.com/watch?v=test123
+          
+          ## Shownote
+          
+          - 00:30 オープニング
+          - 05:45 メインテーマ
+          - 59:59 エンディング
+        CONTENT
+      }
+      allow(Podcast).to  receive(:find_by).with(id: @podcast.id.to_s) { @podcast }
+
+      visit  "/podcasts/#{@podcast.id}"
+      expect(page).to have_http_status(:success)
+      
+      # mm:ss形式のタイムスタンプもYouTubeリンクに変換されているか確認
+      expect(page).to have_link '00:30', href: 'https://youtu.be/test123?t=00m30s'
+      expect(page).to have_link '05:45', href: 'https://youtu.be/test123?t=05m45s'
+      expect(page).to have_link '59:59', href: 'https://youtu.be/test123?t=59m59s'
+    end
   end
 end