fix(ext/node): validate fs.watch boolean options (#33627)

Enables `parallel/test-fs-watch-recursive-validate.js`.

Co-authored-by: Nathan Whitaker <nathan@deno.com>

Author: nathanwhitbot

ext/node/polyfills/fs.ts

@@ -3270,6 +3270,14 @@ function watch(
   // deno-lint-ignore prefer-primordials
   const watchPath = getValidatedPath(filename).toString();
 
+  // Match Node: validate non-boolean `recursive`/`persistent` up front.
+  // https://github.com/nodejs/node/blob/main/lib/internal/fs/recursive_watch.js
+  if (options != null && options.recursive != null) {
+    validateBoolean(options.recursive, "options.recursive");
+  }
+  if (options != null && options.persistent != null) {
+    validateBoolean(options.persistent, "options.persistent");
+  }
   const recursive = options?.recursive || false;
   validateIgnoreOption(options?.ignore, "options.ignore");
   const ignoreMatcher = createIgnoreMatcher(options?.ignore);

tests/node_compat/config.jsonc

@@ -1208,6 +1208,7 @@
     "parallel/test-fs-watch-recursive-add-folder.js": {},
     "parallel/test-fs-watch-recursive-delete.js": {},
     "parallel/test-fs-watch-recursive-linux-parallel-remove.js": {},
+    "parallel/test-fs-watch-recursive-validate.js": {},
     "parallel/test-fs-watch-ref-unref.js": {},
     "parallel/test-fs-watch-stop-async.js": {},
     "parallel/test-fs-watch-stop-sync.js": {},