Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.

Greptile Summary

Fixes multipart passthrough dropping all but the last file when a client sends multiple parts under the same field name (e.g. -F [email protected] -F [email protected]). The root cause was FormData.items() collapsing duplicate keys and a dict keyed by field name overwriting earlier entries.

• make_multipart_http_request now reads form.multi_items() (a stable list in Starlette), builds files as a list of (field_name, (filename, content, content_type)) tuples, and groups repeated non-file fields into list values — both formats httpx encodes as separate multipart parts.
• Tests are updated to pass FormData([...]) instead of a plain dict, assert on the list-of-tuples shape, and include a new regression test verifying that two files under the same field name and two non-file values under the same key are both forwarded.

Confidence Score: 5/5

Safe to merge — the change is a targeted, well-tested fix to multipart forwarding with no impact on other request paths.

The fix is correct: multi_items() returns a list in Starlette so iterating it twice (once for files, once for non-file fields) is safe, and httpx accepts both list-of-tuples for files and dict-of-lists for data. Single-value non-file fields now arrive as ["value"] instead of "value", which httpx encodes identically. All previous tests are updated to match the new shapes, and the new regression test directly exercises the duplicate-key scenario without any assertion-collapsing workarounds.

No files require special attention.

_{Reviews (2): Last reviewed commit: "refactor(passthrough): address multipart..." | Re-trigger Greptile}

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@greptile review again with new commit that resolves the p2 issues