flaglint 0.2.0 → 0.2.2

package/CHANGELOG.md

@@ -5,6 +5,29 @@ 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
+
+- **Parse failure on generic TypeScript arrow functions** — `flagPredicate = <T>(...)` and similar generic arrows in `.ts` files now parse correctly. Root cause: `@typescript-eslint/typescript-estree` wasn't receiving a `filePath`, so it couldn't apply TypeScript's extension-based JSX rules. Adding `filePath` tells the compiler to treat `.ts` files as non-JSX (generics parse cleanly) and `.tsx` as JSX (LDProvider detection still works). Validated against LaunchDarkly's own docs codebase.
+
+### Added
+
+- **`wrappers` config option** — detect custom wrapper functions as flag usages. Add your wrapper names to `.flaglintrc`:
+  ```json
+  { "wrappers": ["flagPredicate", "useFlag", "getFlag", "isEnabled"] }
+  ```
+  FlagLint will treat calls to these functions as `variation`-equivalent. Supports static and dynamic flag keys. Default is `[]` — no behaviour change for existing users.
+
 ## [0.2.0] - 2026-05-22
 
 ### Breaking Changes

package/README.md

@@ -118,6 +118,7 @@ Create `.flaglintrc` in your project root:
 | `exclude` | `string[]` | `["**/node_modules/**", ...]` | Glob patterns to ignore |
 | `provider` | `string` | `"launchdarkly"` | Feature flag provider |
 | `minFileCount` | `number` | `1` | A flag is stale if it appears in ≤ N files (default: 1) |
+| `wrappers` | `string[]` | `[]` | Function names that wrap LD SDK calls. FlagLint will detect calls to these functions as flag usages. Example: `["flagPredicate", "useFlag", "getFlag", "isEnabled"]` |
 | `reportTitle` | `string` | — | Custom title for generated reports |
 | `outputDir` | `string` | `"."` | Default output directory |
 

package/dist/bin/flaglint.js

@@ -76,7 +76,7 @@ function walk(root, visit) {
     }
   }
 }
-function detectUsages(ast, filePath) {
+function detectUsages(ast, filePath, wrappers) {
   const usages = [];
   walk(ast, (node) => {
     if (node.type === "CallExpression") {
@@ -141,6 +141,20 @@ function detectUsages(ast, filePath) {
           return;
         }
       }
+      if (wrappers.length > 0 && callee.type === "Identifier" && wrappers.includes(callee.name) && call.arguments.length >= 1) {
+        const { flagKey, isDynamic } = extractFlagKey(call.arguments[0]);
+        const sig = checkStale(flagKey, filePath);
+        usages.push({
+          flagKey,
+          isDynamic,
+          file: filePath,
+          line: loc.line,
+          column: loc.column,
+          callType: "variation",
+          stalenessSignals: sig ? [sig] : []
+        });
+        return;
+      }
       if (callee.type === "CallExpression" && callee.callee.type === "Identifier" && callee.callee.name === "withLDConsumer") {
         const sig = checkStale("*", filePath);
         usages.push({
@@ -174,7 +188,7 @@ function detectUsages(ast, filePath) {
   });
   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("..")) {
@@ -193,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 {
@@ -202,12 +216,13 @@ async function scan(source, config, onProgress) {
         loc: true,
         range: false,
         comment: false,
-        tokens: false
+        tokens: false,
+        filePath: file
       });
     } catch {
-      return { usages: [], warning: `warn: failed to parse ${file}` };
+      return { usages: [], warning: { kind: "parse-failure", file } };
     }
-    return { usages: detectUsages(ast, file), warning: null };
+    return { usages: detectUsages(ast, file, config.wrappers), warning: null };
   }
   const limit = pLimit(50);
   const results = await Promise.all(
@@ -246,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)
@@ -389,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.0" : "0.1.0";
+  const version = true ? "0.2.2" : "0.1.0";
   return `<!DOCTYPE html>
 <html lang="en">
 <head>
@@ -486,6 +504,7 @@ var FlagLintConfigSchema = z.object({
   provider: z.enum(["launchdarkly", "unleash", "growthbook", "custom"]).default("launchdarkly"),
   // TODO v0.3: replace minFileCount with real date-based staleness via git log
   minFileCount: z.number().int().min(0).default(1),
+  wrappers: z.array(z.string()).default([]),
   reportTitle: z.string().optional(),
   outputDir: z.string().default(".")
 });
@@ -569,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", () => {
@@ -588,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;
@@ -601,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(
@@ -808,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.0" : "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";
@@ -921,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();
@@ -938,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...";
@@ -967,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;
@@ -1006,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.0", "-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

@@ -1,6 +1,6 @@
 {
   "name": "flaglint",
-  "version": "0.2.0",
+  "version": "0.2.2",
   "description": "Find stale feature flags. Detect flag debt. Plan your OpenFeature migration.",
   "type": "module",
   "bin": {
@@ -41,10 +41,6 @@
     "test": "vitest",
     "test:run": "vitest run",
     "test:coverage": "vitest run --coverage",
-    "agent": "tsx scripts/agent/agent.ts",
-    "agent:launch": "tsx scripts/agent/agent.ts launch",
-    "agent:parallel": "tsx scripts/agent/agent.ts parallel",
-    "agent:sync": "tsx scripts/agent/agent.ts sync-docs",
     "release:patch": "tsx scripts/release.ts patch",
     "release:minor": "tsx scripts/release.ts minor",
     "release:major": "tsx scripts/release.ts major"