aidp 0.12.1 → 0.13.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: e672ad3f8f54f1b3ad0cc4a804a63d1fa048709d47112497d5a4871d046dfb8b
-  data.tar.gz: 665d0b12e5dff272a0da11fd20d80299eaa4657ab7a87fe4e8b0f80e039e437d
+  metadata.gz: 0f6484f93a1d57d07f223f49abab2f1a919490d4cb25e6623e76ca85b430a0be
+  data.tar.gz: 6f765c07f09d77cf3ffab88c36aa9a04acf90c39fc44d326a594b7ff4b67759b
 SHA512:
-  metadata.gz: 384734aed97d697dce2c06ac6bf6f52b161b6436081ff4feb3b8e99e8038bef2b62a9aa3613b21dc92705569741598b98dcbf72805868f6cbdf82a1a19c3c6dd
-  data.tar.gz: 3bd5a990c925f10bb1ad863097b1f0a1e84b682e2489b0793242c37cc4034eb6ac8777559cecb6acaef915391dfeb60337074267e52a1cc4c35aee87c793d672
+  metadata.gz: e3219a59423ae33fc74bfaf38ac3ecebc3a157ac4f789c22252527652f02f63a1d98078de3db8b277d323073bf5cec8b86c97e9e7749f9a4e3097c6d849a4c2e
+  data.tar.gz: 3ec1273d7a35839fef07a02fa3a77e9340c008146c87ae790c8829f851f668af5fe1a1df12e28244b2b78c3313dd4ca667cb5e794232a3c4507f09f540d46cd9

data/lib/aidp/analyze/json_file_storage.rb

@@ -14,7 +14,7 @@ module Aidp
 
       # Store data in a JSON file
       def store_data(filename, data)
-        file_path = get_file_path(filename)
+        file_path = file_path(filename)
 
         # Ensure directory exists
         FileUtils.mkdir_p(File.dirname(file_path))
@@ -31,8 +31,8 @@ module Aidp
       end
 
       # Retrieve data from a JSON file
-      def get_data(filename)
-        file_path = get_file_path(filename)
+      def data(filename)
+        file_path = file_path(filename)
 
         return nil unless File.exist?(file_path)
 
@@ -45,12 +45,12 @@ module Aidp
 
       # Check if a JSON file exists
       def data_exists?(filename)
-        File.exist?(get_file_path(filename))
+        File.exist?(file_path(filename))
       end
 
       # Delete a JSON file
       def delete_data(filename)
-        file_path = get_file_path(filename)
+        file_path = file_path(filename)
 
         if File.exist?(file_path)
           File.delete(file_path)
@@ -89,8 +89,8 @@ module Aidp
       end
 
       # Get project configuration
-      def get_project_config
-        get_data("project_config.json")
+      def project_config
+        data("project_config.json")
       end
 
       # Store runtime status
@@ -99,8 +99,8 @@ module Aidp
       end
 
       # Get runtime status
-      def get_runtime_status
-        get_data("runtime_status.json")
+      def runtime_status
+        data("runtime_status.json")
       end
 
       # Store simple metrics
@@ -109,8 +109,8 @@ module Aidp
       end
 
       # Get simple metrics
-      def get_simple_metrics
-        get_data("simple_metrics.json")
+      def simple_metrics
+        data("simple_metrics.json")
       end
 
       # Store analysis session data
@@ -119,8 +119,8 @@ module Aidp
       end
 
       # Get analysis session data
-      def get_analysis_session(session_id)
-        get_data("sessions/#{session_id}.json")
+      def analysis_session(session_id)
+        data("sessions/#{session_id}.json")
       end
 
       # List analysis sessions
@@ -145,8 +145,8 @@ module Aidp
       end
 
       # Get user preferences
-      def get_user_preferences
-        get_data("user_preferences.json")
+      def user_preferences
+        data("user_preferences.json")
       end
 
       # Store cache data
@@ -161,8 +161,8 @@ module Aidp
       end
 
       # Get cache data (respects TTL)
-      def get_cache(cache_key)
-        cache_file_data = get_data("cache/#{cache_key}.json")
+      def cache(cache_key)
+        cache_file_data = data("cache/#{cache_key}.json")
         return nil unless cache_file_data
 
         # Check TTL if specified
@@ -203,7 +203,7 @@ module Aidp
       end
 
       # Get storage statistics
-      def get_storage_statistics
+      def storage_statistics
         files = list_files
 
         {
@@ -226,7 +226,7 @@ module Aidp
 
         files = list_files
         files.each do |file_info|
-          data = get_data(file_info[:filename])
+          data = data(file_info[:filename])
           export_data["files"][file_info[:filename]] = {
             "data" => data,
             "metadata" => {
@@ -249,7 +249,7 @@ module Aidp
 
       # Import data from an exported JSON file
       def import_data(import_filename)
-        import_path = get_file_path(import_filename)
+        import_path = file_path(import_filename)
 
         unless File.exist?(import_path)
           raise "Import file does not exist: #{import_filename}"
@@ -280,7 +280,7 @@ module Aidp
 
       private
 
-      def get_file_path(filename)
+      def file_path(filename)
         File.join(@storage_dir, filename)
       end
 

data/lib/aidp/cli/enhanced_input.rb

@@ -0,0 +1,114 @@
+# frozen_string_literal: true
+
+require "tty-prompt"
+require "reline"
+
+module Aidp
+  class CLI
+    # Enhanced input handler with full readline-style key bindings using Reline
+    class EnhancedInput
+      # Standard key bindings supported by Reline:
+      # - Ctrl-A: Move to beginning of line
+      # - Ctrl-E: Move to end of line
+      # - Ctrl-W: Delete word backward
+      # - Ctrl-K: Kill to end of line
+      # - Ctrl-U: Kill to beginning of line
+      # - Ctrl-D: Delete character forward
+      # - Ctrl-H/Backspace: Delete character backward
+      # - Left/Right arrows: Move cursor
+      # - Alt-F/Alt-B: Move forward/backward by word
+      # - Home/End: Jump to beginning/end
+      # - Ctrl-T: Transpose characters
+      # - And many more Emacs-style bindings
+
+      def initialize(prompt: nil, input: nil, output: nil, use_reline: true)
+        @use_reline = use_reline
+        @input = input || $stdin
+        @output = output || $stdout
+        @prompt = prompt || TTY::Prompt.new(
+          input: @input,
+          output: @output,
+          enable_color: true,
+          interrupt: :exit
+        )
+        @show_hints = false
+      end
+
+      # Ask a question with full readline support
+      # Uses Reline for readline-style editing when use_reline is true
+      def ask(question, **options)
+        # If reline is enabled and we're in a TTY, use reline for better editing
+        if @use_reline && @input.tty?
+          default = options[:default]
+          required = options[:required] || false
+
+          # Display helpful hint on first use
+          if @show_hints
+            @output.puts "💡 Hint: Use Ctrl-A (start), Ctrl-E (end), Ctrl-W (delete word), Ctrl-K (kill line)"
+            @show_hints = false
+          end
+
+          # Use Reline for input with full key binding support
+          loop do
+            prompt_text = question.to_s
+            prompt_text += " (#{default})" if default
+            prompt_text += " "
+
+            # Reline provides full readline editing capabilities
+            Reline.output = @output
+            Reline.input = @input
+            Reline.completion_append_character = " "
+
+            answer = Reline.readline(prompt_text, false)
+
+            # Handle Ctrl-D (nil return)
+            if answer.nil?
+              @output.puts
+              raise Interrupt
+            end
+
+            answer = answer.strip
+            answer = default if answer.empty? && default
+
+            if required && (answer.nil? || answer.empty?)
+              @output.puts "  Value required."
+              next
+            end
+
+            return answer
+          end
+        else
+          # Fall back to TTY::Prompt's ask
+          @prompt.ask(question, **options)
+        end
+      rescue Interrupt
+        @output.puts
+        raise
+      end
+
+      # Enable hints for key bindings
+      def enable_hints!
+        @show_hints = true
+      end
+
+      # Disable Reline (fall back to TTY::Prompt)
+      def disable_reline!
+        @use_reline = false
+      end
+
+      # Enable Reline
+      def enable_reline!
+        @use_reline = true
+      end
+
+      # Delegate other methods to underlying prompt
+      def method_missing(method, *args, **kwargs, &block)
+        @prompt.send(method, *args, **kwargs, &block)
+      end
+
+      def respond_to_missing?(method, include_private = false)
+        @prompt.respond_to?(method, include_private) || super
+      end
+    end
+  end
+end

data/lib/aidp/cli/first_run_wizard.rb

@@ -137,9 +137,6 @@ module Aidp
         @prompt.say("Interactive custom configuration: press Enter to accept defaults shown in [brackets].")
         @prompt.say("")
 
-        # Get available providers for validation
-        available_providers = get_available_providers
-
         # Use TTY::Prompt select for primary provider
         # Find the formatted string that matches the default
         default_option = available_providers.find { |option| option.start_with?("cursor -") } || available_providers.first
@@ -191,9 +188,6 @@ module Aidp
         @prompt.say("Interactive configuration update: press Enter to keep current values shown in [brackets].")
         @prompt.say("")
 
-        # Get available providers for validation
-        available_providers = get_available_providers
-
         # Use TTY::Prompt select for primary provider
         # Find the formatted string that matches the current default
         default_option = available_providers.find { |option| option.start_with?("#{current_default} -") } || available_providers.first
@@ -278,7 +272,7 @@ module Aidp
       end
 
       # Get available providers for validation
-      def get_available_providers
+      def available_providers
         # Get all supported providers from the factory (single source of truth)
         all_providers = Aidp::Harness::ProviderFactory::PROVIDER_CLASSES.keys
 

data/lib/aidp/cli/mcp_dashboard.rb

@@ -57,7 +57,7 @@ module Aidp
         server_matrix = build_server_matrix
         eligible_providers = []
 
-        @configuration.configured_providers.each do |provider|
+        @configuration.provider_names.each do |provider|
           provider_servers = server_matrix[:provider_servers][provider] || []
           enabled_servers = provider_servers.select { |s| s[:enabled] }.map { |s| s[:name] }
 
@@ -70,7 +70,7 @@ module Aidp
         {
           required_servers: required_servers,
           eligible_providers: eligible_providers,
-          total_providers: @configuration.configured_providers.size
+          total_providers: @configuration.provider_names.size
         }
       end
 
@@ -96,7 +96,7 @@ module Aidp
       private
 
       def build_server_matrix
-        providers = @configuration.configured_providers
+        providers = @configuration.provider_names
         all_servers = {} # server_name => {providers: {provider_name => server_info}}
         provider_servers = {} # provider_name => [server_info]
 

data/lib/aidp/cli/terminal_io.rb

@@ -1,6 +1,7 @@
 # frozen_string_literal: true
 
 require "stringio"
+require "tty-reader"
 
 module Aidp
   class CLI
@@ -32,6 +33,31 @@ module Aidp
         @input.gets
       end
 
+      # Enhanced readline-style input with standard key combinations
+      # Supports: Ctrl-A (beginning), Ctrl-E (end), Ctrl-W (delete word), etc.
+      def readline(prompt = "", default: nil)
+        # Use StringIO for testing, otherwise use TTY::Reader for real input
+        if @input.is_a?(StringIO)
+          @output.print(prompt)
+          @output.flush
+          line = @input.gets
+          return line&.chomp if line
+          return default
+        end
+
+        reader = TTY::Reader.new(
+          input: @input,
+          output: @output,
+          interrupt: :exit
+        )
+
+        # Read line with full readline support (Ctrl-A, Ctrl-E, Ctrl-W, etc.)
+        result = reader.read_line(prompt, default: default || "")
+        result&.chomp
+      rescue TTY::Reader::InputInterrupt
+        raise Interrupt
+      end
+
       def write(str)
         @output.write(str)
       end

data/lib/aidp/cli.rb

@@ -618,8 +618,8 @@ module Aidp
             false
           end
         end
-        configuration = Aidp::Harness::Configuration.new(Dir.pwd)
-        pm = Aidp::Harness::ProviderManager.new(configuration, prompt: TTY::Prompt.new)
+        config_manager = Aidp::Harness::ConfigManager.new(Dir.pwd)
+        pm = Aidp::Harness::ProviderManager.new(config_manager, prompt: TTY::Prompt.new)
 
         # Use TTY::Spinner for progress indication
         require "tty-spinner"
@@ -772,11 +772,11 @@ module Aidp
         require "tty-spinner"
 
         provider_name = args.shift
-        configuration = Aidp::Harness::Configuration.new(Dir.pwd)
+        config_manager = Aidp::Harness::ConfigManager.new(Dir.pwd)
         providers_to_refresh = if provider_name
           [provider_name]
         else
-          configuration.configured_providers
+          config_manager.provider_names
         end
 
         display_message("Refreshing provider information...", type: :info)

data/lib/aidp/harness/condition_detector.rb

@@ -1425,7 +1425,7 @@ module Aidp
       end
 
       # Get completion confidence level
-      def get_completion_confidence(completion_info)
+      def completion_confidence(completion_info)
         return 0.0 unless completion_info && completion_info[:confidence]
 
         completion_info[:confidence]
@@ -1433,23 +1433,23 @@ module Aidp
 
       # Check if completion is high confidence
       def high_confidence_completion?(completion_info)
-        get_completion_confidence(completion_info) >= 0.8
+        completion_confidence(completion_info) >= 0.8
       end
 
       # Check if completion is medium confidence
       def medium_confidence_completion?(completion_info)
-        confidence = get_completion_confidence(completion_info)
+        confidence = completion_confidence(completion_info)
         confidence >= 0.5 && confidence < 0.8
       end
 
       # Check if completion is low confidence
       def low_confidence_completion?(completion_info)
-        confidence = get_completion_confidence(completion_info)
+        confidence = completion_confidence(completion_info)
         confidence > 0.0 && confidence < 0.5
       end
 
       # Get next actions from completion info
-      def get_next_actions(completion_info)
+      def next_actions(completion_info)
         return [] unless completion_info && completion_info[:next_actions]
 
         completion_info[:next_actions]
@@ -1495,7 +1495,7 @@ module Aidp
       end
 
       # Get progress status description
-      def get_progress_status_description(completion_info)
+      def progress_status_description(completion_info)
         return "unknown" unless completion_info
 
         case completion_info[:progress_status]

data/lib/aidp/harness/config_loader.rb

@@ -23,7 +23,7 @@ module Aidp
         validation_result = @validator.load_and_validate
 
         if validation_result[:valid]
-          @config_cache = @validator.get_validated_config
+          @config_cache = @validator.validated_config
           @last_loaded = Time.now
 
           # Log warnings if any
@@ -40,7 +40,7 @@ module Aidp
       end
 
       # Get harness configuration with defaults
-      def get_harness_config(force_reload = false)
+      def harness_config(force_reload = false)
         config = load_config(force_reload)
         return nil unless config
 
@@ -48,7 +48,7 @@ module Aidp
       end
 
       # Get provider configuration with defaults
-      def get_provider_config(provider_name, force_reload = false)
+      def provider_config(provider_name, force_reload = false)
         config = load_config(force_reload)
         return nil unless config
 
@@ -57,7 +57,7 @@ module Aidp
       end
 
       # Get all provider configurations
-      def get_all_provider_configs(force_reload = false)
+      def all_provider_configs(force_reload = false)
         config = load_config(force_reload)
         return {} unless config
 
@@ -65,7 +65,7 @@ module Aidp
       end
 
       # Get configured provider names
-      def get_configured_providers(force_reload = false)
+      def configured_providers(force_reload = false)
         config = load_config(force_reload)
         return [] unless config
 
@@ -94,7 +94,7 @@ module Aidp
       end
 
       # Get configuration summary
-      def get_config_summary
+      def config_summary
         @validator.get_summary
       end
 
@@ -109,7 +109,7 @@ module Aidp
       end
 
       # Get configuration with specific overrides
-      def get_config_with_overrides(overrides = {})
+      def config_with_overrides(overrides = {})
         base_config = load_config
         return nil unless base_config
 
@@ -117,17 +117,17 @@ module Aidp
       end
 
       # Get harness configuration with overrides
-      def get_harness_config_with_overrides(overrides = {})
-        harness_config = get_harness_config
-        return nil unless harness_config
+      def harness_config_with_overrides(overrides = {})
+        base_harness_config = harness_config
+        return nil unless base_harness_config
 
         harness_overrides = overrides[:harness] || overrides["harness"] || {}
-        deep_merge(harness_config, harness_overrides)
+        deep_merge(base_harness_config, harness_overrides)
       end
 
       # Get provider configuration with overrides
-      def get_provider_config_with_overrides(provider_name, overrides = {})
-        provider_config = get_provider_config(provider_name)
+      def provider_config_with_overrides(provider_name, overrides = {})
+        provider_config = provider_config(provider_name)
         return nil unless provider_config
 
         provider_overrides = overrides[:providers]&.dig(provider_name.to_sym) ||
@@ -158,34 +158,34 @@ module Aidp
       end
 
       # Get validation errors
-      def get_validation_errors
+      def validation_errors
         validation_result = @validator.validate_existing
         validation_result[:errors] || []
       end
 
       # Get validation warnings
-      def get_validation_warnings
+      def validation_warnings
         validation_result = @validator.validate_existing
         validation_result[:warnings] || []
       end
 
       # Get configuration for specific harness mode
-      def get_mode_config(mode, force_reload = false)
+      def mode_config(mode, force_reload = false)
         config = load_config(force_reload)
         return nil unless config
 
         case mode.to_s
         when "analyze"
-          get_analyze_mode_config(config)
+          analyze_mode_config(config)
         when "execute"
-          get_execute_mode_config(config)
+          execute_mode_config(config)
         else
           config
         end
       end
 
       # Get environment-specific configuration
-      def get_environment_config(environment = nil, force_reload = false)
+      def environment_config(environment = nil, force_reload = false)
         environment ||= ENV["AIDP_ENV"] || "development"
         config = load_config(force_reload)
         return nil unless config
@@ -214,7 +214,7 @@ module Aidp
       end
 
       # Get configuration with feature flags
-      def get_config_with_features(features = {}, force_reload = false)
+      def config_with_features(features = {}, force_reload = false)
         config = load_config(force_reload)
         return nil unless config
 
@@ -250,7 +250,7 @@ module Aidp
       end
 
       # Get configuration with time-based overrides
-      def get_time_based_config(force_reload = false)
+      def time_based_config(force_reload = false)
         config = load_config(force_reload)
         return nil unless config
 
@@ -340,12 +340,12 @@ module Aidp
         result
       end
 
-      def get_analyze_mode_config(config)
+      def analyze_mode_config(config)
         analyze_overrides = config[:analyze_mode] || config["analyze_mode"] || {}
         merge_overrides(config, analyze_overrides)
       end
 
-      def get_execute_mode_config(config)
+      def execute_mode_config(config)
         execute_overrides = config[:execute_mode] || config["execute_mode"] || {}
         merge_overrides(config, execute_overrides)
       end