npm - flaglint - Versions diffs - 0.2.1 → 0.2.2 - Mend

flaglint 0.2.1 → 0.2.2

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.

Files changed (3) hide show

package/CHANGELOG.md CHANGED Viewed

@@ -5,6 +5,15 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
+## [0.2.2] - 2026-05-23
+### Fixed
+- **Config mutation**: `--exclude-tests` no longer mutates the loaded config object in both `scan` and `migrate` commands — uses spread instead of `push()` so the original config is never modified.
+- **Typed scan warnings**: `ScanResult.warnings` is now a typed `ScanWarning` union (`read-failure` | `parse-failure`) instead of opaque strings, preserving structured data at the domain boundary.
+- **StalenessEvaluator wired**: The `StalenessEvaluator` interface now has a call site in `scan()` — pass an `evaluator` to inject API-based staleness signals without touching core scanner logic.
+- **ScanConfig boundary**: `scan()` now accepts `ScanConfig` (scan-relevant fields only) rather than the full `FlagLintConfig`, decoupling the scanner from CLI output concerns (`reportTitle`, `outputDir`).
 ## [0.2.1] - 2026-05-23
 ### Fixed

package/dist/bin/flaglint.js CHANGED Viewed

@@ -188,7 +188,7 @@ function detectUsages(ast, filePath, wrappers) {
   });
   return usages;
 }
-async function scan(source, config, onProgress) {
+async function scan(source, config, onProgress, evaluator) {
   const start = Date.now();
   for (const pattern of config.include) {
     if (pattern.startsWith("/") || pattern.startsWith("..")) {
@@ -207,7 +207,7 @@ async function scan(source, config, onProgress) {
       code = await source.readFile(file);
     } catch (err) {
       const fsCode = err.code ?? "UNKNOWN";
-      return { usages: [], warning: `warn: could not read ${file} (${fsCode})` };
+      return { usages: [], warning: { kind: "read-failure", file, fsCode } };
     }
     let ast;
     try {
@@ -220,7 +220,7 @@ async function scan(source, config, onProgress) {
         filePath: file
       });
     } catch {
-      return { usages: [], warning: `warn: failed to parse ${file}` };
+      return { usages: [], warning: { kind: "parse-failure", file } };
     }
     return { usages: detectUsages(ast, file, config.wrappers), warning: null };
   }
@@ -261,6 +261,9 @@ async function scan(source, config, onProgress) {
       }
     }
   }
