knip 6.34.0 → 6.35.0

package/dist/util/watch.js

@@ -1,7 +1,8 @@
-import { readFileSync } from 'node:fs';
 import { invalidateCache } from '../graph-explorer/cache.js';
 import { debugLog } from './debug.js';
+import { hasErrorCause } from './errors.js';
 import { isFile } from './fs.js';
+import { logError } from './log.js';
 import { updateImportMap } from './module-graph.js';
 import { toAbsolute, toPosix, toRelative } from './path.js';
 import { clearModuleResolutionCaches } from '../typescript/resolve-module-names.js';
@@ -13,11 +14,12 @@ const createUpdate = (options) => {
     return { duration, mem };
 };
 export const getSessionHandler = async (options, { analyzedFiles, analyzeSourceFile, chief, collector, analyze, principal, graph, isIgnored, onFileChange, unreferencedFiles, entryPaths, }) => {
-    const handleFileChanges = async (changes) => {
+    const added = new Set();
+    const deleted = new Set();
+    const modified = new Set();
+    const processFileChanges = async (changes) => {
         const startTime = performance.now();
-        const added = new Set();
-        const deleted = new Set();
-        const modified = new Set();
+        let hasRelevantChanges = false;
         for (const change of changes) {
             const filePath = toAbsolute(change.filePath, options.cwd);
             const relativePath = toRelative(change.filePath, options.cwd);
@@ -28,23 +30,24 @@ export const getSessionHandler = async (options, { analyzedFiles, analyzeSourceF
             const workspace = chief.findWorkspaceByFilePath(filePath);
             if (!workspace)
                 continue;
+            hasRelevantChanges = true;
             switch (change.type) {
                 case 'added':
                     principal.addProjectPath(filePath);
-                    principal.deletedFiles.delete(filePath);
+                    deleted.delete(filePath);
                     if (principal.projectPaths.has(filePath))
                         added.add(filePath);
                     debugLog(workspace.name, `Watcher: + ${relativePath}`);
                     break;
                 case 'deleted':
                     deleted.add(filePath);
+                    added.delete(filePath);
                     analyzedFiles.delete(filePath);
                     principal.removeProjectPath(filePath);
                     debugLog(workspace.name, `Watcher: - ${relativePath}`);
                     break;
                 default: {
-                    const cached = principal.fileManager.sourceTextCache.get(filePath);
-                    if (cached !== undefined && cached === readFileSync(filePath, 'utf8')) {
+                    if (!principal.fileManager.hasChanged(filePath)) {
                         debugLog(workspace.name, `Watcher: = ${relativePath}`);
                         continue;
                     }
@@ -55,31 +58,31 @@ export const getSessionHandler = async (options, { analyzedFiles, analyzeSourceF
             }
             principal.invalidateFile(filePath);
         }
-        if (added.size === 0 && deleted.size === 0 && modified.size === 0)
+        if (!hasRelevantChanges || (added.size === 0 && deleted.size === 0 && modified.size === 0))
             return;
         clearResolverCache();
         clearModuleResolutionCaches();
         clearGlobCache();
         invalidateCache(graph);
+        const filePaths = await principal.getUsedResolvedFiles();
         unreferencedFiles.clear();
         const cachedUnusedFiles = collector.purge();
         for (const filePath of added)
             cachedUnusedFiles.add(filePath);
         for (const filePath of deleted)
             cachedUnusedFiles.delete(filePath);
-        const filePaths = principal.getUsedResolvedFiles();
         if (added.size > 0 || deleted.size > 0) {
             graph.clear();
             for (const filePath of filePaths) {
                 const workspace = chief.findWorkspaceByFilePath(filePath);
                 if (workspace) {
-                    analyzeSourceFile(filePath, principal);
+                    await analyzeSourceFile(filePath);
                 }
             }
         }
         else {
             for (const [filePath, file] of graph) {
-                if (filePaths.includes(filePath)) {
+                if (filePaths.has(filePath)) {
                     file.importedBy = undefined;
                 }
                 else {
@@ -94,7 +97,7 @@ export const getSessionHandler = async (options, { analyzedFiles, analyzeSourceF
                 if (!graph.has(filePath)) {
                     const workspace = chief.findWorkspaceByFilePath(filePath);
                     if (workspace) {
-                        analyzeSourceFile(filePath, principal);
+                        await analyzeSourceFile(filePath);
                     }
                 }
             }
@@ -103,7 +106,7 @@ export const getSessionHandler = async (options, { analyzedFiles, analyzeSourceF
                     const workspace = chief.findWorkspaceByFilePath(filePath);
                     if (workspace) {
                         if (principal.projectPaths.has(filePath) || graph.has(filePath)) {
-                            analyzeSourceFile(filePath, principal);
+                            await analyzeSourceFile(filePath);
                         }
                     }
                 }
@@ -120,17 +123,32 @@ export const getSessionHandler = async (options, { analyzedFiles, analyzeSourceF
         collector.addFileCounts({ processed: analyzedFiles.size, unused: unusedFiles.length });
         for (const issue of collector.getRetainedIssues())
             collector.addIssue(issue);
+        added.clear();
+        deleted.clear();
+        modified.clear();
         const update = createUpdate({ startTime });
         if (onFileChange)
             onFileChange(Object.assign({ issues: getIssues().issues }, update));
         return update;
     };
+    const clearSourceText = () => principal.fileManager.sourceTextCache.clear();
+    let pending = Promise.resolve();
+    const handleFileChanges = (changes) => {
+        const update = pending.then(() => processFileChanges(changes));
+        pending = update.then(clearSourceText, clearSourceText);
+        return update;
+    };
     const listener = (eventType, filePath) => {
         debugLog('*', `(raw) ${eventType} ${filePath}`);
         if (typeof filePath === 'string') {
             const normalizedPath = toPosix(filePath);
             const type = eventType === 'rename' ? (isFile(options.cwd, normalizedPath) ? 'added' : 'deleted') : 'modified';
-            handleFileChanges([{ type, filePath: normalizedPath }]);
+            handleFileChanges([{ type, filePath: normalizedPath }]).catch(error => {
+                const message = error instanceof Error ? error.message : String(error);
+                const reason = error instanceof Error && hasErrorCause(error) ? `\nReason: ${error.cause.message}` : '';
+                logError(message + reason);
+                process.exitCode = 2;
+            });
         }
     };
     const getIssues = () => collector.getIssues();

package/dist/version.d.ts

@@ -1 +1 @@
-export declare const version = "6.34.0";
+export declare const version = "6.35.0";

package/dist/version.js

@@ -1 +1 @@
-export const version = '6.34.0';
+export const version = '6.35.0';

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "knip",
-  "version": "6.34.0",
+  "version": "6.35.0",
   "description": "Find and fix unused dependencies, exports and files in your TypeScript and JavaScript projects",
   "keywords": [
     "analysis",
@@ -85,25 +85,25 @@
     "formatly": "^0.7.0",
     "get-tsconfig": "4.14.3",
     "jiti": "^2.7.0",
-    "oxc-parser": "^0.147.0",
+    "oxc-parser": "^0.148.0",
     "oxc-resolver": "11.24.2",
     "picomatch": "^4.0.7",
     "smol-toml": "^1.8.0",
     "strip-json-comments": "5.0.3",
     "tinyglobby": "^0.2.17",
-    "unbash": "^4.0.10",
+    "unbash": "^4.0.11",
     "yaml": "^2.9.0",
     "zod": "^4.4.3"
   },
   "devDependencies": {
-    "@jest/types": "^30.5.0",
-    "@types/bun": "^1.4.0",
+    "@jest/types": "^30.5.1",
+    "@types/bun": "^1.4.1",
     "@types/picomatch": "^4.0.3",
     "codeclimate-types": "^0.3.1",
     "prettier": "^3.9.6",
-    "tsx": "^4.23.12",
+    "tsx": "^4.23.13",
     "typescript": "7.0.2",
-    "webpack": "^5.110.1"
+    "webpack": "^5.110.3"
   },
   "engines": {
     "node": "^20.19.0 || >=22.12.0"

package/schema.json

@@ -173,6 +173,14 @@
         }
       }
     },
+    "preprocessor": {
+      "title": "Preprocess the results before providing them to the reporter(s)",
+      "oneOf": [{ "type": "string" }, { "$ref": "#/definitions/list" }]
+    },
+    "preprocessorOptions": {
+      "title": "Extra options to pass to the preprocessor",
+      "type": "object"
+    },
     "treatConfigHintsAsErrors": {
       "title": "Exit with non-zero code (1) if there are any configuration hints",
       "type": "boolean"
@@ -650,6 +658,10 @@
           "title": "metro plugin configuration (https://knip.dev/reference/plugins/metro)",
           "$ref": "#/definitions/plugin"
         },
+        "mise": {
+          "title": "mise plugin configuration (https://knip.dev/reference/plugins/mise)",
+          "$ref": "#/definitions/plugin"
+        },
         "mocha": {
           "title": "Mocha plugin configuration (https://knip.dev/reference/plugins/mocha)",
           "$ref": "#/definitions/plugin"