Swagger JSON missing global definitions from schema used in request spec

[mkreis]

I think I found a bug regarding Swagger Schema generation:

Only definitions from schemas referred to in responses are shown as global definitions in /definitions, but those from requests are missing.

With the following Reitit route config

:parameters {:body [:map [:company ::validation/company1]]}
        :responses {200 {:body ::validation/company2}}

only company2 will be in /definitions (although the ref$ to both def’s are there, thus causing a missing/invalid reference to company1).

I'm using Malli 0.14.0 and Reitit 0.7.0-alpha7

[aviflax]

Might be related to metosin/reitit#616

[mkreis]

But only very loosely. As far as I understand, metosin/reitit#616 is about the overall location of definitions for the OpenAPI schema, whereas here it is about missing parts in the definition.

[opqdonut]

Thanks for the report! I only noticed it now since it's in this repo and not reitit. I think I see the where the problem is.

[opqdonut]

I'm unable to reproduce this with malli 0.16.0 and the latest reitit master. The code in malli does look a bit suspicious. Do you have a working reproduction @mkreis ?

[mkreis]

I can try to reproduce it with malli 0.16.0 and add the results here.

[mkreis]

Hi @opqdonut , I managed to recreate it with a simple project: https://github.com/mkreis/malli-issue-1002
As it turned out, the probem is caused by multiple entries in the :parameters map, e.g. parameters for :path, :query and :body at the same time. If there is only one key, then the definitions are exported. With multiple keys, it's broken.
In my sample project above, I have marked the particular section in file service.clj.

[opqdonut]

Thanks! I'll try to have a look tomorrow.

[mkreis]

Cool, thank you!

Also I just noticed that it works if key :body is the first key in the :parameters map. So it might be easy to fix... I hope.