Skip to content

feat(curl): add GET data aggregation#304

Open
subsetpark wants to merge 1 commit into
vercel-labs:mainfrom
flowglad:agent/curl-get-data
Open

feat(curl): add GET data aggregation#304
subsetpark wants to merge 1 commit into
vercel-labs:mainfrom
flowglad:agent/curl-get-data

Conversation

@subsetpark

@subsetpark subsetpark commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Add curl -G/--get support so data options are appended to the query string.
  • Preserve command-line order across repeated inline and file-backed data options.
  • Match curl URL-encoding and default form content-type behavior.

Why

The current implementation overwrites repeated data flags and cannot move their payload to the URL. It also diverges from curl for spaces, percent-escape casing, and the default content type.

Validation

  • 63 focused curl tests
  • 3 live comparisons against system curl
  • pnpm lint
  • pnpm knip
  • pnpm --filter just-bash typecheck
  • pnpm --filter just-bash build
  • Full test:run: 13,627 passed; one unrelated security hypothesis test was flaky in the combined run and passed on immediate isolated retry.

@vercel

vercel Bot commented Jul 16, 2026

Copy link
Copy Markdown

@subsetpark is attempting to deploy a commit to the Vercel Labs Team on Vercel.

A member of the Team first needs to authorize it.

@subsetpark
subsetpark marked this pull request as ready for review July 16, 2026 19:38
@subsetpark
subsetpark requested a review from cramforce as a code owner July 16, 2026 19:38
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