hey-api / openapi-ts

🚀 The OpenAPI to TypeScript codegen. Generate clients, SDKs, validators, and more. Support: @mrlubos
https://heyapi.dev
Other
1.31k stars 102 forks source link

Zod plugin #876

Open ZilvinasAbr opened 3 months ago

ZilvinasAbr commented 3 months ago

Description

Currently openapi-ts generates typescript definitions for the OpenAPI schema. Would it be possible to generate zod schemas also? Something similar like these libraries: https://github.com/astahmer/openapi-zod-client and https://github.com/astahmer/typed-openapi

mrlubos commented 3 months ago

Sure would! Let's keep this issue open to gauge the interest and will prioritise accordingly

jacobdr commented 3 months ago

We sort of go the other way -- we use fastify-type-provider-zod as the backbone to generate the OpenAPI schema. But I see a use case where you would want to generate the zod validators from the OpenAPI schema if you build one by hand, and then import those into a UI package for integration into something like react-hook-form and its zod validation provider

ZilvinasAbr commented 3 months ago

We sort of go the other way -- we use fastify-type-provider-zod as the backbone to generate the OpenAPI schema. But I see a use case where you would want to generate the zod validators from the OpenAPI schema if you build one by hand, and then import those into a UI package for integration into something like react-hook-form and its zod validation provider

Correct. The situation I find myself is a non-js backend which generates OpenAPI schema and it would be nice to be able to get not only type definitions of the OpenAPI schema but also zod schemas

RobertOstermann commented 3 months ago

I agree, it would be super helpful to have the zod schemas generated for scenarios with non-js backends.

mrlubos commented 2 months ago

For people wanting to generate Zod schemas, do you need 1:1 schema matching the endpoint, so you can use it to validate request/response? Which one of those is higher priority?

vyacheslav-korneev commented 2 months ago

@mrlubos In our case we'd like to validate server response to be sure that it complies with the schema that was provided.

Out of curiosity, when may this feature be available if you start working on it? Any ETAs so we can understand whether we need to go with any custom solution?

mrlubos commented 2 months ago

@vyacheslav-korneev Definitely go with custom, I don't do ETAs for now

1saifj commented 2 months ago

love to see this in next release, excellent work dear @mrlubos 👏 👏 👏

mrlubos commented 2 months ago

Hey I said nothing about next release, don't try to bully me into shipping it that fast @1saifj 🤣 Would be good to collect some information first. I assume people here are already using other packages for generating Zod schemas such as those linked above. Why are those not sufficient/what benefit would you gain by having Hey API generate Zod schemas?

1saifj commented 2 months ago

@mrlubos idk i just love all in one stuffs xD

mrlubos commented 2 months ago

We will get there my friend, it's just a question of priorities and what can have the biggest impact

ZilvinasAbr commented 2 months ago

@mrlubos Thanks for the question. I guess for my personal case, for starters it would be most beneficial to have schemas for all the OpenAPI resources/entities. E.g.

For resource/entity User there would also be a zod schema generated.

Also, I noticed that currently this library actually generates JSON Schema of what I just mentioned. I might first try to use some tool to convert those JSON schemas to zod schemas first 🤔

After that, it would be nice to have zod schemas for each endpoint payload and response, but that is probably secondary for now

jasonlor commented 2 months ago

Would be incredible to be able to generate a react query client that uses fetch API, with request/responses validated with zod from an openapi spec

mrlubos commented 2 months ago

@jasonlor I agree. Out of curiosity, why do you need to validate both? Do you not trust the API to uphold the OpenAPI contract?

jasonlor commented 2 months ago

@mrlubos Ah the intent is to help out with overall productivity. I have two developers who are working with each other (server/client) and sometimes encounters issues due to incorrectly formed requests or missing objects on the response side.

This would help enforce the interface and help streamline any unintended errors that might be present. Other benefits w/ react query would be from Tkdodo regarding type inference:

Here's some existing work that I dove into this morning that could be helpful to reference –  https://github.com/mattpocock/zod-fetch

For now I'll be generating the zod schemas from the types.gen.ts file using https://github.com/fabien0102/ts-to-zod And the manually defining the hooks with the recently released react-query plugin

AdamDorwart commented 1 month ago

Just wanted to throw another use case in the pool. I have a Python backend that generates an OpenAPI schema. I would love to:

There's a case for I shouldn't couple input form validation with backend model definitions but sometimes it makes things easier. Zod isn't a requirement - I just want my models with validation rules available and zod is pretty good.

thoroc commented 1 week ago

For people wanting to generate Zod schemas, do you need 1:1 schema matching the endpoint, so you can use it to validate request/response? Which one of those is higher priority?

That would definitely help as the AWS Powertools uses Zod for parsing events https://docs.powertools.aws.dev/lambda/typescript/latest/utilities/parser/#built-in-schemas