+  if (evaluator) {
+    await evaluator.evaluate(allUsages, config);
+  }
   const uniqueFlags = [
     ...new Set(
       allUsages.filter((u) => !u.isDynamic && u.flagKey !== "*").map((u) => u.flagKey)
@@ -404,7 +407,7 @@ function formatHTML(result, options) {
     return `<tr class="${cls}"><td><code>${esc(key)}</code></td><td>${data.usages.length}</td><td>${fileList}</td><td>${[...data.callTypes].map(esc).join(", ")}</td><td>${status}</td></tr>`;
   }).join("\n      ");
   const title = options.title ? esc(options.title) : "FlagLint Scan Report";
-  const version = true ? "0.2.1" : "0.1.0";
+  const version = true ? "0.2.2" : "0.1.0";
   return `<!DOCTYPE html>
 <html lang="en">
 <head>
@@ -585,16 +588,15 @@ Examples:
         process.stderr.write(chalk.red(String(err instanceof Error ? err.message : err)) + "\n");
         process.exit(1);
       }
-      if (options.excludeTests) {
-        config.exclude.push(
-          "**/*.test.ts",
-          "**/*.test.tsx",
-          "**/*.spec.ts",
-          "**/*.spec.tsx",
-          "**/__tests__/**",
-          "**/tests/**"
-        );
-      }
+      const TEST_PATTERNS = [
+        "**/*.test.ts",
+        "**/*.test.tsx",
+        "**/*.spec.ts",
+        "**/*.spec.tsx",
+        "**/__tests__/**",
+        "**/tests/**"
+      ];
+      const scanConfig = options.excludeTests ? { ...config, exclude: [...config.exclude, ...TEST_PATTERNS] } : config;
       const format = options.format;
       const spinner = ora(`Scanning ${dir}...`).start();
       process.once("SIGINT", () => {
@@ -604,7 +606,7 @@ Examples:
       let lastSpinnerUpdate = 0;
       let result;
       try {
-        result = await scan(new LocalFileSource(dir), config, (filesScanned) => {
+        result = await scan(new LocalFileSource(dir), scanConfig, (filesScanned) => {
           if (filesScanned - lastSpinnerUpdate >= 50) {
             spinner.text = `Scanning... (${filesScanned} files)`;
             lastSpinnerUpdate = filesScanned;
@@ -617,7 +619,8 @@ Examples:
         process.exit(1);
       }
       for (const w of result.warnings) {
-        process.stderr.write(chalk.yellow(w + "\n"));
+        const msg = w.kind === "read-failure" ? `warn: could not read ${w.file} (${w.fsCode})` : `warn: failed to parse ${w.file}`;
+        process.stderr.write(chalk.yellow(msg + "\n"));
       }
       if (result.scannedFiles === 0) {
         process.stderr.write(
@@ -824,7 +827,7 @@ function analyze(result) {
 function formatMigrationReport(analysis) {
   const { readinessScore, requiredPackages, items, manualReviewCount, autoMigrateCount } = analysis;
   const date = (/* @__PURE__ */ new Date()).toLocaleDateString();
-  const version = true ? "0.2.1" : "0.1.0";
+  const version = true ? "0.2.2" : "0.1.0";
   let scoreLabel;
   if (readinessScore >= 80) scoreLabel = "\u2713 Your codebase is ready for migration";
   else if (readinessScore >= 50) scoreLabel = "\u26A0 Some manual work required before migration";
@@ -937,16 +940,15 @@ Examples:
         process.stderr.write(chalk2.red(String(err instanceof Error ? err.message : err)) + "\n");
         process.exit(1);
       }
-      if (options.excludeTests) {
-        config.exclude.push(
-          "**/*.test.ts",
-          "**/*.test.tsx",
-          "**/*.spec.ts",
-          "**/*.spec.tsx",
-          "**/__tests__/**",
-          "**/tests/**"
-        );
-      }
+      const TEST_PATTERNS = [
+        "**/*.test.ts",
+        "**/*.test.tsx",
+        "**/*.spec.ts",
+        "**/*.spec.tsx",
+        "**/__tests__/**",
+        "**/tests/**"
+      ];
+      const scanConfig = options.excludeTests ? { ...config, exclude: [...config.exclude, ...TEST_PATTERNS] } : config;
       const spinner = ora2(`Scanning ${dir}...`).start();
       process.once("SIGINT", () => {
         spinner.stop();
@@ -954,7 +956,7 @@ Examples:
       });
       let scanResult;
       try {
-        scanResult = await scan(new LocalFileSource(dir), config, (filesScanned) => {
+        scanResult = await scan(new LocalFileSource(dir), scanConfig, (filesScanned) => {
           spinner.text = `Scanning files... ${filesScanned}`;
         });
         spinner.text = "Analyzing migration readiness...";
@@ -983,7 +985,8 @@ Examples:
       const analysis = analyze(scanResult);
       spinner.stop();
       for (const w of scanResult.warnings) {
-        process.stderr.write(chalk2.yellow(w + "\n"));
+        const msg = w.kind === "read-failure" ? `warn: could not read ${w.file} (${w.fsCode})` : `warn: failed to parse ${w.file}`;
+        process.stderr.write(chalk2.yellow(msg + "\n"));
       }
       const { readinessScore } = analysis;
       const scoreColor = readinessScore >= 80 ? chalk2.green : readinessScore >= 50 ? chalk2.yellow : chalk2.red;
@@ -1022,7 +1025,7 @@ Examples:
 // src/cli.ts
 function createCLI() {
   const program2 = new Command();
-  program2.name("flaglint").description("Find stale feature flags. Detect flag debt. Plan your OpenFeature migration.").version("0.2.1", "-v, --version", "output the current version").addHelpText(
+  program2.name("flaglint").description("Find stale feature flags. Detect flag debt. Plan your OpenFeature migration.").version("0.2.2", "-v, --version", "output the current version").addHelpText(
     "after",
     `
 Examples:

package/package.json CHANGED Viewed

@@ -1,6 +1,6 @@
 {
   "name": "flaglint",
-  "version": "0.2.1",
+  "version": "0.2.2",
   "description": "Find stale feature flags. Detect flag debt. Plan your OpenFeature migration.",
   "type": "module",
   "bin": {