code_healer 0.1.24 → 0.1.32

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 704c23ee702e1f6a10a5fbbdcebe377d0523aa69d880aa40257e7351528130f3
-  data.tar.gz: e6989f6bbe820a4891c4b1268cde29d9231875c7415d4fbcc8ba9e8d5b72bc91
+  metadata.gz: 2a36985eccac2d4e1f55a411b9872478f6ec9e2f5285ebe969f6a839e600e37a
+  data.tar.gz: 7f1c43c592b9b6f6ed953ac70cc13855048d7f62222b65ddbf56b047a711ece5
 SHA512:
-  metadata.gz: e383c0afbc418195f3de44b0763bca3d18014d198d4cf0d204cf65be5b4c0a763a25ae86faef7c90eba11cb694bf76f5ed24c1322ae3ebeab293fe00509c1d25
-  data.tar.gz: 694a3cad9243ece93065c849f5dba28e3316d9a678dc8b299d5b675d7125a96441a886b946f24956b0ec54052a3f6bb2e00b3cd0815b941da35094f26c7935ea
+  metadata.gz: 790508da0467e26544b1d89bab7caece0bc0afd0e8bc10cf1aee0166704d75a5ef717cab75049b32b6c51e296339df1e8a036194684b100c7ca7e840de5a8230
+  data.tar.gz: a34f3ec6ba73155e4adba2ed08ed30a3581dc587ec1d91df2386a69653ec46a83351d2f03bfae02e525e432c64410ea119e9293c44493b08210e7e4a4ba824bd

data/CHANGELOG.md

@@ -1,3 +1,19 @@
+## [0.1.32] - 2025-09-15
+
+### Added
+- Targeted RSpec test-fix loop after code changes in `ClaudeCodeEvolutionHandler`:
+  - Runs specs for files related to recent modifications
+  - Parses failures and re-invokes Claude with failure summary
+  - Iterates up to `test_fix.max_iterations` (default 2)
+- Configuration: `test_fix.max_iterations` with sensible default in `ConfigManager` and `setup.rb`.
+
+### Changed
+- Removed demo mode code and comments across gem: prompts, setup script, logger phrasing, and workspace manager.
+- Switched class targeting to excluded-classes-only model; removed `allowed_classes` usage in config managers and setup generation.
+
+### Notes
+- Backwards compatible; no breaking API changes. Configure `test_fix.max_iterations` in `config/code_healer.yml` to tune retries.
+
 # Changelog
 
 All notable changes to this project will be documented in this file.
@@ -7,6 +23,36 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [0.1.24] - 2025-08-31
 
+## [0.1.26] - 2025-09-03
+
+### Added
+- **Enhanced PresentationLogger**: Improved with backtrace truncation, timing display, and specialized action methods
+- **Smart Value Truncation**: Long arrays and strings are intelligently truncated for clean presentation
+- **Timing Integration**: Total healing time displayed in final outcome
+- **Action-Specific Logging**: `claude_action`, `workspace_action`, `git_action` methods for contextual logging
+
+### Changed
+- **Workspace Logs**: Dramatically reduced verbosity while maintaining key information
+- **Backtrace Display**: Shows only first 3 relevant lines with clean formatting
+- **Git Operations**: Streamlined git status and diff output for presentation clarity
+- **Outcome Display**: Enhanced with timing and better formatting
+
+### Notes
+- Perfect for conference demos with `CODE_HEALER_VERBOSE=true` for detailed debugging
+- No API changes; safe minor release
+
+## [0.1.25] - 2025-09-03
+
+### Added
+- PresentationLogger: clean, emoji-labeled, presentation-friendly logs with optional verbose mode (`CODE_HEALER_VERBOSE=true`)
+- HealingJob and ClaudeCodeEvolutionHandler now use PresentationLogger for concise output
+
+### Changed
+- Suppressed noisy prints in core healing flow in favor of high-signal steps and outcomes
+
+### Notes
+- No API changes; safe minor release. Ideal for conference demos.
+
 ### Added
 - **Aggressive File Filtering**: Implemented comprehensive filtering to prevent `tmp/`, `log/`, and other temporary files from being committed
 - **Pre-Commit Validation**: Added workspace validation before commit to ensure no temporary files slip through

