Skip to content

Conversation

@pavadeli
Copy link
Member

No description provided.

pavadeli and others added 11 commits October 23, 2024 13:37
…ans and numbers (#104)

See rationale in #47.

BREAKING CHANGE: `string.autoCast` no longer supports `null`, `undefined`, symbols, objects and functions and anything other than strings, numbers and booleans really.
Every type used to have a `.autoCast` and `.autoCastAll` getter that returns a clone of the type with autocast parsers added. This PR changes that to be separate functions with the same name. This helps with future changes, because maintaining getters that return the `this` types is becoming increasingly hard.

BREAKING CHANGE: The getters `.autoCast` and `.autoCastAll` are removed and moved to separately exported functions.
BREAKING CHANGE: The `type`, `input` and `details` properties of `ValidationError` used to be writable, they are now marked as readonly.
With this method it is now possible to define a default value for a required property inside an object.
…tersectionType` (#110)

* feat: split `visitObjectLikeType` into `visitObjectType` and `visitIntersectionType`

Recently `withOptional` was changed to no longer return an `IntersectionType`. It returns a normal `InterfaceType` instead. The `Visitor<R>` interface used to combine intersections and "normal objects" into a single `visitObjectLikeType` to prevent confusion, but that confusion is probably gone now. Therefore, the `visitObjectLikeType` is now split into `visitObjectType` and `visitIntersectionType`.

BREAKING CHANGE: `Visitor<R>` interface now requires two new methods `visitObjectType` and `visitIntersectionType` while the existing `visitObjectLikeType` is removed.

* chore: update docs
* feat: Introduce StandardSchemaV1 interface in Skunkteam Types

Hooked into the existing Valid/Invalid Conversion tests.

* chore: Forgot to run pre-pr command

* suggesting simpler messages in standard validate

---------

Co-authored-by: Paco van der Linden <[email protected]>
@pavadeli pavadeli marked this pull request as ready for review February 18, 2025 11:02
@pavadeli pavadeli merged commit 36a3554 into main Feb 18, 2025
5 of 7 checks passed
@github-actions
Copy link

🎉 This PR is included in version 9.0.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants