Integers as Discriminators about ajv

[b-wils]

https://runkit.com/b-wils/ajv-number-discriminator

[jasoniangreen]

Thanks for that. So after reviewing things I can see that it is indeed a deliberate restriction and there was actually a PR a while back to add support for other scalar data types.

I am just in the process of preparing a release (the first in a long time) but I can look into this afterwards and follow up with @epoberezkin.

[jasoniangreen]

Hi there, could you put together a runkit exhibiting the issue? Try and keep the example as simple as possible.

[jasoniangreen]

Judging by the example here https://ajv.js.org/json-schema.html#discriminator your schema is not quite right.

Looks like you are missing propertyName in discriminator: {propertyName: "foo"}.

[b-wils]

Ah yes, my apologies. https://runkit.com/b-wils/ajv-number-discriminator is updated with the correct discriminator declaration and now should demonstrate the issue with numbers as discriminators.

[jasoniangreen]

Hi @b-wils, I have spoken to EP and this is not a change we're open to exploring at this time. I don't have all the context but there are reasons it should be a string and my interpretation of the spec is that it should be a string too. It should be pretty trivial to work around too.

[b-wils]

Thanks for looking into it, if this is isn't part of the spec then that makes sense to not extend it in ajv.

By workaround, you mean change the fields from numbers to strings? Unfortunately our data is already in use and won't be feasible to change existing users. Replacing oneOf with if/then is a (slightly awkward) workaround to reduce error noise, or we may just live with the noise.