data/lib/code_healer/claude_code_evolution_handler.rb

@@ -1,49 +1,44 @@
 require 'timeout'
 require 'open3'
+require_relative 'presentation_logger'
 
 module CodeHealer
   class ClaudeCodeEvolutionHandler
     class << self
       def handle_error_with_claude_code(error, class_name, method_name, file_path)
-        puts "🤖 Claude Code Terminal Evolution Triggered!"
-        puts "Error: #{error.class} - #{error.message}"
-        puts "Class: #{class_name}, Method: #{method_name}"
-        puts "File: #{file_path}"
+        PresentationLogger.section("Claude Code Terminal – Evolution")
+        PresentationLogger.kv("Error", "#{error.class}: #{error.message}")
+        PresentationLogger.kv("Target", "#{class_name}##{method_name}")
+        PresentationLogger.detail("File: #{file_path}")
         
         begin
-          # Build concise, demo-optimized prompt (no repo-wide scan, no tests)
+          # Build concise prompt
           prompt = BusinessContextManager.build_claude_code_prompt(
             error, class_name, method_name, file_path
           )
+
           prompt << "\n\nStrict instructions:" \
                    "\n- Do NOT scan the entire codebase." \
                    "\n- Work only with the provided file/method context and backtrace." \
                    "\n- Return a unified diff (no prose)." \
-                   "\n- Keep changes minimal and safe." \
-                   "\n- Do NOT create or run tests." if CodeHealer::ConfigManager.demo_mode?
+                   "\n- Keep changes minimal and safe."
           
           # Execute Claude Code command
           success = execute_claude_code_fix(prompt, class_name, method_name)
           
           if success
-            puts "✅ Claude Code evolution completed successfully!"
-            # Reload modified files
+            PresentationLogger.success("Claude run completed")
             reload_modified_files
-            
-            # 🚀 Trigger Git operations (commit, push, PR creation)
-            # Note: Git operations are now handled by the isolated workspace manager
-            # to prevent duplication and ensure proper isolation
-            puts "🔄 Git operations will be handled by isolated workspace manager..."
-            
-            return true
+            # Run targeted RSpec and iterate on failures
+            return run_tests_and_iterate_fixes(class_name, method_name)
           else
-            puts "❌ Claude Code evolution failed"
+            PresentationLogger.error("Claude run failed")
             return false
           end
           
         rescue => e
-          puts "❌ Claude Code evolution error: #{e.message}"
-          puts e.backtrace.first(5)
+          PresentationLogger.error("Claude evolution error: #{e.message}")
+          PresentationLogger.detail("Backtrace: #{Array(e.backtrace).first(5).join("\n")}")
           return false
         end
       end
@@ -56,24 +51,24 @@ module CodeHealer
         # Build command
         command = build_claude_command(prompt, config)
         
-        puts "🚀 Executing Claude Code fix..."
-        puts "Command: #{command}"
-        puts "Timeout: #{config['timeout']} seconds"
+        PresentationLogger.claude_action("Executing Claude Code (timeout: #{config['timeout']}s)")
         
         begin
           # Execute with timeout
           Timeout.timeout(config['timeout']) do
             stdout, stderr, status = Open3.capture3(command)
             
-            puts "📤 Claude Code Output:"
             if stdout && !stdout.empty?
-              puts "✅ Response received:"
-              puts stdout
+              PresentationLogger.success("Response received from Claude")
+              PresentationLogger.detail(stdout)
+
+
+
+              # Business context references are intentionally not logged
 
               # Check if Claude Code is asking for permission
               if stdout.include?("permission") || stdout.include?("grant") || stdout.include?("edit")
-                puts "🔐 Claude Code needs permission to edit files"
-                puts "💡 Make sure to grant Edit permissions when prompted"
+                PresentationLogger.warn("Claude requested edit permissions. Ensure permissions are granted.")
               end
               
               # Check if fix was applied
@@ -81,28 +76,28 @@ module CodeHealer
                 puts "🎯 Fix appears to be ready - checking if files were modified..."
               end
             else
-              puts "⚠️  No output received from Claude Code"
+              PresentationLogger.warn("No output received from Claude")
             end
 
             if stderr && !stderr.empty?
