ruby_llm-agents 3.7.2 → 3.9.0

data/lib/generators/ruby_llm_agents/templates/application_agent.rb.tt

@@ -2,15 +2,15 @@
 
 # ApplicationAgent - Base class for all agents in this application
 #
-# All agents inherit from this class. Configure shared settings here
-# that apply to all agents, or override them per-agent as needed.
+# All agents inherit from this class. Configure shared settings here.
 #
-# Example:
+# Quick reference:
 #   class MyAgent < ApplicationAgent
-#     param :query, required: true
+#     system "You are a helpful assistant."
+#     prompt "Answer this: {query}"          # {query} becomes a required param
 #
-#     def user_prompt
-#       query
+#     returns do                             # Optional: structured output
+#       string :answer, description: "The answer"
 #     end
 #   end
 #
@@ -20,50 +20,17 @@
 #   MyAgent.call(query: "hello", skip_cache: true) # Bypass cache
 #
 class ApplicationAgent < RubyLLM::Agents::Base
-  # ============================================
-  # Shared Model Configuration
-  # ============================================
-  # These settings are inherited by all agents
-
-  # model "gemini-2.0-flash"   # Default model for all agents
-  # temperature 0.0            # Default temperature (0.0 = deterministic)
-  # timeout 60                 # Default timeout in seconds
-
-  # ============================================
-  # Shared Caching
-  # ============================================
-
-  # cache 1.hour  # Enable caching for all agents (override per-agent if needed)
-
-  # ============================================
-  # Shared Reliability Settings
-  # ============================================
-  # Configure once here, all agents inherit these settings
-
-  # Automatic retries for all agents
-  # retries max: 2, backoff: :exponential, base: 0.4, max_delay: 3.0
-
-  # Shared fallback models
-  # fallback_models ["gpt-4o-mini", "claude-3-haiku"]
-
-  # Total timeout across retries/fallbacks
-  # total_timeout 30
-
-  # Circuit breaker (per agent-model pair)
-  # circuit_breaker errors: 5, within: 60, cooldown: 300
-
-  # ============================================
-  # Shared Helper Methods
-  # ============================================
-  # Define methods here that can be used by all agents
-
-  # Example: Common system prompt prefix
-  # def system_prompt_prefix
-  #   "You are an AI assistant for #{Rails.application.class.module_parent_name}."
-  # end
-
-  # Example: Common metadata
-  # def metadata
-  #   { app_version: Rails.application.config.version }
+  # Shared settings inherited by all agents.
+  # Override per-agent as needed.
+
+  # model "gpt-4o"              # Override the configured default model
+  # temperature 0.0             # 0.0 = deterministic, 2.0 = creative
+  # cache for: 1.hour           # Enable caching for all agents
+
+  # Shared error handling (uncomment to apply to all agents)
+  # on_failure do
+  #   retries times: 2, backoff: :exponential
+  #   fallback to: ["gpt-4o-mini"]
+  #   timeout 30
   # end
 end

data/lib/generators/ruby_llm_agents/templates/initializer.rb.tt

@@ -6,29 +6,14 @@
 
 RubyLLM::Agents.configure do |config|
   # ============================================
-  # LLM Provider API Keys
+  # Quick Start — set ONE API key to get going
   # ============================================
-  # Configure at least one provider. Set these in your environment
-  # or replace ENV[] calls with your keys directly.
+  # Uncomment one line below, then run: rails ruby_llm_agents:doctor
 
   # config.openai_api_key = ENV["OPENAI_API_KEY"]
   # config.anthropic_api_key = ENV["ANTHROPIC_API_KEY"]
   # config.gemini_api_key = ENV["GOOGLE_API_KEY"]
 
-  # Additional providers:
-  # config.deepseek_api_key = ENV["DEEPSEEK_API_KEY"]
-  # config.openrouter_api_key = ENV["OPENROUTER_API_KEY"]
-  # config.mistral_api_key = ENV["MISTRAL_API_KEY"]
-  # config.xai_api_key = ENV["XAI_API_KEY"]
-
-  # Custom endpoints (e.g., Azure OpenAI, local Ollama):
-  # config.openai_api_base = "https://your-resource.openai.azure.com"
-  # config.ollama_api_base = "http://localhost:11434"
-
-  # Connection settings:
-  # config.request_timeout = 120
-  # config.max_retries = 3
-
   # ============================================
   # Model Defaults
   # ============================================
