makit 0.0.144 → 0.0.145

data/lib/makit/cli/strategy_commands.rb

@@ -1,212 +1,207 @@
-# 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
+  class TraceCommands < Cli::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 < Cli::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