Error Handling for Inaccessible Directories

Author: Artemonim

CHANGELOG.md

@@ -33,6 +33,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ### Fixed
 
+-   **Error Handling for Inaccessible Directories**: Fixed a crash (`PermissionError`) that occurred when scanning directories with restricted read permissions. The application will now skip such directories and print a warning, preventing unintended modifications to files that might have been excluded by an unreadable `.gitignore` or other configuration files.
 -   **Deleting empty lines**: Detected and fixed the removal of empty lines at the end of processed files
 -   **Language-Specific Indentation**: Fixed the indentation in the generated 'Table of Contents' to respect common style conventions for each language (e.g., 4 spaces for Python, 2 for JavaScript).
 

agent_docstrings/core.py

@@ -69,6 +69,8 @@ def parse_gitignore(gitignore_path: Path) -> Set[str]:
                 line = line.strip()
                 if line and not line.startswith("#"):
                     patterns.add(line)
+    except PermissionError:
+        raise
     except Exception:
         pass
     return patterns
@@ -118,6 +120,8 @@ def load_blacklist_whitelist(directory: Path) -> Tuple[Set[str], Set[str]]:
                     for line in f
                     if line.strip() and not line.startswith("#")
                 )
+        except PermissionError:
+            raise
         except Exception:
             pass
     if whitelist_file.exists():
@@ -128,6 +132,8 @@ def load_blacklist_whitelist(directory: Path) -> Tuple[Set[str], Set[str]]:
                     for line in f
                     if line.strip() and not line.startswith("#")
                 )
+        except PermissionError:
+            raise
         except Exception:
             pass
     return blacklist, whitelist
@@ -483,41 +489,45 @@ def discover_and_process_files(paths: List[str], verbose: bool = False, beta: bo
     files_to_process = []
 
     for p_str in paths:
-        path = Path(p_str).resolve()
-        if not path.exists():
-            print(f"Warning: '{p_str}' is not a valid path. Skipping.")
-            continue
-            
-        if path.is_dir():
-            # Collect all gitignore patterns from the directory tree
-            ignore_patterns = set()
-            current_dir = path
-            while current_dir != current_dir.parent:
-                gitignore_path = current_dir / '.gitignore'
-                if gitignore_path.exists():
-                    ignore_patterns.update(parse_gitignore(gitignore_path))
-                current_dir = current_dir.parent
-            
-            # Load blacklist and whitelist from the root directory
-            blacklist_patterns, whitelist_patterns = load_blacklist_whitelist(path)
-
-            for root, dirs, files in os.walk(path):
-                root_path = Path(root)
+        try:
+            path = Path(p_str).resolve()
+            if not path.exists():
+                print(f"Warning: '{p_str}' is not a valid path. Skipping.")
+                continue
 
-                # Filter directories to avoid walking into ignored ones
-                dirs[:] = [d for d in dirs if d not in DEFAULT_IGNORE_DIRS and not is_path_ignored(root_path / d, ignore_patterns, path)]
+            if path.is_dir():
+                # Collect all gitignore patterns from the directory tree
+                ignore_patterns = set()
+                current_dir = path
+                while current_dir != current_dir.parent:
+                    gitignore_path = current_dir / '.gitignore'
+                    if gitignore_path.exists():
+                        ignore_patterns.update(parse_gitignore(gitignore_path))
+                    current_dir = current_dir.parent
 
-                for file in files:
-                    file_path = root_path / file
+                # Load blacklist and whitelist from the root directory
+                blacklist_patterns, whitelist_patterns = load_blacklist_whitelist(path)
+
+                for root, dirs, files in os.walk(path):
+                    root_path = Path(root)
 
-                    # Check if file should be processed
-                    if not should_process_file(file_path, path, ignore_patterns, 
-                                             blacklist_patterns, whitelist_patterns):
-                        continue
+                    # Filter directories to avoid walking into ignored ones
+                    dirs[:] = [d for d in dirs if d not in DEFAULT_IGNORE_DIRS and not is_path_ignored(root_path / d, ignore_patterns, path)]
 
-                    files_to_process.append(file_path)
-        elif path.is_file():
-            files_to_process.append(path)
+                    for file in files:
+                        file_path = root_path / file
+                        
+                        # Check if file should be processed
+                        if not should_process_file(file_path, path, ignore_patterns, 
+                                                 blacklist_patterns, whitelist_patterns):
+                            continue
+                        
+                        files_to_process.append(file_path)
+            elif path.is_file():
+                files_to_process.append(path)
+        except PermissionError:
+            print(f"Warning: Could not read configuration (e.g., .gitignore) in '{p_str}' due to a permission error. Skipping path to ensure no unintended files are modified.")
+            continue
 
     # Process all collected files
     for file_path in sorted(list(set(files_to_process))):