aidp 0.14.1 → 0.15.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 8d97cff94f784545b96549c70d910e913bf19a7c94b908e45dae2a999c57e45b
-  data.tar.gz: 170bf8fe45dbb081cad38ef69eb8a0f58b962d495ed300b3d58d3f78ef553462
+  metadata.gz: 9a7dbc13b3136399f56fc64c00753506be8cbb35d5d3bf0f932799cc12a10504
+  data.tar.gz: 20afb73742ef4379f51fc05a9ae9cfb71ec95269856d79da0c0ce48c07c0aa39
 SHA512:
-  metadata.gz: a371614e261335e71b6e78cbe6d6410f26f03a4910e37e6b4102d3681fc9ef2abf617a8f4199b918f8efcdb43fc978a3eef9c1d04aa485297af90394bd34cbd8
-  data.tar.gz: 549e21d089121d0212698307bd8ed520499452ad829ff70e8ca1dea7fd50b6c63db9be2a55d4b56c9c60d822703710e84f57f91354bad8892e7367347e6b7777
+  metadata.gz: 41837e458506b1dc31724b611d46792d446da2f9b0303a37ded28fddbb8f29cfbaeeb417fddc24a3f34a0ed40f5ef3b9d96bf6c827c548be92436ca5ead751c2
+  data.tar.gz: fd57d77e5f8a85c38b3343c4db7a96ec7764066c420e2810d650ef97b51b4d47461c6779d3d9d856baa6730e15a204708b0c0b151d2bc4c00b71d86df68f988d

data/README.md

@@ -20,11 +20,8 @@ aidp config --interactive
 aidp init
 # Creates LLM_STYLE_GUIDE.md, PROJECT_ANALYSIS.md, CODE_QUALITY_PLAN.md
 
-# Start an interactive workflow
-aidp execute
-
-# Or run in background
-aidp execute --background
+# Start Copilot interactive mode (default)
+aidp
 ```
 
 ### First-Time Setup
@@ -58,16 +55,12 @@ AIDP implements **work loops** - an iterative execution pattern where AI agents
 
 See [Work Loops Guide](docs/WORK_LOOPS_GUIDE.md) for details.
 
-### Background Execution
+### Job Management
 
-Run workflows in the background while monitoring progress from separate terminals:
+Monitor and control background jobs:
 
 ```bash
-# Start in background
-aidp execute --background
-✓ Started background job: 20251005_235912_a1b2c3d4
-
-# Monitor progress
+# List and manage jobs
 aidp jobs list                        # List all jobs
 aidp jobs status <job_id>             # Show job status
 aidp jobs logs <job_id> --tail        # View recent logs
@@ -102,17 +95,14 @@ aidp checkpoint history 20
 
 ## Command Reference
 
-### Execution Modes
+### Copilot Mode
 
 ```bash
-# Execute mode - Build new features
-aidp execute                    # Interactive workflow selection
-aidp execute --background       # Run in background
-aidp execute --background --follow  # Start and follow logs
-
-# Analyze mode - Analyze codebase
-aidp analyze                    # Interactive analysis
-aidp analyze --background       # Background analysis
+# Start interactive Copilot mode (default)
+aidp                            # AI-guided workflow selection
+
+# Copilot can perform both analysis and development based on your needs
+# It will interactively help you choose the right approach
 ```
 
 ### Job Management
@@ -207,6 +197,21 @@ harness:
       - "bundle exec rspec"
     lint_commands:
       - "bundle exec standardrb"
+    units:
+      deterministic:
+        - name: run_full_tests
+          command: "bundle exec rake spec"
+          enabled: false
+          next:
+            success: agentic
+            failure: decide_whats_next
+        - name: wait_for_github
+          type: "wait"
+          metadata:
+            interval_seconds: 60
+      defaults:
+        on_no_next_step: wait_for_github
+        fallback_agentic: decide_whats_next
 
 providers:
   claude:
@@ -329,40 +334,36 @@ The questions file is only created when the AI needs additional information beyo
 ### Standard Interactive Workflow
 
 ```bash
-# Start execute mode
-aidp execute
-
-# Select workflow type (e.g., "Full PRD to Implementation")
-# Answer any questions interactively
-# Review generated files (PRD, architecture, etc.)
-# Workflow runs automatically with harness managing retries
+# Start Copilot mode (default)
+aidp
+
+# Copilot will guide you through:
+# - Understanding your project goals
+# - Selecting the right workflow (analysis, development, or both)
+# - Answering questions about your requirements
+# - Reviewing generated files (PRD, architecture, etc.)
+# - Automatic execution with harness managing retries
 ```
 
-### Background Workflow with Monitoring
+### Project Analysis
 
 ```bash
-# Terminal 1: Start background execution
-aidp execute --background
-✓ Started background job: 20251005_235912_a1b2c3d4
-
-# Terminal 2: Watch progress in real-time
-aidp checkpoint summary --watch
-
-# Terminal 3: Monitor job status
-aidp jobs status 20251005_235912_a1b2c3d4 --follow
+# High-level project analysis and documentation
+aidp init
 
-# Later: Check final results
-aidp checkpoint summary
-aidp jobs logs 20251005_235912_a1b2c3d4 --tail
+# Creates:
+# - LLM_STYLE_GUIDE.md
+# - PROJECT_ANALYSIS.md
+# - CODE_QUALITY_PLAN.md
 ```
 
-### Quick Analysis
+### Progress Monitoring
 
 ```bash
-# Run analysis in background
-aidp analyze --background
+# Watch progress in real-time
+aidp checkpoint summary --watch
 
-# Check progress
+# Check job status
 aidp jobs list
 aidp checkpoint summary
 ```

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)
@@ -189,15 +196,15 @@ module Aidp
         tui.start_display_loop
 
         begin
-          # First question: Choose mode
-          mode = select_mode_interactive(tui)
+          # Copilot is now the default mode - no menu selection
+          # The guided workflow selector will internally choose appropriate mode
+          mode = :guided
 
           # Get workflow configuration (no spinner - may wait for user input)
           workflow_config = workflow_selector.select_workflow(harness_mode: false, mode: mode)
 
