oxlint-harness 1.0.9 → 1.0.11

package/dist/command.js

@@ -56,7 +56,7 @@ The suppression file format uses counts per rule per file:
         tighten: Flags.boolean({
             char: "t",
             description: "Tighten suppressions by reducing counts when violations are fixed",
-            default: true,
+            default: false,
             allowNo: true,
         }),
     };
@@ -89,7 +89,7 @@ The suppression file format uses counts per rule per file:
             this.log("      --show-code <number>      Show code snippets for files with N or fewer errors (default: 3, 0 to disable)");
             this.log("      --results-path <path>     Path to save oxlint JSON results (default: artifacts/oxlint-results.json)");
             this.log("      --no-save-results         Don't save oxlint JSON results to file");
-            this.log("  -t, --tighten                 Tighten suppressions when violations are fixed (default: true)");
+            this.log("  -t, --tighten                 Tighten suppressions when violations are fixed");
             this.log("      --no-tighten              Don't tighten suppressions");
             this.log("      --no-type-aware           Don't pass --type-aware to oxlint (default: passes it)");
             this.log("      --no-type-check           Don't pass --type-check to oxlint (default: passes it)");
@@ -187,7 +187,7 @@ The suppression file format uses counts per rule per file:
                     "results-path": process.env.OXLINT_HARNESS_RESULTS_PATH ||
                         "artifacts/oxlint-results.json",
                     "save-results": true,
-                    tighten: true,
+                    tighten: false,
                 };
                 // Manually parse known flags and collect unknown ones
                 for (let i = 0; i < rawArgs.length; i++) {
@@ -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())}`);
@@ -282,15 +283,13 @@ The suppression file format uses counts per rule per file:
             // Normal mode: check suppressions
             const suppressions = suppressionManager.loadSuppressions();
             const excessErrors = suppressionManager.findExcessErrors(diagnostics, suppressions);
-            // Tighten is on by default; env var can override the flag
-            let shouldTighten = flags.tighten;
-            if (process.env.OXLINT_HARNESS_TIGHTEN_BULK_SUPPRESSION?.toLowerCase() ===
-                "true") {
-                shouldTighten = true;
-            }
+            // Tighten is opt-in via --tighten flag or env var
+            const shouldTighten = flags.tighten ||
+                process.env.OXLINT_HARNESS_TIGHTEN_BULK_SUPPRESSION?.toLowerCase() ===
+                    "true";
             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/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.9",
+  "version": "1.0.11",
   "description": "A harness for oxlint with bulk suppressions support",
   "type": "module",
   "main": "dist/index.js",
@@ -51,5 +51,5 @@
   "engines": {
     "node": ">=22"
   },
-  "packageManager": "pnpm@9.15.9"
+  "packageManager": "pnpm@10.28.0+sha512.05df71d1421f21399e053fde567cea34d446fa02c76571441bfc1c7956e98e363088982d940465fd34480d4d90a0668bc12362f8aa88000a64e83d0b0e47be48"
 }