fix: sanitize "+" in uri

[hunshcn]

No description provided.

[stackblitz]

Run & review this pull request in StackBlitz Codeflow.

[changeset-bot]

🦋 Changeset detected

Latest commit: e8b3058

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]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
hey-api-docs	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Feb 9, 2025 11:20am

[mrlubos]

Hi @hunshcn, just to clarify, are you still working with the legacy parser? What's your configuration?

Regarding this pull request, please add a change log (pnpm changeset) and ideally there'd be an actual specification test with a snapshot. But if this is only for the legacy parser, you can skip that!

[hunshcn]

Hi @hunshcn, just to clarify, are you still working with the legacy parser? What's your configuration?

I has a endpoint contains "+". The uri like /api/v1/resources+overview.

After convert openapi-ts -i path/to/openapi.json -o src/client -c @hey-api/client-fetch, I will get a function named apiV1Resources+overview which is incorrect.

[mrlubos]

Yeah that makes sense. Are you able to add the requested changes?

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 57.16%. Comparing base (1df55ed) to head (e8b3058).
Report is 5 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #1687   +/-   ##
=======================================
  Coverage   57.16%   57.16%           
=======================================
  Files         189      189           
  Lines       25620    25620           
  Branches     1911     1911           
=======================================
  Hits        14646    14646           
  Misses      10965    10965           
  Partials        9        9           

Flag	Coverage Δ
unittests	57.16% <100.00%> (ø)

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.

[pkg-pr-new]

Open in Stackblitz

@hey-api/client-axios

npm i https://pkg.pr.new/hey-api/openapi-ts/@hey-api/client-axios@1687

@hey-api/client-fetch

npm i https://pkg.pr.new/hey-api/openapi-ts/@hey-api/client-fetch@1687

@hey-api/client-next

npm i https://pkg.pr.new/hey-api/openapi-ts/@hey-api/client-next@1687

@hey-api/client-nuxt

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

@hey-api/openapi-ts

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

commit: e8b3058

[mrlubos]

Thank you @hunshcn! I will aim to release this by Monday