chore: fix mypy and update pre-commits (#1369)

Author: vincentsarago

.pre-commit-config.yaml

@@ -1,24 +1,24 @@
 repos:
   - repo: https://github.com/abravalheri/validate-pyproject
-    rev: v0.24
+    rev: v0.25
     hooks:
       - id: validate-pyproject
 
   - repo: https://github.com/PyCQA/isort
-    rev: 5.12.0
+    rev: 8.0.1
     hooks:
       - id: isort
         language_version: python
 
   - repo: https://github.com/astral-sh/ruff-pre-commit
-    rev: v0.8.4
+    rev: v0.15.11
     hooks:
       - id: ruff
         args: ["--fix"]
       - id: ruff-format
 
   - repo: https://github.com/pre-commit/mirrors-mypy
-    rev: v1.19.0
+    rev: v1.20.1
     hooks:
       - id: mypy
         language_version: python
@@ -29,13 +29,13 @@ repos:
         - pydantic~=2.0
 
   - repo: https://github.com/zizmorcore/zizmor-pre-commit
-    rev: v1.23.1
+    rev: v1.24.1
     hooks:
       - id: zizmor
         args: [--min-severity, medium]
 
   - repo: https://github.com/commitizen-tools/commitizen
-    rev: v4.13.9
+    rev: v4.13.10
     hooks:
       - id: commitizen
         stages: [commit-msg]

docs/src/examples/notebooks/Working_with_CloudOptimizedGeoTIFF.ipynb

@@ -373,7 +373,7 @@
    "outputs": [],
    "source": [
     "# Every 15 of each month for all the years\n",
-    "files_15 = list(filter(lambda x: (x.split(\"_\")[2][7:9] == \"15\"), files))"
+    "files_15 = list(filter(lambda x: x.split(\"_\")[2][7:9] == \"15\", files))"
    ]
   },
   {

src/titiler/core/tests/test_factories.py

@@ -1370,9 +1370,9 @@ def __post_init__(self):
                             .split(","),
                         )
                     )
-                    assert (
-                        len(parsed) == 2
-                    ), f"Invalid rescale values: {self.rescale}, should be of form ['min,max', 'min,max'] or [[min,max], [min, max]]"
+                    assert len(parsed) == 2, (
+                        f"Invalid rescale values: {self.rescale}, should be of form ['min,max', 'min,max'] or [[min,max], [min, max]]"
+                    )
                     rescale_array.append(parsed)
 
                 self.rescale = rescale_array  # Noqa

src/titiler/core/titiler/core/dependencies.py

@@ -317,9 +317,9 @@ def __post_init__(self) -> None:
                         r.split(","),
                     )
                 )
-                assert (
-                    len(parsed) == 2
-                ), f"Invalid rescale values: {self.rescale}, should be of form ['min,max', 'min,max'] or [[min,max], [min, max]]"
+                assert len(parsed) == 2, (
+                    f"Invalid rescale values: {self.rescale}, should be of form ['min,max', 'min,max'] or [[min,max], [min, max]]"
+                )
                 rescale_array.append(parsed)
 
             self.rescale: RescaleType = rescale_array  # type: ignore
@@ -443,9 +443,9 @@ def __post_init__(self):
 
         if self.range:
             parsed = list(map(float, self.range.split(",")))
-            assert (
-                len(parsed) == 2
-            ), f"Invalid histogram_range values: {self.range}, should be of form 'min,max'"
+            assert len(parsed) == 2, (
+                f"Invalid histogram_range values: {self.range}, should be of form 'min,max'"
+            )
 
             self.range = parsed  # type: ignore
 

src/titiler/core/titiler/core/utils.py

@@ -503,11 +503,11 @@ def tms_limits_to_xml(limits: list[TMSLimits]) -> list[str]:
     for limit in limits:
         xml_limits.append(
             f"""<TileMatrixLimits>
-                    <TileMatrix>{limit['tileMatrix']}</TileMatrix>
-                    <MinTileRow>{limit['minTileRow']}</MinTileRow>
-                    <MaxTileRow>{limit['maxTileRow']}</MaxTileRow>
-                    <MinTileCol>{limit['minTileCol']}</MinTileCol>
-                    <MaxTileCol>{limit['maxTileCol']}</MaxTileCol>
+                    <TileMatrix>{limit["tileMatrix"]}</TileMatrix>
+                    <MinTileRow>{limit["minTileRow"]}</MinTileRow>
+                    <MaxTileRow>{limit["maxTileRow"]}</MaxTileRow>
+                    <MinTileCol>{limit["minTileCol"]}</MinTileCol>
+                    <MaxTileCol>{limit["maxTileCol"]}</MaxTileCol>
                 </TileMatrixLimits>""",
         )
 

src/titiler/extensions/titiler/extensions/cogeo.py

@@ -23,9 +23,9 @@ class cogValidateExtension(FactoryExtension):
     def register(self, factory: TilerFactory):  # type: ignore [override]
         """Register endpoint to the tiler factory."""
 
-        assert (
-            cog_info is not None
-        ), "'rio-cogeo' must be installed to use CogValidateExtension"
+        assert cog_info is not None, (
+            "'rio-cogeo' must be installed to use CogValidateExtension"
+        )
 
         @factory.router.get(
             "/validate",

src/titiler/extensions/titiler/extensions/stac.py

@@ -46,9 +46,9 @@ class stacExtension(FactoryExtension):
     def register(self, factory: TilerFactory):  # type: ignore [override]
         """Register endpoint to the tiler factory."""
 
-        assert (
-            create_stac_item is not None
-        ), "'rio-stac' must be installed to use stacExtension"
+        assert create_stac_item is not None, (
+            "'rio-stac' must be installed to use stacExtension"
+        )
         assert pystac is not None, "'pystac' must be installed to use stacExtension"
 
         media = [m.value for m in pystac.MediaType] + ["auto"]

src/titiler/extensions/titiler/extensions/wmts.py

@@ -2,7 +2,7 @@
 
 import warnings
 from collections.abc import Callable
-from typing import Annotated, Any
+from typing import Annotated, Any, cast
 from urllib.parse import urlencode
 
 import jinja2
@@ -67,6 +67,7 @@ def register(self, factory: TilerFactory):  # type: ignore [override] # noqa: C9
                 factory.render_dependency,
             ]
         )
+        tile_dependencies = cast(list[Callable], tile_dependencies)
 
         @factory.router.get(
             "/WMTSCapabilities.xml",

src/titiler/mosaic/titiler/mosaic/extensions/wmts.py

@@ -2,7 +2,7 @@
 
 import warnings
 from collections.abc import Callable
-from typing import Annotated, Any
+from typing import Annotated, Any, cast
 from urllib.parse import urlencode
 
 import jinja2
@@ -72,6 +72,7 @@ def register(self, factory: MosaicTilerFactory):  # type: ignore [override] # no
                 factory.backend_dependency,
             ]
         )
+        tile_dependencies = cast(list[Callable], tile_dependencies)
 
         @factory.router.get(
             "/WMTSCapabilities.xml",

src/titiler/xarray/titiler/xarray/io.py

@@ -348,9 +348,9 @@ def fs_open_dataset(  # noqa: C901
 
     # NetCDF arguments
     if any(src_path.lower().endswith(ext) for ext in [".nc", ".nc4"]):
-        assert (
-            h5netcdf is not None
-        ), "'h5netcdf' must be installed to read NetCDF dataset"
+        assert h5netcdf is not None, (
+            "'h5netcdf' must be installed to read NetCDF dataset"
+        )
 
         xr_open_args.update(
             {