aidp 0.24.0 → 0.26.0

data/lib/aidp/watch/runner.rb

@@ -9,6 +9,10 @@ require_relative "state_store"
 require_relative "plan_generator"
 require_relative "plan_processor"
 require_relative "build_processor"
+require_relative "../auto_update"
+require_relative "review_processor"
+require_relative "ci_fix_processor"
+require_relative "change_request_processor"
 
 module Aidp
   module Watch
@@ -26,6 +30,8 @@ module Aidp
         @project_dir = project_dir
         @force = force
         @verbose = verbose
+        @provider_name = provider_name
+        @safety_config = safety_config
 
         owner, repo = RepositoryClient.parse_issues_url(issues_url)
         @repository_client = RepositoryClient.new(owner: owner, repo: repo, gh_available: gh_available)
@@ -52,9 +58,42 @@ module Aidp
           verbose: verbose,
           label_config: label_config
         )
+
+        # Initialize auto-update coordinator
+        @auto_update_coordinator = Aidp::AutoUpdate.coordinator(project_dir: project_dir)
+        @last_update_check = nil
+        @review_processor = ReviewProcessor.new(
+          repository_client: @repository_client,
+          state_store: @state_store,
+          provider_name: provider_name,
+          project_dir: project_dir,
+          label_config: label_config,
+          verbose: verbose
+        )
+        @ci_fix_processor = CiFixProcessor.new(
+          repository_client: @repository_client,
+          state_store: @state_store,
+          provider_name: provider_name,
+          project_dir: project_dir,
+          label_config: label_config,
+          verbose: verbose
+        )
+        @change_request_processor = ChangeRequestProcessor.new(
+          repository_client: @repository_client,
+          state_store: @state_store,
+          provider_name: provider_name,
+          project_dir: project_dir,
+          label_config: label_config,
+          change_request_config: safety_config[:pr_change_requests] || safety_config["pr_change_requests"] || {},
+          safety_config: safety_config[:safety] || safety_config["safety"] || {},
+          verbose: verbose
+        )
       end
 
       def start
+        # Check for and restore from checkpoint (after auto-update)
+        restore_from_checkpoint_if_exists
+
         # Validate safety requirements before starting
         @safety_checker.validate_watch_mode_safety!(force: @force)
 
@@ -91,6 +130,10 @@ module Aidp
       def process_cycle
         process_plan_triggers
         process_build_triggers
+        check_for_updates_if_due
+        process_review_triggers
+        process_ci_fix_triggers
+        process_change_request_triggers
       end
 
       def process_plan_triggers
@@ -149,12 +192,191 @@ module Aidp
         end
       end
 
