fix: copy all frontend resources from JAR files (#22264) (#22270)

Frontend resources in JAR files are filtered to only copy a hard-coded
set of file types like JS, TS, CSS, etc.
Since the resources are collected only from specific folders, the filter
is most likely unecessary and also makes it impossible to use fronted frameworks
that rely on custom extensions (e.g. Vue).
This change removes the file extension filter, allowing all files in META-INF/frontend
and META-INF/resources/frontend to be copied in the frontend generated folder.

Fixes #22208

Co-authored-by: Marco Collovati <marco@vaadin.com>

Author: vaadin-bot

flow-server/src/main/java/com/vaadin/flow/server/frontend/TaskCopyFrontendFiles.java

@@ -32,21 +32,21 @@
 import static com.vaadin.flow.server.Constants.RESOURCES_JAR_DEFAULT;
 
 /**
- * Copies JavaScript and CSS files from JAR files into a given folder.
+ * Copies all frontend resources from JAR files into a given folder.
+ * <p>
+ * The task considers "frontend resources" all files placed in
+ * {@literal META-INF/frontend}, {@literal META-INF/resources/frontend} and
+ * {@literal META-INF/resources/[**]/themes} folders.
  * <p>
  * For internal use only. May be renamed or removed in a future release.
  *
  * @since 2.0
  */
 public class TaskCopyFrontendFiles
         extends AbstractFileGeneratorFallibleCommand {
-    private static final String[] WILDCARD_INCLUSIONS = new String[] {
-            "**/*.js", "**/*.js.map", "**/*.css", "**/*.css.map", "**/*.ts",
-            "**/*.ts.map", "**/*.tsx", "**/*.tsx.map", "**/*.jsx",
-            "**/*.jsx.map" };
     private static final String WILDCARD_INCLUSION_APP_THEME_JAR = "**/themes/**/*";
     private final Options options;
-    private Set<File> resourceLocations = null;
+    private final Set<File> resourceLocations;
 
     /**
      * Scans the jar files given defined by {@code resourcesToScan}.
@@ -97,11 +97,11 @@ public void execute() {
                 handledFiles.addAll(jarContentsManager
                         .copyIncludedFilesFromJarTrimmingBasePath(location,
                                 RESOURCES_FRONTEND_DEFAULT, targetDirectory,
-                                WILDCARD_INCLUSIONS));
+                                "**/*"));
                 handledFiles.addAll(jarContentsManager
                         .copyIncludedFilesFromJarTrimmingBasePath(location,
                                 COMPATIBILITY_RESOURCES_FRONTEND_DEFAULT,
-                                targetDirectory, WILDCARD_INCLUSIONS));
+                                targetDirectory, "**/*"));
                 handledFiles.addAll(jarContentsManager
                         .copyIncludedFilesFromJarTrimmingBasePath(location,
                                 RESOURCES_JAR_DEFAULT, targetDirectory,

flow-server/src/test/java/com/vaadin/flow/server/frontend/FrontendResourcesAreCopiedAfterCleaningTest.java

@@ -57,13 +57,13 @@ public void setup() throws IOException, ExecutionFailedException {
     public void frontendResources_should_beCopiedFromJars_when_TaskUpdatePackagesRemovesThem()
             throws IOException, ExecutionFailedException {
         copyResources();
-        assertCopiedFrontendFileAmount(10);
+        assertCopiedFrontendFileAmount(17);
 
         performPackageClean();
         assertCopiedFrontendFileAmount(0);
 
         copyResources();
-        assertCopiedFrontendFileAmount(10);
+        assertCopiedFrontendFileAmount(17);
     }
 
     private void assertCopiedFrontendFileAmount(int fileCount)

flow-server/src/test/java/com/vaadin/flow/server/frontend/TaskCopyFrontendFilesTest.java

@@ -118,8 +118,9 @@ private void should_collectJsAndCssFilesFromJars(String jarFile,
         task.execute();
 
         List<String> files = TestUtils.listFilesRecursively(frontendDepsFolder);
-        Assert.assertEquals(12, files.size());
+        Assert.assertEquals(19, files.size());
 
+        // Check some resources
         Assert.assertTrue("TS resource should have been copied from jar file",
                 files.contains("example.ts"));
 
@@ -163,6 +164,13 @@ private void should_collectJsAndCssFilesFromJars(String jarFile,
                 "JSX resource source map should have been copied from jar file",
                 files.contains("test.jsx.map"));
 
+        Assert.assertTrue("HTML resource should have been copied from jar file",
+                files.contains("ExampleTemplate.html"));
+
+        Assert.assertFalse(
+                "Resource from unsupported frontend folder location should not have been copied from jar file",
+                files.contains("ignored.js"));
+
         Assert.assertEquals("Generated files should have been tracked",
                 files.stream()
                         .map(path -> frontendDepsFolder.toPath().resolve(path))