refactor(execution): add separate entrypoints file (#4527)

yaacovCR · yaacovCR · commit 34f5c766108a · 2026-02-24T20:40:36.000+02:00
cleans up our long execute file (a bit)

it's potentially a breaking change if these entrypoints were being
imported directly; we will clean that up later by renaming execute.ts to
something else and renaming entrypoints.ts to execute.ts, but removing
the entrypoints first will hopefully allow us to preserve more history.
diff --git a/benchmark/introspectionFromSchema-benchmark.js b/benchmark/introspectionFromSchema-benchmark.js
@@ -1,4 +1,4 @@
-import { executeSync } from 'graphql/execution/execute.js';
+import { executeSync } from 'graphql/execution/entrypoints.js';
 import { parse } from 'graphql/language/parser.js';
 import { buildSchema } from 'graphql/utilities/buildASTSchema.js';
 import { getIntrospectionQuery } from 'graphql/utilities/getIntrospectionQuery.js';
diff --git a/benchmark/list-async-benchmark.js b/benchmark/list-async-benchmark.js
@@ -1,4 +1,4 @@
-import { execute } from 'graphql/execution/execute.js';
+import { execute } from 'graphql/execution/entrypoints.js';
 import { parse } from 'graphql/language/parser.js';
 import { buildSchema } from 'graphql/utilities/buildASTSchema.js';
 
diff --git a/benchmark/list-asyncIterable-benchmark.js b/benchmark/list-asyncIterable-benchmark.js
@@ -1,4 +1,4 @@
-import { execute } from 'graphql/execution/execute.js';
+import { execute } from 'graphql/execution/entrypoints.js';
 import { parse } from 'graphql/language/parser.js';
 import { buildSchema } from 'graphql/utilities/buildASTSchema.js';
 
diff --git a/benchmark/list-sync-benchmark.js b/benchmark/list-sync-benchmark.js
@@ -1,4 +1,4 @@
-import { execute } from 'graphql/execution/execute.js';
+import { execute } from 'graphql/execution/entrypoints.js';
 import { parse } from 'graphql/language/parser.js';
 import { buildSchema } from 'graphql/utilities/buildASTSchema.js';
 
diff --git a/src/execution/__tests__/abstract-test.ts b/src/execution/__tests__/abstract-test.ts
@@ -17,7 +17,7 @@ import { GraphQLSchema } from '../../type/schema.js';
 
 import { buildSchema } from '../../utilities/buildASTSchema.js';
 
-import { execute, executeSync } from '../execute.js';
+import { execute, executeSync } from '../entrypoints.js';
 
 interface Context {
   async: boolean;
diff --git a/src/execution/__tests__/cancellation-test.ts b/src/execution/__tests__/cancellation-test.ts
@@ -11,7 +11,7 @@ import { parse } from '../../language/parser.js';
 
 import { buildSchema } from '../../utilities/buildASTSchema.js';
 
-import { execute, subscribe } from '../execute.js';
+import { execute, subscribe } from '../entrypoints.js';
 
 const schema = buildSchema(`
   type Todo {
diff --git a/src/execution/__tests__/directives-test.ts b/src/execution/__tests__/directives-test.ts
@@ -7,7 +7,7 @@ import { GraphQLObjectType } from '../../type/definition.js';
 import { GraphQLString } from '../../type/scalars.js';
 import { GraphQLSchema } from '../../type/schema.js';
 
-import { executeSync } from '../execute.js';
+import { executeSync } from '../entrypoints.js';
 
 const schema = new GraphQLSchema({
   query: new GraphQLObjectType({
diff --git a/src/execution/__tests__/errorPropagation-test.ts b/src/execution/__tests__/errorPropagation-test.ts
@@ -8,8 +8,8 @@ import { parse } from '../../language/parser.js';
 
 import { buildSchema } from '../../utilities/buildASTSchema.js';
 
+import { execute } from '../entrypoints.js';
 import type { ExecutionResult } from '../execute.js';
-import { execute } from '../execute.js';
 
 const syncError = new Error('bar');
 
diff --git a/src/execution/__tests__/executor-test.ts b/src/execution/__tests__/executor-test.ts
@@ -25,7 +25,7 @@ import {
 } from '../../type/scalars.js';
 import { GraphQLSchema } from '../../type/schema.js';
 
-import { execute, executeSync } from '../execute.js';
+import { execute, executeSync } from '../entrypoints.js';
 
 describe('Execute: Handles basic execution tasks', () => {
   it('executes arbitrary code', async () => {
diff --git a/src/execution/__tests__/lists-test.ts b/src/execution/__tests__/lists-test.ts
@@ -18,8 +18,8 @@ import { GraphQLSchema } from '../../type/schema.js';
 
 import { buildSchema } from '../../utilities/buildASTSchema.js';
 
+import { execute, executeSync } from '../entrypoints.js';
 import type { ExecutionResult } from '../execute.js';
-import { execute, executeSync } from '../execute.js';
 
 describe('Execute: Accepts any iterable as list value', () => {
   function complete(rootValue: unknown) {
diff --git a/src/execution/__tests__/mutations-test.ts b/src/execution/__tests__/mutations-test.ts
@@ -10,7 +10,7 @@ import { GraphQLObjectType } from '../../type/definition.js';
 import { GraphQLInt } from '../../type/scalars.js';
 import { GraphQLSchema } from '../../type/schema.js';
 
-import { execute, executeSync } from '../execute.js';
+import { execute, executeSync } from '../entrypoints.js';
 
 class NumberHolder {
   theNumber: number;
diff --git a/src/execution/__tests__/nonnull-test.ts b/src/execution/__tests__/nonnull-test.ts
@@ -16,8 +16,8 @@ import { GraphQLSchema } from '../../type/schema.js';
 
 import { buildSchema } from '../../utilities/buildASTSchema.js';
 
+import { execute, executeSync } from '../entrypoints.js';
 import type { ExecutionResult } from '../execute.js';
-import { execute, executeSync } from '../execute.js';
 
 const syncError = new Error('sync');
 const syncNonNullError = new Error('syncNonNull');
diff --git a/src/execution/__tests__/oneof-test.ts b/src/execution/__tests__/oneof-test.ts
@@ -6,8 +6,8 @@ import { parse } from '../../language/parser.js';
 
 import { buildSchema } from '../../utilities/buildASTSchema.js';
 
+import { execute } from '../entrypoints.js';
 import type { ExecutionResult } from '../execute.js';
-import { execute } from '../execute.js';
 
 const schema = buildSchema(`
   type Query {
diff --git a/src/execution/__tests__/resolve-test.ts b/src/execution/__tests__/resolve-test.ts
@@ -8,7 +8,7 @@ import { GraphQLObjectType } from '../../type/definition.js';
 import { GraphQLInt, GraphQLString } from '../../type/scalars.js';
 import { GraphQLSchema } from '../../type/schema.js';
 
-import { executeSync } from '../execute.js';
+import { executeSync } from '../entrypoints.js';
 
 describe('Execute: resolve function', () => {
   function testSchema(testField: GraphQLFieldConfig<any, any>) {
diff --git a/src/execution/__tests__/schema-test.ts b/src/execution/__tests__/schema-test.ts
@@ -16,7 +16,7 @@ import {
 } from '../../type/scalars.js';
 import { GraphQLSchema } from '../../type/schema.js';
 
-import { executeSync } from '../execute.js';
+import { executeSync } from '../entrypoints.js';
 
 describe('Execute: Handles execution with a complex schema', () => {
   it('executes using a schema', () => {
diff --git a/src/execution/__tests__/subscribe-test.ts b/src/execution/__tests__/subscribe-test.ts
@@ -20,12 +20,13 @@ import {
 } from '../../type/scalars.js';
 import { GraphQLSchema } from '../../type/schema.js';
 
-import type { ExecutionArgs, ExecutionResult } from '../execute.js';
+import type { ExecutionArgs } from '../entrypoints.js';
 import {
   createSourceEventStream,
   executeSubscriptionEvent,
   subscribe,
-} from '../execute.js';
+} from '../entrypoints.js';
+import type { ExecutionResult } from '../execute.js';
 
 import { SimplePubSub } from './simplePubSub.js';
 
diff --git a/src/execution/__tests__/sync-test.ts b/src/execution/__tests__/sync-test.ts
@@ -13,7 +13,7 @@ import { validate } from '../../validation/validate.js';
 
 import { graphqlSync } from '../../graphql.js';
 
-import { execute, executeSync } from '../execute.js';
+import { execute, executeSync } from '../entrypoints.js';
 
 describe('Execute: synchronously when possible', () => {
   const schema = new GraphQLSchema({
diff --git a/src/execution/__tests__/union-interface-test.ts b/src/execution/__tests__/union-interface-test.ts
@@ -12,7 +12,7 @@ import {
 import { GraphQLBoolean, GraphQLString } from '../../type/scalars.js';
 import { GraphQLSchema } from '../../type/schema.js';
 
-import { execute, executeSync } from '../execute.js';
+import { execute, executeSync } from '../entrypoints.js';
 
 class Dog {
   name: string;
diff --git a/src/execution/__tests__/variables-test.ts b/src/execution/__tests__/variables-test.ts
@@ -32,7 +32,7 @@ import { GraphQLSchema } from '../../type/schema.js';
 
 import { valueFromASTUntyped } from '../../utilities/valueFromASTUntyped.js';
 
-import { executeSync } from '../execute.js';
+import { executeSync } from '../entrypoints.js';
 import { getVariableValues } from '../values.js';
 
 const TestFaultyScalarGraphQLError = new GraphQLError(
diff --git a/src/execution/entrypoints.ts b/src/execution/entrypoints.ts
diff --git a/src/execution/execute.ts b/src/execution/execute.ts
diff --git a/src/execution/index.ts b/src/execution/index.ts
diff --git a/src/graphql.ts b/src/graphql.ts
diff --git a/src/utilities/introspectionFromSchema.ts b/src/utilities/introspectionFromSchema.ts

Original file line number	Diff line number	Diff line change
`@@ -1,4 +1,4 @@`
`1`		`-import { executeSync } from 'graphql/execution/execute.js';`
	`1`	`+import { executeSync } from 'graphql/execution/entrypoints.js';`
`2`	`2`	`import { parse } from 'graphql/language/parser.js';`
`3`	`3`	`import { buildSchema } from 'graphql/utilities/buildASTSchema.js';`
`4`	`4`	`import { getIntrospectionQuery } from 'graphql/utilities/getIntrospectionQuery.js';`
Original file line number	Diff line number	Diff line change
`@@ -1,4 +1,4 @@`
`1`		`-import { execute } from 'graphql/execution/execute.js';`
	`1`	`+import { execute } from 'graphql/execution/entrypoints.js';`
`2`	`2`	`import { parse } from 'graphql/language/parser.js';`
`3`	`3`	`import { buildSchema } from 'graphql/utilities/buildASTSchema.js';`
`4`	`4`