qodev-apollo-api 0.1.2__tar.gz → 0.2.0__tar.gz

{qodev_apollo_api-0.1.2 → qodev_apollo_api-0.2.0}/CHANGELOG.md

@@ -7,6 +7,34 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [0.2.0] - 2026-06-02
+
+### Fixed
+- `create_note()` posted `{"note": <plaintext>}`, which Apollo silently ignores — notes were created with empty content. It now serialises `content` to ProseMirror JSON and posts it in the `content` field (the format Apollo stores and `search_notes()` reads back). Closes #6.
+
+### Added
+- `markdown_to_prosemirror()` in `utils` — inverse of `prosemirror_to_markdown()` (title, paragraphs, bullet/ordered lists).
+- `create_note(..., title=...)` — optional note title (rendered as the ProseMirror `noteTitle`).
+- `ApolloClient.delete_note(note_id)` — `DELETE /notes/{id}`.
+
+### Changed
+- `create_note()` association args (`contact_ids`, `account_ids`, `opportunity_ids`) are now keyword-only, so the new positional `title` can't be confused with them.
+
+## [0.1.3] - 2026-02-23
+
+### Added
+- `ActionItemTask` subclass for `action_item` task type
+- `OtherTask` fallback subclass — `resolve_task()` now returns `OtherTask` for unknown task types instead of raising `ValidationError`
+- `OpportunityContactRoleType` model for role type definitions (Decision Maker, Buyer, etc.)
+- `ApolloClient.list_opportunity_contact_role_types()` — lookup endpoint for role type ID → name mapping (undocumented `POST /opportunity_contact_role_types/search`)
+
+### Fixed
+- `Deal.closed_date`, `Deal.actual_close_date`, `Deal.next_step_date` now parsed as `datetime` (were `str`)
+- `EmploymentHistory.start_date`, `EmploymentHistory.end_date` now parsed as `date` (were `str`)
+- `CallSummaryNextStep.due_at` now parsed as `datetime` (was `str`)
+- `NewsArticle.published_at`, `JobPosting.posted_at` now parsed as `datetime` (were `str`)
+- `search_conversations()` default and max limit corrected from 100 to 25 (Apollo API caps at 25)
+
 ## [0.1.2] - 2026-02-23
 
 ### Added

{qodev_apollo_api-0.1.2 → qodev_apollo_api-0.2.0}/PKG-INFO

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

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

{qodev_apollo_api-0.1.2 → qodev_apollo_api-0.2.0}/src/qodev_apollo_api/__init__.py

