fix(typing): resolve list-shadowing, ship py.typed cleanly; v0.5.1

- Qualify builtin list as builtins.list where a 'list' method shadows it
  (Locations/Reviews/Photos/ConnectedAccounts) so annotations stay checkable
  once py.typed ships (mypy 28 -> 0)
- Fix README clone URL to github.com/listings-api/listingsapi-python-sdk
- Bump version to 0.5.1

Author: Anirudha2604

README.md

@@ -533,7 +533,7 @@ client = listingsapi.ListingsAPI(
 Runnable scripts for every area live in [examples/](examples/), from a 10-line quickstart to a full FastAPI backend.
 
 ```bash
-git clone https://github.com/Listings-API/listingsapi-python-sdk.git
+git clone https://github.com/listings-api/listingsapi-python-sdk.git
 cd listingsapi-python-sdk
 pip install -e ".[dev]"
 pytest

pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "listingsapi"
-version = "0.5.0"
+version = "0.5.1"
 description = "Python SDK for listingsAPI — listings, reviews, posts, and analytics for local marketing"
 readme = "README.md"
 requires-python = ">=3.9"

src/listingsapi/__init__.py

@@ -23,7 +23,7 @@
     ValidationError,
 )
 
-__version__ = "0.5.0"
+__version__ = "0.5.1"
 
 __all__ = [
     # Client

src/listingsapi/resources/connected_accounts.py

@@ -2,6 +2,7 @@
 
 from __future__ import annotations
 
+import builtins
 from typing import Any
 
 from listingsapi._types import APIObject
@@ -43,7 +44,7 @@ def details(self, connected_account_id: str) -> APIObject:
         data = self._get(f"connected-accounts/{connected_account_id}/details")
         return APIObject(data.get("data", {}).get("connectedAccountDetails") or {})
 
-    def folders(self, connected_account_id: str, *, folder_name: str | None = None) -> list[APIObject]:
+    def folders(self, connected_account_id: str, *, folder_name: str | None = None) -> builtins.list[APIObject]:
         """Get folders under a connected Google account."""
         params: dict[str, str] = {}
         if folder_name is not None:
@@ -115,15 +116,15 @@ def disconnect_facebook(self, connected_account_id: str) -> APIObject:
         )
         return APIObject(data.get("data", {}).get("fbBulkDisconnect") or {})
 
-    def trigger_matches(self, connected_account_ids: list[str]) -> APIObject:
+    def trigger_matches(self, connected_account_ids: builtins.list[str]) -> APIObject:
         """Trigger matching of profiles to listingsAPI locations."""
         data = self._post(
             "connected-accounts/trigger-matches",
             {"input": {"connectedAccountIds": connected_account_ids}},
         )
         return APIObject(data.get("data", {}).get("connectedAccountsTriggerMatches") or {})
 
-    def confirm_matches(self, match_record_ids: list[str]) -> APIObject:
+    def confirm_matches(self, match_record_ids: builtins.list[str]) -> APIObject:
         """Confirm suggested matches between connected account listings and locations."""
         data = self._post(
             "connected-accounts/confirm-matches",

src/listingsapi/resources/locations.py

@@ -2,6 +2,7 @@
 
 from __future__ import annotations
 
+import builtins
 import json
 from typing import Any
 
@@ -78,7 +79,7 @@ def retrieve(self, location_id: str | int) -> APIObject:
             raise NotFoundError(f"Location {location_id} not found", status_code=404)
         return results[0]
 
-    def list_by_ids(self, location_ids: list[str | int]) -> list[APIObject]:
+    def list_by_ids(self, location_ids: builtins.list[str | int]) -> builtins.list[APIObject]:
         """Get locations by a list of IDs (numeric or base64)."""
         if not location_ids:
             return []
@@ -87,7 +88,7 @@ def list_by_ids(self, location_ids: list[str | int]) -> list[APIObject]:
         items = data.get("data", {}).get("getLocationsByIds") or []
         return [APIObject(item) for item in items]
 
-    def list_by_store_codes(self, store_codes: list[str]) -> list[APIObject]:
+    def list_by_store_codes(self, store_codes: builtins.list[str]) -> builtins.list[APIObject]:
         """Get locations matching the given store codes."""
         if not store_codes:
             return []
@@ -99,7 +100,7 @@ def search(
         self,
         query: str,
         *,
-        fields: list[str] | None = None,
+        fields: builtins.list[str] | None = None,
         first: int | None = None,
         after: str | None = None,
         before: str | None = None,
@@ -168,11 +169,11 @@ def add(
         website: str | None = None,
         store_id: str | None = None,
         hide_address: bool | None = None,
-        business_hours: list[dict[str, Any]] | None = None,
-        folder_ids: list[str] | None = None,
+        business_hours: builtins.list[dict[str, Any]] | None = None,
+        folder_ids: builtins.list[str] | None = None,
         service_area: dict[str, Any] | None = None,
-        place_action_links: list[dict[str, Any]] | None = None,
-        enabled_site_ids: list[int] | None = None,
+        place_action_links: builtins.list[dict[str, Any]] | None = None,
+        enabled_site_ids: builtins.list[int] | None = None,
         additional_fields: dict[str, Any] | None = None,
     ) -> APIObject:
         """Create a location in one call with every mandatory field as a keyword argument.
@@ -222,7 +223,7 @@ def add(
             )
             print(result.location.id)
         """
-        problems: list[str] = []
+        problems: builtins.list[str] = []
         if not name or not (2 <= len(name.strip()) <= 150):
             problems.append("name must be 2-150 characters")
         if not description or len(description.strip()) < 200:
@@ -276,15 +277,15 @@ def update(self, input: dict[str, Any]) -> APIObject:
         data = self._post("locations/update", {"input": input})
         return APIObject(data.get("data", {}).get("updateLocation") or {})
 
-    def archive(self, location_ids: list[str | int]) -> APIObject:
+    def archive(self, location_ids: builtins.list[str | int]) -> APIObject:
         """Archive one or more locations."""
         encoded = [encode_location_id(lid) for lid in location_ids]
         data = self._post("locations/archive", {"input": {"locationIds": encoded}})
         return APIObject(data.get("data", {}).get("archiveLocations") or {})
 
 
     def cancel_archive(
-        self, location_ids: list[str | int], selection_type: str, changed_by: str
+        self, location_ids: builtins.list[str | int], selection_type: str, changed_by: str
     ) -> APIObject:
         """Cancel a scheduled archival."""
         encoded = [encode_location_id(lid) for lid in location_ids]

src/listingsapi/resources/photos.py

@@ -2,6 +2,7 @@
 
 from __future__ import annotations
 
+import builtins
 from typing import Any
 
 from listingsapi._types import APIObject
@@ -17,29 +18,29 @@ class Photos(APIResource):
         client.photos.add(16808, [{"photo": "https://example.com/img.jpg", "type": "ADDITIONAL"}])
     """
 
-    def list(self, location_id: str | int) -> list[APIObject]:
+    def list(self, location_id: str | int) -> builtins.list[APIObject]:
         """Get photos attached to a location."""
         data = self._location_get(location_id, "photos")
         items = data.get("data", {}).get("mediaFilesOfLocation") or []
         return [APIObject(item) for item in items]
 
-    def add(self, location_id: str | int, photos: list[dict[str, Any]]) -> APIObject:
+    def add(self, location_id: str | int, photos: builtins.list[dict[str, Any]]) -> APIObject:
         """Add photos to a location. Each item needs 'photo' (URL) and 'type' (LOGO, COVER, ADDITIONAL)."""
         data = self._post(
             "locations/photos",
             {"input": {"locationId": encode_location_id(location_id), "photos": photos}},
         )
         return APIObject(data.get("data", {}).get("addLocationPhotos") or {})
 
-    def remove(self, location_id: str | int, photo_ids: list[str]) -> APIObject:
+    def remove(self, location_id: str | int, photo_ids: builtins.list[str]) -> APIObject:
         """Remove photos from a location."""
         data = self._post(
             "locations/photos/remove",
             {"input": {"locationId": encode_location_id(location_id), "photoIds": photo_ids}},
         )
         return APIObject(data.get("data", {}).get("removeLocationPhotos") or {})
 
-    def star(self, location_id: str | int, media_ids: list[str], *, starred: bool = True) -> APIObject:
+    def star(self, location_id: str | int, media_ids: builtins.list[str], *, starred: bool = True) -> APIObject:
         """Star or unstar photos. Max 4 starred photos per account."""
         data = self._post(
             "locations/photos/star",

src/listingsapi/resources/reviews.py

@@ -2,6 +2,7 @@
 
 from __future__ import annotations
 
+import builtins
 import json
 from typing import Any
 
@@ -79,11 +80,11 @@ def list(
         after: str | None = None,
         before: str | None = None,
         last: int | None = None,
-        site_urls: list[str] | None = None,
+        site_urls: builtins.list[str] | None = None,
         start_date: str | None = None,
         end_date: str | None = None,
         category: str | None = None,
-        rating_filters: list[int] | None = None,
+        rating_filters: builtins.list[int] | None = None,
     ) -> SyncPage:
         """List reviews for a location with optional filters."""
         params: dict[str, Any] = {}
@@ -155,15 +156,15 @@ def settings(self, location_id: str | int) -> APIObject:
         data = self._location_get(location_id, "reviews/settings")
         return APIObject(data.get("data", {}).get("interactionsSetting") or {})
 
-    def edit_settings(self, location_id: str | int, site_urls: list[dict[str, Any]]) -> APIObject:
+    def edit_settings(self, location_id: str | int, site_urls: builtins.list[dict[str, Any]]) -> APIObject:
         """Update review source URLs for a location."""
         data = self._post(
             "locations/reviews/settings/edit",
             {"locationId": encode_location_id(location_id), "siteUrls": site_urls},
         )
         return APIObject(data.get("data", {}).get("editInteractionsSetting") or {})
 
-    def details(self, interaction_ids: list[str]) -> APIObject:
+    def details(self, interaction_ids: builtins.list[str]) -> APIObject:
         """Get detailed info for specific reviews by ID."""
         if not interaction_ids:
             return APIObject({})
@@ -172,12 +173,12 @@ def details(self, interaction_ids: list[str]) -> APIObject:
 
     def phrases(
         self,
-        location_ids: list[str],
+        location_ids: builtins.list[str],
         *,
-        site_urls: list[str] | None = None,
+        site_urls: builtins.list[str] | None = None,
         start_date: str | None = None,
         end_date: str | None = None,
-    ) -> list[APIObject]:
+    ) -> builtins.list[APIObject]:
         """Get commonly mentioned phrases from reviews."""
         if not location_ids:
             return []
@@ -192,7 +193,7 @@ def phrases(
         items = data.get("data", {}).get("newReviewPhrases") or []
         return [APIObject(item) for item in items]
 
-    def site_config(self) -> list[APIObject]:
+    def site_config(self) -> builtins.list[APIObject]:
         """Get eligible review sites for the account."""
         data = self._get("reviews/site-config")
         items = data.get("data", {}).get("interactionSiteConfig") or []

src/listingsapi/resources/workflows.py

@@ -134,7 +134,7 @@ def weekly_reputation_report(
 
         review_overview = reviews.analytics.overview(location_id, **date_params).to_dict()
 
-        review_page = reviews.list(location_id, first=50, **date_params)
+        review_page = reviews.list(location_id, first=50, start_date=start_date, end_date=end_date)
         recent_reviews = [r.to_dict() for r in review_page]
 
         google = analytics.google(location_id, from_date=start_date, to_date=end_date).to_dict()

tests/test_listingsapi.py

@@ -388,7 +388,7 @@ def test_analytics_google(client):
 
 
 def test_version():
-    assert listingsapi.__version__ == "0.5.0"
+    assert listingsapi.__version__ == "0.5.1"
 
 
 # --- locations.add (one-call create with validation) ---