Allow providing a default shouldRevalidate implementation

[jeyj0]

Throughout my current project, I am using conform for my form validation. One of its features is server-side form validation, which can occur onInput. When combining these with remix forms (I have only checked behavior with fetcher.Form, but I don't see why Form shouldn't have the same issue), lots of requests are made to the action for validation (expected and wanted), which causes (by default) all loaders for all currently active routes to be revalidated (expected, but not wanted). To avoid these unnecessary requests, I am using a simple shouldRevalidate function for all of my routes (see below for my helper function that still allows additional customization for when it's necessary).

This function is currently used on every route in my project.

It would be nice to be able to configure this default shouldRevalidate function somewhere, so I don't have to remember adding it to all routes.

I'm also open to hearing advice on alternative approaches if they exist.

export function shouldRevalidate_ignoreValidateIntent(
  fn?: ShouldRevalidateFunction,
) {
  return (args: ShouldRevalidateFunctionArgs) => {
    try {
      if (args.formData !== undefined) {
        const intent = args.formData.get("__intent__");
        if (intent !== null && typeof intent === "string") {
          const value = JSON.parse(intent);
          if (
            typeof value === "object" &&
            value !== null &&
            hasOwnProperty(value, "type") &&
            value.type === "validate"
          ) {
            return false;
          }
        }
      }
    } catch (e) {
      // ignore, meaning fall back to default
    }

    return fn === undefined ? args.defaultShouldRevalidate : fn(args);
  };
}