Allow NPM use for install/build

rails · Dec 5, 2023 · e884fd6 · e884fd6
1 parent 95a5472
commit e884fd6
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/lib/tasks/jsbundling/build.rake b/lib/tasks/jsbundling/build.rake
@@ -24,13 +24,15 @@ module Jsbundling
 
     def install_command
       return "bun install" if File.exist?('bun.lockb') || (tool_exists?('bun') && !File.exist?('yarn.lock'))
-      return "yarn install" if File.exist?('yarn.lock') || tool_exists?('yarn')
+      return "yarn install" if File.exist?('yarn.lock') || (tool_exists?('yarn') && !File.exist?('package-lock.json'))
+      return "npm install" if File.exist?('package-lock.json') || tool_exists?('npm')
       raise "jsbundling-rails: No suitable tool found for installing JavaScript dependencies"
     end
 
     def build_command
       return "bun run build" if File.exist?('bun.lockb') || (tool_exists?('bun') && !File.exist?('yarn.lock'))
-      return "yarn build" if File.exist?('yarn.lock') || tool_exists?('yarn')
+      return "yarn build" if File.exist?('yarn.lock') || (tool_exists?('yarn') && !File.exist?('package-lock.json'))
+      return "npm run build" if File.exist?('package-lock.json') || tool_exists?('npm')
       raise "jsbundling-rails: No suitable tool found for building JavaScript"
     end