Skip to content

fix(@hey-api/sdk): correctly map flat parameters#3047

Merged
mrlubos merged 1 commit intomainfrom
fix/sdk-flat-map
Dec 7, 2025
Merged

fix(@hey-api/sdk): correctly map flat parameters#3047
mrlubos merged 1 commit intomainfrom
fix/sdk-flat-map

Conversation

@mrlubos
Copy link
Copy Markdown
Member

@mrlubos mrlubos commented Dec 7, 2025

Fixes #2984

@bolt-new-by-stackblitz
Copy link
Copy Markdown

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

@changeset-bot
Copy link
Copy Markdown

changeset-bot bot commented Dec 7, 2025

🦋 Changeset detected

Latest commit: 55ef279

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
Copy link
Copy Markdown

vercel bot commented Dec 7, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
hey-api-docs Ready Ready Preview Comment Dec 7, 2025 11:25pm

@mrlubos mrlubos marked this pull request as ready for review December 7, 2025 23:04
@dosubot dosubot bot added the size:L This PR changes 100-499 lines, ignoring generated files. label Dec 7, 2025
@dosubot dosubot bot added bug 🔥 Broken or incorrect behavior. client labels Dec 7, 2025
@codecov
Copy link
Copy Markdown

codecov bot commented Dec 7, 2025

Codecov Report

❌ Patch coverage is 0% with 6 lines in your changes missing coverage. Please review.
✅ Project coverage is 33.01%. Comparing base (7f52c34) to head (55ef279).
⚠️ Report is 6 commits behind head on main.

Files with missing lines Patch % Lines
...pi-ts/src/plugins/@hey-api/sdk/shared/operation.ts 0.00% 6 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3047      +/-   ##
==========================================
- Coverage   33.01%   33.01%   -0.01%     
==========================================
  Files         426      426              
  Lines       32792    32797       +5     
  Branches     2126     2126              
==========================================
  Hits        10827    10827              
- Misses      21937    21942       +5     
  Partials       28       28              
Flag Coverage Δ
unittests 33.01% <0.00%> (-0.01%) ⬇️

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.

@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new bot commented Dec 7, 2025

Open in StackBlitz

npm i https://pkg.pr.new/hey-api/openapi-ts/@hey-api/codegen-core@3047
npm i https://pkg.pr.new/hey-api/openapi-ts/@hey-api/nuxt@3047
npm i https://pkg.pr.new/hey-api/openapi-ts/@hey-api/openapi-ts@3047
npm i https://pkg.pr.new/hey-api/openapi-ts/@hey-api/vite-plugin@3047

commit: 55ef279

@mrlubos mrlubos merged commit 2b11229 into main Dec 7, 2025
13 of 15 checks passed
@mrlubos mrlubos deleted the fix/sdk-flat-map branch December 7, 2025 23:37
@hey-api hey-api bot mentioned this pull request Dec 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug 🔥 Broken or incorrect behavior. size:L This PR changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Path parameter incorrectly serialized as /servers/id,<id> instead of /servers/<id> on paramsStructure flat

1 participant