Add environment variable to silence config loader messages

Author: GrantBirki

lib/hooks/core/config_loader.rb

@@ -24,6 +24,10 @@ class ConfigLoader
         normalize_headers: true
       }.freeze
 
+      SILENCE_CONFIG_LOADER_MESSAGES = ENV.fetch(
+        "HOOKS_SILENCE_CONFIG_LOADER_MESSAGES", "false"
+      ).downcase == "true".freeze
+
       # Load and merge configuration from various sources
       #
       # @param config_path [String, Hash] Path to config file or config hash
@@ -65,7 +69,7 @@ def self.load(config_path: nil)
 
         # Log overrides if any were made
         if overrides.any?
-          puts "INFO: Configuration overrides applied from: #{overrides.join(', ')}"
+          puts "INFO: Configuration overrides applied from: #{overrides.join(', ')}" unless SILENCE_CONFIG_LOADER_MESSAGES
         end
 
         return config
@@ -112,7 +116,7 @@ def self.load_config_file(file_path)
         result
       rescue => e
         # Log this error with meaningful information
-        puts "ERROR: Failed to load config file '#{file_path}': #{e.message}"
+        puts "ERROR: Failed to load config file '#{file_path}': #{e.message}" unless SILENCE_CONFIG_LOADER_MESSAGES
         nil
       end
 

spec/integration/global_lifecycle_hooks_spec.rb

@@ -1,5 +1,7 @@
 # frozen_string_literal: true
 
+ENV["HOOKS_SILENCE_CONFIG_LOADER_MESSAGES"] = "true" # Silence config loader messages in tests
+
 require_relative "../../lib/hooks"
 require "rack/test"
 require "json"

spec/integration/hooks_integration_spec.rb

@@ -1,5 +1,7 @@
 # frozen_string_literal: true
 
+ENV["HOOKS_SILENCE_CONFIG_LOADER_MESSAGES"] = "true" # Silence config loader messages in tests
+
 require_relative "../../lib/hooks"
 require "rack/test"
 require "json"

spec/unit/spec_helper.rb

@@ -12,6 +12,7 @@
 
 ENV["GITHUB_WEBHOOK_SECRET"] = FAKE_HMAC_SECRET
 ENV["ALT_WEBHOOK_SECRET"] = FAKE_ALT_HMAC_SECRET
+ENV["HOOKS_SILENCE_CONFIG_LOADER_MESSAGES"] = "true"
 
 COV_DIR = File.expand_path("../../coverage", File.dirname(__FILE__))