@@ -46,6 +31,23 @@ RubyLLM::Agents.configure do |config|
   # When enabled, agents stream responses and track time-to-first-token
   # config.default_streaming = false
 
+  # ============================================
+  # Additional Providers (uncomment as needed)
+  # ============================================
+
+  # config.deepseek_api_key = ENV["DEEPSEEK_API_KEY"]
+  # config.openrouter_api_key = ENV["OPENROUTER_API_KEY"]
+  # config.mistral_api_key = ENV["MISTRAL_API_KEY"]
+  # config.xai_api_key = ENV["XAI_API_KEY"]
+
+  # Custom endpoints (e.g., Azure OpenAI, local Ollama):
+  # config.openai_api_base = "https://your-resource.openai.azure.com"
+  # config.ollama_api_base = "http://localhost:11434"
+
+  # Connection settings:
+  # config.request_timeout = 120
+  # config.max_retries = 3
+
   # ============================================
   # Caching
   # ============================================

data/lib/generators/ruby_llm_agents/templates/migration.rb.tt

@@ -77,6 +77,9 @@ class CreateRubyLLMAgentsExecutions < ActiveRecord::Migration<%= migration_versi
     add_index :ruby_llm_agents_executions, :request_id
     add_index :ruby_llm_agents_executions, :parent_execution_id
     add_index :ruby_llm_agents_executions, :root_execution_id
+    add_index :ruby_llm_agents_executions, [:status, :created_at]
+    add_index :ruby_llm_agents_executions, [:model_id, :status]
+    add_index :ruby_llm_agents_executions, [:cache_hit, :created_at]
 
     # Foreign keys for execution hierarchy
     add_foreign_key :ruby_llm_agents_executions, :ruby_llm_agents_executions,

data/lib/generators/ruby_llm_agents/upgrade_generator.rb

@@ -98,6 +98,25 @@ module RubyLlmAgents
       )
     end
 
+    # Add dashboard performance indexes
+    def create_add_dashboard_performance_indexes_migration
+      unless table_exists?(:ruby_llm_agents_executions)
+        say_status :skip, "executions table does not exist yet", :yellow
+        return
+      end
+
+      if index_exists?(:ruby_llm_agents_executions, [:status, :created_at])
+        say_status :skip, "dashboard performance indexes already exist", :yellow
+        return
+      end
+
+      say_status :upgrade, "Adding dashboard performance indexes", :blue
+      migration_template(
+        "add_dashboard_performance_indexes_migration.rb.tt",
+        File.join(db_migrate_path, "add_dashboard_performance_indexes.rb")
+      )
+    end
+
     def suggest_config_consolidation
       ruby_llm_initializer = File.join(destination_root, "config/initializers/ruby_llm.rb")
       agents_initializer = File.join(destination_root, "config/initializers/ruby_llm_agents.rb")
@@ -192,5 +211,11 @@ module RubyLlmAgents
     rescue
       false
     end
+
+    def index_exists?(table, columns)
+      ActiveRecord::Base.connection.index_exists?(table, columns)
+    rescue
+      false
+    end
   end
 end

data/lib/ruby_llm/agents/base_agent.rb

@@ -332,6 +332,14 @@ module RubyLLM
       # @param temperature [Float] Override the class-level temperature
       # @param options [Hash] Agent parameters defined via the param DSL
       def initialize(model: self.class.model, temperature: self.class.temperature, **options)
+        # Merge tracker defaults (shared options like tenant) — explicit opts win
+        tracker = Thread.current[:ruby_llm_agents_tracker]
+        if tracker
+          options = tracker.defaults.merge(options)
+          @_track_request_id = tracker.request_id
+          @_track_tags = tracker.tags
+        end
+
         @ask_message = options.delete(:_ask_message)
         @parent_execution_id = options.delete(:_parent_execution_id)
         @root_execution_id = options.delete(:_root_execution_id)
@@ -506,6 +514,7 @@ module RubyLLM
           stream_block: (block if streaming_enabled?),
           parent_execution_id: @parent_execution_id,
           root_execution_id: @root_execution_id,
+          debug: @options[:debug],
           options: execution_options
         )
       end
@@ -721,6 +730,10 @@ module RubyLLM
         capture_response(response, context)
         result = build_result(process_response(response), response, context)
         context.output = result
+      rescue RubyLLM::UnauthorizedError, RubyLLM::ForbiddenError => e
+        raise_with_setup_hint(e, context)
+      rescue RubyLLM::ModelNotFoundError => e
+        raise_with_model_hint(e, context)
       ensure
         Thread.current[:ruby_llm_agents_caller_context] = previous_context
       end
@@ -731,8 +744,18 @@ module RubyLLM
       # @return [RubyLLM::Chat] Configured chat client
       def build_client(context = nil)
         effective_model = context&.model || model
-        client = RubyLLM.chat(model: effective_model)
-          .with_temperature(temperature)
+        chat_opts = {model: effective_model}
+
+        # Use scoped RubyLLM::Context for thread-safe per-tenant API keys.
+        # RubyLLM::Context#chat creates a Chat with the scoped config,
+        # so we call .chat on the context instead of RubyLLM.chat.
+        llm_ctx = context&.llm
+        client = if llm_ctx.is_a?(RubyLLM::Context)
+          llm_ctx.chat(**chat_opts)
+        else
+          RubyLLM.chat(**chat_opts)
+        end
+        client = client.with_temperature(temperature)
 
         client = client.with_instructions(system_prompt) if system_prompt
         client = client.with_schema(schema) if schema
@@ -883,8 +906,9 @@ module RubyLLM
       # @param context [Pipeline::Context] The context
       # @return [Result] The result object
       def build_result(content, response, context)
-        Result.new(
+        result_opts = {
           content: content,
+          agent_class_name: self.class.name,
           input_tokens: context.input_tokens,
           output_tokens: context.output_tokens,
           input_cost: context.input_cost,
@@ -901,7 +925,12 @@ module RubyLLM
           streaming: streaming_enabled?,
           attempts_count: context.attempts_made || 1,
           execution_id: context.execution_id
-        )
+        }
+
+        # Attach pipeline trace when debug mode is enabled
+        result_opts[:trace] = context.trace if context.trace_enabled? && context.trace.any?
+
+        Result.new(**result_opts)
       end
 
       # Extracts thinking data from a response for inclusion in Result
@@ -1071,6 +1100,44 @@ module RubyLLM
           tool_call[key] || tool_call[key.to_s]
         end
       end
+
+      # Re-raises auth errors with actionable setup guidance
+      def raise_with_setup_hint(error, context)
+        effective_model = context&.model || model
+        provider = detect_provider(effective_model)
+
+        hint = "#{self.class.name} failed: #{error.message}\n\n" \
+               "The API key for #{provider || "your provider"} is missing or invalid.\n" \
+               "Fix: Set the key in config/initializers/ruby_llm_agents.rb\n" \
+               "     or run: rails ruby_llm_agents:doctor"
+
+        raise RubyLLM::Agents::ConfigurationError, hint
+      end
+
+      # Re-raises model errors with actionable guidance
+      def raise_with_model_hint(error, context)
+        effective_model = context&.model || model
+
+        hint = "#{self.class.name} failed: #{error.message}\n\n" \
+               "Model '#{effective_model}' was not found.\n" \
+               "Fix: Check the model name or set a default in your initializer:\n" \
+               "     config.default_model = \"gpt-4o\""
+
+        raise RubyLLM::Agents::ConfigurationError, hint
+      end
+
+      # Best-effort provider detection from model name
+      def detect_provider(model_id)
+        return nil unless model_id
+
+        case model_id.to_s
+        when /gpt|o[1-9]|dall-e|whisper|tts/i then "OpenAI"
+        when /claude/i then "Anthropic"
+        when /gemini|gemma/i then "Google (Gemini)"
+        when /deepseek/i then "DeepSeek"
+        when /mistral|mixtral/i then "Mistral"
+        end
+      end
     end
   end
 end

data/lib/ruby_llm/agents/core/base.rb

@@ -87,6 +87,10 @@ module RubyLLM
         run_callbacks(:after, context, response)
 
         context.output = build_result(processed_content, response, context)
+      rescue RubyLLM::UnauthorizedError, RubyLLM::ForbiddenError => e
+        raise_with_setup_hint(e, context)
+      rescue RubyLLM::ModelNotFoundError => e
+        raise_with_model_hint(e, context)
       end
 
       # Returns the resolved tenant ID for tracking

