fix(cli): use a separate ignore matcher to ignore watched files (#1681)

Mister-Hope · web-flow · commit 1ab52b65c690 · 2026-03-10T12:10:30.000+08:00
diff --git a/packages/cli/src/commands/dev/watchPageFiles.ts b/packages/cli/src/commands/dev/watchPageFiles.ts
@@ -54,24 +54,24 @@ export const watchPageFiles = (app: App): FSWatcher[] => {
   const sourceDir = app.dir.source()
   const tempDir = app.dir.temp()
   const cacheDir = app.dir.cache()
-  const isPageMatch = picomatch(pagePatterns, {
-    ignore: ignorePatterns,
-    cwd: sourceDir,
-  })
+  const ignoreMatcher = picomatch(ignorePatterns, { cwd: sourceDir })
+  const pageMatcher = picomatch(pagePatterns, { cwd: sourceDir })
   const pagesWatcher = chokidar.watch('.', {
     cwd: sourceDir,
     ignored: (filepath, stats) => {
-      // ignore node_modules
-      if (filepath.includes('node_modules')) {
+      // This is important so that folders like node_modules will be ignored immediately without traversing their children
+      if (ignoreMatcher(filepath)) {
         return true
       }
-      // ignore internal temp files
-      if (filepath.startsWith(tempDir) || filepath.startsWith(cacheDir)) {
-        return true
+
+      // ignore internal temp and cache directories
+      if (stats?.isDirectory()) {
+        return filepath === tempDir || filepath === cacheDir
       }
+
       // ignore non-matched files
       return (
-        !!stats?.isFile() && !isPageMatch(path.relative(sourceDir, filepath))
+        !!stats?.isFile() && !pageMatcher(path.relative(sourceDir, filepath))
       )
     },
     ignoreInitial: true,
