@grafana/create-plugin 6.8.0-canary.2356.20813982803.0 → 6.8.0-canary.2356.20814126191.0

package/dist/codemods/additions/scripts/bundle-grafana-ui/index.js

@@ -2,20 +2,19 @@ import * as v from 'valibot';
 import * as recast from 'recast';
 import { coerce, gte } from 'semver';
 import { additionsDebug } from '../../../utils.js';
+import { getBundlerConfig } from '../../../utils.bundler-config.js';
 import { updateExternalsArray } from '../../../utils.externals.js';
 
 const { builders } = recast.types;
 const PLUGIN_JSON_PATH = "src/plugin.json";
 const MIN_GRAFANA_VERSION = "10.2.0";
-const WEBPACK_CONFIG_PATH = ".config/webpack/webpack.config.ts";
-const RSPACK_CONFIG_PATH = ".config/rspack/rspack.config.ts";
 const schema = v.object({});
 function bundleGrafanaUI(context, _options) {
   additionsDebug("Running bundle-grafana-ui addition...");
   ensureMinGrafanaVersion(context);
   updateExternalsArray(context, createBundleGrafanaUIModifier());
-  updateResolveExtensionsSimple(context);
-  updateModuleRulesSimple(context);
+  updateResolveExtensions(context);
+  updateModuleRules(context);
   return context;
 }
 function isGrafanaRegex(element, pattern) {
@@ -78,15 +77,12 @@ function createBundleGrafanaUIModifier() {
     return removeGrafanaUiAndAddReactInlineSvg(externalsArray);
   };
 }
