robotcode-robot 0.96.0__tar.gz → 0.97.0__tar.gz

{robotcode_robot-0.96.0 → robotcode_robot-0.97.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.3
 Name: robotcode-robot
-Version: 0.96.0
+Version: 0.97.0
 Summary: Support classes for RobotCode for handling Robot Framework projects.
 Project-URL: Homepage, https://robotcode.io
 Project-URL: Donate, https://opencollective.com/robotcode
@@ -10,7 +10,6 @@ Project-URL: Issues, https://github.com/robotcodedev/robotcode/issues
 Project-URL: Source, https://github.com/robotcodedev/robotcode
 Author-email: Daniel Biehl <dbiehl@live.de>
 License: Apache-2.0
-License-File: LICENSE.txt
 Classifier: Development Status :: 5 - Production/Stable
 Classifier: Framework :: Robot Framework
 Classifier: Framework :: Robot Framework :: Tool
@@ -26,7 +25,7 @@ Classifier: Topic :: Utilities
 Classifier: Typing :: Typed
 Requires-Python: >=3.8
 Requires-Dist: platformdirs<4.4.0,>=3.2.0
-Requires-Dist: robotcode-core==0.96.0
+Requires-Dist: robotcode-core==0.97.0
 Requires-Dist: robotframework>=4.1.0
 Requires-Dist: tomli>=1.1.0; python_version < '3.11'
 Description-Content-Type: text/markdown

{robotcode_robot-0.96.0 → robotcode_robot-0.97.0}/pyproject.toml

@@ -30,7 +30,7 @@ dependencies = [
   "robotframework>=4.1.0",
   "tomli>=1.1.0; python_version < '3.11'",
   "platformdirs>=3.2.0,<4.4.0",
-  "robotcode-core==0.96.0",
+  "robotcode-core==0.97.0",
 ]
 dynamic = ["version"]
 

robotcode_robot-0.97.0/src/robotcode/robot/__version__.py

@@ -0,0 +1 @@
+__version__ = "0.97.0"

{robotcode_robot-0.96.0 → robotcode_robot-0.97.0}/src/robotcode/robot/config/loader.py

@@ -55,14 +55,36 @@ class ConfigTypeError(TypeError):
 _ConfigType = TypeVar("_ConfigType", bound=BaseOptions)
 
 
+class InvalidTomlError(ValueError):
+    pass
+
+
+class InvalidTomlFileError(ValueError):
+    pass
+
+
+def _load_toml(data: Union[str, Path]) -> Dict[str, Any]:
+    file = None
+    if isinstance(data, Path):
+        file = data
+        data = data.read_text("utf-8")
+
+    try:
+        return tomllib.loads(data)
+    except tomllib.TOMLDecodeError as e:
+        if file:
+            raise InvalidTomlFileError(f"Invalid TOML file'{file.absolute()}': {e}") from e
+        raise InvalidTomlError(f"Invalid TOML: {e}") from e
+
+
 def load_robot_config_from_robot_toml_str(__s: str) -> RobotConfig:
     return load_config_from_robot_toml_str(RobotConfig, __s)
 
 
 def load_config_from_robot_toml_str(
-    config_type: Type[_ConfigType], data: Union[str, Dict[str, Any]], tool_name: Optional[str] = None
+    config_type: Type[_ConfigType], data: Union[str, Dict[str, Any], Path], tool_name: Optional[str] = None
 ) -> _ConfigType:
-    dict_data = tomllib.loads(data) if isinstance(data, str) else data
+    dict_data = _load_toml(data) if isinstance(data, (str, Path)) else data
 
     if tool_name:
         try:
@@ -76,9 +98,9 @@ def load_config_from_robot_toml_str(
 
 
 def load_config_from_pyproject_toml_str(
-    config_type: Type[_ConfigType], tool_name: str, data: Union[str, Dict[str, Any]]
+    config_type: Type[_ConfigType], tool_name: str, data: Union[str, Dict[str, Any], Path]
 ) -> _ConfigType:
-    dict_data = tomllib.loads(data) if isinstance(data, str) else data
+    dict_data = _load_toml(data) if isinstance(data, (str, Path)) else data
 
     return from_dict(dict_data.get("tool", {}).get(tool_name, {}), config_type)
 
@@ -93,13 +115,13 @@ def _load_config_data_from_path(
     try:
         if path.name == PYPROJECT_TOML:
             return load_config_from_pyproject_toml_str(
-                config_type, pyproject_toml_tool_name, path.read_text("utf-8") if data is None else data
+                config_type, pyproject_toml_tool_name, path if data is None else data
             )
 
         if path.name == ROBOT_TOML or path.name == LOCAL_ROBOT_TOML or path.suffix == ".toml":
             return load_config_from_robot_toml_str(
                 config_type,
-                path.read_text("utf-8") if data is None else data,
+                path if data is None else data,
                 tool_name=robot_toml_tool_name,
             )
         raise TypeError("Unknown config file type.")
@@ -143,7 +165,7 @@ def load_config_from_path(
             verbose_callback(f"Load configuration from {__path if isinstance(__path, Path) else __path[0]}")
 
         p = __path if isinstance(__path, Path) else __path[0]
-        data = tomllib.loads(p.read_text("utf-8"))
+        data = _load_toml(p)
 
         result.add_options(
             _load_config_data_from_path(

{robotcode_robot-0.96.0 → robotcode_robot-0.97.0}/src/robotcode/robot/diagnostics/imports_manager.py

@@ -25,6 +25,7 @@ from typing import (
     final,
 )
 
+from robot.errors import RobotError
 from robot.libraries import STDLIBS
 from robot.utils.text import split_args_from_name_or_path
 from robotcode.core.concurrent import RLock, run_as_task
@@ -32,7 +33,16 @@ from robotcode.core.documents_manager import DocumentsManager
 from robotcode.core.event import event
 from robotcode.core.filewatcher import FileWatcherEntry, FileWatcherManagerBase, FileWatcherManagerDummy
 from robotcode.core.language import language_id
-from robotcode.core.lsp.types import DocumentUri, FileChangeType, FileEvent
+from robotcode.core.lsp.types import (
+    Diagnostic,
+    DiagnosticRelatedInformation,
+    DiagnosticSeverity,
+    DocumentUri,
+    FileChangeType,
+    FileEvent,
+    Location,
+    Range,
+)
 from robotcode.core.text_document import TextDocument
 from robotcode.core.uri import Uri
 from robotcode.core.utils.caching import SimpleLRUCache
@@ -577,6 +587,8 @@ class ImportsManager:
             weakref.WeakKeyDictionary()
         )
 
+        self._diagnostics: List[Diagnostic] = []
+
     def __del__(self) -> None:
         try:
             if self._executor is not None:
@@ -584,6 +596,12 @@ class ImportsManager:
         except RuntimeError:
             pass
 
+    @property
+    def diagnostics(self) -> List[Diagnostic]:
+        self.get_command_line_variables()
+
+        return self._diagnostics
+
     @property
     def environment(self) -> Mapping[str, str]:
         return self._environment
@@ -647,18 +665,54 @@ class ImportsManager:
                             ]
 
                             if lib_doc.errors:
-                                # TODO add diagnostics
                                 for error in lib_doc.errors:
-                                    self._logger.error(
-                                        lambda: f"{error.type_name}: {error.message} in {error.source}:{error.line_no}"
+                                    self._diagnostics.append(
+                                        Diagnostic(
+                                            Range.zero(),
+                                            f"Processing variable file variable file '{name}({', '.join(args)})' failed"
+                                            + ("" if error.source is not None else f": {error.message}"),
+                                            DiagnosticSeverity.ERROR,
+                                            error.type_name,
+                                            related_information=(
+                                                [
+                                                    DiagnosticRelatedInformation(
+                                                        Location(
+                                                            str(Uri.from_path(os.path.abspath(error.source))),
+                                                            Range.from_int_range(
+                                                                (error.line_no - 1) if error.line_no is not None else -1
+                                                            ),
+                                                        ),
+                                                        error.message,
+                                                    )
+                                                ]
+                                                if error.source is not None
+                                                else None
+                                            ),
+                                        )
                                     )
+                    except RobotError as e:
+                        self._diagnostics.append(
+                            Diagnostic(
+                                Range.zero(),
+                                f"Error in command line variable file '{name}({', '.join(args)})': {e}",
+                                DiagnosticSeverity.ERROR,
+                                type(e).__name__,
+                            )
+                        )
                     except (SystemExit, KeyboardInterrupt):
                         raise
                     except BaseException as e:
-                        # TODO add diagnostics
+                        self._diagnostics.append(
+                            Diagnostic(
+                                Range.zero(),
+                                f"Error in command line variable file '{name}({', '.join(args)})': {e}",
+                                DiagnosticSeverity.ERROR,
+                                type(e).__name__,
+                            )
+                        )
                         ex = e
                         self._logger.exception(
-                            lambda: f"Error getting command line variables: {ex}",
+                            lambda: f"Error in command line variable file '{name}({', '.join(args)})'",
                             exc_info=ex,
                         )
 

{robotcode_robot-0.96.0 → robotcode_robot-0.97.0}/src/robotcode/robot/diagnostics/workspace_config.py

@@ -75,6 +75,7 @@ class WorkspaceConfig(ConfigBase):
 
 @dataclass
 class WorkspaceAnalysisConfig:
+    exclude_patterns: List[str] = field(default_factory=list)
     cache: CacheConfig = field(default_factory=CacheConfig)
     robot: AnalysisRobotConfig = field(default_factory=AnalysisRobotConfig)
     modifiers: AnalysisDiagnosticModifiersConfig = field(default_factory=AnalysisDiagnosticModifiersConfig)

robotcode_robot-0.96.0/src/robotcode/robot/__version__.py

@@ -1 +0,0 @@
-__version__ = "0.96.0"