flaglint 0.4.0 → 0.4.1

package/CHANGELOG.md

@@ -7,6 +7,17 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [0.4.1] - 2026-05-25
+
+### Fixed
+
+- Detail evaluation methods are classified as manual review and excluded from safe auto-transform counts.
+- Generated LaunchDarkly OpenFeature provider setup now uses the SDK key constructor correctly.
+- Evaluation-context guidance now states that either OpenFeature `targetingKey` or existing LaunchDarkly `key` is accepted.
+- Default one-file flags no longer trigger staleness solely because they occur in one file; explicit `minFileCount: 1` remains available.
+- Reporter output now says `Flags with review signals` instead of implying flags are safe to remove.
+- Public early-preview messaging now states the Node.js server-side migration scope and review/testing requirement.
+
 ## [0.4.0] - 2026-05-24
 
 ### Added

package/README.md

@@ -21,9 +21,12 @@
   </a>
 </p>
 
-> ⚠️ **Early preview.** Current scope: **LaunchDarkly Node.js server-side SDK** only.
-> React hooks, HOC, and client-side SDK patterns are detected by `scan` but are not
-> automatically migrated.
+> [!WARNING]
+> FlagLint is currently an early preview.
+>
+> Automatic migration currently supports LaunchDarkly Node.js server-side SDK evaluation calls only. Generated changes must be reviewed and tested before merging.
+>
+> React hooks, higher-order components, browser/client-side SDK usage, bulk flag-state calls, detail evaluations, dynamic keys, and custom wrappers are not automatically migrated in this release.
 
 # FlagLint
 
@@ -209,10 +212,8 @@ Run `flaglint migrate --dry-run` to review the migration plan.
 | Dynamic flag key | ✗ manual | Key must be a static string literal |
 | `ldClient.allFlags()` / `allFlagsState()` | ✗ manual | Bulk calls — no single-flag codemod |
 | Unknown fallback type | ✗ manual | Fallback type must be determinable statically |
-| React `useFlags()`, `useLDClient()` | detect only | Client-side — outside Node.js server SDK scope |
-| React HOC / `<LDProvider>` | detect only | Client-side — outside Node.js server SDK scope |
 
-`flaglint scan` and `flaglint migrate --dry-run` report all detected patterns including manual-review cases.
+`flaglint scan` and `flaglint migrate --dry-run` report detected LaunchDarkly Node.js server-side SDK patterns, including manual-review cases.
 `flaglint migrate --apply` rewrites only the ✓ rows above.
 
 ---
@@ -231,15 +232,15 @@ npm install @openfeature/server-sdk \
 Bootstrap file (do not apply automatically — bootstrap is intentionally manual):
 
 ```typescript
-import LaunchDarkly from "@launchdarkly/node-server-sdk";
 import { LaunchDarklyProvider } from "@launchdarkly/openfeature-node-server";
 import { OpenFeature } from "@openfeature/server-sdk";
 
-const ldClient = LaunchDarkly.init(process.env.LD_SDK_KEY!);
-await OpenFeature.setProviderAndWait(new LaunchDarklyProvider(ldClient));
+const ldProvider = new LaunchDarklyProvider(process.env.LD_SDK_KEY!);
+await OpenFeature.setProviderAndWait(ldProvider);
 
-// Evaluation context must include targetingKey (or key):
-// { targetingKey: user.id }
+// Evaluation context needs a targeting key.
+// Use OpenFeature `targetingKey` or keep an existing LaunchDarkly `key`:
+// { targetingKey: user.id } or { key: user.id }
 export const openFeatureClient = OpenFeature.getClient();
 ```
 
@@ -278,7 +279,7 @@ Create `.flaglintrc`, `.flaglintrc.json`, or `flaglint.config.json` in your proj
   "include": ["**/*.{ts,tsx,js,jsx}"],
   "exclude": ["**/node_modules/**", "**/dist/**"],
   "provider": "launchdarkly",
-  "minFileCount": 1,
+  "minFileCount": 0,
   "reportTitle": "My Project Flag Report"
 }
 ```
@@ -288,7 +289,7 @@ Create `.flaglintrc`, `.flaglintrc.json`, or `flaglint.config.json` in your proj
 | `include` | `string[]` | `["**/*.{ts,tsx,js,jsx}"]` | Glob patterns to scan |
 | `exclude` | `string[]` | `["**/node_modules/**", ...]` | Glob patterns to ignore |
 | `provider` | `string` | `"launchdarkly"` | Feature flag provider |
-| `minFileCount` | `number` | `1` | A flag is a staleness candidate if it appears in ≤ N files |
+| `minFileCount` | `number` | `0` | Opt-in staleness heuristic. When set above 0, a flag is a staleness candidate if it appears in ≤ N files |
 | `wrappers` | `string[]` | `[]` | Function names wrapping LD SDK calls. Example: `["flagPredicate", "useFlag"]` |
 | `reportTitle` | `string` | — | Custom title for generated reports |
 | `outputDir` | `string` | `"."` | Default output directory |

package/dist/bin/flaglint.js

@@ -53,6 +53,13 @@ var LD_FLAG_KEY_METHODS = /* @__PURE__ */ new Set([
   "jsonVariationDetail"
 ]);
 var LD_ALL_FLAGS_METHODS = /* @__PURE__ */ new Set(["allFlags", "allFlagsState"]);
+var LD_DETAIL_METHODS = /* @__PURE__ */ new Set([
+  "variationDetail",
+  "boolVariationDetail",
+  "stringVariationDetail",
+  "numberVariationDetail",
+  "jsonVariationDetail"
+]);
 var LD_HOOKS = /* @__PURE__ */ new Set(["useFlags", "useLDClient"]);
 var DEFAULT_EXCLUDE = ["**/node_modules/**", "**/dist/**", "**/build/**", "**/.next/**"];
 function extractFlagKey(arg) {
@@ -120,7 +127,7 @@ function buildMigrationInventoryItem(code, filePath, loc, call, methodName, args
   }
   const fallback = args[2];
   const valueType = inferValueType(methodName, fallback);
-  const manualReviewReason = isDynamic ? "dynamic-key" : valueType === "unknown" ? "unknown-fallback" : void 0;
+  const manualReviewReason = isDynamic ? "dynamic-key" : LD_DETAIL_METHODS.has(methodName) ? "detail-method" : valueType === "unknown" ? "unknown-fallback" : void 0;
   return {
     file: filePath,
     line: loc.line,
@@ -528,7 +535,7 @@ function formatMarkdown(result, options) {
   }
   if (staleFlags.length > 0) {
     lines.push("## \u26A0 Stale Flag Candidates");
-    lines.push("Flags that may be safe to remove:");
+    lines.push("Flags with review signals:");
     lines.push("| Flag Key | Reason | Location |");
     lines.push("|----------|--------|----------|");
     for (const [key, data] of staleFlags) {
@@ -681,7 +688,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.4.0" : "0.1.0";
+  const version = true ? "0.4.1" : "0.1.0";
   return `<!DOCTYPE html>
 <html lang="en">
 <head>
