Skip to content

Conversation

@scottlovegrove
Copy link
Collaborator

Summary

  • Fixed batch POST requests that were failing with 400 errors about missing required arguments
  • The batch request builder now correctly includes a body field for non-GET requests containing URL-encoded parameters
  • Previously, only method and url fields were included in batch requests, which worked for GET requests (where params are in the URL) but not for POST requests (where params need to be in the body)

Test plan

  • Manually tested api.inbox.archiveThread(id, {batch: true}) which now returns 200 instead of 400
  • Run existing test suite to ensure no regressions
  • Consider adding test coverage for batch POST requests

🤖 Generated with Claude Code

POST requests in batch mode were missing the required body field containing URL-encoded parameters. This caused the API to return 400 errors about missing arguments. The batch request builder now correctly includes a body field for non-GET requests with the URL-encoded parameters.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
@scottlovegrove scottlovegrove merged commit f765248 into main Dec 16, 2025
3 checks passed
@scottlovegrove scottlovegrove deleted the scottl/fix-batch-posts branch December 16, 2025 12:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant