remove ensureResult for this PR, add TODO type

src/core/ObservableQuery.ts

@@ -35,6 +35,7 @@ import type { QueryInfo } from "./QueryInfo.js";
 import type { MissingFieldError } from "../cache/index.js";
 import type { MissingTree } from "../cache/core/types/common.js";
 import { equalByQuery } from "./equalByQuery.js";
+import type { TODO } from "../utilities/types/TODO.js";
 
 const { assign, hasOwnProperty } = Object;
 
@@ -927,10 +928,9 @@ Did you mean to call refetch(variables) instead of refetch({ variables })?`,
   public reobserve(
     newOptions?: Partial<WatchQueryOptions<TVariables, TData>>,
     newNetworkStatus?: NetworkStatus
-  ) {
-    return this.reobserveAsConcast(newOptions, newNetworkStatus).promise.then(
-      ensureResult
-    );
+  ): Promise<ApolloQueryResult<TData>> {
+    return this.reobserveAsConcast(newOptions, newNetworkStatus)
+      .promise as TODO;
   }
 
   public resubscribeAfterError(
@@ -1096,13 +1096,3 @@ function skipCacheDataFor(
     fetchPolicy === "standby"
   );
 }
-
-export function ensureResult<TData = any>(
-  value: ApolloQueryResult<TData> | undefined
-): ApolloQueryResult<TData> {
-  invariant(
-    value,
-    "A Concast finished without a result. This in an Apollo Client bug, please file a bug report."
-  );
-  return value;
-}

src/core/QueryManager.ts

@@ -51,11 +51,7 @@ import type {
   ErrorPolicy,
   MutationFetchPolicy,
 } from "./watchQueryOptions.js";
-import {
-  ObservableQuery,
-  ensureResult,
-  logMissingFieldErrors,
-} from "./ObservableQuery.js";
+import { ObservableQuery, logMissingFieldErrors } from "./ObservableQuery.js";
 import { NetworkStatus, isNetworkRequestInFlight } from "./networkStatus.js";
 import type {
   ApolloQueryResult,
@@ -78,6 +74,7 @@ import {
 import type { ApolloErrorOptions } from "../errors/index.js";
 import { PROTOCOL_ERRORS_SYMBOL } from "../errors/index.js";
 import { print } from "../utilities/index.js";
+import type { TODO } from "../utilities/types/TODO.js";
 
 const { hasOwnProperty } = Object.prototype;
 
@@ -620,11 +617,8 @@ export class QueryManager<TStore> {
     options: WatchQueryOptions<TVars, TData>,
     networkStatus?: NetworkStatus
   ): Promise<ApolloQueryResult<TData>> {
-    return this.fetchConcastWithInfo(
-      queryId,
-      options,
-      networkStatus
-    ).concast.promise.then(ensureResult);
+    return this.fetchConcastWithInfo(queryId, options, networkStatus).concast
+      .promise as TODO;
   }
 
   public getQueryStore() {

src/utilities/types/TODO.ts

@@ -0,0 +1,2 @@
+/** @internal */
+export type TODO = any;