aidp 0.23.0 → 0.25.0

data/lib/aidp/watch/state_store.rb

@@ -53,6 +53,49 @@ module Aidp
         save!
       end
 
+      # Review tracking methods
+      def review_processed?(pr_number)
+        reviews.key?(pr_number.to_s)
+      end
+
+      def review_data(pr_number)
+        reviews[pr_number.to_s]
+      end
+
+      def record_review(pr_number, data)
+        payload = {
+          "timestamp" => data[:timestamp] || Time.now.utc.iso8601,
+          "reviewers" => data[:reviewers],
+          "total_findings" => data[:total_findings]
+        }.compact
+
+        reviews[pr_number.to_s] = payload
+        save!
+      end
+
+      # CI fix tracking methods
+      def ci_fix_completed?(pr_number)
+        fix_data = ci_fixes[pr_number.to_s]
+        fix_data && fix_data["status"] == "completed"
+      end
+
+      def ci_fix_data(pr_number)
+        ci_fixes[pr_number.to_s]
+      end
+
+      def record_ci_fix(pr_number, data)
+        payload = {
+          "status" => data[:status],
+          "timestamp" => data[:timestamp] || Time.now.utc.iso8601,
+          "reason" => data[:reason],
+          "root_causes" => data[:root_causes],
+          "fixes_count" => data[:fixes_count]
+        }.compact
+
+        ci_fixes[pr_number.to_s] = payload
+        save!
+      end
+
       private
 
       def ensure_directory
@@ -81,6 +124,8 @@ module Aidp
           base = load_state
           base["plans"] ||= {}
           base["builds"] ||= {}
+          base["reviews"] ||= {}
+          base["ci_fixes"] ||= {}
           base
         end
       end
@@ -93,6 +138,14 @@ module Aidp
         state["builds"]
       end
 
+      def reviews
+        state["reviews"]
+      end
+
+      def ci_fixes
+        state["ci_fixes"]
+      end
+
       def stringify_keys(hash)
         return {} unless hash
 

data/lib/aidp/workflows/guided_agent.rb

@@ -229,16 +229,34 @@ module Aidp
           end
 
           combined_prompt = "#{system_prompt}\n\n#{user_prompt}"
+
+          # Record timing for metrics
+          start_time = Time.now
           result = provider.send_message(prompt: combined_prompt)
+          duration = Time.now - start_time
 
           if result.nil? || result.empty?
+            # Record failed metric
+            @provider_manager.record_metrics(provider_name, success: false, duration: duration, error: StandardError.new("Empty response")) if @provider_manager.respond_to?(:record_metrics)
             raise ConversationError, "Provider request failed: empty response"
           end
 
+          # Record successful metric
+          @provider_manager.record_metrics(provider_name, success: true, duration: duration) if @provider_manager.respond_to?(:record_metrics)
+
           result
         rescue => e
           message = e.message.to_s
 
+          # Record failed metric (if we haven't already from empty response check)
+          if defined?(start_time) && defined?(provider_name) && defined?(duration).nil?
+            duration = Time.now - start_time
+            @provider_manager.record_metrics(provider_name, success: false, duration: duration, error: e) if @provider_manager.respond_to?(:record_metrics)
+          end
+
+          # Extract rate limit reset time from error message (especially Claude)
+          extract_and_save_rate_limit_info(provider_name, message) if defined?(provider_name)
+
           # Classify error type for better handling
           classified = if message =~ /resource[_ ]exhausted/i || message =~ /\[resource_exhausted\]/i
             "resource_exhausted"
@@ -319,6 +337,41 @@ module Aidp
         Aidp.logger.warn("guided_agent", "Failed verbose iteration emit", error: e.message)
       end
 
