fix: improve gitignore pattern handling and add configurable respect option

- Remove early return in repo_analyzer to allow default ignore patterns to be checked
- Add respect_gitignore parameter to DependencyParser constructor for configurable gitignore handling
- Update DependencyGraphBuilder to pass respect_gitignore configuration to parser
- Enhance test coverage to verify gitignore behavior when both enabled and disabled
- Modify test patterns to avoid conflicts with default ignore patterns for more accurate testing

Author: ShangHong-CAI

codewiki/src/be/dependency_analyzer/analysis/repo_analyzer.py

@@ -185,7 +185,7 @@ def _should_exclude_path(self, path: str, filename: str) -> bool:
                         return True
                     if pattern in path.split("/"):
                         return True
-                return False
+                # Allow default ignore patterns to be checked (remove early return)
 
             if (
                 git_ignored is None

codewiki/src/be/dependency_analyzer/ast_parser.py

@@ -18,21 +18,22 @@
 class DependencyParser:
     """Parser for extracting code components from multi-language repositories."""
 
-    def __init__(self, repo_path: str, include_patterns: List[str] = None, exclude_patterns: List[str] = None):
+    def __init__(self, repo_path: str, include_patterns: List[str] = None, exclude_patterns: List[str] = None,respect_gitignore: bool = False):
         """
         Initialize the dependency parser.
         
         Args:
             repo_path: Path to the repository
             include_patterns: File patterns to include (e.g., ["*.cs", "*.py"])
             exclude_patterns: File/directory patterns to exclude (e.g., ["*Tests*"])
+            respect_gitignore: Whether to respect .gitignore patterns
         """
         self.repo_path = os.path.abspath(repo_path)
         self.components: Dict[str, Node] = {}
         self.modules: Set[str] = set()
         self.include_patterns = include_patterns
         self.exclude_patterns = exclude_patterns
-        
+        self.respect_gitignore = respect_gitignore
         self.analysis_service = AnalysisService()
 
     def parse_repository(self, filtered_folders: List[str] = None) -> Dict[str, Node]:
@@ -43,11 +44,14 @@ def parse_repository(self, filtered_folders: List[str] = None) -> Dict[str, Node
             logger.info(f"Using custom include patterns: {self.include_patterns}")
         if self.exclude_patterns:
             logger.info(f"Using custom exclude patterns: {self.exclude_patterns}")
-        
+        if self.respect_gitignore:
+            logger.info(f"Respecting .gitignore patterns")
+
         structure_result = self.analysis_service._analyze_structure(
-            self.repo_path, 
+            self.repo_path,
             include_patterns=self.include_patterns,
-            exclude_patterns=self.exclude_patterns
+            exclude_patterns=self.exclude_patterns,
+            respect_gitignore=self.respect_gitignore,
         )
 
         call_graph_result = self.analysis_service._analyze_call_graph(

codewiki/src/be/dependency_analyzer/dependency_graphs_builder.py

@@ -44,7 +44,8 @@ def build_dependency_graph(self) -> tuple[Dict[str, Any], List[str]]:
         parser = DependencyParser(
             self.config.repo_path,
             include_patterns=include_patterns,
-            exclude_patterns=exclude_patterns
+            exclude_patterns=exclude_patterns,
+            respect_gitignore=self.config.respect_gitignore,
         )
 
         filtered_folders = None

tests/test_gitignore_verification.py

@@ -24,27 +24,24 @@ def main() -> None:
 
     try:
         subprocess.run([git_cmd, "init", "-q"], cwd=temp_dir, check=True)
-        (temp_dir / ".gitignore").write_text("node_modules/\n*.log\n!important.log")
+        # Use *.txt files instead of *.log since *.log is in DEFAULT_IGNORE_PATTERNS
+        (temp_dir / ".gitignore").write_text("node_modules/\n*.txt\nbackend/config.ini")
         (temp_dir / "backend").mkdir()
         (temp_dir / "backend" / ".gitignore").write_text("secrets.py")
 
-        (temp_dir / "app.log").touch()
-        (temp_dir / "important.log").touch()
-        (temp_dir / "backend" / "secrets.py").touch()
-        (temp_dir / "backend" / "api.py").touch()
-        (temp_dir / "force_exclude.py").touch()
-
         print("-" * 60)
-        print("TEST 1: Gitignore Logic (Negation & Nested)")
+        print("TEST 1: Gitignore Logic (Basic & Nested) with respect_gitignore=True")
         analyzer = RepoAnalyzer(respect_gitignore=True, repo_path=str(temp_dir))
 
-        check1 = analyzer._should_exclude_path("app.log", "app.log") is True
-        check2 = analyzer._should_exclude_path("important.log", "important.log") is False
+        check1 = analyzer._should_exclude_path("notes.txt", "notes.txt") is True
+        check2 = analyzer._should_exclude_path("readme.txt", "readme.txt") is True
         check3 = analyzer._should_exclude_path("backend/secrets.py", "secrets.py") is True
+        check4 = analyzer._should_exclude_path("backend/config.ini", "config.ini") is True
 
-        print(f"  [{'✅' if check1 else '❌'}] Basic pattern (*.log)")
-        print(f"  [{'✅' if check2 else '❌'}] Negation pattern (!important.log)")
+        print(f"  [{'✅' if check1 else '❌'}] Basic pattern (*.txt)")
+        print(f"  [{'✅' if check2 else '❌'}] Another txt file (*.txt)")
         print(f"  [{'✅' if check3 else '❌'}] Nested .gitignore")
+        print(f"  [{'✅' if check4 else '❌'}] Path pattern (backend/config.ini)")
 
         print("\nTEST 2: Priority Logic (CLI Override > Git)")
         analyzer_override = RepoAnalyzer(
@@ -55,7 +52,20 @@ def main() -> None:
         )
         print(f"  [{'✅' if check4 else '❌'}] CLI --exclude overrides Git tracking")
 
-        if all([check1, check2, check3, check4]):
+        print("\nTEST 3: Gitignore Disabled (respect_gitignore=False)")
+        analyzer_no_gitignore = RepoAnalyzer(respect_gitignore=False, repo_path=str(temp_dir))
+
+        check5 = analyzer_no_gitignore._should_exclude_path("notes.txt", "notes.txt") is False
+        check6 = analyzer_no_gitignore._should_exclude_path("readme.txt", "readme.txt") is False
+        check7 = (
+            analyzer_no_gitignore._should_exclude_path("backend/secrets.py", "secrets.py") is False
+        )
+
+        print(f"  [{'✅' if check5 else '❌'}] *.txt files NOT excluded by gitignore")
+        print(f"  [{'✅' if check6 else '❌'}] readme.txt NOT excluded by gitignore")
+        print(f"  [{'✅' if check7 else '❌'}] nested .gitignore NOT respected")
+
+        if all([check1, check2, check3, check4, check5, check6, check7]):
             print("\n✨ ALL CHECKS PASSED")
             sys.exit(0)
         else: