Mesh v1 encapsulate and filter transform doesn't remove fields

[ziqiq]

Mesh v1 createEncapsulateTransform and createFilterTransform add @inaccessible directive for the fields being removed, instead of actually removing the fields. This causes naming conflict if subgraphs relies on them for namespacing.

Is that intentional or bug?

[alexfiu4-cyber]

i workaround this with another renamer:

export const renameInaccessibleMethods = ((): FieldRenamer =>
        (opts) => {
            let newFieldName: string = ''
            if (opts.field.extensions && ("inaccessible" in <object>opts.field.extensions["directives"])) {
                    newFieldName = opts.fieldName + 'DoNotUse'
            }
            return newFieldName
        }
)()