aidp 0.12.0 → 0.13.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: a5c07df32790749c9d60716d1ae3133f0a232cda036e7523b74b8c0725a0b066
-  data.tar.gz: b15140f57f3d8cc58dc4b4e79200d3b0ef50152c5f6bcd6f45b030aeca8c863e
+  metadata.gz: 0f6484f93a1d57d07f223f49abab2f1a919490d4cb25e6623e76ca85b430a0be
+  data.tar.gz: 6f765c07f09d77cf3ffab88c36aa9a04acf90c39fc44d326a594b7ff4b67759b
 SHA512:
-  metadata.gz: 1e5e5b0f9c209dea98509a29b4004f264cb49706d961cafe42ad2867a40abc6ecd9e3cf279a541e714c914f975ec25f6ed898a7be7549dc6ae1292e369eaa7b9
-  data.tar.gz: 5f7a286a65cd9e1ecbdbd167182fa1471e9407171a2fc028cfa144d9204606d8c8e3a7138348c8406133713aa1724749e7abcc0fa6904fca64b65b43355bbfd8
+  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

@@ -4,6 +4,7 @@
 require "yaml"
 require "tty-prompt"
 require_relative "../harness/provider_factory"
+require_relative "../config/paths"
 
 module Aidp
   class CLI
@@ -96,13 +97,14 @@ module Aidp
           display_message("Template not found: #{filename}", type: :error)
           return nil
         end
-        dest = File.join(@project_dir, "aidp.yml")
+        dest = Aidp::ConfigPaths.config_file(@project_dir)
+        Aidp::ConfigPaths.ensure_config_dir(@project_dir)
         File.write(dest, File.read(src))
         dest
       end
 
       def write_minimal_config(project_dir)
-        dest = File.join(project_dir, ".aidp", "aidp.yml")
+        dest = Aidp::ConfigPaths.config_file(project_dir)
         return dest if File.exist?(dest)
         data = {
           "harness" => {
@@ -118,26 +120,23 @@ module Aidp
             }
           }
         }
-        FileUtils.mkdir_p(File.dirname(dest))
+        Aidp::ConfigPaths.ensure_config_dir(project_dir)
         File.write(dest, YAML.dump(data))
         dest
       end
 
       def write_example_config(project_dir)
         Aidp::Config.create_example_config(project_dir)
-        File.join(project_dir, "aidp.yml")
+        Aidp::ConfigPaths.config_file(project_dir)
       end
 
       def run_custom
-        dest = File.join(@project_dir, "aidp.yml")
+        dest = Aidp::ConfigPaths.config_file(@project_dir)
         return dest if File.exist?(dest)
 
         @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
@@ -169,12 +168,13 @@ module Aidp
           },
           "providers" => provider_section
         }
+        Aidp::ConfigPaths.ensure_config_dir(@project_dir)
         File.write(dest, YAML.dump(data))
         dest
       end
 
       def run_custom_with_defaults(existing_config)
-        dest = File.join(@project_dir, "aidp.yml")
+        dest = Aidp::ConfigPaths.config_file(@project_dir)
 
         # Extract current values from existing config
         harness_config = existing_config[:harness] || existing_config["harness"] || {}
@@ -188,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
@@ -240,12 +237,13 @@ module Aidp
           "providers" => provider_section
         }
 
+        Aidp::ConfigPaths.ensure_config_dir(@project_dir)
         File.write(dest, YAML.dump(data))
         dest
       end
 
       def load_existing_config
-        config_file = File.join(@project_dir, "aidp.yml")
+        config_file = Aidp::ConfigPaths.config_file(@project_dir)
         return nil unless File.exist?(config_file)
 
         begin
@@ -274,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/config/paths.rb

