Duplicate 'Queries' module generated

[hikurangi]

We've been diving into this library and have discovered an issue:

When running graphql-codegen, it generates a Queries module in both Apollo.re and Graphql.re, potentially unnecessarily(?). That yields the following error:

We've found a bug for you!
 /home/user/repo/graphql/Apollo.re 11:8-14
  9 │ });
 10 │
 11 │ module Queries = {
 12 │   module MyQuery = {
 13 │     include MakeQuery({
 Multiple definition of the module name Queries.
 Names must be unique in a given structure or signature.

Our workaround is to comment the offending Queries module out of the Graphql.re file, allowing us to run the codegen command successfully.

[kgoggin]

Thanks for filing this - sorry it's taken me a bit to respond!

Graphql.re will create a module for each and every type in your underlying schema, and Apollo.re will always contain a Queries and a Mutations module to namespace the modules for each of your operations. Do you have a type in your schema called Queries, perhaps?

I hadn't contemplated the namespace collision here. I think the best solution I can think of would be to allow a config option to not include Graphql.re inside Apollo.re - that way even if you have overlap they won't collide. I added the include there for convenience, really - it's not strictly necessary. Thoughts?