feat(client): added angular, axios, fetch, next & nuxt client type export for external typing purposes;

[alexedme]

feat(client): added angular, axios, fetch, next & nuxt client type export for external typing purposes;

Closes #2533

[bolt-new-by-stackblitz]

Run & review this pull request in StackBlitz Codeflow.

[changeset-bot]

🦋 Changeset detected

Latest commit: efdeedc

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]

@alexedme 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/nuxt@2535

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

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

commit: efdeedc

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 24.00%. Comparing base (59ec7a0) to head (efdeedc).
⚠️ Report is 3 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #2535   +/-   ##
=======================================
  Coverage   24.00%   24.00%           
=======================================
  Files         359      359           
  Lines       35540    35540           
  Branches     1540     1540           
=======================================
  Hits         8533     8533           
  Misses      26994    26994           
  Partials       13       13           

Flag	Coverage Δ
unittests	24.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.

🚀 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]

Why do you need to re-export from both types and index?

[alexedme]

My fault.

The change in the types file is redundant and not necessary. At first, I thought of creating a new type in the following way:

import type { Client } from './bundle/types';

export AxiosClient = Client

But opted afterwards to simply re-export the type with an alias to avoid conflict with Client namespace (packages/openapi-ts/src/plugins/@hey-api/client-core/types.d.ts line 17).

I will revert the changes in the types file.

Thank you for letting me know.

[mrlubos]

Open in StackBlitz

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

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

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

commit: 6757b16

@alexedme you can use the previews to verify it works as expected on your end once actually released

[mrlubos]

Looking good – thank you!

[alexedme]

Looking good – thank you!

I should to be the one thanking you for allowing me to contribute to this great lib. It was a nice experience.