[useMaskedErrors] replace EnvelopError with GraphQLError

[n1ru4l]

Is your feature request related to a problem? Please describe.

Right now we use EnvelopError which extends GraphQLError for identifying whether an error should be masked or not.

https://github.com/dotansimha/envelop/blob/7e90676ce7b90a1880d5b06e18b6694819aed1fa/packages/core/src/plugins/use-masked-errors.ts#L17

We choose to use a custom EnvelopError because the GraphQLError constructor API was cumbersome. We changed this within graphql/graphql-js#3454

Instead of

throw new GraphQLError("Oops.", undefined, undefined, undefined, undefined, { code: "SERVER_GOES_BRRT" })

We can now instantiate a GraphQLError with extensions via the new object constructor overload:

throw new GraphQLError("Oops.", {
  extensions: { code: "SERVER_GOES_BRRT" },
})

Describe the solution you'd like

Replace EnvelopError with GraphQLError. This might be a breaking change though we could potentially work around it by just alias exporting GraphQLError as EnvelopError 🤔 .

import { GraphQLError } from "graphql";

export const EnvelopError = GraphQLError

However, since the constructor changes that would still be a breaking change.

[saihaj]

in engine agnostic envelop #1487 we dropped useMaskedError plugin from core since that required relying on GraphQLError class. Do we want to create a new plug-in for this?

[RIP21]

BTW docs are still outdated and say EnvelopError :)