bump ruff and mypy (#733)

Co-authored-by: lukaspie <lukaspie@github.com>

Author: lukaspie

.pre-commit-config.yaml

@@ -1,14 +1,14 @@
 repos:
   - repo: https://github.com/astral-sh/ruff-pre-commit
-    rev: v0.14.0
+    rev: v0.15.0
     hooks:
       - id: ruff-check
         files: ^(src/pynxtools|tests)/.*\.py$
       - id: ruff-format
         files: ^(src/pynxtools|tests)/.*\.py$
 
   - repo: https://github.com/pre-commit/mirrors-mypy
-    rev: v1.18.1
+    rev: v1.19.1
     hooks:
       - id: mypy  # static type checking
 

pyproject.toml

@@ -58,7 +58,7 @@ docs = [
 ]
 dev = [
     "mypy",
-    "ruff>=0.14.0",
+    "ruff>=0.15.0",
     "pytest",
     "pytest-timeout",
     "pytest-cov",

src/pynxtools/dataconverter/helpers.py

@@ -201,7 +201,7 @@ def _log(self, path: str, log_type: ValidationProblem, value: Any | None, *args)
         elif log_type == ValidationProblem.ExpectedField:
             logger.error(f"Expected a field at {path}, but found a group.")
         elif log_type == ValidationProblem.MissingDocumentation:
-            if "@" in path.rsplit("/")[-1]:
+            if "@" in path.rsplit("/", maxsplit=1)[-1]:
                 logger.warning(f"Attribute {path} has no documentation.")
             else:
                 logger.warning(f"Field {path} has no documentation.")
@@ -901,8 +901,8 @@ def get_custom_attr_path(path: str) -> str:
     Returns:
         str: The modified path string representing the custom attribute path.
     """
-    if path.split("/")[-1].startswith("@"):
-        attr_name = path.split("/")[-1][1:]  # remove "@"
+    if path.rsplit("/", maxsplit=1)[-1].startswith("@"):
+        attr_name = path.rsplit("/", maxsplit=1)[-1][1:]  # remove "@"
         return f"{path}_custom"
     return f"{path}/@custom"
 

src/pynxtools/dataconverter/validation.py

@@ -1598,7 +1598,11 @@ def is_documented(key: str, tree: NexusNode) -> bool:
             node = add_best_matches_for(key, tree, check_types=True)
         except TypeError:
             node = None
-            nx_type = "attribute" if key.split("/")[-1].startswith("@") else "field"
+            nx_type = (
+                "attribute"
+                if key.rsplit("/", maxsplit=1)[-1].startswith("@")
+                else "field"
+            )
 
             collector.collect_and_log(
                 key,

src/pynxtools/nexus/nexus.py

@@ -66,9 +66,9 @@ def decode_if_string(
         # may need to be updated in the future: https://api.h5py.org/h5t.html
         if elem.dtype.kind == "S":  # Check if it's a bytes array (fixed-length strings)
             decoded_array = np.vectorize(
-                lambda x: x.decode(encoding).rstrip("\x00")
-                if isinstance(x, bytes)
-                else x
+                lambda x: (
+                    x.decode(encoding).rstrip("\x00") if isinstance(x, bytes) else x
+                )
             )(elem)
             return decoded_array.astype(str)  # Ensure the dtype is str
 

tests/dataconverter/test_nexus_tree.py

@@ -43,8 +43,10 @@ def test_correct_extension_of_tree():
     def get_node_fields(tree: NexusNode) -> list[tuple[str, Any]]:
         return list(
             filter(
-                lambda x: not x[0].startswith("_")
-                and x[0] not in ("inheritance", "is_a", "parent_of", "nxdl_base"),
+                lambda x: (
+                    not x[0].startswith("_")
+                    and x[0] not in ("inheritance", "is_a", "parent_of", "nxdl_base")
+                ),
                 tree.__dict__.items(),
             )
         )