makit 0.0.140 → 0.0.142

data/lib/makit/cli/strategy_commands.rb

@@ -1,203 +1,203 @@
-# frozen_string_literal: true
-
-require_relative "base"
-
-module Makit
-  module Cli
-    # Commands for managing execution strategies
-    class StrategyCommands < Base
-      desc "strategy", "Show current execution strategy information"
-      option ["--verbose", "-v"], :flag, "Show detailed strategy information"
-      def strategy
-        runner = Makit::Commands::Runner.default
-        info = runner.strategy_info
-
-        puts "Current Execution Strategy:"
-        puts "  Type: #{info[:type]}"
-        puts "  Class: #{info[:class]}"
-        
-        if options[:verbose]
-          puts "\nFactory Information:"
-          factory_info = info[:factory_info]
-          puts "  Current: #{factory_info[:current]}"
-          puts "  Available: #{factory_info[:available].join(', ')}"
-          puts "  ChildProcess Available: #{factory_info[:childprocess_available]}"
-          puts "  Open3 Available: #{factory_info[:open3_available]}"
-        end
-
-        puts "\nEnvironment Variables:"
-        puts "  MAKIT_STRATEGY: #{ENV['MAKIT_STRATEGY'] || 'not set (using auto-detect)'}"
-      end
-
-      desc "strategy test", "Test both strategies and show performance comparison"
-      option ["--timeout", "-t"], "TIMEOUT", "Timeout for test commands", default: "5"
-      def test
-        timeout = options[:timeout].to_i
-        
-        puts "Testing execution strategies..."
-        puts "=" * 50
-
-        # Test ChildProcess strategy
-        puts "\n1. Testing ChildProcess Strategy:"
-        childprocess_result = test_strategy('childprocess', timeout)
-        
-        # Test Open3 strategy  
-        puts "\n2. Testing Open3 Strategy:"
-        open3_result = test_strategy('open3', timeout)
-
-        # Show comparison
-        puts "\n" + "=" * 50
-        puts "Performance Comparison:"
-        puts "  ChildProcess: #{childprocess_result[:duration]}s (#{childprocess_result[:status]})"
-        puts "  Open3:       #{open3_result[:duration]}s (#{open3_result[:status]})"
-      end
-
-      private
-
-      # Test a specific strategy
-      #
-      # @param strategy_type [String] strategy type to test
-      # @param timeout [Integer] timeout for test commands
-      # @return [Hash] test results
-      def test_strategy(strategy_type, timeout)
-        start_time = Time.now
-        
-        begin
-          # Create runner with specific strategy
-          strategy = Makit::Commands::Strategies::Factory.create(strategy: strategy_type)
-          runner = Makit::Commands::Runner.new(strategy: strategy)
-          
-          # Test with a simple command
-          request = Makit::Commands::Request.new(
-            command: "echo",
-            arguments: ["Hello from #{strategy_type}!"],
-            timeout: timeout
-          )
-          
-          result = runner.execute(request)
-          duration = Time.now - start_time
-          
-          status = result.success? ? "SUCCESS" : "FAILED"
-          puts "  Status: #{status}"
-          puts "  Duration: #{duration.round(3)}s"
-          puts "  Output: #{result.stdout.strip}"
-          
-          if result.failure?
-            puts "  Error: #{result.stderr}"
-          end
-          
-          { duration: duration.round(3), status: status, success: result.success? }
-          
-        rescue => e
-          duration = Time.now - start_time
-          puts "  Status: ERROR"
-          puts "  Duration: #{duration.round(3)}s"
-          puts "  Error: #{e.message}"
-          
-        { duration: duration.round(3), status: "ERROR", success: false }
-      end
-    end
-  end
-end
-
-# Add trace commands to the main CLI
-module Makit
-  module Cli
-    # Commands for managing trace functionality
-    class TraceCommands < Base
-      desc "trace", "Show current trace status and configuration"
-      option ["--verbose", "-v"], :flag, "Show detailed trace information"
-      def trace
-        puts "Makit Trace Status:"
-        puts "=" * 30
-        
-        status = Makit::Rake.status
-        
-        puts "Enhanced Tracing: #{Makit::Rake.tracing_enabled? ? 'Enabled' : 'Disabled'}"
-        puts "Rake Trace: #{status[:trace_controller][:rake_trace] || 'Not set'}"
-        puts "Trace Flag: #{status[:trace_controller][:trace_flag] ? 'Present' : 'Not present'}"
-        puts "Makit Trace: #{status[:trace_controller][:makit_trace] || 'Not set'}"
-        puts "Verbose Mode: #{status[:verbose] ? 'Enabled' : 'Disabled'}"
-        
-        if options[:verbose]
-          puts "\nDetailed Information:"
-          puts "  Strategy: #{status[:strategy][:type] || 'Unknown'}"
-          puts "  Class: #{status[:strategy][:class] || 'Unknown'}"
-          puts "  Factory Info: #{status[:strategy][:factory_info] || 'Not available'}"
-        end
-      end
-
-      desc "trace enable", "Enable enhanced tracing"
-      def enable
-        ENV['MAKIT_TRACE'] = 'true'
-        puts "Enhanced tracing enabled"
-        puts "Run 'rake --trace' to see enhanced output"
-      end
-
-      desc "trace disable", "Disable enhanced tracing"
-      def disable
-        ENV.delete('MAKIT_TRACE')
-        puts "Enhanced tracing disabled"
-      end
-
-      desc "trace test", "Test trace functionality with a simple rake task"
-      def test
-        puts "Testing trace functionality..."
-        puts "Run: rake --trace"
-        puts "You should see [MAKIT] prefixed trace output for makit-related tasks"
-      end
-    end
-
-    # Commands for managing timeout configuration
-    class TimeoutCommands < Base
-      desc "timeout", "Show current timeout configuration"
-      option ["--verbose", "-v"], :flag, "Show detailed timeout information"
-      def timeout
-        puts "Makit Timeout Configuration:"
-        puts "=" * 35
-        
-        puts "Global Default: #{Makit::Configuration::Timeout.global_default}s"
-        puts "Environment: #{ENV['MAKIT_DEFAULT_TIMEOUT'] || 'Not set'}"
-        
-        if options[:verbose]
-          puts "\nOperation-Specific Timeouts:"
-          Makit::Configuration::Timeout.all_timeouts.each do |operation, timeout|
-            puts "  #{operation.to_s.ljust(20)}: #{timeout}s"
-          end
-        end
-      end
-
-      desc "timeout set", "Set global default timeout"
-      option ["--timeout", "-t"], "TIMEOUT", "Timeout in seconds", required: true
-      def set
-        timeout = options[:timeout].to_i
-        Makit::Configuration::Timeout.validate_timeout(timeout)
-        ENV['MAKIT_DEFAULT_TIMEOUT'] = timeout.to_s
-        puts "Global timeout set to #{timeout} seconds"
-      end
-
-      desc "timeout reset", "Reset timeout to default value"
-      def reset
-        ENV.delete('MAKIT_DEFAULT_TIMEOUT')
-        puts "Timeout reset to default: #{Makit::Configuration::Timeout.global_default} seconds"
-      end
-
-      desc "timeout test", "Test timeout configuration with a simple command"
-      option ["--timeout", "-t"], "TIMEOUT", "Timeout for test command", default: "5"
-      def test
-        timeout = options[:timeout].to_i
-        puts "Testing timeout configuration with #{timeout}s timeout..."
-        
-        # Test with a simple command
-        result = Makit::Commands::Runner.default.run("echo", "Timeout test", timeout: timeout)
-        
-        if result.success?
-          puts "✓ Timeout test passed"
-        else
-          puts "✗ Timeout test failed: #{result.stderr}"
-        end
-      end
-    end
-  end
-end
-end
+# frozen_string_literal: true
+
+require_relative "base"
+
+module Makit
+  module Cli
+    # Commands for managing execution strategies
+    class StrategyCommands < Base
+      desc "strategy", "Show current execution strategy information"
+      option ["--verbose", "-v"], :flag, "Show detailed strategy information"
+      def strategy
+        runner = Makit::Commands::Runner.default
+        info = runner.strategy_info
+
+        puts "Current Execution Strategy:"
+        puts "  Type: #{info[:type]}"
+        puts "  Class: #{info[:class]}"
+        
+        if options[:verbose]
+          puts "\nFactory Information:"
+          factory_info = info[:factory_info]
+          puts "  Current: #{factory_info[:current]}"
+          puts "  Available: #{factory_info[:available].join(', ')}"
+          puts "  ChildProcess Available: #{factory_info[:childprocess_available]}"
+          puts "  Open3 Available: #{factory_info[:open3_available]}"
+        end
+
+        puts "\nEnvironment Variables:"
+        puts "  MAKIT_STRATEGY: #{ENV['MAKIT_STRATEGY'] || 'not set (using auto-detect)'}"
+      end
+
+      desc "strategy test", "Test both strategies and show performance comparison"
+      option ["--timeout", "-t"], "TIMEOUT", "Timeout for test commands", default: "5"
+      def test
+        timeout = options[:timeout].to_i
+        
+        puts "Testing execution strategies..."
+        puts "=" * 50
+
+        # Test ChildProcess strategy
+        puts "\n1. Testing ChildProcess Strategy:"
+        childprocess_result = test_strategy('childprocess', timeout)
+        
+        # Test Open3 strategy  
+        puts "\n2. Testing Open3 Strategy:"
+        open3_result = test_strategy('open3', timeout)
+
+        # Show comparison
+        puts "\n" + "=" * 50
+        puts "Performance Comparison:"
+        puts "  ChildProcess: #{childprocess_result[:duration]}s (#{childprocess_result[:status]})"
+        puts "  Open3:       #{open3_result[:duration]}s (#{open3_result[:status]})"
+      end
+
+      private
+
+      # Test a specific strategy
+      #
+      # @param strategy_type [String] strategy type to test
+      # @param timeout [Integer] timeout for test commands
+      # @return [Hash] test results
+      def test_strategy(strategy_type, timeout)
+        start_time = Time.now
+        
+        begin
+          # Create runner with specific strategy
+          strategy = Makit::Commands::Strategies::Factory.create(strategy: strategy_type)
+          runner = Makit::Commands::Runner.new(strategy: strategy)
+          
+          # Test with a simple command
+          request = Makit::Commands::Request.new(
+            command: "echo",
+            arguments: ["Hello from #{strategy_type}!"],
+            timeout: timeout
+          )
+          
+          result = runner.execute(request)
+          duration = Time.now - start_time
+          
+          status = result.success? ? "SUCCESS" : "FAILED"
+          puts "  Status: #{status}"
+          puts "  Duration: #{duration.round(3)}s"
+          puts "  Output: #{result.stdout.strip}"
+          
+          if result.failure?
+            puts "  Error: #{result.stderr}"
+          end
+          
+          { duration: duration.round(3), status: status, success: result.success? }
+          
+        rescue => e
+          duration = Time.now - start_time
+          puts "  Status: ERROR"
+          puts "  Duration: #{duration.round(3)}s"
+          puts "  Error: #{e.message}"
+          
+        { duration: duration.round(3), status: "ERROR", success: false }
+      end
+    end
+  end
+end
+
+# Add trace commands to the main CLI
+module Makit
+  module Cli
+    # Commands for managing trace functionality
+    class TraceCommands < Base
+      desc "trace", "Show current trace status and configuration"
+      option ["--verbose", "-v"], :flag, "Show detailed trace information"
+      def trace
+        puts "Makit Trace Status:"
+        puts "=" * 30
+        
+        status = Makit::Rake.status
+        
+        puts "Enhanced Tracing: #{Makit::Rake.tracing_enabled? ? 'Enabled' : 'Disabled'}"
+        puts "Rake Trace: #{status[:trace_controller][:rake_trace] || 'Not set'}"
+        puts "Trace Flag: #{status[:trace_controller][:trace_flag] ? 'Present' : 'Not present'}"
+        puts "Makit Trace: #{status[:trace_controller][:makit_trace] || 'Not set'}"
+        puts "Verbose Mode: #{status[:verbose] ? 'Enabled' : 'Disabled'}"
+        
+        if options[:verbose]
+          puts "\nDetailed Information:"
+          puts "  Strategy: #{status[:strategy][:type] || 'Unknown'}"
+          puts "  Class: #{status[:strategy][:class] || 'Unknown'}"
+          puts "  Factory Info: #{status[:strategy][:factory_info] || 'Not available'}"
+        end
+      end
+
+      desc "trace enable", "Enable enhanced tracing"
+      def enable
+        ENV['MAKIT_TRACE'] = 'true'
+        puts "Enhanced tracing enabled"
+        puts "Run 'rake --trace' to see enhanced output"
+      end
+
+      desc "trace disable", "Disable enhanced tracing"
+      def disable
+        ENV.delete('MAKIT_TRACE')
+        puts "Enhanced tracing disabled"
+      end
+
+      desc "trace test", "Test trace functionality with a simple rake task"
+      def test
+        puts "Testing trace functionality..."
+        puts "Run: rake --trace"
+        puts "You should see [MAKIT] prefixed trace output for makit-related tasks"
+      end
+    end
+
+    # Commands for managing timeout configuration
+    class TimeoutCommands < Base
+      desc "timeout", "Show current timeout configuration"
+      option ["--verbose", "-v"], :flag, "Show detailed timeout information"
+      def timeout
+        puts "Makit Timeout Configuration:"
+        puts "=" * 35
+        
+        puts "Global Default: #{Makit::Configuration::Timeout.global_default}s"
+        puts "Environment: #{ENV['MAKIT_DEFAULT_TIMEOUT'] || 'Not set'}"
+        
+        if options[:verbose]
+          puts "\nOperation-Specific Timeouts:"
+          Makit::Configuration::Timeout.all_timeouts.each do |operation, timeout|
+            puts "  #{operation.to_s.ljust(20)}: #{timeout}s"
+          end
+        end
+      end
+
+      desc "timeout set", "Set global default timeout"
+      option ["--timeout", "-t"], "TIMEOUT", "Timeout in seconds", required: true
+      def set
+        timeout = options[:timeout].to_i
+        Makit::Configuration::Timeout.validate_timeout(timeout)
+        ENV['MAKIT_DEFAULT_TIMEOUT'] = timeout.to_s
+        puts "Global timeout set to #{timeout} seconds"
+      end
+
+      desc "timeout reset", "Reset timeout to default value"
+      def reset
+        ENV.delete('MAKIT_DEFAULT_TIMEOUT')
+        puts "Timeout reset to default: #{Makit::Configuration::Timeout.global_default} seconds"
+      end
+
+      desc "timeout test", "Test timeout configuration with a simple command"
+      option ["--timeout", "-t"], "TIMEOUT", "Timeout for test command", default: "5"
+      def test
+        timeout = options[:timeout].to_i
+        puts "Testing timeout configuration with #{timeout}s timeout..."
+        
+        # Test with a simple command
+        result = Makit::Commands::Runner.default.run("echo", "Timeout test", timeout: timeout)
+        
+        if result.success?
+          puts "✓ Timeout test passed"
+        else
+          puts "✗ Timeout test failed: #{result.stderr}"
+        end
+      end
+    end
+  end
+end
+end