feat: auto-fix issues during review phase

Enhanced the review phase to automatically fix issues:
- If review finds issues, spawn agent to fix them
- Run tests to verify fixes work
- Commit and push fixes if tests pass
- Re-review the PR after fixes
- Only block merge if auto-fix fails or cannot resolve issues

This makes evolve self-healing — it doesn't just block on
issues, it actively tries to resolve them before giving up.

Author: Patel230

internal/evolution/git.go

@@ -233,9 +233,99 @@ func (e *Engine) reviewPR(ctx context.Context, p iteragent.Provider, tools []ite
 		return nil
 	}
 
-	// Reviewer found issues but didn't say LGTM — block the merge.
-	e.logger.Warn("PR self-review did not pass — blocking merge")
-	return fmt.Errorf("review blocked merge: reviewer did not say LGTM")
+	// Reviewer found issues — try to auto-fix them
+	e.logger.Warn("PR self-review found issues — attempting auto-fix")
+
+	fixed, fixErr := e.autoFixIssues(ctx, p, tools, systemPrompt, skills, reviewOutput)
+	if fixErr != nil {
+		e.logger.Error("auto-fix failed", "err", fixErr)
+		e.postReviewComment(ctx, "Auto-fix attempt failed: "+fixErr.Error(), false)
+		return fmt.Errorf("review blocked merge: %w", fixErr)
+	}
+
+	if !fixed {
+		e.logger.Warn("auto-fix could not resolve all issues — blocking merge")
+		return fmt.Errorf("review blocked merge: auto-fix could not resolve all issues")
+	}
+
+	// Re-review after fixes
+	e.logger.Info("auto-fix applied — re-reviewing PR")
+	return e.reviewPR(ctx, p, tools, systemPrompt, skills)
+}
+
+// autoFixIssues attempts to fix issues identified during review.
+// Returns true if fixes were applied and tests pass, false otherwise.
+func (e *Engine) autoFixIssues(ctx context.Context, p iteragent.Provider, tools []iteragent.Tool, systemPrompt string, skills *iteragent.SkillSet, reviewOutput string) (bool, error) {
+	e.logger.Info("starting auto-fix based on review feedback")
+
+	// Build fix prompt with review feedback
+	fixPrompt := fmt.Sprintf(`The code review identified issues:
+
+%s
+
+Fix these issues in the codebase. Run tests to verify the fixes work.
+If you cannot fix an issue, explain why. Only commit if tests pass.`, reviewOutput)
+
+	a := e.newAgent(p, tools, systemPrompt, skills)
+	var fixResult string
+	for ev := range a.Prompt(ctx, fixPrompt) {
+		if e.eventSink != nil {
+			select {
+			case e.eventSink <- ev:
+			default:
+			}
+		}
+		if ev.Type == string(iteragent.EventMessageEnd) {
+			fixResult = ev.Content
+		}
+	}
+	a.Finish()
+
+	// Check if any changes were made
+	status, err := e.runTool(ctx, "bash", map[string]interface{}{
+		"cmd": "git status --porcelain",
+	})
+	if err != nil {
+		return false, fmt.Errorf("failed to check git status: %w", err)
+	}
+
+	if strings.TrimSpace(status) == "" {
+		e.logger.Info("no changes made during auto-fix")
+		return false, nil
+	}
+
+	// Run tests to verify fixes
+	e.logger.Info("running tests to verify auto-fix")
+	_, testErr := e.runTool(ctx, "bash", map[string]interface{}{
+		"cmd": "go test ./... 2>&1 | head -50",
+	})
+	if testErr != nil {
+		e.logger.Warn("tests failed after auto-fix, reverting changes")
+		// Revert failed fixes
+		e.runTool(ctx, "bash", map[string]interface{}{
+			"cmd": "git checkout -- .",
+		})
+		return false, fmt.Errorf("tests failed after auto-fix")
+	}
+
+	// Commit the fixes
+	e.logger.Info("committing auto-fix changes")
+	if _, err := e.runTool(ctx, "bash", map[string]interface{}{
+		"cmd": "git add -A && git commit -m 'fix: auto-fix issues from review'",
+	}); err != nil {
+		return false, fmt.Errorf("failed to commit auto-fix: %w", err)
+	}
+
+	// Push fixes to the PR branch
+	if _, err := e.runTool(ctx, "bash", map[string]interface{}{
+		"cmd": fmt.Sprintf("git push origin %s", e.branchName),
+	}); err != nil {
+		return false, fmt.Errorf("failed to push auto-fix: %w", err)
+	}
+
+	e.logger.Info("auto-fix applied successfully")
+	e.postReviewComment(ctx, fmt.Sprintf("Auto-fix applied:\n\n%s\n\nChanges committed and pushed.", fixResult), true)
+	return true, nil
 }
 
 // postReviewComment posts the reviewer's output as a GitHub PR comment.