fix(client): node compatible mergeHeaders

[volesen]

Closes #2230
Closes #2521
Related #2539

Alternatively, I think the cleanest approach would be to handle Header instances separately, as they always contain string as values.

[bolt-new-by-stackblitz]

Run & review this pull request in StackBlitz Codeflow.

[changeset-bot]

🦋 Changeset detected

Latest commit: 391ced3

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

[vercel]

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

A member of the Team first needs to authorize it.

[pkg-pr-new]

Open in StackBlitz

npm i https://pkg.pr.new/hey-api/openapi-ts/@hey-api/codegen-core@2585

npm i https://pkg.pr.new/hey-api/openapi-ts/@hey-api/nuxt@2585

npm i https://pkg.pr.new/hey-api/openapi-ts/@hey-api/openapi-ts@2585

npm i https://pkg.pr.new/hey-api/openapi-ts/@hey-api/vite-plugin@2585

commit: 391ced3

[codecov]

Codecov Report

❌ Patch coverage is 79.31034% with 6 lines in your changes missing coverage. Please review.
✅ Project coverage is 24.19%. Comparing base (ec26321) to head (391ced3).
⚠️ Report is 5 commits behind head on main.

Files with missing lines	Patch %	Lines
...s/src/plugins/@hey-api/client-nuxt/bundle/utils.ts	25.00%	6 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #2585      +/-   ##
==========================================
+ Coverage   24.16%   24.19%   +0.03%     
==========================================
  Files         363      363              
  Lines       36629    36654      +25     
  Branches     1644     1650       +6     
==========================================
+ Hits         8850     8869      +19     
- Misses      27766    27772       +6     
  Partials       13       13              

Flag	Coverage Δ
unittests	24.19% <79.31%> (+0.03%)	⬆️

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.

[mrlubos]

Hey @volesen can you add the same logic to the other clients so they maintain feature parity? And what's the alternative approach you mention?

[volesen]

@mrlubos, I just updated all instances using Header.entries - the remaining client implementations are using custom header classes

[mrlubos]

This looks great, thank you!