Return response objects from all client methods

[hownowstephen]

Summary

• send_request() now returns the requests.Response object instead of response.text
• Track API methods (identify, track, pageview, backfill, delete, add_device, delete_device, suppress, unsuppress, merge_customers) now return the response instead of None
• APIClient methods (send_email, send_push, etc.) continue to return parsed JSON dicts — switched from json.loads(resp) to resp.json()

Breaking change

Track API methods previously returned None. Code that checked result is None or relied on the falsy return will need updating. However, the most common usage pattern (cio.identify(...) without capturing return) is unaffected.

Example

# Before: no way to inspect the response
cio.identify(id="123", name="Alice")

# After: response available if needed
resp = cio.identify(id="123", name="Alice")
print(resp.status_code)  # 200

Closes #85

Test plan

• Updated test_client_base.py assertions for Response objects
• All existing Track API tests pass (use hooks, don't assert return values)
• All existing APIClient tests pass (return value contract unchanged — still dicts)
• Full test suite passes (28 tests)
• Lint passes

---

Note

Medium Risk
This is a small but behavior-changing API shift: send_request and all Track API calls now return requests.Response, which may break callers that assumed None/string returns or compared return values directly. Core request/response handling is touched, but logic changes are straightforward and covered by updated tests.

Overview
Updates the core HTTP layer so ClientBase.send_request() returns the full requests.Response object (instead of response.text).

Propagates this through the Track client so methods like identify, track, delete, merge_customers, and batch now return the response rather than None, enabling callers to inspect status codes/headers.

Keeps APIClient return values as parsed JSON dicts, but switches parsing to resp.json() and removes the now-unused json import. Tests are updated to assert on Response attributes (e.g., status_code) instead of raw strings.

^{Reviewed by Cursor Bugbot for commit b07b660. Bugbot is set up for automated code reviews on this repo. Configure here.}

[cursor]

Cursor Bugbot has reviewed your changes and found 2 potential issues.

^{❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.}

^{Reviewed by Cursor Bugbot for commit a91619d. Configure here.}