remove the signal handler as it is not used anywhere and things like puma should really be in charge

Author: GrantBirki

docs/design.md

@@ -140,7 +140,6 @@ lib/hooks/
 │   ├── config_loader.rb      # Loads + merges per-endpoint configs
 │   ├── config_validator.rb   # Dry::Schema-based validation
 │   ├── logger_factory.rb     # Structured JSON logger + context enrichment
-│   └── signal_handler.rb     # Trap SIGINT/SIGTERM for graceful shutdown
 │
 ├── handlers/
 │   └── base.rb               # `Hooks::Handlers::Base` interface: defines #call

lib/hooks/app/api.rb

@@ -13,15 +13,14 @@ module App
     # Factory for creating configured Grape API classes
     class API
       # Create a new configured API class
-      def self.create(config:, endpoints:, log:, signal_handler:)
+      def self.create(config:, endpoints:, log:)
         # Store startup time for uptime calculation
         start_time = Time.now
 
         # Capture values in local variables for closure
         captured_config = config
         captured_endpoints = endpoints
         captured_logger = log
-        _captured_signal_handler = signal_handler
         captured_start_time = start_time
 
         # Set global logger instance for plugins/validators

lib/hooks/core/builder.rb

@@ -3,7 +3,6 @@
 require_relative "config_loader"
 require_relative "config_validator"
 require_relative "logger_factory"
-require_relative "signal_handler"
 require_relative "../app/api"
 
 module Hooks
@@ -34,9 +33,6 @@ def build
           )
         end
 
-        # Setup signal handler for graceful shutdown
-        signal_handler = SignalHandler.new(@log)
-
         # Load endpoints
         endpoints = load_endpoints(config)
 
@@ -50,8 +46,7 @@ def build
         Hooks::App::API.create(
           config: config,
           endpoints: endpoints,
-          log: @log,
-          signal_handler: signal_handler
+          log: @log
         )
       end
 

lib/hooks/core/signal_handler.rb

@@ -97,7 +97,6 @@
           expect(args[:config]).to be_a(Hash)
           expect(args[:endpoints]).to eq([])
           expect(args[:log]).to respond_to(:info)
-          expect(args[:signal_handler]).to be_a(Hooks::Core::SignalHandler)
           "mock_api"
         end