Simplify AST validators - #18103

Build successful! You can test your changes in the REPL here: https://babeljs.io/repl/build/61960

Open in StackBlitz

commit: fb150ab

@fisker It seems to me that this PR is ready for review. Are there any pending action items?

Are there any pending action items?

#18103 (comment)

I mean, do we want change

export function isBlockStatement<Opts extends Options<t.BlockStatement>>(
  node: t.Node | null | undefined,
  opts?: Opts | null,
): boolean {
-  return node?.type === "BlockStatement" && shallowEqualN(node, opts);
+  return shallowEqualN(node, "BlockStatement", opts);
}

Are there any pending action items?

#18103 (comment)

I mean, do we want change

export function isBlockStatement<Opts extends Options<t.BlockStatement>>(
  node: t.Node | null | undefined,
  opts?: Opts | null,
): boolean {
-  return node?.type === "BlockStatement" && shallowEqualN(node, opts);
+  return shallowEqualN(node, "BlockStatement", opts);
}

This change looks good to me, in this case we can also consider rename shallowEqualN to something like isType(type: string, node, opts)