@grafana/create-plugin 6.8.0-canary.2356.20813734793.0 → 6.8.0-canary.2356.20814023201.0

package/CHANGELOG.md

@@ -1,3 +1,15 @@
+# v6.7.1 (Thu Jan 08 2026)
+
+#### 🐛 Bug Fix
+
+- fix: stop copying all images to bundle [#2369](https://github.com/grafana/plugin-tools/pull/2369) ([@jackw](https://github.com/jackw))
+
+#### Authors: 1
+
+- Jack Westbrook ([@jackw](https://github.com/jackw))
+
+---
+
 # v6.7.0 (Wed Jan 07 2026)
 
 #### 🚀 Enhancement

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

@@ -2,39 +2,30 @@ import * as v from 'valibot';
 import * as recast from 'recast';
 import { coerce, gte } from 'semver';
 import { additionsDebug } from '../../../utils.js';
-import { updateBundlerConfig } 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());
-  updateBundlerConfig(context, createResolveModifier(), createModuleRulesModifier());
+  updateResolveExtensions(context);
+  updateModuleRules(context);
   return context;
 }
-function isGrafanaUiRegex(element) {
+function isGrafanaRegex(element, pattern) {
   if (element.type === "RegExpLiteral") {
     const regexNode = element;
-    return regexNode.pattern === "^@grafana\\/ui" && regexNode.flags === "i";
+    return regexNode.pattern === pattern && regexNode.flags === "i";
   }
   if (element.type === "Literal" && "regex" in element && element.regex) {
     const regex = element.regex;
-    return regex.pattern === "^@grafana\\/ui" && regex.flags === "i";
-  }
-  return false;
-}
-function isGrafanaDataRegex(element) {
-  if (element.type === "RegExpLiteral") {
-    const regexNode = element;
-    return regexNode.pattern === "^@grafana\\/data" && regexNode.flags === "i";
-  }
-  if (element.type === "Literal" && "regex" in element && element.regex) {
-    const regex = element.regex;
-    return regex.pattern === "^@grafana\\/data" && regex.flags === "i";
+    return regex.pattern === pattern && regex.flags === "i";
   }
   return false;
 }
@@ -46,7 +37,7 @@ function removeGrafanaUiAndAddReactInlineSvg(externalsArray) {
     if (!element) {
       continue;
     }
-    if (isGrafanaUiRegex(element)) {
+    if (isGrafanaRegex(element, "^@grafana\\/ui")) {
       hasGrafanaUiExternal = true;
     }
     if ((element.type === "Literal" || element.type === "StringLiteral") && "value" in element && typeof element.value === "string" && element.value === "react-inlinesvg") {
@@ -58,7 +49,7 @@ function removeGrafanaUiAndAddReactInlineSvg(externalsArray) {
       if (!element) {
         return true;
       }
-      return !isGrafanaUiRegex(element);
+      return !isGrafanaRegex(element, "^@grafana\\/ui");
     });
     hasChanges = true;
     additionsDebug("Removed /^@grafana\\/ui/i from externals array");
@@ -67,7 +58,7 @@ function removeGrafanaUiAndAddReactInlineSvg(externalsArray) {
     let insertIndex = -1;
     for (let i = 0; i < externalsArray.elements.length; i++) {
       const element = externalsArray.elements[i];
-      if (element && isGrafanaDataRegex(element)) {
+      if (element && isGrafanaRegex(element, "^@grafana\\/data")) {
         insertIndex = i + 1;
         break;
       }
@@ -87,106 +78,66 @@ function createBundleGrafanaUIModifier() {
     return removeGrafanaUiAndAddReactInlineSvg(externalsArray);
   };
 }
-function createResolveModifier() {
-  return (resolveObject) => {
-    if (!resolveObject.properties) {
-      return false;
-    }
-    let hasChanges = false;
-    let hasMjsExtension = false;
-    let extensionsProperty = null;
-    for (const prop of resolveObject.properties) {
-      if (!prop || prop.type !== "Property" && prop.type !== "ObjectProperty") {
-        continue;
-      }
-      const key = "key" in prop ? prop.key : null;
-      const value = "value" in prop ? prop.value : null;
-      if (key && key.type === "Identifier") {
-        if (key.name === "extensions" && value && value.type === "ArrayExpression") {
-          extensionsProperty = prop;
-          for (const element of value.elements) {
-            if (element && (element.type === "Literal" || element.type === "StringLiteral") && "value" in element && element.value === ".mjs") {
-              hasMjsExtension = true;
-              break;
-            }
-          }
-        }
-      }
-    }
-    if (!hasMjsExtension && extensionsProperty && "value" in extensionsProperty) {
-      const extensionsArray = extensionsProperty.value;
-      extensionsArray.elements.push(builders.literal(".mjs"));
-      hasChanges = true;
-      additionsDebug("Added '.mjs' to resolve.extensions");
+function updateResolveExtensions(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) {
+    return;
+  }
+  if (content.includes("'.mjs'") || content.includes('".mjs"')) {
+    return;
+  }
+  const updated = content.replace(/(extensions:\s*\[)([^\]]+)(\])/, (match, prefix, extensions, suffix) => {
+    if (extensions.includes(".mjs")) {
+      return match;
     }
-    return hasChanges;
-  };
+    return `${prefix}${extensions}, '.mjs'${suffix}`;
+  });
+  if (updated !== content) {
+    context.updateFile(configPath, updated);
+    additionsDebug("Added '.mjs' to resolve.extensions");
+  }
 }
-function createModuleRulesModifier() {
-  return (moduleObject) => {
-    if (!moduleObject.properties) {
-      return false;
-    }
-    let hasChanges = false;
-    let hasMjsRule = false;
-    let rulesProperty = null;
-    for (const prop of moduleObject.properties) {
-      if (!prop || prop.type !== "Property" && prop.type !== "ObjectProperty") {
-        continue;
-      }
-      const key = "key" in prop ? prop.key : null;
-      const value = "value" in prop ? prop.value : null;
-      if (key && key.type === "Identifier" && key.name === "rules" && value && value.type === "ArrayExpression") {
-        rulesProperty = prop;
-        for (const element of value.elements) {
-          if (element && element.type === "ObjectExpression" && element.properties) {
-            for (const ruleProp of element.properties) {
-              if (ruleProp && (ruleProp.type === "Property" || ruleProp.type === "ObjectProperty") && "key" in ruleProp && ruleProp.key.type === "Identifier" && ruleProp.key.name === "test") {
-                const testValue = "value" in ruleProp ? ruleProp.value : null;
-                if (testValue) {
-                  if (testValue.type === "RegExpLiteral" && "pattern" in testValue && testValue.pattern === "\\.mjs$") {
-                    hasMjsRule = true;
-                    break;
-                  }
-                  if (testValue.type === "Literal" && "regex" in testValue && testValue.regex && typeof testValue.regex === "object" && "pattern" in testValue.regex && testValue.regex.pattern === "\\.mjs$") {
-                    hasMjsRule = true;
-                    break;
-                  }
-                  if (testValue.type === "Literal" && "value" in testValue && typeof testValue.value === "string" && testValue.value.includes(".mjs")) {
-                    hasMjsRule = true;
-                    break;
-                  }
-                }
-              }
-            }
-          }
-          if (hasMjsRule) {
-            break;
-          }
-        }
-        break;
+function updateModuleRules(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) {
+    return;
+  }
+  if (content.includes("test: /\\.mjs$") || content.includes("test: /\\\\.mjs$")) {
+    return;
+  }
+  const mjsRule = `{
+        test: /\\.mjs$/,
+        include: /node_modules/,
+        resolve: {
+          fullySpecified: false,
+        },
+        type: 'javascript/auto',
+      },`;
+  let updated = content;
+  if (content.match(/rules:\s*\[\s*\]/)) {
+    updated = content.replace(/(rules:\s*\[\s*)(\])/, `$1${mjsRule}
+      $2`);
+  } else {
+    updated = content.replace(/(rules:\s*\[\s*)(\{[\s\S]*?\}),(\s*)/, (match, prefix, firstRule, suffix) => {
+      if (match.includes("test: /\\.mjs$")) {
+        return match;
       }
-    }
-    if (!hasMjsRule && rulesProperty && "value" in rulesProperty) {
-      const rulesArray = rulesProperty.value;
-      const mjsRule = builders.objectExpression([
-        builders.property("init", builders.identifier("test"), builders.literal(/\.mjs$/)),
-        builders.property("init", builders.identifier("include"), builders.literal(/node_modules/)),
-        builders.property(
-          "init",
-          builders.identifier("resolve"),
-          builders.objectExpression([
-            builders.property("init", builders.identifier("fullySpecified"), builders.literal(false))
-          ])
-        ),
-        builders.property("init", builders.identifier("type"), builders.literal("javascript/auto"))
-      ]);
-      rulesArray.elements.splice(1, 0, mjsRule);
-      hasChanges = true;
-      additionsDebug("Added module rule for .mjs files in node_modules with resolve.fullySpecified: false");
-    }
-    return hasChanges;
-  };
+      return `${prefix}${firstRule},
+      ${mjsRule}${suffix}`;
+    });
+  }
+  if (updated !== content) {
+    context.updateFile(configPath, updated);
+    additionsDebug("Added module rule for .mjs files in node_modules with resolve.fullySpecified: false");
+  }
 }
 function ensureMinGrafanaVersion(context) {
   if (!context.doesFileExist(PLUGIN_JSON_PATH)) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@grafana/create-plugin",
-  "version": "6.8.0-canary.2356.20813734793.0",
+  "version": "6.8.0-canary.2356.20814023201.0",
   "repository": {
     "directory": "packages/create-plugin",
     "url": "https://github.com/grafana/plugin-tools"
@@ -55,5 +55,5 @@
   "engines": {
     "node": ">=20"
   },
-  "gitHead": "7dc0a04f78794e1d23d24b13fa2dfaa5a4312bc3"
+  "gitHead": "c56736e56659f6e4b808d59225f1807e32e94062"
 }

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

@@ -4,13 +4,14 @@ import { coerce, gte } from 'semver';
 
 import type { Context } from '../../../context.js';
 import { additionsDebug } from '../../../utils.js';
-import { updateBundlerConfig, type ModuleRulesModifier, type ResolveModifier } 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>;
@@ -25,43 +26,29 @@ export default function bundleGrafanaUI(context: Context, _options: BundleGrafan
   updateExternalsArray(context, createBundleGrafanaUIModifier());
 
   // Update bundler resolve configuration to handle ESM imports
-  updateBundlerConfig(context, createResolveModifier(), createModuleRulesModifier());
+  updateResolveExtensions(context);
 
-  return context;
-}
+  // Update module rules directly with simple string manipulation
+  updateModuleRules(context);
 
-/**
- * Checks if an AST node is a regex matching @grafana/ui
- * The pattern in the AST is "^@grafana\/ui" (backslash-escaped forward slash)
- */
-function isGrafanaUiRegex(element: recast.types.namedTypes.ASTNode): boolean {
-  // Handle RegExpLiteral (TypeScript parser)
-  if (element.type === 'RegExpLiteral') {
-    const regexNode = element as recast.types.namedTypes.RegExpLiteral;
-    return regexNode.pattern === '^@grafana\\/ui' && regexNode.flags === 'i';
-  }
-  // Handle Literal with regex property (other parsers)
-  if (element.type === 'Literal' && 'regex' in element && element.regex) {
-    const regex = element.regex as { pattern: string; flags: string };
-    return regex.pattern === '^@grafana\\/ui' && regex.flags === 'i';
-  }
-  return false;
+  return context;
 }
 
 /**
- * Checks if an AST node is a regex matching @grafana/data
- * The pattern in the AST is "^@grafana\/data" (backslash-escaped forward slash)
+ * Checks if an AST node is a regex matching a Grafana package pattern
+ * @param element - The AST node to check
+ * @param pattern - The regex pattern to match (e.g., "^@grafana\\/ui" or "^@grafana\\/data")
  */
-function isGrafanaDataRegex(element: recast.types.namedTypes.ASTNode): boolean {
+function isGrafanaRegex(element: recast.types.namedTypes.ASTNode, pattern: string): boolean {
   // Handle RegExpLiteral (TypeScript parser)
   if (element.type === 'RegExpLiteral') {
     const regexNode = element as recast.types.namedTypes.RegExpLiteral;
-    return regexNode.pattern === '^@grafana\\/data' && regexNode.flags === 'i';
+    return regexNode.pattern === pattern && regexNode.flags === 'i';
   }
   // Handle Literal with regex property (other parsers)
   if (element.type === 'Literal' && 'regex' in element && element.regex) {
     const regex = element.regex as { pattern: string; flags: string };
-    return regex.pattern === '^@grafana\\/data' && regex.flags === 'i';
+    return regex.pattern === pattern && regex.flags === 'i';
   }
   return false;
 }
@@ -82,7 +69,7 @@ function removeGrafanaUiAndAddReactInlineSvg(externalsArray: recast.types.namedT
     }
 
     // Check for /^@grafana\/ui/i regex
-    if (isGrafanaUiRegex(element)) {
+    if (isGrafanaRegex(element, '^@grafana\\/ui')) {
       hasGrafanaUiExternal = true;
     }
 
@@ -103,7 +90,7 @@ function removeGrafanaUiAndAddReactInlineSvg(externalsArray: recast.types.namedT
       if (!element) {
         return true;
       }
-      return !isGrafanaUiRegex(element);
+      return !isGrafanaRegex(element, '^@grafana\\/ui');
     });
     hasChanges = true;
     additionsDebug('Removed /^@grafana\\/ui/i from externals array');
@@ -115,7 +102,7 @@ function removeGrafanaUiAndAddReactInlineSvg(externalsArray: recast.types.namedT
     let insertIndex = -1;
     for (let i = 0; i < externalsArray.elements.length; i++) {
       const element = externalsArray.elements[i];
-      if (element && isGrafanaDataRegex(element)) {
+      if (element && isGrafanaRegex(element, '^@grafana\\/data')) {
         insertIndex = i + 1;
         break;
       }
@@ -145,158 +132,101 @@ function createBundleGrafanaUIModifier(): ExternalsArrayModifier {
 }
 
 /**
- * Creates a modifier function for updateBundlerConfig that adds '.mjs' to resolve.extensions
+ * Updates resolve extensions to add .mjs using string manipulation
  */
-function createResolveModifier(): ResolveModifier {
-  return (resolveObject: recast.types.namedTypes.ObjectExpression): boolean => {
-    if (!resolveObject.properties) {
-      return false;
-    }
-
-    let hasChanges = false;
-    let hasMjsExtension = false;
-    let extensionsProperty: recast.types.namedTypes.Property | null = null;
+function updateResolveExtensions(context: Context): void {
+  const configPath = context.doesFileExist(RSPACK_CONFIG_PATH)
+    ? RSPACK_CONFIG_PATH
+    : context.doesFileExist(WEBPACK_CONFIG_PATH)
+      ? WEBPACK_CONFIG_PATH
+      : null;
+
+  if (!configPath) {
+    return;
+  }
 
-    // Check current state
-    for (const prop of resolveObject.properties) {
-      if (!prop || (prop.type !== 'Property' && prop.type !== 'ObjectProperty')) {
-        continue;
-      }
+  const content = context.getFile(configPath);
+  if (!content) {
+    return;
+  }
 
-      const key = 'key' in prop ? prop.key : null;
-      const value = 'value' in prop ? prop.value : null;
-
-      if (key && key.type === 'Identifier') {
-        if (key.name === 'extensions' && value && value.type === 'ArrayExpression') {
-          extensionsProperty = prop as recast.types.namedTypes.Property;
-          // Check if .mjs is already in the extensions array
-          for (const element of value.elements) {
-            if (
-              element &&
-              (element.type === 'Literal' || element.type === 'StringLiteral') &&
-              'value' in element &&
-              element.value === '.mjs'
-            ) {
-              hasMjsExtension = true;
-              break;
-            }
-          }
-        }
-      }
-    }
+  // Check if .mjs already exists
+  if (content.includes("'.mjs'") || content.includes('".mjs"')) {
+    return;
+  }
 
-    // Add .mjs to extensions if missing
-    if (!hasMjsExtension && extensionsProperty && 'value' in extensionsProperty) {
-      const extensionsArray = extensionsProperty.value as recast.types.namedTypes.ArrayExpression;
-      extensionsArray.elements.push(builders.literal('.mjs'));
-      hasChanges = true;
-      additionsDebug("Added '.mjs' to resolve.extensions");
+  // Add .mjs to extensions array
+  const updated = content.replace(/(extensions:\s*\[)([^\]]+)(\])/, (match, prefix, extensions, suffix) => {
+    if (extensions.includes('.mjs')) {
+      return match;
     }
+    return `${prefix}${extensions}, '.mjs'${suffix}`;
+  });
 
-    return hasChanges;
-  };
+  if (updated !== content) {
+    context.updateFile(configPath, updated);
+    additionsDebug("Added '.mjs' to resolve.extensions");
+  }
 }
 
 /**
- * Creates a modifier function for updateBundlerConfig that adds a module rule for .mjs files
- * in node_modules with resolve.fullySpecified: false
+ * Updates module rules to add .mjs rule using string manipulation
  */
-function createModuleRulesModifier(): ModuleRulesModifier {
-  return (moduleObject: recast.types.namedTypes.ObjectExpression): boolean => {
-    if (!moduleObject.properties) {
-      return false;
-    }
+function updateModuleRules(context: Context): void {
+  const configPath = context.doesFileExist(RSPACK_CONFIG_PATH)
+    ? RSPACK_CONFIG_PATH
+    : context.doesFileExist(WEBPACK_CONFIG_PATH)
+      ? WEBPACK_CONFIG_PATH
+      : null;
+
+  if (!configPath) {
+    return;
+  }
 
-    let hasChanges = false;
-    let hasMjsRule = false;
-    let rulesProperty: recast.types.namedTypes.Property | null = null;
+  const content = context.getFile(configPath);
+  if (!content) {
+    return;
+  }
 
-    // Find the rules property
-    for (const prop of moduleObject.properties) {
-      if (!prop || (prop.type !== 'Property' && prop.type !== 'ObjectProperty')) {
-        continue;
-      }
+  // Check if rule already exists
+  if (content.includes('test: /\\.mjs$') || content.includes('test: /\\\\.mjs$')) {
+    return;
+  }
 
-      const key = 'key' in prop ? prop.key : null;
-      const value = 'value' in prop ? prop.value : null;
-
-      if (key && key.type === 'Identifier' && key.name === 'rules' && value && value.type === 'ArrayExpression') {
-        rulesProperty = prop as recast.types.namedTypes.Property;
-        // Check if .mjs rule already exists
-        for (const element of value.elements) {
-          if (element && element.type === 'ObjectExpression' && element.properties) {
-            for (const ruleProp of element.properties) {
-              if (
-                ruleProp &&
-                (ruleProp.type === 'Property' || ruleProp.type === 'ObjectProperty') &&
-                'key' in ruleProp &&
-                ruleProp.key.type === 'Identifier' &&
-                ruleProp.key.name === 'test'
-              ) {
-                const testValue = 'value' in ruleProp ? ruleProp.value : null;
-                if (testValue) {
-                  // Check for RegExpLiteral with .mjs pattern
-                  if (testValue.type === 'RegExpLiteral' && 'pattern' in testValue && testValue.pattern === '\\.mjs$') {
-                    hasMjsRule = true;
-                    break;
-                  }
-                  // Check for Literal with regex property
-                  if (
-                    testValue.type === 'Literal' &&
-                    'regex' in testValue &&
-                    testValue.regex &&
-                    typeof testValue.regex === 'object' &&
-                    'pattern' in testValue.regex &&
-                    testValue.regex.pattern === '\\.mjs$'
-                  ) {
-                    hasMjsRule = true;
-                    break;
-                  }
-                  // Check for string literal containing .mjs
-                  if (
-                    testValue.type === 'Literal' &&
-                    'value' in testValue &&
-                    typeof testValue.value === 'string' &&
-                    testValue.value.includes('.mjs')
-                  ) {
-                    hasMjsRule = true;
-                    break;
-                  }
-                }
-              }
-            }
-          }
-          if (hasMjsRule) {
-            break;
-          }
-        }
-        break;
+  const mjsRule = `{
+        test: /\\.mjs$/,
+        include: /node_modules/,
+        resolve: {
+          fullySpecified: false,
+        },
+        type: 'javascript/auto',
+      },`;
+
+  // Simple approach: find rules array and insert after first rule
+  let updated = content;
+
+  // Case 1: Empty array - insert at start
+  if (content.match(/rules:\s*\[\s*\]/)) {
+    updated = content.replace(/(rules:\s*\[\s*)(\])/, `$1${mjsRule}\n      $2`);
+  }
+  // Case 2: Find first rule and insert after it
+  else {
+    // Match: rules: [ { ... }, and insert mjs rule after the first rule
+    // The regex finds the first complete rule object (balanced braces)
+    updated = content.replace(/(rules:\s*\[\s*)(\{[\s\S]*?\}),(\s*)/, (match, prefix, firstRule, suffix) => {
+      // Check if we already inserted (avoid double insertion)
+      if (match.includes('test: /\\.mjs$')) {
+        return match;
       }
-    }
-
-    // Add .mjs rule if missing (insert at position 1, after imports-loader rule which must be first)
-    if (!hasMjsRule && rulesProperty && 'value' in rulesProperty) {
-      const rulesArray = rulesProperty.value as recast.types.namedTypes.ArrayExpression;
-      const mjsRule = builders.objectExpression([
-        builders.property('init', builders.identifier('test'), builders.literal(/\.mjs$/)),
-        builders.property('init', builders.identifier('include'), builders.literal(/node_modules/)),
-        builders.property(
-          'init',
-          builders.identifier('resolve'),
-          builders.objectExpression([
-            builders.property('init', builders.identifier('fullySpecified'), builders.literal(false)),
-          ])
-        ),
-        builders.property('init', builders.identifier('type'), builders.literal('javascript/auto')),
-      ]);
-      // Insert at position 1 (second position) to keep imports-loader first
-      rulesArray.elements.splice(1, 0, mjsRule);
-      hasChanges = true;
-      additionsDebug('Added module rule for .mjs files in node_modules with resolve.fullySpecified: false');
-    }
+      // Insert mjs rule after first rule
+      return `${prefix}${firstRule},\n      ${mjsRule}${suffix}`;
+    });
+  }
 
-    return hasChanges;
-  };
+  if (updated !== content) {
+    context.updateFile(configPath, updated);
+    additionsDebug('Added module rule for .mjs files in node_modules with resolve.fullySpecified: false');
+  }
 }
 
 /**

package/templates/common/.config/rspack/rspack.config.ts

@@ -21,7 +21,8 @@ import { externals } from '../bundler/externals';
 const { SubresourceIntegrityPlugin } = rspack.experiments;
 const pluginJson = getPluginJson();
 const cpVersion = getCPConfigVersion();
-
+const logoPaths = Array.from(new Set([pluginJson.info?.logos?.large, pluginJson.info?.logos?.small])).filter(Boolean);
+const screenshotPaths = pluginJson.info?.screenshots?.map((s: { path: string }) => s.path) || [];
 const virtualPublicPath = new RspackVirtualModulePlugin({
   'grafana-public-path': `
 import amdMetaModule from 'amd-module';
@@ -161,14 +162,13 @@ const config = async (env): Promise<Configuration> => {
           { from: 'plugin.json', to: '.' },
           { from: '../LICENSE', to: '.' },
           { from: '../CHANGELOG.md', to: '.', force: true },
-          { from: '**/*.json', to: '.' }, // TODO<Add an error for checking the basic structure of the repo>
-          { from: '**/*.svg', to: '.', noErrorOnMissing: true }, // Optional
-          { from: '**/*.png', to: '.', noErrorOnMissing: true }, // Optional
-          { from: '**/*.html', to: '.', noErrorOnMissing: true }, // Optional
-          { from: 'img/**/*', to: '.', noErrorOnMissing: true }, // Optional
-          { from: 'libs/**/*', to: '.', noErrorOnMissing: true }, // Optional
-          { from: 'static/**/*', to: '.', noErrorOnMissing: true }, // Optional
-          { from: '**/query_help.md', to: '.', noErrorOnMissing: true }, // Optional
+          { from: '**/*.json', to: '.' },
+          { from: '**/query_help.md', to: '.', noErrorOnMissing: true },
+          ...logoPaths.map((logoPath) => ({ from: logoPath, to: logoPath })),
+          ...screenshotPaths.map((screenshotPath) => ({
+            from: screenshotPath,
+            to: screenshotPath,
+          })),
         ],
       }),
       // Replace certain template-variables in the README and plugin.json

package/templates/common/.config/webpack/webpack.config.ts

@@ -24,7 +24,8 @@ import { externals } from '../bundler/externals.ts';
 const pluginJson = getPluginJson();
 const cpVersion = getCPConfigVersion();
 const pluginVersion = getPackageJson().version;
-
+const logoPaths = Array.from(new Set([pluginJson.info?.logos?.large, pluginJson.info?.logos?.small])).filter(Boolean);
+const screenshotPaths = pluginJson.info?.screenshots?.map((s: { path: string }) => s.path) || [];
 const virtualPublicPath = new VirtualModulesPlugin({
   'node_modules/grafana-public-path.js': `
 import amdMetaModule from 'amd-module';
@@ -174,13 +175,12 @@ const config = async (env: Env): Promise<Configuration> => {
           { from: '../LICENSE', to: '.' },
           { from: '../CHANGELOG.md', to: '.', force: true },
           { from: '**/*.json', to: '.' },
-          { from: '**/*.svg', to: '.', noErrorOnMissing: true },
-          { from: '**/*.png', to: '.', noErrorOnMissing: true },
-          { from: '**/*.html', to: '.', noErrorOnMissing: true },
-          { from: 'img/**/*', to: '.', noErrorOnMissing: true },
-          { from: 'libs/**/*', to: '.', noErrorOnMissing: true },
-          { from: 'static/**/*', to: '.', noErrorOnMissing: true },
           { from: '**/query_help.md', to: '.', noErrorOnMissing: true },
+          ...logoPaths.map((logoPath) => ({ from: logoPath, to: logoPath })),
+          ...screenshotPaths.map((screenshotPath) => ({
+            from: screenshotPath,
+            to: screenshotPath,
+          })),
         ],
       }),
       // Replace certain template-variables in the README and plugin.json

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

@@ -1,102 +0,0 @@
-import * as recast from 'recast';
-import * as typeScriptParser from 'recast/parsers/typescript.js';
-import { additionsDebug } from './utils.js';
-
-const WEBPACK_CONFIG_PATH = ".config/webpack/webpack.config.ts";
-const RSPACK_CONFIG_PATH = ".config/rspack/rspack.config.ts";
-function updateBundlerConfig(context, resolveModifier, moduleRulesModifier) {
-  if (!resolveModifier && !moduleRulesModifier) {
-    return;
-  }
-  if (context.doesFileExist(RSPACK_CONFIG_PATH)) {
-    additionsDebug(`Found ${RSPACK_CONFIG_PATH}, updating bundler configuration...`);
-    const rspackContent = context.getFile(RSPACK_CONFIG_PATH);
-    if (rspackContent) {
-      try {
-        const ast = recast.parse(rspackContent, {
-          parser: typeScriptParser
-        });
-        let hasChanges = false;
-        recast.visit(ast, {
-          visitObjectExpression(path) {
-            const { node } = path;
-            const properties = node.properties;
-            if (properties) {
-              for (const prop of properties) {
-                if (prop && (prop.type === "Property" || prop.type === "ObjectProperty")) {
-                  const key = "key" in prop ? prop.key : null;
-                  const value = "value" in prop ? prop.value : null;
-                  if (resolveModifier && key && key.type === "Identifier" && key.name === "resolve" && value && value.type === "ObjectExpression") {
-                    hasChanges = resolveModifier(value) || hasChanges;
-                  }
-                  if (moduleRulesModifier && key && key.type === "Identifier" && key.name === "module" && value && value.type === "ObjectExpression") {
-                    hasChanges = moduleRulesModifier(value) || hasChanges;
-                  }
-                }
-              }
-            }
-            return this.traverse(path);
-          }
-        });
-        if (hasChanges) {
-          const output = recast.print(ast, {
-            tabWidth: 2,
-            trailingComma: true,
-            lineTerminator: "\n"
-          });
-          context.updateFile(RSPACK_CONFIG_PATH, output.code);
-          additionsDebug(`Updated ${RSPACK_CONFIG_PATH}`);
-        }
-      } catch (error) {
-        additionsDebug(`Error updating ${RSPACK_CONFIG_PATH}:`, error);
-      }
-    }
-    return;
-  }
-  if (context.doesFileExist(WEBPACK_CONFIG_PATH)) {
-    additionsDebug(`Found ${WEBPACK_CONFIG_PATH}, updating bundler configuration...`);
-    const webpackContent = context.getFile(WEBPACK_CONFIG_PATH);
-    if (webpackContent) {
-      try {
-        const ast = recast.parse(webpackContent, {
-          parser: typeScriptParser
-        });
-        let hasChanges = false;
-        recast.visit(ast, {
-          visitObjectExpression(path) {
-            const { node } = path;
-            const properties = node.properties;
-            if (properties) {
-              for (const prop of properties) {
-                if (prop && (prop.type === "Property" || prop.type === "ObjectProperty")) {
-                  const key = "key" in prop ? prop.key : null;
-                  const value = "value" in prop ? prop.value : null;
-                  if (resolveModifier && key && key.type === "Identifier" && key.name === "resolve" && value && value.type === "ObjectExpression") {
-                    hasChanges = resolveModifier(value) || hasChanges;
-                  }
-                  if (moduleRulesModifier && key && key.type === "Identifier" && key.name === "module" && value && value.type === "ObjectExpression") {
-                    hasChanges = moduleRulesModifier(value) || hasChanges;
-                  }
-                }
-              }
-            }
-            return this.traverse(path);
-          }
-        });
-        if (hasChanges) {
-          const output = recast.print(ast, {
-            tabWidth: 2,
-            trailingComma: true,
-            lineTerminator: "\n"
-          });
-          context.updateFile(WEBPACK_CONFIG_PATH, output.code);
-          additionsDebug(`Updated ${WEBPACK_CONFIG_PATH}`);
-        }
-      } catch (error) {
-        additionsDebug(`Error updating ${WEBPACK_CONFIG_PATH}:`, error);
-      }
-    }
-  }
-}
-
-export { updateBundlerConfig };