refactor(core): Remove getOperationName helper from @urql/core

.changeset/quick-seahorses-cross.md

@@ -0,0 +1,5 @@
+---
+'@urql/core': patch
+---
+
+Fix incorrect operation name being picked from queries that contain multiple operations.

.changeset/young-lamps-help.md

@@ -0,0 +1,6 @@
+---
+'@urql/core': major
+'@urql/exchange-execute': minor
+---
+
+Remove `getOperationName` export from `@urql/core`

exchanges/execute/src/execute.test.ts

@@ -33,7 +33,6 @@ import {
   makeErrorResult,
   makeOperation,
   Client,
-  getOperationName,
   OperationResult,
 } from '@urql/core';
 
@@ -45,10 +44,8 @@ const exchangeArgs = {
   client: {},
 } as any;
 
-const expectedQueryOperationName = getOperationName(queryOperation.query);
-const expectedSubscribeOperationName = getOperationName(
-  subscriptionOperation.query
-);
+const expectedQueryOperationName = 'getUser';
+const expectedSubscribeOperationName = 'subscribeToUser';
 
 const fetchMock = (global as any).fetch as Mock;
 const mockHttpResponseData = { key: 'value' };

exchanges/execute/src/execute.ts

@@ -17,6 +17,7 @@ import {
   subscribe,
   ExecutionArgs,
   SubscriptionArgs,
+  Kind,
 } from 'graphql';
 
 import {
@@ -27,7 +28,6 @@ import {
   mergeResultPatch,
   Operation,
   OperationResult,
-  getOperationName,
 } from '@urql/core';
 
 export interface ExecuteExchangeArgs {
@@ -150,14 +150,22 @@ export const executeExchange = ({
           }
         }
 
+        let operationName: string | undefined;
+        for (const node of operation.query.definitions) {
+          if (node.kind === Kind.OPERATION_DEFINITION) {
+            operationName = node.name ? node.name.value : undefined;
+            break;
+          }
+        }
+
         return pipe(
           makeExecuteSource(operation, {
             schema,
             document: operation.query,
             rootValue,
             contextValue,
             variableValues,
-            operationName: getOperationName(operation.query),
+            operationName,
             fieldResolver,
             typeResolver,
             subscribeFieldResolver,

packages/core/src/index.ts

@@ -14,5 +14,4 @@ export {
   formatDocument,
   maskTypename,
   makeOperation,
-  getOperationName,
 } from './utils';

packages/core/src/utils/request.ts

@@ -169,8 +169,8 @@ export const createRequest = <
  */
 export const getOperationName = (query: DocumentNode): string | undefined => {
   for (const node of query.definitions) {
-    if (node.kind === Kind.OPERATION_DEFINITION && node.name) {
-      return node.name.value;
+    if (node.kind === Kind.OPERATION_DEFINITION) {
+      return node.name ? node.name.value : undefined;
     }
   }
 };