+      def process_review_triggers
+        review_label = @review_processor.review_label
+        prs = @repository_client.list_pull_requests(labels: [review_label], state: "open")
+        Aidp.log_debug("watch_runner", "review_poll", label: review_label, total: prs.size)
+        prs.each do |pr|
+          unless pr_has_label?(pr, review_label)
+            Aidp.log_debug("watch_runner", "review_skip_label_mismatch", pr: pr[:number], labels: pr[:labels])
+            next
+          end
+
+          detailed = @repository_client.fetch_pull_request(pr[:number])
+
+          # Check author authorization before processing
+          unless @safety_checker.should_process_issue?(detailed, enforce: false)
+            Aidp.log_debug("watch_runner", "review_skip_unauthorized_author", pr: detailed[:number], author: detailed[:author])
+            next
+          end
+
+          Aidp.log_debug("watch_runner", "review_process", pr: detailed[:number])
+          @review_processor.process(detailed)
+        rescue RepositorySafetyChecker::UnauthorizedAuthorError => e
+          Aidp.log_warn("watch_runner", "unauthorized PR author", pr: pr[:number], error: e.message)
+        end
+      end
+
+      def process_ci_fix_triggers
+        ci_fix_label = @ci_fix_processor.ci_fix_label
+        prs = @repository_client.list_pull_requests(labels: [ci_fix_label], state: "open")
+        Aidp.log_debug("watch_runner", "ci_fix_poll", label: ci_fix_label, total: prs.size)
+        prs.each do |pr|
+          unless pr_has_label?(pr, ci_fix_label)
+            Aidp.log_debug("watch_runner", "ci_fix_skip_label_mismatch", pr: pr[:number], labels: pr[:labels])
+            next
+          end
+
+          detailed = @repository_client.fetch_pull_request(pr[:number])
+
+          # Check author authorization before processing
+          unless @safety_checker.should_process_issue?(detailed, enforce: false)
+            Aidp.log_debug("watch_runner", "ci_fix_skip_unauthorized_author", pr: detailed[:number], author: detailed[:author])
+            next
+          end
+
+          Aidp.log_debug("watch_runner", "ci_fix_process", pr: detailed[:number])
+          @ci_fix_processor.process(detailed)
+        rescue RepositorySafetyChecker::UnauthorizedAuthorError => e
+          Aidp.log_warn("watch_runner", "unauthorized PR author", pr: pr[:number], error: e.message)
+        end
+      end
+
+      def process_change_request_triggers
+        change_request_label = @change_request_processor.change_request_label
+        prs = @repository_client.list_pull_requests(labels: [change_request_label], state: "open")
+        Aidp.log_debug("watch_runner", "change_request_poll", label: change_request_label, total: prs.size)
+        prs.each do |pr|
+          unless pr_has_label?(pr, change_request_label)
+            Aidp.log_debug("watch_runner", "change_request_skip_label_mismatch", pr: pr[:number], labels: pr[:labels])
+            next
+          end
+
+          detailed = @repository_client.fetch_pull_request(pr[:number])
+
+          # Check author authorization before processing
+          unless @safety_checker.should_process_issue?(detailed, enforce: false)
+            Aidp.log_debug("watch_runner", "change_request_skip_unauthorized_author", pr: detailed[:number], author: detailed[:author])
+            next
+          end
+
+          Aidp.log_debug("watch_runner", "change_request_process", pr: detailed[:number])
+          @change_request_processor.process(detailed)
+        rescue RepositorySafetyChecker::UnauthorizedAuthorError => e
+          Aidp.log_warn("watch_runner", "unauthorized PR author", pr: pr[:number], error: e.message)
+        end
+      end
+
       def issue_has_label?(issue, label)
         Array(issue[:labels]).any? do |issue_label|
           name = issue_label.is_a?(Hash) ? issue_label["name"] : issue_label.to_s
           name.casecmp(label).zero?
         end
       end
+
+      # Restore from checkpoint if one exists (after auto-update)
+      def restore_from_checkpoint_if_exists
+        return unless @auto_update_coordinator.policy.enabled
+
+        checkpoint = @auto_update_coordinator.restore_from_checkpoint
+        return unless checkpoint
+
+        # Checkpoint exists and was successfully restored
+        display_message("✨ Restored from checkpoint after update to v#{Aidp::VERSION}", type: :success)
+
+        # Override instance variables with checkpoint state
+        if checkpoint.watch_mode? && checkpoint.watch_state
+          @interval = checkpoint.watch_state[:interval] || @interval
+          Aidp.log_info("watch_runner", "checkpoint_restored",
+            checkpoint_id: checkpoint.checkpoint_id,
+            interval: @interval)
+        end
+      rescue => e
+        # Log but don't fail - continue with fresh start
+        Aidp.log_error("watch_runner", "checkpoint_restore_failed",
+          error: e.message)
+        display_message("⚠️  Checkpoint restore failed, starting fresh: #{e.message}", type: :warning)
+      end
+
+      # Check for updates at appropriate intervals
+      def check_for_updates_if_due
+        return unless @auto_update_coordinator.policy.enabled
+        return unless time_for_update_check?
+
+        update_check = @auto_update_coordinator.check_for_update
+
+        if update_check.should_update?
+          display_message("🔄 Update available: #{update_check.current_version} → #{update_check.available_version}", type: :highlight)
+          display_message("   Saving checkpoint and initiating update...", type: :muted)
+
+          initiate_update(update_check)
+          # Never returns - exits with code 75
+        end
+      rescue Aidp::AutoUpdate::UpdateLoopError => e
+        # Restart loop detected - disable auto-update
+        display_message("⚠️  Auto-update disabled: #{e.message}", type: :error)
+        Aidp.log_error("watch_runner", "update_loop_detected", error: e.message)
+      rescue Aidp::AutoUpdate::UpdateError => e
+        # Non-fatal update error - log and continue
+        Aidp.log_error("watch_runner", "update_check_failed", error: e.message)
+      end
+
+      # Determine if it's time to check for updates
+      # @return [Boolean]
+      def time_for_update_check?
+        return true if @last_update_check.nil?
+
+        elapsed = Time.now - @last_update_check
+        elapsed >= @auto_update_coordinator.policy.check_interval_seconds
+      end
+
+      # Initiate update process: capture state, create checkpoint, exit
+      # @param update_check [Aidp::AutoUpdate::UpdateCheck] Update check result
+      def initiate_update(update_check)
+        current_state = capture_current_state
+
+        # This will exit with code 75 if successful
+        @auto_update_coordinator.initiate_update(current_state)
+      end
+
+      # Capture current watch mode state for checkpoint
+      # @return [Hash] Current state
+      def capture_current_state
+        {
+          mode: "watch",
+          watch_state: {
+            repository: @repository_client.full_repo,
+            interval: @interval,
+            provider_name: @provider_name,
+            persona: nil,
+            safety_config: @safety_config,
+            worktree_context: capture_worktree_context,
+            state_store_snapshot: @state_store.send(:state).dup
+          }
+        }
+      end
+
+      # Capture git worktree context
+      # @return [Hash] Worktree information
+      def capture_worktree_context
+        return {} unless system("git rev-parse --git-dir > /dev/null 2>&1")
+
+        {
+          branch: `git rev-parse --abbrev-ref HEAD`.strip,
+          commit_sha: `git rev-parse HEAD`.strip,
+          remote_url: `git config --get remote.origin.url`.strip
+        }
+      rescue => e
+        Aidp.log_debug("watch_runner", "worktree_context_unavailable", error: e.message)
+        {}
+      end
+
+      def pr_has_label?(pr, label)
+        Array(pr[:labels]).any? do |pr_label|
+          name = pr_label.is_a?(Hash) ? pr_label["name"] : pr_label.to_s
+          name.casecmp(label).zero?
+        end
+      end
     end
   end
 end

