search

discordjs / builders

A collection of builders that you can use when creating your bot.
Apache License 2.0
97 stars 37 forks source link

Validation errors #71

Open KokoNeotSide opened 2 years ago

KokoNeotSide commented 2 years ago

Feature

Please revert back the validation errors. This new "Zod" thing is terrible. I've been searching for 2h where the issue is because it doesn't tell you where the error is.

Ideal solution or implementation

Revert back to the previous validator (ow) which said what exact fields and commands are causing issues

Alternative solutions or implementations

No response

Other context

image image

the first image, I'm missing description for one command, there is no way for me to read that from the error. the second image, I accidentally made one command name first letter capital. All I know that my regex is invalid somewhere in 50 commands. This is terrible

kyranet commented 2 years ago

That's not happening, ow switched to ESM-only and has caused many issues in the past due to it.

We'll most likely switch to ShapeShift, which Sapphire devs have been building to improve the experience and performance of the validation process.

KhafraDev commented 2 years ago

@kyranet Are there any benchmarks for shapeshift compared to zod or other validation libs?

kyranet commented 2 years ago

Yes, @vladfrangu has them.