makit 0.0.139 → 0.0.141

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 8914cc91413f619374d83e3dbd6a339d1d8b2ddd3cdf6b97ba60e679a6086791
-  data.tar.gz: aa96a06181f3080162e6e1712f69a23d18a9dd85be0dd963fb263ac06618b397
+  metadata.gz: 5292d7e48fcc8060c1ad09ed8beeb9bb596c6d33fe33075e39f65553480532c2
+  data.tar.gz: be28ff936d5268a37689bd9f7d581144b755fee9ef310cb24e6ee2c85045517a
 SHA512:
-  metadata.gz: '059afb37ecd17ce37de9e74c6885032d6e17bf13c39e3de14e5334603b872c325b7d6a24408a03e3eb42517096ba8c7d9251ba3543cfc9421b7a46a6a9d8698d'
-  data.tar.gz: a068d31da5e80058570e92d9d1a5e26c9ca0f6458d0656c801dfce426935571b7dda46e9e03f0106e7f2f68c9431d8bad0019bb2a44132c1df8282d9fdfbb724
+  metadata.gz: ce30902a329a81775d0d023597daef878bcd78a692ee6c3de0cb85b325077422813638b28bcdd5a52b5991b677e5ca7966335a6cc8b0c1f6267dfd3ff7d86a6a
+  data.tar.gz: f16cb9d563051f87fd8b8693afde7415b21e75ac35d7ac98c22264c789d7065035d053d534153872d942b8add3241b34b7aab8c30f81191ff275cd76e58b152f

data/lib/makit/cli/main.rb

@@ -23,6 +23,7 @@ module Makit
           utility     - System utilities and maintenance (nuget-cache, system-info)
           strategy    - Manage execution strategies (show, test)
           trace       - Manage trace functionality (show, enable, disable, test)
+          timeout     - Manage timeout configuration (show, set, reset, test)
           rake        - Manage rake tasks
 
         Common usage examples:
@@ -61,6 +62,7 @@ module Makit
       subcommand "utility", "System utilities and maintenance", UtilityCommand
       subcommand "strategy", "Manage execution strategies", StrategyCommands
       subcommand "trace", "Manage trace functionality", TraceCommands
+      subcommand "timeout", "Manage timeout configuration", TimeoutCommands
       subcommand "rake", "Manage rake tasks", Makit::Rake::CliCommand
     end
   end

data/lib/makit/cli/strategy_commands.rb

@@ -1,152 +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
-  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

data/lib/makit/commands/request.rb

@@ -55,7 +55,7 @@ module Makit
         @arguments = validate_arguments(arguments)
         @environment = options[:environment] || {}
         @directory = options[:directory] || Dir.pwd
-        @timeout = options[:timeout] || 30
+        @timeout = options[:timeout] || Makit::Configuration::Timeout.global_default
         @metadata = options[:metadata] || {}
         @exit_on_failure = options[:exit_on_failure] || true
         @show_stderr = options[:show_stderr] || true

data/lib/makit/commands/runner.rb

@@ -179,6 +179,18 @@ module Makit
         }
       end
 
+      # Make this method public as it is used by rake hooks and CLI output
+      # Get information about the current strategy
+      #
+      # @return [Hash] strategy information
+      def strategy_info
+        {
+          class: @strategy.class.name,
+          type: @strategy.class.name.split('::').last.downcase,
+          factory_info: Strategies::Factory.strategy_info
+        }
+      end
+
       private
 
       # Execute request with middleware chain.
@@ -320,16 +332,7 @@ module Makit
         raise ArgumentError, "Strategy must respond to #supports?: #{@strategy.class}"
       end
 
-      # Get information about the current strategy
-      #
-      # @return [Hash] strategy information
-      def strategy_info
-        {
-          class: @strategy.class.name,
-          type: @strategy.class.name.split('::').last.downcase,
-          factory_info: Strategies::Factory.strategy_info
-        }
-      end
+      
 
       # Log command execution result using the default logger.
       #