Skip to content

Fix binary object handling in multipart/form-data (File / Blob) when using nuxt#3459

Merged
mrlubos merged 9 commits intohey-api:mainfrom
LeiCraft:fix/binary-object-handling-in-multipart-form-data-in-nuxt-client
Feb 27, 2026
Merged

Fix binary object handling in multipart/form-data (File / Blob) when using nuxt#3459
mrlubos merged 9 commits intohey-api:mainfrom
LeiCraft:fix/binary-object-handling-in-multipart-form-data-in-nuxt-client

Conversation

@LeiCraft
Copy link
Contributor

When the body is FormData and includes a file object, the unwrapRefs logic would incorrectly convert the File / Blob values to plain JavaScript objects.
This caused binary values to be serialized incorrectly, breaking file upload.

so in order to fix the issue i updated unwrapRefs to correctly preserve File and Blob instances.

@bolt-new-by-stackblitz
Copy link

Review PR in StackBlitz Codeflow Run & review this pull request in StackBlitz Codeflow.

@vercel
Copy link

vercel bot commented Feb 26, 2026

@LeiCraft is attempting to deploy a commit to the Hey API Team on Vercel.

A member of the Team first needs to authorize it.

@pullfrog
Copy link

pullfrog bot commented Feb 26, 2026

Leaping into action...

Pullfrog  | View workflow run | Using OpenCode | Triggered by Pullfrogpullfrog.com𝕏

@dosubot dosubot bot added the size:M This PR changes 30-99 lines, ignoring generated files. label Feb 26, 2026
@changeset-bot
Copy link

changeset-bot bot commented Feb 26, 2026

🦋 Changeset detected

Latest commit: 6bef793

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@hey-api/openapi-ts Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@dosubot dosubot bot added the bug 🔥 Broken or incorrect behavior. label Feb 26, 2026
@codecov
Copy link

codecov bot commented Feb 26, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 40.15%. Comparing base (7ef9332) to head (6bef793).
⚠️ Report is 10 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3459      +/-   ##
==========================================
+ Coverage   40.10%   40.15%   +0.04%     
==========================================
  Files         478      478              
  Lines       17494    17496       +2     
  Branches     5305     5306       +1     
==========================================
+ Hits         7016     7025       +9     
+ Misses       8426     8421       -5     
+ Partials     2052     2050       -2     
Flag Coverage Δ
unittests 40.15% <100.00%> (+0.04%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copy link
Member

@mrlubos mrlubos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good, thank you for the fix!

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Feb 27, 2026
@mrlubos mrlubos merged commit 12c76b8 into hey-api:main Feb 27, 2026
5 of 16 checks passed
@hey-api hey-api bot mentioned this pull request Feb 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug 🔥 Broken or incorrect behavior. lgtm This PR has been approved by a maintainer size:M This PR changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants