aidp 0.9.5 → 0.10.0

data/lib/aidp/execute/workflow_selector.rb

@@ -24,9 +24,24 @@ module Aidp
 
       private
 
+      # Helper method for consistent message display using TTY::Prompt
+      def display_message(message, type: :info)
+        color = case type
+        when :error then :red
+        when :success then :green
+        when :warning then :yellow
+        when :info then :blue
+        when :highlight then :cyan
+        when :muted then :bright_black
+        else :white
+        end
+
+        @prompt.say(message, color: color)
+      end
+
       def select_workflow_interactive
-        puts "\n🚀 Welcome to AI Dev Pipeline!"
-        puts "Let's set up your development workflow.\n\n"
+        display_message("\n🚀 Welcome to AI Dev Pipeline!", type: :highlight)
+        display_message("Let's set up your development workflow.\n\n")
 
         # Step 1: Collect project information
         collect_project_info
@@ -45,7 +60,7 @@ module Aidp
       end
 
       def select_workflow_with_defaults
-        puts "\n🚀 Starting harness with default workflow configuration..."
+        display_message("\n🚀 Starting harness with default workflow configuration...", type: :highlight)
 
         # Use default project information
         @user_input = {
@@ -71,7 +86,7 @@ module Aidp
       private
 
       def collect_project_info
-        puts "📋 First, tell us about your project:\n"
+        display_message("📋 First, tell us about your project:\n", type: :highlight)
 
         @user_input[:project_description] = prompt_required(
           "What do you want to build? (Be specific about features and goals)"
@@ -91,17 +106,17 @@ module Aidp
       end
 
       def choose_workflow_type
-        puts "\n🎯 Choose your development approach:\n"
-        puts "1. 🔬 Exploration/Experiment - Quick prototype or proof of concept"
-        puts "   • Fast iteration, minimal documentation"
-        puts "   • Focus on core functionality and validation"
-        puts "   • Steps: PRD → Tasks → Implementation"
-        puts ""
-        puts "2. 🏗️  Full Development - Production-ready feature or system"
-        puts "   • Comprehensive planning and documentation"
-        puts "   • You can customize which steps to include"
-        puts "   • Full enterprise workflow available"
-        puts ""
+        display_message("\n🎯 Choose your development approach:\n", type: :highlight)
+        display_message("1. 🔬 Exploration/Experiment - Quick prototype or proof of concept")
+        display_message("   • Fast iteration, minimal documentation", type: :muted)
+        display_message("   • Focus on core functionality and validation", type: :muted)
+        display_message("   • Steps: PRD → Tasks → Implementation", type: :muted)
+        display_message("")
+        display_message("2. 🏗️  Full Development - Production-ready feature or system")
+        display_message("   • Comprehensive planning and documentation", type: :muted)
+        display_message("   • You can customize which steps to include", type: :muted)
+        display_message("   • Full enterprise workflow available", type: :muted)
+        display_message("")
 
         choice = prompt_choice("Which approach fits your project?", ["1", "2", "exploration", "full"])
 
@@ -111,7 +126,7 @@ module Aidp
         when "2", "full"
           :full
         else
-          puts "Invalid choice. Defaulting to exploration workflow."
+          display_message("Invalid choice. Defaulting to exploration workflow.", type: :warning)
           :exploration
         end
       end
@@ -137,8 +152,8 @@ module Aidp
       end
 
       def full_workflow_steps
-        puts "\n🛠️ Customize your full development workflow:\n"
-        puts "Select the steps you want to include (enter numbers separated by commas):\n"
+        display_message("\n🛠️ Customize your full development workflow:\n", type: :highlight)
+        display_message("Select the steps you want to include (enter numbers separated by commas):\n")
 
         available_steps = {
           "1" => "00_PRD - Product Requirements Document",
@@ -155,8 +170,8 @@ module Aidp
           "12" => "13_DELIVERY_ROLLOUT - Deployment Planning"
         }
 
-        available_steps.each { |num, desc| puts "  #{num}. #{desc}" }
-        puts ""
+        available_steps.each { |num, desc| display_message("  #{num}. #{desc}") }
+        display_message("")
 
         selected = prompt_required("Enter step numbers (e.g., 1,3,5,9,10): ")
         selected_numbers = selected.split(",").map(&:strip).map(&:to_i)
@@ -193,7 +208,7 @@ module Aidp
           input = @prompt.ask("#{question}:")
 
           if input.nil? || input.strip.empty?
-            puts "❌ This field is required. Please provide an answer."
+            display_message("❌ This field is required. Please provide an answer.", type: :error)
             next
           end
 

data/lib/aidp/harness/enhanced_runner.rb

@@ -50,7 +50,7 @@ module Aidp
         @configuration = Configuration.new(project_dir)
         @state_manager = StateManager.new(project_dir, @mode)
         @condition_detector = ConditionDetector.new
-        @provider_manager = ProviderManager.new(@configuration)
+        @provider_manager = ProviderManager.new(@configuration, prompt: TTY::Prompt.new)
         @error_handler = ErrorHandler.new(@provider_manager, @configuration)
         @completion_checker = CompletionChecker.new(@project_dir, @workflow_type)
       end
@@ -312,9 +312,9 @@ module Aidp
       def get_mode_runner
         case @mode
         when :analyze
-          Aidp::Analyze::Runner.new(@project_dir, self)
+          Aidp::Analyze::Runner.new(@project_dir, self, prompt: TTY::Prompt.new)
         when :execute
-          Aidp::Execute::Runner.new(@project_dir, self)
+          Aidp::Execute::Runner.new(@project_dir, self, prompt: TTY::Prompt.new)
         else
           raise ArgumentError, "Unsupported mode: #{@mode}"
         end

data/lib/aidp/harness/provider_factory.rb

@@ -7,6 +7,7 @@ require_relative "../providers/anthropic"
 require_relative "../providers/gemini"
 require_relative "../providers/macos_ui"
 require_relative "../providers/opencode"
+require_relative "../providers/github_copilot"
 
 module Aidp
   module Harness
@@ -17,7 +18,8 @@ module Aidp
         "anthropic" => Aidp::Providers::Anthropic,
         "gemini" => Aidp::Providers::Gemini,
         "macos" => Aidp::Providers::MacOSUI,
-        "opencode" => Aidp::Providers::Opencode
+        "opencode" => Aidp::Providers::Opencode,
+        "github_copilot" => Aidp::Providers::GithubCopilot
       }.freeze
 
       def initialize(config_manager = nil)

data/lib/aidp/harness/provider_manager.rb

@@ -1,13 +1,15 @@
 # frozen_string_literal: true
 
+require "tty-prompt"
 require_relative "provider_factory"
 
 module Aidp
   module Harness
     # Manages provider switching and fallback logic
     class ProviderManager
-      def initialize(configuration)
+      def initialize(configuration, prompt: TTY::Prompt.new)
         @configuration = configuration
+        @prompt = prompt
         @current_provider = nil
         @current_model = nil
         @provider_history = []
@@ -1199,20 +1201,37 @@ module Aidp
         [delay, 60].min # Cap at 60 seconds
       end
 
+      private
+
+      def display_message(message, type: :info)
+        color = case type
+        when :error then :red
+        when :success then :green
+        when :warning then :yellow
+        when :info then :blue
+        when :highlight then :cyan
+        when :muted then :bright_black
+        else :white
+        end
+        @prompt.say(message, color: color)
+      end
+
+      public
+
       # Log provider switch
       def log_provider_switch(from_provider, to_provider, reason, context)
-        puts "🔄 Provider switch: #{from_provider} → #{to_provider} (#{reason})"
+        display_message("🔄 Provider switch: #{from_provider} → #{to_provider} (#{reason})", type: :info)
         if context.any?
-          puts "   Context: #{context.inspect}"
+          display_message("   Context: #{context.inspect}", type: :muted)
         end
       end
 
       # Log no providers available
       def log_no_providers_available(reason, context)
-        puts "❌ No providers available for switching (#{reason})"
-        puts "   All providers are rate limited, unhealthy, or circuit breaker open"
+        display_message("❌ No providers available for switching (#{reason})", type: :error)
+        display_message("   All providers are rate limited, unhealthy, or circuit breaker open", type: :warning)
         if context.any?
-          puts "   Context: #{context.inspect}"
+          display_message("   Context: #{context.inspect}", type: :muted)
         end
       end
 
@@ -1220,26 +1239,26 @@ module Aidp
       def log_circuit_breaker_event(provider_name, event)
         case event
         when "opened"
-          puts "🔴 Circuit breaker opened for provider: #{provider_name}"
+          display_message("🔴 Circuit breaker opened for provider: #{provider_name}", type: :error)
         when "reset"
-          puts "🟢 Circuit breaker reset for provider: #{provider_name}"
+          display_message("🟢 Circuit breaker reset for provider: #{provider_name}", type: :success)
         end
       end
 
       # Log model switch
       def log_model_switch(from_model, to_model, reason, context)
-        puts "🔄 Model switch: #{from_model} → #{to_model} (#{reason})"
+        display_message("🔄 Model switch: #{from_model} → #{to_model} (#{reason})", type: :info)
         if context.any?
-          puts "   Context: #{context.inspect}"
+          display_message("   Context: #{context.inspect}", type: :muted)
         end
       end
 
       # Log no models available
       def log_no_models_available(provider_name, reason, context)
-        puts "❌ No models available for provider #{provider_name} (#{reason})"
-        puts "   All models are rate limited, unhealthy, or circuit breaker open"
+        display_message("❌ No models available for provider #{provider_name} (#{reason})", type: :error)
+        display_message("   All models are rate limited, unhealthy, or circuit breaker open", type: :warning)
         if context.any?
-          puts "   Context: #{context.inspect}"
+          display_message("   Context: #{context.inspect}", type: :muted)
         end
       end
 
@@ -1247,9 +1266,9 @@ module Aidp
       def log_model_circuit_breaker_event(provider_name, model_name, event)
         case event
         when "opened"
-          puts "🔴 Circuit breaker opened for model: #{provider_name}:#{model_name}"
+          display_message("🔴 Circuit breaker opened for model: #{provider_name}:#{model_name}", type: :error)
         when "reset"
-          puts "🟢 Circuit breaker reset for model: #{provider_name}:#{model_name}"
+          display_message("🟢 Circuit breaker reset for model: #{provider_name}:#{model_name}", type: :success)
         end
       end
 

data/lib/aidp/harness/runner.rb

@@ -27,6 +27,9 @@ module Aidp
         error: "error"
       }.freeze
 
