aidp 0.32.0 → 0.33.0

data/lib/aidp/watch/runner.rb

@@ -2,21 +2,6 @@
 
 require "tty-prompt"
 
-require_relative "../message_display"
-require_relative "repository_client"
-require_relative "repository_safety_checker"
-require_relative "state_store"
-require_relative "github_state_extractor"
-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"
-require_relative "auto_processor"
-require_relative "auto_pr_processor"
-
 module Aidp
   module Watch
     # Coordinates the watch mode loop: monitors issues, handles plan/build
@@ -522,14 +507,21 @@ module Aidp
         return unless @auto_update_coordinator.policy.enabled
         return unless time_for_update_check?
 
+        @last_update_check = Time.now
         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
+          # Prefer hot reloading if available (Zeitwerk enabled with reloading)
+          if @auto_update_coordinator.hot_reload_available?
+            perform_hot_reload(update_check)
+          else
+            # Fall back to checkpoint + exit approach
+            display_message("   Saving checkpoint and initiating update...", type: :muted)
+            initiate_update(update_check)
+            # Never returns - exits with code 75
+          end
         end
       rescue Aidp::AutoUpdate::UpdateLoopError => e
         # Restart loop detected - disable auto-update
@@ -540,6 +532,25 @@ module Aidp
         Aidp.log_error("watch_runner", "update_check_failed", error: e.message)
       end
 
+      # Perform hot code reload without restarting
+      # @param update_check [Aidp::AutoUpdate::UpdateCheck] Update check result
+      def perform_hot_reload(update_check)
+        display_message("   Performing hot code reload (no restart needed)...", type: :muted)
+
+        if @auto_update_coordinator.hot_reload_update(update_check)
+          display_message("✨ Hot reload complete! Now running #{Aidp::VERSION}", type: :success)
+          Aidp.log_info("watch_runner", "hot_reload_success",
+            version: Aidp::VERSION)
+        else
+          display_message("⚠️  Hot reload skipped (no update needed)", type: :muted)
+        end
+      rescue Aidp::AutoUpdate::UpdateError => e
+        display_message("⚠️  Hot reload failed: #{e.message}", type: :warning)
+        display_message("   Falling back to checkpoint + restart...", type: :muted)
+        # Fall back to cold restart
+        initiate_update(update_check)
+      end
+
       # Determine if it's time to check for updates
       # @return [Boolean]
       def time_for_update_check?

data/lib/aidp/watch.rb

@@ -1,9 +1,7 @@
 # frozen_string_literal: true
 
-require_relative "message_display"
-require_relative "watch/repository_client"
-require_relative "watch/state_store"
-require_relative "watch/plan_generator"
-require_relative "watch/plan_processor"
-require_relative "watch/build_processor"
-require_relative "watch/runner"
+module Aidp
+  # Watch mode functionality for monitoring GitHub issues and PRs
+  module Watch
+  end
+end

data/lib/aidp/workstream_cleanup.rb

@@ -1,7 +1,5 @@
 # frozen_string_literal: true
 
-require_relative "worktree"
-require_relative "workstream_state"
 require "open3"
 require "tty-prompt"
 

data/lib/aidp/workstream_executor.rb

@@ -2,10 +2,6 @@
 
 require "concurrent-ruby"
 require "time"
-require_relative "worktree"
-require_relative "workstream_state"
-require_relative "harness/runner"
-require_relative "message_display"
 
 module Aidp
   # Executes multiple workstreams in parallel using concurrent-ruby.

data/lib/aidp/worktree.rb

@@ -3,7 +3,6 @@
 require "fileutils"
 require "json"
 require "open3"
-require_relative "workstream_state"
 
 module Aidp
   # Manages git worktree operations for parallel workstreams.

data/lib/aidp.rb

@@ -1,115 +1,30 @@
 # frozen_string_literal: true
 
-# Core extensions
-require_relative "aidp/core_ext/class_attribute"
+# Bootstrap: Load essential files before Zeitwerk
+# These files must be loaded first and are excluded from autoloading:
+# - version.rb: Needed for version checks during load
+# - core_ext: Ruby core extensions that need to be available globally
+# - logger.rb: Logging infrastructure used throughout loading
 
-# Shared modules
 require_relative "aidp/version"
-require_relative "aidp/config"
-require_relative "aidp/util"
-require_relative "aidp/rescue_logging"
-require_relative "aidp/message_display"
-require_relative "aidp/concurrency"
-require_relative "aidp/setup/wizard"
-require_relative "aidp/init"
-require_relative "aidp/watch"
-require_relative "aidp/cli"
-
-# Jobs and background execution
-require_relative "aidp/jobs/background_runner"
-
-# CLI commands
-require_relative "aidp/cli/jobs_command"
-
-# Providers
-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"
-
-# Simple file-based storage
-require_relative "aidp/storage/json_storage"
-require_relative "aidp/storage/csv_storage"
-require_relative "aidp/storage/file_manager"
-
-# Analyze mode (simplified - file-based storage only)
-require_relative "aidp/analyze/json_file_storage"
-require_relative "aidp/analyze/error_handler"
-require_relative "aidp/analyze/ruby_maat_integration"
-require_relative "aidp/analyze/runner"
-require_relative "aidp/analyze/steps"
-require_relative "aidp/analyze/progress"
-
-# Tree-sitter analysis
-require_relative "aidp/analyze/tree_sitter_grammar_loader"
-require_relative "aidp/analyze/seams"
-require_relative "aidp/analyze/tree_sitter_scan"
-require_relative "aidp/analyze/kb_inspector"
-
-# Metadata system
-require_relative "aidp/metadata/tool_metadata"
-require_relative "aidp/metadata/validator"
-require_relative "aidp/metadata/parser"
-require_relative "aidp/metadata/scanner"
-require_relative "aidp/metadata/compiler"
-require_relative "aidp/metadata/query"
-require_relative "aidp/metadata/cache"
-
-# Workflows
-require_relative "aidp/workflows/definitions"
-require_relative "aidp/workflows/selector"
-
-# Execute mode
-require_relative "aidp/execute/steps"
-require_relative "aidp/execute/runner"
-require_relative "aidp/execute/progress"
-require_relative "aidp/execute/checkpoint"
-require_relative "aidp/execute/checkpoint_display"
-require_relative "aidp/execute/work_loop_state"
-require_relative "aidp/execute/instruction_queue"
-require_relative "aidp/execute/persistent_tasklist"
-require_relative "aidp/execute/async_work_loop_runner"
-require_relative "aidp/execute/interactive_repl"
-
-# Logging
+require_relative "aidp/core_ext/class_attribute"
 require_relative "aidp/logger"
 
-# Daemon mode
-require_relative "aidp/daemon/process_manager"
-require_relative "aidp/daemon/runner"
-
-# Workstream/worktree management
-require_relative "aidp/worktree"
-require_relative "aidp/workstream_state"
-require_relative "aidp/workstream_executor"
+# Now set up Zeitwerk autoloader for the rest of the codebase
+require_relative "aidp/loader"
 
-# Harness mode
-require_relative "aidp/harness/configuration"
-require_relative "aidp/harness/config_schema"
-require_relative "aidp/harness/config_validator"
-require_relative "aidp/harness/config_loader"
-require_relative "aidp/harness/config_manager"
-require_relative "aidp/harness/ruby_llm_registry"
-require_relative "aidp/harness/model_registry"
-require_relative "aidp/harness/condition_detector"
-require_relative "aidp/harness/user_interface"
-require_relative "aidp/harness/simple_user_interface"
-require_relative "aidp/harness/provider_manager"
-require_relative "aidp/harness/provider_config"
-require_relative "aidp/harness/provider_factory"
-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"
+# Configure Zeitwerk based on environment
+# In watch mode or development, enable reloading for hot code updates
+# In production, disable reloading and eager load for performance
+reloading_enabled = ENV["AIDP_ENABLE_RELOADING"] == "1" ||
+  ENV["AIDP_WATCH_MODE"] == "1"
 
-# UI components
-require_relative "aidp/harness/ui/spinner_helper"
+Aidp::Loader.setup(
+  enable_reloading: reloading_enabled,
+  eager_load: !reloading_enabled && ENV["AIDP_EAGER_LOAD"] == "1"
+)
 
-# CLI commands
+# Manually require files that contain multiple constants (not autoloadable by Zeitwerk)
+require_relative "aidp/errors"
+require_relative "aidp/auto_update/errors"
+require_relative "aidp/harness/state/errors"

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: aidp
 version: !ruby/object:Gem::Version
-  version: 0.32.0
+  version: 0.33.0
 platform: ruby
 authors:
 - Bart Agapinan
@@ -52,61 +52,89 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '1.5'
 - !ruby/object:Gem::Dependency
-  name: thor
+  name: ostruct
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.3'
+        version: '0.6'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.3'
+        version: '0.6'
 - !ruby/object:Gem::Dependency
-  name: tty-cursor
+  name: pastel
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.7'
+        version: '0.8'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.7'
+        version: '0.8'
 - !ruby/object:Gem::Dependency
-  name: tty-screen
+  name: ruby_llm
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.8'
+        version: '1.9'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.8'
+        version: '1.9'
 - !ruby/object:Gem::Dependency
-  name: tty-reader
+  name: ruby_tree_sitter
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.9'
+        version: '2.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.9'
+        version: '2.0'
+- !ruby/object:Gem::Dependency
+  name: thor
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.3'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.3'
+- !ruby/object:Gem::Dependency
+  name: zeitwerk
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '2.7'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '2.7'
 - !ruby/object:Gem::Dependency
   name: tty-box
   requirement: !ruby/object:Gem::Requirement
@@ -122,47 +150,47 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0.7'
 - !ruby/object:Gem::Dependency
-  name: tty-table
+  name: tty-command
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.12'
+        version: '0.10'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.12'
+        version: '0.10'
 - !ruby/object:Gem::Dependency
-  name: tty-progressbar
+  name: tty-cursor
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.18'
+        version: '0.7'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.18'
+        version: '0.7'
 - !ruby/object:Gem::Dependency
-  name: tty-spinner
+  name: tty-progressbar
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.9'
+        version: '0.18'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.9'
+        version: '0.18'
 - !ruby/object:Gem::Dependency
   name: tty-prompt
   requirement: !ruby/object:Gem::Requirement
@@ -178,66 +206,66 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0.23'
 - !ruby/object:Gem::Dependency
-  name: pastel
+  name: tty-reader
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.8'
+        version: '0.9'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.8'
+        version: '0.9'
 - !ruby/object:Gem::Dependency
-  name: ruby_tree_sitter
+  name: tty-screen
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.0'
+        version: '0.8'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.0'
+        version: '0.8'
 - !ruby/object:Gem::Dependency
-  name: tty-command
+  name: tty-spinner
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.10'
+        version: '0.9'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.10'
+        version: '0.9'
 - !ruby/object:Gem::Dependency
-  name: ruby_llm
+  name: tty-table
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.9'
+        version: '0.12'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.9'
+        version: '0.12'
 description: The AI-Dev-Pipeline (AIDP) CLI provides a powerful, markdown-driven workflow
   for software development. It supports in-depth project analysis to understand existing
   codebases and an execution mode to systematically implement new features.
 email:
-- bart@sonic.next
+- bart@sonic.net
 executables:
 - aidp
 extensions: []
@@ -284,12 +312,13 @@ files:
 - lib/aidp/cli/providers_command.rb
 - lib/aidp/cli/terminal_io.rb
 - lib/aidp/cli/tools_command.rb
+- lib/aidp/comment_consolidator.rb
 - lib/aidp/concurrency.rb
 - lib/aidp/concurrency/backoff.rb
 - lib/aidp/concurrency/exec.rb
 - lib/aidp/concurrency/wait.rb
 - lib/aidp/config.rb
-- lib/aidp/config/paths.rb
+- lib/aidp/config_paths.rb
 - lib/aidp/core_ext/class_attribute.rb
 - lib/aidp/daemon/process_manager.rb
 - lib/aidp/daemon/runner.rb
@@ -316,6 +345,7 @@ files:
 - lib/aidp/execute/workflow_selector.rb
 - lib/aidp/firewall/provider_requirements_collector.rb
 - lib/aidp/harness/ai_decision_engine.rb
+- lib/aidp/harness/ai_filter_factory.rb
 - lib/aidp/harness/capability_registry.rb
 - lib/aidp/harness/completion_checker.rb
 - lib/aidp/harness/condition_detector.rb
@@ -327,11 +357,14 @@ files:
 - lib/aidp/harness/deprecation_cache.rb
 - lib/aidp/harness/enhanced_runner.rb
 - lib/aidp/harness/error_handler.rb
+- lib/aidp/harness/filter_definition.rb
 - lib/aidp/harness/filter_strategy.rb
+- lib/aidp/harness/generated_filter_strategy.rb
 - lib/aidp/harness/generic_filter_strategy.rb
 - lib/aidp/harness/model_cache.rb
 - lib/aidp/harness/model_registry.rb
 - lib/aidp/harness/output_filter.rb
+- lib/aidp/harness/output_filter_config.rb
 - lib/aidp/harness/provider_config.rb
 - lib/aidp/harness/provider_factory.rb
 - lib/aidp/harness/provider_info.rb
@@ -379,8 +412,10 @@ files:
 - lib/aidp/init/project_analyzer.rb
 - lib/aidp/init/runner.rb
 - lib/aidp/jobs/background_runner.rb
+- lib/aidp/loader.rb
 - lib/aidp/logger.rb
 - lib/aidp/message_display.rb
+- lib/aidp/metadata.rb
 - lib/aidp/metadata/cache.rb
 - lib/aidp/metadata/compiler.rb
 - lib/aidp/metadata/parser.rb
@@ -445,6 +480,7 @@ files:
 - lib/aidp/storage/csv_storage.rb
 - lib/aidp/storage/file_manager.rb
 - lib/aidp/storage/json_storage.rb
+- lib/aidp/style_guide/selector.rb
 - lib/aidp/tooling_detector.rb
 - lib/aidp/util.rb
 - lib/aidp/utils/devcontainer_detector.rb

