[Question] Using libraries such as join-monster, which rely on mutating schema instance

[jbroadice]

Hi again.

I've recently discovered a few GraphQL libraries that work by decorating / mutating the schema (GraphQLSchema) instance, and depend on those changes being persisted and not overwritten, such that those decorations can be accessed in resolvers with the resolverInfo argument (fourth argument). For example, join-monster requires adding custom properties to GraphQLObjectType constructors in order to be configured. When using Apollo, this can be achieved using an adapter, such as join-monster-graphql-tools-adapter, which works by decorating / mutating the schema based on a custom object you pass to it.

As far as I can tell, I cannot rely on mutating the schema instance in such a way when using GraphQL Modules, as the schema are built using a factory method, which is called at strategic points in the control flow.

This has lead me to abandon the idea of being able to use such libraries with GraphQL Modules, viewing them as being idiomatically incompatible with one another. Is this an accurate conclusion, or have I missed something? Perhaps the solution could be as simple as formalising an interface for mutating / transforming a schema at a strategic point before the resolvers get called?

Cheers.

[ardatan]

We can add a field for this kind of middleware tools.

new GraphQLModule({
  // some definition
  schemaMiddleware: (schema: GraphQLSchema) => {
   //some logic here
   return modifiedSchema;
}
})

[jbroadice]

Great, that would do the trick!

[ardatan]

After v0.2.10 you can mutate schema by using

new GraphQLModule({
  // some definition
  middleware({schema}){
              //your logic
     return {
            schema
      };
   }
}
})

Let me know if that worked for you.

[jbroadice]

Thanks! I tried that, but it didn't seem to work.

As a test, I tried:

new GraphQLModule({
  // ...
  middleware({ schema }) {
    schema['__DIRTY__'] = true;
    return { schema };
  }
})

I see that the middleware function is getting called once, which seems to be when the GraphQLModule is initially bootstrapped.

However, when inspecting resolverInfo.schema from within a resolver, I see that the schema remains unchanged (e.g. I can't see the __DIRTY__ property).

[ardatan]

Hi @JChapple!
I added tests for your case, and it passes.
24eac3d

Hello!

Thanks for putting the test together. I've investigated further and discovered that the problem seems to occur when adding middleware to a module that is being imported by another module.

In my case, I have an entry AppModule:

export const AppModule = new GraphQLModule<AppModuleConfig, {}, AppModuleContext>({
  imports: () => [
    AuthModule,
  ],
});

and I am attempting to add the middleware to AuthModule, which I have stripped down for testing to:

export const AuthModule = new GraphQLModule<{}, AuthModuleRequest, AuthModuleContext>({
  typeDefs, // imported above

  resolvers: {
    Query: {
      me: (_root, _args, _context, info) => {
        console.log('resolver', info.schema);
      }
    }
  },

  middleware: ({ schema }) => {
    schema['__DIRTY__'] = true;
    return { schema };
  },
});

If ApolloServer is instantiated using AuthModule as the root entry point (e.g. const { schema, context } = AuthModule.forRoot({});), I see the __DIRTY__ property when resolving, in info.schema, as expected. However, if I instantiate Apollo using AppModule as the entry point, the resolver does not contain the mutated property.

[ardatan]

Could you try the following versions?

Changes:
 - @graphql-modules/core: 0.3.0-alpha.aca8a400
 - @graphql-modules/epoxy:  0.3.0-alpha.aca8a400
 - @graphql-modules/sonar: 0.3.0-alpha.aca8a400
 - @graphql-modules/utils:  0.3.0-alpha.aca8a400

Great, that seems to have fixed it. Many thanks! 🎉

[ardatan]

Great!!! So, we can close this issue then :)

[BeeHiveJava]

Could you try the following versions?

Changes:
 - @graphql-modules/core: 0.3.0-alpha.aca8a400
 - @graphql-modules/epoxy:  0.3.0-alpha.aca8a400
 - @graphql-modules/sonar: 0.3.0-alpha.aca8a400
 - @graphql-modules/utils:  0.3.0-alpha.aca8a400

I just tested this today and it does not seem to work. I tested versions 0.4.2 and 0.5.0-alpha.e4344183. The modified schema only seems to get through when the middleware is added to the root module (so changing the root module or moving the middleware to the root module).

I'm using the exact sample supplied above by @ghost so I'm not certain where to go from here @ardatan?

[ardatan]

#335 We are planning to change our merge logic, so you won't lose your modifications on child modules if you use middlewares.
Could you try it with the following versions?

Changes:
 - @graphql-modules/core: 0.4.2 => 0.5.0-alpha.b12e4322
 - @graphql-modules/di: 0.4.2 => 0.5.0-alpha.b12e4322