react-relay 15.0.0 → 16.1.0

package/lib/ReactRelayQueryRendererContext.js

@@ -1,14 +1,3 @@
-/**
- * Copyright (c) Meta Platforms, Inc. and affiliates.
- *
- * This source code is licensed under the MIT license found in the
- * LICENSE file in the root directory of this source tree.
- *
- * 
- * @format
- * @oncall relay
- */
-
 'use strict';
 
 var React = require('react');

package/lib/ReactRelayRefetchContainer.js

@@ -1,14 +1,3 @@
-/**
- * Copyright (c) Meta Platforms, Inc. and affiliates.
- *
- * This source code is licensed under the MIT license found in the
- * LICENSE file in the root directory of this source tree.
- *
- * 
- * @format
- * @oncall relay
- */
-
 'use strict';
 
 var _interopRequireDefault = require("@babel/runtime/helpers/interopRequireDefault")["default"];
@@ -38,31 +27,20 @@ var _require3 = require('relay-runtime'),
   getVariablesFromObject = _require3.getVariablesFromObject,
   isScalarAndEqual = _require3.isScalarAndEqual;
 var warning = require("fbjs/lib/warning");
-/**
- * Composes a React component class, returning a new class that intercepts
- * props, resolving them with the provided fragments and subscribing for
- * updates.
- */
 function createContainerWithFragments(Component, fragments, taggedNode) {
   var _class;
   var containerName = getContainerName(Component);
   return _class = /*#__PURE__*/function (_React$Component) {
     (0, _inheritsLoose2["default"])(_class, _React$Component);
-    // $FlowFixMe[missing-local-annot]
-
     function _class(props) {
       var _props$__rootIsQueryR, _this;
       _this = _React$Component.call(this, props) || this;
       (0, _defineProperty2["default"])((0, _assertThisInitialized2["default"])(_this), "_handleFragmentDataUpdate", function () {
         var resolverFromThisUpdate = _this.state.resolver;
         _this.setState(function (updatedState) {
-          return (
-            // If this event belongs to the current data source, update.
-            // Otherwise we should ignore it.
-            resolverFromThisUpdate === updatedState.resolver ? {
-              data: updatedState.resolver.resolve()
-            } : null
-          );
+          return resolverFromThisUpdate === updatedState.resolver ? {
+            data: updatedState.resolver.resolve()
+          } : null;
         });
       });
       (0, _defineProperty2["default"])((0, _assertThisInitialized2["default"])(_this), "_refetch", function (refetchVariables, renderVariables, observerOrCallback, options) {
@@ -85,22 +63,13 @@ function createContainerWithFragments(Component, fragments, taggedNode) {
           cacheConfig.metadata = options === null || options === void 0 ? void 0 : options.metadata;
         }
         var observer = typeof observerOrCallback === 'function' ? {
-          // callback is not exectued on complete or unsubscribe
-          // for backward compatibility
           next: observerOrCallback,
           error: observerOrCallback
         } : observerOrCallback || {};
         var query = getRequest(taggedNode);
         var operation = createOperationDescriptor(query, fetchVariables, cacheConfig);
-
-        // TODO: T26288752 find a better way
         _this.state.localVariables = fetchVariables;
-
-        // Cancel any previously running refetch.
         _this._refetchSubscription && _this._refetchSubscription.unsubscribe();
-
-        // Declare refetchSubscription before assigning it in .start(), since
-        // synchronous completion may call callbacks .subscribe() returns.
         var refetchSubscription;
         var storeSnapshot = _this._getQueryFetcher().lookupInStore(environment, operation, options === null || options === void 0 ? void 0 : options.fetchPolicy);
         if (storeSnapshot != null) {
@@ -123,7 +92,6 @@ function createContainerWithFragments(Component, fragments, taggedNode) {
         _this._getQueryFetcher().execute({
           environment: environment,
           operation: operation,
-          // TODO (T26430099): Cleanup old references
           preservePreviousReferences: true
         }).mergeMap(function (response) {
           _this.state.resolver.setVariables(fragmentVariables, operation.request.node);
@@ -141,8 +109,6 @@ function createContainerWithFragments(Component, fragments, taggedNode) {
             });
           });
         })["finally"](function () {
-          // Finalizing a refetch should only clear this._refetchSubscription
-          // if the finizing subscription is the most recent call.
           if (_this._refetchSubscription === refetchSubscription) {
             _this._refetchSubscription = null;
           }
@@ -161,10 +127,6 @@ function createContainerWithFragments(Component, fragments, taggedNode) {
       var relayContext = assertRelayContext(props.__relayContext);
       var rootIsQueryRenderer = (_props$__rootIsQueryR = props.__rootIsQueryRenderer) !== null && _props$__rootIsQueryR !== void 0 ? _props$__rootIsQueryR : false;
       _this._refetchSubscription = null;
-      // Do not provide a subscription/callback here.
-      // It is possible for this render to be interrupted or aborted,
-      // In which case the subscription would cause a leak.
-      // We will add the subscription in componentDidMount().
       var resolver = createFragmentSpecResolver(relayContext, containerName, fragments, props, rootIsQueryRenderer);
       _this.state = {
         data: resolver.resolve(),
@@ -184,11 +146,6 @@ function createContainerWithFragments(Component, fragments, taggedNode) {
       this._subscribeToNewResolverAndRerenderIfStoreHasChanged();
     };
     _proto.componentDidUpdate = function componentDidUpdate(prevProps, prevState) {
-      // If the environment has changed or props point to new records then
-      // previously fetched data and any pending fetches no longer apply:
-      // - Existing references are on the old environment.
-      // - Existing references are based on old variables.
-      // - Pending fetches are for the previous records.
       if (this.state.resolver !== prevState.resolver) {
         prevState.resolver.dispose();
         this._queryFetcher && this._queryFetcher.dispose();
@@ -197,16 +154,9 @@ function createContainerWithFragments(Component, fragments, taggedNode) {
       } else {
         this._rerenderIfStoreHasChanged();
       }
-    }
-
-    /**
-     * When new props are received, read data for the new props and add it to
-     * state. Props may be the same in which case previous data can be reused.
-     */;
+    };
     _class.getDerivedStateFromProps = function getDerivedStateFromProps(nextProps, prevState) {
       var _nextProps$__rootIsQu;
-      // Any props change could impact the query, so we mirror props in state.
-      // This is an unusual pattern, but necessary for this container usecase.
       var prevProps = prevState.prevProps;
       var relayContext = assertRelayContext(nextProps.__relayContext);
       var rootIsQueryRenderer = (_nextProps$__rootIsQu = nextProps.__rootIsQueryRenderer) !== null && _nextProps$__rootIsQu !== void 0 ? _nextProps$__rootIsQu : false;
@@ -215,17 +165,7 @@ function createContainerWithFragments(Component, fragments, taggedNode) {
       var prevRootVariables = getRootVariablesForFragments(fragments, prevProps);
       var nextRootVariables = getRootVariablesForFragments(fragments, nextProps);
       var resolver = prevState.resolver;
-
-      // If the environment has changed or props point to new records then
-      // previously fetched data and any pending fetches no longer apply:
-      // - Existing references are on the old environment.
-      // - Existing references are based on old variables.
-      // - Pending fetches are for the previous records.
       if (prevState.prevPropsContext.environment !== relayContext.environment || !areEqual(prevRootVariables, nextRootVariables) || !areEqual(prevIDs, nextIDs)) {
-        // Do not provide a subscription/callback here.
-        // It is possible for this render to be interrupted or aborted,
-        // In which case the subscription would cause a leak.
-        // We will add the subscription in componentDidUpdate().
         resolver = createFragmentSpecResolver(relayContext, containerName, fragments, nextProps, rootIsQueryRenderer);
         return {
           data: resolver.resolve(),
@@ -255,12 +195,9 @@ function createContainerWithFragments(Component, fragments, taggedNode) {
       this._refetchSubscription && this._refetchSubscription.unsubscribe();
     };
     _proto.shouldComponentUpdate = function shouldComponentUpdate(nextProps, nextState) {
-      // Short-circuit if any Relay-related data has changed
       if (nextState.data !== this.state.data || nextState.relayProp !== this.state.relayProp) {
         return true;
       }
-      // Otherwise, for convenience short-circuit if all non-Relay props
-      // are scalar and equal
       var keys = Object.keys(nextProps);
       for (var ii = 0; ii < keys.length; ii++) {
         var key = keys[ii];
@@ -280,8 +217,6 @@ function createContainerWithFragments(Component, fragments, taggedNode) {
       var _this$state = this.state,
         data = _this$state.data,
         resolver = _this$state.resolver;
-      // External values could change between render and commit.
-      // Check for this case, even though it requires an extra store read.
       var maybeNewData = resolver.resolve();
       if (data !== maybeNewData) {
         this.setState({
@@ -294,23 +229,13 @@ function createContainerWithFragments(Component, fragments, taggedNode) {
         data = _this$state2.data,
         resolver = _this$state2.resolver;
       var maybeNewData = resolver.resolve();
-
-      // Event listeners are only safe to add during the commit phase,
-      // So they won't leak if render is interrupted or errors.
       resolver.setCallback(this.props, this._handleFragmentDataUpdate);
-
-      // External values could change between render and commit.
-      // Check for this case, even though it requires an extra store read.
       if (data !== maybeNewData) {
         this.setState({
           data: maybeNewData
         });
       }
-    }
-
-    /**
-     * Render new data for the existing props/context.
-     */;
+    };
     _proto._getFragmentVariables = function _getFragmentVariables() {
       return getVariablesFromObject(fragments, this.props);
     };
@@ -320,9 +245,7 @@ function createContainerWithFragments(Component, fragments, taggedNode) {
       }
       return this._queryFetcher;
     };
-    // $FlowFixMe[missing-local-annot]
     _proto.render = function render() {
-      // eslint-disable-next-line no-unused-vars
       var _this$props = this.props,
         componentRef = _this$props.componentRef,
         __relayContext = _this$props.__relayContext,
@@ -347,16 +270,7 @@ function getRelayProp(environment, refetch) {
     refetch: refetch
   };
 }
-
-/**
- * Wrap the basic `createContainer()` function with logic to adapt to the
- * `context.relay.environment` in which it is rendered. Specifically, the
- * extraction of the environment-specific version of fragments in the
- * `fragmentSpec` is memoized once per environment, rather than once per
- * instance of the container constructed/rendered.
- */
 function createContainer(Component, fragmentSpec, taggedNode) {
-  // $FlowFixMe[incompatible-return]
   return buildReactRelayContainer(Component, fragmentSpec, function (ComponentClass, fragments) {
     return createContainerWithFragments(ComponentClass, fragments, taggedNode);
   });

package/lib/ReactRelayTestMocker.js

@@ -1,14 +1,3 @@
-/**
- * Copyright (c) Meta Platforms, Inc. and affiliates.
- *
- * This source code is licensed under the MIT license found in the
- * LICENSE file in the root directory of this source tree.
- *
- * 
- * @format
- * @oncall relay
- */
-
 'use strict';
 
 var _interopRequireDefault = require("@babel/runtime/helpers/interopRequireDefault")["default"];
@@ -21,9 +10,6 @@ var _require = require('relay-runtime'),
   createOperationDescriptor = _require.createOperationDescriptor,
   isRelayModernEnvironment = _require.isRelayModernEnvironment;
 var warning = require("fbjs/lib/warning");
-/**
- * The next id to return from `generateId()`.
- */
 var nextId = 0;
 var ReactRelayTestMocker = /*#__PURE__*/function () {
   function ReactRelayTestMocker(env) {
@@ -38,37 +24,15 @@ var ReactRelayTestMocker = /*#__PURE__*/function () {
   }
   ReactRelayTestMocker.mockOutEnvironment = function mockOutEnvironment(env) {
     return new ReactRelayTestMocker(env);
-  }
-
-  /**
-   * Get a unique id number (as a string). Note: will wrap around after 2^32
-   * calls, if your test needs that many IDs.
-   *
-   * @returns a unique id string
-   */;
+  };
   ReactRelayTestMocker.generateId = function generateId() {
     var toRet = nextId.toString();
     nextId++;
     return toRet;
-  }
-
-  /**
-   * Create a unique identifier for a (query, variables) pair.
-   * @param request: the request associated with the query
-   * @param variables: the variables associated with this invocation of the
-   * query
-   *
-   * @returns a string which can later be used to uniquely identify this query
-   * in the list of pending queries
-   */;
+  };
   ReactRelayTestMocker.getIdentifier = function getIdentifier(request) {
     return request.name;
-  }
-
-  /**
-   * Remove variables that we don't need from the query that will make it more
-   * annoying to test (e.g. client_mutation_id, actor_id)
-   */;
+  };
   ReactRelayTestMocker.stripUnused = function stripUnused(variables) {
     if (variables.input) {
       var toRemove = ['client_mutation_id', 'actor_id', 'clientMutationId', 'actorId'];
@@ -81,15 +45,7 @@ var ReactRelayTestMocker = /*#__PURE__*/function () {
       return strippedVariables;
     }
     return variables;
-  }
-
-  /**
-   * Replace the environment's network layer with a mocked out one to allow for
-   * better testing. Mocking the network allows testing without using a mocked
-   * out QueryRenderer, and will allow for easier testing of components wrapped
-   * in refetch containers, for example. It also allows test writers to see how
-   * their components behave under error conditions.
-   */;
+  };
   var _proto = ReactRelayTestMocker.prototype;
   _proto._mockNetworkLayer = function _mockNetworkLayer(env) {
     var _this = this;
@@ -102,8 +58,6 @@ var ReactRelayTestMocker = /*#__PURE__*/function () {
       });
       var strippedVars = ReactRelayTestMocker.stripUnused(variables);
       var ident = ReactRelayTestMocker.getIdentifier(request);
-
-      // there's a default value for this query, use it
       if (_this._defaults[ident]) {
         var payload = _this._defaults[ident];
         return typeof payload === 'function' ? payload(strippedVars) : payload;
@@ -148,35 +102,18 @@ var ReactRelayTestMocker = /*#__PURE__*/function () {
     env.hasMockedNetwork = true;
     env.__setNet(Network.create(fetch));
     return env;
-  }
-
-  /**
-   * set a default payload for a given query
-   */;
+  };
   _proto.setDefault = function setDefault(toSet) {
     var query = toSet.query,
       payload = toSet.payload;
     var ident = ReactRelayTestMocker.getIdentifier(query.params);
     this._defaults[ident] = payload;
-  }
-
-  /**
-   * remove a default payload for a given query
-   */;
+  };
   _proto.unsetDefault = function unsetDefault(toUnset) {
     var query = toUnset.query;
     var ident = ReactRelayTestMocker.getIdentifier(query.params);
     delete this._defaults[ident];
-  }
-
-  /**
-   * Write directly to the Relay store instead of trying to resolve a query that
-   * was sent via the network.
-   *
-   * Use this method when testing a component wrapped in a fragment container
-   * (via `createFragmentContainer`). The component under test should also be
-   * wrapped in a `RelayTestRenderer`.
-   */;
+  };
   _proto.dataWrite = function dataWrite(config) {
     var query = config.query,
       variables = config.variables,
@@ -184,16 +121,7 @@ var ReactRelayTestMocker = /*#__PURE__*/function () {
     var operationDescriptor = createOperationDescriptor(query, variables);
     !(payload.data != null && payload.errors === undefined) ? process.env.NODE_ENV !== "production" ? invariant(false, 'Only `data` can be written when using `writeDirect`. You may need to ' + 'wrap your payload in an object like `{data: payload}`.') : invariant(false) : void 0;
     this._environment.commitPayload(operationDescriptor, payload.data);
-  }
-
-  /**
-   * Write the data specified in config's payload to the instance's environment.
-   * NOTE: callers may need to invoke `jest.runOnlyPendingTimers()` after
-   * calling this function.
-   *
-   * @param config: an object containing the data to write and the query and
-   * variables that the payload is simulating a response to
-   */;
+  };
   _proto.networkWrite = function networkWrite(config) {
     !this._environment.hasMockedNetwork ? process.env.NODE_ENV !== "production" ? invariant(false, 'You cannot resolve queries without a mocked environment. Did you mean ' + 'to use `writeDirect` instead?') : invariant(false) : void 0;
     var query = config.query,
@@ -221,11 +149,7 @@ var ReactRelayTestMocker = /*#__PURE__*/function () {
     });
     var varMessage = usedVars ? ' - variables: ' + JSON.stringify(usedVars) : '';
     !toResolve ? process.env.NODE_ENV !== "production" ? invariant(false, 'You are attempting to resolve a query that has not been fetched ' + '(%s%s).\n\tPlease ensure you passed the correct variables, or use ' + '`writeDirect` instead.', ident, varMessage) : invariant(false) : void 0;
-    var realPayload =
-    // $FlowFixMe[incompatible-call]
-    typeof payload === 'function' ? payload(toResolve.variables) : payload;
-
-    // if there are errors, reject the query
+    var realPayload = typeof payload === 'function' ? payload(toResolve.variables) : payload;
     if (realPayload.errors != null && realPayload.errors.length > 0) {
       this._environment.mock.rejectQuery(toResolve, {
         error: realPayload.errors[0]

package/lib/ReactRelayTypes.js

@@ -1,12 +1 @@
-/**
- * Copyright (c) Meta Platforms, Inc. and affiliates.
- *
- * This source code is licensed under the MIT license found in the
- * LICENSE file in the root directory of this source tree.
- *
- * 
- * @format
- * @oncall relay
- */
-
 'use strict';

package/lib/RelayContext.js

@@ -1,32 +1,11 @@
-/**
- * Copyright (c) Meta Platforms, Inc. and affiliates.
- *
- * This source code is licensed under the MIT license found in the
- * LICENSE file in the root directory of this source tree.
- *
- * 
- * @format
- * @oncall relay
- */
-
 'use strict';
 
 var isRelayEnvironment = require('./isRelayEnvironment');
 var invariant = require('invariant');
-
-/**
- * Asserts that the input is a matches the `RelayContext` type defined in
- * `RelayEnvironmentTypes` and returns it as that type.
- */
 function assertRelayContext(relay) {
   !isRelayContext(relay) ? process.env.NODE_ENV !== "production" ? invariant(false, 'RelayContext: Expected `context.relay` to be an object conforming to ' + 'the `RelayContext` interface, got `%s`.', relay) : invariant(false) : void 0;
   return relay;
 }
-
-/**
- * Determine if the input is a plain object that matches the `RelayContext`
- * type defined in `RelayEnvironmentTypes`.
- */
 function isRelayContext(context) {
   return typeof context === 'object' && context !== null && !Array.isArray(context) && isRelayEnvironment(context.environment);
 }

package/lib/assertFragmentMap.js

@@ -1,21 +1,6 @@
-/**
- * Copyright (c) Meta Platforms, Inc. and affiliates.
- *
- * This source code is licensed under the MIT license found in the
- * LICENSE file in the root directory of this source tree.
- *
- * 
- * @format
- * @oncall relay
- */
-
 'use strict';
 
 var invariant = require('invariant');
-
-/**
- * Fail fast if the user supplies invalid fragments as input.
- */
 function assertFragmentMap(componentName, fragmentSpec) {
   !(fragmentSpec && typeof fragmentSpec === 'object') ? process.env.NODE_ENV !== "production" ? invariant(false, 'Could not create Relay Container for `%s`. ' + 'Expected a set of GraphQL fragments, got `%s` instead.', componentName, fragmentSpec) : invariant(false) : void 0;
   for (var key in fragmentSpec) {

package/lib/buildReactRelayContainer.js

@@ -1,14 +1,3 @@
-/**
- * Copyright (c) Meta Platforms, Inc. and affiliates.
- *
- * This source code is licensed under the MIT license found in the
- * LICENSE file in the root directory of this source tree.
- *
- * 
- * @format
- * @oncall relay
- */
-
 'use strict';
 
 var _interopRequireDefault = require("@babel/runtime/helpers/interopRequireDefault")["default"];
@@ -24,12 +13,7 @@ var React = require('react');
 var _require2 = require('relay-runtime'),
   getFragment = _require2.getFragment;
 var useContext = React.useContext;
-/**
- * Helper to create the Relay HOCs with ref forwarding, setting the displayName
- * and reading the React context.
- */
 function buildReactRelayContainer(ComponentClass, fragmentSpec, createContainerWithFragments) {
-  // Sanity-check user-defined fragment input
   var containerName = getContainerName(ComponentClass);
   assertFragmentMap(getComponentName(ComponentClass), fragmentSpec);
   var fragments = {};
@@ -52,12 +36,9 @@ function buildReactRelayContainer(ComponentClass, fragmentSpec, createContainerW
   forwardRef.displayName = containerName;
   var ForwardContainer = React.forwardRef(forwardRef);
   if (process.env.NODE_ENV !== "production") {
-    // Used by RelayModernTestUtils
     ForwardContainer.__ComponentClass = ComponentClass;
     ForwardContainer.displayName = containerName;
   }
-
-  // $FlowFixMe[incompatible-return]
   return ForwardContainer;
 }
 module.exports = buildReactRelayContainer;

package/lib/getRootVariablesForFragments.js

@@ -1,14 +1,3 @@
-/**
- * Copyright (c) Meta Platforms, Inc. and affiliates.
- *
- * This source code is licensed under the MIT license found in the
- * LICENSE file in the root directory of this source tree.
- *
- * 
- * @format
- * @oncall relay
- */
-
 'use strict';
 
 var _interopRequireDefault = require("@babel/runtime/helpers/interopRequireDefault")["default"];
@@ -17,9 +6,6 @@ var _require = require('relay-runtime'),
   getSelector = _require.getSelector;
 function getRootVariablesForFragments(fragments, props) {
   var rootVariables = {};
-  // NOTE: For extra safety, we make sure the rootVariables include the
-  // variables from all owners in this fragmentSpec, even though they
-  // should all point to the same owner
   Object.keys(fragments).forEach(function (key) {
     var _selector$selectors$, _selector$selectors$2, _selector$owner$varia;
     var fragmentNode = fragments[key];

package/lib/hooks.js

@@ -1,14 +1,3 @@
-/**
- * Copyright (c) Meta Platforms, Inc. and affiliates.
- *
- * This source code is licensed under the MIT license found in the
- * LICENSE file in the root directory of this source tree.
- *
- * 
- * @format
- * @oncall relay
- */
-
 'use strict';
 
 var EntryPointContainer = require('./relay-hooks/EntryPointContainer.react');
@@ -29,10 +18,6 @@ var useRelayEnvironment = require('./relay-hooks/useRelayEnvironment');
 var useSubscribeToInvalidationState = require('./relay-hooks/useSubscribeToInvalidationState');
 var useSubscription = require('./relay-hooks/useSubscription');
 var RelayRuntime = require('relay-runtime');
-/**
- * The public interface for Relay Hooks.
- * This will eventually become the main public interface for react-relay.
- */
 module.exports = {
   ConnectionHandler: RelayRuntime.ConnectionHandler,
   applyOptimisticMutation: RelayRuntime.applyOptimisticMutation,

package/lib/index.js

@@ -1,14 +1,3 @@
-/**
- * Copyright (c) Meta Platforms, Inc. and affiliates.
- *
- * This source code is licensed under the MIT license found in the
- * LICENSE file in the root directory of this source tree.
- *
- * 
- * @format
- * @oncall relay
- */
-
 'use strict';
 
 var ReactRelayContext = require('./ReactRelayContext');
@@ -36,11 +25,6 @@ var useRelayEnvironment = require('./relay-hooks/useRelayEnvironment');
 var useSubscribeToInvalidationState = require('./relay-hooks/useSubscribeToInvalidationState');
 var useSubscription = require('./relay-hooks/useSubscription');
 var RelayRuntime = require('relay-runtime');
-/**
- * The public interface to react-relay.
- * Currently contains both Relay Hooks and legacy Container APIs.
- * Will eventually only export the interface from ./hooks.js.
- */
 module.exports = {
   ConnectionHandler: RelayRuntime.ConnectionHandler,
   QueryRenderer: ReactRelayQueryRenderer,
@@ -58,7 +42,6 @@ module.exports = {
   graphql: RelayRuntime.graphql,
   readInlineData: RelayRuntime.readInlineData,
   requestSubscription: RelayRuntime.requestSubscription,
-  // Relay Hooks
   EntryPointContainer: EntryPointContainer,
   RelayEnvironmentProvider: RelayEnvironmentProvider,
   ProfilerContext: ProfilerContext,

package/lib/isRelayEnvironment.js

@@ -1,23 +1,6 @@
-/**
- * Copyright (c) Meta Platforms, Inc. and affiliates.
- *
- * This source code is licensed under the MIT license found in the
- * LICENSE file in the root directory of this source tree.
- *
- * 
- * @format
- * @oncall relay
- */
-
 'use strict';
 
-/**
- * Determine if a given value is an object that implements the `Environment`
- * interface defined in `RelayEnvironmentTypes`.
- */
 function isRelayEnvironment(environment) {
-  return typeof environment === 'object' && environment !== null &&
-  // TODO: add applyMutation/sendMutation once ready in both cores
-  typeof environment.check === 'function' && typeof environment.lookup === 'function' && typeof environment.retain === 'function' && typeof environment.execute === 'function' && typeof environment.subscribe === 'function';
+  return typeof environment === 'object' && environment !== null && typeof environment.check === 'function' && typeof environment.lookup === 'function' && typeof environment.retain === 'function' && typeof environment.execute === 'function' && typeof environment.subscribe === 'function';
 }
 module.exports = isRelayEnvironment;

package/lib/jest-react/enqueueTask.js

@@ -1,15 +1,3 @@
-/**
- * Copyright (c) Meta Platforms, Inc. and affiliates.
- *
- * This source code is licensed under the MIT license found in the
- * LICENSE file in the root directory of this source tree.
- *
- * @format
- * @oncall relay
- */
-
-// This file is sync'd from https://github.com/facebook/react/tree/main/packages/jest-react
-
 'use strict';
 
 var didWarnAboutMessageChannel = false;
@@ -17,17 +5,10 @@ var enqueueTaskImpl = null;
 function enqueueTask(task) {
   if (enqueueTaskImpl === null) {
     try {
-      // read require off the module object to get around the bundlers.
-      // we don't want them to detect a require and bundle a Node polyfill.
       var requireString = ('require' + Math.random()).slice(0, 7);
       var nodeRequire = module && module[requireString];
-      // assuming we're in node, let's try to get node's
-      // version of setImmediate, bypassing fake timers if any.
       enqueueTaskImpl = nodeRequire.call(module, 'timers').setImmediate;
     } catch (_unused) {
-      // we're in a browser
-      // we can't use regular timers because they may still be faked
-      // so we try MessageChannel+postMessage instead
       enqueueTaskImpl = function enqueueTaskImpl(callback) {
         if (process.env.NODE_ENV !== "production") {
           if (didWarnAboutMessageChannel === false) {
@@ -37,7 +18,6 @@ function enqueueTask(task) {
             }
           }
         }
-        /*global MessageChannel*/
         var channel = new MessageChannel();
         channel.port1.onmessage = callback;
         channel.port2.postMessage(undefined);