(chore) - remove useDevtoolsContext (#387)

* (chore) - remove usage of useDevtoolsContext

* (chore) - remove tests relate'd to useDevtoolsContext

* (chore) - save size by not spreading context

* (fix) - fix test by defaulting to an empty object

* (fix) - ensure useMutation executes with an empty object as context to prevent potential crashes

src/hooks/index.ts

@@ -1,4 +1,3 @@
 export * from './useMutation';
 export * from './useQuery';
 export * from './useSubscription';
-export * from './useDevtoolsContext';

src/hooks/useMutation.test.tsx

@@ -107,16 +107,6 @@ describe('on execute', () => {
     );
   });
 
-  it('calls executeMutation with source component info', () => {
-    renderer.create(<MutationUser {...props} />);
-    act(() => {
-      execute(vars);
-    });
-    expect(client.executeMutation.mock.calls[0][1]).toHaveProperty('meta', {
-      source: 'MutationUser',
-    });
-  });
-
   it('can adjust context in executeMutation', () => {
     renderer.create(<MutationUser {...props} />);
     act(() => {

src/hooks/useMutation.ts

@@ -5,7 +5,6 @@ import { Context } from '../context';
 import { OperationResult, OperationContext } from '../types';
 import { CombinedError, createRequest } from '../utils';
 import { useImmediateState } from './useImmediateState';
-import { useDevtoolsContext } from './useDevtoolsContext';
 
 export interface UseMutationState<T> {
   fetching: boolean;
@@ -22,7 +21,6 @@ export type UseMutationResponse<T, V> = [
 export const useMutation = <T = any, V = object>(
   query: DocumentNode | string
 ): UseMutationResponse<T, V> => {
-  const devtoolsContext = useDevtoolsContext();
   const client = useContext(Context);
   const [state, setState] = useImmediateState<UseMutationState<T>>({
     fetching: false,
@@ -43,15 +41,15 @@ export const useMutation = <T = any, V = object>(
       const request = createRequest(query, variables as any);
 
       return pipe(
-        client.executeMutation(request, { ...devtoolsContext, ...context }),
+        client.executeMutation(request, context || {}),
         toPromise
       ).then(result => {
         const { data, error, extensions } = result;
         setState({ fetching: false, data, error, extensions });
         return result;
       });
     },
-    [client, devtoolsContext, query, setState]
+    [client, query, setState]
   );
 
   return [state, executeMutation];

src/hooks/useQuery.spec.ts

@@ -80,7 +80,6 @@ describe('useQuery', () => {
         variables: mockVariables,
       },
       {
-        meta: { source: 'TestHook' },
         requestPolicy: undefined,
         url: 'test',
       }

src/hooks/useQuery.test.tsx

@@ -88,19 +88,6 @@ describe('on initial useEffect', () => {
       })
     );
   });
-
-  it('passes source component name to executeQuery', () => {
-    renderer.create(<QueryUser {...props} />);
-
-    expect(client.executeQuery).toBeCalledWith(
-      expect.any(Object),
-      expect.objectContaining({
-        meta: {
-          source: 'QueryUser',
-        },
-      })
-    );
-  });
 });
 
 describe('on subscription', () => {

src/hooks/useQuery.ts

@@ -4,7 +4,6 @@ import { pipe, subscribe } from 'wonka';
 import { Context } from '../context';
 import { OperationContext, RequestPolicy } from '../types';
 import { CombinedError, noop } from '../utils';
-import { useDevtoolsContext } from './useDevtoolsContext';
 import { useRequest } from './useRequest';
 import { useImmediateEffect } from './useImmediateEffect';
 import { useImmediateState } from './useImmediateState';
@@ -33,7 +32,6 @@ export type UseQueryResponse<T> = [
 export const useQuery = <T = any, V = object>(
   args: UseQueryArgs<V>
 ): UseQueryResponse<T> => {
-  const devtoolsContext = useDevtoolsContext();
   const unsubscribe = useRef(noop);
   const client = useContext(Context);
 
@@ -62,7 +60,6 @@ export const useQuery = <T = any, V = object>(
           pollInterval: args.pollInterval,
           ...args.context,
           ...opts,
-          ...devtoolsContext,
         }),
         subscribe(({ data, error, extensions }) => {
           setState({ fetching: false, data, error, extensions });
@@ -74,7 +71,6 @@ export const useQuery = <T = any, V = object>(
       args.requestPolicy,
       args.pollInterval,
       client,
-      devtoolsContext,
       request,
       setState,
     ]

src/hooks/useSubscription.test.tsx

@@ -62,14 +62,6 @@ describe('on initial useEffect', () => {
       expect.any(Object)
     );
   });
-
-  it('passes source component info to executeSubscription', () => {
-    renderer.create(<SubscriptionUser q={query} />);
-    expect(client.executeSubscription).toBeCalledWith(
-      expect.any(Object),
-      expect.objectContaining({ meta: { source: 'SubscriptionUser' } })
-    );
-  });
 });
 
 it('should support setting context in useSubscription params', () => {
@@ -85,7 +77,6 @@ it('should support setting context in useSubscription params', () => {
     },
     {
       url: 'test',
-      meta: { source: 'SubscriptionUser' },
     }
   );
 });

src/hooks/useSubscription.ts

@@ -3,7 +3,6 @@ import { useCallback, useContext, useEffect, useRef } from 'react';
 import { pipe, subscribe } from 'wonka';
 import { Context } from '../context';
 import { CombinedError, noop } from '../utils';
-import { useDevtoolsContext } from './useDevtoolsContext';
 import { useRequest } from './useRequest';
 import { useImmediateState } from './useImmediateState';
 import { OperationContext } from '../types';
@@ -29,7 +28,6 @@ export const useSubscription = <T = any, R = T, V = object>(
   args: UseSubscriptionArgs<V>,
   handler?: SubscriptionHandler<T, R>
 ): UseSubscriptionResponse<R> => {
-  const devtoolsContext = useDevtoolsContext();
   const unsubscribe = useRef(noop);
   const client = useContext(Context);
 
@@ -48,10 +46,7 @@ export const useSubscription = <T = any, R = T, V = object>(
     unsubscribe.current();
 
     [unsubscribe.current] = pipe(
-      client.executeSubscription(request, {
-        ...devtoolsContext,
-        ...args.context,
-      }),
+      client.executeSubscription(request, args.context || {}),
       subscribe(({ data, error, extensions }) => {
         setState(s => ({
           fetching: true,
@@ -61,7 +56,7 @@ export const useSubscription = <T = any, R = T, V = object>(
         }));
       })
     );
-  }, [client, devtoolsContext, handler, request, setState, args.context]);
+  }, [client, handler, request, setState, args.context]);
 
   // Trigger subscription on query change
   // We don't use useImmediateEffect here as we have no way of