feat: Install Node.js in version-specific directories with fallback support

flow-plugins/flow-dev-bundle-plugin/src/main/java/com/vaadin/flow/plugin/maven/BuildDevBundleMojo.java

@@ -134,14 +134,6 @@ public class BuildDevBundleMojo extends AbstractMojo
     @Parameter(property = InitParameters.NODE_VERSION, defaultValue = FrontendTools.DEFAULT_NODE_VERSION)
     private String nodeVersion;
 
-    /**
-     * Setting defining if the automatically installed node version may be
-     * updated to the default Vaadin node version.
-     */
-    @Parameter(property = InitParameters.NODE_AUTO_UPDATE, defaultValue = ""
-            + Constants.DEFAULT_NODE_AUTO_UPDATE)
-    private boolean nodeAutoUpdate;
-
     @Parameter(defaultValue = "${project}", readonly = true, required = true)
     MavenProject project;
 
@@ -412,11 +404,6 @@ public URI nodeDownloadRoot() throws URISyntaxException {
         }
     }
 
-    @Override
-    public boolean nodeAutoUpdate() {
-        return nodeAutoUpdate;
-    }
-
     @Override
     public String nodeVersion() {
         return nodeVersion;

flow-plugins/flow-gradle-plugin/src/functionalTest/kotlin/com/vaadin/gradle/GradleVersionSupportTest.kt

@@ -61,7 +61,7 @@ class GradleVersionSupportTest(private val versionUnderTest: GradleVersion) : Ab
                 implementation("org.slf4j:slf4j-simple:$slf4jVersion")
             }
             vaadin {
-                nodeAutoUpdate = true // test the vaadin{} block by changing some innocent property with limited side-effect
+                eagerServerLoad = false // test the vaadin{} block by changing some innocent property with limited side-effect
             }
         """
         )

flow-plugins/flow-gradle-plugin/src/functionalTest/kotlin/com/vaadin/gradle/MiscMultiModuleTest.kt

@@ -60,7 +60,7 @@ class MiscMultiModuleTest : AbstractGradleTest() {
                 }
 
                 vaadin {
-                    nodeAutoUpdate = true // test the vaadin{} block by changing some innocent property with limited side-effect
+                    eagerServerLoad = false // test the vaadin{} block by changing some innocent property with limited side-effect
                 }
             }
         """.trimIndent())
@@ -104,7 +104,7 @@ class MiscMultiModuleTest : AbstractGradleTest() {
                 }
 
                 vaadin {
-                    nodeAutoUpdate = true // test the vaadin{} block by changing some innocent property with limited side-effect
+                    eagerServerLoad = false // test the vaadin{} block by changing some innocent property with limited side-effect
                 }
             }
         """.trimIndent())
@@ -161,7 +161,7 @@ class MiscMultiModuleTest : AbstractGradleTest() {
             }
 
             vaadin {
-                nodeAutoUpdate = true // test the vaadin{} block by changing some innocent property with limited side-effect
+                eagerServerLoad = false // test the vaadin{} block by changing some innocent property with limited side-effect
             }
         """.trimIndent())
 
@@ -212,7 +212,7 @@ class MiscMultiModuleTest : AbstractGradleTest() {
             }
 
             vaadin {
-                nodeAutoUpdate = true // test the vaadin{} block by changing some innocent property with limited side-effect
+                eagerServerLoad = false // test the vaadin{} block by changing some innocent property with limited side-effect
                 applicationIdentifier = 'MY_APP_ID'
             }
         """.trimIndent())
@@ -266,7 +266,7 @@ class MiscMultiModuleTest : AbstractGradleTest() {
             }
 
             vaadin {
-                nodeAutoUpdate = true // test the vaadin{} block by changing some innocent property with limited side-effect
+                eagerServerLoad = false // test the vaadin{} block by changing some innocent property with limited side-effect
                 applicationIdentifier = 'MY_APP_ID'
             }
         """.trimIndent())

flow-plugins/flow-gradle-plugin/src/functionalTest/kotlin/com/vaadin/gradle/MiscSingleModuleTest.kt

@@ -194,7 +194,7 @@ class MiscSingleModuleTest : AbstractGradleTest() {
             }
             def jettyVersion = "11.0.12"
             vaadin {
-                nodeAutoUpdate = true // test the vaadin{} block by changing some innocent property with limited side-effect
+                eagerServerLoad = false // test the vaadin{} block by changing some innocent property with limited side-effect
             }
             dependencies {
                 implementation("com.vaadin:flow:$flowVersion")

flow-plugins/flow-gradle-plugin/src/functionalTest/kotlin/com/vaadin/gradle/VaadinSmokeTest.kt

@@ -57,7 +57,7 @@ class VaadinSmokeTest : AbstractGradleTest() {
                 implementation("org.slf4j:slf4j-simple:$slf4jVersion")
             }
             vaadin {
-                nodeAutoUpdate = true // test the vaadin{} block by changing some innocent property with limited side-effect
+                eagerServerLoad = false // test the vaadin{} block by changing some innocent property with limited side-effect
             }
         """)
     }
@@ -285,7 +285,7 @@ class VaadinSmokeTest : AbstractGradleTest() {
                 implementation("org.slf4j:slf4j-simple:$slf4jVersion")
             }
             vaadin {
-                nodeAutoUpdate = true // test the vaadin{} block by changing some innocent property with limited side-effect
+                eagerServerLoad = false // test the vaadin{} block by changing some innocent property with limited side-effect
             }
         """)
         testProject.newFolder("libs")