-              puts "⚠️  Claude Code Warnings/Errors:"
-              puts stderr
+              PresentationLogger.warn("Claude warnings/errors present")
+              PresentationLogger.detail(stderr)
             end
             
             if status.success?
-              puts "✅ Claude Code execution completed successfully"
+              PresentationLogger.success("Claude execution succeeded")
               return true
             else
-              puts "❌ Claude Code execution failed with status: #{status.exitstatus}"
+              PresentationLogger.error("Claude execution failed (status #{status.exitstatus})")
               return false
             end
           end
           
         rescue Timeout::Error
-          puts "⏰ Claude Code execution timed out after #{config['timeout']} seconds"
+          PresentationLogger.error("Claude execution timed out after #{config['timeout']}s")
           return false
         rescue => e
-          puts "❌ Claude Code execution error: #{e.message}"
+          PresentationLogger.error("Claude execution error: #{e.message}")
           return false
         end
       end
@@ -117,28 +112,121 @@ module CodeHealer
         # Replace placeholder
         command = command_template.gsub('{prompt}', escaped_prompt)
     
-        # Add demo mode specific instructions
-        if CodeHealer::ConfigManager.demo_mode?
-          command += " --append-system-prompt 'DEMO MODE: Focus on quick fixes, skip tests, limit file changes to #{config['max_file_changes'] || 3} files maximum'"
-        else
-          if config['include_tests']
-            command += " --append-system-prompt 'Include tests when fixing the code'"
-          end
-          
-          if config['max_file_changes']
-            command += " --append-system-prompt 'Limit changes to #{config['max_file_changes']} files maximum'"
-          end
+        # Add limits and testing hints
+        if config['include_tests']
+          command += " --append-system-prompt 'Include tests when fixing the code'"
+        end
+        
+        if config['max_file_changes']
+          command += " --append-system-prompt 'Limit changes to #{config['max_file_changes']} files maximum'"
         end
     
-        # Add business context instructions
-        command += " --append-system-prompt 'CRITICAL: Before fixing any code, you MUST use the Atlassian MCP tools to search Confluence for business context. Search for documentation about the class and method you are fixing. Only proceed with the fix after reviewing the business requirements from Confluence. If no documentation is found, note this in your response.'"
-    
+        # Add business context instructions and require a delimited summary we can parse (Confluence only)
+        # command += " --append-system-prompt 'CRITICAL: Before fixing any code, use the Atlassian MCP tools to fetch business context from Confluence ONLY. Summarize the relevant findings in a concise, bullet list between the markers <<CONTEXT_START>> and <<CONTEXT_END>>. Include Confluence page titles and links, and key rules. Keep to <=5 bullets. Then proceed with the fix.'"
+
+        # Explicit Confluence page fetch (env override with default fallback)
+        explicit_confluence_page_id = (ENV['CONFLUENCE_PAGE_ID'] || '4949770295').to_s.strip
+        unless explicit_confluence_page_id.empty?
+          command += " --append-system-prompt 'CRITICAL: Before fixing any code, Explicitly fetch Confluence page ID #{explicit_confluence_page_id} using Atlassian MCP (mcp__atlassian), extract applicable business logic/rules, and APPLY those rules in the fix.'"
+        end
+          
         # Return command
         command
       end