data/lib/aidp/config/paths.rb

@@ -1,131 +0,0 @@
-# frozen_string_literal: true
-
-require "fileutils"
-
-module Aidp
-  # Centralized path management for all AIDP internal files
-  # Ensures consistent file locations and prevents path-related bugs
-  module ConfigPaths
-    # Get the main AIDP directory for a project
-    def self.aidp_dir(project_dir = Dir.pwd)
-      File.join(project_dir, ".aidp")
-    end
-
-    # Get the main configuration file path
-    def self.config_file(project_dir = Dir.pwd)
-      File.join(aidp_dir(project_dir), "aidp.yml")
-    end
-
-    # Get the configuration directory path
-    def self.config_dir(project_dir = Dir.pwd)
-      aidp_dir(project_dir)
-    end
-
-    # Get the progress directory path
-    def self.progress_dir(project_dir = Dir.pwd)
-      File.join(aidp_dir(project_dir), "progress")
-    end
-
-    # Get the execute progress file path
-    def self.execute_progress_file(project_dir = Dir.pwd)
-      File.join(progress_dir(project_dir), "execute.yml")
-    end
-
-    # Get the analyze progress file path
-    def self.analyze_progress_file(project_dir = Dir.pwd)
-      File.join(progress_dir(project_dir), "analyze.yml")
-    end
-
-    # Get the harness state directory path
-    def self.harness_state_dir(project_dir = Dir.pwd)
-      File.join(aidp_dir(project_dir), "harness")
-    end
-
-    # Get the harness state file path for a specific mode
-    def self.harness_state_file(mode, project_dir = Dir.pwd)
-      File.join(harness_state_dir(project_dir), "#{mode}_state.json")
-    end
-
-    # Get the providers directory path
-    def self.providers_dir(project_dir = Dir.pwd)
-      File.join(aidp_dir(project_dir), "providers")
-    end
-
-    # Get the provider info file path
-    def self.provider_info_file(provider_name, project_dir = Dir.pwd)
-      File.join(providers_dir(project_dir), "#{provider_name}_info.yml")
-    end
-
-    # Get the jobs directory path
-    def self.jobs_dir(project_dir = Dir.pwd)
-      File.join(aidp_dir(project_dir), "jobs")
-    end
-
-    # Get the checkpoint file path
-    def self.checkpoint_file(project_dir = Dir.pwd)
-      File.join(aidp_dir(project_dir), "checkpoint.yml")
-    end
-
-    # Get the checkpoint history file path
-    def self.checkpoint_history_file(project_dir = Dir.pwd)
-      File.join(aidp_dir(project_dir), "checkpoint_history.jsonl")
-    end
-
-    # Get the JSON storage directory path
-    def self.json_storage_dir(project_dir = Dir.pwd)
-      File.join(aidp_dir(project_dir), "json")
-    end
-
-    # Check if the main configuration file exists
-    def self.config_exists?(project_dir = Dir.pwd)
-      File.exist?(config_file(project_dir))
-    end
-
-    # Ensure the main AIDP directory exists
-    def self.ensure_aidp_dir(project_dir = Dir.pwd)
-      dir = aidp_dir(project_dir)
-      FileUtils.mkdir_p(dir) unless Dir.exist?(dir)
-      dir
-    end
-
-    # Ensure the configuration directory exists
-    def self.ensure_config_dir(project_dir = Dir.pwd)
-      ensure_aidp_dir(project_dir)
-    end
-
-    # Ensure the progress directory exists
-    def self.ensure_progress_dir(project_dir = Dir.pwd)
-      dir = progress_dir(project_dir)
-      FileUtils.mkdir_p(dir) unless Dir.exist?(dir)
-      dir
-    end
-
-    # Ensure the harness state directory exists
-    def self.ensure_harness_state_dir(project_dir = Dir.pwd)
-      dir = harness_state_dir(project_dir)
-      FileUtils.mkdir_p(dir) unless Dir.exist?(dir)
-      dir
-    end
-
-    # Ensure the providers directory exists
-    def self.ensure_providers_dir(project_dir = Dir.pwd)
-      dir = providers_dir(project_dir)
-      FileUtils.mkdir_p(dir) unless Dir.exist?(dir)
-      dir
-    end
-
-    # Ensure the jobs directory exists
-    def self.ensure_jobs_dir(project_dir = Dir.pwd)
-      dir = jobs_dir(project_dir)
-      FileUtils.mkdir_p(dir) unless Dir.exist?(dir)
-      dir
-    end
-
-    # Ensure the JSON storage directory exists
-    def self.ensure_json_storage_dir(project_dir = Dir.pwd)
-      dir = json_storage_dir(project_dir)
-      FileUtils.mkdir_p(dir) unless Dir.exist?(dir)
-      dir
-    end
-  end
-end