aidp 0.8.1 → 0.8.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: a295d267b134bc6f1c28aef126969254203d4f07f98d39e686c85c4479954eca
-  data.tar.gz: 23b7bef369a2d423d3e141492f857da0d66640c337d4cd530befa52d6f3c9392
+  metadata.gz: b9c14dc26b8cedb7e1beb09f9799dcbbcedf3e6aeb33b908256d7c67bd864b22
+  data.tar.gz: 3af751e93b47bf20566796c4ebf7519aa425e9851531b954be059b7bf649fc7d
 SHA512:
-  metadata.gz: 2544ee1a9ba91033d2b8b9eee057df06c2a15aa3d90725028f72a839c09d3d3f1b95545b937cc9206e360bb8f137174cb495b591ae9890fc26ef5e58e235c0d2
-  data.tar.gz: 4e634535c9cfdcae56400ac073b79334e5a0311342f53e26d6a9b1ace929c5bae5210ae92467e0c47cca3afa902935857d315886bdbc1a6dd07880b55878dd07
+  metadata.gz: 3991a8477fc0f4293473a06083b734f7ea544d46847747688cda8c0c7d9b59e4dde2bbb41faaa92a9edaad88e920a0d6faca939c11d3a32c4da6f9b3740c62fe
+  data.tar.gz: b11ab471251fd86f4d9ff2acbd8c3c8fb0e9bf360292dc51e6a9697ba97b0e520dbcf9dc115331e1eb6800268dc08055b0e7fd4584f5af9e7a6609a7d3fac058

data/lib/aidp/analyze/progress_visualizer.rb

@@ -4,7 +4,6 @@ require "tty-spinner"
 require "tty-progressbar"
 require "tty-table"
 require "pastel"
-require "colorize"
 
 module Aidp
   class ProgressVisualizer
@@ -295,21 +294,21 @@ module Aidp
     end
 
     def colorize(text, color)
-      return text unless defined?((+"").colorize)
+      return text unless @pastel
 
-      text.to_s.colorize(color)
+      @pastel.decorate(text.to_s, color)
     end
 
     def coverage_colorize(percentage, color)
-      return percentage unless defined?((+"").colorize)
+      return percentage unless @pastel
 
-      percentage.to_s.colorize(color)
+      @pastel.decorate(percentage.to_s, color)
     end
 
     def priority_colorize(text, color)
-      return text unless defined?((+"").colorize)
+      return text unless @pastel
 
-      text.to_s.colorize(color)
+      @pastel.decorate(text.to_s, color)
     end
   end
 end

data/lib/aidp/execute/workflow_selector.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require "readline"
+require "tty-prompt"
 
 module Aidp
   module Execute
@@ -8,6 +8,7 @@ module Aidp
     class WorkflowSelector
       def initialize
         @user_input = {}
+        @prompt = TTY::Prompt.new
       end
 
       # Main entry point for interactive workflow selection
@@ -189,22 +190,23 @@ module Aidp
 
       def prompt_required(question)
         loop do
-          response = Readline.readline("#{question}: ", true)&.strip
-          return response unless response.nil? || response.empty?
-          puts "This field is required. Please provide an answer."
+          input = @prompt.ask("#{question}:")
+
+          if input.nil? || input.strip.empty?
+            puts "❌ This field is required. Please provide an answer."
+            next
+          end
+
+          return input.strip
         end
       end
 
       def prompt_optional(question)
-        Readline.readline("#{question} (optional): ", true)&.strip
+        @prompt.ask("#{question} (optional):")
       end
 
       def prompt_choice(question, valid_choices)
-        loop do
-          response = Readline.readline("#{question} (#{valid_choices.join("/")}): ", true)&.strip&.downcase
-          return response if valid_choices.map(&:downcase).include?(response)
-          puts "Please choose one of: #{valid_choices.join(", ")}"
-        end
+        @prompt.select("#{question}:", valid_choices, per_page: valid_choices.length)
       end
     end
   end

data/lib/aidp/harness/ui/enhanced_tui.rb

@@ -28,7 +28,7 @@ module Aidp
           # Headless (non-interactive) detection for test/CI environments:
           # - RSpec defined or RSPEC_RUNNING env set
           # - STDIN not a TTY (captured by PTY/tmux harness)
-          @headless = !!(defined?(RSpec) || ENV["RSPEC_RUNNING"] || !$stdin.tty?)
+          @headless = !!(defined?(RSpec) || ENV["RSPEC_RUNNING"] || $stdin.nil? || !$stdin.tty?)
           @current_mode = nil
           @workflow_active = false
           @current_step = nil
@@ -382,7 +382,7 @@ module Aidp
         # Very lightweight key listener just for spec expectations (F1 help, Ctrl shortcuts)
         def start_key_listener
           return if @key_thread
-          return unless $stdin.tty? || @headless
+          return unless $stdin&.tty? || @headless
 
           @key_thread = Thread.new do
             while @display_active

data/lib/aidp/harness/user_interface.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require "readline"
+require "tty-prompt"
 
 module Aidp
   module Harness
@@ -9,6 +9,47 @@ module Aidp
       def initialize
         @input_history = []
         @file_selection_enabled = false
+        @prompt = TTY::Prompt.new
+        @control_mutex = Mutex.new
+        @pause_requested = false
+        @stop_requested = false
+        @resume_requested = false
+        @control_interface_enabled = false
+        @control_thread = nil
+      end
+
+      private
+
+      # Helper method to handle input consistently with TTY::Prompt
+      # Fixed to avoid keystroke loss issues with TTY::Prompt's required validation
+      def get_input_with_prompt(message, required: false, default: nil)
+        loop do
+          # Always use simple ask without built-in validation to avoid echo issues
+          input = if default
+            @prompt.ask("#{message} (default: #{default}): ")
+          else
+            @prompt.ask("#{message}: ")
+          end
+
+          # Handle empty input
+          if input.nil? || input.strip.empty?
+            if default
+              return default
+            elsif required
+              puts "❌ This field is required. Please provide a response."
+              next
+            else
+              return nil
+            end
+          end
+
+          return input.strip
+        end
+      end
+
+      public
+
+      def setup_control_interface
         @control_interface_enabled = true
         @pause_requested = false
         @stop_requested = false
@@ -465,9 +506,14 @@ module Aidp
           puts "  3. Get help"
           puts "  4. Cancel all questions"
 
-          choice = Readline.readline("Your choice (1-4): ", true)
+          choice = @prompt.select("Choose an option:", {
+            "Try again" => "1",
+            "Skip this question" => "2",
+            "Get help" => "3",
+            "Cancel all questions" => "4"
+          })
 
-          case choice&.strip
+          case choice
           when "1"
             puts "🔄 Retrying..."
             :retry
@@ -529,7 +575,7 @@ module Aidp
         end
 
         puts "\nPress Enter to continue..."
-        Readline.readline
+        @prompt.keypress("Press any key to continue...")
       end
 
       # Enhanced error handling and validation display
@@ -563,7 +609,7 @@ module Aidp
           end
           puts "\nPress Enter to continue or type 'fix' to correct..."
 
-          input = Readline.readline("", true)
+          input = @prompt.ask("")
           return input&.strip&.downcase == "fix"
         end
         false
@@ -573,21 +619,16 @@ module Aidp
       def get_text_response(expected_input, default_value, required, options = {})
         prompt = "Your response"
         prompt += " (default: #{default_value})" if default_value
-        prompt += required ? ": " : " (optional): "
+        prompt_text = prompt + (required ? "" : " (optional)")
 
         loop do
-          input = Readline.readline(prompt, true)
+          input = get_input_with_prompt(prompt_text, required: required, default: default_value)
 
-          # Handle empty input
-          if input.nil? || input.strip.empty?
-            if default_value
-              return default_value
-            elsif required
-              puts "❌ This field is required. Please provide a response."
-              next
-            else
-              return nil
-            end
+          # get_input_with_prompt already handles required validation and returns non-empty input
+          # Only validate the type/format if we got input
+          if input.nil?
+            # This should only happen for non-required fields
+            return nil
           end
 
           # Enhanced validation
@@ -622,7 +663,7 @@ module Aidp
           prompt += " (default: #{default_value})" if default_value
           prompt += required ? ": " : " (optional): "
 
-          input = Readline.readline(prompt, true)
+          input = @prompt.ask(prompt)
 
           if input.nil? || input.strip.empty?
             if default_value
@@ -666,7 +707,7 @@ module Aidp
         prompt += required ? ": " : " (optional): "
 
         loop do
-          input = Readline.readline(prompt, true)
+          input = @prompt.ask(prompt)
 
           if input.nil? || input.strip.empty?
             return default
@@ -702,7 +743,7 @@ module Aidp
         prompt += required ? ": " : " (optional): "
 
         loop do
-          input = Readline.readline(prompt, true)
+          input = @prompt.ask(prompt)
 
           if input.nil? || input.strip.empty?
             if default_value
@@ -745,7 +786,7 @@ module Aidp
         prompt += required ? ": " : " (optional): "
 
         loop do
-          input = Readline.readline(prompt, true)
+          input = @prompt.ask(prompt)
 
           if input.nil? || input.strip.empty?
             if default_value
@@ -792,7 +833,7 @@ module Aidp
         prompt += required ? ": " : " (optional): "
 
         loop do
-          input = Readline.readline(prompt, true)
+          input = @prompt.ask(prompt)
 
           if input.nil? || input.strip.empty?
             if default_value
@@ -829,7 +870,7 @@ module Aidp
         prompt += required ? ": " : " (optional): "
 
         loop do
