fix!(QueryList): Generic fixes (#453)

Author: tony

CHANGES

@@ -15,6 +15,10 @@ $ pip install --user --upgrade --pre libvcs
 
 <!-- Maintainers, insert changes / features for the next release here -->
 
+### Improvement
+
+- `QueryList` generic support improved (#453)
+
 ## libvcs 0.27.0 (2024-02-06)
 
 ### Development

src/libvcs/_internal/query_list.py

@@ -7,9 +7,9 @@
 import re
 import traceback
 import typing as t
-from collections.abc import Mapping, Sequence
+from collections.abc import Iterable, Mapping, Sequence
 
-T = t.TypeVar("T", t.Any, t.Any)
+T = t.TypeVar("T")
 no_arg = object()
 
 
@@ -312,7 +312,7 @@ def __init__(self, op: str, *args: object):
         return super().__init__(f"{op} not in LOOKUP_NAME_MAP")
 
 
-class QueryList(list[T]):
+class QueryList(t.Generic[T], list[T]):
     """Filter list of object/dictionaries. For small, local datasets.
 
     *Experimental, unstable*.
@@ -464,7 +464,10 @@ class QueryList(list[T]):
     data: Sequence[T]
     pk_key: t.Optional[str]
 
-    def items(self) -> list[T]:
+    def __init__(self, items: t.Optional["Iterable[T]"] = None) -> None:
+        super().__init__(items if items is not None else [])
+
+    def items(self) -> list[tuple[str, T]]:
         if self.pk_key is None:
             raise PKRequiredException()
         return [(getattr(item, self.pk_key), item) for item in self]
@@ -522,7 +525,7 @@ def filter_lookup(obj: t.Any) -> bool:
             _filter = matcher
         elif matcher is not None:
 
-            def val_match(obj: t.Union[str, list[t.Any]]) -> bool:
+            def val_match(obj: t.Union[str, list[t.Any], T]) -> bool:
                 if isinstance(matcher, list):
                     return obj in matcher
                 else:

tests/_internal/test_query_list.py

@@ -1,5 +1,5 @@
 import dataclasses
-from typing import Any, Optional, Union
+import typing as t
 
 import pytest
 
@@ -249,9 +249,9 @@ class Obj:
     ],
 )
 def test_filter(
-    items: list[dict[str, Any]],
-    filter_expr: Optional[dict[str, Union[str, list[str]]]],
-    expected_result: Union[QueryList[Any], list[dict[str, Any]]],
+    items: list[dict[str, t.Any]],
+    filter_expr: t.Optional[t.Union[t.Callable[[t.Any], bool], t.Any]],
+    expected_result: t.Union[QueryList[t.Any], list[dict[str, t.Any]]],
 ) -> None:
     qs = QueryList(items)
     if filter_expr is not None: