zap context.errors

src/execution/execute.js

@@ -94,7 +94,6 @@ export type ExecutionContext = {
  operation: OperationDefinitionNode,
  variableValues: { [variable: string]: mixed },
  fieldResolver: GraphQLFieldResolver<any, any>,
- errors: Array<GraphQLError>,
 };
 
 /**
@@ -235,21 +234,17 @@ function executeImpl(
  // field and its descendants will be omitted, and sibling fields will still
  // be executed. An execution which encounters errors will still result in a
  // resolved Promise.
- const result = executeOperation(context, context.operation, rootValue);
- return buildResponse(context, result);
+ const data = executeOperation(context, context.operation, rootValue);
+ return buildResponse(data);
 }
 
 /**
- * Given a completed execution context and data, build the { errors, data }
- * response defined by the "Response" section of the GraphQL specification.
+ * Strip out `errors` if empty.
  */
-function buildResponse(
- context: ExecutionContext,
- result: ExecutionPartialResult<mixed>,
-) {
+function buildResponse(result: ExecutionPartialResult<mixed>) {
  const promise = getPromise(result);
  if (promise) {
- return promise.then(resolved => buildResponse(context, resolved));
+ return promise.then(resolved => buildResponse(resolved));
  }
  if (result.data && (!result.errors || !result.errors.length)) {
  return { data: result.data };
@@ -389,7 +384,6 @@ export function buildExecutionContext(
  operation,
  variableValues,
  fieldResolver: fieldResolver || defaultFieldResolver,
- errors,
  };
 }