refactor: IPアドレス検証ロジックをSakuraServerUserAgentクラスに統一

- DRY原則に従い、重複していたIP検証ロジックを共通化
- valid_ip_address?メソッド: Rubyらしく?付きで真偽値を返す
- normalize_ip_addressメソッド: IP正規化を別途提供
- 不要なラッパーメソッドとVALID_IP_PATTERN定数を削除

Author: yasulab

Rakefile

@@ -1,5 +1,4 @@
 require "rspec/core/rake_task"
-require 'ipaddr'
 require 'fileutils'
 require 'json'
 require 'time'
@@ -94,27 +93,16 @@ namespace :server do
             "または: IP_ADDRESS=192.168.1.1 rake server:find_by_ip"
     end
 
-    # セキュリティのためRubyのIPAddrを使用してIPアドレスを検証
-    begin
-      validated_ip = IPAddr.new(ip)
-      
-      # プライベート/特殊IPをチェック
-      if validated_ip.private? || validated_ip.loopback? || validated_ip.link_local?
-        abort "❌ エラー: プライベートまたは特殊IPアドレスは許可されていません: #{ip}"
-      end
-      
-      # さくらクラウドのIP範囲の追加検証（オプション）
-      if ENV['VALIDATE_SAKURA_RANGE'] == 'true'
-        unless in_sakura_cloud_range?(validated_ip)
-          abort "❌ エラー: IPアドレスがさくらクラウドの範囲外です: #{ip}"
-        end
-      end
-      
-      validated_ip_str = validated_ip.to_s
-    rescue IPAddr::InvalidAddressError => e
-      abort "❌ エラー: 無効なIPアドレス形式: #{ip}\n#{e.message}"
+    # IPアドレスの検証（SakuraServerUserAgentの共通メソッドを使用）
+    require_relative 'scripts/sakura_server_user_agent'
+    
+    unless SakuraServerUserAgent.valid_ip_address?(ip)
+      abort "❌ エラー: 無効なIPアドレス形式: #{ip}"
     end
 
+    # IPアドレスを正規化
+    validated_ip_str = SakuraServerUserAgent.normalize_ip_address(ip)
+    
     puts "✅ 有効なIPアドレス: #{validated_ip_str}"
     puts "🔍 サーバー情報を検索中..."
     puts "-" * 50
@@ -359,51 +347,53 @@ namespace :server do
   end
 end
 
-# ヘルパーメソッド（将来のフェーズで拡張予定）
-def in_sakura_cloud_range?(ip_addr)
-  # さくらクラウドのIP範囲（現時点では簡略化）
-  sakura_ranges = [
-    IPAddr.new("153.127.0.0/16"),  # 石狩第二ゾーン
-    IPAddr.new("163.43.0.0/16"),   # 東京ゾーン
-    IPAddr.new("133.242.0.0/16"),  # 大阪ゾーン
-  ]
-  
-  sakura_ranges.any? { |range| range.include?(ip_addr) }
-end
+# ヘルパーメソッド（将来の拡張用に保持）
+# 注: 現在は使用されていません（YAGNI原則により簡素化）
+# def in_sakura_cloud_range?(ip_addr)
+#   sakura_ranges = [
+#     IPAddr.new("153.127.0.0/16"),  # 石狩第二ゾーン
+#     IPAddr.new("163.43.0.0/16"),   # 東京ゾーン
+#     IPAddr.new("133.242.0.0/16"),  # 大阪ゾーン
+#   ]
+#   sakura_ranges.any? { |range| range.include?(ip_addr) }
+# end
 
 # ================================================================
-# 並列実行タスク（Rakeの高度な機能）
+# 並列実行タスク（将来の実装用にコメントアウト）
 # ================================================================
-namespace :parallel do
-  desc "複数サーバーの状態を並列チェック"
-  multitask :check_all => ['server:validate_env'] do
-    # servers.csvから全サーバーをチェック
-    servers = CSV.read('servers.csv', headers: true)
-    
-    # 並列でステータスチェックを実行
-    threads = servers.map do |server|
-      Thread.new do
-        begin
-          result = `ruby scripts/initialize_server.rb --find #{server['Name']} 2>&1`
-          { name: server['Name'], status: $?.success? ? 'OK' : 'ERROR', details: result }
-        rescue => e
-          { name: server['Name'], status: 'ERROR', details: e.message }
-        end
-      end
-    end
-    
-    results = threads.map(&:value)
-    
-    # 結果をサマリー表示
-    puts "\n" + "=" * 50
-    puts "サーバーステータスサマリー"
-    puts "=" * 50
-    results.each do |r|
-      status_icon = r[:status] == 'OK' ? '✅' : '❌'
-      puts "#{status_icon} #{r[:name]}: #{r[:status]}"
-    end
-  end
-end
+# YAGNI原則により、実際に必要になるまでコメントアウト
+# 注意: 200サーバーの並列チェックはAPI制限のリスクあり
+#
+# namespace :parallel do
+#   desc "複数サーバーの状態を並列チェック"
+#   multitask :check_all => ['server:validate_env'] do
+#     # servers.csvから全サーバーをチェック
+#     servers = CSV.read('servers.csv', headers: true)
+#     
+#     # 並列でステータスチェックを実行
+#     threads = servers.map do |server|
+#       Thread.new do
+#         begin
+#           result = `ruby scripts/initialize_server.rb --find #{server['Name']} 2>&1`
+#           { name: server['Name'], status: $?.success? ? 'OK' : 'ERROR', details: result }
+#         rescue => e
+#           { name: server['Name'], status: 'ERROR', details: e.message }
+#         end
+#       end
+#     end
+#     
+#     results = threads.map(&:value)
+#     
+#     # 結果をサマリー表示
+#     puts "\n" + "=" * 50
+#     puts "サーバーステータスサマリー"
+#     puts "=" * 50
+#     results.each do |r|
+#       status_icon = r[:status] == 'OK' ? '✅' : '❌'
+#       puts "#{status_icon} #{r[:name]}: #{r[:status]}"
+#     end
+#   end
+# end
 
 # ================================================================
 # クリーンタスク（Rake標準機能の活用）

