Unstable order when reading spec from `.json` file about createapi HOT 6 CLOSED

I had a go at doing this, but I found a bug in Yams that is a slight blocker (it caused one of the specs that decodes a json configuration to fail):

• jpsim/Yams#351

Other than the empty string/optional issue though, Yams has been pretty good at handling our 31,500 line OpenAPI schema in JSON format, we've just been renaming it to .yml (so that CreateAPI uses Yams and preserves the ordering) but not converting it from JSON.

from createapi.

Unfortunately, that seems to be the case. It's recommended to use YAML. OpenAPIKit uses ordered dictionaries to preserve the order of the keys when it can (YAML).

To convert, you can use Swagger Editor. See File / Convert and save as JSON.

There is also a CreateAPI option to sort properties alphabetically which is disabled by default:

entities:
  isSortingPropertiesAlphabetically: true

from createapi.

The primary objective of this revision is to bring YAML into compliance with JSON as an official subset.

— YAML 1.2 Specification (2009)

It should be perfectly valid to use YAMLDecoder for json documents.
This of course hinges on how closely Yams follows the spec.

I didn't run into any issues with the OpenAPI specs I tried (two small internal ones), but that feels like too small of a sample size to conclusively rule out any nasty edge cases.

from createapi.

If retaining order is important for your use-case, I recommend the Yams and FineJSON libraries for YAML and JSON respectively. Also keep in mind that JSON is entirely valid YAML and therefore you will likely get good results from Yams decoding of JSON as well (it just won't encode as valid JSON).

— OpenAPIKit Documentation

Not sure if "you will likely get good results" is good enough, but I'd argue that entity generation with JSON as input should currently be considered broken, as its behavior is unexpected for anyone unfamiliar with the JSON specification.

Would you be interested in a PR that switches the JSON parser over to Yams and adds tests?
It wouldn't be necessary to check in any JSON specs, as decoding a YAML spec with Yams and consequently encoding it with JSONEncoder does retain order.

from createapi.

should currently be considered broke

As in the ordering, or are there more issues?

Would you be interested in a PR that switches the JSON parser over to Yams and adds tests?

Yeah, absolutely. Let's give it a try. If it resolved the ordering issue, that would be fantastic.

I have a private repo with 1KK lines of specs, but they are all YAML. I'll give you access in case you need it. It's not perfectly structured. It was initially part of this project, so I just copied it and symlinked the sources for now.

But I think adding a JSON+YAML test to this repo should be enough.

from createapi.

Fixed in 0.0.4! Thanks 🚀

from createapi.