aidp 0.14.1 → 0.14.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 8d97cff94f784545b96549c70d910e913bf19a7c94b908e45dae2a999c57e45b
-  data.tar.gz: 170bf8fe45dbb081cad38ef69eb8a0f58b962d495ed300b3d58d3f78ef553462
+  metadata.gz: e9ed49c611f18d153eef9794e7ad5077a4bd369988b21dc6849b26b4f9489691
+  data.tar.gz: 97d9812b8cb6437471b4e4bdfdff218b588e20d0187fe6878dd03fdc6fed9b25
 SHA512:
-  metadata.gz: a371614e261335e71b6e78cbe6d6410f26f03a4910e37e6b4102d3681fc9ef2abf617a8f4199b918f8efcdb43fc978a3eef9c1d04aa485297af90394bd34cbd8
-  data.tar.gz: 549e21d089121d0212698307bd8ed520499452ad829ff70e8ca1dea7fd50b6c63db9be2a55d4b56c9c60d822703710e84f57f91354bad8892e7367347e6b7777
+  metadata.gz: f88b7ff6548415eb12a364782a7b2f1a8dfdbc2983c9a776a4714c8d2cc0c58501c4180795ff62734b6dee26e2448d0cbfcab1a1a0e5ecb577b49bb24d2184b9
+  data.tar.gz: 67867d11f4909e48f2b6954f966e06291a3b4e3ad2196fc993bf5ebe0ad5f411547f6ed0bdd76bb864a0f0850148a87a2d0ba5265207a447788bb020f72b29b4

data/lib/aidp/cli.rb

@@ -8,11 +8,13 @@ require_relative "harness/ui/enhanced_tui"
 require_relative "harness/ui/enhanced_workflow_selector"
 require_relative "harness/enhanced_runner"
 require_relative "cli/first_run_wizard"
+require_relative "rescue_logging"
 
 module Aidp
   # CLI interface for AIDP
   class CLI
     include Aidp::MessageDisplay
+    include Aidp::RescueLogging
 
     # Simple options holder for instance methods (used in specs)
     attr_accessor :options
@@ -126,6 +128,7 @@ module Aidp
         {harness: {state: "unknown"}}
       end
     rescue => e
+      log_rescue(e, component: "cli", action: "fetch_harness_status", fallback: {harness: {state: "error"}}, mode: mode)
       {harness: {state: "error", error: e.message}}
     end
 
@@ -159,6 +162,10 @@ module Aidp
           return 0
         end
 
+        # Initialize logger from aidp.yml config
+        # Priority: ENV variable > aidp.yml > default (info)
+        setup_logging(Dir.pwd)
+
         # Start the interactive TUI
         display_message("AIDP initializing...", type: :info)
         display_message("   Press Ctrl+C to stop\n", type: :highlight)
@@ -216,6 +223,7 @@ module Aidp
           display_message("\n\n⏹️  Interrupted by user", type: :warning)
           1
         rescue => e
+          log_rescue(e, component: "cli", action: "run_harness", fallback: 1, mode: actual_mode)
           display_message("\n❌ Error: #{e.message}", type: :error)
           1
         ensure
@@ -225,6 +233,29 @@ module Aidp
 
       private
 
+      def setup_logging(project_dir)
+        # Load logging config from aidp.yml
+        config_path = File.join(project_dir, ".aidp", "aidp.yml")
+        logging_config = {}
+
+        if File.exist?(config_path)
+          require "yaml"
+          full_config = YAML.load_file(config_path)
+          logging_config = full_config["logging"] || full_config[:logging] || {}
+        end
+
+        # Set up logger with config (ENV variable AIDP_LOG_LEVEL takes precedence)
+        Aidp.setup_logger(project_dir, logging_config)
+
+        # Log initialization
+        Aidp.logger.info("cli", "AIDP starting", version: Aidp::VERSION, log_level: Aidp.logger.level)
+      rescue => e
+        log_rescue(e, component: "cli", action: "setup_logger", fallback: "default_config", project_dir: project_dir)
+        # If logging setup fails, continue with default logger
+        Aidp.setup_logger(project_dir, {})
+        Aidp.logger.warn("cli", "Failed to load logging config, using defaults", error: e.message)
+      end
+
       def parse_options(args)
         options = {}
 