@@ -779,7 +786,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),
+  minFileCount: z.number().int().min(0).default(0),
   wrappers: z.array(z.string()).default([]),
   reportTitle: z.string().optional(),
   outputDir: z.string().default(".")
@@ -1011,6 +1018,8 @@ function reasonLabel(reason) {
       return "dynamic key";
     case "unknown-fallback":
       return "unsupported or unknown fallback";
+    case "detail-method":
+      return "detail method";
     case "bulk-inventory-call":
       return "bulk inventory call";
     default:
@@ -1094,7 +1103,7 @@ function formatMigrationReport(analysis) {
     unsupportedUnknownCount
   } = analysis;
   const date = (/* @__PURE__ */ new Date()).toLocaleDateString();
-  const version = true ? "0.4.0" : "0.1.0";
+  const version = true ? "0.4.1" : "0.1.0";
   const lines = [];
   lines.push("# OpenFeature Migration Inventory");
   lines.push(`Generated by FlagLint v${version} on ${date}`);
@@ -1179,6 +1188,9 @@ function methodForType(valueType) {
 function manualReason(item) {
   if (item.manualReviewReason === "dynamic-key") return "dynamic key requires manual review";
   if (item.manualReviewReason === "unknown-fallback") return "unknown fallback type requires manual review";
+  if (item.manualReviewReason === "detail-method") {
+    return "detail methods skipped: OpenFeature detail APIs exist, but LaunchDarkly/OpenFeature detail result parity requires manual review";
+  }
   if (item.manualReviewReason === "bulk-inventory-call") return "bulk inventory call has no single-flag codemod";
   return "manual review required";
 }
@@ -1268,7 +1280,7 @@ function formatProviderSetupSection() {
   lines.push('import { OpenFeature } from "@openfeature/server-sdk";');
   lines.push('import { LaunchDarklyProvider } from "@launchdarkly/openfeature-node-server";');
   lines.push("");
-  lines.push('const ldProvider = new LaunchDarklyProvider("<your-sdk-key>");');
+  lines.push("const ldProvider = new LaunchDarklyProvider(process.env.LD_SDK_KEY!);");
   lines.push("await OpenFeature.setProviderAndWait(ldProvider);");
   lines.push("");
   lines.push("// Share this client across your application.");
@@ -1278,11 +1290,14 @@ function formatProviderSetupSection() {
   lines.push("");
   lines.push("### 3. Evaluation context \u2014 targeting key");
   lines.push("");
-  lines.push("LaunchDarkly requires a `targetingKey` field in every evaluation context.");
-  lines.push("Replace the context arguments shown in the diffs with an object that includes it:");
+  lines.push("LaunchDarkly requires a targeting key in every evaluation context. The");
+  lines.push("provider accepts either OpenFeature `targetingKey` or an existing");
+  lines.push("LaunchDarkly `key`.");
+  lines.push("Keep your existing LaunchDarkly `key` contexts, or use `targetingKey`");
+  lines.push("for new OpenFeature-native contexts:");
   lines.push("");
   lines.push("```typescript");
-  lines.push("{ targetingKey: user.key, ...otherAttributes }");
+  lines.push("{ targetingKey: user.id } // or { key: user.id }");
   lines.push("```");
   lines.push("");
   return lines;
@@ -1881,7 +1896,7 @@ Examples:
 // src/cli.ts
 function createCLI() {
   const program2 = new Command();
-  program2.name("flaglint").description("LaunchDarkly Node.js server SDK -> OpenFeature migration.").version("0.4.0", "-v, --version", "output the current version").addHelpText(
+  program2.name("flaglint").description("LaunchDarkly Node.js server SDK -> OpenFeature migration.").version("0.4.1", "-v, --version", "output the current version").addHelpText(
     "after",
     `
 Examples:

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "flaglint",
-  "version": "0.4.0",
+  "version": "0.4.1",
   "description": "LaunchDarkly Node.js server SDK -> OpenFeature migration.",
   "type": "module",
   "bin": {