CUST-5289 [v3] Python SDK ListGrantsQueryParams uses camelCase keys that are silently ignored by the API

License
I confirm that this contribution is made under the terms of the MIT license and that I have the authority necessary to make this contribution on behalf of its copyright owner.

Fix ListGrantsQueryParams key handling in Python SDK

- Fix bug where `sortBy`, `orderBy`, and `grantStatus` are sent as-is and ignored by the API.
- Normalize grants query aliases to `sort_by`, `order_by`, and `grant_status` at request serialization time.
- Align type hints with snake_case API contract while keeping camelCase aliases for backwards compatibility.
- Preserve existing SDK behavior for callers already using snake_case keys.

✅ Tests checks:

- `pytest tests/handler/test_http_client.py -k test_build_query_params -q` (100%)
- Manual serialization check for grants aliases via `_build_query_params` (verified)

Made-with: Cursor

Author: samLRodrigues

nylas/handler/http_client.py

@@ -49,15 +49,22 @@ def _validate_response(response: Response) -> Tuple[Dict, CaseInsensitiveDict]:
 
 def _build_query_params(base_url: str, query_params: dict = None) -> str:
     query_param_parts = []
+    query_param_key_aliases = {
+        "sortBy": "sort_by",
+        "orderBy": "order_by",
+        "grantStatus": "grant_status",
+    }
+
     for key, value in query_params.items():
+        normalized_key = query_param_key_aliases.get(key, key)
         if isinstance(value, list):
             for item in value:
-                query_param_parts.append(f"{key}={quote(str(item))}")
+                query_param_parts.append(f"{normalized_key}={quote(str(item))}")
         elif isinstance(value, dict):
             for k, v in value.items():
-                query_param_parts.append(f"{key}={k}:{quote(str(v))}")
+                query_param_parts.append(f"{normalized_key}={k}:{quote(str(v))}")
         else:
-            query_param_parts.append(f"{key}={quote(str(value))}")
+            query_param_parts.append(f"{normalized_key}={quote(str(value))}")
 
     query_string = "&".join(query_param_parts)
     return f"{base_url}?{query_string}"

nylas/models/grants.py

@@ -83,23 +83,30 @@ class ListGrantsQueryParams(TypedDict):
         limit: The maximum number of objects to return.
             This field defaults to 10. The maximum allowed value is 200.
         offset: Offset grant results by this number.
-        sortBy: Sort entries by field name
-        orderBy: Specify ascending or descending order.
+        sort_by: Sort entries by field name.
+        order_by: Specify ascending or descending order.
         since: Scope grants from a specific point in time by Unix timestamp.
         before: Scope grants to a specific point in time by Unix timestamp.
         email: Filtering your query based on grant email address (if applicable)
-        grantStatus: Filtering your query based on grant email status (if applicable)
+        grant_status: Filtering your query based on grant email status (if applicable)
         ip: Filtering your query based on grant IP address
         provider: Filtering your query based on OAuth provider
+        sortBy: Deprecated camelCase alias for sort_by.
+        orderBy: Deprecated camelCase alias for order_by.
+        grantStatus: Deprecated camelCase alias for grant_status.
     """
 
     limit: NotRequired[int]
     offset: NotRequired[int]
-    sortBy: NotRequired[str]
-    orderBy: NotRequired[str]
+    sort_by: NotRequired[str]
+    order_by: NotRequired[str]
     since: NotRequired[int]
     before: NotRequired[int]
     email: NotRequired[str]
-    grantStatus: NotRequired[str]
+    grant_status: NotRequired[str]
     ip: NotRequired[str]
     provider: NotRequired[Provider]
+    # Backward-compatible aliases for callers still passing camelCase keys.
+    sortBy: NotRequired[str]
+    orderBy: NotRequired[str]
+    grantStatus: NotRequired[str]