-          input = Readline.readline(prompt, true)
+          input = @prompt.ask(prompt)
 
           if input.nil? || input.strip.empty?
             if default_value
@@ -1278,7 +1319,7 @@ module Aidp
       # Get user input with support for file selection
       def get_user_input(prompt)
         loop do
-          input = Readline.readline(prompt, true)
+          input = @prompt.ask(prompt)
 
           # Handle empty input
           if input.nil? || input.strip.empty?
@@ -1622,7 +1663,7 @@ module Aidp
       # Get advanced file selection from user
       def get_advanced_file_selection(max_files, _search_options)
         loop do
-          input = Readline.readline("Select file (0-#{max_files}, -1=refine, p=preview, h=help): ", true)
+          input = @prompt.ask("Select file (0-#{max_files}, -1=refine, p=preview, h=help): ")
 
           if input.nil? || input.strip.empty?
             puts "Please enter a selection."
@@ -1715,7 +1756,7 @@ module Aidp
         end
 
         puts "\nPress Enter to continue..."
-        Readline.readline
+        @prompt.keypress("Press any key to continue...")
       end
 
       # Get file selection from user (legacy method for compatibility)
@@ -1729,7 +1770,7 @@ module Aidp
         prompt = "#{message} [#{default_text}]: "
 
         loop do
-          input = Readline.readline(prompt, true)
+          input = @prompt.ask(prompt)
 
           if input.nil? || input.strip.empty?
             return default
@@ -1756,7 +1797,7 @@ module Aidp
         end
 
         loop do
-          input = Readline.readline("Your choice (1-#{options.size}): ", true)
+          input = @prompt.ask("Your choice (1-#{options.size}): ")
 
           if input.nil? || input.strip.empty?
             return default if default
@@ -1927,14 +1968,14 @@ module Aidp
         if should_show_help?(questions.first&.dig(:type), seen_types)
           show_help
           puts "\nPress Enter to continue..."
-          Readline.readline
+          @prompt.keypress("Press any key to continue...")
         end
 
         # Display question summary if verbose
         if @verbose_mode
           display_question_summary(questions)
           puts "\nPress Enter to start answering questions..."
-          Readline.readline
+          @prompt.keypress("Press any key to continue...")
         end
 
         # Collect feedback
@@ -2041,17 +2082,17 @@ module Aidp
 
       # Check if pause is requested
       def pause_requested?
-        @control_mutex.synchronize { @pause_requested }
+        @control_mutex.synchronize { !!@pause_requested }
       end
 
       # Check if stop is requested
       def stop_requested?
-        @control_mutex.synchronize { @stop_requested }
+        @control_mutex.synchronize { !!@stop_requested }
       end
 
       # Check if resume is requested
       def resume_requested?
-        @control_mutex.synchronize { @resume_requested }
+        @control_mutex.synchronize { !!@resume_requested }
       end
 
       # Request pause
@@ -2124,7 +2165,7 @@ module Aidp
         puts "=" * 50
 
         loop do
-          input = Readline.readline("Paused> ", true)
+          input = @prompt.ask("Paused>")
 
           case input&.strip&.downcase
           when "r", "resume"
@@ -2189,7 +2230,7 @@ module Aidp
       # Control interface main loop
       def control_interface_loop
         loop do
-          input = Readline.readline("Control> ", true)
+          input = @prompt.ask("Control> ")
 
           case input&.strip&.downcase
           when "p", "pause"
@@ -2253,11 +2294,11 @@ module Aidp
       def get_control_status
         @control_mutex.synchronize do
           {
-            enabled: @control_interface_enabled,
-            pause_requested: @pause_requested,
-            stop_requested: @stop_requested,
-            resume_requested: @resume_requested,
-            control_thread_alive: @control_thread&.alive? || false
+            enabled: !!@control_interface_enabled,
+            pause_requested: !!@pause_requested,
+            stop_requested: !!@stop_requested,
+            resume_requested: !!@resume_requested,
+            control_thread_alive: !!@control_thread&.alive?
           }
         end
       end
@@ -2291,7 +2332,7 @@ module Aidp
         puts "=" * 50
 
         loop do
-          choice = Readline.readline("Select option (1-8): ", true)
+          choice = @prompt.ask("Select option (1-8): ")
 
           case choice&.strip
           when "1"

data/lib/aidp/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Aidp
-  VERSION = "0.8.1"
+  VERSION = "0.8.2"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: aidp
 version: !ruby/object:Gem::Version
-  version: 0.8.1
+  version: 0.8.2
 platform: ruby
 authors:
 - Bart Agapinan
@@ -23,20 +23,6 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '0.11'
-- !ruby/object:Gem::Dependency
-  name: colorize
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '1.1'
-  type: :runtime
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '1.1'
 - !ruby/object:Gem::Dependency
   name: csv
   requirement: !ruby/object:Gem::Requirement