Skip to content

fix: omit empty comma array query params#1656

Open
ynachiket wants to merge 1 commit into
anthropics:mainfrom
ynachiket:ynachiket/test-querystring-comma-empty-20260606
Open

fix: omit empty comma array query params#1656
ynachiket wants to merge 1 commit into
anthropics:mainfrom
ynachiket:ynachiket/test-querystring-comma-empty-20260606

Conversation

@ynachiket
Copy link
Copy Markdown

Summary

  • omit comma-formatted array query params when the array has no serializable values
  • add coverage for empty, None-only, and nested empty comma arrays

Validation

  • PYTHONPATH=src /tmp/anthropic-pdlc042-venv/bin/python -m pytest tests/test_qs.py -q
  • PYTHONPATH=src /tmp/anthropic-pdlc042-venv/bin/python -m py_compile src/anthropic/_qs.py tests/test_qs.py
  • git diff --check

@ynachiket ynachiket requested a review from a team as a code owner June 6, 2026 10:24
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