forest_admin_rails 1.12.0 → 1.12.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 12744bc9381af2c455a8ab1dd56e68443bc813d6f5df92f2b0f56a6fce4d3fa8
-  data.tar.gz: d080d3ccbb57fecc113f6c0b41d871f115fc80e06dcb87a4fc6e0ee99fa1afc3
+  metadata.gz: 3f8e76e19bc58b11c12dd6d31f125fbe0543d6f33ff4463d9454e68ec43fdb4f
+  data.tar.gz: 130779408d7ecf0879ec80cc1cf1fef5be06b7f48848e7e0dcb9ca741ee43783
 SHA512:
-  metadata.gz: 903af7b0abb45535e2e5e56bf358f8d3103f4518f48a0ff79b1bad9157bcd3ae19f5f323ff8a0530905deab1864f52034e2fc23403236fc612d4915e60328b13
-  data.tar.gz: 34302839a5df138d04551410c2031764f1fee00be5fda460ae2a3d3cee9c089da6303e6cd5d220665175b405a17f5c3404d5317499607cbc9b0e94830007b94a
+  metadata.gz: 6116cedc55a13e68579903a099466eff89536c679f46c87dcbe787c74010ad3268983f5188d33adac1ebce34e4ad177cd0cd6195faa94e20e4aefef25dba18d3
+  data.tar.gz: 484024339f54c3a375ef3d65ac8d2ca9f5c1154be65491e0fa33eab698876d8e2319717bb1a3ece1850f6a2b5251524c77dd2ce340f0386b3495faf702529fe0

data/app/controllers/forest_admin_rails/forest_controller.rb

@@ -7,8 +7,8 @@ module ForestAdminRails
     skip_forgery_protection
 
     def index
-      if ForestAdminAgent::Http::Router.routes.key? params['route_alias']
-        route = ForestAdminAgent::Http::Router.routes[params['route_alias']]
+      if ForestAdminAgent::Http::Router.cached_routes.key? params['route_alias']
+        route = ForestAdminAgent::Http::Router.cached_routes[params['route_alias']]
 
         begin
           forest_response route[:closure].call({ params: params.to_unsafe_h, headers: request.headers.to_h })

data/config/initializers/forest_admin_route_cache.rb

@@ -0,0 +1,29 @@
+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
+      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/config/routes.rb

@@ -1,15 +1,20 @@
 ForestAdminRails::Engine.routes.draw do
-  Rails.error.handle(ForestAdminDatasourceToolkit::Exceptions::ForestException) do
-    scope '/forest' do
-      ForestAdminAgent::Http::Router.routes.each do |name, agent_route|
-        match agent_route[:uri],
-              defaults: { format: agent_route[:format] },
-              to: 'forest#index',
-              via: agent_route[:method],
-              as: name,
-              route_alias: name,
-              constraints: { id: /[a-zA-Z0-9\.]*+/ }
-      end
+  scope '/forest' do
+    # Use cached_routes to avoid recomputing routes during Rails route initialization
+    ForestAdminAgent::Http::Router.cached_routes.each do |name, agent_route|
+      match agent_route[:uri],
+            defaults: { format: agent_route[:format] },
+            to: 'forest#index',
+            via: agent_route[:method],
+            as: name,
+            route_alias: name,
+            constraints: { id: /[a-zA-Z0-9\.]*+/ }
     end
+  rescue StandardError => e
+    if defined?(Rails) && Rails.logger
+      Rails.logger.error("[ForestAdmin] CRITICAL: Failed to initialize routes: #{e.class} - #{e.message}")
+      Rails.logger.error(e.backtrace.join("\n"))
+    end
+    raise e
   end
 end

data/lib/forest_admin_rails/engine.rb

@@ -24,6 +24,10 @@ module ForestAdminRails
 
     extend ActiveSupport::Concern
 
+    initializer 'forest_admin_rails.add_autoload_paths', before: :set_autoload_paths do |app|
+      app.config.autoload_paths << Rails.root.join('lib')
+    end
+
     initializer 'forest_admin_rails.error_subscribe' do
       Rails.error.subscribe(ForestAdminErrorSubscriber.new)
     end
@@ -39,7 +43,28 @@ module ForestAdminRails
 
     def load_configuration
       return unless running_web_server?
-      return unless File.exist?(Rails.root.join('app', 'lib', 'forest_admin_rails', 'create_agent.rb'))
+
+      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!
+
+          The file 'app/lib/forest_admin_rails/create_agent.rb' should now be located at:
+          'lib/forest_admin_rails/create_agent.rb'
+
+          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
+
+          This will become a hard requirement in a future version.
+        WARNING
+      end
+
+      return unless File.exist?(new_path) || File.exist?(old_path)
 
       # force eager loading models
       Rails.application.eager_load!

data/lib/forest_admin_rails/version.rb

@@ -1,3 +1,3 @@
 module ForestAdminRails
-  VERSION = "1.12.0"
+  VERSION = "1.12.2"
 end

data/lib/generators/forest_admin_rails/install_generator.rb

@@ -8,7 +8,7 @@ module ForestAdminRails
         @auth_secret = SecureRandom.hex(20)
         @env_secret = env_secret
         template 'initializers/config.rb', 'config/initializers/forest_admin_rails.rb'
-        template 'create_agent.rb', 'app/lib/forest_admin_rails/create_agent.rb'
+        template 'create_agent.rb', 'lib/forest_admin_rails/create_agent.rb'
         route 'mount ForestAdminRails::Engine => "#{ForestAdminRails.config[:prefix]}"'
       end
     end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: forest_admin_rails
 version: !ruby/object:Gem::Version
-  version: 1.12.0
+  version: 1.12.2
 platform: ruby
 authors:
 - Matthieu
@@ -68,6 +68,7 @@ files:
 - app/controllers/forest_admin_rails/forest_controller.rb
 - app/helpers/forest_admin_rails/application_helper.rb
 - config/initializers/forest_admin_error_subscriber.rb
+- config/initializers/forest_admin_route_cache.rb
 - config/routes.rb
 - lib/forest_admin_rails.rb
 - lib/forest_admin_rails/engine.rb