@@ -699,6 +730,7 @@ module Aidp
         table = TTY::Table.new header, table_rows
         display_message(table.render(:basic), type: :info)
       rescue => e
+        log_rescue(e, component: "cli", action: "display_provider_health", fallback: "error_message")
         display_message("Failed to display provider health: #{e.message}", type: :error)
       end
 
@@ -1049,6 +1081,7 @@ module Aidp
         )
         runner.start
       rescue ArgumentError => e
+        log_rescue(e, component: "cli", action: "start_watch_command", fallback: "error_display")
         display_message("❌ #{e.message}", type: :error)
       end
 

data/lib/aidp/debug_mixin.rb

@@ -1,7 +1,5 @@
 # frozen_string_literal: true
 
-require_relative "debug_logger"
-
 module Aidp
   # Mixin module for easy debug integration across the codebase
   module DebugMixin
@@ -27,7 +25,7 @@ module Aidp
 
     # Shared logger instance across all classes using DebugMixin
     def self.shared_logger
-      @shared_logger ||= Aidp::DebugLogger.new
+      Aidp.logger
     end
 
     # Instance-level debug methods
@@ -56,7 +54,8 @@ module Aidp
     def debug_log(message, level: :info, data: nil)
       return unless debug_enabled?
 
-      debug_logger.log(message, level: level, data: data)
+      debug_logger.log(level, component_name, message, **data) if data
+      debug_logger.log(level, component_name, message) unless data
     end
 
     # Log command execution with debug details
@@ -65,30 +64,30 @@ module Aidp
 
       command_str = [cmd, *args].join(" ")
 
-      debug_log("🔧 Executing command: #{command_str}", level: :info)
+      debug_logger.info(component_name, "🔧 Executing command: #{command_str}")
 
       if input
         if input.is_a?(String) && input.length > 200
           # If input is long, show first 100 chars and indicate it's truncated
-          debug_log("📝 Input (truncated): #{input[0..100]}...", level: :info)
+          debug_logger.info(component_name, "📝 Input (truncated): #{input[0..100]}...")
         elsif input.is_a?(String) && File.exist?(input)
-          debug_log("📝 Input file: #{input}", level: :info)
+          debug_logger.info(component_name, "📝 Input file: #{input}")
         else
-          debug_log("📝 Input: #{input}", level: :info)
+          debug_logger.info(component_name, "📝 Input: #{input}")
         end
       end
 
       if error && !error.empty?
-        debug_log("❌ Error output: #{error}", level: :error)
+        debug_logger.error(component_name, "❌ Error output: #{error}")
       end
 
       if debug_verbose?
         if output && !output.empty?
-          debug_log("📤 Output: #{output}", level: :info)
+          debug_logger.debug(component_name, "📤 Output: #{output}")
         end
 
         if exit_code
-          debug_log("🏁 Exit code: #{exit_code}", level: :info)
+          debug_logger.debug(component_name, "🏁 Exit code: #{exit_code}")
         end
       end
     end
@@ -98,25 +97,15 @@ module Aidp
       return unless debug_basic?
 
       message = "🔄 #{action}: #{step_name}"
-      if details.any?
-        detail_str = details.map { |k, v| "#{k}=#{v}" }.join(", ")
-        message += " (#{detail_str})"
-      end
-
-      debug_log(message, level: :info, data: details)
+      debug_logger.info(component_name, message, **details)
     end
 
     # Log provider interaction
     def debug_provider(provider_name, action, details = {})
       return unless debug_basic?
 
-      message = "🤖 #{provider_name}: #{action}"
-      if details.any?
-        detail_str = details.map { |k, v| "#{k}=#{v}" }.join(", ")
-        message += " (#{detail_str})"
-      end
-
-      debug_log(message, level: :info, data: details)
+      message = "🤖 #{action}"
+      debug_logger.info("provider_#{provider_name}", message, **details)
     end
 
     # Log error with debug context
@@ -124,10 +113,10 @@ module Aidp
       return unless debug_basic?
 
       error_message = "💥 Error: #{error.class.name}: #{error.message}"
-      debug_log(error_message, level: :error, data: {error: error, context: context})
+      debug_logger.error(component_name, error_message, error: error.class.name, **context)
 
       if debug_verbose? && error.backtrace
-        debug_log("📍 Backtrace: #{error.backtrace.first(5).join("\n")}", level: :error)
+        debug_logger.debug(component_name, "📍 Backtrace: #{error.backtrace.first(5).join("\n")}")
       end
     end
 
@@ -136,12 +125,7 @@ module Aidp
       return unless debug_verbose?
 
       message = "⏱️  #{operation}: #{duration.round(2)}s"
-      if details.any?
-        detail_str = details.map { |k, v| "#{k}=#{v}" }.join(", ")
-        message += " (#{detail_str})"
-      end
-
-      debug_log(message, level: :info, data: {duration: duration, details: details})
+      debug_logger.debug(component_name, message, duration: duration, **details)
     end
 
     # Execute command with debug logging
@@ -151,7 +135,7 @@ module Aidp
       command_str = [cmd, *args].join(" ")
       start_time = Time.now
 
-      debug_log("🚀 Starting command execution: #{command_str}", level: :info)
+      debug_logger.info(component_name, "🚀 Starting command execution: #{command_str}")
 
       begin
         # Configure printer based on streaming mode
@@ -190,5 +174,22 @@ module Aidp
         raise
       end
     end
+
+    private
+
+    # Safely derive a component name for logging (memoized).
+    # Handles anonymous classes and modules gracefully.
+    def component_name
+      @component_name ||= begin
+        klass = self.class
+        name = klass.name
+        return "anonymous" unless name && !name.empty?
+        # Take the last constant segment, normalize to snake-ish lowercase
+        segment = name.split("::").last
+        segment.gsub(/([a-z\d])([A-Z])/, '\\1_\\2').downcase
+      rescue
+        "anonymous"
+      end
+    end
   end
 end

data/lib/aidp/execute/async_work_loop_runner.rb

@@ -3,6 +3,7 @@
 require_relative "work_loop_runner"
 require_relative "work_loop_state"
 require_relative "instruction_queue"
+require_relative "../rescue_logging"
 
 module Aidp
   module Execute
@@ -16,6 +17,8 @@ module Aidp
     # - Stream output to main thread for display
     # - Handle graceful cancellation with checkpoint save
     class AsyncWorkLoopRunner
+      include Aidp::RescueLogging
+
       attr_reader :state, :instruction_queue, :work_thread
 
       def initialize(project_dir, provider_manager, config, options = {})
@@ -42,6 +45,7 @@ module Aidp
         @work_thread = Thread.new do
           run_async_loop
         rescue => e
+          log_rescue(e, component: "async_work_loop_runner", action: "thread_execution", fallback: "error_state", step: @step_name)
           @state.error!(e)
           @state.append_output("Work loop error: #{e.message}", type: :error)
         ensure
@@ -149,6 +153,7 @@ module Aidp
 
         result
       rescue => e
+        log_rescue(e, component: "async_work_loop_runner", action: "run_async_loop", fallback: "error_state", step: @step_name, iteration: @state.iteration)
         @state.error!(e)
         @state.append_output("Error in work loop: #{e.message}\n#{e.backtrace.first(3).join("\n")}", type: :error)
         raise

data/lib/aidp/execute/checkpoint.rb

@@ -3,12 +3,15 @@
 require "yaml"
 require "time"
 require "json"
+require "aidp/rescue_logging"
 
 module Aidp
   module Execute
     # Manages periodic checkpoints during work loop execution
     # Tracks progress metrics, code quality, and task completion
     class Checkpoint
+      include Aidp::RescueLogging
+
       attr_reader :project_dir, :checkpoint_file, :history_file
 
       def initialize(project_dir)
@@ -104,7 +107,11 @@ module Aidp
         end
 
         total_lines
-      rescue
+      rescue => e
+        log_rescue(e,
+          component: "checkpoint",
+          action: "count_lines_of_code",
+          fallback: 0)
         0
       end
 
@@ -121,7 +128,11 @@ module Aidp
         end
 
         count