+      # Extract rate limit reset time from error message and save to provider manager
+      def extract_and_save_rate_limit_info(provider_name, error_message)
+        return unless @provider_manager.respond_to?(:mark_rate_limited)
+        return unless error_message
+
+        # Claude error format: "please try again in 1m23s" or "retry after X seconds/minutes"
+        reset_time = nil
+
+        if error_message =~ /try again in (\d+)m(\d+)s/i
+          minutes = $1.to_i
+          seconds = $2.to_i
+          reset_time = Time.now + (minutes * 60) + seconds
+        elsif error_message =~ /try again in (\d+)s/i
+          seconds = $1.to_i
+          reset_time = Time.now + seconds
+        elsif error_message =~ /try again in (\d+)m/i
+          minutes = $1.to_i
+          reset_time = Time.now + (minutes * 60)
+        elsif error_message =~ /retry after (\d+) seconds?/i
+          seconds = $1.to_i
+          reset_time = Time.now + seconds
+        elsif error_message =~ /retry after (\d+) minutes?/i
+          minutes = $1.to_i
+          reset_time = Time.now + (minutes * 60)
+        end
+
+        # Mark provider as rate limited with extracted reset time
+        if reset_time
+          @provider_manager.mark_rate_limited(provider_name, reset_time)
+          Aidp.logger.info("guided_agent", "Extracted rate limit reset time", provider: provider_name, reset_at: reset_time.iso8601)
+        end
+      rescue => e
+        Aidp.logger.warn("guided_agent", "Failed to extract rate limit info", error: e.message)
+      end
+
       def validate_provider_configuration!
         configured = @provider_manager.configured_providers
         if configured.nil? || configured.empty?

data/lib/aidp/worktree.rb

@@ -2,6 +2,7 @@
 
 require "fileutils"
 require "json"
+require "open3"
 require_relative "workstream_state"
 
 module Aidp
@@ -32,16 +33,14 @@ module Aidp
           raise WorktreeExists, "Worktree already exists at #{worktree_path}"
         end
 
-        # Create the worktree
-        cmd = ["git", "worktree", "add", "-b", branch, worktree_path]
-        cmd << base_branch if base_branch
-
-        Dir.chdir(project_dir) do
-          success = system(*cmd, out: File::NULL, err: File::NULL)
-          unless success
-            raise Error, "Failed to create worktree: #{$?.exitstatus}"
-          end
-        end
+        branch_exists = branch_exists?(project_dir, branch)
+        run_worktree_add!(
+          project_dir: project_dir,
+          branch: branch,
+          branch_exists: branch_exists,
+          worktree_path: worktree_path,
+          base_branch: base_branch
+        )
 
         # Initialize .aidp directory in the worktree
         ensure_aidp_dir(worktree_path)
@@ -203,6 +202,64 @@ module Aidp
       def registry_file_path(project_dir)
         File.join(project_dir, ".aidp", "worktrees.json")
       end
+
+      def branch_exists?(project_dir, branch)
+        Dir.chdir(project_dir) do
+          system("git", "show-ref", "--verify", "--quiet", "refs/heads/#{branch}")
+        end
+      end
+
+      def run_worktree_add!(project_dir:, branch:, branch_exists:, worktree_path:, base_branch:)
+        prune_attempted = false
+
+        loop do
+          cmd = build_worktree_command(branch_exists: branch_exists, branch: branch, worktree_path: worktree_path, base_branch: base_branch)
+          stdout, stderr, status = Dir.chdir(project_dir) { Open3.capture3(*cmd) }
+
+          return if status.success?
+
+          error_output = stderr.strip.empty? ? stdout.strip : stderr.strip
+
+          if !branch_exists && branch_already_exists?(error_output, branch)
+            Aidp.log_debug("worktree", "branch_exists_retry", branch: branch)
+            branch_exists = true
+            next
+          end
+
+          if !prune_attempted && missing_registered_worktree?(error_output)
+            Aidp.log_debug("worktree", "prune_missing_worktree", branch: branch, path: worktree_path)
+            Dir.chdir(project_dir) { Open3.capture3("git", "worktree", "prune") }
+            prune_attempted = true
+            next
+          end
+
+          raise Error, "Failed to create worktree (status=#{status.exitstatus}): #{error_output}"
+        end
+      end
+
+      def build_worktree_command(branch_exists:, branch:, worktree_path:, base_branch:)
+        if branch_exists
+          ["git", "worktree", "add", worktree_path, branch]
+        else
+          cmd = ["git", "worktree", "add", "-b", branch, worktree_path]
+          cmd << base_branch if base_branch
+          cmd
+        end
+      end
+
+      def branch_already_exists?(error_output, branch)
+        return false if error_output.nil? || error_output.empty?
+
+        normalized = error_output.downcase
+        normalized.include?("branch '#{branch.downcase}' already exists") ||
+          normalized.include?("a branch named '#{branch.downcase}' already exists")
+      end
+
+      def missing_registered_worktree?(error_output)
+        return false if error_output.nil? || error_output.empty?
+
+        error_output.downcase.include?("missing but already registered worktree")
+      end
     end
   end
 end

