fix: use shared shouldIgnore/isSupportedFile in watcher (#864)

* fix: use shared shouldIgnore/isSupportedFile in watcher (#838)

The watcher duplicated shouldIgnore and isSupportedFile logic locally,
causing the shared exports in constants.ts to be misclassified as
test-only. Reuse the shared functions to fix role classification and
eliminate duplication.

* fix: correct missing imports in watcher (#864)

Replace raw IGNORE_DIRS/EXTENSIONS references in collectTrackedFiles
with the already-imported shouldIgnore/isSupportedFile helpers, fixing
the TypeScript compilation error.

Author: carlos-alm

src/domain/graph/watcher.ts

@@ -2,20 +2,16 @@ import fs from 'node:fs';
 import path from 'node:path';
 import { closeDb, getNodeId as getNodeIdQuery, initSchema, openDb } from '../../db/index.js';
 import { debug, info } from '../../infrastructure/logger.js';
-import { EXTENSIONS, IGNORE_DIRS, normalizePath } from '../../shared/constants.js';
+import { isSupportedFile, normalizePath, shouldIgnore } from '../../shared/constants.js';
 import { DbError } from '../../shared/errors.js';
 import { createParseTreeCache, getActiveEngine } from '../parser.js';
 import { type IncrementalStmts, rebuildFile } from './builder/incremental.js';
 import { appendChangeEvents, buildChangeEvent, diffSymbols } from './change-journal.js';
 import { appendJournalEntries } from './journal.js';
 
-function shouldIgnore(filePath: string): boolean {
+function shouldIgnorePath(filePath: string): boolean {
   const parts = filePath.split(path.sep);
-  return parts.some((p) => IGNORE_DIRS.has(p));
-}
-
-function isTrackedExt(filePath: string): boolean {
-  return EXTENSIONS.has(path.extname(filePath));
+  return parts.some((p) => shouldIgnore(p));
 }
 
 /** Prepare all SQL statements needed by the watcher's incremental rebuild. */
@@ -146,11 +142,11 @@ function collectTrackedFiles(dir: string, result: string[]): void {
     return;
   }
   for (const entry of entries) {
-    if (IGNORE_DIRS.has(entry.name) || entry.name.startsWith('.')) continue;
+    if (shouldIgnore(entry.name)) continue;
     const full = path.join(dir, entry.name);
     if (entry.isDirectory()) {
       collectTrackedFiles(full, result);
-    } else if (EXTENSIONS.has(path.extname(entry.name))) {
+    } else if (isSupportedFile(entry.name)) {
       result.push(full);
     }
   }
@@ -268,8 +264,8 @@ function startPollingWatcher(ctx: WatcherContext, pollIntervalMs: number): () =>
 function startNativeWatcher(ctx: WatcherContext): () => void {
   const watcher = fs.watch(ctx.rootDir, { recursive: true }, (_eventType, filename) => {
     if (!filename) return;
-    if (shouldIgnore(filename)) return;
-    if (!isTrackedExt(filename)) return;
+    if (shouldIgnorePath(filename)) return;
+    if (!isSupportedFile(filename)) return;
 
     ctx.pending.add(path.join(ctx.rootDir, filename));
     scheduleDebouncedProcess(ctx);