aidp 0.15.2 → 0.17.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: '0691816c2748d6704d61bba9ed4c992f1b7d3ac8f70385ac94cb199916b71491'
-  data.tar.gz: e0a8b4a7c9ece98972565635894d7312c4253c2adb3341192c7622e8e28d3a12
+  metadata.gz: 84451cdcc0886f1a70e799ab53d9b8898633f324cdd80c5f07a5f204848a3de2
+  data.tar.gz: e93982382c700564db24fac6ff1de2276e5b0301b91f6cade19aa8dfacd3f7d3
 SHA512:
-  metadata.gz: 0d3ffce4b6aa8913658862e4412c165c4f20e8164845267d9d85c574b340d4140e63abbae359c47387255081c2738cbdaae215c159e1aeb4d36e7a84b5405dfc
-  data.tar.gz: d2a809ea1b79bbe97e8adfe90903ecb610851837d7b744da208c20e74574346e18e4e3a25ebb136f702fb2501cf5855f1aac0d70d31b6be618c0843a51d5d90f
+  metadata.gz: 3de783cb25c3eedc9056f77c2b294918a2f5ab46c009280897b6094dd157c45c681fc5c8fe22c6eea401ba74e0c4151c132a85e3c8ceedfef7722e9580f96f43
+  data.tar.gz: de86c77e08a8f9b837d501d8e69786b6f969c0e2a1538f1a9d2062267f01d6e8c855f0885c31d0bdc3f2c9362fa78439fccdcfe5e09a12cd1ab61282365cdc4a

data/README.md

@@ -93,6 +93,34 @@ aidp checkpoint history 20
 - PRD task completion percentage
 - File count and growth trends
 
+### Parallel Workstreams
+
+Work on multiple tasks simultaneously using isolated git worktrees:
+
+```bash
+# Create a workstream for each task
+aidp ws new issue-123-fix-auth
+aidp ws new feature-dashboard
+
+# List all workstreams
+aidp ws list
+
+# Check status
+aidp ws status issue-123-fix-auth
+
+# Remove when complete
+aidp ws rm issue-123-fix-auth --delete-branch
+```
+
+**Benefits:**
+
+- Complete isolation between tasks
+- Separate git branches for each workstream
+- Independent state and history
+- Work on multiple features in parallel
+
+See [Workstreams Guide](docs/WORKSTREAMS.md) for detailed usage.
+
 ## Command Reference
 
 ### Copilot Mode
@@ -147,6 +175,25 @@ aidp checkpoint clear
 aidp checkpoint clear --force     # Skip confirmation
 ```
 
+### Workstream Commands
+
+```bash
+# Create a new workstream
+aidp ws new <slug>
+aidp ws new <slug> --base-branch <branch>
+
+# List all workstreams
+aidp ws list
+
+# Check workstream status
+aidp ws status <slug>
+
+# Remove a workstream
+aidp ws rm <slug>
+aidp ws rm <slug> --delete-branch  # Also delete git branch
+aidp ws rm <slug> --force          # Skip confirmation
+```
+
 ### System Commands
 
 ```bash

data/lib/aidp/analyze/error_handler.rb

@@ -1,6 +1,19 @@
 # frozen_string_literal: true
 
 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
@@ -23,7 +36,7 @@ module Aidp
           context: context,
           step: step,
           retry_count: retry_count,
-          timestamp: Time.current
+          timestamp: Time.now
         }
 
         log_error(error_info)
@@ -38,21 +51,19 @@ module Aidp
 
       # Recovery strategies
       def retry_with_backoff(operation, max_retries: 3, base_delay: 1)
-        retry_count = 0
-        begin
+        Aidp::Concurrency::Backoff.retry(
+          max_attempts: max_retries,
+          base: base_delay,
+          strategy: :exponential,
+          jitter: 0.2,
+          on: [StandardError]
+        ) do
           operation.call
-        rescue => e
-          retry_count += 1
-          if retry_count <= max_retries
-            delay = base_delay * (2**(retry_count - 1))
-            logger.warn("Retrying operation in #{delay} seconds (attempt #{retry_count}/#{max_retries})")
-            Async::Task.current.sleep(delay)
-            retry
-          else
-            logger.error("Operation failed after #{max_retries} retries: #{e.message}")
-            raise e
-          end
         end
+      rescue Aidp::Concurrency::MaxAttemptsError => e
+        logger.error("Operation failed after #{max_retries} retries")
+        # Re-raise the MaxAttemptsError to preserve error chain and context
+        raise e
       end
 
       def skip_step_with_warning(step_name, error)
