Question: How can we rely on specific fields in rules? about graphql-shield

[kratam]

Yes, thank you! For anyone else having the same problem, it should look like this:

const { fragmentReplacements, schema } = applyMiddleware(
  plainSchema,
  permissions,
)

// create transforms array for transformSchema
const transforms = [new ReplaceFieldWithFragment(schema, fragmentReplacements)]
// apply transformations
const finalSchema = transformSchema(schema, transforms)

[maticzav]

Hey @nemcek 👋

That's an interesting question, and indeed you are right! The current version doesn't support fragments. Fragments are a way of telling graphql resolver which information your resolver requires to work correctly.

The next version of shield should introduce this with the following syntax. Right now, we fully depend on graphql-tools PR and grpahql-middleware fragments support which should land in no time.

const nestedRule = rule({
  cache: 'strict',
  fragment: 'fragment UserID on User { id }',
})(async ({ id }, args, ctx, info) => {
  return somethingWithID(id)
})

I hope this covers your idea. Tell me if you have any thoughts on this as well! 🙂

[maticzav]

Hey @kratam 👋

Could you also provide schema and rules? Also, you should note thay you have to use fragmentReplacements returned by middleware; maybe we should indicate that in the docs.

[lastmjs]

What libraries are transformSchema and ReplaceFieldWithFragment from?

[lastmjs]

I believe transformSchema from graphql-tools is now called wrapSchema

[nemcek]

Oh, sweet!

Just to be sure I understood it correctly. In the next version if I use rule with a fragment, for example as:

const isOwner = rule({
    fragment: 'fragment UserID on User { id }',
})(async (parent, args, ctx, info) => {
  return ctx.user.items.some(id => id === parent.id)
})

then I can be sure that parent.id will be populated, right?
If that's the case then shield solves all my use-cases. Love it!

[maticzav]

Yeah, that’s exactly that! Perfect 🚀🎉

[kratam]

Is it live? parent is always undefined even when I set cache: 'strict' (the fragment part doesn't even matter):

const isOwner = rule({
  cache: 'strict',
  fragment: 'fragment Id on Item { id }',
})(async (parent, args, ctx: Context, info) => {
  console.log(parent) // undefined
  return true
})

Note: I'm using:

"apollo-server-express": "^2.0.5",
"graphql-middleware": "^1.7.1",
"graphql-shield": "^3.2.0",

[kratam]

Wow, super fast response! :) How can I pass fragmentReplacements to ApolloServer?

[maticzav]

Hey 👋

Sorry for the delayed response. fragmentReplacements are part of schema forwarding; graphql-binding supports this using fragmentReplacements parameter.

Besides that, you could also use standalone transforms which are supported by ApolloServer, especially ReplaceFieldWithFragment

https://www.apollographql.com/docs/graphql-tools/schema-transforms.html#Other

I hope this helps you with your problem 🙂

[forrestwilkins]

It looks like ReplaceFieldWithFragment should be accessible from @graphql-tools/delegate according to this thread, but I'm getting the following error:
Module '"@graphql-tools/delegate"' has no exported member 'ReplaceFieldWithFragment'

[forrestwilkins]

Has anyone been able to configure fragmentReplacements for NestJS and Apollo Server? It looks like ReplaceFieldWithFragment is deprecated now.