aidp 0.16.0 → 0.17.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 9bbab73392b0bc0fdcd74bb0f0e0bbf09716b2d2451b04aaa64c4ca4ccc7638e
-  data.tar.gz: af0d8b6d8937a1781b3b379ab6328ff17c5f0c96ea00c80f5886ce15b2265e2f
+  metadata.gz: 949670fdea4721406643f4fd8da096e943740effbd836407311b812350919b14
+  data.tar.gz: 8034a3c798571e4626b273c4a9abe9a5da38cef5dd4f25de8794a0ef63bf6022
 SHA512:
-  metadata.gz: ee40afa8126a8f8d6f92a7df6fbc634c6eb054499bb7dfdec53946f05ecf8faf8c1bcae1eb02274a7441bdd7685751363e20207587df818c8bb8fe300bfbc50f
-  data.tar.gz: 010a5afdd1a06905f09a423efe3bfde2ef84aca498f939667871191b1c9174168c31807959cbc3d50e7f7ab2cc552301032c1d232f6e70331ee8918debb7aba5
+  metadata.gz: cdabc6ba04d06a89dbc4ad19b27a64862e960daa350029db4fada5a4f6d32d3fba27cb32dda291871a79ad8f319d27e7dd1348ddcb7fdb8b0f0a1390b9493fc1
+  data.tar.gz: 39c384f4c01bf00bef550ac9190d27ddd268437be92fc4cdfcdeac3dff09c79c055b2e5b5582e553af6fa838b3f904cce354e9db62ee3638f658eec05673e4f5

data/lib/aidp/analyze/error_handler.rb

@@ -3,6 +3,18 @@
 require "logger"
 require_relative "../concurrency"
 
+begin
+  require "net/http"
+rescue LoadError
+  # Net::HTTP might not be available in all environments
+end
+
+begin
+  require "sqlite3"
+rescue LoadError
+  # SQLite3 might not be available in all environments
+end
+
 module Aidp
   module Analyze
     # Comprehensive error handling system for analyze mode
@@ -24,7 +36,7 @@ module Aidp
           context: context,
           step: step,
           retry_count: retry_count,
-          timestamp: Time.current
+          timestamp: Time.now
         }
 
         log_error(error_info)
@@ -59,7 +71,7 @@ module Aidp
         {
           status: "skipped",
           reason: error.message,
-          timestamp: Time.current
+          timestamp: Time.now
         }
       end
 
@@ -88,8 +100,8 @@ module Aidp
 
       def setup_logger(log_file, verbose)
         output_stream = log_file || @output || $stdout
-        logger = Logger.new(output_stream)
-        logger.level = verbose ? Logger::DEBUG : Logger::INFO
+        logger = ::Logger.new(output_stream)
+        logger.level = verbose ? ::Logger::DEBUG : ::Logger::INFO
         logger.formatter = proc do |severity, datetime, progname, msg|
           "#{datetime.strftime("%Y-%m-%d %H:%M:%S")} [#{severity}] #{msg}\n"
         end
@@ -97,19 +109,28 @@ module Aidp
       end
 
       def setup_recovery_strategies