data/lib/ruby_llm/agents/core/configuration.rb

@@ -349,6 +349,7 @@ module RubyLLM
         perplexity_api_key
         xai_api_key
         gpustack_api_key
+        inception_api_key
         openai_api_base
         openai_organization_id
         openai_project_id
@@ -818,6 +819,16 @@ module RubyLLM
         tenant_resolver&.call
       end
 
+      # Returns a concise string representation for debugging
+      #
+      # @return [String] Summary of key configuration values
+      def inspect
+        "#<#{self.class} model=#{default_model.inspect} temperature=#{default_temperature} " \
+          "timeout=#{default_timeout} streaming=#{default_streaming} " \
+          "multi_tenancy=#{multi_tenancy_enabled} async_logging=#{async_logging} " \
+          "track_executions=#{track_executions}>"
+      end
+
       # Returns whether the async gem is available
       #
       # @return [Boolean] true if async gem is loaded

data/lib/ruby_llm/agents/core/instrumentation.rb

@@ -2,31 +2,27 @@
 
 module RubyLLM
   module Agents
-    # Instrumentation concern for tracking agent executions
+    # @deprecated This module is deprecated and will be removed in a future version.
+    #   All agents now use {Pipeline::Middleware::Instrumentation} automatically
+    #   via the middleware pipeline. This module is no longer included in any
+    #   production class. It remains only for backward compatibility with code
+    #   that explicitly includes it.
     #
-    # Provides comprehensive execution tracking including:
-    # - Timing metrics (started_at, completed_at, duration_ms)
-    # - Token usage tracking (input, output, cached)
-    # - Cost calculation via RubyLLM pricing data
-    # - Error and timeout handling with status tracking
-    # - Safe parameter sanitization for logging
-    #
-    # Included automatically in {RubyLLM::Agents::Base}.
-    #
-    # @example Adding custom metadata to executions
-    #   class MyAgent < ApplicationAgent
-    #     def metadata
-    #       { user_id: Current.user&.id, request_id: request.uuid }
-    #     end
-    #   end
-    #
-    # @see RubyLLM::Agents::Execution
-    # @see RubyLLM::Agents::ExecutionLoggerJob
+    # @see Pipeline::Middleware::Instrumentation
     # @api private
     module Instrumentation
       extend ActiveSupport::Concern
 
       included do
+        if defined?(RubyLLM::Agents::Deprecations)
+          RubyLLM::Agents::Deprecations.warn(
+            "RubyLLM::Agents::Instrumentation is deprecated. " \
+            "All agents now use Pipeline::Middleware::Instrumentation automatically. " \
+            "Remove `include RubyLLM::Agents::Instrumentation` from #{name || "your class"}.",
+            caller
+          )
+        end
+
         # @!attribute [rw] execution_id
         #   The ID of the current execution record
         #   @return [Integer, nil]

data/lib/ruby_llm/agents/core/version.rb

@@ -4,6 +4,6 @@ module RubyLLM
   module Agents
     # Current version of the RubyLLM::Agents gem
     # @return [String] Semantic version string
-    VERSION = "3.7.2"
+    VERSION = "3.9.0"
   end
 end

data/lib/ruby_llm/agents/infrastructure/alert_manager.rb

@@ -83,8 +83,8 @@ module RubyLLM
         # @return [void]
         def emit_notification(event, payload)
           ActiveSupport::Notifications.instrument("ruby_llm_agents.alert.#{event}", payload)
-        rescue
-          # Ignore notification failures
+        rescue => e
+          Rails.logger.debug("[RubyLLM::Agents::AlertManager] Notification failed: #{e.message}") if defined?(Rails) && Rails.logger
         end
 
         # Stores the alert in cache for dashboard display
@@ -106,8 +106,8 @@ module RubyLLM
           alerts = alerts.first(50)
 
           cache.write(key, alerts, expires_in: 24.hours)
-        rescue
-          # Ignore cache failures
+        rescue => e
+          Rails.logger.debug("[RubyLLM::Agents::AlertManager] Cache store failed: #{e.message}") if defined?(Rails) && Rails.logger
         end
 
         # Formats a human-readable message for the event

data/lib/ruby_llm/agents/infrastructure/budget_tracker.rb

