aidp 0.9.6 → 0.10.0

data/lib/aidp/provider_manager.rb

@@ -1,5 +1,6 @@
 # frozen_string_literal: true
 
+require "tty-prompt"
 require_relative "harness/provider_factory"
 
 module Aidp
@@ -13,7 +14,8 @@ module Aidp
         end
 
         # Fallback to legacy method
-        create_legacy_provider(provider_type)
+        prompt = options[:prompt] || TTY::Prompt.new
+        create_legacy_provider(provider_type, prompt: prompt)
       end
 
       def load_from_config(config = {}, options = {})
@@ -132,16 +134,18 @@ module Aidp
 
       private
 
-      def create_legacy_provider(provider_type)
+      def create_legacy_provider(provider_type, prompt: TTY::Prompt.new)
         case provider_type
         when "cursor"
-          Aidp::Providers::Cursor.new
+          Aidp::Providers::Cursor.new(prompt: prompt)
         when "anthropic"
-          Aidp::Providers::Anthropic.new
+          Aidp::Providers::Anthropic.new(prompt: prompt)
         when "gemini"
-          Aidp::Providers::Gemini.new
+          Aidp::Providers::Gemini.new(prompt: prompt)
         when "macos_ui"
-          Aidp::Providers::MacOSUI.new
+          Aidp::Providers::MacOSUI.new(prompt: prompt)
+        when "github_copilot"
+          Aidp::Providers::GithubCopilot.new(prompt: prompt)
         end
       end
     end

data/lib/aidp/providers/anthropic.rb

@@ -58,7 +58,7 @@ module Aidp
         # 4. Default timeout
 
         if ENV["AIDP_QUICK_MODE"]
-          puts "⚡ Quick mode enabled - 2 minute timeout"
+          display_message("⚡ Quick mode enabled - 2 minute timeout", type: :highlight)
           return 120
         end
 
@@ -69,12 +69,12 @@ module Aidp
         # Adaptive timeout based on step type
         step_timeout = get_adaptive_timeout
         if step_timeout
-          puts "🧠 Using adaptive timeout: #{step_timeout} seconds"
+          display_message("🧠 Using adaptive timeout: #{step_timeout} seconds", type: :info)
           return step_timeout
         end
 
         # Default timeout (5 minutes for interactive use)
-        puts "📋 Using default timeout: 5 minutes"
+        display_message("📋 Using default timeout: 5 minutes", type: :info)
         300
       end
 

data/lib/aidp/providers/base.rb

@@ -1,5 +1,7 @@
 # frozen_string_literal: true
 
+require "tty-prompt"
+
 module Aidp
   module Providers
     class Base
@@ -17,7 +19,7 @@ module Aidp
 
       attr_reader :activity_state, :last_activity_time, :start_time, :step_name
 
-      def initialize
+      def initialize(output: nil, prompt: TTY::Prompt.new)
         @activity_state = :idle
         @last_activity_time = Time.now
         @start_time = nil
@@ -28,6 +30,8 @@ module Aidp
         @last_output_time = Time.now
         @job_context = nil
         @harness_context = nil