data/lib/aidp/watch/state_store.rb

@@ -27,14 +27,26 @@ module Aidp
         plans[issue_number.to_s]
       end
 
+      def plan_iteration_count(issue_number)
+        plan = plans[issue_number.to_s]
+        return 0 unless plan
+        plan["iteration"] || 1
+      end
+
       def record_plan(issue_number, data)
+        existing_plan = plans[issue_number.to_s]
+        iteration = existing_plan ? (existing_plan["iteration"] || 1) + 1 : 1
+
         payload = {
           "summary" => data[:summary],
           "tasks" => data[:tasks],
           "questions" => data[:questions],
           "comment_body" => data[:comment_body],
           "comment_hint" => data[:comment_hint],
-          "posted_at" => data[:posted_at] || Time.now.utc.iso8601
+          "comment_id" => data[:comment_id],
+          "posted_at" => data[:posted_at] || Time.now.utc.iso8601,
+          "iteration" => iteration,
+          "previous_iteration_at" => existing_plan ? existing_plan["posted_at"] : nil
         }.compact
 
         plans[issue_number.to_s] = payload
@@ -53,6 +65,77 @@ 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
+
+      # Change request tracking methods
+      def change_request_processed?(pr_number)
+        change_requests.key?(pr_number.to_s)
+      end
+
+      def change_request_data(pr_number)
+        change_requests[pr_number.to_s]
+      end
+
+      def record_change_request(pr_number, data)
+        payload = {
+          "status" => data[:status],
+          "timestamp" => data[:timestamp] || Time.now.utc.iso8601,
+          "changes_applied" => data[:changes_applied],
+          "commits" => data[:commits],
+          "reason" => data[:reason],
+          "clarification_count" => data[:clarification_count]
+        }.compact
+
+        change_requests[pr_number.to_s] = payload
+        save!
+      end
+
+      def reset_change_request_state(pr_number)
+        change_requests.delete(pr_number.to_s)
+        save!
+      end
+
       private
 
       def ensure_directory
@@ -81,6 +164,9 @@ module Aidp
           base = load_state
           base["plans"] ||= {}
           base["builds"] ||= {}
+          base["reviews"] ||= {}
+          base["ci_fixes"] ||= {}
+          base["change_requests"] ||= {}
           base
         end
       end
@@ -93,6 +179,18 @@ module Aidp
         state["builds"]
       end
 
+      def reviews
+        state["reviews"]
+      end
+
+      def ci_fixes
+        state["ci_fixes"]
+      end
+
+      def change_requests
+        state["change_requests"]
+      end
+
       def stringify_keys(hash)
         return {} unless hash
 

data/lib/aidp/workstream_executor.rb

@@ -156,8 +156,11 @@ module Aidp
         )
 
         # Log error and exit