scripts/initialize_server.rb

@@ -28,7 +28,8 @@ class ServerInitializer
   IP_PATTERN = /(?:IPアドレス|IP)[：:]\s*【?(\d{1,3}\.\d{1,3}\.\d{1,3}\.\d{1,3})】?/
 
   # IPアドレスの厳密な検証パターン
-  VALID_IP_PATTERN = /\A(?:(?:25[0-5]|2[0-4][0-9]|[01]?[0-9][0-9]?)\.){3}(?:25[0-5]|2[0-4][0-9]|[01]?[0-9][0-9]?)\z/
+  # 注: RakefileでIPAddrクラスによる検証を行うため、ここでの重複検証は不要（YAGNI）
+  # VALID_IP_PATTERN = /\A(?:(?:25[0-5]|2[0-4][0-9]|[01]?[0-9][0-9]?)\.){3}(?:25[0-5]|2[0-4][0-9]|[01]?[0-9][0-9]?)\z/
 
 
   # テスト用サーバー名の安全管理（誤削除防止）
@@ -121,7 +122,7 @@ def show_help
   # 安全性チェック: テスト用サーバー以外の削除には追加確認
   def confirm_safe_deletion(target)
     # IPアドレスの場合はサーバー名を取得
-    if valid_ip_address?(target)
+    if SakuraServerUserAgent.valid_ip_address?(target)
       server_info = find_server_by_ip(target)
       server_name = server_info ? server_info['Name'] : nil
     else
@@ -161,7 +162,7 @@ def run_delete_mode
     puts ""
 
     # IPアドレスの検証
-    unless valid_ip_address?(@input)
+    unless SakuraServerUserAgent.valid_ip_address?(@input)
       puts "❌ エラー: 無効なIPアドレス形式です: #{@input}"
       puts ""
       puts "正しいIPアドレス形式で指定してください（例: 192.168.1.1）"
@@ -366,11 +367,6 @@ def find_by_name
     display_deletion_plan(server_info, ip_address, dojo_name)
   end
 
-  # IPアドレスの検証
-  def valid_ip_address?(ip)
-    return false if ip.nil? || ip.empty?
-    !!(ip =~ VALID_IP_PATTERN)
-  end
 
   # サーバー情報の詳細表示（削除用）
   def display_server_details_for_deletion(server)

scripts/sakura_server_user_agent.rb

@@ -1,5 +1,6 @@
 #!/usr/bin/env ruby
 require_relative 'smart_wait_helper'
+require 'ipaddr'  # IP検証用
 
 class SakuraServerUserAgent
   include SmartWaitHelper
@@ -26,6 +27,33 @@ class SakuraServerUserAgent
   # gh-pagesブランチで公開される実際のサーバー情報
   INSTANCES_CSV_URL = "https://raw.githubusercontent.com/coderdojo-japan/dojopaas/refs/heads/gh-pages/instances.csv"
 
+  # IPアドレス検証用のクラスメソッド（共通化）
+  # @param ip [String] 検証するIPアドレス
+  # @return [Boolean] 有効なIPアドレスの場合true、無効な場合false
+  def self.valid_ip_address?(ip)
+    return false if ip.nil? || ip.empty?
+    
+    begin
+      IPAddr.new(ip)
+      true
+    rescue IPAddr::InvalidAddressError
+      false
+    end
+  end
+  
+  # IPアドレスを正規化して返す
+  # @param ip [String] 正規化するIPアドレス
+  # @return [String, nil] 正規化されたIPアドレス、無効な場合はnil
+  def self.normalize_ip_address(ip)
+    return nil if ip.nil? || ip.empty?
+    
+    begin
+      IPAddr.new(ip).to_s
+    rescue IPAddr::InvalidAddressError
+      nil
+    end
+  end
+
   # jsのserver.createで使っているフィールドを参考
   def initialize(zone:0, packet_filter_id:nil, name:nil, description:nil, zone_id:"is1b",
                  tags:nil, pubkey:nil, resolve:nil, verbose:false)