nativekeel 0.1.3 → 0.1.4

package/bin/nativekeel.js

@@ -156,7 +156,9 @@ async function main() {
   return result.findings.some((f) => SEVERITIES.indexOf(f.severity) <= threshold) ? 1 : 0;
 }
 
+// Exit only after stdout has flushed: process.exit() right after a large write to a pipe
+// cuts the output off at 8 KB (`npx nativekeel --json | jq` got broken JSON).
 main().then(
-  (code) => process.exit(code),
+  (code) => process.stdout.write('', () => process.exit(code)),
   (e) => fail(e.stack || e.message),
 );

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "nativekeel",
-  "version": "0.1.3",
+  "version": "0.1.4",
   "description": "Health check for React Native apps: version support, New Architecture blockers, unmaintained native modules and leaked secrets.",
   "type": "module",
   "bin": {

package/src/analyze.js

@@ -4,7 +4,7 @@ import { cleanVersion, findPackageDir, installedVersion } from './project.js';
 import { createRegistry } from './registry.js';
 import { scanSecrets } from './secrets.js';
 import { nativeChecks } from './native.js';
-import { findUnused } from './usage.js';
+import { findUnused, filesImportingWith } from './usage.js';
 import { matchKnownIssues } from './known-issues.js';
 import { COMPAT, bestRange, checkCompat } from './compat.js';
 import { interopProblems } from './interop.js';
@@ -47,7 +47,9 @@ function isNativeModule(root, name, dir) {
       return false;
     }
   }
-  // Without node_modules, trust React Native Directory's platform flags.
+  // Without node_modules, ask React Native Directory. Its ios/android flags mean "works on",
+  // not "has native code" (pure JS packages set them too), so prefer its hasNativeCode.
+  if (dir && dir.github && typeof dir.github.hasNativeCode === 'boolean') return dir.github.hasNativeCode;
   return !!(dir && (dir.ios || dir.android) && !dir.web);
 }
 