+
+      # Run targeted specs for changed files and iterate fixes up to a configured limit
+      def run_tests_and_iterate_fixes(class_name, method_name)
+        max_iters = ConfigManager.max_test_fix_iterations
+        it = 0
+        loop do
+          it += 1
+          PresentationLogger.step("RSpec run #{it}/#{max_iters}")
+          failures = run_targeted_rspec_for_changes
+          if failures.nil?
+            PresentationLogger.warn("No RSpec detected or no changed files with specs. Skipping test loop.")
+            return true
+          end
+          if failures.empty?
+            PresentationLogger.success("All targeted specs passed")
+            return true
+          end
+          PresentationLogger.warn("Failures detected (#{failures.size}). Attempting fix iteration...")
+          break if it >= max_iters
+          attempt_fix_from_failures(failures, class_name, method_name)
+        end
+        PresentationLogger.warn("Reached max test-fix iterations (#{max_iters}).")
+        false
+      end
+
+      def run_targeted_rspec_for_changes
+        changed = get_recently_modified_files.select { |f| f.end_with?('.rb') }
+        spec_files = changed.map { |f| f.sub(%r{^app/}, 'spec/').sub(/\.rb\z/, '_spec.rb') }
+        spec_files.select! { |s| File.exist?(s) }
+        return nil if spec_files.empty? || !File.exist?('spec')
+        cmd = ["bundle exec rspec --format documentation --no-color", spec_files.map { |s| "'#{s}'" }.join(' ')].join(' ')
+        stdout, stderr, status = Open3.capture3(cmd)
+        PresentationLogger.detail(stdout) if stdout && !stdout.empty?
+        PresentationLogger.warn(stderr) if stderr && !stderr.empty?
+        parse_rspec_failures(stdout)
+      rescue => e
+        PresentationLogger.warn("RSpec execution failed: #{e.message}")
+        nil
+      end
+
+      def parse_rspec_failures(output)
+        return [] unless output
+        failures = []
+        current = nil
+        output.each_line do |line|
+          if line =~ /^\s*\d+\)\s+(.*)$/
+            current = { title: $1.strip, details: [] }
+            failures << current
+          elsif current
+            current[:details] << line
+          end
+        end
+        failures
+      end
+
+      def attempt_fix_from_failures(failures, class_name, method_name)
+        summary = failures.map { |f| "- #{f[:title]}\n  #{f[:details].first(5).join}" }.join("\n")
+        PresentationLogger.claude_action("Sending failure summary to Claude for iterative fix")
+        iterative_prompt = <<~PROMPT
+          The previous fix compiled, but targeted RSpec tests failed. Here is a concise failure summary:\n\n#{summary}\n\nUpdate the relevant code to make these tests pass. Return only a unified diff. Keep changes minimal and safe.
+        PROMPT
+        config = ConfigManager.claude_code_settings
+        command = build_claude_command(iterative_prompt, config)
+        stdout, stderr, status = Open3.capture3(command)
+        PresentationLogger.detail(stdout) if stdout && !stdout.empty?
+        PresentationLogger.warn(stderr) if stderr && !stderr.empty?
+        if status.success?
+          PresentationLogger.success("Iterative Claude run succeeded")
+          reload_modified_files
+          true
+        else
+          PresentationLogger.error("Iterative Claude run failed (status #{status.exitstatus})")
+          false
+        end
+      end
+      
+      # Parse Claude Terminal output for Confluence links/titles (Confluence only)
+      def extract_business_context_references(text)
+        refs = []
+        return refs unless text
+        
+        # Match Confluence URLs
+        confluence_regex = /(https?:\/\/[^\s]+confluence[^\s]+\/(display|spaces|pages)\/[^\s)"']+)/i
+        text.scan(confluence_regex).each do |match|
+          url = match[0]
+          title = url.split('/').last.gsub('-', ' ')[0..80]
+          refs << { source: 'Confluence', display: "#{title} (#{url})" }
+        end
+        
+        refs.uniq { |r| r[:display] }
+      end
       
       def reload_modified_files
-        puts "🔄 Reloading modified files..."
+        PresentationLogger.step("Reloading modified files")
         
         # Get list of recently modified files (last 5 minutes)
         recent_files = get_recently_modified_files
@@ -147,14 +235,14 @@ module CodeHealer
           if file_path.include?('/app/')
             begin
               load file_path
-              puts "✅ Reloaded: #{file_path}"
+              PresentationLogger.detail("Reloaded: #{file_path}")
             rescue => e
-              puts "⚠️  Failed to reload #{file_path}: #{e.message}"
+              PresentationLogger.warn("Failed to reload #{file_path}: #{e.message}")
             end
           end
         end
         
-        puts "🔄 File reloading completed"
+        PresentationLogger.detail("File reloading completed")
       end
       
       def get_recently_modified_files
@@ -191,7 +279,7 @@ module CodeHealer
           f.puts(log_entry.to_json)
         end
         
-        puts "📝 Evolution attempt logged to #{log_file}"
+        PresentationLogger.detail("Evolution attempt logged to #{log_file}")
       end
       
 

data/lib/code_healer/config_manager.rb

@@ -34,8 +34,9 @@ module CodeHealer
         config['allowed_error_types'] || []
       end
 
+      # Deprecated: allowed classes are no longer used. We rely solely on excluded_classes.
       def allowed_classes
-        config['allowed_classes'] || []
+        []
       end
 
       def excluded_classes
@@ -45,8 +46,7 @@ module CodeHealer
       def can_evolve_class?(class_name)
         return false unless enabled?
         return false if excluded_classes.include?(class_name)
-        return true if allowed_classes.empty?
-        allowed_classes.include?(class_name)
+        true
       end
 
       def can_handle_error?(error)
@@ -82,6 +82,15 @@ module CodeHealer
         config['claude_code'] || {}
       end
 
+      # Test-fix iteration settings
+      def test_fix_settings
+        config['test_fix'] || {}
+      end
+
+      def max_test_fix_iterations
+        (test_fix_settings['max_iterations'] || 2).to_i
+      end
+
       def claude_persist_session?
         claude_code_settings['persist_session'] == true
       end
@@ -163,21 +172,6 @@ module CodeHealer
         config['api'] || {}
       end
 
-      # Demo Configuration
-      def demo_settings
-        config['demo'] || {}
-      end
-
-      def demo_mode?
-        demo_settings['enabled'] == true
-      end
-
-      def demo_skip_tests?
-        demo_mode? && demo_settings['skip_tests'] != false
-      end
-
-
-
       def git_settings
         config['git'] || {}
       end
@@ -337,7 +331,6 @@ module CodeHealer
           'max_evolutions_per_day' => 10,
           'auto_generate_tests' => true,
           'allowed_error_types' => ['ZeroDivisionError', 'NoMethodError', 'ArgumentError', 'TypeError'],
-          'allowed_classes' => ['User', 'Order', 'PaymentProcessor'],
           'excluded_classes' => ['ApplicationController', 'ApplicationRecord', 'ApplicationJob', 'ApplicationMailer'],
           'evolution_strategy' => {
             'method' => 'api',
@@ -355,6 +348,9 @@ module CodeHealer
               'spec/business_context_specs.rb'
             ]
           },
