aidp 0.8.2 → 0.8.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: b9c14dc26b8cedb7e1beb09f9799dcbbcedf3e6aeb33b908256d7c67bd864b22
-  data.tar.gz: 3af751e93b47bf20566796c4ebf7519aa425e9851531b954be059b7bf649fc7d
+  metadata.gz: 60b61133b8f3b07ef7fbdd76bce1485bd89b75f78c4130f195c786331f383ded
+  data.tar.gz: e81f6387d6c6941da8b2ae1ff2b3153a3911fd2bb0405c4982f52b166681fc64
 SHA512:
-  metadata.gz: 3991a8477fc0f4293473a06083b734f7ea544d46847747688cda8c0c7d9b59e4dde2bbb41faaa92a9edaad88e920a0d6faca939c11d3a32c4da6f9b3740c62fe
-  data.tar.gz: b11ab471251fd86f4d9ff2acbd8c3c8fb0e9bf360292dc51e6a9697ba97b0e520dbcf9dc115331e1eb6800268dc08055b0e7fd4584f5af9e7a6609a7d3fac058
+  metadata.gz: 42b8c458bd6dcbfce249d12bb7a1e7eab1d09a61259ea231b935fd4d28c39170baf57d099bb3a5fd3037a3501a4d5daa9abc3696e3c7b91e07137ffce24b22a6
+  data.tar.gz: e2f038cc38c7b32f9bda535ac6bfb8a0bc39c54ef252dc9d7f333f66c71de41dc8e88a6479e8319bad64e5e9dc8adffb2d4dd03febddb82081df2ddf91e8efe0

data/lib/aidp/harness/runner.rb

@@ -6,7 +6,7 @@ require_relative "configuration"
 require_relative "state_manager"
 require_relative "condition_detector"
 require_relative "provider_manager"
-require_relative "user_interface"
+require_relative "simple_user_interface"
 require_relative "error_handler"
 require_relative "status_display"
 require_relative "completion_checker"
@@ -47,7 +47,7 @@ module Aidp
         @state_manager = StateManager.new(project_dir, @mode)
         @condition_detector = ConditionDetector.new
         @provider_manager = ProviderManager.new(@configuration)
-        @user_interface = UserInterface.new
+        @user_interface = SimpleUserInterface.new
         @error_handler = ErrorHandler.new(@provider_manager, @configuration)
         @status_display = StatusDisplay.new
         @completion_checker = CompletionChecker.new(@project_dir, @workflow_type)

data/lib/aidp/harness/simple_user_interface.rb

@@ -0,0 +1,125 @@
+# frozen_string_literal: true
+
+require "tty-prompt"
+
+module Aidp
+  module Harness
+    # Simple, focused user interface for collecting feedback
+    # Replaces the bloated UserInterface with minimal, clean code
+    class SimpleUserInterface
+      def initialize
+        @prompt = TTY::Prompt.new
+      end
+
+      # Main method - collect responses for questions
+      def collect_feedback(questions, context = nil)
+        show_context(context) if context
+
+        responses = {}
+        questions.each_with_index do |question_data, index|
+          key = "question_#{question_data[:number] || index + 1}"
+          responses[key] = ask_question(question_data)
+        end
+
+        responses
+      end
+
+      private
+
+      def show_context(context)
+        puts "\n🤖 Agent needs feedback"
+        puts "Context: #{context[:description]}" if context[:description]
+        puts ""
+      end
+
+      def ask_question(question_data)
+        question = question_data[:question]
+        type = question_data[:type] || "text"
+        default = question_data[:default]
+        required = question_data[:required] != false
+        options = question_data[:options]
+
+        puts "\n#{question}"
+
+        case type
+        when "text"
+          ask_text(question, default, required)
+        when "choice"
+          @prompt.select("Choose:", options, default: default)
+        when "confirmation"
+          @prompt.yes?("#{question}?", default: default)
+        when "file"
+          ask_file(question, default, required)
+        when "number"
+          ask_number(question, default, required)
+        when "email"
+          ask_email(question, default, required)
+        when "url"
+          ask_url(question, default, required)
+        else
+          ask_text(question, default, required)
+        end
+      end
+
+      def ask_text(question, default, required)
+        options = {}
+        options[:default] = default if default
+        options[:required] = required
+
+        @prompt.ask("Response:", **options)
+      end
+
+      def ask_file(question, default, required)
+        input = @prompt.ask("File path:", default: default, required: required)
+
+        # Handle @ file selection
+        if input&.start_with?("@")
+          search_term = input[1..].strip
+          files = find_files(search_term)
+
+          return nil if files.empty?
+
+          @prompt.select("Select file:", files, per_page: 15)
+        else
+          input
+        end
+      end
+
+      def ask_number(question, default, required)
+        @prompt.ask("Number:", default: default, required: required) do |q|
+          q.convert :int
+          q.validate(/^\d+$/, "Please enter a valid number")
+        end
+      end
+
+      def ask_email(question, default, required)
+        @prompt.ask("Email:", default: default, required: required) do |q|
+          q.validate(/\A[\w+\-.]+@[a-z\d-]+(\.[a-z\d-]+)*\.[a-z]+\z/i, "Please enter a valid email")
+        end
+      end
+
+      def ask_url(question, default, required)
+        @prompt.ask("URL:", default: default, required: required) do |q|
+          q.validate(/\Ahttps?:\/\/.+/i, "Please enter a valid URL (http:// or https://)")
+        end
+      end
+
+      def find_files(search_term)
+        if search_term.empty?
+          # Show common files
+          Dir.glob("**/*").select { |f| File.file?(f) }.first(20)
+        elsif search_term.start_with?(".")
+          # Extension search (e.g., .rb)
+          Dir.glob("**/*#{search_term}").select { |f| File.file?(f) }
+        elsif search_term.end_with?("/")
+          # Directory search (e.g., lib/)
+          dir = search_term.chomp("/")
+          Dir.glob("#{dir}/**/*").select { |f| File.file?(f) }
+        else
+          # Name search
+          Dir.glob("**/*#{search_term}*").select { |f| File.file?(f) }
+        end
+      end
+    end
+  end
+end

data/lib/aidp/version.rb

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

data/lib/aidp.rb

@@ -51,6 +51,7 @@ require_relative "aidp/harness/config_loader"
 require_relative "aidp/harness/config_manager"
 require_relative "aidp/harness/condition_detector"
 require_relative "aidp/harness/user_interface"
+require_relative "aidp/harness/simple_user_interface"
 require_relative "aidp/harness/provider_manager"
 require_relative "aidp/harness/provider_config"
 require_relative "aidp/harness/provider_factory"

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: aidp
 version: !ruby/object:Gem::Version
-  version: 0.8.2
+  version: 0.8.3
 platform: ruby
 authors:
 - Bart Agapinan
@@ -272,6 +272,7 @@ files:
 - lib/aidp/harness/provider_manager.rb
 - lib/aidp/harness/provider_type_checker.rb
 - lib/aidp/harness/runner.rb
+- lib/aidp/harness/simple_user_interface.rb
 - lib/aidp/harness/state/errors.rb
 - lib/aidp/harness/state/metrics.rb
 - lib/aidp/harness/state/persistence.rb