fix: use typing.List/Optional to avoid mypy list method shadowing

Classes with a list() method cause mypy to interpret list[str] as a
method call. Switch to typing.List and typing.Optional in api_keys.py
and webhooks.py, with per-file ruff ignores for the UP006/UP035/UP045
rules.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: mivertowski

pyproject.toml

@@ -42,6 +42,12 @@ line-length = 100
 [tool.ruff.lint]
 select = ["E", "F", "I", "UP", "B", "SIM"]
 
+[tool.ruff.lint.per-file-ignores]
+# These files define a list() method which shadows the builtin;
+# mypy needs typing.List to avoid interpreting list[str] as the method.
+"src/vynfi/resources/api_keys.py" = ["UP006", "UP035", "UP045"]
+"src/vynfi/resources/webhooks.py" = ["UP006", "UP035", "UP045"]
+
 [tool.mypy]
 python_version = "3.9"
 strict = true

src/vynfi/resources/api_keys.py

@@ -2,7 +2,7 @@
 
 from __future__ import annotations
 
-from typing import Any
+from typing import Any, List, Optional
 
 from .._client import SyncClient
 from .._types import ApiKey, ApiKeyCreated
@@ -18,8 +18,8 @@ def create(
         self,
         *,
         name: str,
-        scopes: list[str] | None = None,
-        expires_in_days: int | None = None,
+        scopes: Optional[List[str]] = None,
+        expires_in_days: Optional[int] = None,
     ) -> ApiKeyCreated:
         """Create a new API key.
 
@@ -33,7 +33,7 @@ def create(
         data = self._client.request("POST", "/v1/api-keys", json=body)
         return ApiKeyCreated.model_validate(data)
 
-    def list(self) -> list[ApiKey]:
+    def list(self) -> List[ApiKey]:
         """List all API keys (masked)."""
         data = self._client.request("GET", "/v1/api-keys")
         if isinstance(data, dict):
@@ -49,8 +49,8 @@ def update(
         self,
         key_id: str,
         *,
-        name: str | None = None,
-        scopes: list[str] | None = None,
+        name: Optional[str] = None,
+        scopes: Optional[List[str]] = None,
     ) -> ApiKey:
         """Update an API key."""
         body: dict[str, Any] = {}

src/vynfi/resources/webhooks.py

@@ -2,7 +2,7 @@
 
 from __future__ import annotations
 
-from typing import Any
+from typing import Any, List, Optional
 
 from .._client import SyncClient
 from .._types import Webhook, WebhookCreated
@@ -14,7 +14,7 @@ class Webhooks:
     def __init__(self, client: SyncClient) -> None:
         self._client = client
 
-    def create(self, *, url: str, events: list[str]) -> WebhookCreated:
+    def create(self, *, url: str, events: List[str]) -> WebhookCreated:
         """Create a new webhook.
 
         Args:
@@ -25,7 +25,7 @@ def create(self, *, url: str, events: list[str]) -> WebhookCreated:
         data = self._client.request("POST", "/v1/webhooks", json={"url": url, "events": events})
         return WebhookCreated.model_validate(data)
 
-    def list(self) -> list[Webhook]:
+    def list(self) -> List[Webhook]:
         """List all webhooks."""
         data = self._client.request("GET", "/v1/webhooks")
         if isinstance(data, dict):
@@ -41,9 +41,9 @@ def update(
         self,
         webhook_id: str,
         *,
-        url: str | None = None,
-        events: list[str] | None = None,
-        status: str | None = None,
+        url: Optional[str] = None,
+        events: Optional[List[str]] = None,
+        status: Optional[str] = None,
     ) -> Webhook:
         """Update a webhook."""
         body: dict[str, Any] = {}
@@ -60,7 +60,6 @@ def delete(self, webhook_id: str) -> None:
         """Delete a webhook."""
         self._client.request("DELETE", f"/v1/webhooks/{webhook_id}")
 
-    def test(self, webhook_id: str) -> dict[str, Any]:
+    def test(self, webhook_id: str) -> Any:
         """Send a test event to a webhook."""
-        data = self._client.request("POST", f"/v1/webhooks/{webhook_id}/test")
-        return data  # type: ignore[return-value]
+        return self._client.request("POST", f"/v1/webhooks/{webhook_id}/test")