[Issue]: OpenApiV2 introspection throws error for the required params when used with discrimnator keyword for OAS3 about wundergraph

[Pagebakers]

I did some further digging and while writing the unit test I figured out the issue.

The problem is when you use allOf the top level required doesn't apply to the child schemas.

In this case - $ref: '#/components/schemas/SomeSchema' has correctly required someproprtyId

However this object schema doesn't have a require on someproprtyId

- type: object
                  properties:
                      someproprtyId:
                          type: string

The correct schema would be:

components:
     schemas:
            SomeSchema:
               required:
               - someproprtyId
               type: object
               properties:
                   someproprtyId:
                       type: string
                       ...
               discriminator:
                   propertyName: someproprtyId
                ....
            OtherSchema:
               required:
               - id
               - someproprtyId
               type: object
               allOf:
               - $ref: '#/components/schemas/SomeSchema'
               - type: object
                  required: # required needs to be added here
                  - someproprtyId
                  properties:
                      someproprtyId:
                          type: string
                  data:
                    ...

I checked this with wunderctl generate and the graphql schema is generated correctly.

[Pagebakers]

Supplied a patch as a workaround. This is still WIP with the upstream package.

[stale]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.