refactor reloader

brakeman.ignore

@@ -6,7 +6,7 @@
       "fingerprint": "1cb8570b8c91f38317cdf909e01e7016359846174f427e86011633c344d30fc3",
       "check_name": "ForgerySetting",
       "message": "`protect_from_forgery` should be called in `Plutonium::Resource::Controller`",
-      "file": "lib/plutonium/reactor/resource_controller.rb",
+      "file": "lib/plutonium/resource/controller.rb",
       "line": 10,
       "link": "https://brakemanscanner.org/docs/warning_types/cross-site_request_forgery/",
       "code": null,

lib/plutonium/railtie.rb

@@ -40,7 +40,8 @@ class Railtie < Rails::Railtie
     end
 
     config.after_initialize do
-      Plutonium::Reactor::Core.achieve_criticality!
+      Plutonium::Reloader.start! if Rails.application.config.plutonium.enable_hotreload
+      Plutonium::ZEITWERK_LOADER.eager_load if Rails.env.production?
     end
   end
 end

lib/plutonium/reloader.rb

@@ -0,0 +1,99 @@
+# Be sure to restart your server when you modify this file.
+# This is a glorified initializer
+
+module Plutonium
+  class Reloader
+    class << self
+      def start!
+        puts "=> [plutonium] starting reloader"
+
+        @listener&.stop
+        @listener = initialize_listener
+      end
+
+      private
+
+      def initialize_listener
+        require "listen"
+
+        reload_paths = gather_reload_paths
+        return unless reload_paths.any?
+
+        listener = Listen.to(*reload_paths, only: /\.rb$/) do |modified, added, removed|
+          handle_file_changes(modified, added, removed)
+        end
+        listener.start
+        listener
+      end
+
+      def gather_reload_paths
+        reload_paths = []
+
+        if Plutonium.development?
+          reload_paths << Plutonium.lib_root.to_s
+          reload_paths << Plutonium.root.join("app", "views", "components").to_s
+          reload_paths << Plutonium.root.join("config", "initializers").to_s
+        end
+
+        packages_dir = Rails.root.join("packages/").to_s
+        reload_paths << packages_dir if File.directory?(packages_dir)
+
+        reload_paths
+      end
+
+      def handle_file_changes(modified, added, removed)
+        (modified + added).each do |file|
+          Plutonium.logger.debug "[plutonium] change detected: #{file}"
+
+          if file == __FILE__
+            reload_file(file)
+            start!
+          elsif file_starts_with_packages_dir?(file)
+            handle_package_file_changes(file, added)
+          else
+            reload_framework_and_file(file)
+          end
+        rescue => e
+          log_reload_failure(file, e)
+        end
+      end
+
+      def file_starts_with_packages_dir?(file)
+        packages_dir = Rails.root.join("packages/").to_s
+        file.starts_with?(packages_dir)
+      end
+
+      def handle_package_file_changes(file, added)
+        return if added.include?(file)
+
+        case File.basename(file)
+        when "engine.rb"
+          reload_engine_and_routes(file)
+        else
+          # Non-engine package files are reloaded by Rails automatically
+        end
+      end
+
+      def reload_engine_and_routes(file)
+        Plutonium.logger.debug "[plutonium] reloading #{file}"
+        load file
+        Rails.application.reload_routes!
+      end
+
+      def reload_framework_and_file(file)
+        Plutonium.logger.debug "[plutonium] reloading framework"
+        Plutonium::ZEITWERK_LOADER.reload
+        load Plutonium.root.join("app", "views", "components", "base.rb")
+        load file # Ensure files that do not contain constants are loaded
+      end
+
+      def reload_file(file)
+        load file
+      end
+
+      def log_reload_failure(file, error)
+        Plutonium.logger.error "\n[plutonium] reload failed #{file}\n\n#{error}\n"
+      end
+    end
+  end
+end