chore (provider-utils): replace zod with standard-schema where possible #6184
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
lgrammel
reviewed
May 14, 2025
| schema: Schema<ELEMENT>, | ||
| ): OutputStrategy<ELEMENT[], ELEMENT[], AsyncIterableStream<ELEMENT>> => { | ||
| const arrayOutputStrategy = <OBJECT>( | ||
| schema: Schema<OBJECT>, |
There was a problem hiding this comment.
ELEMENT was on purpose because it is an array element
gr2m
force-pushed
the
v5-gr2m/5766-replace-zod-with-standard-schema
branch
from
6cf46ef to
1d4bb39
Compare
gr2m
commented
May 15, 2025
| @@ -19,7 +20,7 @@ export type Validator<OBJECT = unknown> = { | |||
| * Optional. Validates that the structure of a value matches this schema, | |||
| * and returns a typed version of the value if it does. | |||
| */ | |||
| readonly validate?: (value: unknown) => ValidationResult<OBJECT>; | |||
| readonly validate?: (value: unknown) => ValidationResult<OBJECT> | PromiseLike<ValidationResult<OBJECT>>; | |||
There was a problem hiding this comment.
no sure if making the validate() method optionally async could cause any problems down the road, but it made the transition easier.
gr2m
commented
May 15, 2025
| @@ -105,6 +105,7 @@ export function processUIMessageStream<MESSAGE_METADATA = unknown>({ | |||
| if (messageMetadataSchema != null) { | |||
| await validateTypes({ | |||
| value: mergedMetadata, | |||
| // @ts-ignore TODO: Couldn't figure this one out yet ~@gr2m | |||
There was a problem hiding this comment.
@lgrammel if you like have a look at this type error. It's probably something obvious that I'll figure out quickly after some rest.
…://github.com/vercel/ai into v5-gr2m/5766-replace-zod-with-standard-schema
lgrammel
approved these changes
May 15, 2025
lgrammel
changed the title
zod with standard-schema
|
Thanks for getting it over the line @lgrammel 🙏🏼 |
This was referenced May 15, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Background
Standard schema allows validation with any schema library that implements it. While we still need to support specific schema libraries in many places because of the JSONSchema conversion, switching to standard schema support where possible will increase the flexibility of the AI SDK.
Summary
Replace
zodwithstandard-schemawhere users can pass their own schemas for validation.Related Issues
Closes #5766