@@ -9,6 +9,7 @@ from .models import (
     AccountDetail,
     AccountPlaybookStatus,
     AccountQueue,
+    ActionItemTask,
     ApolloModel,
     BaseTask,
     CalendarEvent,
@@ -53,8 +54,10 @@ from .models import (
     NewsArticle,
     Note,
     OpportunityContactRole,
+    OpportunityContactRoleType,
     OpportunityRoleEntry,
     OrganizationRef,
+    OtherTask,
     OutreachTaskMessage,
     PaginatedResponse,
     PhoneEntry,
@@ -79,6 +82,7 @@ __all__ = [
     "AccountDetail",
     "AccountPlaybookStatus",
     "AccountQueue",
+    "ActionItemTask",
     "ApolloClient",
     "ApolloError",
     "ApolloModel",
@@ -126,8 +130,10 @@ __all__ = [
     "NewsArticle",
     "Note",
     "OpportunityContactRole",
+    "OpportunityContactRoleType",
     "OpportunityRoleEntry",
     "OrganizationRef",
+    "OtherTask",
     "OutreachTaskMessage",
     "PaginatedResponse",
     "PhoneEntry",
@@ -145,4 +151,4 @@ __all__ = [
     "resolve_task",
 ]
 
-__version__ = "0.1.2"
+__version__ = "0.2.0"

{qodev_apollo_api-0.1.2 → qodev_apollo_api-0.2.0}/src/qodev_apollo_api/client.py

@@ -27,6 +27,7 @@ from .models import (
     LinkedInConnectTask,
     LinkedInMessageTask,
     Note,
+    OpportunityContactRoleType,
     PaginatedResponse,
     Pipeline,
     SortOrder,
@@ -37,7 +38,7 @@ from .models import (
     TaskType,
     resolve_task,
 )
-from .utils import normalize_linkedin_url, prosemirror_to_markdown
+from .utils import markdown_to_prosemirror, normalize_linkedin_url, prosemirror_to_markdown
 
 
 class ApolloClient:
@@ -493,6 +494,28 @@ class ApolloClient:
             page=1,
         )
 
+    async def list_opportunity_contact_role_types(
+        self,
+    ) -> PaginatedResponse[OpportunityContactRoleType]:
+        """List all opportunity contact role types (undocumented endpoint).
+
+        Returns role type definitions (e.g., Decision Maker, Buyer, Champion)
+        that map to OpportunityRoleEntry.opportunity_contact_role_type_id.
+
+        Returns:
+            Paginated response with OpportunityContactRoleType items
+        """
+        result = await self._post("/opportunity_contact_role_types/search", {})
+        role_types = [
+            OpportunityContactRoleType.model_validate(rt)
+            for rt in result.get("opportunity_contact_role_types", [])
+        ]
+        return PaginatedResponse[OpportunityContactRoleType](
+            items=role_types,
+            total=len(role_types),
+            page=1,
+        )
+
     # ========================================================================
     # ENRICHMENT
     # ========================================================================
@@ -574,14 +597,23 @@ class ApolloClient:
     async def create_note(
         self,
         content: str,
+        title: str | None = None,
+        *,
         contact_ids: list[str] | None = None,
         account_ids: list[str] | None = None,
         opportunity_ids: list[str] | None = None,
     ) -> dict:
         """Create a note.
 
+        Apollo stores note bodies in the ``content`` field as ProseMirror JSON,
+        so the plain text / Markdown ``content`` is converted via
+        :func:`~qodev_apollo_api.utils.markdown_to_prosemirror` before posting.
+        (Posting ``{"note": <plaintext>}`` is silently ignored by Apollo and
+        produces an empty note.)
+
         Args:
-            content: Note content (plain text or Markdown)
+            content: Note body as plain text / Markdown.
+            title: Optional note title (rendered as the ProseMirror noteTitle).
             contact_ids: List of contact IDs to associate
             account_ids: List of account IDs to associate
             opportunity_ids: List of opportunity IDs to associate
@@ -589,7 +621,9 @@ class ApolloClient:
         Returns:
             Raw API response with created note data
         """
-        data: dict[str, str | list[str]] = {"note": content}
+        data: dict[str, str | list[str]] = {
+            "content": markdown_to_prosemirror(content, title=title)
+        }
         if contact_ids:
             data["contact_ids"] = contact_ids
         if account_ids:
@@ -599,6 +633,17 @@ class ApolloClient:
 
         return await self._post("/notes", data)
 
+    async def delete_note(self, note_id: str) -> dict:
+        """Delete a note by ID.
+
+        Args:
+            note_id: Apollo note ID.
+
+        Returns:
+            Raw API response.
+        """
+        return await self._request("DELETE", f"/notes/{note_id}")
+
     # ========================================================================
     # ACTIVITIES
     # ========================================================================
@@ -1058,19 +1103,19 @@ class ApolloClient:
     # ========================================================================
 
     async def search_conversations(
-        self, page: int = 1, limit: int = 100, **filters
+        self, page: int = 1, limit: int = 25, **filters
     ) -> PaginatedResponse[Conversation]:
         """Search recorded conversations (Zoom/Teams/Meet).
 
         Args:
             page: Page number (default 1)
-            limit: Results per page (default 100, max 100)
+            limit: Results per page (default 25, max 25)
             **filters: Additional filters
 
         Returns:
             Paginated response with Conversation items
         """
-        data = {"page": page, "per_page": min(limit, 100), **filters}
+        data = {"page": page, "per_page": min(limit, 25), **filters}
         result = await self._post("/conversations/search", data)
 
         conversations = [Conversation.model_validate(c) for c in result.get("conversations", [])]

{qodev_apollo_api-0.1.2 → qodev_apollo_api-0.2.0}/src/qodev_apollo_api/models.py

@@ -6,12 +6,12 @@ All models use extra="allow" to capture any new/undocumented API fields.
 
 from __future__ import annotations
 
-from datetime import datetime
+from datetime import date, datetime
 from decimal import Decimal
 from enum import StrEnum
 from typing import Annotated, Any, Generic, Literal, TypeAlias, TypeVar
 
-from pydantic import BaseModel, ConfigDict, Discriminator, Field, TypeAdapter
+from pydantic import BaseModel, ConfigDict, Discriminator, Field, TypeAdapter, ValidationError
 
 
 class ApolloModel(BaseModel):
@@ -183,6 +183,17 @@ class OpportunityContactRole(ApolloModel):
     updated_at: datetime | None = None
 
 
+class OpportunityContactRoleType(ApolloModel):
+    """Role type definition for opportunity contact roles (e.g., Decision Maker, Buyer)."""
+
+    id: str
+    name: str | None = None
+    team_id: str | None = None
+    crm_api_name: str | None = None
+    crm_label: str | None = None
+    display_order: float | None = None
+
+
 class CrmNote(ApolloModel):
     """CRM-synced note reference."""
 
@@ -346,8 +357,8 @@ class EmploymentHistory(ApolloModel):
     title: str | None = None
     organization_name: str | None = None
     organization_id: str | None = None
-    start_date: str | None = None
-    end_date: str | None = None
+    start_date: date | None = None
+    end_date: date | None = None
 
     # Additional fields from API
     key: str | None = None
@@ -554,8 +565,8 @@ class Deal(ApolloModel):
     created_by_id: str | None = None
 
     # Close
-    closed_date: str | None = None
-    actual_close_date: str | None = None
+    closed_date: datetime | None = None
+    actual_close_date: datetime | None = None
     is_closed: bool | None = None
     is_won: bool | None = None
     closed_lost_reason: str | None = None
@@ -563,7 +574,7 @@ class Deal(ApolloModel):
 
     # Next Steps
     next_step: str | None = None
-    next_step_date: str | None = None
+    next_step_date: datetime | None = None
     next_step_last_updated_at: datetime | None = None
     current_solutions: str | None = None
     deal_source: str | None = None
@@ -775,6 +786,8 @@ class TaskType(StrEnum):
     LINKEDIN_ACTIONS = "linkedin_actions"
     CONTACT_ACTION_ITEM = "contact_action_item"
     ACCOUNT_ACTION_ITEM = "account_action_item"
+    ACTION_ITEM = "action_item"
+    OTHER = "other"
 
 
 class LinkedInTemplate(ApolloModel):
@@ -919,6 +932,24 @@ class AccountActionItemTask(BaseTask):
     type: Literal[TaskType.ACCOUNT_ACTION_ITEM] = TaskType.ACCOUNT_ACTION_ITEM
 
 
+class ActionItemTask(BaseTask):
+    """Generic action item task."""
+
+    type: Literal[TaskType.ACTION_ITEM] = TaskType.ACTION_ITEM
+
+
+class OtherTask(BaseTask):
+    """Fallback for task types not yet modelled in the library.
+
+    Used by resolve_task() when the API returns a type value that doesn't
+    match any known TaskType variant.  The raw type string from the API
+    is preserved in ``original_type``.
+    """
+
+    type: Literal[TaskType.OTHER] = TaskType.OTHER
+    original_type: str
+
+
 # ---------------------------------------------------------------------------
 # Discriminated union for polymorphic task deserialization
 # ---------------------------------------------------------------------------
@@ -934,7 +965,8 @@ _DiscriminatedTask = Annotated[
     | LinkedInViewProfileTask
     | LinkedInActionsTask
     | ContactActionItemTask
-    | AccountActionItemTask,
+    | AccountActionItemTask
+    | ActionItemTask,
     Discriminator("type"),
 ]
 
@@ -950,6 +982,8 @@ Task: TypeAlias = (
     | LinkedInActionsTask
     | ContactActionItemTask
     | AccountActionItemTask
+    | ActionItemTask
+    | OtherTask
 )
 
 _task_adapter: TypeAdapter[_DiscriminatedTask] = TypeAdapter(_DiscriminatedTask)
@@ -958,9 +992,18 @@ _task_adapter: TypeAdapter[_DiscriminatedTask] = TypeAdapter(_DiscriminatedTask)
 def resolve_task(data: dict[str, Any]) -> Task:
     """Validate a raw task dict into the most specific Task subclass.
 
-    Raises ValidationError if the type field is missing or unrecognized.
+    Falls back to OtherTask for unknown task types.  The raw type string
+    is preserved in ``OtherTask.original_type``.
+    Raises ValidationError only if the data is structurally invalid
+    (e.g. missing required ``id`` field).
     """
-    return _task_adapter.validate_python(data)
+    try:
+        return _task_adapter.validate_python(data)
+    except ValidationError:
+        raw_type = data.get("type")
+        if raw_type is None:
+            raise
+        return OtherTask.model_validate({**data, "type": "other", "original_type": raw_type})
 
 
 class Email(ApolloModel):
@@ -1127,7 +1170,7 @@ class CallSummaryNextStep(ApolloModel):
 
     id: str | None = None
     step: str | None = None
-    due_at: str | None = None
+    due_at: datetime | None = None
     action_type: str | None = None
     task_id: str | None = None
     participant_id: str | None = None
@@ -1209,7 +1252,7 @@ class NewsArticle(BaseModel):
 
     title: str | None = None
     url: str | None = None
-    published_at: str | None = None
+    published_at: datetime | None = None
     category: str | None = None
     summary: str | None = None
 
@@ -1221,7 +1264,7 @@ class JobPosting(BaseModel):
     location: str | None = None
     department: str | None = None
     url: str | None = None
-    posted_at: str | None = None
+    posted_at: datetime | None = None
 
 
 # ============================================================================

{qodev_apollo_api-0.1.2 → qodev_apollo_api-0.2.0}/src/qodev_apollo_api/utils.py

@@ -1,9 +1,12 @@
 """Utility functions for Apollo client.
 
-Includes ProseMirror JSON to Markdown conversion for notes.
+Includes ProseMirror JSON <-> Markdown conversion for notes.
 """
 
 import json
+import re
+
+_ORDERED_ITEM_RE = re.compile(r"^\d+\.\s+(.*)$")
 
 
 def prosemirror_to_markdown(content_json: str) -> tuple[str, str]:
@@ -103,6 +106,66 @@ def _extract_text_from_list_item(item: dict) -> str:
     return " ".join(text_parts)
 
 
+def markdown_to_prosemirror(content: str, title: str | None = None) -> str:
+    """Convert plain text / lightweight Markdown to a ProseMirror JSON string.
+
+    Apollo's ``POST /notes`` stores the note body in the ``content`` field as a
+    ProseMirror JSON string; this is the inverse of :func:`prosemirror_to_markdown`
+    for the node types Apollo notes use.
+
+    Supports:
+      * an optional note title (``noteTitle`` node)
+      * paragraphs (one per line; blank lines become empty paragraphs)
+      * bullet lists (lines starting with ``- `` or ``* ``)
+      * ordered lists (lines starting with ``1. ``)
+
+    Args:
+        content: Note body as plain text / Markdown.
+        title: Optional note title (rendered as the ProseMirror ``noteTitle``).
+
+    Returns:
+        ProseMirror document as a JSON string, ready to post to ``content``.
+    """
+    nodes: list[dict] = []
+    if title:
+        nodes.append({"type": "noteTitle", "content": [{"type": "text", "text": title}]})
+
+    def _paragraph(text: str) -> dict:
+        # Empty text nodes are invalid in ProseMirror — emit a bare paragraph.
+        if not text:
+            return {"type": "paragraph"}
+        return {"type": "paragraph", "content": [{"type": "text", "text": text}]}
+
+    def _list_item(text: str) -> dict:
+        return {"type": "listItem", "content": [_paragraph(text)]}
+
+    lines = (content or "").split("\n")
+    i = 0
+    while i < len(lines):
+        stripped = lines[i].strip()
+
+        if stripped[:2] in ("- ", "* "):
+            items = []
+            while i < len(lines) and lines[i].strip()[:2] in ("- ", "* "):
+                items.append(_list_item(lines[i].strip()[2:].strip()))
+                i += 1
+            nodes.append({"type": "bulletList", "content": items})
+            continue
+
+        if _ORDERED_ITEM_RE.match(stripped):
+            items = []
+            while i < len(lines) and (m := _ORDERED_ITEM_RE.match(lines[i].strip())):
+                items.append(_list_item(m.group(1).strip()))
+                i += 1
+            nodes.append({"type": "orderedList", "content": items})
+            continue
+
+        nodes.append(_paragraph(lines[i]))
+        i += 1
+
+    return json.dumps({"type": "doc", "content": nodes}, ensure_ascii=False)
+
+
 def normalize_linkedin_url(url: str) -> str:
     """Normalize LinkedIn URL for comparison.
 

{qodev_apollo_api-0.1.2 → qodev_apollo_api-0.2.0}/tests/test_client.py

@@ -1,5 +1,6 @@
 """Tests for Apollo client methods."""
 
+import json
 import os
 from datetime import UTC, datetime
 from unittest.mock import AsyncMock, MagicMock, patch
@@ -27,6 +28,7 @@ from qodev_apollo_api.models import (
     LinkedInConnectTask,
     LinkedInMessageTask,
     Note,
+    OpportunityContactRoleType,
     PaginatedResponse,
     Pipeline,
     SortOrder,
@@ -412,7 +414,21 @@ async def test_search_conversations(client: ApolloClient):
     assert len(result.items) == 1
     assert isinstance(result.items[0], Conversation)
 
-    assert client._client.request.call_args[0] == ("POST", "/conversations/search")
+    call_args = client._client.request.call_args
+    assert call_args[0] == ("POST", "/conversations/search")
+    assert call_args[1]["json"]["per_page"] == 25
+
+
+async def test_search_conversations_caps_limit(client: ApolloClient):
+    """Test search_conversations caps limit at 25 (Apollo API maximum)."""
+    client._client.request.return_value = _make_response(
+        {"conversations": [], "pagination": {"total_entries": 0}}
+    )
+
+    await client.search_conversations(limit=100)
+
+    payload = client._client.request.call_args[1]["json"]
+    assert payload["per_page"] == 25
 
 
 # ============================================================================
@@ -545,6 +561,31 @@ async def test_list_all_stages(client: ApolloClient):
     assert len(result.items) == 2
 
 
+async def test_list_opportunity_contact_role_types(client: ApolloClient):
+    """Test POST /opportunity_contact_role_types/search returns role types."""
+    client._client.request.return_value = _make_response(
+        {
+            "opportunity_contact_role_types": [
+                {"id": "rt1", "name": "Decision Maker", "display_order": 3.0},
+                {"id": "rt2", "name": "Buyer", "display_order": 1.0},
+            ]
+        }
+    )
+
+    result = await client.list_opportunity_contact_role_types()
+
+    assert isinstance(result, PaginatedResponse)
+    assert len(result.items) == 2
+    assert all(isinstance(rt, OpportunityContactRoleType) for rt in result.items)
+    assert result.items[0].name == "Decision Maker"
+    assert result.items[1].name == "Buyer"
+    assert result.total == 2
+
+    call_args = client._client.request.call_args
+    assert call_args[0] == ("POST", "/opportunity_contact_role_types/search")
+    assert call_args[1]["json"] == {}
+
+
 async def test_get_contact_stages(client: ApolloClient):
     """Test GET /contact_stages returns list[dict]."""
     client._client.request.return_value = _make_response(
@@ -1069,18 +1110,23 @@ async def test_search_people(client: ApolloClient):
 
 
 async def test_create_note(client: ApolloClient):
-    """Test POST /notes."""
-    client._client.request.return_value = _make_response({"note": {"id": "n1", "content": "Hello"}})
+    """create_note posts ProseMirror JSON in `content` (not plaintext in `note`)."""
+    client._client.request.return_value = _make_response({"note": {"id": "n1"}})
 
-    result = await client.create_note("Hello")
+    result = await client.create_note("Hello", title="Greeting")
 
-    assert result == {"note": {"id": "n1", "content": "Hello"}}
+    assert result == {"note": {"id": "n1"}}
 
     call_args = client._client.request.call_args
     assert call_args[0] == ("POST", "/notes")
     payload = call_args[1]["json"]
-    assert payload["note"] == "Hello"
+    assert "note" not in payload  # old (ignored) field must be gone
     assert "contact_ids" not in payload
+    doc = json.loads(payload["content"])
+    assert doc["type"] == "doc"
+    assert doc["content"][0]["type"] == "noteTitle"
+    assert doc["content"][0]["content"][0]["text"] == "Greeting"
+    assert doc["content"][1]["content"][0]["text"] == "Hello"
 
 
 async def test_create_note_with_associations(client: ApolloClient):
@@ -1095,12 +1141,21 @@ async def test_create_note_with_associations(client: ApolloClient):
     )
 
     payload = client._client.request.call_args[1]["json"]
-    assert payload["note"] == "Meeting notes"
+    assert "Meeting notes" in payload["content"]  # body serialised into ProseMirror
     assert payload["contact_ids"] == ["c1", "c2"]
     assert payload["account_ids"] == ["a1"]
     assert payload["opportunity_ids"] == ["o1"]
 
 
+async def test_delete_note(client: ApolloClient):
+    """delete_note issues DELETE /notes/{id}."""
+    client._client.request.return_value = _make_response({})
+
+    await client.delete_note("n1")
+
+    client._client.request.assert_called_once_with("DELETE", "/notes/n1")
+
+
 async def test_get_api_usage(client: ApolloClient):
     """Test POST /usage_stats/api_usage_stats."""
     usage_data = {

{qodev_apollo_api-0.1.2 → qodev_apollo_api-0.2.0}/tests/test_models.py

@@ -1,6 +1,6 @@
 """Tests for Apollo models."""
 
-from datetime import datetime
+from datetime import UTC, datetime
 
 import pytest
 from pydantic import ValidationError
@@ -12,6 +12,7 @@ from qodev_apollo_api.models import (
     AccountDetail,
     AccountPlaybookStatus,
     AccountQueue,
+    ActionItemTask,
     ApolloModel,
     BaseTask,
     Call,
@@ -51,8 +52,10 @@ from qodev_apollo_api.models import (
     LinkedInViewProfileTask,
     Note,
     OpportunityContactRole,
+    OpportunityContactRoleType,
     OpportunityRoleEntry,
     OrganizationRef,
+    OtherTask,
     PaginatedResponse,
     PhoneEntry,
     Pipeline,
@@ -564,6 +567,36 @@ def test_deal_with_embedded_account():
     assert deal.account.name == "Acme Corp"
 
 
+# ============================================================================
+# OPPORTUNITY CONTACT ROLE TYPE
+# ============================================================================
+
+
+def test_opportunity_contact_role_type_model():
+    """Test OpportunityContactRoleType model validation."""
+    rt = OpportunityContactRoleType.model_validate(
+        {
+            "id": "rt1",
+            "name": "Decision Maker",
+            "team_id": "team1",
+            "crm_api_name": None,
+            "crm_label": None,
+            "display_order": 3.0,
+        }
+    )
+    assert rt.id == "rt1"
+    assert rt.name == "Decision Maker"
+    assert rt.display_order == 3.0
+
+
+def test_opportunity_contact_role_type_minimal():
+    """Test OpportunityContactRoleType with only required fields."""
+    rt = OpportunityContactRoleType.model_validate({"id": "rt1"})
+    assert rt.id == "rt1"
+    assert rt.name is None
+    assert rt.display_order is None
+
+
 # ============================================================================
 # PIPELINE & STAGE
 # ============================================================================
@@ -811,6 +844,12 @@ def test_resolve_task_account_action_item():
     assert isinstance(result, AccountActionItemTask)
 
 
+def test_resolve_task_action_item():
+    """Test resolve_task returns ActionItemTask."""
+    result = resolve_task({"id": "1", "type": "action_item"})
+    assert isinstance(result, ActionItemTask)
+
+
 def test_resolve_task_linkedin_interact():
     """Test resolve_task returns LinkedInInteractTask."""
     result = resolve_task({"id": "1", "type": "linkedin_step_interact_post"})
@@ -842,10 +881,13 @@ def test_resolve_task_missing_type_raises():
         resolve_task({"id": "1", "status": "complete"})
 
 
-def test_resolve_task_unknown_type_raises():
-    """Test resolve_task raises ValidationError for unknown type values."""
-    with pytest.raises(ValidationError):
-        resolve_task({"id": "1", "type": "some_future_task_type"})
+def test_resolve_task_unknown_type_falls_back_to_other_task():
+    """Test resolve_task falls back to OtherTask for unknown type values."""
+    result = resolve_task({"id": "1", "type": "some_future_task_type"})
+    assert isinstance(result, OtherTask)
+    assert result.type == TaskType.OTHER
+    assert result.original_type == "some_future_task_type"
+    assert result.id == "1"
 
 
 def test_task_with_full_emailer_message():
@@ -1036,7 +1078,7 @@ def test_call_summary_with_structured_items():
     assert len(summary.next_steps) == 1
     assert isinstance(summary.next_steps[0], CallSummaryNextStep)
     assert summary.next_steps[0].step == "Schedule demo"
-    assert summary.next_steps[0].due_at == "2026-02-24T00:00:00Z"
+    assert summary.next_steps[0].due_at == datetime(2026, 2, 24, tzinfo=UTC)
     assert len(summary.pain_points) == 1
     assert isinstance(summary.pain_points[0], CallSummaryPoint)
     assert summary.pain_points[0].text == "Catching bugs early"

{qodev_apollo_api-0.1.2 → qodev_apollo_api-0.2.0}/tests/test_utils.py

@@ -2,7 +2,11 @@
 
 import json
 
-from qodev_apollo_api.utils import normalize_linkedin_url, prosemirror_to_markdown
+from qodev_apollo_api.utils import (
+    markdown_to_prosemirror,
+    normalize_linkedin_url,
+    prosemirror_to_markdown,
+)
 
 
 def test_prosemirror_to_markdown_simple():
@@ -111,3 +115,60 @@ def test_normalize_linkedin_url_empty():
     """Test normalization of empty URL."""
     assert normalize_linkedin_url("") == ""
     assert normalize_linkedin_url(None) == ""
+
+
+def test_markdown_to_prosemirror_simple():
+    """Title + body become a noteTitle node and a paragraph node."""
+    doc = json.loads(markdown_to_prosemirror("Hello world", title="My Title"))
+    assert doc["type"] == "doc"
+    assert doc["content"][0] == {
+        "type": "noteTitle",
+        "content": [{"type": "text", "text": "My Title"}],
+    }
+    assert doc["content"][1] == {
+        "type": "paragraph",
+        "content": [{"type": "text", "text": "Hello world"}],
+    }
+
+
+def test_markdown_to_prosemirror_blank_line_is_empty_paragraph():
+    """Blank lines become bare paragraphs (empty text nodes are invalid)."""
+    doc = json.loads(markdown_to_prosemirror("a\n\nb"))
+    assert doc["content"] == [
+        {"type": "paragraph", "content": [{"type": "text", "text": "a"}]},
+        {"type": "paragraph"},
+        {"type": "paragraph", "content": [{"type": "text", "text": "b"}]},
+    ]
+
+
+def test_markdown_to_prosemirror_no_title():
+    """Without a title there is no noteTitle node."""
+    doc = json.loads(markdown_to_prosemirror("body only"))
+    assert all(n["type"] != "noteTitle" for n in doc["content"])
+
+
+def test_markdown_to_prosemirror_lists():
+    """Bullet and ordered list lines become bulletList / orderedList nodes."""
+    doc = json.loads(markdown_to_prosemirror("- one\n- two\n1. first\n2. second"))
+    types = [n["type"] for n in doc["content"]]
+    assert types == ["bulletList", "orderedList"]
+    assert len(doc["content"][0]["content"]) == 2  # two bullet items
+    assert doc["content"][0]["content"][0]["content"][0]["content"][0]["text"] == "one"
+    assert doc["content"][1]["content"][1]["content"][0]["content"][0]["text"] == "second"
+
+
+def test_markdown_prosemirror_roundtrip():
+    """markdown_to_prosemirror output is readable back by prosemirror_to_markdown."""
+    pm = markdown_to_prosemirror("First para\n\nSecond para", title="Round Trip")
+    title, markdown = prosemirror_to_markdown(pm)
+    assert title == "Round Trip"
+    assert markdown == "First para\n\nSecond para"
+
+
+def test_markdown_to_prosemirror_empty():
+    """Empty content yields a valid empty doc."""
+    doc = json.loads(markdown_to_prosemirror(""))
+    assert doc["type"] == "doc"
+    # No invalid empty text nodes anywhere.
+    for node in doc["content"]:
+        assert node.get("content") != [{"type": "text", "text": ""}]

{qodev_apollo_api-0.1.2 → qodev_apollo_api-0.2.0}/uv.lock

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