Use full path in json definition generation about schemars

[miraclx]

Was just about to submit a ticket for this. This is really relevant. We can probably come up with a different key discriminant than the type name here:

schemars/schemars/src/gen.rs

Lines 225 to 227 in 5268080

	if !self.definitions.contains_key(&name) {
	self.insert_new_subschema_for::<T>(name);
	}

Perhaps a Hash of std::any::TypeId?

[ewoolsey]

Commenting here to mention that I have also run into this problem. Would love to see a fix for this, Should be fairly straightforward I imagine.

[miraclx]

Hi @ewoolsey, #178 is a potential fix. Although, as highlighted in there, I believe a different approach should be considered. An approach that, in-turn, won't be a simple fix.

[ewoolsey]

Hi @ewoolsey, #178 is a potential fix. Although, as highlighted in there, I believe a different approach should be considered. An approach that, in-turn, won't be a simple fix.

Haha things are never as straightforward as they first seem. @miraclx any Idea when we can expect either of these solutions to be implemented? I'm working on an interactive parser built on top of schemers and serde to generate complex types from user input. Using it to build some cool development tools. Should I proceed with a workaround or wait it out?

[miraclx]

@ewoolsey, depending on your use case, #178 should suffice for schema generation.

For schema consumption, however, clients have to read the title field for type names. Which, unless you're writing your own jsonschema consumer, probably won't use.

But if you are, it should work perfectly.

One last thing to note is that PR aids exhaustive schema generation.

But you should be careful about espousing reductionism when consuming it.

Effectively, you need to be careful not to remap it to {TypeName: TypeSchema}.

Avoiding these classes of errors both when generating and consuming the schema will require a different approach for encoding information in the jsonschema that helps clients further disambiguate types. Module paths might be an option. But that's tricky to extract from macros.

[ewoolsey]

Unfortunately because I'm dealing with a large library that implements JsonSchema on all its types, I'll have to wait for a new release on crates.io :/ Really appreciate the info though! And good luck finding a more robust solution!

[karelispanagiotis]

I encountered the same issue with structs having the same name in different mods.