Remove hidden attributes from required params

- Update `required_params` method to skip hidden attributes when determining required parameters.
- Modify `response_model_spec` to reflect changes in required attributes, removing `hidden_attr`.
- Add a test in `parser_spec` to ensure hidden attributes are not marked as required.

Author: bogdan

lib/grape-swagger/entity/parser.rb

@@ -136,8 +136,11 @@ def parse_nested(entity_name, entity_options, parent_model = nil)
 
       def required_params(params)
         params.each_with_object(Set.new) do |(key, options), accum|
-          required = if options.fetch(:documentation, {}).key?(:required)
-                       options.dig(:documentation, :required)
+          documentation = options.fetch(:documentation, {})
+          next if documentation[:hidden]
+
+          required = if documentation.key?(:required)
+                       documentation[:required]
                      else
                        !options.key?(:if) && !options.key?(:unless) && options[:expose_nil] != false
                      end

spec/grape-swagger/entities/response_model_spec.rb

@@ -67,8 +67,7 @@ def app
             'code' => { 'type' => 'string', 'description' => 'Error code' },
             'message' => { 'type' => 'string', 'description' => 'Error message' },
             'attr' => { 'type' => 'string', 'description' => 'Attribute' } },
-      'required' => %w[text colors hidden_attr created_at kind kind2 kind3 tags relation
-                       attr code message]
+      'required' => %w[text colors created_at kind kind2 kind3 tags relation attr code message]
     )
 
     expect(subject['definitions'].keys).to include 'ThisApi_Entities_Kind'

spec/grape-swagger/entity/parser_spec.rb

@@ -21,6 +21,10 @@
           expect(properties[:kind3]['$ref']).to eq('#/definitions/Kind')
         end
 
+        it 'does not mark hidden attributes as required' do
+          expect(required).not_to include(:hidden_attr)
+        end
+
         it 'merges attributes that have merge: true defined' do
           expect(properties[:merged_attribute]).to be_nil
           expect(properties[:code][:type]).to eq('string')