coerce is a little over zealous

[deanlandolt]

it looks like the patching of json-schema's coerce is a little over zealous -- it will coerce for required values that don't actually have keys, which causes inserts that should fail to succeed with, for instance, an empty string

[kriszyp]

So undefined shouldn't be coerced?

[deanlandolt]

well, void 0 should probably, sure...but !key in object shouldn't be, if you know what i'm saying...

it forces all keys to exist (even the required ones) and creates false positives in schema validation

[deanlandolt]

and shouldn't it be an opt-in? like, shouldn't you have to have a coerce: true member on your schema descriptor?

[kriszyp]

Does this fix it?
kriszyp/commonjs-utils@f9ed902

[deanlandolt]

perfect...if you think it's better to coerce by default -- i guess it doesn't hurt in a postel kinda way, perhaps a "coerce" config setting would also be useful...just a thought, but this works for me