@grafana/react-detect 0.3.0 → 0.4.0

package/CHANGELOG.md

@@ -1,3 +1,15 @@
+# v0.4.0 (Thu Jan 15 2026)
+
+#### 🚀 Enhancement
+
+- feat: allow skipping dependency and build tooling analysis [#2397](https://github.com/grafana/plugin-tools/pull/2397) ([@jackw](https://github.com/jackw))
+
+#### Authors: 1
+
+- Jack Westbrook ([@jackw](https://github.com/jackw))
+
+---
+
 # v0.3.0 (Thu Jan 15 2026)
 
 #### 🚀 Enhancement

package/README.md

@@ -21,3 +21,5 @@ npx @grafana/react-detect
 
 - `--pluginRoot`: Pass the root directory of the plugin. Defaults to the current working directory.
 - `--json`: Output the results as json.
+- `--skipBuildTooling`: Skip webpack configuration checks (jsx-runtime externalization). Useful when build config is missing or not applicable.
+- `--skipDependencies`: Skip dependency tree analysis. Useful when lockfile is missing or for faster analysis of source code only.

package/dist/bin/run.js

@@ -3,7 +3,15 @@ import minimist from 'minimist';
 import { detect19 } from '../commands/detect19.js';
 
 const args = process.argv.slice(2);
-const argv = minimist(args);
+const argv = minimist(args, {
+  boolean: ["json", "skipBuildTooling", "skipDependencies"],
+  string: ["pluginRoot"],
+  default: {
+    json: false,
+    skipBuildTooling: false,
+    skipDependencies: false
+  }
+});
 const commands = {
   detect19
 };

package/dist/commands/detect19.js

@@ -10,16 +10,36 @@ import { output } from '../utils/output.js';
 async function detect19(argv) {
   try {
     const pluginRoot = argv.pluginRoot || process.cwd();
+    const skipDependencies = argv.skipDependencies || false;
+    const skipBuildTooling = argv.skipBuildTooling || false;
     const allMatches = await getAllMatches(pluginRoot);
-    const depContext = new DependencyContext();
-    await depContext.loadDependencies(pluginRoot);
-    const matchesWithRootDependency = allMatches.map((match) => {
+    let depContext = null;
+    if (!skipDependencies) {
+      depContext = new DependencyContext();
+      try {
+        await depContext.loadDependencies(pluginRoot);
+      } catch (error) {
+        output.warning({
+          title: "Failed to load dependencies",
+          body: [
+            error.message,
+            "Continuing without dependency analysis.",
+            "Use --skipDependencies to suppress this warning."
+          ]
+        });
+        depContext = null;
+      }
+    }
+    const matchesWithRootDependency = depContext === null ? allMatches : allMatches.map((match) => {
       if (match.type === "dependency" && match.packageName) {
         return { ...match, rootDependency: depContext.findRootDependency(match.packageName) };
       }
       return match;
     });
-    const results = generateAnalysisResults(matchesWithRootDependency, pluginRoot, depContext);
+    const results = generateAnalysisResults(matchesWithRootDependency, pluginRoot, depContext, {
+      skipBuildTooling,
+      skipDependencies
+    });
     if (argv.json) {
       jsonReporter(results);
     } else {

package/dist/reporters/console.js

@@ -14,6 +14,25 @@ const summaryMsg = [
   `Grafana developer documentation: ${output.formatUrl("https://grafana.com/developers/plugin-tools/set-up/set-up-docker")}`
 ];
 function consoleReporter(results) {
+  if (!results.summary.analyzedBuildTooling || !results.summary.analyzedDependencies) {
+    const skippedFeatures = [];
+    if (!results.summary.analyzedBuildTooling) {
+      skippedFeatures.push("Build tooling analysis (webpack config checks)");
+    }
+    if (!results.summary.analyzedDependencies) {
+      skippedFeatures.push("Dependency tree analysis (lockfile parsing)");
+    }
+    output.warning({
+      title: "Analysis Skipped",
+      body: [
+        "The following analyses were skipped:",
+        ...output.bulletList(skippedFeatures),
+        "",
+        "Results may be incomplete. Remove skip flags for full analysis."
+      ]
+    });
+    output.addHorizontalLine("yellow");
+  }
   if (results.summary.totalIssues === 0) {
     output.success({
       title: "No React 19 breaking changes detected.",

package/dist/results.js

@@ -2,8 +2,8 @@ import { getPattern } from './patterns/definitions.js';
 import { getPluginJson, hasExternalisedJsxRuntime } from './utils/plugin.js';
 import path from 'node:path';
 
-function generateAnalysisResults(matches, pluginRoot, depContext) {
-  const filtered = filterMatches(matches);
+function generateAnalysisResults(matches, pluginRoot, depContext, options = { skipBuildTooling: false, skipDependencies: false }) {
+  const filtered = filterMatches(matches, options.skipBuildTooling);
   const pluginJson = getPluginJson(pluginRoot);
   const sourceMatches = filtered.filter((m) => m.type === "source");
   const dependencyMatches = filtered.filter((m) => m.type === "dependency");
@@ -36,7 +36,9 @@ function generateAnalysisResults(matches, pluginRoot, depContext) {
       sourceIssuesCount: sourceMatches.length,
       dependencyIssuesCount: dependencyMatches.length,
       status: totalIssues > 0 ? "action_required" : "no_action_required",
-      affectedDependencies: Array.from(affectedDeps)
+      affectedDependencies: Array.from(affectedDeps),
+      analyzedBuildTooling: !options.skipBuildTooling,
+      analyzedDependencies: !options.skipDependencies
     },
     issues: {
       critical,
@@ -45,8 +47,8 @@ function generateAnalysisResults(matches, pluginRoot, depContext) {
     }
   };
 }
-function filterMatches(matches) {
-  const externalisedJsxRuntime = hasExternalisedJsxRuntime();
+function filterMatches(matches, skipBuildTooling) {
+  const externalisedJsxRuntime = skipBuildTooling ? false : hasExternalisedJsxRuntime();
   const filtered = matches.filter((match) => {
     if (match.type === "source" && (match.confidence === "none" || match.confidence === "unknown")) {
       return false;
@@ -109,7 +111,7 @@ function buildDependencyIssues(dependencyMatches, depContext) {
   const issues = [];
   for (const [packageName, matches] of byPackage) {
     const rootDep = matches[0].rootDependency || packageName;
-    const version = depContext.getVersion(packageName) || null;
+    const version = depContext?.getVersion(packageName) || null;
     issues.push({
       packageName,
       version: version || "unknown",

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@grafana/react-detect",
   "description": "Run various checks to detect if a Grafana plugin is compatible with React.",
-  "version": "0.3.0",
+  "version": "0.4.0",
   "repository": {
     "directory": "packages/react-detect",
     "url": "https://github.com/grafana/plugin-tools"
@@ -40,5 +40,5 @@
   "engines": {
     "node": ">=20"
   },
-  "gitHead": "41320751786a4b7fa6c5addf92fb7f81a4df6702"
+  "gitHead": "1875c3b665a6bb447a6d2fdea23df11de9c066f2"
 }

package/src/bin/run.ts

@@ -4,7 +4,15 @@ import minimist from 'minimist';
 import { detect19 } from '../commands/detect19.js';
 
 const args = process.argv.slice(2);
-const argv = minimist(args);
+const argv = minimist(args, {
+  boolean: ['json', 'skipBuildTooling', 'skipDependencies'],
+  string: ['pluginRoot'],
+  default: {
+    json: false,
+    skipBuildTooling: false,
+    skipDependencies: false,
+  },
+});
 
 const commands: Record<string, (argv: minimist.ParsedArgs) => Promise<void>> = {
   detect19,

package/src/commands/detect19.ts

@@ -13,19 +13,46 @@ import { output } from '../utils/output.js';
 export async function detect19(argv: minimist.ParsedArgs) {
   try {
     const pluginRoot = argv.pluginRoot || process.cwd();
+    const skipDependencies = argv.skipDependencies || false;
+    const skipBuildTooling = argv.skipBuildTooling || false;
 
     const allMatches = await getAllMatches(pluginRoot);
-    const depContext = new DependencyContext();
-    await depContext.loadDependencies(pluginRoot);
 
-    const matchesWithRootDependency = allMatches.map((match) => {
-      if (match.type === 'dependency' && match.packageName) {
-        return { ...match, rootDependency: depContext.findRootDependency(match.packageName) };
+    // Conditionally load dependencies
+    let depContext: DependencyContext | null = null;
+    if (!skipDependencies) {
+      depContext = new DependencyContext();
+      try {
+        await depContext.loadDependencies(pluginRoot);
+      } catch (error) {
+        // Log warning but continue with null context
+        output.warning({
+          title: 'Failed to load dependencies',
+          body: [
+            (error as Error).message,
+            'Continuing without dependency analysis.',
+            'Use --skipDependencies to suppress this warning.',
+          ],
+        });
+        depContext = null;
       }
-      return match;
-    });
+    }
 
-    const results = generateAnalysisResults(matchesWithRootDependency, pluginRoot, depContext);
+    // Enrich matches with root dependency only if we have context
+    const matchesWithRootDependency =
+      depContext === null
+        ? allMatches
+        : allMatches.map((match) => {
+            if (match.type === 'dependency' && match.packageName) {
+              return { ...match, rootDependency: depContext.findRootDependency(match.packageName) };
+            }
+            return match;
+          });
+
+    const results = generateAnalysisResults(matchesWithRootDependency, pluginRoot, depContext, {
+      skipBuildTooling,
+      skipDependencies,
+    });
 
     if (argv.json) {
       jsonReporter(results);

package/src/reporters/console.ts

@@ -16,6 +16,27 @@ const summaryMsg = [
 ];
 
 export function consoleReporter(results: PluginAnalysisResults) {
+  if (!results.summary.analyzedBuildTooling || !results.summary.analyzedDependencies) {
+    const skippedFeatures: string[] = [];
+    if (!results.summary.analyzedBuildTooling) {
+      skippedFeatures.push('Build tooling analysis (webpack config checks)');
+    }
+    if (!results.summary.analyzedDependencies) {
+      skippedFeatures.push('Dependency tree analysis (lockfile parsing)');
+    }
+
+    output.warning({
+      title: 'Analysis Skipped',
+      body: [
+        'The following analyses were skipped:',
+        ...output.bulletList(skippedFeatures),
+        '',
+        'Results may be incomplete. Remove skip flags for full analysis.',
+      ],
+    });
+    output.addHorizontalLine('yellow');
+  }
+
   if (results.summary.totalIssues === 0) {
     output.success({
       title: 'No React 19 breaking changes detected.',

package/src/results.ts

@@ -5,12 +5,18 @@ import { getPluginJson, hasExternalisedJsxRuntime } from './utils/plugin.js';
 import { DependencyContext } from './utils/dependencies.js';
 import path from 'node:path';
 
+export interface AnalysisOptions {
+  skipBuildTooling: boolean;
+  skipDependencies: boolean;
+}
+
 export function generateAnalysisResults(
   matches: AnalyzedMatch[],
   pluginRoot: string,
-  depContext: DependencyContext
+  depContext: DependencyContext | null,
+  options: AnalysisOptions = { skipBuildTooling: false, skipDependencies: false }
 ): PluginAnalysisResults {
-  const filtered = filterMatches(matches);
+  const filtered = filterMatches(matches, options.skipBuildTooling);
   const pluginJson = getPluginJson(pluginRoot);
   const sourceMatches = filtered.filter((m) => m.type === 'source');
   const dependencyMatches = filtered.filter((m) => m.type === 'dependency');
@@ -49,6 +55,8 @@ export function generateAnalysisResults(
       dependencyIssuesCount: dependencyMatches.length,
       status: totalIssues > 0 ? 'action_required' : 'no_action_required',
       affectedDependencies: Array.from(affectedDeps),
+      analyzedBuildTooling: !options.skipBuildTooling,
+      analyzedDependencies: !options.skipDependencies,
     },
     issues: {
       critical,
@@ -58,8 +66,9 @@ export function generateAnalysisResults(
   };
 }
 
-function filterMatches(matches: AnalyzedMatch[]): AnalyzedMatch[] {
-  const externalisedJsxRuntime = hasExternalisedJsxRuntime();
+function filterMatches(matches: AnalyzedMatch[], skipBuildTooling: boolean): AnalyzedMatch[] {
+  // Only check webpack config if NOT skipping build tooling
+  const externalisedJsxRuntime = skipBuildTooling ? false : hasExternalisedJsxRuntime();
   const filtered = matches.filter((match) => {
     // TODO: add mode for strict / loose filtering
     if (match.type === 'source' && (match.confidence === 'none' || match.confidence === 'unknown')) {
@@ -139,7 +148,10 @@ function generateResult(match: AnalyzedMatch): AnalysisResult {
 /**
  * Build DependencyIssue objects grouped by package
  */
-function buildDependencyIssues(dependencyMatches: AnalyzedMatch[], depContext: DependencyContext): DependencyIssue[] {
+function buildDependencyIssues(
+  dependencyMatches: AnalyzedMatch[],
+  depContext: DependencyContext | null
+): DependencyIssue[] {
   // Group by package
   const byPackage = new Map<string, AnalyzedMatch[]>();
   for (const match of dependencyMatches) {
@@ -154,7 +166,8 @@ function buildDependencyIssues(dependencyMatches: AnalyzedMatch[], depContext: D
   const issues: DependencyIssue[] = [];
   for (const [packageName, matches] of byPackage) {
     const rootDep = matches[0].rootDependency || packageName;
-    const version = depContext.getVersion(packageName) || null;
+    // Handle null depContext gracefully with optional chaining
+    const version = depContext?.getVersion(packageName) || null;
 
     issues.push({
       packageName,

package/src/types/reporters.ts

@@ -32,6 +32,8 @@ export interface PluginAnalysisResults {
     dependencyIssuesCount: number;
     status: 'action_required' | 'no_action_required';
     affectedDependencies: string[];
+    analyzedBuildTooling: boolean;
+    analyzedDependencies: boolean;
   };
   issues: {
     critical: AnalysisResult[];