Should pipes abort early by default?

[IlyaSemenov]

Following #618, I am wondering why pipes don't abort early by default? What I mean is, take the example from valibot documentation:

  const EmailSchema = v.pipe(v.string(), v.email(), v.endsWith('@example.com'));
  const email = v.parse(EmailSchema, '[email protected]', {
    abortPipeEarly: true,
  });

Naturally, there is no point in testing (and reporting) the string ending if it's not even an email address, and that's an inner property of a particular pipe, not the whole schema. Arguably, the majority of pipes will be structured similarly, and only select few pipes will test different aspects independently. For those cases, I'd say pipe is not even a good name. In real life, if something is stuck in the pipe (=fails validation) it's not moved further along. If liquid goes in parallel, it's a manifold, not a pipe.

I propose to set abortPipeEarly by default, and allow to revert this with v.config() — opposite to how it works now.

[fabian-hiller]

For context, in the beginning Valibot only returned a single issue for a path, and later expanded that because we had several people requesting it. I think there are legitimate examples for both preferences. For anyone who prefers abortPipeEarly to be true by default, you can add the following line to your project to change the default configuration.

import * as v from 'valibot';

v.setGlobalConfig({ abortPipeEarly: true });

[IlyaSemenov]

Thanks for the heads up. It didn't occur to me that setGlobalConfig could change this behaviour, too.

With that in mind, we shouldn't change the default, or it'll be kind of a footgun. If abortPipeEarly: true were the default and everyone was relying on it (like in EmailSchema example), then changing it in global config would make many schemas emit weird/duplicate errors or even crash.

So it should be either kept as is, or it should be early aborting by default without having a way to override this in global config.