useBaseQuery - non-reactive useEffect dependencies?

[AjaxSolutions]

https://github.com/apollographql/apollo-client/blob/master/src/react/hooks/utils/useBaseQuery.ts#L89

Is this dep list correct?

  useEffect(() => queryData.afterExecute({ lazy }), [
    queryResult.loading,
    queryResult.networkStatus,
    queryResult.error,
    queryResult.data,
  ]);

facebook/react#16121

"Dependencies should only include values that participate in top-down React data flow. Such as props, state, and what you calculate from them. "

[AjaxSolutions]

I'm looking at React code checking deps in areHookInputsEqual. I don't see anything here that would favor React state or props. They just use Object.is polyfill to compare deps.

https://github.com/facebook/react/blob/e71f5df9c32b3b486bf4efce861834389c395d47/packages/react-dom/src/server/ReactPartialRendererHooks.js#L124

[hwillson]

The dependency tracking we have in place is intentional and we would like to keep it as is for now. If you notice any issues caused because of this, definitely let us know. Thanks!