@@ -0,0 +1,131 @@
+# frozen_string_literal: true
+
+require "fileutils"
+
+module Aidp
+  # Centralized path management for all AIDP internal files
+  # Ensures consistent file locations and prevents path-related bugs
+  module ConfigPaths
+    # Get the main AIDP directory for a project
+    def self.aidp_dir(project_dir = Dir.pwd)
+      File.join(project_dir, ".aidp")
+    end
+
+    # Get the main configuration file path
+    def self.config_file(project_dir = Dir.pwd)
+      File.join(aidp_dir(project_dir), "aidp.yml")
+    end
+
+    # Get the configuration directory path
+    def self.config_dir(project_dir = Dir.pwd)
+      aidp_dir(project_dir)
+    end
+
+    # Get the progress directory path
+    def self.progress_dir(project_dir = Dir.pwd)
+      File.join(aidp_dir(project_dir), "progress")
+    end
+
+    # Get the execute progress file path
+    def self.execute_progress_file(project_dir = Dir.pwd)
+      File.join(progress_dir(project_dir), "execute.yml")
+    end
+
+    # Get the analyze progress file path
+    def self.analyze_progress_file(project_dir = Dir.pwd)
+      File.join(progress_dir(project_dir), "analyze.yml")
+    end
+
+    # Get the harness state directory path
+    def self.harness_state_dir(project_dir = Dir.pwd)
+      File.join(aidp_dir(project_dir), "harness")
+    end
+
+    # Get the harness state file path for a specific mode
+    def self.harness_state_file(mode, project_dir = Dir.pwd)
+      File.join(harness_state_dir(project_dir), "#{mode}_state.json")
+    end
+
+    # Get the providers directory path
+    def self.providers_dir(project_dir = Dir.pwd)
+      File.join(aidp_dir(project_dir), "providers")
+    end
+
+    # Get the provider info file path
+    def self.provider_info_file(provider_name, project_dir = Dir.pwd)
+      File.join(providers_dir(project_dir), "#{provider_name}_info.yml")
+    end
+
+    # Get the jobs directory path
+    def self.jobs_dir(project_dir = Dir.pwd)
+      File.join(aidp_dir(project_dir), "jobs")
+    end
+
+    # Get the checkpoint file path
+    def self.checkpoint_file(project_dir = Dir.pwd)
+      File.join(aidp_dir(project_dir), "checkpoint.yml")
+    end
+
+    # Get the checkpoint history file path
+    def self.checkpoint_history_file(project_dir = Dir.pwd)
+      File.join(aidp_dir(project_dir), "checkpoint_history.jsonl")
+    end
+
+    # Get the JSON storage directory path
+    def self.json_storage_dir(project_dir = Dir.pwd)
+      File.join(aidp_dir(project_dir), "json")
+    end
+
+    # Check if the main configuration file exists
+    def self.config_exists?(project_dir = Dir.pwd)
+      File.exist?(config_file(project_dir))
+    end
+
+    # Ensure the main AIDP directory exists
+    def self.ensure_aidp_dir(project_dir = Dir.pwd)
+      dir = aidp_dir(project_dir)
+      FileUtils.mkdir_p(dir) unless Dir.exist?(dir)
+      dir
+    end
+
+    # Ensure the configuration directory exists
+    def self.ensure_config_dir(project_dir = Dir.pwd)
+      ensure_aidp_dir(project_dir)
+    end
+
+    # Ensure the progress directory exists
+    def self.ensure_progress_dir(project_dir = Dir.pwd)
+      dir = progress_dir(project_dir)
+      FileUtils.mkdir_p(dir) unless Dir.exist?(dir)
+      dir
+    end
+
+    # Ensure the harness state directory exists
+    def self.ensure_harness_state_dir(project_dir = Dir.pwd)
+      dir = harness_state_dir(project_dir)
+      FileUtils.mkdir_p(dir) unless Dir.exist?(dir)
+      dir
+    end
+
+    # Ensure the providers directory exists
+    def self.ensure_providers_dir(project_dir = Dir.pwd)
+      dir = providers_dir(project_dir)
+      FileUtils.mkdir_p(dir) unless Dir.exist?(dir)
+      dir
+    end
+
+    # Ensure the jobs directory exists
+    def self.ensure_jobs_dir(project_dir = Dir.pwd)
+      dir = jobs_dir(project_dir)
+      FileUtils.mkdir_p(dir) unless Dir.exist?(dir)
+      dir
+    end
+
+    # Ensure the JSON storage directory exists
+    def self.ensure_json_storage_dir(project_dir = Dir.pwd)
+      dir = json_storage_dir(project_dir)
+      FileUtils.mkdir_p(dir) unless Dir.exist?(dir)
+      dir
+    end
+  end
+end

data/lib/aidp/config.rb

@@ -1,6 +1,7 @@
 # frozen_string_literal: true
 
 require "yaml"
+require_relative "config/paths"
 
 module Aidp
   # Configuration management for both execute and analyze modes
@@ -165,7 +166,7 @@ module Aidp
     }.freeze
 
     def self.load(project_dir = Dir.pwd)
-      config_file = File.join(project_dir, ".aidp", "aidp.yml")
+      config_file = ConfigPaths.config_file(project_dir)
 
       if File.exist?(config_file)
         load_yaml_config(config_file)
@@ -244,15 +245,15 @@ module Aidp
 
     # Check if configuration file exists
     def self.config_exists?(project_dir = Dir.pwd)
-      File.exist?(File.join(project_dir, ".aidp", "aidp.yml"))
+      ConfigPaths.config_exists?(project_dir)
     end
 
     # Create example configuration file
     def self.create_example_config(project_dir = Dir.pwd)
-      config_path = File.join(project_dir, ".aidp", "aidp.yml")
+      config_path = ConfigPaths.config_file(project_dir)
       return false if File.exist?(config_path)
 
-      FileUtils.mkdir_p(File.dirname(config_path))
+      ConfigPaths.ensure_config_dir(project_dir)
 
       example_config = {
         harness: {
@@ -283,6 +284,19 @@ module Aidp
       true
     end
 
+    # Expose path methods for convenience
+    def self.config_file(project_dir = Dir.pwd)
+      ConfigPaths.config_file(project_dir)
+    end
+
+    def self.config_dir(project_dir = Dir.pwd)
+      ConfigPaths.config_dir(project_dir)
+    end
+
+    def self.aidp_dir(project_dir = Dir.pwd)
+      ConfigPaths.aidp_dir(project_dir)
+    end
+
     private_class_method def self.load_yaml_config(config_file)
       YAML.load_file(config_file) || {}
     rescue => e

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]