fix: use standard list and widen type to allow tuples etc.

Author: Izaak Gough

src/firebase_functions/private/manifest.py

@@ -22,6 +22,7 @@
 from collections.abc import Mapping as _Mapping
 from collections.abc import Sequence as _Sequence
 from enum import Enum as _Enum
+from zoneinfo import ZoneInfo as _ZoneInfo
 
 import typing_extensions as _typing_extensions
 
@@ -40,14 +41,7 @@
 )
 
 SpecValue: _typing.TypeAlias = (
-    str
-    | int
-    | float
-    | bool
-    | _util.Sentinel
-    | list["SpecValue"]
-    | dict[str, "SpecValue"]
-    | None
+    str | int | float | bool | _util.Sentinel | list["SpecValue"] | dict[str, "SpecValue"] | None
 )
 
 
@@ -176,7 +170,7 @@ class ManifestEndpoint:
     """A definition of a function as appears in the Manifest."""
 
     entryPoint: str | None = None
-    region: list[str] | None = _dataclasses.field(default_factory=list[str])
+    region: list[str] | None = _dataclasses.field(default_factory=list)
     platform: str | None = "gcfv2"
     availableMemoryMb: int | _params.Expression[int] | _util.Sentinel | None = None
     maxInstances: int | _params.Expression[int] | _util.Sentinel | None = None
@@ -189,7 +183,7 @@ class ManifestEndpoint:
     labels: dict[str, str] | None = None
     ingressSettings: str | None | _util.Sentinel = None
     secretEnvironmentVariables: list[SecretEnvironmentVariable] | _util.Sentinel | None = (
-        _dataclasses.field(default_factory=list[SecretEnvironmentVariable])
+        _dataclasses.field(default_factory=list)
     )
     httpsTrigger: HttpsTrigger | None = None
     callableTrigger: CallableTrigger | None = None
@@ -208,10 +202,8 @@ class ManifestRequiredApi(_typing.TypedDict):
 class ManifestStack:
     endpoints: dict[str, ManifestEndpoint]
     specVersion: str = "v1alpha1"
-    params: _Sequence[ManifestParamBase] | None = _dataclasses.field(default_factory=list[ManifestParamBase])
-    requiredAPIs: list[ManifestRequiredApi] = _dataclasses.field(
-        default_factory=list[ManifestRequiredApi]
-    )
+    params: _Sequence[ManifestParamBase] | None = _dataclasses.field(default_factory=list)
+    requiredAPIs: list[ManifestRequiredApi] = _dataclasses.field(default_factory=list)
 
 
 def _param_input_to_spec(
@@ -296,23 +288,26 @@ def _param_to_spec(param: ManifestParamBase) -> dict[str, SpecValue]:
 
 def _object_to_spec(data: object) -> SpecValue:
     if isinstance(data, _Enum):
-        return data.value
+        result: SpecValue = data.value
     elif isinstance(data, _params.Expression):
-        return f"{data}"
+        result = f"{data}"
+    elif isinstance(data, _ZoneInfo):
+        result = data.key
     elif _dataclasses.is_dataclass(data):
-        return _dataclass_to_spec(_typing.cast(_DataclassInstance, data))
-    elif isinstance(data, list):
-        return list(map(_object_to_spec, data))
-    elif isinstance(data, dict):
-        return _dict_to_spec(data)
+        result = _dataclass_to_spec(_typing.cast(_DataclassInstance, data))
+    elif isinstance(data, _Mapping):
+        result = _dict_to_spec(data)
+    elif isinstance(data, _Sequence) and not isinstance(data, str | bytes | bytearray):
+        result = list(map(_object_to_spec, data))
     elif data is None:
-        return None
+        result = None
     elif isinstance(data, _util.Sentinel):
-        return data
-    elif isinstance(data, (str, int, float, bool)):
-        return data
+        result = data
+    elif isinstance(data, str | int | float | bool):
+        result = data
     else:
         raise TypeError(f"Unsupported manifest spec value: {type(data)!r}")
+    return result
 
 
 def _dict_factory(data: list[tuple[str, object]]) -> dict[str, SpecValue]: