correct definition of iter_files

Author: d-biehl

robotcode/analyzer/__main__.py

@@ -1,7 +1,31 @@
-def main() -> None:
-    # TODO: implement commandline analyzer
-    pass
+import asyncio
+from pathlib import Path
+from typing import List
+
+from robotcode.language_server.robotframework.diagnostics.library_doc import (
+    RESOURCE_FILE_EXTENSION,
+    ROBOT_FILE_EXTENSION,
+)
+from robotcode.utils.glob_path import iter_files
+
+exclude_patterns: List[str] = [
+    "**/.git/**",
+    "**/node_modules/**",
+    "**/.pytest_cache/**",
+    "**/__pycache__/**",
+    "**/.mypy_cache/**",
+]
+
+
+async def main() -> None:
+    async for f in iter_files(
+        Path("c:/develop/robot/robotframework"),
+        f"**/*.{{{ROBOT_FILE_EXTENSION[1:]},{RESOURCE_FILE_EXTENSION[1:]}}}",
+        ignore_patterns=exclude_patterns,
+        absolute=True,
+    ):
+        print(f)
 
 
 if __name__ == "__main__":
-    main()
+    asyncio.run(main())

robotcode/language_server/robotframework/parts/robot_workspace.py

@@ -66,7 +66,7 @@ async def _load_workspace_documents(self, sender: Any) -> List[WorkspaceDocument
                     async for f in iter_files(
                         folder.uri.to_path(),
                         f"**/*.{{{ROBOT_FILE_EXTENSION[1:]},{RESOURCE_FILE_EXTENSION[1:]}}}",
-                        ignore_patterns=config.workspace.exclude_patterns or [],  # type: ignore
+                        ignore_patterns=config.workspace.exclude_patterns or [],
                         absolute=True,
                     )
                 ]

robotcode/utils/glob_path.py

@@ -3,7 +3,7 @@
 import os
 import re
 from pathlib import Path
-from typing import Any, AsyncGenerator, Iterable, List, Union, cast
+from typing import Any, AsyncGenerator, Iterable, Sequence, Union, cast
 
 
 def _glob_pattern_to_re(pattern: str) -> str:
@@ -84,8 +84,8 @@ def globmatches(pattern: str, path: Union[Path, str, os.PathLike[Any]]) -> bool:
 
 async def iter_files(
     path: Union[Path, str, os.PathLike[str]],
-    patterns: Union[List[Union[Pattern, str]], Pattern, str, None] = None,
-    ignore_patterns: Union[List[Union[Pattern, str]], Pattern, str, None] = None,
+    patterns: Union[Sequence[Union[Pattern, str]], Pattern, str, None] = None,
+    ignore_patterns: Union[Sequence[Union[Pattern, str]], Pattern, str, None] = None,
     *,
     absolute: bool = False,
     _base_path: Union[Path, str, os.PathLike[str], None] = None,
@@ -99,12 +99,12 @@ async def iter_files(
         if not isinstance(_base_path, Path):
             path = Path(_base_path)
 
-    if patterns is not None and not isinstance(patterns, (set, list, tuple)):
+    if patterns is not None and isinstance(patterns, (str, Pattern)):
         patterns = [patterns]
     if patterns is not None:
         patterns = list(map(lambda p: p if isinstance(p, Pattern) else Pattern(p), patterns))
 
-    if ignore_patterns is not None and not isinstance(ignore_patterns, (set, list, tuple)):
+    if ignore_patterns is not None and isinstance(ignore_patterns, (str, Pattern)):
         ignore_patterns = [ignore_patterns]
     if ignore_patterns is not None:
         ignore_patterns = list(map(lambda p: p if isinstance(p, Pattern) else Pattern(p), ignore_patterns))