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

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

@@ -7,6 +7,18 @@ 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
+
+- `update_opportunity_roles(...)` now types its `roles` parameter as `list[RoleAssignment]` (a `TypedDict` with a required `contact_id` and optional `opportunity_contact_role_type_id` / `is_primary`) instead of the loose `list[dict]`, giving callers type checking and autocomplete. `RoleAssignment` is exported from the package root. Non-breaking — plain dicts still satisfy it structurally.
+
 ## [0.3.0] - 2026-07-08
 
 ### Added

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

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: qodev-apollo-api
-Version: 0.3.0
+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.0 → qodev_apollo_api-0.3.2}/pyproject.toml

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

{qodev_apollo_api-0.3.0 → qodev_apollo_api-0.3.2}/src/qodev_apollo_api/__init__.py

@@ -63,6 +63,7 @@ from .models import (
     PaginatedResponse,
     PhoneEntry,
     Pipeline,
+    RoleAssignment,
     SortOrder,
     Stage,
     Task,
@@ -141,6 +142,7 @@ __all__ = [
     "PhoneEntry",
     "Pipeline",
     "RateLimitError",
+    "RoleAssignment",
     "SortOrder",
     "Stage",
     "Task",

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

@@ -33,6 +33,7 @@ from .models import (
     OpportunityContactRoleType,
     PaginatedResponse,
     Pipeline,
+    RoleAssignment,
     SortOrder,
     Stage,
     Task,
@@ -521,7 +522,9 @@ class ApolloClient:
             page=1,
         )
 
-    async def update_opportunity_roles(self, opportunity_id: str, roles: list[dict]) -> Deal:
+    async def update_opportunity_roles(
+        self, opportunity_id: str, roles: list[RoleAssignment]
+    ) -> Deal:
         """Set the contact roles on a deal/opportunity (undocumented endpoint).
 
         This **replaces** the full set of contact roles on the opportunity, so
@@ -756,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.0 → qodev_apollo_api-0.3.2}/src/qodev_apollo_api/models.py

@@ -9,7 +9,7 @@ from __future__ import annotations
 from datetime import date, datetime
 from decimal import Decimal
 from enum import StrEnum
-from typing import Annotated, Any, Generic, Literal, TypeAlias, TypeVar
+from typing import Annotated, Any, Generic, Literal, NotRequired, TypeAlias, TypedDict, TypeVar
 
 from pydantic import BaseModel, ConfigDict, Discriminator, Field, TypeAdapter, ValidationError
 
@@ -183,6 +183,18 @@ class OpportunityContactRole(ApolloModel):
     updated_at: datetime | None = None
 
 
+class RoleAssignment(TypedDict):
+    """A single contact-role entry accepted by ``update_opportunity_roles``.
+
+    ``contact_id`` is required; ``opportunity_contact_role_type_id`` and
+    ``is_primary`` are optional.
+    """
+
+    contact_id: str
+    opportunity_contact_role_type_id: NotRequired[str | None]
+    is_primary: NotRequired[bool]
+
+
 class OpportunityContactRoleType(ApolloModel):
     """Role type definition for opportunity contact roles (e.g., Decision Maker, Buyer)."""
 

{qodev_apollo_api-0.3.0 → 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.0 → 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.0 → qodev_apollo_api-0.3.2}/uv.lock

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