Update chokidar (#464)

* Update chokidar to v4

chokidar v3 is no longer supported. v4 requires node 20 so

1. Added a check for v20+

2. Make script exit if any spawn gets an error.

   This required watching for processes to exit and
   killing the other processes.

3. Update to use glob as chokidar v4 no longer does it

Also fixed a minor warning in sample/utis.ts

Author: greggman

build/lib/copyAndWatch.js

@@ -1,6 +1,7 @@
 import chokidar from 'chokidar';
 import fs from 'fs';
 import path from 'path';
+import { globSync } from 'glob';
 
 const debug = console.log; //() => {};
 const removeLeadingSlash = (s) => s.replace(/^\//, '');
@@ -20,7 +21,7 @@ const removeLeadingSlash = (s) => s.replace(/^\//, '');
  */
 export function copyAndWatch(paths, { watch } = { watch: true }) {
   for (const { src, srcPrefix, dst } of paths) {
-    const watcher = chokidar.watch(src, {
+    const watcher = chokidar.watch(globSync(src), {
       ignored: /(^|[\/\\])\../, // ignore dot files
       persistent: watch,
     });

build/tools/serve.js

@@ -3,17 +3,32 @@ import { mkdirSync } from 'fs';
 
 mkdirSync('out', { recursive: true });
 
-spawn('npm', ['run', 'watch'], {
+const spawns = new Set();
+
+function spawnAndCheck(cmd, args, options) {
+  const s = spawn(cmd, args, options);
+  spawns.add(s);
+  s.on('close', (code) => {
+    spawns.delete(s);
+    if (code !== 0) {
+      console.error(cmd, 'exited with code:', code);
+      [...spawns].forEach((s) => s.kill());
+      process.exit(code);
+    }
+  });
+}
+
+spawnAndCheck('npm', ['run', 'watch'], {
   shell: true,
   stdio: 'inherit',
 });
 
-spawn('node', ['build/tools/copy.js', '1'], {
+spawnAndCheck('node', ['build/tools/copy.js', '1'], {
   shell: true,
   stdio: 'inherit',
 });
 
-spawn('npm', ['run', 'server'], {
+spawnAndCheck('npm', ['run', 'server'], {
   shell: true,
   stdio: 'inherit',
 });

package-lock.json

@@ -43,12 +43,12 @@
     "@types/stats.js": "^0.17.3",
     "@typescript-eslint/eslint-plugin": "^7.18.0",
     "@webgpu/types": "^0.1.49",
-    "chokidar": "^3.6.0",
+    "chokidar": "^4.0.1",
     "eslint": "^8.57.1",
     "eslint-config-prettier": "^8.10.0",
     "eslint-plugin-html": "^8.1.2",
     "eslint-plugin-prettier": "^4.2.1",
-    "glob": "^10.4.5",
+    "glob": "^11.0.0",
     "prettier": "^2.8.8",
     "rollup": "^4.24.0",
     "rollup-plugin-copy": "^3.5.0",

package.json

@@ -5,6 +5,12 @@ import typescript from '@rollup/plugin-typescript';
 import commonjs from '@rollup/plugin-commonjs';
 import { readDirSyncRecursive } from './build/lib/readdir.js';
 
+const nodeVersion = parseInt(process.version.substring(1));
+if (isNaN(nodeVersion) || nodeVersion < 20) {
+  console.error('need node >= v20');
+  process.exit(1);
+}
+
 const outPath = 'out';
 
 function wgslPlugin() {

rollup.config.js

@@ -22,6 +22,7 @@ export function quitIfWebGPUNotAvailable(
   if (!device) {
     quitIfAdapterNotAvailable(adapter);
     fail('Unable to get a device for an unknown reason');
+    return;
   }
 
   device.lost.then((reason) => {