-      rescue
+      rescue => e
+        log_rescue(e,
+          component: "checkpoint",
+          action: "count_project_files",
+          fallback: 0)
         0
       end
 
@@ -134,7 +145,11 @@ module Aidp
 
         coverage_ratio = (test_files.to_f / source_files * 100).round(2)
         [coverage_ratio, 100].min
-      rescue
+      rescue => e
+        log_rescue(e,
+          component: "checkpoint",
+          action: "estimate_test_coverage",
+          fallback: 0)
         0
       end
 
@@ -173,7 +188,11 @@ module Aidp
         offense_count = data["summary"]["offense_count"] || 0
         # Simple scoring: fewer offenses = higher score
         [100 - (offense_count / total_files.to_f * 10), 0].max.round(2)
-      rescue
+      rescue => e
+        log_rescue(e,
+          component: "checkpoint",
+          action: "run_rubocop_check",
+          fallback: nil)
         nil
       end
 
@@ -191,7 +210,11 @@ module Aidp
         return 0 if total_tasks == 0
 
         (completed_tasks.to_f / total_tasks * 100).round(2)
-      rescue
+      rescue => e
+        log_rescue(e,
+          component: "checkpoint",
+          action: "calculate_prd_task_progress",
+          fallback: 0)
         0
       end
 

data/lib/aidp/execute/interactive_repl.rb

@@ -4,6 +4,7 @@ require "tty-prompt"
 require "tty-spinner"
 require_relative "async_work_loop_runner"
 require_relative "repl_macros"
+require_relative "../rescue_logging"
 
 module Aidp
   module Execute
@@ -19,6 +20,8 @@ module Aidp
     #   repl = InteractiveRepl.new(project_dir, provider_manager, config)
     #   repl.start_work_loop(step_name, step_spec, context)
     class InteractiveRepl
+      include Aidp::RescueLogging
+
       def initialize(project_dir, provider_manager, config, options = {})
         @project_dir = project_dir
         @provider_manager = provider_manager
@@ -85,6 +88,7 @@ module Aidp
           rescue Interrupt
             handle_interrupt
           rescue => e
+            log_rescue(e, component: "interactive_repl", action: "repl_command_loop", fallback: "error_display")
             @prompt.error("REPL error: #{e.message}")
           end
         end
@@ -98,6 +102,7 @@ module Aidp
         command = $stdin.gets&.chomp
         command&.strip
       rescue => e
+        log_rescue(e, component: "interactive_repl", action: "read_command", fallback: nil)
         @prompt.error("Input error: #{e.message}")
         nil
       end
@@ -165,6 +170,7 @@ module Aidp
           @prompt.error(result[:message])
         end
       rescue => e
+        log_rescue(e, component: "interactive_repl", action: "handle_command", fallback: "error_display", command: result[:data])
         @prompt.error("Command error: #{e.message}")
       end
 
@@ -215,6 +221,7 @@ module Aidp
           message: success ? "Reset #{count} commit(s)" : output
         }
       rescue => e
+        log_rescue(e, component: "interactive_repl", action: "git_reset", fallback: "error_result", count: count)
         {success: false, message: e.message}
       end
 

data/lib/aidp/execute/work_loop_runner.rb

@@ -63,6 +63,8 @@ module Aidp
         @iteration_count = 0
         transition_to(:ready)
 
+        Aidp.logger.info("work_loop", "Starting fix-forward execution", step: step_name, max_iterations: MAX_ITERATIONS)
+
         display_message("🔄 Starting fix-forward work loop for step: #{step_name}", type: :info)
         display_message("  State machine: READY → APPLY_PATCH → TEST → {PASS → DONE | FAIL → DIAGNOSE → NEXT_PATCH}", type: :info)
 
@@ -77,7 +79,10 @@ module Aidp
           @iteration_count += 1
           display_message("  Iteration #{@iteration_count} [State: #{STATES[@current_state]}]", type: :info)
 
-          break if @iteration_count > MAX_ITERATIONS
+          if @iteration_count > MAX_ITERATIONS
+            Aidp.logger.error("work_loop", "Max iterations exceeded", step: @step_name, iterations: @iteration_count)
+            break
+          end
 
           # State: READY - Starting new iteration
           transition_to(:ready) unless @current_state == :ready
