forest_admin_rpc_agent 1.13.4 → 1.14.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: d53c87e67caf6c3d8d25e0153e5e2d9d70c9c2729068a8fdf9b1ae87fc1d2a9c
-  data.tar.gz: 33e47c98fa34e2334d1e11a6bf24a4402d3d71ef7fee86339de5b5a201577c96
+  metadata.gz: 9fb58efd1964b8dbf5c8e9c10b6557297196df287759952fdeff7eef3949e836
+  data.tar.gz: 01f76fae8c2ee42d73cd2e454fe9bdaee2a16aea47cf37e38081af45931e884e
 SHA512:
-  metadata.gz: 9a8955692b0f1e9665c1449422d59ae40f3806c3be9e16f2a4de9a43a4a5e907ce1d0eafa0699fbdbf5b48cafb9b772cdbdfde12b45f1b9d0b95f819e50fb49c
-  data.tar.gz: 5bceed442425fdcbf317aab144eb58f05414d857183fe81a47339504701e1086c8c88938672854705952e5907e2a314b5a9377ddcd480dbdd371a3f03035c2a4
+  metadata.gz: 25c49b286bce40a650d73bf684ca6b50b6bd5dfec85d546f5460dfc64dd4babaf01fd67d45f05fea865ab69d94490e53f65626ebc186eb03f3f3c5815680b6de
+  data.tar.gz: 3507ba4f38bb3b4344f47958899d797e11a53e8aece6ab3e2cd45b2201630f84b3d7d82bdf4b427cbc87cdf22118dedb9b5bb01ff84d41021211afde3728b36b

data/config/routes.rb

@@ -1,14 +1,19 @@
 ForestAdminRpcAgent::Engine.routes.draw do
-  route_classes = ForestAdminRpcAgent::Routes.constants.reject { |route| route.to_s == 'BaseRoute' }
+  next if defined?(Rake) && Rake.respond_to?(:application) && Rake.application&.top_level_tasks&.any?
 
-  route_classes.each do |route|
-    route_class = ForestAdminRpcAgent::Routes.const_get(route)
-
-    route_instance = route_class.new
-    if route_instance.respond_to?(:registered)
+  begin
+    # Use cached_route_instances to avoid recomputing routes during Rails initialization
+    ForestAdminRpcAgent::Http::Router.cached_route_instances.each do |route_instance|
       route_instance.registered(self)
-    else
-      Rails.logger.warn "Skipping route: #{route_class} (does not respond to :registered)"
     end
+  rescue StandardError => e
+    error_message = "[ForestAdminRpcAgent] CRITICAL: Failed to initialize routes: #{e.class} - #{e.message}\n" \
+                    "#{e.backtrace.join("\n")}"
+    begin
+      ForestAdminRpcAgent::Facades::Container.logger.log('Error', error_message)
+    rescue StandardError
+      puts error_message
+    end
+    raise e
   end
 end

data/lib/forest_admin_rpc_agent/http/router.rb

@@ -0,0 +1,92 @@
+module ForestAdminRpcAgent
+  module Http
+    class Router
+      # Mutex for thread-safe cache operations
+      @mutex = Mutex.new
+
+      def self.cached_route_instances
+        return route_instances.freeze if cache_disabled?
+
+        return @cached_route_instances if @cached_route_instances
+
+        @mutex.synchronize do
+          @cached_route_instances ||= begin
+            start_time = Time.now
+            computed_routes = route_instances
+            elapsed = ((Time.now - start_time) * 1000).round(2)
+
+            log_message = "[ForestAdminRpcAgent] Computed #{computed_routes.size} routes " \
+                          "in #{elapsed}ms (caching enabled)"
+            log_to_available_logger('Info', log_message)
+
+            computed_routes.freeze
+          end
+        end
+      end
+
+      def self.cache_disabled?
+        config = ForestAdminRpcAgent::Facades::Container.config_from_cache
+        config&.dig(:disable_route_cache) == true
+      rescue StandardError
+        # Config not available, default to caching enabled
+        false
+      end
+
+      def self.reset_cached_route_instances!
+        @mutex.synchronize do
+          @cached_route_instances = nil
+        end
+      end
+
+      def self.log_to_available_logger(level, message)
+        ForestAdminRpcAgent::Facades::Container.logger.log(level, message)
+      rescue StandardError
+        puts message
+      end
+
+      def self.route_instances
+        route_classes = ForestAdminRpcAgent::Routes.constants.reject { |route| route.to_s == 'BaseRoute' }
+
+        route_instances = []
+
+        route_classes.each do |route_name|
+          route_class = ForestAdminRpcAgent::Routes.const_get(route_name)
+
+          # Skip if it's not a class or if it doesn't look like a route
+          unless route_class.is_a?(Class)
+            log_to_available_logger(
+              'Warn',
+              "Skipping constant: #{route_name} (not a class)"
+            )
+            next
+          end
+
+          begin
+            route_instance = route_class.new
+
+            unless route_instance.respond_to?(:registered)
+              log_to_available_logger(
+                'Warn',
+                "Skipping route: #{route_class} (does not respond to :registered)"
+              )
+              next
+            end
+
+            route_instances << route_instance
+          rescue ArgumentError => e
+            # Skip classes that require constructor arguments (not routes)
+            log_to_available_logger(
+              'Warn',
+              "Skipping constant: #{route_name} (requires constructor arguments: #{e.message})"
+            )
+            next
+          rescue StandardError => e
+            raise e.class, "Failed to instantiate route '#{route_name}': #{e.message}"
+          end
+        end
+
+        route_instances
+      end
+    end
+  end
+end

data/lib/forest_admin_rpc_agent/version.rb

@@ -1,3 +1,3 @@
 module ForestAdminRpcAgent
-  VERSION = "1.13.4"
+  VERSION = "1.14.0"
 end

data/lib/forest_admin_rpc_agent.rb

@@ -23,6 +23,7 @@ module ForestAdminRpcAgent
   setting :logger_level, default: 'info'
   setting :logger, default: nil
   setting :customize_error_message, default: nil
+  setting :disable_route_cache, default: false
 
   begin
     require 'thor'

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: forest_admin_rpc_agent
 version: !ruby/object:Gem::Version
-  version: 1.13.4
+  version: 1.14.0
 platform: ruby
 authors:
 - Matthieu
@@ -9,7 +9,7 @@ authors:
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2025-11-06 00:00:00.000000000 Z
+date: 2025-11-07 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: base64
@@ -162,6 +162,7 @@ files:
 - lib/forest_admin_rpc_agent/extensions/config_loader.rb
 - lib/forest_admin_rpc_agent/extensions/sinatra_extension.rb
 - lib/forest_admin_rpc_agent/facades/container.rb
+- lib/forest_admin_rpc_agent/http/router.rb
 - lib/forest_admin_rpc_agent/middleware/authentication.rb
 - lib/forest_admin_rpc_agent/routes/action_execute.rb
 - lib/forest_admin_rpc_agent/routes/action_form.rb