+        @output = output
+        @prompt = prompt
         @harness_metrics = {
           total_requests: 0,
           successful_requests: 0,
@@ -327,6 +331,21 @@ module Aidp
         # Weighted health score
         (success_rate * 50) + ((1 - rate_limit_ratio) * 30) + (response_time_score * 0.2)
       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/providers/cursor.rb

@@ -95,13 +95,11 @@ module Aidp
         else
           print "\r🔄 cursor-agent is running... (#{seconds}s)"
         end
-        $stdout.flush
       end
 
       def clear_activity_status
         # Clear the activity status line
         print "\r" + " " * 50 + "\r"
-        $stdout.flush
       end
 
       def calculate_timeout
@@ -112,7 +110,7 @@ module Aidp
         # 4. Default timeout
 
         if ENV["AIDP_QUICK_MODE"]
-          puts "⚡ Quick mode enabled - 2 minute timeout"
+          display_message("⚡ Quick mode enabled - 2 minute timeout", type: :highlight)
           return 120
         end
 
@@ -123,12 +121,12 @@ module Aidp
         # Adaptive timeout based on step type
         step_timeout = get_adaptive_timeout
         if step_timeout
-          puts "🧠 Using adaptive timeout: #{step_timeout} seconds"
+          display_message("🧠 Using adaptive timeout: #{step_timeout} seconds", type: :info)
           return step_timeout
         end
 
         # Default timeout (5 minutes for interactive use)
-        puts "📋 Using default timeout: 5 minutes"
+        display_message("📋 Using default timeout: 5 minutes", type: :info)
         300
       end
 
@@ -161,11 +159,11 @@ module Aidp
         # Only print static messages for state changes
         case state
         when :stuck
-          puts "\n⚠️  cursor appears stuck: #{message}"
+          display_message("\n⚠️  cursor appears stuck: #{message}", type: :warning)
         when :completed
-          puts "\n✅ cursor completed: #{message}"
+          display_message("\n✅ cursor completed: #{message}", type: :success)
         when :failed
-          puts "\n❌ cursor failed: #{message}"
+          display_message("\n❌ cursor failed: #{message}", type: :error)
         end
       end
     end

data/lib/aidp/providers/gemini.rb

@@ -54,7 +54,7 @@ module Aidp
         # 4. Default timeout
 
         if ENV["AIDP_QUICK_MODE"]
-          puts "⚡ Quick mode enabled - 2 minute timeout"
+          display_message("⚡ Quick mode enabled - 2 minute timeout", type: :highlight)
           return 120
         end
 
@@ -65,12 +65,12 @@ module Aidp
         # Adaptive timeout based on step type
         step_timeout = get_adaptive_timeout
         if step_timeout
-          puts "🧠 Using adaptive timeout: #{step_timeout} seconds"
+          display_message("🧠 Using adaptive timeout: #{step_timeout} seconds", type: :info)
           return step_timeout
         end
 
         # Default timeout (5 minutes for interactive use)
-        puts "📋 Using default timeout: 5 minutes"
+        display_message("📋 Using default timeout: 5 minutes", type: :info)
         300
       end
 

data/lib/aidp/providers/github_copilot.rb

@@ -0,0 +1,264 @@
+# frozen_string_literal: true
+
+require "timeout"
+require_relative "base"
+require_relative "../util"
+require_relative "../debug_mixin"
+
+module Aidp
+  module Providers
+    class GithubCopilot < Base
+      include Aidp::DebugMixin
+
+      def self.available?
+        !!Aidp::Util.which("copilot")
+      end
+
+      def name
+        "github_copilot"
+      end
+
+      def available?
+        return false unless self.class.available?
+
+        # Additional check to ensure the CLI is properly configured
+        begin
+          result = Aidp::Util.execute_command("copilot", ["--version"], timeout: 10)
+          result.exit_status == 0
+        rescue
+          false
+        end
+      end
+
+      def send(prompt:, session: nil)
+        raise "copilot CLI not available" unless self.class.available?
+
+        # Smart timeout calculation
+        timeout_seconds = calculate_timeout
+
+        debug_provider("copilot", "Starting execution", {timeout: timeout_seconds})
+        debug_log("📝 Sending prompt to copilot (length: #{prompt.length})", level: :info)
+
+        # Set up activity monitoring
+        setup_activity_monitoring("copilot", method(:activity_callback))
+        record_activity("Starting copilot execution")
+
+        # Start activity display thread with timeout
+        activity_display_thread = Thread.new do
+          start_time = Time.now
+          loop do
+            sleep 0.5 # Update every 500ms to reduce spam
+            elapsed = Time.now - start_time
+
+            # Break if we've been running too long or state changed
+            break if elapsed > timeout_seconds || @activity_state == :completed || @activity_state == :failed
+
+            print_activity_status(elapsed)
+          end
+        end
+
+        begin
+          # Use non-interactive mode for automation
+          args = ["-p", prompt, "--allow-all-tools"]
+
+          # Add session support if provided
+          if session && !session.empty?
+            args += ["--resume", session]
+          end
+
+          # Use debug_execute_command for better debugging (no input since prompt is in args)
+          result = debug_execute_command("copilot", args: args, timeout: timeout_seconds)
+
+          # Log the results
+          debug_command("copilot", args: args, input: prompt, output: result.out, error: result.err, exit_code: result.exit_status)
+
+          # Stop activity display
+          activity_display_thread.kill if activity_display_thread.alive?
+          activity_display_thread.join(0.1) # Give it 100ms to finish
+          clear_activity_status
+
+          if result.exit_status == 0
+            mark_completed
+            result.out
+          else
+            mark_failed("copilot failed with exit code #{result.exit_status}")
+            debug_error(StandardError.new("copilot failed"), {exit_code: result.exit_status, stderr: result.err})
+            raise "copilot failed with exit code #{result.exit_status}: #{result.err}"
+          end
+        rescue => e
+          # Stop activity display
+          activity_display_thread.kill if activity_display_thread.alive?
+          activity_display_thread.join(0.1) # Give it 100ms to finish
+          clear_activity_status
+          mark_failed("copilot execution failed: #{e.message}")
+          debug_error(e, {provider: "github_copilot", prompt_length: prompt.length})
+          raise
+        end
+      end
+
+      # Enhanced send method with additional options
+      def send_with_options(prompt:, session: nil, tools: nil, log_level: nil, config_file: nil, directories: nil)
+        args = ["-p", prompt]
+
+        # Add session support
+        if session && !session.empty?
+          args += ["--resume", session]
+        end
+
+        # Add tool permissions
+        if tools && !tools.empty?
+          if tools.include?("all")
+            args += ["--allow-all-tools"]
+          else
+            tools.each do |tool|
+              args += ["--allow-tool", tool]
+            end
+          end
+        else
+          # Default to allowing all tools for automation
+          args += ["--allow-all-tools"]
+        end
+
+        # Add logging level
+        if log_level
+          args += ["--log-level", log_level]
+        end
+
+        # Add allowed directories
+        if directories && !directories.empty?
+          directories.each do |dir|
+            args += ["--add-dir", dir]
+          end
+        end
+
+        # Use the enhanced version of send
+        send_with_custom_args(prompt: prompt, args: args)
+      end
+
+      # Override health check for GitHub Copilot specific considerations
+      def harness_healthy?
+        return false unless super
+
+        # Additional health checks specific to GitHub Copilot CLI
+        # Check if we can access GitHub (basic connectivity test)
+        begin
+          result = Aidp::Util.execute_command("copilot", ["--help"], timeout: 5)
+          result.exit_status == 0
+        rescue
+          false
+        end
+      end
+
+      private
+
+      def send_with_custom_args(prompt:, args:)
+        timeout_seconds = calculate_timeout
+
+        debug_provider("copilot", "Starting execution", {timeout: timeout_seconds, args: args})
+        debug_log("📝 Sending prompt to copilot with custom args", level: :info)
+
+        setup_activity_monitoring("copilot", method(:activity_callback))
+        record_activity("Starting copilot execution with custom args")
+
+        begin
+          result = debug_execute_command("copilot", args: args, timeout: timeout_seconds)
+          debug_command("copilot", args: args, output: result.out, error: result.err, exit_code: result.exit_status)
+
+          if result.exit_status == 0
+            mark_completed
+            result.out
+          else
+            mark_failed("copilot failed with exit code #{result.exit_status}")
+            debug_error(StandardError.new("copilot failed"), {exit_code: result.exit_status, stderr: result.err})
+            raise "copilot failed with exit code #{result.exit_status}: #{result.err}"
+          end
+        rescue => e
+          mark_failed("copilot execution failed: #{e.message}")
+          debug_error(e, {provider: "github_copilot", prompt_length: prompt.length})
+          raise
+        end
+      end
+
+      def print_activity_status(elapsed)
+        # Print activity status during execution with elapsed time
+        minutes = (elapsed / 60).to_i
+        seconds = (elapsed % 60).to_i
+
+        if minutes > 0
+          print "\r🤖 GitHub Copilot CLI is running... (#{minutes}m #{seconds}s)"
+        else
+          print "\r🤖 GitHub Copilot CLI is running... (#{seconds}s)"
+        end
+      end
+
+      def clear_activity_status
+        # Clear the activity status line
+        print "\r" + " " * 60 + "\r"
+      end
+
+      def calculate_timeout
+        # Priority order for timeout calculation:
+        # 1. Quick mode (for testing)
+        # 2. Environment variable override
+        # 3. Adaptive timeout based on step type
+        # 4. Default timeout
+
+        if ENV["AIDP_QUICK_MODE"]
+          display_message("⚡ Quick mode enabled - 2 minute timeout", type: :highlight)
+          return 120
+        end
+
+        if ENV["AIDP_GITHUB_COPILOT_TIMEOUT"]
+          return ENV["AIDP_GITHUB_COPILOT_TIMEOUT"].to_i
+        end
+
+        # Adaptive timeout based on step type
+        step_timeout = get_adaptive_timeout
+        if step_timeout
+          display_message("🧠 Using adaptive timeout: #{step_timeout} seconds", type: :info)
+          return step_timeout
+        end
+
+        # Default timeout (5 minutes for interactive use)
+        display_message("📋 Using default timeout: 5 minutes", type: :info)
+        300
+      end
+
+      def get_adaptive_timeout
+        # Timeout recommendations based on step type patterns
+        step_name = ENV["AIDP_CURRENT_STEP"] || ""
+
+        case step_name
+        when /REPOSITORY_ANALYSIS/
+          180  # 3 minutes - repository analysis can be quick
+        when /ARCHITECTURE_ANALYSIS/
+          600  # 10 minutes - architecture analysis needs more time
+        when /TEST_ANALYSIS/
+          300  # 5 minutes - test analysis is moderate
+        when /FUNCTIONALITY_ANALYSIS/
+          600  # 10 minutes - functionality analysis is complex
+        when /DOCUMENTATION_ANALYSIS/
+          300  # 5 minutes - documentation analysis is moderate
+        when /STATIC_ANALYSIS/
+          450  # 7.5 minutes - static analysis can be intensive
+        when /REFACTORING_RECOMMENDATIONS/
+          600  # 10 minutes - refactoring recommendations are complex
+        else
+          nil  # Use default
+        end
+      end
+
+      def activity_callback(state, message, provider)
+        # Handle activity state changes
+        case state
+        when :stuck
+          display_message("\n⚠️  GitHub Copilot CLI appears stuck: #{message}", type: :warning)
+        when :completed
+          display_message("\n✅ GitHub Copilot CLI completed: #{message}", type: :success)
+        when :failed
+          display_message("\n❌ GitHub Copilot CLI failed: #{message}", type: :error)
+        end
+      end
+    end
+  end
+end

data/lib/aidp/providers/opencode.rb

@@ -95,13 +95,11 @@ module Aidp
         else
           print "\r🔄 opencode is running... (#{seconds}s)"
         end
-        $stdout.flush
       end
 
       def clear_activity_status
         # Clear the activity status line
         print "\r" + " " * 50 + "\r"
-        $stdout.flush
       end
 
       def calculate_timeout
@@ -112,7 +110,7 @@ module Aidp
         # 4. Default timeout
 
         if ENV["AIDP_QUICK_MODE"]
-          puts "⚡ Quick mode enabled - 2 minute timeout"
+          display_message("⚡ Quick mode enabled - 2 minute timeout", type: :highlight)
           return 120
         end
 
@@ -123,12 +121,12 @@ module Aidp
         # Adaptive timeout based on step type
         step_timeout = get_adaptive_timeout
         if step_timeout
-          puts "🧠 Using adaptive timeout: #{step_timeout} seconds"
+          display_message("🧠 Using adaptive timeout: #{step_timeout} seconds", type: :info)
           return step_timeout
         end
 
         # Default timeout (5 minutes for interactive use)
-        puts "📋 Using default timeout: 5 minutes"
+        display_message("📋 Using default timeout: 5 minutes", type: :info)
         300
       end
 
@@ -161,11 +159,11 @@ module Aidp
         # Only print static messages for state changes
         case state
         when :starting
-          puts "🚀 Starting opencode execution..."
+          display_message("🚀 Starting opencode execution...", type: :info)
         when :completed
-          puts "✅ opencode execution completed"
+          display_message("✅ opencode execution completed", type: :success)
         when :failed
-          puts "❌ opencode execution failed: #{message}"
+          display_message("❌ opencode execution failed: #{message}", type: :error)
         end
       end
 

data/lib/aidp/version.rb

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

data/lib/aidp.rb

@@ -33,10 +33,10 @@ require_relative "aidp/analyze/steps"
 require_relative "aidp/analyze/progress"
 
 # Tree-sitter analysis
-require_relative "aidp/analysis/tree_sitter_grammar_loader"
-require_relative "aidp/analysis/seams"
-require_relative "aidp/analysis/tree_sitter_scan"
-require_relative "aidp/analysis/kb_inspector"
+require_relative "aidp/analyze/tree_sitter_grammar_loader"
+require_relative "aidp/analyze/seams"
+require_relative "aidp/analyze/tree_sitter_scan"
+require_relative "aidp/analyze/kb_inspector"
 
 # Execute mode
 require_relative "aidp/execute/steps"

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: aidp
 version: !ruby/object:Gem::Version
-  version: 0.9.6
+  version: 0.10.0
 platform: ruby
 authors:
 - Bart Agapinan
@@ -233,20 +233,19 @@ files:
 - README.md
 - bin/aidp
 - lib/aidp.rb
-- lib/aidp/analysis/kb_inspector.rb
-- lib/aidp/analysis/seams.rb
-- lib/aidp/analysis/tree_sitter_grammar_loader.rb
-- lib/aidp/analysis/tree_sitter_scan.rb
 - lib/aidp/analyze/error_handler.rb
 - lib/aidp/analyze/feature_analyzer.rb
 - lib/aidp/analyze/json_file_storage.rb
+- lib/aidp/analyze/kb_inspector.rb
 - lib/aidp/analyze/prioritizer.rb
 - lib/aidp/analyze/progress.rb
-- lib/aidp/analyze/progress_visualizer.rb
 - lib/aidp/analyze/report_generator.rb
 - lib/aidp/analyze/ruby_maat_integration.rb
 - lib/aidp/analyze/runner.rb
+- lib/aidp/analyze/seams.rb
 - lib/aidp/analyze/steps.rb
+- lib/aidp/analyze/tree_sitter_grammar_loader.rb
+- lib/aidp/analyze/tree_sitter_scan.rb
 - lib/aidp/cli.rb
 - lib/aidp/cli/first_run_wizard.rb
 - lib/aidp/cli/jobs_command.rb
@@ -307,6 +306,7 @@ files:
 - lib/aidp/providers/base.rb
 - lib/aidp/providers/cursor.rb
 - lib/aidp/providers/gemini.rb
+- lib/aidp/providers/github_copilot.rb
 - lib/aidp/providers/macos_ui.rb
 - lib/aidp/providers/opencode.rb
 - lib/aidp/storage/csv_storage.rb