+      # Public accessors for testing and integration
+      attr_reader :current_provider, :current_step, :user_input, :execution_log, :provider_manager
+
       def initialize(project_dir, mode = :analyze, options = {})
         @project_dir = project_dir
         @mode = mode.to_sym
@@ -37,6 +40,7 @@ module Aidp
         @current_provider = nil
         @user_input = options[:user_input] || {}  # Include user input from workflow selection
         @execution_log = []
+        @prompt = options[:prompt] || TTY::Prompt.new
 
         # Store workflow configuration
         @selected_steps = options[:selected_steps]
@@ -46,7 +50,7 @@ module Aidp
         @configuration = Configuration.new(project_dir)
         @state_manager = StateManager.new(project_dir, @mode)
         @condition_detector = ConditionDetector.new
-        @provider_manager = ProviderManager.new(@configuration)
+        @provider_manager = ProviderManager.new(@configuration, prompt: @prompt)
         @user_interface = SimpleUserInterface.new
         @error_handler = ErrorHandler.new(@provider_manager, @configuration)
         @status_display = StatusDisplay.new
@@ -96,8 +100,8 @@ module Aidp
               log_execution("Harness completed successfully - all criteria met", completion_status)
             else
               log_execution("Steps completed but completion criteria not met", completion_status)
-              puts "\n⚠️  All steps completed but some completion criteria not met:"
-              puts completion_status[:summary]
+              display_message("\n⚠️  All steps completed but some completion criteria not met:", type: :warning)
+              display_message(completion_status[:summary], type: :info)
 
               # Ask user if they want to continue anyway
               if @user_interface.get_confirmation("Continue anyway? This may indicate issues that should be addressed.", default: false)
@@ -181,9 +185,9 @@ module Aidp
       def get_mode_runner
         case @mode
         when :analyze
-          Aidp::Analyze::Runner.new(@project_dir, self)
+          Aidp::Analyze::Runner.new(@project_dir, self, prompt: TTY::Prompt.new)
         when :execute
-          Aidp::Execute::Runner.new(@project_dir, self)
+          Aidp::Execute::Runner.new(@project_dir, self, prompt: TTY::Prompt.new)
         else
           raise ArgumentError, "Unsupported mode: #{@mode}"
         end
@@ -406,6 +410,21 @@ module Aidp
           "Harness finished in state: #{@state}"
         end
       end
+
+      private
+
+      def display_message(message, type: :info)
+        color = case type
+        when :error then :red
+        when :success then :green
+        when :warning then :yellow
+        when :info then :blue
+        when :highlight then :cyan
+        when :muted then :bright_black
+        else :white
+        end
+        @prompt.say(message, color: color)
+      end
     end
   end
 end

data/lib/aidp/harness/simple_user_interface.rb

@@ -26,10 +26,25 @@ module Aidp
 
       private
 
+      # Helper method for consistent message display using TTY::Prompt
+      def display_message(message, type: :info)
+        color = case type
+        when :error then :red
+        when :success then :green
+        when :warning then :yellow
+        when :info then :blue
+        when :highlight then :cyan
+        when :muted then :bright_black
+        else :white
+        end
+
+        @prompt.say(message, color: color)
+      end
+
       def show_context(context)
-        puts "\n🤖 Agent needs feedback"
-        puts "Context: #{context[:description]}" if context[:description]
-        puts ""
+        display_message("\n🤖 Agent needs feedback", type: :info)
+        display_message("Context: #{context[:description]}", type: :info) if context[:description]
+        display_message("", type: :info)
       end
 
       def ask_question(question_data)
@@ -39,7 +54,7 @@ module Aidp
         required = question_data[:required] != false
         options = question_data[:options]
 
-        puts "\n#{question}"
+        display_message("\n#{question}", type: :info)
 
         case type
         when "text"