fix(breaking): check for any existing manifest path before cleaning

Author: ElMassimo

Gemfile.lock

@@ -97,7 +97,7 @@ GEM
       tzinfo (~> 2.0)
     ansi (1.5.0)
     ast (2.4.2)
-    base64 (0.1.1)
+    base64 (0.2.0)
     benchmark-ips (2.12.0)
     bigdecimal (3.1.4)
     builder (3.2.4)
@@ -108,7 +108,7 @@ GEM
     crass (1.0.6)
     date (3.3.3)
     docile (1.4.0)
-    drb (2.1.1)
+    drb (2.2.0)
       ruby2_keywords
     dry-cli (1.0.0)
     erubi (1.12.0)
@@ -117,11 +117,11 @@ GEM
     i18n (1.14.1)
       concurrent-ruby (~> 1.0)
     io-console (0.6.0)
-    irb (1.8.1)
+    irb (1.9.0)
       rdoc
       reline (>= 0.3.8)
     json (2.6.3)
-    loofah (2.21.3)
+    loofah (2.22.0)
       crass (~> 1.0.2)
       nokogiri (>= 1.12.0)
     m (1.6.2)
@@ -135,15 +135,15 @@ GEM
     marcel (1.0.2)
     method_source (1.0.0)
     mini_mime (1.1.5)
-    mini_portile2 (2.8.4)
+    mini_portile2 (2.8.5)
     minitest (5.20.0)
     minitest-reporters (1.6.1)
       ansi
       builder
       minitest (>= 5.0)
       ruby-progressbar
     minitest-stub_any_instance (1.0.3)
-    mutex_m (0.1.2)
+    mutex_m (0.2.0)
     net-imap (0.4.0)
       date
       net-protocol
@@ -167,9 +167,9 @@ GEM
     pry-byebug (3.10.1)
       byebug (~> 11.0)
       pry (>= 0.13, < 0.15)
-    psych (5.1.0)
+    psych (5.1.1.1)
       stringio
-    racc (1.7.1)
+    racc (1.7.3)
     rack (3.0.8)
     rack-proxy (0.7.7)
       rack
@@ -210,11 +210,11 @@ GEM
       thor (~> 1.0, >= 1.2.2)
       zeitwerk (~> 2.6)
     rainbow (3.1.1)
-    rake (13.0.6)
-    rdoc (6.5.0)
+    rake (13.1.0)
+    rdoc (6.6.0)
       psych (>= 4.0.0)
     regexp_parser (2.8.1)
-    reline (0.3.9)
+    reline (0.4.0)
       io-console (~> 0.5)
     rexml (3.2.6)
     rubocop (1.24.1)
@@ -241,8 +241,8 @@ GEM
       simplecov-html (~> 0.10.0)
     simplecov-html (0.10.2)
     spring (2.1.1)
-    stringio (3.0.8)
-    thor (1.2.2)
+    stringio (3.0.9)
+    thor (1.3.0)
     timeout (0.4.0)
     tzinfo (2.0.6)
       concurrent-ruby (~> 1.0)

test/config_test.rb

@@ -61,7 +61,7 @@ def test_ruby_config_file
   end
 
   def test_manifest_path
-    assert_path 'test_app/public/vite-production/manifest.json', @config.manifest_path
+    assert_path 'test_app/public/vite-production/.vite/manifest.json', @config.manifest_paths.first
   end
 
   def test_build_cache_dir

test/manifest_test.rb

@@ -52,20 +52,22 @@ def test_lookup_exception!
     stub_builder(build_successful: true) {
       asset_file = 'calendar.js'
       error = assert_raises_manifest_missing_entry_error { path_for(asset_file) }
-      assert_match "Vite Ruby can't find entrypoints/#{ asset_file } in #{ manifest_path }", error.message
+      assert_match "Vite Ruby can't find entrypoints/#{ asset_file } in the manifests", error.message
       assert_match '"autoBuild" is set to `false`', error.message
 
       asset_file = 'images/logo.gif'
       error = assert_raises_manifest_missing_entry_error { path_for(asset_file) }
-      assert_match "Vite Ruby can't find #{ asset_file } in #{ manifest_path }", error.message
+      assert_match "Vite Ruby can't find #{ asset_file } in the manifests", error.message
 
       asset_file = '/app/styles/theme.css'
       error = assert_raises_manifest_missing_entry_error { path_for(asset_file) }
-      assert_match "Vite Ruby can't find ../styles/theme.css in #{ manifest_path }", error.message
+      assert_match "Vite Ruby can't find ../styles/theme.css in the manifests", error.message
 
       asset_file = '~/favicon.ico'
       error = assert_raises_manifest_missing_entry_error { path_for(asset_file) }
-      assert_match "Vite Ruby can't find favicon.ico in #{ manifest_path }", error.message
+      assert_match "Vite Ruby can't find favicon.ico in the manifests", error.message
+
+      assert_match "Manifest files found:\n  #{ manifest_path }", error.message
     }
   end
 
@@ -77,7 +79,7 @@ def test_lookup_exception_when_auto_build
         path_for(asset_file)
       end
 
-      assert_match "Vite Ruby can't find entrypoints/#{ asset_file } in #{ manifest_path }", error.message
+      assert_match "Vite Ruby can't find entrypoints/#{ asset_file } in the manifests", error.message
       assert_match 'The file path is incorrect.', error.message
     }
   end
@@ -94,7 +96,7 @@ def test_lookup_exception_when_build_failed
         path_for(asset_file)
       end
 
-      assert_match "Vite Ruby can't find entrypoints/#{ asset_file } in #{ manifest_path }", error.message
+      assert_match "Vite Ruby can't find entrypoints/#{ asset_file } in the manifests", error.message
       assert_match 'The last build failed.', error.message
       assert_match "  #{ error_lines[0] }", error.message
       assert_match "  #{ error_lines[1] }", error.message
@@ -108,7 +110,7 @@ def test_lookup_with_type_exception!
       path_for(asset_file, type: :javascript)
     end
 
-    assert_match "Vite Ruby can't find entrypoints/#{ asset_file }.js in #{ manifest_path }", error.message
+    assert_match "Vite Ruby can't find entrypoints/#{ asset_file }.js in the manifests", error.message
   end
 
   def test_lookup_success!
@@ -229,7 +231,7 @@ def assert_raises_manifest_missing_entry_error(auto_build: false, &block)
   end
 
   def manifest_path
-    'public/vite-production/manifest.json'
+    'public/vite-production/.vite/manifest.json'
   end
 
   def prefixed(file)

…lic/vite-production/manifest-assets.json …te-production/.vite/manifest-assets.jsontest/test_app/public/vite-production/manifest-assets.json renamed to test/test_app/public/vite-production/.vite/manifest-assets.json test/test_app/public/vite-production/manifest-assets.json renamed to test/test_app/public/vite-production/.vite/manifest-assets.json

@@ -131,7 +131,7 @@ def print_info
   def_delegators :@vite_ruby, :config, :builder, :manifest, :logger, :logger=
 
   def may_clean?
-    config.build_output_dir.exist? && config.manifest_path.exist?
+    config.build_output_dir.exist? && config.manifest_paths.any?
   end
 
   def clean_files(files)