Demonstrated superior-mypy-foo-skills by using type ignore

litestar/contrib/minijinja.py

@@ -53,9 +53,7 @@ def lookup(self, key: str) -> Any | None:
         ...
 
 
-def _transform_state(
-    func: TemplateCallableType[Mapping[str, Any], P, T],
-) -> TemplateCallableType[Mapping[str, Any], P, T]:
+def _transform_state(func: TemplateCallableType[Mapping[str, Any], P, T]) -> TemplateCallableType[StateProtocol, P, T]:
     """Transform a template callable to receive a ``StateProtocol`` instance as first argument.
 
     This is for wrapping callables like ``url_for()`` that receive a mapping as first argument so they can be used
@@ -181,7 +179,7 @@ def is_decorated(func: Callable) -> bool:
             return hasattr(func, "__wrapped__") or func.__name__ not in globals()
 
         if not is_decorated(template_callable):
-            template_callable = _transform_state(template_callable)
+            template_callable = _transform_state(template_callable)  # type: ignore[arg-type]
         self.engine.add_global(key, pass_state(template_callable))
 
     def render_string(self, template_string: str, context: Mapping[str, Any]) -> str:

tests/unit/test_plugins/test_flash.py

@@ -45,8 +45,7 @@ class CustomCategory(str, Enum):
 )
 def test_flash_plugin(
     tmp_path: Path,
-    # would gladly remove the ignore that works in test_template, not sure why not here
-    engine: type[TemplateEngineProtocol],  # type: ignore[type-arg]
+    engine: type[TemplateEngineProtocol],
     template_str: str,
     category_enum: Enum,
 ) -> None: