aidp 0.10.0 → 0.11.0

data/lib/aidp/harness/status_display.rb

@@ -1,17 +1,21 @@
 # frozen_string_literal: true
 
 require "tty-prompt"
+require "tty-cursor"
 
 module Aidp
   module Harness
     # Real-time status updates and monitoring interface
     class StatusDisplay
+      include Aidp::MessageDisplay
+
       def initialize(provider_manager = nil, metrics_manager = nil, circuit_breaker_manager = nil, error_logger = nil, prompt: TTY::Prompt.new)
         @provider_manager = provider_manager
         @metrics_manager = metrics_manager
         @circuit_breaker_manager = circuit_breaker_manager
         @error_logger = error_logger
         @prompt = prompt
+        @cursor = TTY::Cursor
 
         @start_time = nil
         @current_step = nil
@@ -41,21 +45,6 @@ module Aidp
         @display_animator = DisplayAnimator.new
       end
 
-      # 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
-
       # Start real-time status updates
       def start_status_updates(display_mode = :compact)
         return if @running
@@ -755,8 +744,14 @@ module Aidp
       end
 
       def clear_display
-        # Clear the current line and move cursor to beginning
-        print "\r" + " " * 80 + "\r"
+        # Clear the current line using TTY::Cursor
+        print_to_stderr(@cursor.clear_line, @cursor.column(1))
+      end
+
+      # Helper method to print to stderr with flush
+      def print_to_stderr(*parts)
+        parts.each { |part| $stderr.print part }
+        $stderr.flush
       end
 
       def format_duration(seconds)

data/lib/aidp/harness/test_runner.rb

@@ -0,0 +1,83 @@
+# frozen_string_literal: true
+
+require "open3"
+
+module Aidp
+  module Harness
+    # Executes test and linter commands configured in aidp.yml
+    # Returns results with exit status and output
+    class TestRunner
+      def initialize(project_dir, config)
+        @project_dir = project_dir
+        @config = config
+      end
+
+      # Run all configured tests
+      # Returns: { success: boolean, output: string, failures: array }
+      def run_tests
+        test_commands = @config.test_commands || []
+        return {success: true, output: "", failures: []} if test_commands.empty?
+
+        results = test_commands.map { |cmd| execute_command(cmd, "test") }
+        aggregate_results(results, "Tests")
+      end
+
+      # Run all configured linters
+      # Returns: { success: boolean, output: string, failures: array }
+      def run_linters
+        lint_commands = @config.lint_commands || []
+        return {success: true, output: "", failures: []} if lint_commands.empty?
+
+        results = lint_commands.map { |cmd| execute_command(cmd, "linter") }
+        aggregate_results(results, "Linters")
+      end
+
+      private
+
+      def execute_command(command, type)
+        stdout, stderr, status = Open3.capture3(command, chdir: @project_dir)
+
+        {
+          command: command,
+          type: type,
+          success: status.success?,
+          exit_code: status.exitstatus,
+          stdout: stdout,
+          stderr: stderr
+        }
+      end
+
+      def aggregate_results(results, category)
+        failures = results.reject { |r| r[:success] }
+        success = failures.empty?
+
+        output = if success
+          "#{category}: All passed"
+        else
+          format_failures(failures, category)
+        end
+
+        {
+          success: success,
+          output: output,
+          failures: failures
+        }
+      end
+
+      def format_failures(failures, category)
+        output = ["#{category} Failures:", ""]
+
+        failures.each do |failure|
+          output << "Command: #{failure[:command]}"
+          output << "Exit Code: #{failure[:exit_code]}"
+          output << "--- Output ---"
+          output << failure[:stdout] unless failure[:stdout].strip.empty?
+          output << failure[:stderr] unless failure[:stderr].strip.empty?
+          output << ""
+        end
+
+        output.join("\n")
+      end
+    end
+  end
+end

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

@@ -15,7 +15,9 @@ module Aidp
       # Enhanced TUI system using TTY libraries, inspired by Claude Code and modern LLM agents
       class EnhancedTUI
         class TUIError < StandardError; end
+
         class InputError < TUIError; end
+
         class DisplayError < TUIError; end
 
         def initialize(prompt: TTY::Prompt.new)

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

@@ -1,6 +1,7 @@
 # frozen_string_literal: true
 
 require_relative "enhanced_tui"
+require_relative "../../workflows/selector"
 
 module Aidp
   module Harness
