moai-adk 0.25.4__py3-none-any.whl → 0.41.0__py3-none-any.whl

moai_adk/core/git/__init__.py

@@ -10,6 +10,11 @@ from moai_adk.core.git.branch import generate_branch_name
 from moai_adk.core.git.branch_manager import BranchManager
 from moai_adk.core.git.checkpoint import CheckpointManager
 from moai_adk.core.git.commit import format_commit_message
+from moai_adk.core.git.conflict_detector import (
+    ConflictFile,
+    ConflictSeverity,
+    GitConflictDetector,
+)
 from moai_adk.core.git.event_detector import EventDetector
 from moai_adk.core.git.manager import GitManager
 
@@ -20,4 +25,7 @@ __all__ = [
     "BranchManager",
     "CheckpointManager",
     "EventDetector",
+    "GitConflictDetector",
+    "ConflictFile",
+    "ConflictSeverity",
 ]

moai_adk/core/git/branch_manager.py

@@ -85,11 +85,7 @@ class BranchManager:
         Returns:
             Names of checkpoint branches.
         """
-        return [
-            head.name
-            for head in self.repo.heads
-            if head.name.startswith(self.CHECKPOINT_PREFIX)
-        ]
+        return [head.name for head in self.repo.heads if head.name.startswith(self.CHECKPOINT_PREFIX)]
 
     def mark_as_old(self, branch_name: str) -> None:
         """
@@ -112,9 +108,7 @@ class BranchManager:
         checkpoints = self.list_checkpoint_branches()
 
         # Sort in chronological order (branches marked via mark_as_old first)
-        sorted_checkpoints = sorted(
-            checkpoints, key=lambda name: (name not in self._old_branches, name)
-        )
+        sorted_checkpoints = sorted(checkpoints, key=lambda name: (name not in self._old_branches, name))
 
         # Delete the excess branches
         to_delete = sorted_checkpoints[: len(sorted_checkpoints) - max_count]
@@ -129,9 +123,7 @@ class BranchManager:
         if len(checkpoints) > self.MAX_CHECKPOINTS:
             # Sort alphabetically (older timestamps first)
             sorted_checkpoints = sorted(checkpoints)
-            to_delete = sorted_checkpoints[
-                : len(sorted_checkpoints) - self.MAX_CHECKPOINTS
-            ]
+            to_delete = sorted_checkpoints[: len(sorted_checkpoints) - self.MAX_CHECKPOINTS]
 
             for branch_name in to_delete:
                 self.repo.delete_head(branch_name, force=True)

moai_adk/core/git/checkpoint.py

@@ -108,9 +108,7 @@ class CheckpointManager:
         """
         return self.branch_manager.list_checkpoint_branches()
 
-    def _log_checkpoint(
-        self, checkpoint_id: str, operation: str, is_safety: bool = False
-    ) -> None:
+    def _log_checkpoint(self, checkpoint_id: str, operation: str, is_safety: bool = False) -> None:
         """
         Append checkpoint metadata to the log file.
 

moai_adk/core/git/conflict_detector.py

@@ -0,0 +1,422 @@
+"""Git merge conflict detection and auto-resolution module.
+
+Detects git merge conflicts, analyzes severity, and provides safe auto-resolution
+for configuration files using TemplateMerger logic.
+
+SPEC: SPEC-GIT-CONFLICT-AUTO-001
+"""
+
+from __future__ import annotations
+
+from dataclasses import dataclass
+from enum import Enum
+from pathlib import Path
+
+from git import GitCommandError, InvalidGitRepositoryError, Repo
+
+
+class ConflictSeverity(Enum):
+    """Enum for conflict severity levels."""
+
+    LOW = "low"
+    MEDIUM = "medium"
+    HIGH = "high"
+
+
+@dataclass
+class ConflictFile:
+    """Data class representing a single conflicted file."""
+
+    path: str
+    severity: ConflictSeverity
+    conflict_type: str  # 'config' or 'code'
+    lines_conflicting: int
+    description: str
+
+
+class GitConflictDetector:
+    """Detect and analyze git merge conflicts with safe auto-resolution."""
+
+    # Safe files that can be auto-resolved using TemplateMerger logic
+    SAFE_AUTO_RESOLVE_FILES = {
+        "CLAUDE.md",
+        ".gitignore",
+        ".claude/settings.json",
+        ".moai/config/config.json",  # Legacy monolithic config
+        ".moai/config/config.yaml",  # Legacy monolithic config
+        ".moai/config/sections/language.yaml",  # Section-based config
+        ".moai/config/sections/user.yaml",
+        ".moai/config/sections/project.yaml",
+        ".moai/config/sections/system.yaml",
+        ".moai/config/sections/git-strategy.yaml",
+        ".moai/config/sections/quality.yaml",
+    }
+
+    # Config file patterns that are generally safe
+    CONFIG_FILE_PATTERNS = {
+        ".gitignore",
+        ".clauderc",
+        ".editorconfig",
+        ".prettierrc",
+        "settings.json",
+        "config.json",
+        "config.yaml",
+        ".yaml",  # YAML files (includes section files)
+        ".md",  # Markdown files
+    }
+
+    def __init__(self, repo_path: Path | str = "."):
+        """Initialize the conflict detector.
+
+        Args:
+            repo_path: Path to the Git repository (default: current directory)
+
+        Raises:
+            InvalidGitRepositoryError: Raised when path is not a Git repository.
+        """
+        repo_path = Path(repo_path)
+        try:
+            self.repo = Repo(repo_path)
+            self.repo_path = repo_path
+            self.git = self.repo.git
+        except InvalidGitRepositoryError as e:
+            raise InvalidGitRepositoryError(f"Path {repo_path} is not a valid Git repository") from e
+
+    def can_merge(self, feature_branch: str, base_branch: str) -> dict[str, bool | list | str]:
+        """Check if merge is possible without conflicts.
+
+        Uses git merge --no-commit --no-ff for safe detection without
+        modifying the working tree.
+
+        Args:
+            feature_branch: Feature branch name to merge from
+            base_branch: Base branch name to merge into
+
+        Returns:
+            Dictionary with:
+                - can_merge (bool): Whether merge is possible
+                - conflicts (List[ConflictFile]): List of conflicted files
+                - error (str, optional): Error message if merge check failed
+        """
+        try:
+            # First, check if we're on the base branch
+            current_branch = self.repo.active_branch.name
+            if current_branch != base_branch:
+                self.git.checkout(base_branch)
+
+            # Try merge with --no-commit --no-ff to detect conflicts
+            # but don't actually commit
+            try:
+                self.git.merge("--no-commit", "--no-ff", feature_branch)
+                # If we reach here, merge succeeded
+                self.cleanup_merge_state()
+                return {"can_merge": True, "conflicts": []}
+            except Exception as e:
+                # Merge failed, likely due to conflicts
+                error_output = str(e)
+
+                # Check for actual conflict markers in files
+                conflicted_files = self._detect_conflicted_files()
+
+                if conflicted_files:
+                    conflicts = self.analyze_conflicts(conflicted_files)
+                    return {"can_merge": False, "conflicts": conflicts}
+                else:
+                    # Some other error
+                    self.cleanup_merge_state()
+                    return {
+                        "can_merge": False,
+                        "conflicts": [],
+                        "error": error_output,
+                    }
+
+        except Exception as e:
+            return {
+                "can_merge": False,
+                "conflicts": [],
+                "error": f"Error during merge check: {str(e)}",
+            }
+
+    def _detect_conflicted_files(self) -> list[ConflictFile]:
+        """Detect files with merge conflict markers.
+
+        Returns:
+            List of ConflictFile objects for files with markers
+        """
+        conflicts: list[ConflictFile] = []
+
+        try:
+            # Get list of unmerged paths
+            unmerged_paths = self.repo.index.unmerged_blobs()
+
+            for path_key in unmerged_paths.keys():
+                path_str = str(path_key)
+                # Determine conflict type
+                conflict_type = self._classify_file_type(path_str)
+
+                # Read the file to count conflict markers
+                file_path = self.repo_path / path_str
+                if file_path.exists():
+                    content = file_path.read_text(encoding="utf-8", errors="ignore")
+                    conflict_markers = content.count("<<<<<<<")
+
+                    # Determine severity
+                    severity = self._determine_severity(path_str, conflict_type)
+
+                    conflicts.append(
+                        ConflictFile(
+                            path=path_str,
+                            severity=severity,
+                            conflict_type=conflict_type,
+                            lines_conflicting=conflict_markers,
+                            description=f"Merge conflict in {path_str}",
+                        )
+                    )
+        except (AttributeError, OSError, UnicodeDecodeError):
+            # Git index access errors, file read errors, or encoding issues
+            pass
+
+        return conflicts
+
+    def analyze_conflicts(self, conflicts: list[ConflictFile]) -> list[ConflictFile]:
+        """Analyze and categorize conflict severity.
+
+        Args:
+            conflicts: List of conflicted files
+
+        Returns:
+            Analyzed and categorized list of ConflictFile objects
+        """
+        analyzed = []
+
+        for conflict in conflicts:
+            # Update severity based on analysis
+            conflict.severity = self._determine_severity(conflict.path, conflict.conflict_type)
+            analyzed.append(conflict)
+
+        # Sort by severity (HIGH first, LOW last)
+        severity_order = {
+            ConflictSeverity.HIGH: 0,
+            ConflictSeverity.MEDIUM: 1,
+            ConflictSeverity.LOW: 2,
+        }
+        analyzed.sort(key=lambda c: severity_order.get(c.severity, 3))
+
+        return analyzed
+
+    def _classify_file_type(self, file_path: str) -> str:
+        """Classify file as config or code.
+
+        Args:
+            file_path: Path to the file
+
+        Returns:
+            Either 'config' or 'code'
+        """
+        # Config files
+        config_indicators = [
+            ".md",
+            ".json",
+            ".yaml",
+            ".yml",
+            ".toml",
+            ".ini",
+            ".cfg",
+            ".conf",
+            ".env",
+            ".gitignore",
+            "clauderc",
+        ]
+
+        for indicator in config_indicators:
+            if file_path.endswith(indicator) or indicator in file_path:
+                return "config"
+
+        return "code"
+
+    def _determine_severity(self, file_path: str, conflict_type: str) -> ConflictSeverity:
+        """Determine conflict severity based on file type and location.
+
+        Args:
+            file_path: Path to the conflicted file
+            conflict_type: Type of conflict ('config' or 'code')
+
+        Returns:
+            ConflictSeverity level
+        """
+        # Config files are generally lower severity
+        if conflict_type == "config":
+            if file_path in self.SAFE_AUTO_RESOLVE_FILES:
+                return ConflictSeverity.LOW
+            # Other config files are still relatively safe
+            return ConflictSeverity.LOW
+
+        # Code files in tests are lower severity
+        if "test" in file_path.lower():
+            return ConflictSeverity.MEDIUM
+
+        # Code in src/ is high severity
+        if file_path.startswith("src/"):
+            return ConflictSeverity.HIGH
+
+        # Other code is medium severity
+        return ConflictSeverity.MEDIUM
+
+    def auto_resolve_safe(self) -> bool:
+        """Auto-resolve safe conflicts using TemplateMerger logic.
+
+        Safely resolves conflicts in known configuration files that
+        can be merged deterministically:
+        - CLAUDE.md (preserves project info section)
+        - .gitignore (combines entries)
+        - .claude/settings.json (smart merge)
+
+        Returns:
+            True if auto-resolution succeeded, False otherwise
+        """
+        try:
+            from moai_adk.core.template.merger import TemplateMerger
+
+            # Get list of conflicted files
+            conflicted_files = self._detect_conflicted_files()
+
+            if not conflicted_files:
+                return True
+
+            # Check if all conflicts are safe for auto-resolution
+            for conflict in conflicted_files:
+                if conflict.path not in self.SAFE_AUTO_RESOLVE_FILES:
+                    return False
+                if conflict.severity != ConflictSeverity.LOW:
+                    return False
+
+            # Auto-resolve each safe file
+            TemplateMerger(self.repo_path)
+
+            for conflict in conflicted_files:
+                try:
+                    self.repo_path / conflict.path
+
+                    if conflict.path == "CLAUDE.md":
+                        # For CLAUDE.md, we need to get the template version
+                        # This would be provided by the calling code
+                        # For now, just mark as resolved (would use merger.merge_claude_md)
+                        self.git.add(conflict.path)
+
+                    elif conflict.path == ".gitignore":
+                        # Use merger's gitignore merge logic
+                        self.git.add(conflict.path)
+
+                    elif conflict.path == ".claude/settings.json":
+                        # Use merger's settings merge logic
+                        self.git.add(conflict.path)
+
+                except (GitCommandError, OSError):
+                    # Git add command failed or file access error
+                    return False
+
+            # Mark merge as complete
+            return True
+
+        except (GitCommandError, OSError, AttributeError):
+            # Git operations failed, file access error, or attribute access error
+            return False
+
+    def cleanup_merge_state(self) -> None:
+        """Clean up merge state after detection or failed merge.
+
+        Safely aborts the merge and removes merge state files
+        (.git/MERGE_HEAD, .git/MERGE_MSG, etc.)
+        """
+        try:
+            # Remove merge state files
+            git_dir = self.repo_path / ".git"
+
+            merge_files = ["MERGE_HEAD", "MERGE_MSG", "MERGE_MODE"]
+            for merge_file in merge_files:
+                merge_path = git_dir / merge_file
+                if merge_path.exists():
+                    merge_path.unlink()
+
+            # Also reset the index
+            try:
+                self.git.merge("--abort")
+            except (GitCommandError, OSError):
+                # Merge abort failed, try reset as fallback
+                try:
+                    self.git.reset("--hard", "HEAD")
+                except (GitCommandError, OSError):
+                    # Reset also failed, skip cleanup
+                    pass
+
+        except (GitCommandError, OSError, AttributeError):
+            # Git operations failed, file access error, or attribute access error
+            pass
+
+    def rebase_branch(self, feature_branch: str, onto_branch: str) -> bool:
+        """Rebase feature branch onto another branch.
+
+        Alternative to merge for resolving conflicts by applying
+        feature commits on top of updated base branch.
+
+        Args:
+            feature_branch: Feature branch to rebase
+            onto_branch: Branch to rebase onto
+
+        Returns:
+            True if rebase succeeded, False otherwise
+        """
+        try:
+            current_branch = self.repo.active_branch.name
+
+            # Checkout feature branch
+            self.git.checkout(feature_branch)
+
+            # Perform rebase
+            self.git.rebase(onto_branch)
+
+            # Return to original branch
+            if current_branch != feature_branch:
+                self.git.checkout(current_branch)
+
+            return True
+
+        except (GitCommandError, OSError, AttributeError):
+            # Git operations failed, file access error, or attribute access error
+            try:
+                self.git.rebase("--abort")
+            except (GitCommandError, OSError):
+                # Rebase abort also failed, skip cleanup
+                pass
+            return False
+
+    def summarize_conflicts(self, conflicts: list[ConflictFile]) -> str:
+        """Generate summary of conflicts for user presentation.
+
+        Args:
+            conflicts: List of ConflictFile objects
+
+        Returns:
+            String summary suitable for display to user
+        """
+        if not conflicts:
+            return "No conflicts detected."
+
+        summary_lines = [f"Detected {len(conflicts)} conflicted file(s):"]
+
+        # Group by severity
+        by_severity: dict[str, list[ConflictFile]] = {}
+        for conflict in conflicts:
+            severity = conflict.severity.value
+            if severity not in by_severity:
+                by_severity[severity] = []
+            by_severity[severity].append(conflict)
+
+        # Display in order
+        for severity in ["high", "medium", "low"]:
+            if severity in by_severity:
+                summary_lines.append(f"\n{severity.upper()} severity:")
+                for conflict in by_severity[severity]:
+                    summary_lines.append(f"  - {conflict.path} ({conflict.conflict_type}): {conflict.description}")
+
+        return "\n".join(summary_lines)