@@ -60,7 +71,7 @@ module Aidp
         {
           status: "skipped",
           reason: error.message,
-          timestamp: Time.current
+          timestamp: Time.now
         }
       end
 
@@ -89,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
@@ -98,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)
@@ -176,8 +196,8 @@ module Aidp
         logger.warn("HTTP error: #{error.message}")
         case error.response&.code
         when "429" # Rate limited
-          Async::Task.current.sleep(60) # Wait 1 minute
-          retry_with_backoff(-> { context[:operation].call }, max_retries: 2)
+          # For rate limiting, use longer base delay
+          retry_with_backoff(-> { context[:operation].call }, max_retries: 2, base_delay: 60)
         when "500".."599" # Server errors
           retry_with_backoff(-> { context[:operation].call }, max_retries: 3)
         else
@@ -251,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/progress.rb

@@ -67,7 +67,7 @@ module Aidp
         end
 
         @progress = if File.exist?(@progress_file)
-          YAML.load_file(@progress_file) || {}
+          YAML.safe_load_file(@progress_file, permitted_classes: [Date, Time, Symbol], aliases: true) || {}
         else
           {}
         end

data/lib/aidp/analyze/runner.rb

@@ -5,6 +5,7 @@ require_relative "steps"
 require_relative "progress"
 require_relative "../storage/file_manager"
 require_relative "../debug_mixin"
+require_relative "../skills"
 
 module Aidp
   module Analyze
@@ -18,6 +19,8 @@ module Aidp
         @is_harness_mode = !harness_runner.nil?
         @file_manager = Aidp::Storage::FileManager.new(File.join(project_dir, ".aidp"))
         @prompt = prompt
+        @skills_registry = nil # Lazy-loaded
+        @skills_composer = Aidp::Skills::Composer.new
       end
 
       def progress
@@ -199,18 +202,37 @@ module Aidp
         step_spec = Aidp::Analyze::Steps::SPEC[step_name]
         raise "Step '#{step_name}' not found" unless step_spec
 
+        # Load template
         template_name = step_spec["templates"].first
         template_path = find_template(template_name)
         raise "Template not found for step #{step_name}" unless template_path
-
         template = File.read(template_path)
 
-        # Replace template variables in the format {{key}} with option values
-        options.each do |key, value|
-          template = template.gsub("{{#{key}}}", value.to_s)
+        # Load skill if specified
+        skill = nil
+        if step_spec["skill"]
+          skill = skills_registry.find(step_spec["skill"])
+          if skill.nil?
+            Aidp.log_warn(
+              "skills",
+              "Skill not found for step",
+              step: step_name,
+              skill_id: step_spec["skill"]
+            )
+          end
         end
 
-        template
+        # Compose skill + template
+        @skills_composer.compose(skill: skill, template: template, options: options)
+      end
+
+      def skills_registry
+        @skills_registry ||= begin
+          provider = get_harness_provider_safely if @is_harness_mode
+          registry = Aidp::Skills::Registry.new(project_dir: @project_dir, provider: provider)
+          registry.load_skills
+          registry
+        end
       end
 
       # Compose prompt with harness context and user input

data/lib/aidp/analyze/steps.rb

@@ -5,20 +5,24 @@ module Aidp
     module Steps
       # Analysis step specifications
       # Templates are organized by purpose and named with action verbs
+      # Skills define WHO the agent is, templates define WHAT task to do
       SPEC = {
         "01_REPOSITORY_ANALYSIS" => {
+          "skill" => "repository_analyst",
           "templates" => ["analysis/analyze_repository.md"],
           "description" => "Initial code-maat based repository mining",
           "outs" => ["docs/analysis/repository_analysis.md"],
           "gate" => false
         },
         "02_ARCHITECTURE_ANALYSIS" => {
+          "skill" => "architecture_analyst",
           "templates" => ["analysis/analyze_architecture.md"],
           "description" => "Identify architectural patterns, dependencies, and violations",
           "outs" => ["docs/analysis/architecture_analysis.md"],
           "gate" => true
         },
         "03_TEST_ANALYSIS" => {
+          "skill" => "test_analyzer",
           "templates" => ["analysis/analyze_tests.md"],
           "description" => "Analyze existing test coverage and identify gaps",
           "outs" => ["docs/analysis/test_analysis.md"],

data/lib/aidp/cli/jobs_command.rb

@@ -206,7 +206,8 @@ module Aidp
             # Exit if job is done
             break unless status[:running]
 
-            # Wait before next update
+            # Periodic status polling - acceptable for UI updates
+            # Alternative: event-driven updates via IPC/file watching
             sleep 2
           end
         rescue Interrupt