code_healer 0.1.5 → 0.1.7

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 053ae42e7bf5fe56e20fefb00f6a5474bf2e4e74622738f81b45073bfe735da3
-  data.tar.gz: e3349ed04f2cb27fec077f0003f5bde4b63b9a9a220158bbfb32b864ea1af75b
+  metadata.gz: fb4a7ded404d0efca6e6e111b0186cc2bdeaef8a6eaf4041fd9799fcc447d044
+  data.tar.gz: 95f4317a52fa36bc1aae2923d497a01c282486de0680548a5e80ff1f15ff6468
 SHA512:
-  metadata.gz: 2bac061a4947215ba717645dccf034ebf1e130eb98f58861d4f88ad8249d17cd83380853b965a1b7f539a5c6af9bbeefd795eb45b521023f447ac99cc222f8e1
-  data.tar.gz: da855be2022f1f6e1cd67dd48ea8a42985444b20895560fd34fe506734d2837fde99a374411c07f10622657b2acd5fc4ed329a5f04990aea6b30575ef0f3b5c2
+  metadata.gz: ad4af34d619b3d1361a61b5dd5e7a285bb9cdc399b72d200209a84ce565f2e7740917878684e4b094254ba3fbc858765e4dc2af2a3d425dd8d343a71e180477e
+  data.tar.gz: 89b963d38d992791a738ff3ab91716939e39a1e8b84d056574bc6ffae504d66e6c192306ba28f5671efeec54967fa0eefca044e17121538055875a91875f4dbd

data/CHANGELOG.md

@@ -5,7 +5,21 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
-## [0.1.5] - 2025-01-14
+## [0.1.7] - 2025-01-14
+
+### Fixed
+- **Git operations in isolated healing workspace** - Preserved .git directory during cloning for proper Git operations
+- **Branch creation and commit operations** now work correctly in the isolated workspace
+- **Workspace cleanup** properly removes .git directory to prevent conflicts
+
+## [0.1.6] - 2025-01-14
+
+### Added
+- **Code heal directory permission validation** during interactive setup
+- **Repository access testing** to ensure the directory can clone and push to the target repo
+- **Write permission verification** for the code heal directory
+- **Automatic directory creation** if it doesn't exist
+- **Comprehensive error messages** with troubleshooting tips for permission issues
 
 ### Fixed
 - **Duplicate HealingJob class definition** that was preventing isolated healing workspace system from working

data/lib/code_healer/healing_workspace_manager.rb

@@ -161,6 +161,13 @@ module CodeHealer
         
         return unless Dir.exist?(workspace_path)
         
+        # Remove .git directory first to avoid conflicts
+        git_dir = File.join(workspace_path, '.git')
+        if Dir.exist?(git_dir)
+          puts "🧹 [WORKSPACE] Removing .git directory to prevent conflicts..."
+          FileUtils.rm_rf(git_dir)
+        end
+        
         puts "🧹 [WORKSPACE] Removing workspace directory..."
         FileUtils.rm_rf(workspace_path)
         puts "🧹 [WORKSPACE] Workspace cleanup completed"
@@ -207,10 +214,9 @@ module CodeHealer
           puts "🌿 [WORKSPACE] Clone result: #{result ? 'SUCCESS' : 'FAILED'}"
           
           if result
-            puts "🌿 [WORKSPACE] Removing .git to avoid conflicts..."
-            # Remove .git to avoid conflicts
-            FileUtils.rm_rf(File.join(workspace_path, '.git'))
-            puts "🌿 [WORKSPACE] .git removed successfully"
+            puts "🌿 [WORKSPACE] Git repository preserved for healing operations"
+            # Keep .git for Git operations during healing
+            # We'll clean it up later in cleanup_workspace
           else
             puts "🌿 [WORKSPACE] Clone failed, checking workspace..."
             puts "🌿 [WORKSPACE] Workspace exists: #{Dir.exist?(workspace_path)}"
@@ -237,6 +243,7 @@ module CodeHealer
               checkout_result = system("git checkout #{current_branch}")
               puts "🌿 [WORKSPACE] Checkout result: #{checkout_result ? 'SUCCESS' : 'FAILED'}"
             end
+            puts "🌿 [WORKSPACE] Git repository preserved for healing operations"
           else
             puts "🌿 [WORKSPACE] Full repo clone failed"
           end

data/lib/code_healer/setup.rb

@@ -7,6 +7,7 @@
 require 'fileutils'
 require 'yaml'
 require 'optparse'
+require 'octokit'
 
 # Parse command line options
 options = {}
@@ -88,6 +89,109 @@ def read_file_content(file_path)
   File.read(file_path) if file_exists?(file_path)
 end
 
+def normalize_repository_url(repository_input, github_token = nil)
+  return nil if repository_input.nil?
+  input = repository_input.strip
+  return nil if input.empty?
+
+  # If it's already a URL, return as-is
+  if input.include?('://')
+    return input
+  end
+
+  # Build HTTPS URL from owner/repo
+  base = "https://github.com/#{input}.git"
+  return base if github_token.nil? || github_token.strip.empty?
+
+  # Embed token for validation clone only (do not log the token)
+  "https://#{github_token}@github.com/#{input}.git"
+end
+
+# Permission validation method
+def validate_code_heal_directory_permissions(directory_path, repository_url, github_token = nil)
+  puts "  📁 Checking directory: #{directory_path}"
+  
+  # Check if directory exists or can be created
+  begin
+    if Dir.exist?(directory_path)
+      puts "    ✅ Directory exists"
+    else
+      puts "    🔨 Creating directory..."
+      Dir.mkdir(directory_path)
+      puts "    ✅ Directory created successfully"
+    end
+  rescue => e
+    puts "    ❌ Cannot create directory: #{e.message}"
+    return false
+  end
+  
+  # Check write permissions
+  begin
+    test_file = File.join(directory_path, '.permission_test')
+    File.write(test_file, 'test')
+    File.delete(test_file)
+    puts "    ✅ Write permissions verified"
+  rescue => e
+    puts "    ❌ Write permission failed: #{e.message}"
+    return false
+  end
+
+  # If we have a token and a repo, verify push permissions via GitHub API
+  if github_token && !github_token.strip.empty? && repository_url && !repository_url.strip.empty?
+    begin
+      client = Octokit::Client.new(access_token: github_token)
+      repo_full_name = if repository_url.include?('github.com')
+        path = repository_url.split('github.com/').last.to_s
+        path.sub(/\.git\z/, '')
+      else
+        repository_url
+      end
+      repo = client.repository(repo_full_name)
+      perms = (repo.respond_to?(:permissions) ? repo.permissions : repo[:permissions]) || {}
+      can_push = perms[:push] == true || perms['push'] == true
+      if can_push
+        puts "    ✅ GitHub token has push permission to #{repo_full_name}"
+      else
+        puts "    ⚠️  GitHub token does not have push permission to #{repo_full_name}"
+      end
+    rescue => e
+      puts "    ⚠️  Could not verify push permission via GitHub API: #{e.message}"
+    end
+  else
+    puts "    ⚠️  Skipping GitHub push-permission check (missing token or repository)"
+  end
+  
+  # Check if we can clone the repository (read access)
+  puts "    🔍 Testing repository access..."
+  begin
+    require 'git'
+    require 'fileutils'
+    
+    # Create a temporary test directory
+    test_dir = File.join(directory_path, 'test_clone_' + Time.now.to_i.to_s)
+    
+    # Try to clone the repository
+    puts "    📥 Attempting to clone repository..."
+    # Avoid printing token in logs
+    safe_url = repository_url.to_s.gsub(/:\/\/[A-Za-z0-9_\-]+@/, '://***@')
+    # Perform clone
+    Git.clone(repository_url, test_dir, depth: 1)
+    
+    # Clean up test clone
+    FileUtils.rm_rf(test_dir)
+    puts "    ✅ Repository access verified (#{safe_url})"
+    
+    return true
+  rescue => e
+    puts "    ❌ Repository access failed: #{e.message}"
+    puts "    💡 Make sure:"
+    puts "       - Your GitHub token has repo access"
+    puts "       - The repository URL is correct (e.g., https://github.com/owner/repo.git or owner/repo)"
+    puts "       - You have network access to GitHub"
+    return false
+  end
+end
+
 # Main setup logic
 puts "🏥 Welcome to CodeHealer Setup! 🚀"
 puts "=" * 50
@@ -119,7 +223,7 @@ else
     puts "📝 Would add 'code_healer' to Gemfile"
   else
     File.write(gemfile_path, new_gemfile_content)
-    puts "✅ Added 'code_healer' to Gemfile"
+  puts "✅ Added 'code_healer' to Gemfile"
   end
 end
 
@@ -145,7 +249,8 @@ puts "Get one at: https://github.com/settings/tokens"
 puts
 
 github_token = ask_for_input("Enter your GitHub personal access token (or press Enter to skip for now):")
-github_repo = ask_for_input("Enter your GitHub repository (username/repo):")
+github_repo = ask_for_input("Enter your GitHub repository (username/repo or full URL):")
+github_repo_url = normalize_repository_url(github_repo, github_token)
 
 # Git Branch Configuration
 puts
@@ -161,6 +266,29 @@ puts "CodeHealer will clone your current branch here before making fixes."
 puts
 
 code_heal_directory = ask_for_input("Enter code heal directory path (default: /tmp/code_healer_workspaces):", default: "/tmp/code_healer_workspaces")
+
+# Validate code heal directory permissions
+puts "🔍 Validating code heal directory permissions..."
+if github_repo_url && !github_repo_url.strip.empty?
+  if validate_code_heal_directory_permissions(code_heal_directory, github_repo_url, github_token)
+    puts "✅ Code heal directory permissions validated successfully!"
+  else
+    puts "❌ Code heal directory permission validation failed!"
+    puts "Please ensure the directory has proper write permissions and can access the repository."
+    code_heal_directory = ask_for_input("Enter a different code heal directory path:", default: "/tmp/code_healer_workspaces")
+    
+    # Retry validation
+    if validate_code_heal_directory_permissions(code_heal_directory, github_repo_url, github_token)
+      puts "✅ Code heal directory permissions validated successfully!"
+    else
+      puts "⚠️  Permission validation failed again. You may need to fix permissions manually."
+    end
+  end
+else
+  puts "⚠️  Skipping repository access validation (no repository URL provided)"
+  puts "   Directory permissions will be validated when you run the actual setup"
+end
+
 auto_cleanup = ask_for_yes_no("Automatically clean up healing workspaces after use?", default: true)
 cleanup_after_hours = ask_for_input("Clean up workspaces after how many hours? (default: 24):", default: "24")
 
@@ -198,12 +326,12 @@ puts "📝 Step 3: Creating Configuration Files"
 puts
 
 # Create actual .env file (will be ignored by git)
-env_content = <<~ENV
+  env_content = <<~ENV
   # CodeHealer Configuration
-  # OpenAI Configuration
+    # OpenAI Configuration
   OPENAI_API_KEY=#{openai_key}
-  
-  # GitHub Configuration
+    
+    # GitHub Configuration
   GITHUB_TOKEN=#{github_token}
   GITHUB_REPOSITORY=#{github_repo}
   
@@ -214,81 +342,81 @@ ENV
 create_file_with_content('.env', env_content, dry_run: options[:dry_run])
 
 # Create code_healer.yml