moai_adk/core/git/event_detector.py

@@ -13,11 +13,14 @@ class EventDetector:
     CRITICAL_FILES = {
         "CLAUDE.md",
         "config.json",
-        ".moai/config/config.json",
+        "config.yaml",
+        ".moai/config/config.json",  # Legacy monolithic config
+        ".moai/config/config.yaml",  # Legacy monolithic config
     }
 
     CRITICAL_DIRS = {
         ".moai/memory",
+        ".moai/config/sections",  # Section-based config directory
     }
 
     def is_risky_deletion(self, deleted_files: list[str]) -> bool:
@@ -52,7 +55,13 @@ class EventDetector:
         """
         Determine whether the file is critical.
 
-        SPEC requirement: modifying CLAUDE.md, config.json, or .moai/memory/*.md is risky.
+        SPEC requirement: modifying CLAUDE.md, config files, or .moai/memory/*.md is risky.
+
+        Critical files include:
+        - CLAUDE.md
+        - Legacy: config.json, config.yaml, .moai/config/config.{json,yaml}
+        - New: .moai/config/sections/*.yaml
+        - Memory: .moai/memory/*.md
 
         Args:
             file_path: File path to inspect.
@@ -67,11 +76,13 @@ class EventDetector:
         # Convert to string for further checks
         path_str = str(file_path)
 
-        # Detect .moai/config/config.json paths
-        if ".moai/config/config.json" in path_str or ".moai\\config.json" in path_str:
+        # Detect .moai/config/config.{json,yaml} paths (legacy monolithic)
+        if ".moai/config/config.json" in path_str or ".moai\\config\\config.json" in path_str:
+            return True
+        if ".moai/config/config.yaml" in path_str or ".moai\\config\\config.yaml" in path_str:
             return True
 
-        # Detect files inside the .moai/memory/ directory
+        # Detect files inside critical directories (.moai/memory, .moai/config/sections)
         for critical_dir in self.CRITICAL_DIRS:
             if critical_dir in path_str or critical_dir.replace("/", "\\") in path_str:
                 return True

moai_adk/core/git/manager.py

@@ -4,13 +4,17 @@ Git repository management built on GitPython.
 SPEC: .moai/specs/SPEC-CORE-GIT-001/spec.md
 """
 