+          'test_fix' => {
+            'max_iterations' => 2
+          },
           'business_context' => {
             'enabled' => true,
             'sources' => ['docs/business_rules.md']
@@ -365,10 +361,7 @@ module CodeHealer
             'max_tokens' => 2000,
             'temperature' => 0.1
           },
-          'demo' => {
-            'enabled' => false,
-            'skip_tests' => true
-          },
+          
           'git' => {
             'auto_commit' => true,
             'auto_push' => true,

data/lib/code_healer/core.rb

@@ -1,6 +1,7 @@
 require 'logger'
 require 'git'
 require 'octokit'
+require_relative 'presentation_logger'
 
 module CodeHealer
   class Core
@@ -180,15 +181,25 @@ module CodeHealer
       end
       
       def patch_classes_for_evolution
-        # Get allowed classes from config
-        allowed_classes = CodeHealer::ConfigManager.allowed_classes
-        
-        allowed_classes.each do |class_name|
-          begin
-            klass = class_name.constantize
-            patch_class_for_evolution(klass)
-          rescue NameError => e
-            puts "⚠️  Could not load class #{class_name}: #{e.message}"
+        # Patch all autoloaded classes except excluded ones
+        excluded = CodeHealer::ConfigManager.excluded_classes
+        # Attempt to iterate over loaded application classes
+        if defined?(Rails) && Rails.application
+          app_paths = [Rails.root.join('app', 'models'), Rails.root.join('app', 'controllers')]
+          app_paths.each do |path|
+            Dir.glob(File.join(path.to_s, '**', '*.rb')).each do |file|
+              begin
+                relative = file.sub(Rails.root.join('app').to_s + '/', '')
+                class_name = relative.gsub('.rb', '').split('/').map(&:classify).join('::')
+                next if class_name.nil? || class_name.empty?
+                next if excluded.include?(class_name)
+                klass = class_name.constantize rescue nil
+                next unless klass.is_a?(Class) || klass.is_a?(Module)
+                patch_class_for_evolution(klass)
+              rescue => e
+                puts "⚠️  Skipping #{file}: #{e.message}"
+              end
+            end
           end
         end
       end
@@ -229,26 +240,26 @@ module CodeHealer
         def extract_from_backtrace(backtrace)
           return [nil, nil] unless backtrace
           
-          puts "🔍 DEBUG: Starting backtrace analysis..."
-          puts "🔍 DEBUG: First 5 backtrace lines:"
-          backtrace.first(5).each_with_index { |line, i| puts "  #{i}: #{line}" }
+          PresentationLogger.detail("Starting backtrace analysis...")
+          PresentationLogger.detail("First 5 backtrace lines:")
+          backtrace.first(5).each_with_index { |line, i| PresentationLogger.detail("  #{i}: #{line}") }
           
           # Use the exact working implementation from SelfRuby
           core_methods = %w[* + - / % ** == != < > <= >= <=> === =~ !~ & | ^ ~ << >> [] []= `]
           app_file_line = backtrace.find { |line| line.include?('/app/') }
           return [nil, nil] unless app_file_line
           
-          puts "🔍 DEBUG: Found app file line: #{app_file_line}"
+          PresentationLogger.detail("Found app file line: #{app_file_line}")
           
           if app_file_line =~ /(.+):(\d+):in `(.+)'/
             file_path = $1
             method_name = $3
             
-            puts "🔍 DEBUG: Extracted file_path=#{file_path}, method_name=#{method_name}"
+            PresentationLogger.detail("Extracted file_path=#{file_path}, method_name=#{method_name}")
             
             # If it's a core method, look deeper in the backtrace
             if core_methods.include?(method_name)
-              puts "🔍 DEBUG: #{method_name} is a core method, looking deeper..."
+              PresentationLogger.detail("#{method_name} is a core method, looking deeper...")
               deeper_app_line = backtrace.find do |line| 
                 line.include?('/app/') && 
                 line =~ /in `(.+)'/ && 
@@ -260,14 +271,14 @@ module CodeHealer
               end
               
               if deeper_app_line
-                puts "🔍 DEBUG: Found deeper app line: #{deeper_app_line}"
+                PresentationLogger.detail("Found deeper app line: #{deeper_app_line}")
                 if deeper_app_line =~ /(.+):(\d+):in `(.+)'/
                   file_path = $1
                   method_name = $3
-                  puts "🔍 DEBUG: Updated to file_path=#{file_path}, method_name=#{method_name}"
+                  PresentationLogger.detail("Updated to file_path=#{file_path}, method_name=#{method_name}")
                 end
               else
-                puts "🔍 DEBUG: No deeper app line found"
+                PresentationLogger.detail("No deeper app line found")
               end
             end
             
@@ -279,7 +290,7 @@ module CodeHealer
               method_name.include?('reduce') ||
               method_name.include?('sum')
             )
-              puts "🔍 DEBUG: #{method_name} is a block/iterator, looking for containing method..."
+              PresentationLogger.detail("#{method_name} is a block/iterator, looking for containing method...")
               # Look for the FIRST valid method in the backtrace, not just any method
               containing_line = backtrace.find do |line|
                 line.include?('/app/') && 
@@ -294,14 +305,14 @@ module CodeHealer
               end
               
               if containing_line
-                puts "🔍 DEBUG: Found containing line: #{containing_line}"
+                PresentationLogger.detail("Found containing line: #{containing_line}")
                 if containing_line =~ /(.+):(\d+):in `(.+)'/
                   file_path = $1
                   method_name = $3
-                  puts "🔍 DEBUG: Updated to file_path=#{file_path}, method_name=#{method_name}"
+                  PresentationLogger.detail("Updated to file_path=#{file_path}, method_name=#{method_name}")
                 end
               else
-                puts "🔍 DEBUG: No containing line found"
+                PresentationLogger.detail("No containing line found")
               end
             end
             
@@ -320,13 +331,13 @@ module CodeHealer
                 end
               end
               
-              puts "🔍 DEBUG: Final result - class_name=#{class_name}, method_name=#{method_name}"
-              puts "🔍 Extracted: #{class_name}##{method_name} from #{file_path}"
+              PresentationLogger.detail("Final result - class_name=#{class_name}, method_name=#{method_name}")
+              PresentationLogger.detail("Extracted: #{class_name}##{method_name} from #{file_path}")
               return [class_name, method_name]
             end
           end
           
-          puts "🔍 DEBUG: No valid method found in backtrace"
+          PresentationLogger.detail("No valid method found in backtrace")
           [nil, nil]
         end