fix: keep async capture_ai from raising on a non-string event name

Author: eli-r-ph

posthog/async_client.py

@@ -587,7 +587,7 @@ def capture_ai(
         return self._capture(event, self._ai_lane, kwargs)
 
     def _log_non_ai_event(self, event: str) -> None:
-        if not event.startswith("$ai_"):
+        if isinstance(event, str) and not event.startswith("$ai_"):
             self.log.debug(
                 "capture_ai called with non-AI event name %r; routing it to the AI endpoint anyway.",
                 event,

posthog/test/test_async_client.py

@@ -374,6 +374,19 @@ async def send_batch(api_key, host, batch, **kwargs):
     assert client._all_worker_tasks() == []
 
 
+@pytest.mark.asyncio
+@pytest.mark.parametrize("event", [None, 123])
+async def test_ai_capture_handles_a_non_string_event_name_like_capture(event):
+    client = AsyncPosthog("test-key", send=False)
+    try:
+        expected = client.capture(event, distinct_id="user-1") is None
+        assert (client.capture_ai(event, distinct_id="user-1") is None) is expected
+        immediate = await client.capture_ai_immediate(event, distinct_id="user-1")
+        assert (immediate is None) is expected
+    finally:
+        await client.shutdown()
+
+
 @pytest.mark.asyncio
 async def test_send_false_accepts_without_starting_workers_or_transport():
     with mock.patch("posthog.async_client._build_client") as build_client: