fix: improve npm resolution for non-windows (#23810) (#23815)

vaadin-bot · mcollovati · web-flow · commit b021a69af6a3 · 2026-03-10T06:58:17.000Z
Adds another potential path to the npm resolver algorithm
to make it compatible with node installation performed
by frontend-maven-plugin

Co-authored-by: Marco Collovati &lt;marco@vaadin.com&gt;
diff --git a/flow-server/src/main/java/com/vaadin/flow/server/frontend/NodeResolver.java b/flow-server/src/main/java/com/vaadin/flow/server/frontend/NodeResolver.java
@@ -585,6 +585,7 @@ private String findNpmCliScript(File nodeFolder, String nodeVersion) {
         } else {
             searchPaths.add("lib/node_modules/npm/bin/npm-cli.js");
             searchPaths.add("../lib/node_modules/npm/bin/npm-cli.js");
+            searchPaths.add("node_modules/npm/bin/npm-cli.js");
         }
 
         for (String path : searchPaths) {

Original file line number	Diff line number	Diff line change
`@@ -585,6 +585,7 @@ private String findNpmCliScript(File nodeFolder, String nodeVersion) {`
`585`	`585`	`} else {`
`586`	`586`	`searchPaths.add("lib/node_modules/npm/bin/npm-cli.js");`
`587`	`587`	`searchPaths.add("../lib/node_modules/npm/bin/npm-cli.js");`
	`588`	`+ searchPaths.add("node_modules/npm/bin/npm-cli.js");`
`588`	`589`	`}`
`589`	`590`
`590`	`591`	`for (String path : searchPaths) {`