-        warn("Error in workstream #{slug}: #{e.message}")
-        warn(e.backtrace.first(5).join("\n"))
+        # Suppress backtrace noise during tests while keeping it for production debugging
+        unless ENV["RSPEC_RUNNING"] == "true"
+          warn("Error in workstream #{slug}: #{e.message}")
+          warn(e.backtrace.first(5).join("\n"))
+        end
         exit(1)
       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"
 
@@ -92,6 +93,10 @@ require_relative "aidp/harness/state_manager"
 require_relative "aidp/harness/error_handler"
 require_relative "aidp/harness/status_display"
 require_relative "aidp/harness/runner"
+require_relative "aidp/harness/filter_strategy"
+require_relative "aidp/harness/generic_filter_strategy"
+require_relative "aidp/harness/rspec_filter_strategy"
+require_relative "aidp/harness/output_filter"
 
 # UI components
 require_relative "aidp/harness/ui/spinner_helper"

data/templates/aidp.yml.example

@@ -708,6 +708,56 @@ devcontainer:
       # - "api.example.com"
       # - "registry.example.com"
 
+# Watch mode configuration
+# Configures automated monitoring and processing of GitHub issues/PRs
+watch:
+  # Label configuration for different triggers
+  labels:
+    plan_trigger: "aidp-plan"                    # Trigger plan generation
+    needs_input: "aidp-needs-input"              # Request clarification
+    ready_to_build: "aidp-ready"                 # Ready for implementation
+    build_trigger: "aidp-build"                  # Trigger implementation
+    review_trigger: "aidp-review"                # Trigger code review
+    ci_fix_trigger: "aidp-fix-ci"                # Trigger CI fix
+    change_request_trigger: "aidp-request-changes"  # Trigger PR change requests
+
+  # Safety configuration
+  safety:
+    # Allow watch mode on public repositories (default: false for safety)
+    allow_public_repos: false
+
+    # List of GitHub usernames allowed to trigger automated actions
+    # Empty list means all authenticated users (for private repos)
+    author_allowlist: []
+      # - "user1"
+      # - "user2"
+
+    # Require running in a container environment (additional safety)
+    require_container: false
+
+  # PR change request configuration
+  pr_change_requests:
+    # Enable/disable PR change request feature
+    enabled: true
+
+    # Allow changes across multiple files in a single request
+    allow_multi_file_edits: true
+
+    # Run tests and linters before pushing changes
+    # If tests fail, changes are committed locally but not pushed
+    run_tests_before_push: true
+
+    # Prefix for commit messages created by change request processor
+    commit_message_prefix: "aidp: pr-change"
+
+    # Require at least one comment before processing change request
+    # This ensures there's context for what changes are requested
+    require_comment_reference: true
+
+    # Maximum PR diff size (in lines) to process
+    # Large PRs are skipped to avoid overwhelming the AI analysis
+    max_diff_size: 2000
+
 # Configuration tips:
 # - Set max_tokens based on your API plan limits
 # - Use default_flags to customize provider behavior
@@ -724,3 +774,6 @@ devcontainer:
 # - Use time-based configs for different usage patterns
 # - Use step-specific configs for different workflow steps
 # - Use user-specific configs for personalized experiences
+# - Configure watch mode safety settings before using on public repositories
+# - Set author_allowlist to restrict who can trigger automated actions
+# - Adjust max_diff_size based on your typical PR sizes and AI capabilities

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: aidp
 version: !ruby/object:Gem::Version
-  version: 0.24.0
+  version: 0.26.0
 platform: ruby
 authors:
 - Bart Agapinan
@@ -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
@@ -293,12 +305,16 @@ files:
 - lib/aidp/harness/configuration.rb
 - lib/aidp/harness/enhanced_runner.rb
 - lib/aidp/harness/error_handler.rb
+- lib/aidp/harness/filter_strategy.rb
+- lib/aidp/harness/generic_filter_strategy.rb
+- lib/aidp/harness/output_filter.rb
 - lib/aidp/harness/provider_config.rb
 - 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/rspec_filter_strategy.rb
 - lib/aidp/harness/runner.rb
 - lib/aidp/harness/simple_user_interface.rb
 - lib/aidp/harness/state/errors.rb
@@ -357,6 +373,7 @@ files:
 - 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
@@ -387,10 +404,17 @@ files:
 - lib/aidp/version.rb
 - lib/aidp/watch.rb
 - lib/aidp/watch/build_processor.rb
+- lib/aidp/watch/change_request_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