@@ -487,7 +487,7 @@ class VaadinSmokeTest : AbstractGradleTest() {
         assertContains(result.output, "Configuration cache entry stored")
 
         val buildFile = testProject.buildFile.readText()
-            .replace("nodeAutoUpdate = true", "nodeAutoUpdate = false")
+            .replace("eagerServerLoad = false", "eagerServerLoad = true")
         testProject.buildFile.writeText(buildFile)
 
         val result2 = testProject.build("--configuration-cache", "vaadinPrepareFrontend", checkTasksSuccessful = false)

flow-plugins/flow-gradle-plugin/src/main/kotlin/com/vaadin/gradle/GradlePluginAdapter.kt

@@ -199,8 +199,6 @@ internal class GradlePluginAdapter private constructor(
     override fun nodeDownloadRoot(): URI =
         URI.create(config.nodeDownloadRoot.get())
 
-    override fun nodeAutoUpdate(): Boolean = config.nodeAutoUpdate.get()
-
     override fun nodeVersion(): String = config.nodeVersion.get()
 
     override fun npmFolder(): File = config.npmFolder.get()

flow-plugins/flow-gradle-plugin/src/main/kotlin/com/vaadin/gradle/PrepareFrontendInputProperties.kt

@@ -125,9 +125,6 @@ internal class PrepareFrontendInputProperties(
     @Input
     fun getNodeDownloadRoot(): Provider<String> = config.nodeDownloadRoot
 
-    @Input
-    fun getNodeAutoUpdate(): Provider<Boolean> = config.nodeAutoUpdate
-
     @Input
     fun getProjectBuildDir(): Provider<String> = config.projectBuildDir
 

flow-plugins/flow-gradle-plugin/src/main/kotlin/com/vaadin/gradle/VaadinFlowPluginExtension.kt

@@ -189,11 +189,6 @@ public abstract class VaadinFlowPluginExtension @Inject constructor(private val
      */
     public abstract val nodeDownloadRoot: Property<String>
 
-    /**
-     * Allow automatic update of node installed to alternate location. Default `false`
-     */
-    public abstract val nodeAutoUpdate: Property<Boolean>
-
     /**
      * Defines the output directory for generated non-served resources, such as
      * the token file. Defaults to `build/vaadin-generated` folder.
@@ -502,9 +497,6 @@ public class PluginEffectiveConfiguration(
     public val nodeDownloadRoot: Property<String> = extension.nodeDownloadRoot
         .convention(Platform.guess().nodeDownloadRoot)
 
-    public val nodeAutoUpdate: Property<Boolean> = extension.nodeAutoUpdate
-        .convention(false)
-
     public val resourceOutputDirectory: Property<File> =
         extension.resourceOutputDirectory
             .convention(
@@ -671,7 +663,6 @@ public class PluginEffectiveConfiguration(
             "generatedTsFolder=${generatedTsFolder.get()}, " +
             "nodeVersion=${nodeVersion.get()}, " +
             "nodeDownloadRoot=${nodeDownloadRoot.get()}, " +
-            "nodeAutoUpdate=${nodeAutoUpdate.get()}, " +
             "resourceOutputDirectory=${resourceOutputDirectory.get()}, " +
             "projectBuildDir=${projectBuildDir.get()}, " +
             "postinstallPackages=${postinstallPackages.get()}, " +

flow-plugins/flow-maven-plugin/src/main/java/com/vaadin/flow/plugin/maven/FlowModeAbstractMojo.java

@@ -133,14 +133,6 @@ public abstract class FlowModeAbstractMojo extends AbstractMojo
     @Parameter(property = InitParameters.NODE_VERSION, defaultValue = FrontendTools.DEFAULT_NODE_VERSION)
     private String nodeVersion;
 
-    /**
-     * Setting defining if the automatically installed node version may be
-     * updated to the default Vaadin node version.
-     */
-    @Parameter(property = InitParameters.NODE_AUTO_UPDATE, defaultValue = ""
-            + Constants.DEFAULT_NODE_AUTO_UPDATE)
-    private boolean nodeAutoUpdate;
-
     /**
      * The folder where `package.json` file is located. Default is project root
      * dir.
@@ -586,11 +578,6 @@ public URI nodeDownloadRoot() throws URISyntaxException {
         }
     }
 
-    @Override
-    public boolean nodeAutoUpdate() {
-        return nodeAutoUpdate;
-    }
-
     @Override
     public String nodeVersion() {
 

flow-plugins/flow-maven-plugin/src/main/java/com/vaadin/flow/plugin/maven/GenerateNpmBOMMojo.java

@@ -173,7 +173,6 @@ protected void executeInternal()
                         .withFrontendGeneratedFolder(generatedTsFolder())
                         .withNodeVersion(nodeVersion())
                         .withNodeDownloadRoot(nodeDownloadRoot())
-                        .setNodeAutoUpdate(nodeAutoUpdate())
                         .withHomeNodeExecRequired(requireHomeNodeExec())
                         .setJavaResourceFolder(javaResourceFolder())
                         .withProductionMode(productionMode)

flow-plugins/flow-plugin-base/src/main/java/com/vaadin/flow/plugin/base/BuildFrontendUtil.java

@@ -146,7 +146,6 @@ public static void prepareFrontend(PluginAdapterBase adapter)
 
         FrontendToolsSettings settings = getFrontendToolsSettings(adapter);
         FrontendTools tools = new FrontendTools(settings);
-        tools.validateNodeAndNpmVersion();
 
         ClassFinder classFinder = adapter.getClassFinder();
         Lookup lookup = adapter.createLookup(classFinder);
@@ -166,7 +165,6 @@ public static void prepareFrontend(PluginAdapterBase adapter)
                         getGeneratedFrontendDirectory(adapter))
                 .withNodeVersion(adapter.nodeVersion())
                 .withNodeDownloadRoot(nodeDownloadRootURI)
-                .setNodeAutoUpdate(adapter.nodeAutoUpdate())
                 .withHomeNodeExecRequired(adapter.requireHomeNodeExec())
                 .setJavaResourceFolder(adapter.javaResourceFolder())
                 .withProductionMode(false).withReact(adapter.isReactEnabled())
@@ -210,7 +208,6 @@ private static FrontendToolsSettings getFrontendToolsSettings(
                 () -> FrontendUtils.getVaadinHomeDirectory().getAbsolutePath());
         settings.setNodeDownloadRoot(adapter.nodeDownloadRoot());
         settings.setNodeVersion(adapter.nodeVersion());
-        settings.setAutoUpdate(adapter.nodeAutoUpdate());
         settings.setUseGlobalPnpm(adapter.useGlobalPnpm());
         settings.setForceAlternativeNode(adapter.requireHomeNodeExec());
         settings.setIgnoreVersionChecks(
@@ -361,7 +358,6 @@ public static void runNodeUpdater(PluginAdapterBuild adapter,
                     .withHomeNodeExecRequired(adapter.requireHomeNodeExec())
                     .withNodeVersion(adapter.nodeVersion())
                     .withNodeDownloadRoot(nodeDownloadRootURI)
-                    .setNodeAutoUpdate(adapter.nodeAutoUpdate())
                     .setJavaResourceFolder(adapter.javaResourceFolder())
                     .withPostinstallPackages(adapter.postinstallPackages())
                     .withCiBuild(adapter.ciBuild())
@@ -434,7 +430,6 @@ public static void runDevBuildNodeUpdater(PluginAdapterBuild adapter)
                     .withHomeNodeExecRequired(adapter.requireHomeNodeExec())
                     .withNodeVersion(adapter.nodeVersion())
                     .withNodeDownloadRoot(nodeDownloadRootURI)
-                    .setNodeAutoUpdate(adapter.nodeAutoUpdate())
                     .setJavaResourceFolder(adapter.javaResourceFolder())
                     .withPostinstallPackages(adapter.postinstallPackages())
                     .withBundleBuild(true)
@@ -507,7 +502,6 @@ public static void runFrontendBuild(PluginAdapterBase adapter)
 
         FrontendToolsSettings settings = getFrontendToolsSettings(adapter);
         FrontendTools tools = new FrontendTools(settings);
-        tools.validateNodeAndNpmVersion();
         BuildFrontendUtil.runVite(adapter, tools);
         String tokenContent = "";
         File tokenFile = getTokenFile(adapter);
@@ -570,12 +564,7 @@ private static void runFrontendBuildTool(PluginAdapterBase adapter,
                     toolName, buildExecutable.getAbsolutePath()));
         }
 
-        String nodePath;
-        if (adapter.requireHomeNodeExec()) {
-            nodePath = frontendTools.forceAlternativeNodeExecutable();
-        } else {
-            nodePath = frontendTools.getNodeExecutable();
-        }
+        String nodePath = frontendTools.getNodeExecutable();
 
         List<String> command = new ArrayList<>();
         command.add(nodePath);

flow-plugins/flow-plugin-base/src/main/java/com/vaadin/flow/plugin/base/ConvertPolymerCommand.java

@@ -196,7 +196,6 @@ private FrontendToolsSettings getFrontendToolsSettings()
                 () -> FrontendUtils.getVaadinHomeDirectory().getAbsolutePath());
         settings.setNodeDownloadRoot(adapter.nodeDownloadRoot());
         settings.setNodeVersion(adapter.nodeVersion());
-        settings.setAutoUpdate(adapter.nodeAutoUpdate());
         settings.setUseGlobalPnpm(adapter.useGlobalPnpm());
         settings.setForceAlternativeNode(adapter.requireHomeNodeExec());
         settings.setIgnoreVersionChecks(

flow-plugins/flow-plugin-base/src/main/java/com/vaadin/flow/plugin/base/PluginAdapterBase.java

@@ -193,13 +193,6 @@ default Lookup createLookup(ClassFinder classFinder) {
      */
     URI nodeDownloadRoot() throws URISyntaxException;
 
-    /**
-     * Whether the alternative node may be auto-updated or not.
-     *
-     * @return {@code true} to update node if older than default
-     */
-    boolean nodeAutoUpdate();
-
     /**
      * The node.js version to be used when node.js is installed automatically by
      * Vaadin, for example `"v12.18.3"`. Defaults to null which uses the

flow-server/src/main/java/com/vaadin/flow/server/Constants.java

@@ -139,11 +139,6 @@ public final class Constants implements Serializable {
      */
     public static final boolean GLOBAL_PNPM_DEFAULT = false;
 
-    /**
-     * The default value for {@link InitParameters#NODE_AUTO_UPDATE}.
-     */
-    public static final boolean DEFAULT_NODE_AUTO_UPDATE = true;
-
     /**
      * The default value for
      * {@link InitParameters#REQUIRE_HOME_NODE_EXECUTABLE}.

flow-server/src/main/java/com/vaadin/flow/server/InitParameters.java

@@ -198,13 +198,6 @@ public class InitParameters implements Serializable {
      */
     public static final String REQUIRE_HOME_NODE_EXECUTABLE = "require.home.node";
 
-    /**
-     * Configuration parameter name for requiring node executable installed in
-     * home directory.
-     *
-     */
-    public static final String NODE_AUTO_UPDATE = "node.auto.update";
-
     /**
      * Configuration name for the parameter that sets the compiled web
      * components path. The path should be the same as