fix(cli): correct watch ignore logic

Author: grypez

packages/cli/src/app.ts

@@ -116,8 +116,6 @@ await yargs(hideBin(process.argv))
       const closeHandlers: (() => Promise<void>)[] = [];
       const resolvedDir = path.resolve(args.dir);
 
-      await bundleSource(resolvedDir, logger);
-
       const handleClose = async (): Promise<void> => {
         await Promise.all(
           closeHandlers.map(async (close) => withTimeout(close(), 400)),

packages/cli/src/commands/watch.ts

@@ -1,7 +1,7 @@
 import { makePromiseKit } from '@endo/promise-kit';
 import { Logger } from '@ocap/utils';
 import { watch } from 'chokidar';
-import type { FSWatcher, MatchFunction } from 'chokidar';
+import type { FSWatcher } from 'chokidar';
 import { unlink } from 'fs/promises';
 import { resolve } from 'path';
 
@@ -56,9 +56,6 @@ export const makeWatchEvents = (
   error: (error: Error) => throwError(error),
 });
 
-export const filterOnlyJsFiles: MatchFunction = (file, stats) =>
-  (stats?.isFile() ?? false) && file.endsWith('.js');
-
 /**
  * Start a watcher that bundles `.js` files in the target dir.
  *
@@ -77,14 +74,14 @@ export function watchDir(dir: string, logger: Logger): WatchDirReturn {
   const { reject: throwError, promise: errorPromise } = makePromiseKit<never>();
 
   let watcher = watch(resolvedDir, {
-    ignoreInitial: true,
-    ignored: [
-      '**/node_modules/**',
-      '**/*.test.js',
-      '**/*.test.ts',
-      '**/*.bundle',
-      filterOnlyJsFiles,
-    ],
+    ignoreInitial: false,
+    ignored: (file, stats) =>
+      // Ignore node_modules
+      file.includes('node_modules') ||
+      // Watch directories (and the files they contain)
+      ((stats?.isFile() ?? false) &&
+        // But only trigger on JS files
+        !file.endsWith('.js')),
   });
 
   const events = makeWatchEvents(watcher, readyResolve, throwError, logger);