forest_admin_rails 1.12.11 → 1.12.12

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 14239daa21f3456795d8a5f86e379f8290f024a99fea0a9743a4ed9184b577b6
-  data.tar.gz: 475e7e2ef5c27294e433d8b09ea38c77c29dea40d277bb2785322889095fd94b
+  metadata.gz: bdd36d163cb4d1210f6680782fd4338aca9d3c2273fbcfab9ee90cfec91af2b9
+  data.tar.gz: 1444cac70cd8990f48df0a80579bf0da629898a9f71a2f4082a8bdd4b19a66d3
 SHA512:
-  metadata.gz: 2ba32705f89e4e93e0995c987a389edbcaf2d7efd166ac31589cdc73bdef3189b72828cfba742c86b23807536a803a434d07176cdf768302e0cdea8c4ca832f9
-  data.tar.gz: e8babcc8148642abd65660b8eca6e83c073d8cf34fb1501f11f99bce839ff77435f43413fb1695f50324a31ecc3ba6d271f73c2299172e64cbca397aa74f3889
+  metadata.gz: 84dfe389f748592cc51544ad223f7203da823074bfb5e577c4d49dc96ae2ed3810cc7bc53628a8c0a8a2516e1a0f4557fbe90fda3b504f2710ae5709329e2b96
+  data.tar.gz: 74757f2ff05b7c69ba45ef14aa51a60d970a940cc5a033c868305b6195e21054d5b8c37cf399b4406311aeb4c5faa4b49636ef828ccea2a31d28401bfaaa7cc8

data/config/initializers/forest_admin_route_cache.rb

@@ -1,31 +1,11 @@
+# Route pre-caching has been moved to engine.rb after agent.build() completes
+#
+# This initializer is kept for informational purposes only
 if defined?(ForestAdminAgent::Http::Router)
   cache_disabled = ForestAdminAgent::Http::Router.cache_disabled?
 
   if cache_disabled
     Rails.logger.warn('[ForestAdmin] Route caching is DISABLED - this will impact performance')
     Rails.logger.warn('[ForestAdmin] To enable caching, remove disable_route_cache: true from configuration')
-  else
-    Rails.application.config.after_initialize do
-      next if defined?(Rake) && Rake.respond_to?(:application) && Rake.application&.top_level_tasks&.any?
-
-      start_time = Time.now
-      routes = ForestAdminAgent::Http::Router.cached_routes
-      elapsed = ((Time.now - start_time) * 1000).round(2)
-
-      Rails.logger.info(
-        "[ForestAdmin] Successfully pre-cached #{routes.size} routes in #{elapsed}ms"
-      )
-    rescue StandardError => e
-      Rails.logger.error(
-        "[ForestAdmin] CRITICAL: Failed to pre-cache routes: #{e.class} - #{e.message}"
-      )
-      Rails.logger.error(e.backtrace.first(10).join("\n"))
-
-      raise e if Rails.env.production?
-
-      Rails.logger.warn(
-        '[ForestAdmin] Routes will be computed on first request (performance degradation)'
-      )
-    end
   end
 end

data/lib/forest_admin_rails/engine.rb

@@ -44,41 +44,64 @@ module ForestAdminRails
     def load_configuration
       return unless running_web_server?
 
+      check_create_agent_location
+      return unless create_agent_file_exists?
+
+      # force eager loading models
+      Rails.application.eager_load!
+
+      setup_agent_and_cache_routes
+
+      sse = ForestAdminAgent::Services::SSECacheInvalidation
+      sse.run if ForestAdminRails.config[:instant_cache_refresh]
+    end
+
+    def check_create_agent_location
       old_path = Rails.root.join('app', 'lib', 'forest_admin_rails', 'create_agent.rb')
       new_path = Rails.root.join('lib', 'forest_admin_rails', 'create_agent.rb')
 
-      # Check for old location and warn user
-      if File.exist?(old_path) && !File.exist?(new_path)
-        logger = ActiveSupport::Logger.new($stdout)
-        logger.warn <<~WARNING
-          ⚠️  DEPRECATION WARNING: create_agent.rb detected in old location!
+      return unless File.exist?(old_path) && !File.exist?(new_path)
 
-          The file 'app/lib/forest_admin_rails/create_agent.rb' should now be located at:
-          'lib/forest_admin_rails/create_agent.rb'
+      logger = ActiveSupport::Logger.new($stdout)
+      logger.warn <<~WARNING
+        ⚠️  DEPRECATION WARNING: create_agent.rb detected in old location!
 
-          Please move your file to the new location:
-            mkdir -p lib/forest_admin_rails
-            mv app/lib/forest_admin_rails/create_agent.rb lib/forest_admin_rails/create_agent.rb
+        The file 'app/lib/forest_admin_rails/create_agent.rb' should now be located at:
+        'lib/forest_admin_rails/create_agent.rb'
 
-          This will become a hard requirement in a future version.
-        WARNING
-      end
+        Please move your file to the new location:
+          mkdir -p lib/forest_admin_rails
+          mv app/lib/forest_admin_rails/create_agent.rb lib/forest_admin_rails/create_agent.rb
 
-      return unless File.exist?(new_path) || File.exist?(old_path)
+        This will become a hard requirement in a future version.
+      WARNING
+    end
 
-      # force eager loading models
-      Rails.application.eager_load!
+    def create_agent_file_exists?
+      old_path = Rails.root.join('app', 'lib', 'forest_admin_rails', 'create_agent.rb')
+      new_path = Rails.root.join('lib', 'forest_admin_rails', 'create_agent.rb')
 
-      begin
-        ForestAdminRails::CreateAgent.setup!
-      rescue StandardError => e
-        logger = ActiveSupport::Logger.new($stdout)
-        logger.warn 'An error has occurred during setup of the Forest Admin agent.'
-        raise e.message
-      end
+      File.exist?(new_path) || File.exist?(old_path)
+    end
 
-      sse = ForestAdminAgent::Services::SSECacheInvalidation
-      sse.run if ForestAdminRails.config[:instant_cache_refresh]
+    def setup_agent_and_cache_routes
+      ForestAdminRails::CreateAgent.setup!
+      precache_routes
+    rescue StandardError => e
+      logger = ActiveSupport::Logger.new($stdout)
+      logger.warn 'An error has occurred during setup of the Forest Admin agent.'
+      raise e.message
+    end
+
+    def precache_routes
+      return if ForestAdminAgent::Http::Router.cache_disabled?
+
+      start_time = Time.now
+      routes = ForestAdminAgent::Http::Router.cached_routes
+      elapsed = ((Time.now - start_time) * 1000).round(2)
+
+      logger = ActiveSupport::Logger.new($stdout)
+      logger.info("[ForestAdmin] Successfully pre-cached #{routes.size} routes in #{elapsed}ms")
     end
 
     def running_web_server?

data/lib/forest_admin_rails/version.rb

@@ -1,3 +1,3 @@
 module ForestAdminRails
-  VERSION = "1.12.11"
+  VERSION = "1.12.12"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: forest_admin_rails
 version: !ruby/object:Gem::Version
-  version: 1.12.11
+  version: 1.12.12
 platform: ruby
 authors:
 - Matthieu