data/lib/aidp.rb

@@ -26,6 +26,7 @@ require_relative "aidp/providers/base"
 require_relative "aidp/providers/cursor"
 require_relative "aidp/providers/anthropic"
 require_relative "aidp/providers/gemini"
+require_relative "aidp/providers/kilocode"
 # Supervised providers removed - using direct execution model
 require_relative "aidp/provider_manager"
 

data/templates/work_loop/decide_whats_next.md

@@ -0,0 +1,21 @@
+# Decide the Next Work Loop Unit
+
+You are operating inside the Aidp hybrid work loop. Review the recent deterministic outputs, consider the latest agent summary, and choose the next unit by emitting `NEXT_UNIT: <unit_name>` on its own line.
+
+## Deterministic Outputs
+
+{{DETERMINISTIC_OUTPUTS}}
+
+## Previous Agent Summary
+
+{{PREVIOUS_AGENT_SUMMARY}}
+
+## Guidance
+
+- Pick whichever unit will unblock progress the fastest. Options include deterministic unit names (`run_full_tests`, `run_lint`, etc.), `agentic` to resume coding, or `wait_for_github` if the system must await an external event.
+- Cite concrete evidence from the outputs above when selecting a unit so future readers understand the decision.
+- Keep the rationale tight—two or three sentences max.
+
+## Rationale
+
+Provide the reasoning for your `NEXT_UNIT` decision here.

data/templates/work_loop/diagnose_failures.md

@@ -0,0 +1,21 @@
+# Diagnose Recent Failures
+
+You are operating inside the Aidp work loop. Analyze the most recent deterministic failures and decide how to unblock progress. Finish by emitting `NEXT_UNIT: <unit_name>` on its own line.
+
+## Recent Deterministic Outputs
+
+{{DETERMINISTIC_OUTPUTS}}
+
+## Previous Agent Summary
+
+{{PREVIOUS_AGENT_SUMMARY}}
+
+## Instructions
+
+- Identify the concrete failures and hypothesize root causes.
+- Propose the next best action (run another deterministic unit, resume agentic editing, or wait for GitHub updates).
+- Keep the analysis crisp—focus on evidence and required follow-up work.
+
+## Analysis
+
+Summarize key findings here, then state your `NEXT_UNIT` decision.

metadata

@@ -1,11 +1,11 @@
 --- !ruby/object:Gem::Specification
 name: aidp
 version: !ruby/object:Gem::Version
-  version: 0.23.0
+  version: 0.25.0
 platform: ruby
 authors:
 - Bart Agapinan
-bindir: bin
+bindir: exe
 cert_chain: []
 date: 1980-01-02 00:00:00.000000000 Z
 dependencies:
@@ -231,7 +231,7 @@ extra_rdoc_files: []
 files:
 - LICENSE
 - README.md
-- bin/aidp
+- exe/aidp
 - lib/aidp.rb
 - lib/aidp/analyze/error_handler.rb
 - lib/aidp/analyze/feature_analyzer.rb
