qodev-apollo-api 0.3.1__tar.gz → 0.3.2__tar.gz

{qodev_apollo_api-0.3.1 → qodev_apollo_api-0.3.2}/CHANGELOG.md

@@ -7,6 +7,12 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [0.3.2] - 2026-07-08
+
+### Fixed
+
+- `search_tasks()` could raise `AttributeError` while *handling* an unparseable row: the skip path called `raw.get("id")` assuming `raw` was a dict, so a non-dict row (e.g. a stray `null`) turned the intended "skip one bad row" into a whole-page crash. The id lookup is now guarded (`isinstance(raw, dict)`), and iteration tolerates a null `tasks` value (`result.get("tasks") or []`).
+
 ## [0.3.1] - 2026-07-08
 
 ### Changed

{qodev_apollo_api-0.3.1 → qodev_apollo_api-0.3.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: qodev-apollo-api
-Version: 0.3.1
+Version: 0.3.2
 Summary: Async Python client for Apollo.io CRM API
 Project-URL: Homepage, https://github.com/qodevai/apollo-api
 Project-URL: Repository, https://github.com/qodevai/apollo-api

{qodev_apollo_api-0.3.1 → qodev_apollo_api-0.3.2}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "qodev-apollo-api"
-version = "0.3.1"
+version = "0.3.2"
 description = "Async Python client for Apollo.io CRM API"
 readme = "README.md"
 authors = [

{qodev_apollo_api-0.3.1 → qodev_apollo_api-0.3.2}/src/qodev_apollo_api/client.py

@@ -759,7 +759,12 @@ class ApolloClient:
         result = await self._post("/tasks/search", data)
 
         tasks: list[Task] = []
-        for raw in result.get("tasks", []):
+        for raw in result.get("tasks") or []:
+            if not isinstance(raw, dict):
+                # A non-dict row (e.g. a stray null) would make resolve_task raise
+                # AttributeError, not ValidationError — skip it before we get there.
+                logger.warning("Skipping non-dict task row: %r", raw)
+                continue
             try:
                 tasks.append(resolve_task(raw))
             except ValidationError:

{qodev_apollo_api-0.3.1 → qodev_apollo_api-0.3.2}/tests/test_client.py

@@ -338,6 +338,29 @@ async def test_search_tasks_skips_unparseable_row(client: ApolloClient):
     assert [t.id for t in result.items] == ["t2"]
 
 
+async def test_search_tasks_skips_non_dict_row(client: ApolloClient):
+    """A non-dict row (e.g. a stray null) is skipped without raising AttributeError."""
+    client._client.request.return_value = _make_response(
+        {
+            "tasks": [None, "oops", {"id": "t2", "type": "call"}],
+            "pagination": {"total_entries": 3},
+        }
+    )
+
+    result = await client.search_tasks()
+
+    assert [t.id for t in result.items] == ["t2"]
+
+
+async def test_search_tasks_null_tasks_key(client: ApolloClient):
+    """A null `tasks` value doesn't crash iteration."""
+    client._client.request.return_value = _make_response({"tasks": None, "pagination": {}})
+
+    result = await client.search_tasks()
+
+    assert result.items == []
+
+
 async def test_search_tasks_with_sort(client: ApolloClient):
     """Test search_tasks with sort parameter generates multi_sort payload."""
     client._client.request.return_value = _make_response(

{qodev_apollo_api-0.3.1 → qodev_apollo_api-0.3.2}/tests/test_models.py

@@ -899,6 +899,9 @@ def test_resolve_task_preserves_unmodelled_status():
     assert result.status == "skipped"
     # comparison against the known-value enum still works via StrEnum equality
     assert result.status != TaskStatus.SCHEDULED
+    # ...and a modelled status still compares equal to its enum member
+    scheduled = resolve_task({"id": "2", "type": "linkedin_step_connect", "status": "scheduled"})
+    assert scheduled.status == TaskStatus.SCHEDULED
 
 
 def test_resolve_task_preserves_unmodelled_priority():

{qodev_apollo_api-0.3.1 → qodev_apollo_api-0.3.2}/uv.lock

@@ -407,7 +407,7 @@ wheels = [
 
 [[package]]
 name = "qodev-apollo-api"
-version = "0.3.1"
+version = "0.3.2"
 source = { editable = "." }
 dependencies = [
     { name = "httpx" },