makit 0.0.111 → 0.0.112

data/lib/makit/commands/middleware/base.rb

@@ -1,73 +1,73 @@
-# frozen_string_literal: true
-
-module Makit
-  module Commands
-    module Middleware
-      # Base class for all command execution middleware.
-      #
-      # Middleware provides a way to add cross-cutting concerns like logging,
-      # caching, validation, and timing to command execution without modifying
-      # the core execution logic.
-      #
-      # @example Creating custom middleware
-      #   class CustomMiddleware < Base
-      #     def call(request, &block)
-      #       # Pre-execution logic
-      #       puts "About to execute: #{request.command}"
-      #
-      #       # Execute the command
-      #       result = block.call(request)
-      #
-      #       # Post-execution logic
-      #       puts "Execution completed: #{result.success? ? 'SUCCESS' : 'FAILED'}"
-      #
-      #       result
-      #     end
-      #   end
-      class Base
-        # Execute middleware logic.
-        #
-        # This method must be implemented by subclasses to provide the actual
-        # middleware functionality. The pattern is to perform any pre-execution
-        # logic, call the block to continue the middleware chain, then perform
-        # any post-execution logic.
-        #
-        # @param request [Request] the command request to process
-        # @yield [Request] yields the processed request to the next middleware
-        # @yieldreturn [Result] the execution result
-        # @return [Result] the final execution result
-        # @raise [NotImplementedError] if not overridden by subclass
-        def call(request, &block)
-          raise NotImplementedError, "#{self.class.name} must implement #call"
-        end
-
-        # Check if this middleware should be applied to the given request.
-        #
-        # Override this method to provide conditional middleware application
-        # based on request properties.
-        #
-        # @param request [Request] the command request
-        # @return [Boolean] true if middleware should be applied
-        def applicable?(_request)
-          true
-        end
-
-        # Get middleware name for logging and debugging.
-        #
-        # @return [String] middleware name
-        def name
-          self.class.name.split("::").last
-        end
-
-        # Get middleware configuration.
-        #
-        # Override this method to provide middleware-specific configuration.
-        #
-        # @return [Hash] middleware configuration
-        def config
-          {}
-        end
-      end
-    end
-  end
-end
+# frozen_string_literal: true
+
+module Makit
+  module Commands
+    module Middleware
+      # Base class for all command execution middleware.
+      #
+      # Middleware provides a way to add cross-cutting concerns like logging,
+      # caching, validation, and timing to command execution without modifying
+      # the core execution logic.
+      #
+      # @example Creating custom middleware
+      #   class CustomMiddleware < Base
+      #     def call(request, &block)
+      #       # Pre-execution logic
+      #       puts "About to execute: #{request.command}"
+      #
+      #       # Execute the command
+      #       result = block.call(request)
+      #
+      #       # Post-execution logic
+      #       puts "Execution completed: #{result.success? ? 'SUCCESS' : 'FAILED'}"
+      #
+      #       result
+      #     end
+      #   end
+      class Base
+        # Execute middleware logic.
+        #
+        # This method must be implemented by subclasses to provide the actual
+        # middleware functionality. The pattern is to perform any pre-execution
+        # logic, call the block to continue the middleware chain, then perform
+        # any post-execution logic.
+        #
+        # @param request [Request] the command request to process
+        # @yield [Request] yields the processed request to the next middleware
+        # @yieldreturn [Result] the execution result
+        # @return [Result] the final execution result
+        # @raise [NotImplementedError] if not overridden by subclass
+        def call(request, &block)
+          raise NotImplementedError, "#{self.class.name} must implement #call"
+        end
+
+        # Check if this middleware should be applied to the given request.
+        #
+        # Override this method to provide conditional middleware application
+        # based on request properties.
+        #
+        # @param request [Request] the command request
+        # @return [Boolean] true if middleware should be applied
+        def applicable?(_request)
+          true
+        end
+
+        # Get middleware name for logging and debugging.
+        #
+        # @return [String] middleware name
+        def name
+          self.class.name.split("::").last
+        end
+
+        # Get middleware configuration.
+        #
+        # Override this method to provide middleware-specific configuration.
+        #
+        # @return [Hash] middleware configuration
+        def config
+          {}
+        end
+      end
+    end
+  end
+end