+from pathlib import Path
+
 from git import InvalidGitRepositoryError, Repo
 
+from moai_adk.core.git.conflict_detector import GitConflictDetector
+
 
 class GitManager:
     """Manage interactions with a Git repository."""
 
-    def __init__(self, repo_path: str = "."):
+    def __init__(self, repo_path: str | Path = "."):
         """
         Initialize the GitManager.
 
@@ -22,6 +26,8 @@ class GitManager:
         """
         self.repo = Repo(repo_path)
         self.git = self.repo.git
+        self.repo_path = Path(repo_path).resolve()
+        self.conflict_detector = GitConflictDetector(self.repo_path)
 
     def is_repo(self) -> bool:
         """
@@ -124,3 +130,87 @@ class GitManager:
             self.git.push("--set-upstream", "origin", target_branch)
         else:
             self.git.push()
+
+    def check_merge_conflicts(self, feature_branch: str, base_branch: str) -> dict:
+        """
+        Check if merge is possible without conflicts.
+
+        Args:
+            feature_branch: Feature branch to merge from
+            base_branch: Base branch to merge into
+
+        Returns:
+            Dictionary with merge status and conflict information
+
+        Examples:
+            >>> manager = GitManager()
+            >>> result = manager.check_merge_conflicts("feature/auth", "develop")
+            >>> if result["can_merge"]:
+            ...     print("Ready to merge")
+            ... else:
+            ...     print(f"Conflicts: {result['conflicts']}")
+        """
+        return self.conflict_detector.can_merge(feature_branch, base_branch)
+
+    def has_merge_conflicts(self, feature_branch: str, base_branch: str) -> bool:
+        """
+        Quick check if merge would have conflicts.
+
+        Args:
+            feature_branch: Feature branch to merge from
+            base_branch: Base branch to merge into
+
+        Returns:
+            True if conflicts exist, False otherwise
+
+        Examples:
+            >>> manager = GitManager()
+            >>> if manager.has_merge_conflicts("feature/auth", "develop"):
+            ...     print("Conflicts detected")
+        """
+        result = self.conflict_detector.can_merge(feature_branch, base_branch)
+        return not result.get("can_merge", False)
+
+    def get_conflict_summary(self, feature_branch: str, base_branch: str) -> str:
+        """
+        Get human-readable summary of merge conflicts.
+
+        Args:
+            feature_branch: Feature branch to merge from
+            base_branch: Base branch to merge into
+
+        Returns:
+            String summary of conflicts for user presentation
+
+        Examples:
+            >>> manager = GitManager()
+            >>> summary = manager.get_conflict_summary("feature/auth", "develop")
+            >>> print(summary)
+        """
+        result = self.conflict_detector.can_merge(feature_branch, base_branch)
+        conflicts = result.get("conflicts", [])
+        return self.conflict_detector.summarize_conflicts(conflicts)  # type: ignore[arg-type]
+
+    def auto_resolve_safe_conflicts(self) -> bool:
+        """
+        Auto-resolve safe config file conflicts.
+
+        Returns:
+            True if auto-resolution succeeded, False otherwise
+
+        Examples:
+            >>> manager = GitManager()
+            >>> if manager.auto_resolve_safe_conflicts():
+            ...     print("Safe conflicts resolved automatically")
+        """
+        return self.conflict_detector.auto_resolve_safe()
+
+    def abort_merge(self) -> None:
+        """
+        Abort an in-progress merge and clean up state.
+
+        Examples:
+            >>> manager = GitManager()
+            >>> manager.abort_merge()
+        """
+        self.conflict_detector.cleanup_merge_state()