-config_content = <<~YAML
-  # CodeHealer Configuration
-  enabled: true
-  
+  config_content = <<~YAML
+    # CodeHealer Configuration
+    enabled: true
+    
   # Allowed classes for healing (customize as needed)
-  allowed_classes:
-    - User
-    - Order
-    - PaymentProcessor
+    allowed_classes:
+      - User
+      - Order
+      - PaymentProcessor
     - OrderProcessor
-  
+    
   # Excluded classes (never touch these)
-  excluded_classes:
-    - ApplicationController
-    - ApplicationRecord
-    - ApplicationJob
-    - ApplicationMailer
+    excluded_classes:
+      - ApplicationController
+      - ApplicationRecord
+      - ApplicationJob
+      - ApplicationMailer
     - ApplicationHelper
-  
-  # Allowed error types for healing
-  allowed_error_types:
+    
+    # Allowed error types for healing
+    allowed_error_types:
     - ZeroDivisionError
     - NoMethodError
-    - ArgumentError
+      - ArgumentError
     - TypeError
-    - NameError
-    - ValidationError
-  
+      - NameError
+      - ValidationError
+    
   # Evolution Strategy Configuration
-  evolution_strategy:
+    evolution_strategy:
     method: #{evolution_method}  # Options: api, claude_code_terminal, hybrid
     fallback_to_api: #{fallback_to_api}  # If Claude Code fails, fall back to API
-  
+    
   # Claude Code Terminal Configuration
-  claude_code:
+    claude_code:
     enabled: #{evolution_method == 'claude_code_terminal' || evolution_method == 'hybrid'}
     timeout: 300  # Timeout in seconds
-    max_file_changes: 10
-    include_tests: true
+      max_file_changes: 10
+      include_tests: true
     command_template: "claude --print '{prompt}' --output-format text --permission-mode acceptEdits --allowedTools Edit"
-    business_context_sources:
-      - "config/business_rules.yml"
-      - "docs/business_logic.md"
-      - "spec/business_context_specs.rb"
-  
+      business_context_sources:
+        - "config/business_rules.yml"
+        - "docs/business_logic.md"
+        - "spec/business_context_specs.rb"
+    
   # Business Context Configuration
-  business_context:
-    enabled: true
+    business_context:
+      enabled: true
     sources:
       - "docs/business_rules.md"
-  
-  # OpenAI API configuration
-  api:
-    provider: openai
+    
+    # OpenAI API configuration
+    api:
+      provider: openai
     model: gpt-4
-    max_tokens: 2000
-    temperature: 0.1
-  
-  # Git operations
-  git:
-    auto_commit: true
-    auto_push: true
+      max_tokens: 2000
+      temperature: 0.1
+    
+    # Git operations
+    git:
+      auto_commit: true
+      auto_push: true
     branch_prefix: "#{branch_prefix}"
     commit_message_template: 'Fix {{class_name}}\#\#{{method_name}}: {{error_type}}'
     pr_target_branch: "#{pr_target_branch}"
   
   # Pull Request Configuration
-  pull_request:
-    enabled: true
-    auto_create: true
-    labels:
-      - "auto-fix"
+    pull_request:
+      enabled: true
+      auto_create: true
+      labels:
+        - "auto-fix"
       - "self-evolving"
-      - "bug-fix"
-  
+        - "bug-fix"
+    
   # Safety Configuration
   safety:
     backup_before_evolution: true
@@ -369,6 +497,8 @@ YAML
 
 create_file_with_content('config/sidekiq.yml', sidekiq_config_content, dry_run: options[:dry_run])
 
+# Permission validation method moved to top of file
+
 # Final instructions
 puts
 if options[:dry_run]

data/lib/code_healer/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module CodeHealer
-  VERSION = "0.1.5"
+  VERSION = "0.1.7"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: code_healer
 version: !ruby/object:Gem::Version
-  version: 0.1.5
+  version: 0.1.7
 platform: ruby
 authors:
 - Deepan Kumar