-          # For guided mode, use the mode determined by the guided workflow selector
-          # Otherwise use the initially selected mode
-          actual_mode = workflow_config[:mode] || mode
+          # Use the mode determined by the guided workflow selector
+          actual_mode = workflow_config[:mode] || :execute
 
           # Pass workflow configuration to harness
           harness_options = {
@@ -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 = {}
 
@@ -234,8 +265,7 @@ module Aidp
           opts.separator "AI Development Pipeline - Autonomous development workflow automation"
           opts.separator ""
           opts.separator "Commands:"
-          opts.separator "  analyze [--background]   Start analyze mode workflow"
-          opts.separator "  execute [--background]   Start execute mode workflow"
+          opts.separator "  (no command)             Start Copilot interactive mode (default)"
           opts.separator "  init                     Analyse project and bootstrap quality docs"
           opts.separator "  watch <issues_url>       Run fully automatic watch mode"
           opts.separator "  status                   Show current system status"
@@ -271,9 +301,12 @@ module Aidp
 
           opts.separator ""
           opts.separator "Examples:"
-          opts.separator "  # Start background execution"
-          opts.separator "  aidp execute --background"
-          opts.separator "  aidp execute --background --follow    # Start and follow logs"
+          opts.separator "  # Start interactive Copilot mode"
+          opts.separator "  aidp"
+          opts.separator ""
+          opts.separator "  # Project bootstrap"
+          opts.separator "  aidp init                             # High-level analysis and docs"
+          opts.separator "  aidp config --interactive             # Configure providers"
           opts.separator ""
           opts.separator "  # Monitor background jobs"
           opts.separator "  aidp jobs list                        # List all jobs"
@@ -284,20 +317,14 @@ module Aidp
           opts.separator "  aidp checkpoint summary --watch       # Auto-refresh every 5s"
           opts.separator "  aidp checkpoint summary --watch --interval 10"
           opts.separator ""
-          opts.separator "  # Project bootstrap"
-          opts.separator "  aidp init"
-          opts.separator "  aidp config --interactive"
           opts.separator "  # Fully automatic orchestration"
-          opts.separator ""
           opts.separator "  aidp watch https://github.com/<org>/<repo>/issues"
           opts.separator "  aidp watch owner/repo --interval 120 --provider claude"
           opts.separator ""
           opts.separator "  # Other commands"
           opts.separator "  aidp providers                        # Check provider health"
           opts.separator "  aidp providers info claude            # Show detailed provider info"
-          opts.separator "  aidp providers refresh                # Refresh all provider info"
           opts.separator "  aidp mcp                              # Show MCP server dashboard"
-          opts.separator "  aidp mcp check dash-api filesystem    # Check provider eligibility"
           opts.separator "  aidp checkpoint history 20            # Show last 20 checkpoints"
           opts.separator ""
           opts.separator "For more information, visit: https://github.com/viamin/aidp"
@@ -311,7 +338,7 @@ module Aidp
       # Determine if the invocation is a subcommand style call
       def subcommand?(args)
         return false if args.nil? || args.empty?
-        %w[status jobs kb harness execute analyze providers checkpoint mcp issue config init watch].include?(args.first)
+        %w[status jobs kb harness providers checkpoint mcp issue config init watch].include?(args.first)
       end
 
       def run_subcommand(args)
@@ -321,8 +348,6 @@ module Aidp
         when "jobs" then run_jobs_command(args)
         when "kb" then run_kb_command(args)
         when "harness" then run_harness_command(args)
-        when "execute" then run_execute_command(args)
-        when "analyze" then run_execute_command(args, mode: :analyze) # symmetry
         when "providers" then run_providers_command(args)
         when "checkpoint" then run_checkpoint_command(args)
         when "mcp" then run_mcp_command(args)
@@ -699,6 +724,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
 
@@ -891,25 +917,6 @@ module Aidp
         mode&.to_sym
       end
 
-      def select_mode_interactive(tui)
-        mode_options = [
-          "🤖 Guided Workflow (Copilot) - AI helps you choose the right workflow",
-          "🔬 Analyze Mode - Analyze your codebase for insights and recommendations",
-          "🏗️ Execute Mode - Build new features with guided development workflow"
-        ]
-        selected = tui.single_select("Welcome to AI Dev Pipeline! Choose your mode", mode_options, default: 1)
-        # Announce mode explicitly in headless contexts (handled internally otherwise)
-        if (defined?(RSpec) || ENV["RSPEC_RUNNING"]) && tui.respond_to?(:announce_mode)
-          tui.announce_mode(:guided) if selected == mode_options[0]
-          tui.announce_mode(:analyze) if selected == mode_options[1]
-          tui.announce_mode(:execute) if selected == mode_options[2]
-        end
-        return :guided if selected == mode_options[0]
-        return :analyze if selected == mode_options[1]
-        return :execute if selected == mode_options[2]
-        :analyze
-      end
-
       def display_harness_result(result)
         case result[:status]
         when "completed"
@@ -1049,6 +1056,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/agent_signal_parser.rb

@@ -0,0 +1,46 @@
+# frozen_string_literal: true
+
+module Aidp
+  module Execute
+    module AgentSignalParser
+      def self.extract_next_unit(output)
+        return nil unless output
+
+        output.to_s.each_line do |line|
+          token = token_from_line(line)
+          next unless token
+
+          return normalize_token(token)
+        end
+
+        nil
+      end
+
+      def self.normalize_token(raw)
+        return nil if raw.nil? || raw.empty?
+
+        token = raw.downcase.strip
+        token.gsub!(/\s+/, "_")
+        token.to_sym
+      end
+
+      def self.token_from_line(line)
+        return nil unless line
+
+        trimmed = line.lstrip
+        separator_index = trimmed.index(":") || trimmed.index("=")
+        return nil unless separator_index
+
+        key = trimmed[0...separator_index].strip
+        value = trimmed[(separator_index + 1)..]&.strip
+
+        return nil unless key && value
+        return value if key.casecmp("next_unit").zero? || key.casecmp("next_step").zero?
+
+        nil
+      end
+
+      private_class_method :token_from_line
+    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