@@ -12,6 +13,7 @@ module Aidp
         def initialize(tui = nil)
           @tui = tui || EnhancedTUI.new
           @user_input = {}
+          @workflow_selector = Aidp::Workflows::Selector.new
         end
 
         def select_workflow(harness_mode: false, mode: :analyze)
@@ -29,7 +31,7 @@ module Aidp
           when :analyze
             select_analyze_workflow_interactive
           when :execute
-            select_execute_workflow_interactive
+            select_execute_workflow_interactive_new(mode)
           else
             raise ArgumentError, "Unknown mode: #{mode}"
           end
@@ -65,6 +67,22 @@ module Aidp
           }
         end
 
+        def select_execute_workflow_interactive_new(mode)
+          # Step 1: Collect project information
+          collect_project_info_interactive
+
+          # Step 2: Use new workflow selector
+          result = @workflow_selector.select_workflow(mode)
+
+          {
+            workflow_type: result[:workflow_key],
+            steps: result[:steps],
+            user_input: @user_input,
+            workflow: result[:workflow]
+          }
+        end
+
+        # Legacy method - kept for backward compatibility if needed
         def select_execute_workflow_interactive
           # Step 1: Collect project information
           collect_project_info_interactive
@@ -160,10 +178,10 @@ module Aidp
 
         def generate_exploration_steps
           [
-            "00_PRD",           # Generate PRD from user input (no manual gate)
+            "00_PRD", # Generate PRD from user input (no manual gate)
             "10_TESTING_STRATEGY", # Ensure we have tests
-            "11_STATIC_ANALYSIS",  # Code quality
-            "16_IMPLEMENTATION"    # Special step for actual development work
+            "11_STATIC_ANALYSIS", # Code quality
+            "16_IMPLEMENTATION" # Special step for actual development work
           ]
         end
 

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

@@ -9,9 +9,13 @@ module Aidp
       # Centralized error handling for UI components
       class ErrorHandler
         class UIError < StandardError; end
+
         class ComponentError < UIError; end
+
         class ValidationError < UIError; end
+
         class DisplayError < UIError; end
+
         class InteractionError < UIError; end
 
         def initialize(ui_components = {})

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

@@ -9,7 +9,9 @@ module Aidp
       # Handles nested framing using CLI UI frames
       class FrameManager < Base
         class FrameError < StandardError; end
+
         class InvalidFrameError < FrameError; end
+
         class DisplayError < FrameError; end
 
         def initialize(ui_components = {})
@@ -56,7 +58,7 @@ module Aidp
             @frame_stats[:status_counts][frame_data[:status]] += 1
           end
 
-          if block_given?
+          if block
             content = yield
             display_message(@frame.frame(formatted_title, content, width: 80))
           else
@@ -77,7 +79,7 @@ module Aidp
           @frame_stack.push({type: frame_type, title: title, data: frame_data})
 
           frame_result = @frame.frame(formatted_title, width: 80) do
-            if block_given?
+            if block
               yield || ""
             else
               ""
@@ -200,7 +202,7 @@ module Aidp
         def frame_with_block(frame_type, title, frame_data = nil, &block)
           validate_frame_type(frame_type)
           validate_title(title)
-          raise ArgumentError, "Block required for frame_with_block" unless block_given?
+          raise ArgumentError, "Block required for frame_with_block" unless block
 
           formatted_title = @formatter ? @formatter.format_frame_title(frame_type, title, frame_data) : title.to_s
           @frame_open = true
@@ -220,7 +222,7 @@ module Aidp
           rescue => e
             @frame_open = false
             @frame_stack.pop unless @frame_stack.empty?
-            raise e  # Re-raise the original exception
+            raise e # Re-raise the original exception
           end
         end
 
@@ -237,7 +239,7 @@ module Aidp
           validate_title(title)
 
           formatted_title = @formatter ? @formatter.format_section_title(title) : "📋 #{title}"
-          if block_given?
+          if block
             content = yield
             display_message(@frame.frame(formatted_title, content, width: 80))
           else
@@ -252,7 +254,7 @@ module Aidp
 
           formatted_title = @formatter ? @formatter.format_subsection_title(title) : "📝 #{title}"
           display_message(@frame.frame(formatted_title, width: 80) do
-            yield if block_given?
+            yield if block
           end)
         rescue => e
           raise DisplayError, "Failed to create subsection: #{e.message}"
@@ -263,7 +265,7 @@ module Aidp
 
           formatted_title = @formatter ? @formatter.format_workflow_title(workflow_name) : "⚙️ #{workflow_name}"
           display_message(@frame.frame(formatted_title, width: 80) do
-            yield if block_given?
+            yield if block
           end)
         rescue => e
           raise DisplayError, "Failed to create workflow frame: #{e.message}"
@@ -274,7 +276,7 @@ module Aidp
 
           formatted_title = @formatter ? @formatter.format_step_title(step_name, step_number, total_steps) : "🔧 #{step_name} (#{step_number}/#{total_steps})"
           display_message(@frame.frame(formatted_title, width: 80) do
-            yield if block_given?
+            yield if block
           end)
         rescue => e
           raise DisplayError, "Failed to create step frame: #{e.message}"

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

@@ -12,7 +12,9 @@ module Aidp
       # Real-time job monitoring and status tracking
       class JobMonitor < Base
         class JobMonitorError < StandardError; end
+
         class JobNotFoundError < JobMonitorError; end
+
         class MonitorError < JobMonitorError; end
 
         JOB_STATUSES = {

data/lib/aidp/harness/ui/navigation/main_menu.rb

@@ -14,7 +14,9 @@ module Aidp
         # Main hierarchical navigation menu system
         class MainMenu < Base
           class MenuError < StandardError; end
+
           class InvalidMenuError < MenuError; end
+
           class NavigationError < MenuError; end
 
           def initialize(ui_components = {}, prompt: nil)
@@ -27,7 +29,7 @@ module Aidp
             @menu_items = []
             @current_level = 0
             @breadcrumb = []
-            @navigation_history = []  # Track all navigation actions
+            @navigation_history = [] # Track all navigation actions
           end
 
           def add_menu_item(item)
@@ -113,7 +115,7 @@ module Aidp
             @menu_items = menu_items
             display_menu_items
 
-            max_attempts = 10  # Prevent infinite loops
+            max_attempts = 10 # Prevent infinite loops
             attempts = 0
 
             loop do

data/lib/aidp/harness/ui/navigation/menu_item.rb

@@ -7,6 +7,7 @@ module Aidp
         # Represents a single menu item in the navigation system
         class MenuItem
           class MenuItemError < StandardError; end
+
           class InvalidTypeError < MenuItemError; end
 
           VALID_TYPES = [:action, :submenu, :workflow, :separator].freeze

data/lib/aidp/harness/ui/navigation/menu_state.rb

@@ -7,6 +7,7 @@ module Aidp
         # Manages navigation state and history
         class MenuState
           class StateError < StandardError; end
+
           class InvalidStateError < StateError; end
 
           def initialize

data/lib/aidp/harness/ui/navigation/submenu.rb

@@ -9,6 +9,7 @@ module Aidp
         # Specialized submenu for drill-down functionality
         class SubMenu < MainMenu
           class SubMenuError < MenuError; end
+
           class InvalidSubMenuError < SubMenuError; end
 
           def initialize(title, parent_menu = nil, ui_components = {})

data/lib/aidp/harness/ui/navigation/workflow_selector.rb

@@ -12,7 +12,9 @@ module Aidp
         # Handles workflow mode selection (simple vs advanced)
         class WorkflowSelector < Base
           class WorkflowError < StandardError; end
+
           class InvalidModeError < WorkflowError; end
+
           class SelectionError < WorkflowError; end
 
           WORKFLOW_MODES = {

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

@@ -10,8 +10,12 @@ module Aidp
     module UI
       # Handles progress display using CLI UI progress bars
       class ProgressDisplay < Base
+        include Aidp::MessageDisplay
+
         class ProgressError < StandardError; end
+
         class InvalidProgressError < ProgressError; end
+
         class DisplayError < ProgressError; end
 
         attr_reader :refresh_interval
@@ -178,7 +182,7 @@ module Aidp
 
         def execute_progress_steps(bar, total_steps, &block)
           total_steps.times do
-            yield(bar) if block_given?
+            yield(bar) if block
             bar.tick
           end
         end
@@ -188,7 +192,7 @@ module Aidp
           total_substeps.times do |index|
             substep_title = @formatter.format_substep_title(title, index + 1, total_substeps)
             bar.update_title(substep_title)
-            yield(bar, index) if block_given?
+            yield(bar, index) if block
             bar.tick
           end
         end
@@ -256,18 +260,10 @@ module Aidp
 
         private
 
+        # Use mixin display_message; fallback to stdout if no prompt
         def display_message(message, type: :info)
           if @prompt
-            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)
+            super
           elsif @output
             @output.puts(message)
           else

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

@@ -9,7 +9,9 @@ module Aidp
       # Handles interactive question collection using CLI UI prompts
       class QuestionCollector < Base
         class QuestionError < StandardError; end
+
         class ValidationError < QuestionError; end
+
         class CollectionError < QuestionError; end
 
         def initialize(ui_components = {}, prompt: nil)

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

@@ -9,7 +9,9 @@ module Aidp
       # Handles concurrent operations using CLI UI spinner groups
       class SpinnerGroup < Base
         class SpinnerGroupError < StandardError; end
+
         class InvalidOperationError < SpinnerGroupError; end
+
         class ExecutionError < SpinnerGroupError; end
 
         def initialize(ui_components = {})

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

@@ -18,7 +18,7 @@ module Aidp
 
         # Main method: automatically manages spinner around a block
         def with_spinner(message, format: :dots, success_message: nil, error_message: nil, &block)
-          raise ArgumentError, "Block required for with_spinner" unless block_given?
+          raise ArgumentError, "Block required for with_spinner" unless block
 
           spinner = create_spinner(message, format)
           start_spinner(spinner)

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

@@ -12,7 +12,9 @@ module Aidp
       # Real-time status updates using CLI UI spinners
       class StatusManager < Base
         class StatusError < StandardError; end
+
         class InvalidStatusError < StatusError; end
+
         class UpdateError < StatusError; end
 
         def initialize(ui_components = {})
@@ -164,7 +166,7 @@ module Aidp
         end
 
         def track_workflow_status(workflow_name, spinner, &block) # Will be updated dynamically
-          yield(spinner) if block_given?
+          yield(spinner) if block
           @status_widget.show_success_status("Completed #{workflow_name}")
         rescue => e
           @status_widget.show_error_status("Failed #{workflow_name}: #{e.message}")
@@ -172,7 +174,7 @@ module Aidp
         end
 
         def track_step_status(step_name, spinner, &block)
-          yield(spinner) if block_given?
+          yield(spinner) if block
           @status_widget.show_success_status("Completed #{step_name}")
         rescue => e
           @status_widget.show_error_status("Failed #{step_name}: #{e.message}")

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

@@ -10,7 +10,9 @@ module Aidp
       # Handles status display using CLI UI spinners
       class StatusWidget < Base
         class StatusError < StandardError; end
+
         class InvalidStatusError < StatusError; end
+
         class DisplayError < StatusError; end
 
         def initialize(ui_components = {})
@@ -29,7 +31,7 @@ module Aidp
 
           formatted_message = @formatter.format_status_message(message)
           @spinner.spin(formatted_message) do |spinner|
-            yield(spinner) if block_given?
+            yield(spinner) if block
           end
         rescue => e
           raise DisplayError, "Failed to show status: #{e.message}"

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

@@ -11,7 +11,9 @@ module Aidp
       # Workflow control interface for pause/resume/cancel/stop
       class WorkflowController < Base
         class WorkflowError < StandardError; end
+
         class InvalidStateError < WorkflowError; end
+
         class ControlError < WorkflowError; end
 
         WORKFLOW_STATES = {

data/lib/aidp/harness/user_interface.rb

@@ -1,15 +1,19 @@
 # frozen_string_literal: true
 
 require "tty-prompt"
+require "tty-cursor"
 
 module Aidp
   module Harness
     # Handles user interaction and feedback collection
     class UserInterface
+      include Aidp::MessageDisplay
+
       def initialize(prompt: TTY::Prompt.new)
         @input_history = []
         @file_selection_enabled = false
         @prompt = prompt
+        @cursor = TTY::Cursor
         @control_mutex = Mutex.new
         @pause_requested = false
         @stop_requested = false
@@ -20,21 +24,6 @@ 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
-
       # 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)
@@ -1835,12 +1824,18 @@ module Aidp
 
       # Display progress message
       def show_progress(message)
-        print "\r#{message}".ljust(80)
+        print_to_stderr(@cursor.clear_line, @cursor.column(1), message)
       end
 
       # Clear progress message
       def clear_progress
-        print "\r" + " " * 80 + "\r"
+        print_to_stderr(@cursor.clear_line, @cursor.column(1))
+      end
+
+      # Helper method to print to stderr with flush
+      def print_to_stderr(*parts)
+        parts.each { |part| $stderr.print part }
+        $stderr.flush
       end
 
       # Get input history