[Gel]: Add support for Gel-only column types in drizzle-zod/-arktype/-valibot/-typebox#6033
Open
allocsys wants to merge 1 commit into
Open
[Gel]: Add support for Gel-only column types in drizzle-zod/-arktype/-valibot/-typebox#6033allocsys wants to merge 1 commit into
allocsys wants to merge 1 commit into
Conversation
…-valibot/-typebox Previously, the 6 Gel-only column types (dateDuration, duration, relDuration, localDate, localTime, localDateTime) fell through to a permissive any-type schema in all four schema-integration packages, so schemas generated for Gel tables silently skipped validation on these columns. - Add real schema validation for the 6 Gel-only column types in drizzle-zod, drizzle-arktype, drizzle-valibot, and drizzle-typebox - Add gel as an optional peerDependency and devDependency in each of the 4 packages - drizzle-arktype: use 6 concrete per-type schemas instead of a generic instanceof-based schema, since arktype's validateChainedAsArgs type check rejects the generic form - Add tests per package verifying the new schemas accept real Gel instances and reject non-matching values
allocsys
marked this pull request as ready for review
July 18, 2026 15:34
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
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.
What
Adds real schema validation for the 6 Gel-only column types across the four schema-integration packages:
dateDuration,duration,relDuration,localDate,localTime,localDateTime.Previously, these types fell through to a permissive any-type schema in
drizzle-zod,drizzle-arktype,drizzle-valibot, anddrizzle-typebox, so schemas generated for Gel tables silently skipped validation on these columns.Changes
drizzle-zod,drizzle-arktype,drizzle-valibot, anddrizzle-typeboxgelas an optionalpeerDependencyanddevDependencyin each of the 4 packagesdrizzle-arktype: use 6 concrete per-type schemas instead of a generic instanceof-based schema, since arktype'svalidateChainedAsArgstype check rejects the generic formTesting
Added
tests/gel.test.tsin each of the 4 packages, covering both acceptance of real Gel instances and rejection of non-matching values for all 6 new column types.Notes for reviewers
Gel) per the[<dialect name>]: <subject>convention rather than a specific package name.