refactor: Git履歴抽出タスクを1つに統合して冪等性を実装

- 引数なし: 全ての非アクティブDojoを処理してYAMLを更新
- 引数あり: 特定のDojoの情報を表示のみ（読み取り専用）
- 既に inactivated_at が設定されている場合はスキップ
- コードの重複を排除してメンテナンス性を向上

Author: yasulab

lib/tasks/dojos_inactivated_at.rake

@@ -1,6 +1,6 @@
 namespace :dojos do
-  desc 'Git履歴からinactivated_at日付を抽出してYAMLファイルに反映'
-  task extract_inactivated_at_from_git: :environment do
+  desc 'Git履歴からinactivated_at日付を抽出してYAMLファイルに反映（引数でDojo IDを指定可能）'
+  task :extract_inactivated_at_from_git, [:dojo_id] => :environment do |t, args|
     require 'git'
 
     yaml_path = Rails.root.join('db', 'dojos.yaml')
@@ -9,16 +9,23 @@ namespace :dojos do
     # YAMLファイルの内容を行番号付きで読み込む
     yaml_lines = File.readlines(yaml_path)
 
-    # 非アクティブなDojoを取得
-    inactive_dojos = Dojo.inactive.where(inactivated_at: nil)
+    # 対象Dojoを決定（引数があれば特定のDojo、なければ全ての非アクティブDojo）
+    target_dojos = if args[:dojo_id]
+      dojo = Dojo.find(args[:dojo_id])
+      puts "=== 特定のDojoのinactivated_at を抽出 ==="
+      puts "対象Dojo: #{dojo.name} (ID: #{dojo.id})"
+      [dojo]
+    else
+      inactive_dojos = Dojo.inactive.where(inactivated_at: nil)
+      puts "=== Git履歴から inactivated_at を抽出 ==="
+      puts "対象となる非アクティブDojo数: #{inactive_dojos.count}"
+      inactive_dojos
+    end
 
-    puts "=== Git履歴から inactivated_at を抽出 ==="
-    puts "対象となる非アクティブDojo数: #{inactive_dojos.count}"
     puts ""
-    
     updated_count = 0
 
-    inactive_dojos.each do |dojo|
+    target_dojos.each do |dojo|
       puts "処理中: #{dojo.name} (ID: #{dojo.id})"
 
       # is_active: false が記載されている行を探す
@@ -54,6 +61,15 @@ namespace :dojos do
           commit = git.gcommit(commit_id)
           inactivated_date = commit.author_date
 
+          # 特定Dojoモードの場合は情報表示のみ
+          if args[:dojo_id]
+            puts "✓ is_active: false に設定された日時: #{inactivated_date.strftime('%Y-%m-%d %H:%M:%S')}"
+            puts "  コミット: #{commit_id[0..7]}"
+            puts "  作者: #{commit.author.name}"
+            puts "  メッセージ: #{commit.message.lines.first.strip}"
+            next
+          end
+          
           # YAMLファイルのDojoブロックを見つけて更新
           yaml_updated = false
           yaml_lines.each_with_index do |line, index|
@@ -63,23 +79,30 @@ namespace :dojos do
               while insert_index < yaml_lines.length && !yaml_lines[insert_index].match?(/^- id:/)
                 # is_active: false の次の行に挿入したい
                 if yaml_lines[insert_index - 1].match?(/is_active: false/)
+                  # 既に inactivated_at がある場合はスキップ（冪等性）
+                  if yaml_lines[insert_index].match?(/^\s*inactivated_at:/)
+                    puts "  - inactivated_at は既に設定されています"
+                    yaml_updated = false
+                    break
+                  end
+                  
                   yaml_lines.insert(insert_index, 
                     "  inactivated_at: '#{inactivated_date.strftime('%Y-%m-%d %H:%M:%S')}'\n")
                   yaml_updated = true
                   break
                 end
                 insert_index += 1
               end
-              break if yaml_updated
+              break
             end
           end
 
           if yaml_updated
             updated_count += 1
             puts "  ✓ inactivated_at を追加: #{inactivated_date.strftime('%Y-%m-%d %H:%M:%S')}"
             puts "    コミット: #{commit_id[0..7]} by #{commit.author.name}"
-          else
-            puts "  ✗ YAMLファイルの更新に失敗"
+          elsif !args[:dojo_id]
+            puts "  - スキップ（既に設定済みまたは更新失敗）"
           end
         else
           puts "  ✗ コミット情報の取得に失敗"
@@ -91,8 +114,8 @@ namespace :dojos do
       puts ""
     end
 
-    if updated_count > 0
-      # YAMLファイルを書き戻す
+    # 全Dojoモードで更新があった場合のみYAMLファイルを書き戻す
+    if !args[:dojo_id] && updated_count > 0
       File.write(yaml_path, yaml_lines.join)
 
       puts "=== 完了 ==="
@@ -102,60 +125,9 @@ namespace :dojos do
       puts "1. db/dojos.yaml の変更内容を確認"
       puts "2. rails dojos:update_db_by_yaml を実行してDBに反映"
       puts "3. 変更をコミット"
-    else
+    elsif !args[:dojo_id]
       puts "=== 完了 ==="
-      puts "更新対象のDojoはありませんでした"
-    end
-  end
-  
-  desc '特定のDojoのinactivated_at日付をGit履歴から抽出'
-  task :extract_inactivated_at_for_dojo, [:dojo_id] => :environment do |t, args|
-    require 'git'
-    
-    dojo = Dojo.find(args[:dojo_id])
-    yaml_path = Rails.root.join('db', 'dojos.yaml')
-    git = Git.open(Rails.root)
-    
-    puts "対象Dojo: #{dojo.name} (ID: #{dojo.id})"
-    
-    # YAMLファイルの内容を読み込む
-    yaml_lines = File.readlines(yaml_path)
-    
-    # is_active: false が記載されている行を探す
-    target_line_number = nil
-    in_dojo_block = false
-    
-    yaml_lines.each_with_index do |line, index|
-      if line.match?(/^- id: #{dojo.id}$/)
-        in_dojo_block = true
-      elsif line.match?(/^- id: \d+$/)
-        in_dojo_block = false
-      end
-      
-      if in_dojo_block && line.match?(/^\s*is_active: false/)
-        target_line_number = index + 1
-        break
-      end
-    end
-    
-    if target_line_number
-      blame_cmd = "git blame #{yaml_path} -L #{target_line_number},+1 --porcelain"
-      blame_output = `#{blame_cmd}`.strip
-      commit_id = blame_output.lines[0].split.first
-      
-      if commit_id && commit_id.match?(/^[0-9a-f]{40}$/)
-        commit = git.gcommit(commit_id)
-        inactivated_date = commit.author_date
-        
-        puts "✓ is_active: false に設定された日時: #{inactivated_date.strftime('%Y-%m-%d %H:%M:%S')}"
-        puts "  コミット: #{commit_id[0..7]}"
-        puts "  作者: #{commit.author.name}"
-        puts "  メッセージ: #{commit.message.lines.first.strip}"
-      else
-        puts "✗ コミット情報の取得に失敗しました"
-      end
-    else
-      puts "✗ YAMLファイル内で 'is_active: false' 行が見つかりません"
+      puts "更新対象のDojoはありませんでした（または既に設定済み）"
     end
   end
 end