Extend type with cacheControl doesn't work

[eduardojmatos]

Hi people! It's my first issue here, and I have a specific problem encountered on my project using stitching and cache with apollo engine.

We're extending types to add some fields. This is the scenario:

graph/schemas/extend/category.js

export default `
  extend type Category @cacheControl(maxAge: 1000, scope: PUBLIC) {
    services(first: Int): ServiceConnection @cacheControl(maxAge: 2000, scope: PUBLIC)
  }
`;

The field services is a stitching of another server that has a ServiceConnection type. We make the realtionship with Category is a categorySlug field. But this is not the problem!

The result, on the "extensions" property return this:

{
  "cacheControl": {
    "version": 1,
    "hints": [
      {
        "path": [
          "category"
        ],
        "maxAge": 0
      },
      {
        "path": [
          "category",
          "services"
        ],
        "maxAge": 2000,
        "scope": "PUBLIC"
      }
    ]
  }
}

The maxAge of path category it's not set as 1000. I don't know if this is the expected behavior.

[januszhou]

Having the same issue here, any suggestions?

[hastarin]

I'd also tried this. It would be great to proxy a remote schema, such as AppSync which doesn't allow custom directives, and stitch in cacheControl directives to then serve it up via Apollo Server.

Sadly I ran into a number of issues.

The first being with graphql@14.0.x there is an SDL validation failure when trying to merge for the custom directive which I was unable to get around.

After downgrading I then hit the same issue reported here.

[yaacovCR]

This seems to be a bug within the cacheControl extension support by Apollo. A workaround would be to create a way to enhance using directives as metadata, possibly, for example, automatically copying the directives from the extensionAstNodes to the primary astNode.

[yaacovCR]

Related, but different, what if you can't even use extend, like with existing fields, as in #451.

[yaacovCR]

You would use mergeSchemas which should hopefully merge directives...

Closing for now as there is no bug in tools...