data/lib/makit/commands/middleware/cache.rb

@@ -1,248 +1,248 @@
-# frozen_string_literal: true
-
-require "fileutils"
-require "digest"
-
-module Makit
-  module Commands
-    module Middleware
-      # Caching middleware for command execution results
-      #
-      # This middleware provides caching functionality similar to the legacy CommandRunner,
-      # allowing commands to be cached based on their content and timestamp.
-      #
-      # @example Basic usage
-      #   runner = Commands::Runner.new(middleware: [Cache.new])
-      #   result = runner.execute("git --version")
-      #
-      # @example With custom cache directory
-      #   cache = Cache.new(cache_dir: "custom/cache")
-      #   runner = Commands::Runner.new(middleware: [cache])
-      class Cache < Base
-        # @!attribute [r] cache_dir
-        #   @return [String] cache directory path
-        attr_reader :cache_dir
-
-        # @!attribute [r] enabled
-        #   @return [Boolean] whether caching is enabled
-        attr_reader :enabled
-
-        # Initialize caching middleware
-        #
-        # @param cache_dir [String] directory to store cache files
-        # @param enabled [Boolean] whether caching is enabled
-        # @param options [Hash] additional options
-        def initialize(cache_dir: nil, enabled: true, **options)
-          @cache_dir = cache_dir || default_cache_dir
-          @enabled = enabled
-          @options = options
-
-          # Ensure cache directory exists
-          FileUtils.mkdir_p(@cache_dir) if @enabled
-        end
-
-        # Check if this middleware applies to the given request
-        #
-        # @param request [Request] command request
-        # @return [Boolean] true if caching should be applied
-        def applicable?(request)
-          @enabled && request.metadata[:cache_key].present?
-        end
-
-        # Execute the middleware chain with caching
-        #
-        # @param request [Request] command request
-        # @yield [Request] yields to next middleware or execution
-        # @return [Result] execution result
-        def call(request, &block)
-          return block.call(request) unless applicable?(request)
-
-          cache_key = request.metadata[:cache_key]
-          cache_timestamp = request.metadata[:cache_timestamp]
-          cache_file = get_cache_file(cache_key)
-
-          # Check if we have a valid cached result
-          if cache_file && File.exist?(cache_file) && cache_timestamp
-            cached_result = load_cached_result(cache_file, cache_timestamp)
-            return cached_result if cached_result
-          end
-
-          # Execute the command
-          result = block.call(request)
-
-          # Cache the result if successful
-          cache_result(cache_file, result) if result.success?
-
-          result
-        end
-
-        # Get middleware configuration
-        #
-        # @return [Hash] configuration hash
-        def config
-          {
-            cache_dir: @cache_dir,
-            enabled: @enabled,
-            options: @options,
-          }
-        end
-
-        private
-
-        # Get the default cache directory
-        #
-        # @return [String] default cache directory path
-        def default_cache_dir
-          File.join(Makit::Directories::PROJECT_ARTIFACTS, "commands", "cache")
-        end
-
-        # Get cache file path for a given key
-        #
-        # @param cache_key [String] cache key
-        # @return [String] cache file path
-        def get_cache_file(cache_key)
-          return nil unless cache_key
-
-          File.join(@cache_dir, "#{cache_key}.pb")
-        end
-
-        # Load cached result if it's newer than the timestamp
-        #
-        # @param cache_file [String] path to cache file
-        # @param timestamp [Time] minimum timestamp for cache validity
-        # @return [Result, nil] cached result or nil if not valid
-        def load_cached_result(cache_file, timestamp)
-          return nil unless File.exist?(cache_file)
-
-          cache_mtime = File.mtime(cache_file)
-          return nil if cache_mtime <= timestamp
-
-          begin
-            # Load the cached command result
-            cached_command = Makit::Serializer.open(cache_file, Makit::V1::Command)
-
-            # Convert to new Result format
-            convert_cached_command_to_result(cached_command)
-          rescue => e
-            Makit::Logging.debug("Failed to load cached result", {
-              cache_file: cache_file,
-              error: e.message,
-            })
-            nil
-          end
-        end
-
-        # Cache a successful result
-        #
-        # @param cache_file [String] path to cache file
-        # @param result [Result] result to cache
-        def cache_result(cache_file, result)
-          return unless cache_file && result.success?
-
-          begin
-            # Convert Result to legacy Command format for caching
-            legacy_command = convert_result_to_legacy_command(result)
-
-            # Save to cache
-            Makit::Serializer.save_as(cache_file, legacy_command)
-
-            Makit::Logging.debug("Cached command result", {
-              cache_file: cache_file,
-              command: result.command,
-              success: result.success?,
-            })
-          rescue => e
-            Makit::Logging.warn("Failed to cache result", {
-              cache_file: cache_file,
-              error: e.message,
-            })
-          end
-        end
-
-        # Convert cached Command to Result format
-        #
-        # @param cached_command [Makit::V1::Command] cached command
-        # @return [Result] converted result
-        def convert_cached_command_to_result(cached_command)
-          # Create a Result object from the cached command
-          Result.new(
-            command: "#{cached_command.name} #{cached_command.arguments.join(" ")}",
-            exit_code: cached_command.exit_code,
-            stdout: cached_command.output,
-            stderr: cached_command.error,
-            started_at: convert_protobuf_timestamp_to_time(cached_command.started_at),
-            duration: convert_protobuf_duration_to_float(cached_command.duration),
-            metadata: {
-              cached: true,
-              directory: cached_command.directory,
-            },
-          )
-        end
-
-        # Convert Result to legacy Command format for caching
-        #
-        # @param result [Result] result to convert
-        # @return [Makit::V1::Command] legacy command object
-        def convert_result_to_legacy_command(result)
-          command_parts = result.command.split
-          name = command_parts.first
-          arguments = command_parts[1..] || []
-
-          Makit::V1::Command.new(
-            name: name,
-            arguments: arguments,
-            exit_code: result.exit_code,
-            output: result.stdout,
-            error: result.stderr,
-            started_at: convert_time_to_protobuf_timestamp(result.started_at),
-            duration: convert_duration_to_protobuf(result.duration),
-            directory: result.metadata[:directory] || Dir.pwd,
-          )
-        end
-
-        # Convert protobuf timestamp to Time
-        #
-        # @param timestamp [Google::Protobuf::Timestamp] protobuf timestamp
-        # @return [Time, nil] converted time
-        def convert_protobuf_timestamp_to_time(timestamp)
-          return nil unless timestamp
-
-          Time.at(timestamp.seconds, timestamp.nanos, :nsec)
-        end
-
-        # Convert protobuf duration to float seconds
-        #
-        # @param duration [Google::Protobuf::Duration] protobuf duration
-        # @return [Float, nil] duration in seconds
-        def convert_protobuf_duration_to_float(duration)
-          return nil unless duration
-
-          duration.seconds + (duration.nanos / 1_000_000_000.0)
-        end
-
-        # Convert Time to protobuf timestamp
-        #
-        # @param time [Time] time to convert
-        # @return [Google::Protobuf::Timestamp] protobuf timestamp
-        def convert_time_to_protobuf_timestamp(time)
-          return nil unless time
-
-          Google::Protobuf::Timestamp.new(seconds: time.to_i, nanos: time.nsec)
-        end
-
-        # Convert duration to protobuf duration
-        #
-        # @param duration [Float] duration in seconds
-        # @return [Google::Protobuf::Duration] protobuf duration
-        def convert_duration_to_protobuf(duration)
-          return nil unless duration
-
-          seconds = duration.to_i
-          nanos = ((duration - seconds) * 1_000_000_000).to_i
-
-          Google::Protobuf::Duration.new(seconds: seconds, nanos: nanos)
-        end
-      end
-    end
-  end
-end
+# frozen_string_literal: true
+
+require "fileutils"
+require "digest"
+
+module Makit
+  module Commands
+    module Middleware
+      # Caching middleware for command execution results
+      #
+      # This middleware provides caching functionality for command execution,
+      # allowing commands to be cached based on their content and timestamp.
+      #
+      # @example Basic usage
+      #   runner = Commands::Runner.new(middleware: [Cache.new])
+      #   result = runner.execute("git --version")
+      #
+      # @example With custom cache directory
+      #   cache = Cache.new(cache_dir: "custom/cache")
+      #   runner = Commands::Runner.new(middleware: [cache])
+      class Cache < Base
+        # @!attribute [r] cache_dir
+        #   @return [String] cache directory path
+        attr_reader :cache_dir
+
+        # @!attribute [r] enabled
+        #   @return [Boolean] whether caching is enabled
+        attr_reader :enabled
+
+        # Initialize caching middleware
+        #
+        # @param cache_dir [String] directory to store cache files
+        # @param enabled [Boolean] whether caching is enabled
+        # @param options [Hash] additional options
+        def initialize(cache_dir: nil, enabled: true, **options)
+          @cache_dir = cache_dir || default_cache_dir
+          @enabled = enabled
+          @options = options
+
+          # Ensure cache directory exists
+          FileUtils.mkdir_p(@cache_dir) if @enabled
+        end
+
+        # Check if this middleware applies to the given request
+        #
+        # @param request [Request] command request
+        # @return [Boolean] true if caching should be applied
+        def applicable?(request)
+          @enabled && request.metadata[:cache_key].present?
+        end
+
+        # Execute the middleware chain with caching
+        #
+        # @param request [Request] command request
+        # @yield [Request] yields to next middleware or execution
+        # @return [Result] execution result
+        def call(request, &block)
+          return block.call(request) unless applicable?(request)
+
+          cache_key = request.metadata[:cache_key]
+          cache_timestamp = request.metadata[:cache_timestamp]
+          cache_file = get_cache_file(cache_key)
+
+          # Check if we have a valid cached result
+          if cache_file && File.exist?(cache_file) && cache_timestamp
+            cached_result = load_cached_result(cache_file, cache_timestamp)
+            return cached_result if cached_result
+          end
+
+          # Execute the command
+          result = block.call(request)
+
+          # Cache the result if successful
+          cache_result(cache_file, result) if result.success?
+
+          result
+        end
+
+        # Get middleware configuration
+        #
+        # @return [Hash] configuration hash
+        def config
+          {
+            cache_dir: @cache_dir,
+            enabled: @enabled,
+            options: @options,
+          }
+        end
+
+        private
+
+        # Get the default cache directory
+        #
+        # @return [String] default cache directory path
+        def default_cache_dir
+          File.join(Makit::Directories::PROJECT_ARTIFACTS, "commands", "cache")
+        end
+
+        # Get cache file path for a given key
+        #
+        # @param cache_key [String] cache key
+        # @return [String] cache file path
+        def get_cache_file(cache_key)
+          return nil unless cache_key
+
+          File.join(@cache_dir, "#{cache_key}.pb")
+        end
+
+        # Load cached result if it's newer than the timestamp
+        #
+        # @param cache_file [String] path to cache file
+        # @param timestamp [Time] minimum timestamp for cache validity
+        # @return [Result, nil] cached result or nil if not valid
+        def load_cached_result(cache_file, timestamp)
+          return nil unless File.exist?(cache_file)
+
+          cache_mtime = File.mtime(cache_file)
+          return nil if cache_mtime <= timestamp
+
+          begin
+            # Load the cached command result
+            cached_command = Makit::Serializer.open(cache_file, Makit::V1::Command)
+
+            # Convert to new Result format
+            convert_cached_command_to_result(cached_command)
+          rescue => e
+            Makit::Logging.debug("Failed to load cached result", {
+              cache_file: cache_file,
+              error: e.message,
+            })
+            nil
+          end
+        end
+
+        # Cache a successful result
+        #
+        # @param cache_file [String] path to cache file
+        # @param result [Result] result to cache
+        def cache_result(cache_file, result)
+          return unless cache_file && result.success?
+
+          begin
+            # Convert Result to legacy Command format for caching
+            legacy_command = convert_result_to_legacy_command(result)
+
+            # Save to cache
+            Makit::Serializer.save_as(cache_file, legacy_command)
+
+            Makit::Logging.debug("Cached command result", {
+              cache_file: cache_file,
+              command: result.command,
+              success: result.success?,
+            })
+          rescue => e
+            Makit::Logging.warn("Failed to cache result", {
+              cache_file: cache_file,
+              error: e.message,
+            })
+          end
+        end
+
+        # Convert cached Command to Result format
+        #
+        # @param cached_command [Makit::V1::Command] cached command
+        # @return [Result] converted result
+        def convert_cached_command_to_result(cached_command)
+          # Create a Result object from the cached command
+          Result.new(
+            command: "#{cached_command.name} #{cached_command.arguments.join(" ")}",
+            exit_code: cached_command.exit_code,
+            stdout: cached_command.output,
+            stderr: cached_command.error,
+            started_at: convert_protobuf_timestamp_to_time(cached_command.started_at),
+            duration: convert_protobuf_duration_to_float(cached_command.duration),
+            metadata: {
+              cached: true,
+              directory: cached_command.directory,
+            },
+          )
+        end
+
+        # Convert Result to legacy Command format for caching
+        #
+        # @param result [Result] result to convert
+        # @return [Makit::V1::Command] legacy command object
+        def convert_result_to_legacy_command(result)
+          command_parts = result.command.split
+          name = command_parts.first
+          arguments = command_parts[1..] || []
+
+          Makit::V1::Command.new(
+            name: name,
+            arguments: arguments,
+            exit_code: result.exit_code,
+            output: result.stdout,
+            error: result.stderr,
+            started_at: convert_time_to_protobuf_timestamp(result.started_at),
+            duration: convert_duration_to_protobuf(result.duration),
+            directory: result.metadata[:directory] || Dir.pwd,
+          )
+        end
+
+        # Convert protobuf timestamp to Time
+        #
+        # @param timestamp [Google::Protobuf::Timestamp] protobuf timestamp
+        # @return [Time, nil] converted time
+        def convert_protobuf_timestamp_to_time(timestamp)
+          return nil unless timestamp
+
+          Time.at(timestamp.seconds, timestamp.nanos, :nsec)
+        end
+
+        # Convert protobuf duration to float seconds
+        #
+        # @param duration [Google::Protobuf::Duration] protobuf duration
+        # @return [Float, nil] duration in seconds
+        def convert_protobuf_duration_to_float(duration)
+          return nil unless duration
+
+          duration.seconds + (duration.nanos / 1_000_000_000.0)
+        end
+
+        # Convert Time to protobuf timestamp
+        #
+        # @param time [Time] time to convert
+        # @return [Google::Protobuf::Timestamp] protobuf timestamp
+        def convert_time_to_protobuf_timestamp(time)
+          return nil unless time
+
+          Google::Protobuf::Timestamp.new(seconds: time.to_i, nanos: time.nsec)
+        end
+
+        # Convert duration to protobuf duration
+        #
+        # @param duration [Float] duration in seconds
+        # @return [Google::Protobuf::Duration] protobuf duration
+        def convert_duration_to_protobuf(duration)
+          return nil unless duration
+
+          seconds = duration.to_i
+          nanos = ((duration - seconds) * 1_000_000_000).to_i
+
+          Google::Protobuf::Duration.new(seconds: seconds, nanos: nanos)
+        end
+      end
+    end
+  end
+end