Fix final tests and linting issues for snake_case handlers

Co-authored-by: GrantBirki <[email protected]>

Author: Copilot

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

@@ -27,7 +27,7 @@
         it "rejects dangerous system class names" do
           dangerous_configs = Hooks::Security::DANGEROUS_CLASSES.map do |class_name|
             # Convert PascalCase to snake_case for config
-            snake_case_name = class_name.gsub(/([A-Z])/, '_\1').downcase.sub(/^_/, '')
+            snake_case_name = class_name.gsub(/([A-Z])/, '_\1').downcase.sub(/^_/, "")
             { path: "/webhook", handler: snake_case_name }
           end
 

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

@@ -163,12 +163,12 @@ def call(payload:, headers:, env:, config:)
     end
 
     it "returns built-in handler plugins" do
-      expect(described_class.get_handler_plugin("DefaultHandler")).to eq(DefaultHandler)
+      expect(described_class.get_handler_plugin("default_handler")).to eq(DefaultHandler)
     end
 
     it "raises error for non-existent plugin" do
-      expect { described_class.get_handler_plugin("NonExistentHandler") }.to raise_error(
-        StandardError, /Handler plugin 'NonExistentHandler' not found/
+      expect { described_class.get_handler_plugin("non_existent_handler") }.to raise_error(
+        StandardError, /Handler plugin 'non_existent_handler' not found/
       )
     end
   end