Refactor JSON parsing comments and update warning messages for clarity

Author: GrantBirki

lib/hooks/app/helpers.rb

@@ -40,7 +40,7 @@ def enforce_request_limits(config)
         # Note: Timeout enforcement would typically be handled at the server level (Puma, etc.)
       end
 
-      # Parse request payload with security limits
+      # Parse request payload
       #
       # @param raw_body [String] The raw request body
       # @param headers [Hash] The request headers
@@ -59,9 +59,8 @@ def parse_payload(raw_body, headers, symbolize: true)
             return parsed_payload
           rescue JSON::ParserError, ArgumentError => e
             # If JSON parsing fails or security limits exceeded, return raw body
-            # Log security violations at debug level to avoid log spam
             if e.message.include?("nesting") || e.message.include?("depth")
-              log.warn("JSON parsing security limit exceeded: #{e.message}")
+              log.warn("JSON parsing limit exceeded: #{e.message}")
             end
           end
         end
@@ -87,7 +86,7 @@ def load_handler(handler_class_name)
 
       private
 
-      # Safely parse JSON with security limits to prevent JSON bombs
+      # Safely parse JSON
       #
       # @param json_string [String] The JSON string to parse
       # @return [Hash, Array] Parsed JSON object

spec/unit/lib/hooks/app/helpers_spec.rb

@@ -236,7 +236,7 @@ def error!(message, code)
         # Mock logger to capture debug messages
         logger = instance_double("Logger")
         allow(helper).to receive(:log).and_return(logger)
-        expect(logger).to receive(:warn).with(/JSON parsing security limit exceeded/)
+        expect(logger).to receive(:warn).with(/JSON parsing limit exceeded/)
 
         # Mock the safe_json_parse method to simulate nesting limit exceeded
         allow(helper).to receive(:safe_json_parse).and_raise(ArgumentError, "nesting exceeded")