test(serve): cover watched-file matcher

Author: cossssmin

src/serve.ts

@@ -1,5 +1,5 @@
 import { readFileSync } from 'node:fs'
-import { dirname, resolve, relative, basename, matchesGlob } from 'node:path'
+import { dirname, resolve, basename } from 'node:path'
 import { fileURLToPath } from 'node:url'
 import { createRequire } from 'node:module'
 import { createServer, createLogger, type ViteDevServer } from 'vite'
@@ -18,6 +18,7 @@ import { serveCompatibility } from './server/compatibility.ts'
 import { serveLint } from './server/linter.ts'
 import { sendEmail } from './server/email.ts'
 import { normalizeComponentSources } from './utils/componentSources.ts'
+import { createWatchedFileMatcher } from './utils/watchPaths.ts'
 import type { MaizzleConfig } from './types/index.ts'
 
 const __dirname = dirname(fileURLToPath(import.meta.url))
@@ -166,15 +167,7 @@ function maizzleDevPlugin(
 
       const userWatchPaths = config.server?.watch ?? []
       const watchPaths = [...defaultWatchPaths, ...userWatchPaths]
-      // Strip a leading "./" so chokidar-style relative globs match correctly.
-      const normalizePattern = (p: string) => p.replace(/^\.\//, '')
-      // Chokidar emits absolute paths on change. Match against project-relative
-      // form so relative globs like "locales/**" actually hit.
-      const cwd = config.root ?? process.cwd()
-      const isWatchedFile = (file: string) => {
-        const rel = relative(cwd, file).replace(/\\/g, '/')
-        return watchPaths.some(p => matchesGlob(rel, normalizePattern(p)))
-      }
+      const isWatchedFile = createWatchedFileMatcher(watchPaths, config.root ?? process.cwd())
 
       for (const watchPath of watchPaths) {
         server.watcher.add(watchPath)

src/tests/utils/watchPaths.test.ts

@@ -0,0 +1,32 @@
+import { describe, it, expect } from 'vitest'
+import { createWatchedFileMatcher } from '../../utils/watchPaths.ts'
+
+describe('createWatchedFileMatcher', () => {
+  const cwd = '/project'
+
+  it('matches an absolute path against a project-relative glob', () => {
+    const matcher = createWatchedFileMatcher(['locales/**'], cwd)
+    expect(matcher('/project/locales/en.json')).toBe(true)
+    expect(matcher('/project/locales/nested/en.json')).toBe(true)
+  })
+
+  it('strips a leading "./" from user-supplied patterns', () => {
+    const matcher = createWatchedFileMatcher(['./locales/**/*.json'], cwd)
+    expect(matcher('/project/locales/en.json')).toBe(true)
+  })
+
+  it('matches top-level files like maizzle.config.ts', () => {
+    const matcher = createWatchedFileMatcher(['maizzle.config.ts'], cwd)
+    expect(matcher('/project/maizzle.config.ts')).toBe(true)
+  })
+
+  it('returns false for files outside any watched pattern', () => {
+    const matcher = createWatchedFileMatcher(['locales/**'], cwd)
+    expect(matcher('/project/emails/welcome.vue')).toBe(false)
+  })
+
+  it('returns false for files outside the project root', () => {
+    const matcher = createWatchedFileMatcher(['locales/**'], cwd)
+    expect(matcher('/elsewhere/locales/en.json')).toBe(false)
+  })
+})

src/utils/watchPaths.ts

@@ -0,0 +1,15 @@
+import { matchesGlob, relative } from 'node:path'
+
+/**
+ * Build a predicate that tells whether an absolute file path emitted by
+ * chokidar matches any of the given globs. Patterns are interpreted as
+ * project-relative; a leading `./` is stripped so user-supplied globs like
+ * `./locales/**` behave identically to `locales/**`.
+ */
+export function createWatchedFileMatcher(patterns: string[], cwd: string) {
+  const normalized = patterns.map(p => p.replace(/^\.\//, ''))
+  return (file: string) => {
+    const rel = relative(cwd, file).replace(/\\/g, '/')
+    return normalized.some(p => matchesGlob(rel, p))
+  }
+}