@@ -244,6 +244,18 @@ files:
 - lib/aidp/analyze/steps.rb
 - lib/aidp/analyze/tree_sitter_grammar_loader.rb
 - lib/aidp/analyze/tree_sitter_scan.rb
+- lib/aidp/auto_update.rb
+- lib/aidp/auto_update/bundler_adapter.rb
+- lib/aidp/auto_update/checkpoint.rb
+- lib/aidp/auto_update/checkpoint_store.rb
+- lib/aidp/auto_update/coordinator.rb
+- lib/aidp/auto_update/errors.rb
+- lib/aidp/auto_update/failure_tracker.rb
+- lib/aidp/auto_update/rubygems_api_adapter.rb
+- lib/aidp/auto_update/update_check.rb
+- lib/aidp/auto_update/update_logger.rb
+- lib/aidp/auto_update/update_policy.rb
+- lib/aidp/auto_update/version_detector.rb
 - lib/aidp/cli.rb
 - lib/aidp/cli/devcontainer_commands.rb
 - lib/aidp/cli/enhanced_input.rb
@@ -297,6 +309,7 @@ files:
 - lib/aidp/harness/provider_factory.rb
 - lib/aidp/harness/provider_info.rb
 - lib/aidp/harness/provider_manager.rb
+- lib/aidp/harness/provider_metrics.rb
 - lib/aidp/harness/provider_type_checker.rb
 - lib/aidp/harness/runner.rb
 - lib/aidp/harness/simple_user_interface.rb
@@ -347,12 +360,16 @@ files:
 - lib/aidp/prompt_optimization/style_guide_indexer.rb
 - lib/aidp/prompt_optimization/template_indexer.rb
 - lib/aidp/provider_manager.rb
+- lib/aidp/providers/adapter.rb
 - lib/aidp/providers/anthropic.rb
 - lib/aidp/providers/base.rb
+- lib/aidp/providers/capability_registry.rb
 - lib/aidp/providers/codex.rb
 - lib/aidp/providers/cursor.rb
+- lib/aidp/providers/error_taxonomy.rb
 - lib/aidp/providers/gemini.rb
 - lib/aidp/providers/github_copilot.rb
+- lib/aidp/providers/kilocode.rb
 - lib/aidp/providers/opencode.rb
 - lib/aidp/rescue_logging.rb
 - lib/aidp/safe_directory.rb
@@ -360,6 +377,7 @@ files:
 - lib/aidp/setup/devcontainer/generator.rb
 - lib/aidp/setup/devcontainer/parser.rb
 - lib/aidp/setup/devcontainer/port_manager.rb
+- lib/aidp/setup/provider_registry.rb
 - lib/aidp/setup/wizard.rb
 - lib/aidp/skills.rb
 - lib/aidp/skills/composer.rb
@@ -382,10 +400,16 @@ files:
 - lib/aidp/version.rb
 - lib/aidp/watch.rb
 - lib/aidp/watch/build_processor.rb
+- lib/aidp/watch/ci_fix_processor.rb
 - lib/aidp/watch/plan_generator.rb
 - lib/aidp/watch/plan_processor.rb
 - lib/aidp/watch/repository_client.rb
 - lib/aidp/watch/repository_safety_checker.rb
+- lib/aidp/watch/review_processor.rb
+- lib/aidp/watch/reviewers/base_reviewer.rb
+- lib/aidp/watch/reviewers/performance_reviewer.rb
+- lib/aidp/watch/reviewers/security_reviewer.rb
+- lib/aidp/watch/reviewers/senior_dev_reviewer.rb
 - lib/aidp/watch/runner.rb
 - lib/aidp/watch/state_store.rb
 - lib/aidp/workflows/definitions.rb
@@ -438,6 +462,8 @@ files:
 - templates/skills/product_strategist/SKILL.md
 - templates/skills/repository_analyst/SKILL.md
 - templates/skills/test_analyzer/SKILL.md
+- templates/work_loop/decide_whats_next.md
+- templates/work_loop/diagnose_failures.md
 homepage: https://github.com/viamin/aidp
 licenses:
 - MIT