-        {
-          Net::TimeoutError => :retry_with_backoff,
-          Net::HTTPError => :retry_with_backoff,
-          SocketError => :retry_with_backoff,
+        strategies = {
           Errno::ENOENT => :skip_step_with_warning,
           Errno::EACCES => :skip_step_with_warning,
           Errno::ENOSPC => :critical_error,
-          SQLite3::BusyException => :retry_with_backoff,
-          SQLite3::CorruptException => :critical_error,
           AnalysisTimeoutError => :chunk_and_retry,
           AnalysisDataError => :continue_with_partial_data,
           AnalysisToolError => :log_and_continue
         }
+
+        # Add network error classes if available
+        strategies[Net::TimeoutError] = :retry_with_backoff if defined?(Net::TimeoutError)
+
+        strategies[Net::HTTPError] = :retry_with_backoff if defined?(Net::HTTPError)
+
+        strategies[SocketError] = :retry_with_backoff if defined?(SocketError)
+
+        # Add SQLite error classes if available
+        strategies[SQLite3::BusyException] = :retry_with_backoff if defined?(SQLite3::BusyException)
+
+        strategies[SQLite3::CorruptException] = :critical_error if defined?(SQLite3::CorruptException)
+
+        strategies
       end
 
       def log_error(error_info)
@@ -250,9 +271,7 @@ module Aidp
         tool_name = context[:tool_name] || "analysis tool"
         error_msg = "#{tool_name} failed: #{error.message}"
 
-        if context[:installation_guide]
-          error_msg += "\n\nTo install #{tool_name}:\n#{context[:installation_guide]}"
-        end
+        error_msg += "\n\nTo install #{tool_name}:\n#{context[:installation_guide]}" if context[:installation_guide]
 
         raise AnalysisToolError.new(error_msg)
       end

data/lib/aidp/analyze/kb_inspector.rb

@@ -80,7 +80,7 @@ module Aidp
         display_message(box)
       end
 
-      def load_kb_data
+      def load_kb_data(suppress_parse_warnings: false)
         data = {}
 
         %w[symbols imports calls metrics seams hotspots tests cycles].each do |type|
@@ -89,8 +89,7 @@ module Aidp
             begin
               data[type.to_sym] = JSON.parse(File.read(file_path), symbolize_names: true)
             rescue JSON::ParserError => e
-              # Suppress warnings in test mode to avoid CI failures
-              unless ENV["RACK_ENV"] == "test" || defined?(RSpec)
+              unless suppress_parse_warnings
                 display_message("Warning: Could not parse #{file_path}: #{e.message}", type: :warn)
               end
               data[type.to_sym] = []

data/lib/aidp/analyze/progress.rb

@@ -9,9 +9,10 @@ module Aidp
     class Progress
       attr_reader :project_dir, :progress_file
 
-      def initialize(project_dir)
+      def initialize(project_dir, skip_persistence: false)
         @project_dir = project_dir
         @progress_file = File.join(project_dir, ".aidp", "progress", "analyze.yml")
+        @skip_persistence = skip_persistence
         load_progress
       end
 
@@ -60,26 +61,20 @@ module Aidp
       private
 
       def load_progress
-        # In test mode, only skip file operations if no progress file exists
-        if (ENV["RACK_ENV"] == "test" || defined?(RSpec)) && !File.exist?(@progress_file)
+        if @skip_persistence && !File.exist?(@progress_file)
           @progress = {}
           return
         end
-
-        @progress = if File.exist?(@progress_file)
-          YAML.load_file(@progress_file) || {}
+        @progress = if !@skip_persistence && File.exist?(@progress_file)
+          YAML.safe_load_file(@progress_file, permitted_classes: [Date, Time, Symbol], aliases: true) || {}
         else
           {}
         end
-
-        # Ensure @progress is never nil
         @progress = {} if @progress.nil?
       end
 
       def save_progress
-        # In test mode, skip file operations to avoid hanging
-        return if ENV["RACK_ENV"] == "test" || defined?(RSpec)
-
+        return if @skip_persistence
         FileUtils.mkdir_p(File.dirname(@progress_file))
         File.write(@progress_file, @progress.to_yaml)
       end

data/lib/aidp/cli/mcp_dashboard.rb

@@ -102,7 +102,7 @@ module Aidp
 
         providers.each do |provider|
           provider_info = Aidp::Harness::ProviderInfo.new(provider, @root_dir)
-          info = provider_info.get_info
+          info = provider_info.info
 
           next unless info[:mcp_support]
 

data/lib/aidp/cli.rb

@@ -147,6 +147,10 @@ module Aidp
     class << self
       extend Aidp::MessageDisplay::ClassMethods
 
+      def create_prompt
+        ::TTY::Prompt.new
+      end
+
       def run(args = ARGV)
         # Handle subcommands first (status, jobs, kb, harness)
         return run_subcommand(args) if subcommand?(args)
@@ -173,7 +177,7 @@ module Aidp
 
         # Handle configuration setup
         # Create a prompt for the wizard
-        prompt = TTY::Prompt.new
+        prompt = create_prompt
 
         if options[:setup_config]
           # Force setup/reconfigure even if config exists
@@ -241,7 +245,7 @@ module Aidp
 
         if File.exist?(config_path)
           require "yaml"
-          full_config = YAML.load_file(config_path)
+          full_config = YAML.safe_load_file(config_path, permitted_classes: [Date, Time, Symbol], aliases: true)
           logging_config = full_config["logging"] || full_config[:logging] || {}
         end
 
@@ -398,7 +402,7 @@ module Aidp
 
       def run_jobs_command(args = [])
         require_relative "cli/jobs_command"
-        jobs_cmd = Aidp::CLI::JobsCommand.new(prompt: TTY::Prompt.new)
+        jobs_cmd = Aidp::CLI::JobsCommand.new(prompt: create_prompt)
         subcommand = args.shift
         jobs_cmd.run(subcommand, args)
       end
@@ -504,21 +508,6 @@ module Aidp
         if step
           display_message("Running #{mode} step '#{step}' with enhanced TUI harness", type: :highlight)
           display_message("progress indicators", type: :info)
-          if step.start_with?("00_PRD") && (defined?(RSpec) || ENV["RSPEC_RUNNING"])
-            # Simulate questions & completion similar to TUI test mode
-            root = ENV["AIDP_ROOT"] || Dir.pwd
-            file = Dir.glob(File.join(root, "templates", (mode == :execute) ? "EXECUTE" : "ANALYZE", "00_PRD*.md")).first
-            if file && File.file?(file)
-              content = File.read(file)
-              questions_section = content.split(/## Questions/i)[1]
-              if questions_section
-                questions_section.lines.select { |l| l.strip.start_with?("-") }.each do |line|
-                  display_message(line.strip.sub(/^-\s*/, ""), type: :info)
-                end
-              end
-            end
-            display_message("PRD completed", type: :success)
-          end
           return
         end
         display_message("Starting enhanced TUI harness", type: :highlight)
@@ -601,7 +590,7 @@ module Aidp
         when "clear"
           force = args.include?("--force")
           unless force
-            prompt = TTY::Prompt.new
+            prompt = create_prompt
             confirm = prompt.yes?("Are you sure you want to clear all checkpoint data?")
             return unless confirm
           end
@@ -696,7 +685,7 @@ module Aidp
           end
         end
         config_manager = Aidp::Harness::ConfigManager.new(Dir.pwd)
-        pm = Aidp::Harness::ProviderManager.new(config_manager, prompt: TTY::Prompt.new)
+        pm = Aidp::Harness::ProviderManager.new(config_manager, prompt: create_prompt)
 
         # Use TTY::Spinner for progress indication
         require "tty-spinner"
@@ -738,7 +727,12 @@ module Aidp
           end
           tokens = (r[:total_tokens].to_i > 0) ? r[:total_tokens].to_s : "0"
           reason = r[:unhealthy_reason] || "-"
-          if no_color || !$stdout.tty?
+          is_tty = begin
+            $stdout.respond_to?(:tty?) && $stdout.tty?
+          rescue
+            false
+          end
+          if no_color || !is_tty
             [r[:provider], r[:status], (r[:available] ? "yes" : "no"), cb, rl, tokens, last_used, reason]
           else
             [
@@ -757,7 +751,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")
+        Aidp.logger.warn("cli", "Failed to display provider health", error_class: e.class.name, error_message: e.message)
         display_message("Failed to display provider health: #{e.message}", type: :error)
       end
 
@@ -777,7 +771,7 @@ module Aidp
         display_message("=" * 60, type: :muted)
 
         provider_info = Aidp::Harness::ProviderInfo.new(provider_name, Dir.pwd)
-        info = provider_info.get_info(force_refresh: force_refresh)
+        info = provider_info.info(force_refresh: force_refresh)
 
         if info.nil?
           display_message("No information available for provider: #{provider_name}", type: :error)
@@ -1044,7 +1038,7 @@ module Aidp
           return
         end
 
-        wizard = Aidp::Setup::Wizard.new(Dir.pwd, prompt: TTY::Prompt.new, dry_run: dry_run)
+        wizard = Aidp::Setup::Wizard.new(Dir.pwd, prompt: create_prompt, dry_run: dry_run)
         wizard.run
       end
 
@@ -1071,7 +1065,7 @@ module Aidp
         end
 
         require_relative "init/runner"
-        runner = Aidp::Init::Runner.new(Dir.pwd, prompt: TTY::Prompt.new, options: options)
+        runner = Aidp::Init::Runner.new(Dir.pwd, prompt: create_prompt, options: options)
         runner.run
       end
 
@@ -1127,7 +1121,7 @@ module Aidp
           project_dir: Dir.pwd,
           once: once,
           use_workstreams: use_workstreams,
-          prompt: TTY::Prompt.new
+          prompt: create_prompt
         )
         runner.start
       rescue ArgumentError => e
@@ -1244,7 +1238,7 @@ module Aidp
 
           # Confirm removal unless --force
           unless force
-            prompt = TTY::Prompt.new
+            prompt = create_prompt
             confirm = prompt.yes?("Remove workstream '#{slug}'?#{" (will also delete branch)" if delete_branch}")
             return unless confirm
           end
@@ -1722,10 +1716,10 @@ module Aidp
 
             by_source = registry.by_source
 
-            if by_source[:builtin].any?
-              display_message("Built-in Skills", type: :highlight)
+            if by_source[:template].any?
+              display_message("Template Skills", type: :highlight)
               display_message("=" * 80, type: :muted)
-              table_rows = by_source[:builtin].map do |skill_id|
+              table_rows = by_source[:template].map do |skill_id|
                 skill = registry.find(skill_id)
                 [skill_id, skill.version, skill.description[0, 60]]
               end
@@ -1735,10 +1729,10 @@ module Aidp
               display_message("", type: :info)
             end
 
-            if by_source[:custom].any?
-              display_message("Custom Skills", type: :highlight)
+            if by_source[:project].any?
+              display_message("Project Skills", type: :highlight)
               display_message("=" * 80, type: :muted)
-              table_rows = by_source[:custom].map do |skill_id|
+              table_rows = by_source[:project].map do |skill_id|
                 skill = registry.find(skill_id)
                 [skill_id, skill.version, skill.description[0, 60]]
               end
@@ -1846,6 +1840,204 @@ module Aidp
             display_message("Failed to search skills: #{e.message}", type: :error)
           end
 
+        when "preview"
+          # Preview full skill content
+          skill_id = args.shift
+
+          unless skill_id
+            display_message("Usage: aidp skill preview <skill-id>", type: :info)
+            return
+          end
+
+          begin
+            registry = Aidp::Skills::Registry.new(project_dir: Dir.pwd)
+            registry.load_skills
+
+            skill = registry.find(skill_id)
+
+            unless skill
+              display_message("Skill not found: #{skill_id}", type: :error)
+              display_message("Use 'aidp skill list' to see available skills", type: :muted)
+              return
+            end
+
+            require_relative "skills/wizard/builder"
+            require_relative "skills/wizard/template_library"
+
+            builder = Aidp::Skills::Wizard::Builder.new
+            full_content = builder.to_skill_md(skill)
+
+            # Check if this is a project skill with a matching template
+            source_info = registry.by_source[skill_id]
+            inheritance_info = ""
+            if source_info == :project
+              template_library = Aidp::Skills::Wizard::TemplateLibrary.new(project_dir: Dir.pwd)
+              template_skill = template_library.templates.find { |s| s.id == skill.id }
+              if template_skill
+                inheritance_info = " (inherits from template)"
+              end
+            elsif source_info == :template
+              inheritance_info = " (template)"
+            end
+
+            display_message("\n" + "=" * 60, type: :info)
+            display_message("Skill: #{skill.name} (#{skill.id}) v#{skill.version}#{inheritance_info}", type: :highlight)
+            display_message("=" * 60 + "\n", type: :info)
+            display_message(full_content, type: :info)
+            display_message("\n" + "=" * 60, type: :info)
+          rescue => e
+            display_message("Failed to preview skill: #{e.message}", type: :error)
+          end
+
+        when "diff"
+          # Show diff between project skill and template
+          skill_id = args.shift
+
+          unless skill_id
+            display_message("Usage: aidp skill diff <skill-id>", type: :info)
+            return
+          end
+
+          begin
+            require_relative "skills/wizard/template_library"
+            require_relative "skills/wizard/differ"
+
+            registry = Aidp::Skills::Registry.new(project_dir: Dir.pwd)
+            registry.load_skills
+
+            project_skill = registry.find(skill_id)
+
+            unless project_skill
+              display_message("Skill not found: #{skill_id}", type: :error)
+              return
+            end
+
+            # Check if it's a project skill
+            unless registry.by_source[:project].include?(skill_id)
+              display_message("Skill '#{skill_id}' is a template skill, not a project skill", type: :info)
+              display_message("Only project skills can be diffed against templates", type: :muted)
+              return
+            end
+
+            # Find the template
+            template_library = Aidp::Skills::Wizard::TemplateLibrary.new(project_dir: Dir.pwd)
+            template_skill = template_library.find(skill_id)
+
+            unless template_skill
+              display_message("No template found for skill '#{skill_id}'", type: :info)
+              display_message("This is a custom skill without a template base", type: :muted)
+              return
+            end
+
+            # Show diff
+            differ = Aidp::Skills::Wizard::Differ.new
+            diff_result = differ.diff(template_skill, project_skill)
+            differ.display(diff_result)
+          rescue => e
+            display_message("Failed to diff skill: #{e.message}", type: :error)
+          end
+
+        when "edit"
+          # Edit an existing skill
+          skill_id = args.shift
+
+          unless skill_id
+            display_message("Usage: aidp skill edit <skill-id>", type: :info)
+            return
+          end
+
+          begin
+            require_relative "skills/wizard/controller"
+
+            registry = Aidp::Skills::Registry.new(project_dir: Dir.pwd)
+            registry.load_skills
+
+            skill = registry.find(skill_id)
+
+            unless skill
+              display_message("Skill not found: #{skill_id}", type: :error)
+              display_message("Use 'aidp skill list' to see available skills", type: :muted)
+              return
+            end
+
+            # Check if it's editable (must be project skill or willing to copy template)
+            if registry.by_source[:template].include?(skill_id)
+              display_message("'#{skill_id}' is a template skill", type: :info)
+              display_message("Editing will create a project override in .aidp/skills/", type: :muted)
+            end
+
+            # Parse options
+            options = {}
+            while args.first&.start_with?("--")
+              opt = args.shift
+              case opt
+              when "--dry-run"
+                options[:dry_run] = true
+              when "--open-editor"
+                options[:open_editor] = true
+              else
+                display_message("Unknown option: #{opt}", type: :error)
+                return
+              end
+            end
+
+            # Pre-fill wizard with existing skill data
+            options[:id] = skill.id
+            options[:name] = skill.name
+            options[:edit_mode] = true
+            options[:existing_skill] = skill
+
+            # Run wizard in edit mode
+            wizard = Aidp::Skills::Wizard::Controller.new(
+              project_dir: Dir.pwd,
+              options: options
+            )
+            wizard.run
+          rescue => e
+            display_message("Failed to edit skill: #{e.message}", type: :error)
+          end
+
+        when "new"
+          # Create a new skill using the wizard
+          begin
+            require_relative "skills/wizard/controller"
+
+            # Parse options
+            options = {}
+            while args.first&.start_with?("--")
+              opt = args.shift
+              case opt
+              when "--minimal"
+                options[:minimal] = true
+              when "--dry-run"
+                options[:dry_run] = true
+              when "--yes", "-y"
+                options[:yes] = true
+              when "--id"
+                options[:id] = args.shift
+              when "--name"
+                options[:name] = args.shift
+              when "--from-template"
+                options[:from_template] = args.shift
+              when "--clone"
+                options[:clone] = args.shift
+              else
+                display_message("Unknown option: #{opt}", type: :error)
+                return
+              end
+            end
+
+            # Run wizard
+            wizard = Aidp::Skills::Wizard::Controller.new(
+              project_dir: Dir.pwd,
+              options: options
+            )
+            wizard.run
+          rescue => e
+            display_message("Failed to create skill: #{e.message}", type: :error)
+            Aidp.log_error("cli", "Skill wizard failed", error: e.message, backtrace: e.backtrace.first(5))
+          end
+
         when "validate"
           # Validate skill file format
           skill_path = args.shift
@@ -1893,19 +2085,94 @@ module Aidp
             end
           end
 
+        when "delete"
+          # Delete a project skill
+          skill_id = args.shift
+
+          unless skill_id
+            display_message("Usage: aidp skill delete <skill-id>", type: :info)
+            return
+          end
+
+          begin
+            registry = Aidp::Skills::Registry.new(project_dir: Dir.pwd)
+            registry.load_skills
+
+            skill = registry.find(skill_id)
+
+            unless skill
+              display_message("Skill not found: #{skill_id}", type: :error)
+              return
+            end
+
+            # Check if it's a project skill
+            source = registry.by_source[skill_id]
+            unless source == :project
+              display_message("Cannot delete template skill '#{skill_id}'", type: :error)
+              display_message("Only project skills in .aidp/skills/ can be deleted", type: :muted)
+              return
+            end
+
+            # Get skill directory
+            skill_dir = File.dirname(skill.source_path)
+
+            # Confirm deletion
+            require "tty-prompt"
+            prompt = create_prompt
+            confirmed = prompt.yes?("Delete skill '#{skill.name}' (#{skill_id})? This cannot be undone.")
+
+            unless confirmed
+              display_message("Deletion cancelled", type: :info)
+              return
+            end
+
+            # Delete the skill directory
+            require "fileutils"
+            FileUtils.rm_rf(skill_dir)
+
+            display_message("✓ Deleted skill: #{skill.name} (#{skill_id})", type: :success)
+          rescue => e
+            display_message("Failed to delete skill: #{e.message}", type: :error)
+            Aidp.log_error("cli", "Skill deletion failed", error: e.message, backtrace: e.backtrace.first(5))
+          end
+
         else
           display_message("Usage: aidp skill <command>", type: :info)
           display_message("", type: :info)
           display_message("Commands:", type: :info)
           display_message("  list                List all available skills (default)", type: :info)
           display_message("  show <id>           Show detailed skill information", type: :info)
+          display_message("  preview <id>        Preview full SKILL.md content", type: :info)
+          display_message("  diff <id>           Show diff between project skill and template", type: :info)
           display_message("  search <query>      Search skills by keyword", type: :info)
+          display_message("  new [options]       Create a new skill using the wizard", type: :info)
+          display_message("  edit <id> [options] Edit an existing skill", type: :info)
+          display_message("  delete <id>         Delete a project skill", type: :info)
           display_message("  validate [path]     Validate skill file format", type: :info)
           display_message("", type: :info)
+          display_message("New Skill Options:", type: :info)
+          display_message("  --minimal           Skip optional sections", type: :info)
+          display_message("  --dry-run           Preview without saving", type: :info)
+          display_message("  --yes, -y           Skip confirmation prompts", type: :info)
+          display_message("  --id <skill_id>     Pre-set skill ID", type: :info)
+          display_message("  --name <name>       Pre-set skill name", type: :info)
+          display_message("", type: :info)
+          display_message("Edit Skill Options:", type: :info)
+          display_message("  --dry-run           Preview changes without saving", type: :info)
+          display_message("  --open-editor       Open content in $EDITOR", type: :info)
+          display_message("", type: :info)
           display_message("Examples:", type: :info)
           display_message("  aidp skill list                                # List all skills", type: :info)
           display_message("  aidp skill show repository_analyst             # Show skill details", type: :info)
+          display_message("  aidp skill preview repository_analyst          # Preview full content", type: :info)
+          display_message("  aidp skill diff my_skill                       # Show diff with template", type: :info)
           display_message("  aidp skill search git                          # Search for git-related skills", type: :info)
+          display_message("  aidp skill new                                 # Create new skill (interactive)", type: :info)
+          display_message("  aidp skill new --minimal --id my_skill         # Create with minimal prompts", type: :info)
+          display_message("  aidp skill new --from-template repo_analyst    # Inherit from template", type: :info)
+          display_message("  aidp skill new --clone my_existing_skill       # Clone existing skill", type: :info)
+          display_message("  aidp skill edit repository_analyst             # Edit existing skill", type: :info)
+          display_message("  aidp skill delete my_custom_skill              # Delete a project skill", type: :info)
           display_message("  aidp skill validate skills/my_skill/SKILL.md   # Validate specific skill", type: :info)
           display_message("  aidp skill validate                            # Validate all skills", type: :info)
         end

data/lib/aidp/config.rb

@@ -312,7 +312,7 @@ module Aidp
     end
 
     private_class_method def self.load_yaml_config(config_file)
-      YAML.load_file(config_file) || {}
+      YAML.safe_load_file(config_file, permitted_classes: [Date, Time, Symbol], aliases: true) || {}
     rescue => e
       warn "Failed to load configuration file #{config_file}: #{e.message}"
       {}

data/lib/aidp/execute/async_work_loop_runner.rb

@@ -26,6 +26,7 @@ module Aidp
         @provider_manager = provider_manager
         @config = config
         @options = options
+        @cancel_timeout = options[:cancel_timeout] || 5 # seconds to wait for graceful shutdown
         @state = WorkLoopState.new
         @instruction_queue = InstructionQueue.new
         @work_thread = nil
@@ -92,7 +93,7 @@ module Aidp
         @state.append_output("Cancellation requested, waiting for safe stopping point...", type: :warning)
 
         # Wait for thread to notice cancellation
-        @work_thread&.join(5) # 5 second timeout
+        @work_thread&.join(@cancel_timeout)
 
         if save_checkpoint && @sync_runner
           @state.append_output("Saving checkpoint before exit...", type: :info)