@@ -74,14 +76,32 @@ export async function analyze(project, { get, now = new Date(), offline: forcedO
   if (!project.hasNodeModules) {
     warnings.push('node_modules not found: versions are read from package.json ranges and native detection is less accurate. Install dependencies for a precise report.');
   }
+  const rnSpec = project.deps['react-native'] || project.devDeps['react-native'];
+  if (!project.rnVersion && rnSpec) {
+    warnings.push(`React Native is declared as "${rnSpec}", which has no version number (a git fork or a workspace reference). Install dependencies so NativeKeel can read the installed version; version checks were skipped.`);
+  }
+  if (project.isLibrary) {
+    warnings.push('This looks like a React Native library, not an app (react-native is a peer dependency, or android/ builds a library). App Store and Google Play checks were skipped; scan the app that uses it for those.');
+  }
   if (!offline && project.rnVersion && !rn) warnings.push('Could not fetch React Native releases from npm; version and New Architecture checks were skipped. Run again.');
   if (!offline && project.expoVersion && !expo) warnings.push('Could not fetch Expo releases from npm; the SDK check was skipped. Run again.');
 
   // 1. React Native version support
   const current = project.rnVersion;
+  // In an Expo project the SDK pins React Native: upgrading the SDK is how React Native moves,
+  // so the Expo finding carries the severity and this one only explains the link.
+  const rnFollowsExpo = !!(project.expoVersion && expo);
   if (rn && current) {
     const behind = rn.lines[0] - minorOf(current);
-    if (behind >= SUPPORTED_RN_MINORS) {
+    if (rnFollowsExpo && behind > 0) {
+      add({
+        id: 'rn-via-expo',
+        severity: 'info',
+        area: 'react-native',
+        title: `React Native ${current} comes with Expo SDK ${majorOf(project.expoVersion)}`,
+        detail: `Latest React Native is ${rn.latest} (${behind} minor${behind === 1 ? '' : 's'} behind). Upgrading the Expo SDK upgrades React Native with it.`,
+      });
+    } else if (behind >= SUPPORTED_RN_MINORS) {
       add({
         id: 'rn-unsupported',
         severity: behind >= CRITICAL_RN_MINORS_BEHIND ? 'critical' : 'high',
@@ -140,7 +160,7 @@ export async function analyze(project, { get, now = new Date(), offline: forcedO
   }
 
   // 4. Google Play target SDK
-  const target = project.android && project.android.targetSdk;
+  const target = !project.isLibrary && project.android && project.android.targetSdk;
   if (target) {
     const today = now.toISOString().slice(0, 10);
     const { updates, visibility } = PLAY_TARGET_SDK;
@@ -173,11 +193,16 @@ export async function analyze(project, { get, now = new Date(), offline: forcedO
   const published = forcedOffline ? {} : await registry.publishDates(recencyCandidates);
   const daysSince = (date) => (date ? Math.floor((now - new Date(`${date}T00:00:00Z`)) / 86400000) : null);
 
+  const nativeOf = Object.fromEntries(names.map((n) => [n, isNativeModule(project.root, n, directory[n])]));
+  const versionOf = (n) => cleanVersion(installedVersion(project.root, n) || declaredRange(n));
+  const archSuspects = names.filter((n) => nativeOf[n] && directory[n] && directory[n].newArchitecture === false);
+  const codegen = forcedOffline ? {} : await registry.codegenSupport(archSuspects.map((n) => ({ name: n, version: versionOf(n) })));
+
   const deps = [];
   for (const name of names) {
     const dir = directory[name];
-    const version = cleanVersion(installedVersion(project.root, name) || declaredRange(name));
-    const native = isNativeModule(project.root, name, dir);
+    const version = versionOf(name);
+    const native = nativeOf[name];
     const release = published[name] || {};
     const age = daysSince(release.date);
     const revived = !!(dir && dir.unmaintained && age !== null && age <= RECENT_RELEASE_DAYS);
@@ -192,7 +217,22 @@ export async function analyze(project, { get, now = new Date(), offline: forcedO
     };
     deps.push(dep);
 
-    const noNewArch = native && dir && dir.newArchitecture === false;
+    const cg = codegen[name];
+    // Directory says no New Architecture, but npm may know better.
+    const archFixedIn = native && dir && dir.newArchitecture === false && cg && !cg.usedHas && cg.latestHas ? cg.latest : null;
+    const noNewArch = native && dir && dir.newArchitecture === false && !(cg && (cg.usedHas || cg.latestHas));
+    if (cg && cg.usedHas) dep.newArch = true;
+    if (archFixedIn) {
+      add({
+        id: `dep-risk:${name}`,
+        severity: 'medium',
+        area: 'dependency',
+        title: `${name} ${version || ''}: New Architecture support needs ${archFixedIn}`.replace('  ', ' '),
+        detail: `React Native Directory lists it without New Architecture support, but ${archFixedIn} ships a codegen spec. Update to ${archFixedIn} (check its changelog for breaking changes) instead of replacing it.`,
+        fix: { kind: 'bump-dep', name, from: version, to: archFixedIn, native },
+      });
+      continue;
+    }
     // Only code that has to follow React Native or the OS goes stale; a finished JS utility does not.
     const stale = !dep.unmaintained && !revived && age !== null && age > STALE_RELEASE_DAYS && (native || !!release.rnLink);
     const alternatives = dir && dir.alternatives && dir.alternatives.length ? dir.alternatives : REPLACEMENTS[name] || [];
@@ -261,13 +301,22 @@ export async function analyze(project, { get, now = new Date(), offline: forcedO
   const newArchRequired =
     findings.some((f) => f.id === 'new-arch-disabled') || (archOff && current && minorOf(current) < NEW_ARCH_ONLY_MINOR);
   for (const { issue, dep } of matchKnownIssues({ deps, rnVersion: current, newArchOn, newArchRequired })) {
+    let { severity, title, detail } = issue;
+    if (issue.evidence) {
+      const files = filesImportingWith(project.root, issue.evidence.imports, issue.evidence.pattern);
+      if (files.length) {
+        detail = `Found in ${files.slice(0, 3).join(', ')}${files.length > 3 ? ` and ${files.length - 3} more` : ''}. ${detail}`;
+      } else {
+        ({ severity, title, detail } = { ...issue, ...issue.withoutEvidence });
+      }
+    }
     add({
       id: `known:${issue.id}`,
-      severity: issue.severity,
+      severity,
       area: 'dependency',
-      title: issue.title,
-      detail: `${issue.detail} (installed: ${dep.name} ${dep.version}; seen in: ${issue.source})`,
-      fix: { kind: 'known-issue', name: dep.name, title: issue.title, detail: issue.detail },
+      title,
+      detail: `${detail} (installed: ${dep.name} ${dep.version}; seen in: ${issue.source})`,
+      fix: { kind: 'known-issue', name: dep.name, title, detail },
     });
   }
 
@@ -322,6 +371,15 @@ export async function analyze(project, { get, now = new Date(), offline: forcedO
         detail: `${value}Anyone who downloads the app can extract it. Revoke it now, then move the call to a server.`,
         fix: { kind: 'secret', label: s.label, file: s.file, line: s.line, bundled: true },
       });
+    } else if (s.committed !== false && s.inComment) {
+      add({
+        id: `secret:${s.rule}:${s.file}`,
+        severity: 'medium',
+        area: 'secret',
+        title: `${s.label} in a commented-out line (${where})`,
+        detail: `${value}Comments are not shipped in the app, but the line is in the repository and its git history. If the value is real, rotate it; either way, delete the line.`,
+        fix: { kind: 'secret', label: s.label, file: s.file, line: s.line, bundled: false },
+      });
     } else if (s.committed !== false) {
       add({
         id: `secret:${s.rule}:${s.file}`,

package/src/known-issues.js

@@ -38,9 +38,19 @@ export const KNOWN_ISSUES = [
     rnFrom: 76,
     when: 'newArch',
     severity: 'high',
-    title: 'react-native-linear-gradient 2.x crashes the New Architecture when a screen unmounts',
-    detail: "2.x has no Fabric component, so BVLinearGradient runs through the interop layer. Its children are not detached before React Native recycles them: \"RCTComponentViewRegistry: Attempt to recycle a mounted view\" (an abort on iOS, right after login in the app where it was found). React Native 0.76+ draws gradients itself: replace it with a small component that sets experimental_backgroundImage: 'linear-gradient(...)' and drop the native package.",
-    source: 'real upgrade of an RN 0.80 app, 2026-10-04 (UI test after login)',
+    title: 'LinearGradient around a Modal: a known New Architecture crash',
+    detail: "react-native-linear-gradient 2.x has no Fabric component, so it runs through the interop layer. When a LinearGradient contains a Modal (or a Modal-based bottom sheet), closing it can abort the app: \"RCTComponentViewRegistry: Attempt to recycle a mounted view\" (upstream issue #718). React Native 0.76+ draws gradients itself: experimental_backgroundImage: 'linear-gradient(...)' on a View.",
+    source: 'real upgrade of an RN 0.80 app, 2026-10-04 (UI test after login); upstream issue #718',
+    // Only high when the code has the crashing shape; many apps use 2.x under Fabric without it.
+    evidence: {
+      imports: 'react-native-linear-gradient',
+      pattern: '<Modal\\b|<RBSheet\\b|<BottomSheetModal\\b|react-native-modal|react-native-raw-bottom-sheet',
+    },
+    withoutEvidence: {
+      severity: 'low',
+      title: 'react-native-linear-gradient 2.x runs through the New Architecture interop layer',
+      detail: "It has no Fabric component. No LinearGradient next to a Modal was found, which is the layout known to crash (upstream issue #718). React Native 0.76+ can draw gradients itself (experimental_backgroundImage), which removes a native dependency.",
+    },
   },
 ];
 

package/src/native.js

@@ -131,7 +131,8 @@ export function nativeChecks(project, { rnLatest = null, now = new Date() } = {}
   const appGradle = read(path.join(root, 'android', 'app', 'build.gradle')) || read(path.join(root, 'android', 'app', 'build.gradle.kts')) || '';
   const gradleProps = read(path.join(root, 'android', 'gradle.properties')) || '';
   const podfile = read(path.join(root, 'ios', 'Podfile')) || '';
-  if (/com\.facebook\.flipper|FLIPPER_VERSION/.test(appGradle + gradleProps) || /use_flipper!|FlipperConfiguration\.enabled/.test(podfile)) {
+  const flipperWired = /com\.facebook\.flipper/.test(appGradle) || /^\s*[^#\n]*(use_flipper!|FlipperConfiguration\.enabled)/m.test(podfile);
+  if (flipperWired) {
     add({
       id: 'flipper',
       severity: 'medium',
@@ -140,6 +141,15 @@ export function nativeChecks(project, { rnLatest = null, now = new Date() } = {}
       detail: 'Flipper was removed from React Native in 0.74 and its native libraries are not 16 KB aligned. Remove the Flipper dependencies and initialization code before upgrading; use React Native DevTools instead.',
       fix: { kind: 'remove-flipper' },
     });
+  } else if (/^\s*FLIPPER_VERSION\s*=/m.test(gradleProps)) {
+    // A leftover property from an old template: harmless, but worth deleting.
+    add({
+      id: 'flipper-leftover',
+      severity: 'low',
+      area: 'native',
+      title: 'Leftover FLIPPER_VERSION in android/gradle.properties',
+      detail: 'Nothing uses it (Flipper is not wired into the build). Delete the line so nobody wonders whether Flipper is still there.',
+    });
   }
 
   // Podfile still loading the CLI's native_modules.rb: React Native provides use_native_modules!
@@ -278,7 +288,7 @@ export function nativeChecks(project, { rnLatest = null, now = new Date() } = {}
 
   // iOS privacy manifest: required by App Store Connect since 1 May 2024. React Native itself
   // uses "required reason" APIs, so every React Native app needs one.
-  if (hasIos) {
+  if (hasIos && !project.isLibrary) {
     const manifests = findFiles(path.join(root, 'ios'), (name) => name.endsWith('.xcprivacy'));
     if (!manifests.length) {
       add({
@@ -365,7 +375,14 @@ export function nativeChecks(project, { rnLatest = null, now = new Date() } = {}
   // someone publish as you. The debug keystore and its public 'android' password are fine.
   const tracked = gitTracked(root);
   if (tracked) {
-    const signing = tracked.filter((f) => /\.(jks|keystore)$/i.test(f) && !/debug/i.test(path.basename(f)));
+    const nonEmpty = (f) => {
+      try {
+        return fs.statSync(path.join(root, f)).size > 0;
+      } catch {
+        return false;
+      }
+    };
+    const signing = tracked.filter((f) => /\.(jks|keystore)$/i.test(f) && !/debug/i.test(path.basename(f)) && nonEmpty(f));
     for (const f of signing) {
       add({
         id: `signing-keystore:${f}`,
@@ -376,7 +393,16 @@ export function nativeChecks(project, { rnLatest = null, now = new Date() } = {}
         fix: { kind: 'secret', label: 'Release keystore', file: f, line: 1, bundled: false },
       });
     }
-    for (const f of tracked.filter((x) => /(^|\/)AuthKey_[A-Z0-9]+\.p8$/.test(x))) {
+    // An empty placeholder file is not a key (some repos commit one so fastlane paths resolve).
+    const hasContent = (f, marker) => {
+      try {
+        const text = fs.readFileSync(path.join(root, f), 'utf8');
+        return marker ? text.includes(marker) : text.length > 0;
+      } catch {
+        return false;
+      }
+    };
+    for (const f of tracked.filter((x) => /(^|\/)AuthKey_[A-Z0-9]+\.p8$/.test(x) && hasContent(x, 'PRIVATE KEY'))) {
       add({
         id: `signing-asc-key:${f}`,
         severity: 'critical',
@@ -386,7 +412,7 @@ export function nativeChecks(project, { rnLatest = null, now = new Date() } = {}
         fix: { kind: 'secret', label: 'App Store Connect API key', file: f, line: 1, bundled: false },
       });
     }
-    for (const f of tracked.filter((x) => /\.p12$/i.test(x))) {
+    for (const f of tracked.filter((x) => /\.p12$/i.test(x) && nonEmpty(x))) {
       add({
         id: `signing-p12:${f}`,
         severity: 'high',
@@ -401,7 +427,8 @@ export function nativeChecks(project, { rnLatest = null, now = new Date() } = {}
       const text = read(path.join(root, f)) || '';
       const leaked = [
         ...[...text.matchAll(/^\s*([A-Z0-9_]*(?:STORE|KEY)_PASSWORD)\s*=\s*(\S+)\s*$/gm)].map((m) => ({ name: m[1], value: m[2] })),
-        ...[...text.matchAll(/\b(storePassword|keyPassword)\s*=?\s*["']([^"']+)["']/g)].map((m) => ({ name: m[1], value: m[2] })),
+        // Not `props['storePassword']`: there the name is a lookup key, not an assignment.
+        ...[...text.matchAll(/(?<![\w'"[])(storePassword|keyPassword)\s*=?\s*["']([^"'\n]+)["']/g)].map((m) => ({ name: m[1], value: m[2] })),
       ].filter((l) => l.value !== 'android' && !/^System\.getenv|^\$\{?[A-Z_]+\}?$/.test(l.value));
       if (leaked.length) {
         add({

package/src/plan.js

@@ -350,6 +350,10 @@ export function buildPlan(result) {
   if (expo) {
     const steps = [];
     for (let v = expo.from + 1; v <= expo.to; v++) steps.push(`SDK ${v - 1} → ${v}: \`npx expo install expo@^${v}.0.0 --fix\`, read the SDK ${v} changelog, run \`npx expo-doctor\`, build.`);
+    steps.push('Each SDK brings its own React Native version; do not upgrade React Native separately.');
+    if (!result.project.managed) {
+      steps.push('This app has its own android/ and ios/ folders: after each SDK, apply the native changes from Expo\'s native project upgrade helper (docs.expo.dev/bare/upgrade), then `pod install` and build both platforms.');
+    }
     phases.push({ title: `Upgrade Expo SDK ${expo.from} → ${expo.to}`, why: 'Expo only supports one SDK step at a time reliably.', steps });
   }
 

package/src/project.js

@@ -42,12 +42,71 @@ export function installedVersion(root, name) {
   return pkg ? pkg.version : null;
 }
 
+// pnpm and Bun "catalogs": a dependency declared as `catalog:` or `catalog:<name>` takes its
+// version from the workspace root (pnpm-workspace.yaml, or package.json `workspaces.catalog`).
+// Returns { default: {pkg: range}, <name>: {...} } or null.
+export function readCatalogs(root) {
+  let dir = root;
+  while (true) {
+    const yaml = readText(path.join(dir, 'pnpm-workspace.yaml'));
+    if (yaml) return parsePnpmCatalogs(yaml);
+    const pkg = dir !== root ? readJson(path.join(dir, 'package.json')) : null;
+    const ws = pkg && pkg.workspaces && !Array.isArray(pkg.workspaces) ? pkg.workspaces : pkg;
+    if (ws && (ws.catalog || ws.catalogs)) return { default: ws.catalog || {}, ...(ws.catalogs || {}) };
+    const parent = path.dirname(dir);
+    if (parent === dir) return null;
+    dir = parent;
+  }
+}
+
+// Just enough YAML for the two catalog sections: `catalog:` and `catalogs: <name>:` maps of
+// "package": version lines. Anything else in the file is ignored.
+function parsePnpmCatalogs(text) {
+  const out = { default: {} };
+  let section = null; // 'default' or a named catalog
+  let inCatalogs = false;
+  const unquote = (v) => v.trim().replace(/^["']|["']$/g, '');
+  for (const raw of text.split('\n')) {
+    const line = raw.replace(/\s+#.*$/, '');
+    if (!line.trim()) continue;
+    const indent = line.length - line.trimStart().length;
+    const m = line.trim().match(/^("[^"]+"|'[^']+'|[^:\s]+)\s*:\s*(.*)$/);
+    if (!m) continue;
+    const key = unquote(m[1]);
+    const value = m[2];
+    if (indent === 0) {
+      inCatalogs = key === 'catalogs';
+      section = key === 'catalog' ? 'default' : null;
+    } else if (inCatalogs && !value) {
+      section = key;
+      out[section] = out[section] || {};
+    } else if (section && value) {
+      out[section][key] = unquote(value);
+    }
+  }
+  return out;
+}
+
+function resolveCatalogRefs(deps, catalogs) {
+  if (!catalogs) return deps;
+  const out = { ...deps };
+  for (const [name, spec] of Object.entries(deps)) {
+    if (typeof spec !== 'string' || !spec.startsWith('catalog:')) continue;
+    const which = spec.slice('catalog:'.length) || 'default';
+    const version = catalogs[which] && catalogs[which][name];
+    if (version) out[name] = version;
+  }
+  return out;
+}
+
 export function loadProject(root) {
   const pkg = readJson(path.join(root, 'package.json'));
   if (!pkg) throw new Error(`No package.json found in ${root}`);
-  const deps = { ...(pkg.dependencies || {}) };
+  const usesCatalog = JSON.stringify([pkg.dependencies, pkg.devDependencies]).includes('"catalog:');
+  const catalogs = usesCatalog ? readCatalogs(root) : null;
+  const deps = resolveCatalogRefs({ ...(pkg.dependencies || {}) }, catalogs);
   // Monorepos often keep react-native in devDependencies; autolinking still links native modules from there.
-  const devDeps = { ...(pkg.devDependencies || {}) };
+  const devDeps = resolveCatalogRefs({ ...(pkg.devDependencies || {}) }, catalogs);
   const declared = (name) => deps[name] || devDeps[name];
   if (!declared('react-native') && !declared('expo')) {
     throw new Error('This does not look like a React Native app (no react-native or expo dependency). Run it inside your app folder.');
@@ -57,9 +116,16 @@ export function loadProject(root) {
   const hasIos = fs.existsSync(path.join(root, 'ios'));
   const appConfig = readJson(path.join(root, 'app.json'));
 
+  // A React Native library (or an example inside one) is not an app: store checks do not apply.
+  const androidGradle = readText(path.join(root, 'android', 'build.gradle')) || '';
+  const isLibrary =
+    !!(pkg.peerDependencies && pkg.peerDependencies['react-native']) ||
+    /apply plugin:\s*['"]com\.android\.library['"]|id\s*\(?\s*['"]com\.android\.library['"]/.test(androidGradle);
+
   return {
     root,
     name: pkg.name,
+    isLibrary,
     deps,
     devDeps,
     rnVersion: cleanVersion(installedVersion(root, 'react-native') || declared('react-native') || ''),

package/src/registry.js

@@ -69,6 +69,26 @@ export function createRegistry(get = fetchJson) {
       return out;
     },
 
+    // Whether the version in use, and the latest, ship a codegen spec (`codegenConfig` in their
+    // package.json), i.e. native New Architecture support. React Native Directory's flag lags
+    // releases, so this is the tie-breaker before calling a package a blocker.
+    async codegenSupport(list, concurrency = 8) {
+      const out = {};
+      const queue = [...list];
+      const manifest = (name, tag) => get(`https://registry.npmjs.org/${name.replace('/', '%2F')}/${tag}`);
+      const worker = async () => {
+        while (queue.length) {
+          const { name, version } = queue.shift();
+          const latest = await manifest(name, 'latest');
+          if (!latest) continue;
+          const used = version && version !== latest.version ? await manifest(name, version) : latest;
+          out[name] = { latest: latest.version, latestHas: !!latest.codegenConfig, usedHas: !!(used && used.codegenConfig) };
+        }
+      };
+      await Promise.all(Array.from({ length: concurrency }, worker));
+      return out;
+    },
+
     // Stable versions of a package, newest release line first.
     async releaseLines(name, pick) {
       // The abbreviated "install" document is a fraction of the full packument's size.

package/src/secrets.js

@@ -18,7 +18,7 @@ const RULES = [
   { id: 'openai-key', label: 'OpenAI API key', re: /\bsk-(?:proj-)?[A-Za-z0-9_-]{32,}\b/g },
   { id: 'anthropic-key', label: 'Anthropic API key', re: /\bsk-ant-[A-Za-z0-9_-]{32,}\b/g },
   { id: 'github-token', label: 'GitHub token', re: /\bgh[pousr]_[A-Za-z0-9]{36,}\b/g },
-  { id: 'slack-token', label: 'Slack token', re: /\bxox[baprs]-[A-Za-z0-9-]{10,}\b/g },
+  { id: 'slack-token', label: 'Slack token', re: /\bxox[baprs]-\d{6,}-[A-Za-z0-9-]{10,}\b/g },
   { id: 'sendgrid-key', label: 'SendGrid API key', re: /\b(?<v>SG\.[A-Za-z0-9_-]{22}\.[A-Za-z0-9_-]{43})\b/g },
   { id: 'twilio-key', label: 'Twilio API key', re: /\b(?<v>SK[0-9a-fA-F]{32})\b/g },
   { id: 'google-oauth-secret', label: 'Google OAuth client secret', re: /\b(?<v>GOCSPX-[A-Za-z0-9_-]{28})\b/g },
@@ -40,6 +40,14 @@ const RULES = [
   },
 ];
 
+// Values that are obviously not real: docs examples, test fixtures, templates.
+const PLACEHOLDER = /example|test|dummy|fake|sample|placeholder|xxxx|changeme|redacted|your[_-]?(?:api[_-]?)?(?:key|token|secret)/i;
+
+// Test code is never part of the app bundle.
+const TEST_PATH = /(^|\/)(__tests__|__mocks__|__fixtures__|test|tests|e2e|fixtures)\/|\.(test|spec)\.[cm]?[jt]sx?$/;
+
+const isCommentLine = (line) => /^\s*(\/\/|\/\*|\*|#|<!--)/.test(line);
+
 export function mask(value) {
   if (value.length <= 8) return '****';
   return `${value.slice(0, 4)}…${value.slice(-2)}`;
@@ -77,19 +85,26 @@ export function scanSecrets(root) {
     } catch {
       continue;
     }
+    const lines = text.split('\n');
     for (const rule of RULES) {
       for (const m of text.matchAll(rule.re)) {
         if (rule.validate && !rule.validate(m)) continue;
+        if (PLACEHOLDER.test(m[0])) continue;
         const rel = path.relative(root, file);
+        const posix = rel.split(path.sep).join('/');
         const value = m.groups && m.groups.v;
+        const line = text.slice(0, m.index).split('\n').length;
+        // Comments are stripped from the bundle; a key there is committed, not shipped.
+        const inComment = isCommentLine(lines[line - 1] || '');
         findings.push({
           rule: rule.id,
           label: rule.label,
           file: rel,
-          line: text.slice(0, m.index).split('\n').length,
+          line,
           preview: value ? mask(value) : null,
-          inAppBundle: !SERVER_DIRS.has(rel.split(path.sep)[0]),
-          committed: tracked ? tracked.has(rel.split(path.sep).join('/')) : null,
+          inAppBundle: !SERVER_DIRS.has(posix.split('/')[0]) && !TEST_PATH.test(posix) && !inComment,
+          inComment,
+          committed: tracked ? tracked.has(posix) : null,
         });
       }
     }

package/src/usage.js

@@ -125,3 +125,15 @@ export function findUnused(project) {
     .filter((n) => !nativeText.includes(n) && !nativeTokens(root, n).some((t) => nativeText.includes(t)))
     .map((name) => ({ name, native: nativeTokens(root, name).length > 0 }));
 }
+
+// Source files (relative paths) that import `pkg` and whose text matches `pattern`.
+export function filesImportingWith(root, pkg, pattern) {
+  const re = new RegExp(pattern);
+  const out = [];
+  for (const file of walk(root, JS_EXT)) {
+    const text = readSmall(file);
+    if (!text.includes(pkg) || !importedPackages(text).has(pkg)) continue;
+    if (re.test(text)) out.push(path.relative(root, file));
+  }
+  return out.sort();
+}