@@ -44,7 +44,7 @@ module RubyLLM
         # @raise [Reliability::BudgetExceededError] If hard cap is exceeded
         # @return [void]
         def check_budget!(agent_type, tenant_id: nil, tenant_config: nil)
-          tenant_id = Budget::ConfigResolver.resolve_tenant_id(tenant_id)
+          tenant_id = resolve_tid(tenant_id)
           budget_config = Budget::ConfigResolver.resolve_budget_config(tenant_id, runtime_config: tenant_config)
 
           return unless budget_config[:enabled]
@@ -61,7 +61,7 @@ module RubyLLM
         # @raise [Reliability::BudgetExceededError] If hard cap is exceeded
         # @return [void]
         def check_token_budget!(agent_type, tenant_id: nil, tenant_config: nil)
-          tenant_id = Budget::ConfigResolver.resolve_tenant_id(tenant_id)
+          tenant_id = resolve_tid(tenant_id)
           budget_config = Budget::ConfigResolver.resolve_budget_config(tenant_id, runtime_config: tenant_config)
 
           return unless budget_config[:enabled]
@@ -80,7 +80,7 @@ module RubyLLM
         def record_spend!(agent_type, amount, tenant_id: nil, tenant_config: nil)
           return if amount.nil? || amount <= 0
 
-          tenant_id = Budget::ConfigResolver.resolve_tenant_id(tenant_id)
+          tenant_id = resolve_tid(tenant_id)
           budget_config = Budget::ConfigResolver.resolve_budget_config(tenant_id, runtime_config: tenant_config)
 
           Budget::SpendRecorder.record_spend!(agent_type, amount, tenant_id: tenant_id, budget_config: budget_config)
@@ -96,7 +96,7 @@ module RubyLLM
         def record_tokens!(agent_type, tokens, tenant_id: nil, tenant_config: nil)
           return if tokens.nil? || tokens <= 0
 
-          tenant_id = Budget::ConfigResolver.resolve_tenant_id(tenant_id)
+          tenant_id = resolve_tid(tenant_id)
           budget_config = Budget::ConfigResolver.resolve_budget_config(tenant_id, runtime_config: tenant_config)
 
           Budget::SpendRecorder.record_tokens!(agent_type, tokens, tenant_id: tenant_id, budget_config: budget_config)
@@ -110,7 +110,7 @@ module RubyLLM
         # @param tenant_id [String, nil] Optional tenant identifier (uses resolver if not provided)
         # @return [Float] Current spend in USD
         def current_spend(scope, period, agent_type: nil, tenant_id: nil)
-          tenant_id = Budget::ConfigResolver.resolve_tenant_id(tenant_id)
+          tenant_id = resolve_tid(tenant_id)
           Budget::BudgetQuery.current_spend(scope, period, agent_type: agent_type, tenant_id: tenant_id)
         end
 
@@ -120,7 +120,7 @@ module RubyLLM
         # @param tenant_id [String, nil] Optional tenant identifier (uses resolver if not provided)
         # @return [Integer] Current token usage
         def current_tokens(period, tenant_id: nil)
-          tenant_id = Budget::ConfigResolver.resolve_tenant_id(tenant_id)
+          tenant_id = resolve_tid(tenant_id)
           Budget::BudgetQuery.current_tokens(period, tenant_id: tenant_id)
         end
 
@@ -132,7 +132,7 @@ module RubyLLM
         # @param tenant_id [String, nil] Optional tenant identifier (uses resolver if not provided)
         # @return [Float, nil] Remaining budget in USD, or nil if no limit configured
         def remaining_budget(scope, period, agent_type: nil, tenant_id: nil)
-          tenant_id = Budget::ConfigResolver.resolve_tenant_id(tenant_id)
+          tenant_id = resolve_tid(tenant_id)
           budget_config = Budget::ConfigResolver.resolve_budget_config(tenant_id)
 
           Budget::BudgetQuery.remaining_budget(scope, period, agent_type: agent_type, tenant_id: tenant_id, budget_config: budget_config)
@@ -144,7 +144,7 @@ module RubyLLM
         # @param tenant_id [String, nil] Optional tenant identifier (uses resolver if not provided)
         # @return [Integer, nil] Remaining token budget, or nil if no limit configured
         def remaining_token_budget(period, tenant_id: nil)
