Add directory containing npx to PATH when starting RPC

Author: knutwannheden

rewrite-core/src/main/java/org/openrewrite/rpc/RewriteRpc.java

@@ -475,6 +475,10 @@ protected <P> P send(String method, @Nullable RpcRequest body, Class<P> response
 
             // If we get here, we've hit the total timeout
             throw new RuntimeException("Request timed out after " + timeout.getSeconds() + " seconds");
+        } catch (RuntimeException e) {
+            // Check if process crashed during the request
+            checkLiveness();
+            throw e;
         } catch (ExecutionException | InterruptedException e) {
             // Check if process crashed during the request
             checkLiveness();

rewrite-javascript/src/main/java/org/openrewrite/javascript/rpc/JavaScriptRewriteRpc.java

@@ -275,6 +275,11 @@ public JavaScriptRewriteRpc get() {
                 }
             }
             process.environment().put("NODE_OPTIONS", nodeOptions.toString());
+            if (npxPath.getParent() != null) {
+                // `npx` is typically a shebang script alongside the `node` executable
+                process.environment().put("PATH", npxPath.getParent() + File.pathSeparator +
+                        System.getenv("PATH"));
+            }
             process.start();
 
             try {