-function updateResolveExtensionsSimple(context) {
-  const configPath = context.doesFileExist(RSPACK_CONFIG_PATH) ? RSPACK_CONFIG_PATH : context.doesFileExist(WEBPACK_CONFIG_PATH) ? WEBPACK_CONFIG_PATH : null;
-  if (!configPath) {
-    return;
-  }
-  const content = context.getFile(configPath);
-  if (!content) {
+function updateResolveExtensions(context) {
+  const config = getBundlerConfig(context);
+  if (!config) {
     return;
   }
+  const { path: configPath, content } = config;
   if (content.includes("'.mjs'") || content.includes('".mjs"')) {
     return;
   }
@@ -101,15 +97,12 @@ function updateResolveExtensionsSimple(context) {
     additionsDebug("Added '.mjs' to resolve.extensions");
   }
 }
-function updateModuleRulesSimple(context) {
-  const configPath = context.doesFileExist(RSPACK_CONFIG_PATH) ? RSPACK_CONFIG_PATH : context.doesFileExist(WEBPACK_CONFIG_PATH) ? WEBPACK_CONFIG_PATH : null;
-  if (!configPath) {
-    return;
-  }
-  const content = context.getFile(configPath);
-  if (!content) {
+function updateModuleRules(context) {
+  const config = getBundlerConfig(context);
+  if (!config) {
     return;
   }
+  const { path: configPath, content } = config;
   if (content.includes("test: /\\.mjs$") || content.includes("test: /\\\\.mjs$")) {
     return;
   }

package/dist/codemods/utils.bundler-config.js

@@ -0,0 +1,19 @@
+import 'recast';
+import 'recast/parsers/typescript.js';
+import './utils.js';
+
+const WEBPACK_CONFIG_PATH = ".config/webpack/webpack.config.ts";
+const RSPACK_CONFIG_PATH = ".config/rspack/rspack.config.ts";
+function getBundlerConfig(context) {
+  const configPath = context.doesFileExist(RSPACK_CONFIG_PATH) ? RSPACK_CONFIG_PATH : context.doesFileExist(WEBPACK_CONFIG_PATH) ? WEBPACK_CONFIG_PATH : null;
+  if (!configPath) {
+    return null;
+  }
+  const content = context.getFile(configPath);
+  if (!content) {
+    return null;
+  }
+  return { path: configPath, content };
+}
+
+export { getBundlerConfig };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@grafana/create-plugin",
-  "version": "6.8.0-canary.2356.20813982803.0",
+  "version": "6.8.0-canary.2356.20814126191.0",
   "repository": {
     "directory": "packages/create-plugin",
     "url": "https://github.com/grafana/plugin-tools"
@@ -55,5 +55,5 @@
   "engines": {
     "node": ">=20"
   },
-  "gitHead": "fedff1c23ee3079d90f0ed2d6984c78fb294b1b9"
+  "gitHead": "cbc5b5a6579bc798c0db9c3caee8443e0c9bf8c7"
 }

package/src/codemods/additions/scripts/bundle-grafana-ui/index.ts

@@ -4,14 +4,13 @@ import { coerce, gte } from 'semver';
 
 import type { Context } from '../../../context.js';
 import { additionsDebug } from '../../../utils.js';
+import { getBundlerConfig } from '../../../utils.bundler-config.js';
 import { updateExternalsArray, type ExternalsArrayModifier } from '../../../utils.externals.js';
 
 const { builders } = recast.types;
 
 const PLUGIN_JSON_PATH = 'src/plugin.json';
 const MIN_GRAFANA_VERSION = '10.2.0';
-const WEBPACK_CONFIG_PATH = '.config/webpack/webpack.config.ts';
-const RSPACK_CONFIG_PATH = '.config/rspack/rspack.config.ts';
 
 export const schema = v.object({});
 type BundleGrafanaUIOptions = v.InferOutput<typeof schema>;
@@ -26,10 +25,10 @@ export default function bundleGrafanaUI(context: Context, _options: BundleGrafan
   updateExternalsArray(context, createBundleGrafanaUIModifier());
 
   // Update bundler resolve configuration to handle ESM imports
-  updateResolveExtensionsSimple(context);
+  updateResolveExtensions(context);
 
   // Update module rules directly with simple string manipulation
-  updateModuleRulesSimple(context);
+  updateModuleRules(context);
 
   return context;
 }
@@ -132,24 +131,15 @@ function createBundleGrafanaUIModifier(): ExternalsArrayModifier {
 }
 
 /**
- * Updates resolve extensions to add .mjs using simple string manipulation
- * This is a simplified approach that may not handle all edge cases, but is much simpler
+ * Updates resolve extensions to add .mjs using string manipulation
  */
-function updateResolveExtensionsSimple(context: Context): void {
-  const configPath = context.doesFileExist(RSPACK_CONFIG_PATH)
-    ? RSPACK_CONFIG_PATH
-    : context.doesFileExist(WEBPACK_CONFIG_PATH)
-      ? WEBPACK_CONFIG_PATH
-      : null;
-
-  if (!configPath) {
+function updateResolveExtensions(context: Context): void {
+  const config = getBundlerConfig(context);
+  if (!config) {
     return;
   }
 
-  const content = context.getFile(configPath);
-  if (!content) {
-    return;
-  }
+  const { path: configPath, content } = config;
 
   // Check if .mjs already exists
   if (content.includes("'.mjs'") || content.includes('".mjs"')) {
@@ -171,24 +161,15 @@ function updateResolveExtensionsSimple(context: Context): void {
 }
 
 /**
- * Updates module rules to add .mjs rule using simple string manipulation
- * This is a simplified approach that may not handle all edge cases, but is much simpler
+ * Updates module rules to add .mjs rule using string manipulation
  */
-function updateModuleRulesSimple(context: Context): void {
-  const configPath = context.doesFileExist(RSPACK_CONFIG_PATH)
-    ? RSPACK_CONFIG_PATH
-    : context.doesFileExist(WEBPACK_CONFIG_PATH)
-      ? WEBPACK_CONFIG_PATH
-      : null;
-
-  if (!configPath) {
+function updateModuleRules(context: Context): void {
+  const config = getBundlerConfig(context);
+  if (!config) {
     return;
   }
 
-  const content = context.getFile(configPath);
-  if (!content) {
-    return;
-  }
+  const { path: configPath, content } = config;
 
   // Check if rule already exists
   if (content.includes('test: /\\.mjs$') || content.includes('test: /\\\\.mjs$')) {

package/src/codemods/utils.bundler-config.ts

@@ -7,6 +7,29 @@ import { additionsDebug } from './utils.js';
 const WEBPACK_CONFIG_PATH = '.config/webpack/webpack.config.ts';
 const RSPACK_CONFIG_PATH = '.config/rspack/rspack.config.ts';
 
+/**
+ * Gets the bundler config file path and content, preferring rspack over webpack
+ * @returns Object with path and content, or null if no config file exists
+ */
+export function getBundlerConfig(context: Context): { path: string; content: string } | null {
+  const configPath = context.doesFileExist(RSPACK_CONFIG_PATH)
+    ? RSPACK_CONFIG_PATH
+    : context.doesFileExist(WEBPACK_CONFIG_PATH)
+      ? WEBPACK_CONFIG_PATH
+      : null;
+
+  if (!configPath) {
+    return null;
+  }
+
+  const content = context.getFile(configPath);
+  if (!content) {
+    return null;
+  }
+
+  return { path: configPath, content };
+}
+
 /**
  * Type for a function that modifies a resolve object expression
  * @param resolveObject - The AST node representing the resolve configuration