-          tenant_id = Budget::ConfigResolver.resolve_tenant_id(tenant_id)
+          tenant_id = resolve_tid(tenant_id)
           budget_config = Budget::ConfigResolver.resolve_budget_config(tenant_id)
 
           Budget::BudgetQuery.remaining_token_budget(period, tenant_id: tenant_id, budget_config: budget_config)
@@ -156,7 +156,7 @@ module RubyLLM
         # @param tenant_id [String, nil] Optional tenant identifier (uses resolver if not provided)
         # @return [Hash] Budget status information
         def status(agent_type: nil, tenant_id: nil)
-          tenant_id = Budget::ConfigResolver.resolve_tenant_id(tenant_id)
+          tenant_id = resolve_tid(tenant_id)
           budget_config = Budget::ConfigResolver.resolve_budget_config(tenant_id)
 
           Budget::BudgetQuery.status(agent_type: agent_type, tenant_id: tenant_id, budget_config: budget_config)
@@ -167,7 +167,7 @@ module RubyLLM
         # @param tenant_id [String, nil] Optional tenant identifier (uses resolver if not provided)
         # @return [Hash, nil] Forecast information
         def calculate_forecast(tenant_id: nil)
-          tenant_id = Budget::ConfigResolver.resolve_tenant_id(tenant_id)
+          tenant_id = resolve_tid(tenant_id)
           budget_config = Budget::ConfigResolver.resolve_budget_config(tenant_id)
 
           Budget::Forecaster.calculate_forecast(tenant_id: tenant_id, budget_config: budget_config)
@@ -178,7 +178,7 @@ module RubyLLM
         # @param tenant_id [String, nil] Optional tenant identifier to reset only that tenant's counters
         # @return [void]
         def reset!(tenant_id: nil)
-          tenant_id = Budget::ConfigResolver.resolve_tenant_id(tenant_id)
+          tenant_id = resolve_tid(tenant_id)
           tenant_part = Budget::SpendRecorder.tenant_key_part(tenant_id)
           today = Budget::SpendRecorder.date_key_part(:daily)
           month = Budget::SpendRecorder.date_key_part(:monthly)
@@ -192,6 +192,14 @@ module RubyLLM
 
         private
 
+        # Resolves tenant ID, falling back to the configured resolver
+        #
+        # @param tenant_id [String, nil] Explicit tenant ID or nil
+        # @return [String, nil] Resolved tenant ID
+        def resolve_tid(tenant_id)
+          Budget::ConfigResolver.resolve_tenant_id(tenant_id)
+        end
+
         # Checks budget limits and raises error if exceeded
         #
         # @param agent_type [String] The agent class name

data/lib/ruby_llm/agents/pipeline/builder.rb

@@ -170,7 +170,8 @@ module RubyLLM
           # @return [Array<Hash>] Middleware entries
           def global_middleware_entries
             RubyLLM::Agents.configuration.middleware_stack
-          rescue
+          rescue => e
+            Rails.logger.debug("[RubyLLM::Agents::Pipeline] Failed to load global middleware: #{e.message}") if defined?(Rails) && Rails.logger
             []
           end
 
@@ -182,7 +183,8 @@ module RubyLLM
             return [] unless agent_class&.respond_to?(:agent_middleware)
 
             agent_class.agent_middleware
-          rescue
+          rescue => e
+            Rails.logger.debug("[RubyLLM::Agents::Pipeline] Failed to load agent middleware: #{e.message}") if defined?(Rails) && Rails.logger
             []
           end
 
@@ -207,7 +209,8 @@ module RubyLLM
           # @return [Boolean]
           def budgets_enabled?
             RubyLLM::Agents.configuration.budgets_enabled?
-          rescue
+          rescue => e
+            Rails.logger.debug("[RubyLLM::Agents::Pipeline] Failed to check budgets_enabled: #{e.message}") if defined?(Rails) && Rails.logger
             false
           end
 
@@ -248,7 +251,8 @@ module RubyLLM
 
             (retries.is_a?(Integer) && retries.positive?) ||
               (fallbacks.is_a?(Array) && fallbacks.any?)
-          rescue
+          rescue => e
+            Rails.logger.debug("[RubyLLM::Agents::Pipeline] Failed to check reliability_enabled: #{e.message}") if defined?(Rails) && Rails.logger
             false
           end
         end