Wrap local results in Promise for query methods (#29161)

These methods return promises. When we had the result already, either from another query request by the client or from an optimistic update, we were returning the raw value without wrapping it in a promise.

GitOrigin-RevId: 956a13525ddc1ef4c46467da12e320c2e8f03203

npm-packages/convex/src/browser/simple_client.ts

@@ -368,9 +368,9 @@ export class ConvexClient {
   ): Promise<Awaited<Query["_returnType"]>> {
     if (this.disabled) throw new Error("ConvexClient is disabled");
     const value = this.client.localQueryResult(getFunctionName(query), args) as
-      | Query["_returnType"]
+      | Awaited<Query["_returnType"]>
       | undefined;
-    if (value !== undefined) return value;
+    if (value !== undefined) return Promise.resolve(value);
 
     return new Promise((resolve, reject) => {
       const { unsubscribe } = this.onUpdate(

npm-packages/convex/src/react/client.ts

@@ -456,7 +456,7 @@ export class ConvexReactClient {
     const watch = this.watchQuery(query, ...args);
     const existingResult = watch.localQueryResult();
     if (existingResult !== undefined) {
-      return existingResult;
+      return Promise.resolve(existingResult);
     }
     return new Promise((resolve, reject) => {
       const unsubscribe = watch.onUpdate(() => {