oxlint-harness 1.0.8 → 1.0.10

package/dist/command.js

@@ -270,10 +270,11 @@ The suppression file format uses counts per rule per file:
             }
             // Handle suppression logic
             const suppressionManager = new SuppressionManager(flags.suppressions);
+            const targetedPaths = oxlintArgs.filter(arg => !arg.startsWith('-'));
             if (flags.update) {
                 // Update mode: generate/update suppression file
                 const currentSuppressions = suppressionManager.loadSuppressions();
-                const updatedSuppressions = suppressionManager.updateSuppressions(currentSuppressions, diagnostics);
+                const updatedSuppressions = suppressionManager.updateSuppressions(currentSuppressions, diagnostics, targetedPaths.length > 0 ? targetedPaths : undefined);
                 suppressionManager.saveSuppressions(updatedSuppressions);
                 this.log(`${colors.success("Updated suppression file:")} ${colors.filename(flags.suppressions)}`);
                 this.log(`${colors.info("Total diagnostics:")} ${colors.emphasis(diagnostics.length.toString())}`);
@@ -290,7 +291,7 @@ The suppression file format uses counts per rule per file:
             }
             if (shouldTighten) {
                 // Tighten suppressions by removing/reducing cleaned-up violations
-                const tightenedSuppressions = suppressionManager.tightenSuppressions(suppressions, diagnostics);
+                const tightenedSuppressions = suppressionManager.tightenSuppressions(suppressions, diagnostics, targetedPaths.length > 0 ? targetedPaths : undefined);
                 suppressionManager.saveSuppressions(tightenedSuppressions);
                 this.log(`${colors.success("Tightened suppression file:")} ${colors.filename(flags.suppressions)}`);
             }

package/dist/oxlint-runner.js

@@ -58,15 +58,7 @@ export class OxlintRunner {
                 try {
                     // oxlint exits with non-zero when linting issues are found
                     // We still want to parse the JSON output
-                    const diagnostics = this.parseOxlintOutput(stdout);
-                    // If oxlint produced no diagnostics but exited with a code other than
-                    // 0 (success) or 1 (lint violations found), treat it as a failure.
-                    // This prevents tighten from wiping the suppression file when oxlint
-                    // fails silently (e.g. missing tsconfig for --type-aware).
-                    if (diagnostics.length === 0 && code !== 0 && code !== 1) {
-                        reject(new Error(`oxlint exited with code ${code} and produced no output.\nStderr: ${stderr}`));
-                        return;
-                    }
+                    const diagnostics = this.parseOxlintOutput(stdout, code, stderr);
                     resolve(diagnostics);
                 }
                 catch (error) {
@@ -78,9 +70,12 @@ export class OxlintRunner {
             });
         });
     }
-    parseOxlintOutput(output) {
+    parseOxlintOutput(output, exitCode = null, stderr = '') {
         if (!output.trim()) {
-            return [];
+            // oxlint -f json should always produce JSON output. Empty stdout means
+            // it failed to run (missing tsconfig, binary not found, signal kill, etc).
+            // Returning [] here would cause tighten to wipe the suppression file.
+            throw new Error(`oxlint produced no output (exit code: ${exitCode}).${stderr ? `\nStderr: ${stderr}` : ''}`);
         }
         try {
             const parsed = JSON.parse(output);

package/dist/suppression-manager.d.ts

@@ -7,6 +7,7 @@ export declare class SuppressionManager {
     saveSuppressions(suppressions: SuppressionFile): void;
     generateSuppressions(diagnostics: ProcessedDiagnostic[]): SuppressionFile;
     findExcessErrors(diagnostics: ProcessedDiagnostic[], suppressions: SuppressionFile): ExcessError[];
-    updateSuppressions(currentSuppressions: SuppressionFile, diagnostics: ProcessedDiagnostic[]): SuppressionFile;
-    tightenSuppressions(currentSuppressions: SuppressionFile, diagnostics: ProcessedDiagnostic[]): SuppressionFile;
+    private isFileCoveredByPaths;
+    updateSuppressions(currentSuppressions: SuppressionFile, diagnostics: ProcessedDiagnostic[], targetedPaths?: string[]): SuppressionFile;
+    tightenSuppressions(currentSuppressions: SuppressionFile, diagnostics: ProcessedDiagnostic[], targetedPaths?: string[]): SuppressionFile;
 }

package/dist/suppression-manager.js

@@ -31,6 +31,9 @@ export class SuppressionManager {
         }
     }
     saveSuppressions(suppressions) {
+        if (Object.keys(suppressions).length === 0) {
+            return;
+        }
         try {
             const sorted = this.sortSuppressions(suppressions);
             const content = JSON.stringify(sorted, null, 2);
@@ -93,10 +96,24 @@ export class SuppressionManager {
         }
         return excessErrors;
     }
-    updateSuppressions(currentSuppressions, diagnostics) {
+    isFileCoveredByPaths(filename, paths) {
+        return paths.some(path => {
+            const normalizedPath = path.replace(/\/+$/, '');
+            return filename === normalizedPath || filename.startsWith(normalizedPath + '/');
+        });
+    }
+    updateSuppressions(currentSuppressions, diagnostics, targetedPaths) {
         const newSuppressions = this.generateSuppressions(diagnostics);
         // Merge with existing suppressions, using new counts
         const updated = { ...currentSuppressions };
+        // Remove entries for targeted files that have no new diagnostics
+        if (targetedPaths && targetedPaths.length > 0) {
+            for (const filename of Object.keys(updated)) {
+                if (this.isFileCoveredByPaths(filename, targetedPaths) && !newSuppressions[filename]) {
+                    delete updated[filename];
+                }
+            }
+        }
         for (const [filename, rules] of Object.entries(newSuppressions)) {
             updated[filename] = { ...updated[filename], ...rules };
         }
@@ -108,7 +125,7 @@ export class SuppressionManager {
         }
         return this.sortSuppressions(updated);
     }
-    tightenSuppressions(currentSuppressions, diagnostics) {
+    tightenSuppressions(currentSuppressions, diagnostics, targetedPaths) {
         // Group diagnostics by file and rule to get actual counts
         const actualCounts = new Map();
         for (const diagnostic of diagnostics) {
@@ -123,6 +140,10 @@ export class SuppressionManager {
         const tightened = { ...currentSuppressions };
         // Process each file in current suppressions
         for (const [filename, rules] of Object.entries(tightened)) {
+            // Skip files not covered by targeted paths
+            if (targetedPaths && targetedPaths.length > 0 && !this.isFileCoveredByPaths(filename, targetedPaths)) {
+                continue;
+            }
             const fileRules = { ...rules };
             // Process each rule in the file
             for (const [rule, suppression] of Object.entries(fileRules)) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "oxlint-harness",
-  "version": "1.0.8",
+  "version": "1.0.10",
   "description": "A harness for oxlint with bulk suppressions support",
   "type": "module",
   "main": "dist/index.js",