.validate(schema) should not dereference the schema

Validator should only check if the schema is correct, but validate() modified the input object.

(Oh my it spent my a whole hour to find validate() mutate the object…)

Workaround: validate(_.cloneDeep(schema))

[Jack-Works]

You’re right, there are always something community will argue about I’ll wait for the isolated validator, thank you

On Tue, Jan 9, 2018, 22:31 James Messinger [e-mail] wrote:

I agree that mutating the object is bad behavior. But here are the reasons that I chose to do it that way:

1. I don’t want to bloat the library by adding a deep-clone library
2. There are many different deep-cloning algorithms, each with their own trade-offs, and regardless of which one I chose, some people would complain
3. Most people use Swagger Parser to parse/validate files, not in-memory objects. So this mutation behavior only affects a minority of users

That said, there is a solution coming soon. In an upcoming release, I’ll be removing the validate() method altogether and moving it to a separate package. So the swagger-parser will only contain logic for reading, parsing, resolving, and dereferencing. The swagger-validator package (or whatever I end up calling it) will contain the validation logic. There are several benefits to this:

1. It reduces the size of the swagger-parser package significantly, since its largest dependency is the JSON Schema validator. This is especially valuable for people who don’t need the validate() method.
2. It decouples the parsing and validation logic
3. It gets closer to the Node principle of small, single-purpose packages
4. It allows people to write their own validators on top of swagger-parser. So if want to use a different JSON Schema parser, or bundle-in a deep-cloning algorithm, you can do that.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/BigstickCarpet/swagger-parser/issues/80#issuecomment-356299811, or mute the thread https://github.com/notifications/unsubscribe-auth/AFJBf1mq2aJTezIGDgQoWHUZ3ekLukYFks5tI3hCgaJpZM4RXyTc .

[JamesMessinger]

I agree that mutating the object is bad behavior. But here are the reasons that I chose to do it that way:

1. I don’t want to bloat the library by adding a deep-clone library
2. There are many different deep-cloning algorithms, each with their own trade-offs, and regardless of which one I chose, some people would complain
3. Most people use Swagger Parser to parse/validate files, not in-memory objects. So this mutation behavior only affects a minority of users

That said, there is a solution coming soon. In an upcoming release, I’ll be removing the validate() method altogether and moving it to a separate package. So the swagger-parser will only contain logic for reading, parsing, resolving, and dereferencing. The swagger-validator package (or whatever I end up calling it) will contain the validation logic. There are several benefits to this:

1. It reduces the size of the swagger-parser package significantly, since its largest dependency is the JSON Schema validator. This is especially valuable for people who don’t need the validate() method.
2. It decouples the parsing and validation logic
3. It gets closer to the Node principle of small, single-purpose packages
4. It allows people to write their own validators on top of swagger-parser. So if want to use a different JSON Schema parser, or bundle-in a deep-cloning algorithm, you can do that.