fix: handle empty 2xx responses

[devin-ai-integration]

This PR adds a check to ensure empty successful responses don't cause JSON deserialization errors.

Changes

Modified the request method in Connection class to check for response content before attempting JSON deserialization:

if r.ok:
    if r.content and len(r.content) > 0:
        try:
            return r.json()
        except JSONDecodeError:
            return None
    return None

This change specifically addresses endpoints like users.delete() that return a 2xx response without a body, preventing unnecessary JSON decode errors while maintaining the existing behavior for responses with content.

Link to Devin run: https://app.devin.ai/sessions/70fbcf177d2747d4bf61393c76788909

[devin-ai-integration]

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

• Address comments on this PR. Add "(aside)" to your comment to have me ignore it.
• Look at CI failures and help fix them

⚙️ Control Options:

• Disable automatic comment and CI monitoring

[mattmikolay]

Looks reasonable to me!