@@ -143,6 +148,7 @@ module Aidp
       def transition_to(new_state)
         raise "Invalid state: #{new_state}" unless STATES.key?(new_state)
 
+        old_state = @current_state
         @state_history << {
           from: @current_state,
           to: new_state,
@@ -150,6 +156,7 @@ module Aidp
           timestamp: Time.now
         }
         @current_state = new_state
+        Aidp.logger.debug("work_loop", "State transition", from: old_state, to: new_state, iteration: @iteration_count, step: @step_name)
       end
 
       # Display summary of state transitions

data/lib/aidp/harness/enhanced_runner.rb

@@ -81,6 +81,8 @@ module Aidp
         @state = STATES[:running]
         @start_time = Time.now
 
+        Aidp.logger.info("harness_runner", "Starting harness execution", mode: @mode, workflow_type: @workflow_type, steps_count: @selected_steps.size)
+
         @tui.show_message("🚀 Starting #{@mode.to_s.capitalize} Mode", :info)
 
         begin

data/lib/aidp/harness/provider_info.rb

@@ -3,11 +3,14 @@
 require "json"
 require "yaml"
 require "fileutils"
+require_relative "../rescue_logging"
 
 module Aidp
   module Harness
     # Stores detailed information about AI providers gathered from their CLI tools
     class ProviderInfo
+      include Aidp::RescueLogging
+
       attr_reader :provider_name, :info_file_path
 
       def initialize(provider_name, root_dir = nil)
@@ -56,6 +59,7 @@ module Aidp
 
         YAML.safe_load_file(@info_file_path, permitted_classes: [Time, Symbol])
       rescue => e
+        log_rescue(e, component: "provider_info", action: "load_yaml", fallback: nil, provider: @provider_name, path: @info_file_path)
         warn "Failed to load provider info for #{@provider_name}: #{e.message}"
         nil
       end
@@ -133,7 +137,8 @@ module Aidp
 
         last_checked = Time.parse(info[:last_checked].to_s)
         (Time.now - last_checked) > max_age
-      rescue
+      rescue => e
+        log_rescue(e, component: "provider_info", action: "parse_last_checked_time", fallback: true, provider: @provider_name, timestamp: info[:last_checked])
         true
       end
 
@@ -147,6 +152,7 @@ module Aidp
 
         provider_instance.fetch_mcp_servers
       rescue => e
+        log_rescue(e, component: "provider_info", action: "fetch_mcp_servers", fallback: [], provider: @provider_name)
         warn "Failed to fetch MCP servers for #{@provider_name}: #{e.message}" if ENV["AIDP_DEBUG"]
         []
       end
@@ -157,7 +163,8 @@ module Aidp
         # Try to find the binary
         path = begin
           Aidp::Util.which(binary_name)
-        rescue
+        rescue => e
+          log_rescue(e, component: "provider_info", action: "locate_provider_binary", fallback: nil, provider: @provider_name, binary: binary_name)
           nil
         end
         return nil unless path
@@ -183,7 +190,8 @@ module Aidp
               Process.kill("TERM", pid)
               sleep 0.1
               Process.kill("KILL", pid)
-            rescue
+            rescue => e
+              log_rescue(e, component: "provider_info", action: "kill_timeout_provider_command", fallback: nil, provider: @provider_name, binary: binary_name, pid: pid)
               nil
             end
             return nil
@@ -192,7 +200,8 @@ module Aidp
           output = r.read
           r.close
           output
-        rescue
+        rescue => e
+          log_rescue(e, component: "provider_info", action: "execute_provider_command", fallback: nil, provider: @provider_name, binary: binary_name, args: args)
           nil
         end
       end
@@ -355,6 +364,7 @@ module Aidp
         # Create provider instance
         @provider_instance = provider_class.new
       rescue => e
+        log_rescue(e, component: "provider_info", action: "create_provider_instance", fallback: nil, provider: @provider_name)
         warn "Failed to create provider instance for #{@provider_name}: #{e.message}" if ENV["AIDP_DEBUG"]
         nil
       end