vite_ruby 3.9.1 → 4.0.0.alpha1

data/lib/vite_ruby/commands.rb

@@ -9,7 +9,7 @@ class ViteRuby::Commands
 
   # Public: Defaults to production, and exits if the build fails.
   def build_from_task(*args)
-    with_node_env(ENV.fetch("NODE_ENV", "production")) {
+    with_node_env(ENV.fetch('NODE_ENV', 'production')) {
       ensure_log_goes_to_stdout {
         build(*args) || exit!
       }
@@ -23,30 +23,52 @@ class ViteRuby::Commands
 
   # Public: Removes all build cache and previously compiled assets.
   def clobber
-    dirs = [config.build_output_dir, config.ssr_output_dir, config.build_cache_dir, config.vite_cache_dir]
+    dirs = [config.build_output_dir, config.build_cache_dir, config.vite_cache_dir]
     dirs.each { |dir| dir.rmtree if dir.exist? }
-    $stdout.puts "Removed vite cache and output dirs:\n\t#{dirs.join("\n\t")}"
+    $stdout.puts "Removed vite cache and output dirs:\n\t#{ dirs.join("\n\t") }"
   end
 
-  # Internal: Installs the binstub for the CLI in the appropriate path.
-  def install_binstubs
-    `bundle binstub vite_ruby --path #{config.root.join("bin")}`
-    `bundle config --delete bin`
+  # Public: Receives arguments from a rake task.
+  def clean_from_task(args)
+    ensure_log_goes_to_stdout {
+      clean(keep_up_to: Integer(args.keep || 2), age_in_seconds: Integer(args.age || 3600))
+    }
   end
 
-  # Internal: Checks if the npm version is 6 or lower.
-  def legacy_npm_version?
-    `npm --version`.to_i < 7 rescue false
+  # Public: Cleanup old assets in the output directory.
+  #
+  # keep_up_to - Max amount of backups to preserve.
+  # age_in_seconds - Amount of time to look back in order to preserve them.
+  #
+  # NOTE: By default keeps the last version, or 2 if created in the past hour.
+  #
+  # Examples:
+  #   To force only 1 backup to be kept: clean(1, 0)
+  #   To only keep files created within the last 10 minutes: clean(0, 600)
+  def clean(keep_up_to: 2, age_in_seconds: 3600)
+    return false unless may_clean?
+
+    versions
+      .each_with_index
+      .drop_while { |(mtime, _), index|
+        max_age = [0, Time.now - Time.at(mtime)].max
+        max_age < age_in_seconds || index < keep_up_to
+      }
+      .each do |(_, files), _|
+        clean_files(files)
+      end
+    true
   end
 
-  # Internal: Checks if the yarn version is 1.x.
-  def legacy_yarn_version?
-    `yarn --version`.to_i < 2 rescue false
+  # Internal: Installs the binstub for the CLI in the appropriate path.
+  def install_binstubs
+    `bundle binstub vite_ruby --path #{ config.root.join('bin') }`
+    `bundle config --delete bin`
   end
 
   # Internal: Verifies if ViteRuby is properly installed.
   def verify_install
-    unless File.exist?(config.root.join("bin/vite"))
+    unless File.exist?(config.root.join('bin/vite'))
       warn <<~WARN
 
         vite binstub not found.
@@ -59,7 +81,7 @@ class ViteRuby::Commands
     unless config_path.exist?
       warn <<~WARN
 
-        Configuration #{config_path} file for vite-plugin-ruby not found.
+        Configuration #{ config_path } file for vite-plugin-ruby not found.
         Make sure `bundle exec vite install` has run successfully before running dependent tasks.
       WARN
       exit!
@@ -68,24 +90,24 @@ class ViteRuby::Commands
 
   # Internal: Prints information about ViteRuby's environment.
   def print_info
-    config.within_root do
-      $stdout.puts "bin/vite present?: #{File.exist? "bin/vite"}"
+    Dir.chdir(config.root) do
+      $stdout.puts "bin/vite present?: #{ File.exist? 'bin/vite' }"
 
-      $stdout.puts "vite_ruby: #{ViteRuby::VERSION}"
+      $stdout.puts "vite_ruby: #{ ViteRuby::VERSION }"
       ViteRuby.framework_libraries.each do |framework, library|
-        $stdout.puts "#{library.name}: #{library.version}"
-        $stdout.puts "#{framework}: #{Gem.loaded_specs[framework]&.version}"
+        $stdout.puts "#{ library.name }: #{ library.version }"
+        $stdout.puts "#{ framework }: #{ Gem.loaded_specs[framework]&.version }"
       end
 
-      $stdout.puts "ruby: #{`ruby --version`}"
-      $stdout.puts "node: #{`node --version`}"
-
-      pkg = config.package_manager
-      $stdout.puts "#{pkg}: #{`#{pkg} --version` rescue nil}"
+      $stdout.puts "node: #{ `node --version` }"
+      $stdout.puts "npm: #{ `npm --version` }"
+      $stdout.puts "yarn: #{ `yarn --version` rescue nil }"
+      $stdout.puts "pnpm: #{ `pnpm --version` rescue nil }"
+      $stdout.puts "ruby: #{ `ruby --version` }"
 
       $stdout.puts "\n"
       packages = `npm ls vite vite-plugin-ruby`
-      packages_msg = packages.include?("vite@") ? "installed packages:\n#{packages}" : "❌ Check that vite and vite-plugin-ruby have been added as development dependencies and installed."
+      packages_msg = packages.include?('vite@') ? "installed packages:\n#{ packages }" : '❌ Check that vite and vite-plugin-ruby have been added as development dependencies and installed.'
       $stdout.puts packages_msg
 
       ViteRuby::CompatibilityCheck.verify_plugin_version(config.root)
@@ -98,19 +120,42 @@ private
 
   def_delegators :@vite_ruby, :config, :builder, :manifest, :logger, :logger=
 
+  def may_clean?
+    config.build_output_dir.exist? && config.manifest_path.exist?
+  end
+
+  def clean_files(files)
+    files.select { |file| File.file?(file) }.each do |file|
+      File.delete(file)
+      logger.info("Removed #{ file }")
+    end
+  end
+
+  def versions
+    all_files = Dir.glob("#{ config.build_output_dir }/**/*")
+    entries = all_files - [config.manifest_path] - current_version_files
+    entries.reject { |file| File.directory?(file) }
+      .group_by { |file| File.mtime(file).utc.to_i }
+      .sort.reverse
+  end
+
+  def current_version_files
+    Dir.glob(manifest.refresh.values.map { |value| config.build_output_dir.join("#{ value['file'] }*") })
+  end
+
   def with_node_env(env)
-    original = ENV["NODE_ENV"]
-    ENV["NODE_ENV"] = env
+    original = ENV['NODE_ENV']
+    ENV['NODE_ENV'] = env
     yield
   ensure
-    ENV["NODE_ENV"] = original
+    ENV['NODE_ENV'] = original
   end
 
   def ensure_log_goes_to_stdout
     old_logger, original_sync = logger, $stdout.sync
 
     $stdout.sync = true
-    self.logger = Logger.new($stdout, formatter: proc { |_, _, progname, msg| (progname == "vite") ? msg : "#{msg}\n" })
+    self.logger = Logger.new($stdout, formatter: proc { |_, _, progname, msg| progname == 'vite' ? msg : "#{ msg }\n" })
     yield
   ensure
     self.logger, $stdout.sync = old_logger, original_sync

data/lib/vite_ruby/compatibility_check.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require "json"
+require 'json'
 
 # Internal: Verifies that the installed vite-plugin-ruby version is compatible
 # with the current version of vite_ruby.
@@ -11,9 +11,9 @@ module ViteRuby::CompatibilityCheck
   class << self
     # Public: Attempt to verify that the vite-plugin-ruby version is compatible.
     def verify_plugin_version(root)
-      package = JSON.parse(root.join("package.json").read) rescue {}
-      requirement = package.dig("devDependencies", "vite-plugin-ruby") ||
-        package.dig("dependencies", "vite-plugin-ruby")
+      package = JSON.parse(root.join('package.json').read) rescue {}
+      requirement = package.dig('devDependencies', 'vite-plugin-ruby') ||
+                    package.dig('dependencies', 'vite-plugin-ruby')
 
       raise_unless_satisfied(requirement, ViteRuby::DEFAULT_PLUGIN_VERSION)
     end
@@ -22,9 +22,9 @@ module ViteRuby::CompatibilityCheck
     def raise_unless_satisfied(npm_req, ruby_req)
       unless compatible_plugin?(npm_req, ruby_req)
         raise ArgumentError, <<~ERROR
-          vite-plugin-ruby@#{npm_req} might not be compatible with vite_ruby-#{ViteRuby::VERSION}
+          vite-plugin-ruby@#{ npm_req } might not be compatible with vite_ruby-#{ ViteRuby::VERSION }
 
-          You may disable this check if needed: https://vite-ruby.netlify.app/config/#skipcompatibilitycheck
+          You may disable this check if needed: https://vite-ruby.netlify.app/config/#skipCompatibilityCheck
 
           You may upgrade both by running:
 
@@ -37,12 +37,12 @@ module ViteRuby::CompatibilityCheck
     # requirement.
     def compatible_plugin?(npm_req, ruby_req)
       npm_req, ruby_req = [npm_req, ruby_req]
-        .map { |req| Gem::Requirement.new(req.sub("^", "~>")) }
+        .map { |req| Gem::Requirement.new(req.sub('^', '~>')) }
 
       current_version = npm_req.requirements.first.second
 
       ruby_req.satisfied_by?(current_version)
-    rescue
+    rescue StandardError
       true
     end
   end

data/lib/vite_ruby/config.rb

@@ -1,36 +1,26 @@
 # frozen_string_literal: true
 
-require "json"
+require 'json'
 
 # Public: Allows to resolve configuration sourced from `config/vite.json` and
 # environment variables, combining them with the default options.
 class ViteRuby::Config
-  def origin
-    "#{protocol}://#{host_with_port}"
-  end
-
   def protocol
-    https ? "https" : "http"
+    https ? 'https' : 'http'
   end
 
   def host_with_port
-    "#{host}:#{port}"
+    "#{ host }:#{ port }"
   end
 
-  # Internal: Path to the manifest files generated by Vite and vite-plugin-ruby.
-  def known_manifest_paths
-    [
-      # NOTE: Generated by Vite when `manifest: true`, which vite-plugin-ruby enables.
-      build_output_dir.join(".vite/manifest.json"),
-
-      # NOTE: Path where vite-plugin-ruby outputs the assets manifest file.
-      build_output_dir.join(".vite/manifest-assets.json"),
-    ]
+  # Internal: Path where Vite outputs the manifest file.
+  def manifest_path
+    build_output_dir.join('manifest.json')
   end
 
-  # Internal: Path to the manifest files generated by Vite and vite-plugin-ruby.
-  def manifest_paths
-    known_manifest_paths.select(&:exist?)
+  # Internal: Path where vite-plugin-ruby outputs the assets manifest file.
+  def assets_manifest_path
+    build_output_dir.join('manifest-assets.json')
   end
 
   # Public: The directory where Vite will store the built assets.
@@ -45,7 +35,7 @@ class ViteRuby::Config
 
   # Internal: The directory where Vite stores its processing cache.
   def vite_cache_dir
-    root.join("node_modules/.vite")
+    root.join('node_modules/.vite')
   end
 
   # Public: The directory that Vite uses as root.
@@ -55,63 +45,46 @@ class ViteRuby::Config
 
   # Public: Loads an optional config/vite.rb file that can modify ViteRuby.env
   def load_ruby_config
-    rb_config_path = File.expand_path(config_path.sub(/.json$/, ".rb"), root)
+    rb_config_path = File.expand_path(config_path.sub(/.json$/, '.rb'), root)
     load rb_config_path if File.exist?(rb_config_path)
   end
 
   # Public: Sets additional environment variables for vite-plugin-ruby.
-  def to_env(env_vars = ViteRuby.env)
+  def to_env
     CONFIGURABLE_WITH_ENV.each_with_object({}) do |option, env|
       unless (value = @config[option]).nil?
-        env["#{ViteRuby::ENV_PREFIX}_#{option.upcase}"] = value.to_s
+        env["#{ ViteRuby::ENV_PREFIX }_#{ option.upcase }"] = value.to_s
       end
-    end.merge(env_vars)
+    end.merge(ViteRuby.env)
   end
 
   # Internal: Files and directories that should be watched for changes.
   def watched_paths
     [
       *(watch_additional_paths + additional_entrypoints).reject { |dir|
-        dir.start_with?("~/") || dir.start_with?(source_code_dir)
+        dir.start_with?('~/') || dir.start_with?(source_code_dir)
       },
-      "#{source_code_dir}/**/*",
-      config_path.sub(/.json$/, ".{rb,json}"),
+      "#{ source_code_dir }/**/*",
+      config_path.sub(/.json$/, '.{rb,json}'),
       *DEFAULT_WATCHED_PATHS,
     ].freeze
   end
 
-  # Internal: Changes the current directory to the root dir.
-  def within_root(&block)
-    Dir.chdir(File.expand_path(root), &block)
-  end
-
 private
 
   # Internal: Coerces all the configuration values, in case they were passed
   # as environment variables which are always strings.
   def coerce_values(config)
-    config["mode"] = config["mode"].to_s
-    config["port"] = config["port"].to_i
-    config["root"] = root = Pathname.new(config["root"])
-    config["build_cache_dir"] = root.join(config["build_cache_dir"])
-    config["ssr_output_dir"] = root.join(config["ssr_output_dir"])
-    coerce_booleans(config, "auto_build", "hide_build_console_output", "https", "skip_compatibility_check", "skip_proxy")
-    config["package_manager"] ||= detect_package_manager(root)
+    config['mode'] = config['mode'].to_s
+    config['port'] = config['port'].to_i
+    config['root'] = Pathname.new(config['root'])
+    config['build_cache_dir'] = config['root'].join(config['build_cache_dir'])
+    coerce_booleans(config, 'auto_build', 'hide_build_console_output', 'https', 'skip_compatibility_check')
   end
 
   # Internal: Coerces configuration options to boolean.
   def coerce_booleans(config, *names)
-    truthy = [true, "true"]
-    names.each { |name| config[name] = truthy.include?(config[name]) }
-  end
-
-  def detect_package_manager(root)
-    return "npm" if root.join("package-lock.json").exist?
-    return "pnpm" if root.join("pnpm-lock.yaml").exist?
-    return "bun" if root.join("bun.lockb").exist?
-    return "yarn" if root.join("yarn.lock").exist?
-
-    "npm"
+    names.each { |name| config[name] = [true, 'true'].include?(config[name]) }
   end
 
   def initialize(attrs)
@@ -125,8 +98,8 @@ private
     # Public: Returns the project configuration for Vite.
     def resolve_config(**attrs)
       config = config_defaults.merge(attrs.transform_keys(&:to_s))
-      file_path = File.join(config["root"], config["config_path"])
-      file_config = config_from_file(file_path, mode: config["mode"])
+      file_path = File.join(config['root'], config['config_path'])
+      file_config = config_from_file(file_path, mode: config['mode'])
       new DEFAULT_CONFIG.merge(file_config).merge(config_from_env).merge(config)
     end
 
@@ -134,21 +107,21 @@ private
 
     # Internal: Converts camelCase to snake_case.
     SNAKE_CASE = ->(camel_cased_word) {
-      camel_cased_word.to_s.gsub("::", "/")
+      camel_cased_word.to_s.gsub(/::/, '/')
         .gsub(/([A-Z]+)([A-Z][a-z])/, '\1_\2')
         .gsub(/([a-z\d])([A-Z])/, '\1_\2')
-        .tr("-", "_")
+        .tr('-', '_')
         .downcase
     }
 
     # Internal: Default values for a Ruby application.
-    def config_defaults(asset_host: nil, mode: ENV.fetch("RACK_ENV", "development"), root: Dir.pwd)
+    def config_defaults(asset_host: nil, mode: ENV.fetch('RACK_ENV', 'development'), root: Dir.pwd)
       {
-        "asset_host" => option_from_env("asset_host") || asset_host,
-        "config_path" => option_from_env("config_path") || DEFAULT_CONFIG.fetch("config_path"),
-        "mode" => option_from_env("mode") || mode,
-        "root" => option_from_env("root") || root,
-      }.select { |_, value| value }
+        'asset_host' => option_from_env('asset_host') || asset_host,
+        'config_path' => option_from_env('config_path') || DEFAULT_CONFIG.fetch('config_path'),
+        'mode' => option_from_env('mode') || mode,
+        'root' => option_from_env('root') || root,
+      }
     end
 
     # Internal: Used to load a JSON file from the specified path.
@@ -162,7 +135,7 @@ private
 
     # Internal: Retrieves a configuration option from environment variables.
     def option_from_env(name)
-      ViteRuby.env["#{ViteRuby::ENV_PREFIX}_#{name.upcase}"]
+      ViteRuby.env["#{ ViteRuby::ENV_PREFIX }_#{ name.upcase }"]
     end
 
     # Internal: Extracts the configuration options provided as env vars.
@@ -177,16 +150,16 @@ private
     # Internal: Loads the configuration options provided in a JSON file.
     def config_from_file(path, mode:)
       multi_env_config = load_json(path)
-      multi_env_config.fetch("all", {})
+      multi_env_config.fetch('all', {})
         .merge(multi_env_config.fetch(mode, {}))
     rescue Errno::ENOENT => error
-      $stderr << "Check that your vite.json configuration file is available in the load path:\n\n\t#{error.message}\n\n"
+      $stderr << "Check that your vite.json configuration file is available in the load path:\n\n\t#{ error.message }\n\n"
       {}
     end
   end
 
   # Internal: Shared configuration with the Vite plugin for Ruby.
-  DEFAULT_CONFIG = load_json("#{__dir__}/../../default.vite.json").freeze
+  DEFAULT_CONFIG = load_json("#{ __dir__ }/../../default.vite.json").freeze
 
   # Internal: Configuration options that can not be provided as env vars.
   NOT_CONFIGURABLE_WITH_ENV = %w[additional_entrypoints watch_additional_paths].freeze
@@ -196,15 +169,12 @@ private
 
   # Internal: If any of these files is modified the build won't be skipped.
   DEFAULT_WATCHED_PATHS = %w[
-    bun.lockb
     package-lock.json
     package.json
     pnpm-lock.yaml
     postcss.config.js
     tailwind.config.js
     vite.config.js
-    vite.config.mjs
-    vite.config.mts
     vite.config.ts
     windi.config.ts
     yarn.lock

data/lib/vite_ruby/dev_server_proxy.rb

@@ -1,15 +1,15 @@
 # frozen_string_literal: true
 
-require "rack/proxy"
+require 'rack/proxy'
 
 # Public: Allows to relay asset requests to the Vite development server.
 class ViteRuby::DevServerProxy < Rack::Proxy
   HOST_WITH_PORT_REGEX = %r{^(.+?)(:\d+)/}
-  VITE_DEPENDENCY_PREFIX = "/@"
+  VITE_DEPENDENCY_PREFIX = '/@'
 
   def initialize(app = nil, options = {})
     @vite_ruby = options.delete(:vite_ruby) || ViteRuby.instance
-    options[:streaming] = false if config.mode == "test" && !options.key?(:streaming)
+    options[:streaming] = false if config.mode == 'test' && !options.key?(:streaming)
     super
   end
 
@@ -17,7 +17,7 @@ class ViteRuby::DevServerProxy < Rack::Proxy
   def perform_request(env)
     if vite_should_handle?(env) && dev_server_running?
       forward_to_vite_dev_server(env)
-      super
+      super(env)
     else
       @app.call(env)
     end
@@ -30,47 +30,41 @@ private
   def_delegators :@vite_ruby, :config, :dev_server_running?
 
   def rewrite_uri_for_vite(env)
-    uri = env.fetch("REQUEST_URI") { [env["PATH_INFO"], env["QUERY_STRING"]].reject { |str| str.to_s.strip.empty? }.join("?") }
-    env["PATH_INFO"], env["QUERY_STRING"] = (env["REQUEST_URI"] = normalize_uri(uri)).split("?")
+    uri = env.fetch('REQUEST_URI') { [env['PATH_INFO'], env['QUERY_STRING']].reject { |str| str.to_s.strip.empty? }.join('?') }
+    env['PATH_INFO'], env['QUERY_STRING'] = (env['REQUEST_URI'] = normalize_uri(uri)).split('?')
   end
 
   def normalize_uri(uri)
     uri
-      .sub(HOST_WITH_PORT_REGEX, "/") # Hanami adds the host and port.
-      .sub(".ts.js", ".ts") # Hanami's javascript helper always adds the extension.
-      .sub(/\.(sass|scss|styl|stylus|less|pcss|postcss)\.css$/, '.\1') # Rails' stylesheet_link_tag always adds the extension.
+      .sub(HOST_WITH_PORT_REGEX, '/') # Hanami adds the host and port.
+      .sub('.ts.js', '.ts') # Hanami's javascript helper always adds the extension.
+      .sub(/(\.(?!min|module)\w+)\.css$/, '\1') # Rails' stylesheet_link_tag always adds the extension.
   end
 
   def forward_to_vite_dev_server(env)
     rewrite_uri_for_vite(env)
-    env["QUERY_STRING"] ||= ""
-    env["HTTP_HOST"] = env["HTTP_X_FORWARDED_HOST"] = config.host
-    env["HTTP_X_FORWARDED_SERVER"] = config.host_with_port
-    env["HTTP_PORT"] = env["HTTP_X_FORWARDED_PORT"] = config.port.to_s
-    env["HTTP_X_FORWARDED_PROTO"] = env["HTTP_X_FORWARDED_SCHEME"] = config.protocol
-    env["HTTPS"] = env["HTTP_X_FORWARDED_SSL"] = "off" unless config.https
-    env["SCRIPT_NAME"] = ""
+    env['QUERY_STRING'] ||= ''
+    env['HTTP_HOST'] = env['HTTP_X_FORWARDED_HOST'] = config.host
+    env['HTTP_X_FORWARDED_SERVER'] = config.host_with_port
+    env['HTTP_PORT'] = env['HTTP_X_FORWARDED_PORT'] = config.port.to_s
+    env['HTTP_X_FORWARDED_PROTO'] = env['HTTP_X_FORWARDED_SCHEME'] = config.protocol
+    env['HTTPS'] = env['HTTP_X_FORWARDED_SSL'] = 'off' unless config.https
+    env['SCRIPT_NAME'] = ''
   end
 
   def vite_should_handle?(env)
-    path = normalize_uri(env["PATH_INFO"])
-    return true if path.start_with?(vite_url_prefix) # Vite asset
-
-    true if file_in_vite_root?(path) # Fallback if Vite can serve the file
+    path = normalize_uri(env['PATH_INFO'])
+    return true if path.start_with?(vite_asset_url_prefix) # Vite asset
+    return true if path.start_with?(VITE_DEPENDENCY_PREFIX) # Packages and imports
+    return true if file_in_vite_root?(path) # Fallback if Vite can serve the file
   end
 
-  # NOTE: When using an empty 'public_output_dir', we need to rely on a
-  # filesystem check to check whether Vite should serve the request.
   def file_in_vite_root?(path)
-    path.include?(".") && # Check for extension, avoid filesystem if possible.
-      config.vite_root_dir.join(path.start_with?("/") ? path[1..] : path).file?
+    path.include?('.') && # Check for extension, avoid filesystem if possible.
+      config.vite_root_dir.join(path.start_with?('/') ? path[1..-1] : path).file?
   end
 
-  # NOTE: Vite is configured to use 'public_output_dir' as the base, which can
-  # be customized by the user in development to not match any of the routes.
-  #
-  # If the path starts with that prefix, it will be redirected to Vite.
-  def vite_url_prefix
-    @vite_url_prefix ||= config.public_output_dir.empty? ? VITE_DEPENDENCY_PREFIX : "/#{config.public_output_dir}/"
+  def vite_asset_url_prefix
+    @vite_asset_url_prefix ||= config.public_output_dir.empty? ? "\0" : "/#{ config.public_output_dir }/"
   end
 end

data/lib/vite_ruby/error.rb

@@ -3,7 +3,7 @@
 # Internal: Provides common functionality for errors.
 class ViteRuby::Error < StandardError
   def message
-    super.sub(":troubleshooting:", <<~MSG)
+    super.sub(':troubleshooting:', <<~MSG)
       Visit the Troubleshooting guide for more information:
         https://vite-ruby.netlify.app/guide/troubleshooting.html#troubleshooting
     MSG

data/lib/vite_ruby/io.rb

@@ -1,13 +1,13 @@
 # frozen_string_literal: true
 
-require "open3"
+require 'open3'
 
 # Public: Builds on top of Ruby I/O open3 providing a friendlier experience.
 module ViteRuby::IO
   class << self
     # Internal: A modified version of capture3 that can continuosly print stdout.
     # NOTE: Streaming output provides a better UX when running bin/vite build.
-    def capture(*cmd, with_output: $stdout.method(:puts), stdin_data: "", **opts)
+    def capture(*cmd, with_output: $stdout.method(:puts), stdin_data: '', **opts)
       return Open3.capture3(*cmd, **opts) unless with_output
 
       Open3.popen3(*cmd, **opts) { |stdin, stdout, stderr, wait_threads|
@@ -15,13 +15,13 @@ module ViteRuby::IO
         stdin.close
         out = Thread.new { read_lines(stdout, &with_output) }
         err = Thread.new { stderr.read }
-        [out.value, err.value.to_s, wait_threads.value]
+        [out.value, err.value, wait_threads.value]
       }
     end
 
     # Internal: Reads and yield every line in the stream. Returns the full content.
     def read_lines(io)
-      buffer = +""
+      buffer = +''
       while line = io.gets
         buffer << line
         yield line