make the handler_plugin_dir a required field

Author: GrantBirki

lib/hooks/core/config_validator.rb

@@ -12,7 +12,7 @@ class ValidationError < StandardError; end
 
       # Global configuration schema
       GLOBAL_CONFIG_SCHEMA = Dry::Schema.Params do
-        optional(:handler_plugin_dir).filled(:string)
+        required(:handler_plugin_dir).filled(:string)
         optional(:auth_plugin_dir).maybe(:string)
         optional(:lifecycle_plugin_dir).maybe(:string)
         optional(:instruments_plugin_dir).maybe(:string)

spec/unit/lib/hooks/core/config_validator_spec.rb

@@ -24,15 +24,18 @@
       end
 
       it "returns validated configuration with minimal fields" do
-        config = {}
+        config = { handler_plugin_dir: "/path/to/handlers" }
 
         result = described_class.validate_global_config(config)
 
-        expect(result).to eq({})
+        expect(result).to eq({ handler_plugin_dir: "/path/to/handlers" })
       end
 
       it "accepts production environment" do
-        config = { environment: "production" }
+        config = {
+          environment: "production",
+          handler_plugin_dir: "/path/to/handlers"
+        }
 
         result = described_class.validate_global_config(config)
 
@@ -41,7 +44,10 @@
 
       it "accepts valid log levels" do
         %w[debug info warn error].each do |log_level|
-          config = { log_level: log_level }
+          config = {
+            log_level: log_level,
+            handler_plugin_dir: "/path/to/handlers"
+          }
 
           result = described_class.validate_global_config(config)
 
@@ -114,7 +120,8 @@
       it "coerces boolean-like string values" do
         config = {
           use_catchall_route: "true",
-          normalize_headers: "1"
+          normalize_headers: "1",
+          handler_plugin_dir: "/path/to/handlers"
         }
 
         result = described_class.validate_global_config(config)
@@ -138,7 +145,8 @@
       it "coerces string numeric values" do
         config = {
           request_limit: "1024",
-          request_timeout: "30"
+          request_timeout: "30",
+          handler_plugin_dir: "/path/to/handlers"
         }
 
         result = described_class.validate_global_config(config)
@@ -164,7 +172,10 @@
       end
 
       it "coerces float values to integers by truncating" do
-        config = { request_timeout: 30.5 }
+        config = {
+          request_timeout: 30.5,
+          handler_plugin_dir: "/path/to/handlers"
+        }
 
         result = described_class.validate_global_config(config)