fix: use zod instead of zod/v4 for tool type compatibility #130
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.
Summary
zodimportszod/v4, but users typically import fromzod, causing TypeScript errors due to internal type metadata differencesChanges
ZodType,ZodObject,ZodRawShapefromzodinstead ofzod/v4intool-types.ts,tool.ts, andtool-executor.tstoJSONSchema,ZodError) fromzod/v4as they're only available thereconvertZodToJsonSchemato bridge the type gap betweenzodandzod/v4toJSONSchematarget fromopenapi-3.0todraft-7(former isn't supported in current Zod version)draft-7targetError Before Fix
Test plan
pnpm run build)npx vitest --run)zodimports