[dynamicIO] use new heuristic to track whether server render is dynamic

Instead of relying on onError just use the timing of the prerender resolution to determine whether the server render completed before aborting.

Author: gnoff

packages/next/src/server/app-render/app-render.tsx

@@ -2716,11 +2716,12 @@ async function prerenderToStream(
           ctx,
           res.statusCode === 404
         )
+        let prerenderIsPending = true
         const reactServerResult = (reactServerPrerenderResult =
           await createReactServerPrerenderResult(
             prerenderAndAbortInSequentialTasks(
-              () =>
-                workUnitAsyncStorage.run(
+              async () => {
+                const prerenderResult = await workUnitAsyncStorage.run(
                   // The store to scope
                   finalRenderPrerenderStore,
                   // The function to run
@@ -2730,17 +2731,36 @@ async function prerenderToStream(
                   clientReferenceManifest.clientModules,
                   {
                     onError: (err: unknown) => {
+                      // TODO we can remove this once https://github.com/facebook/react/pull/31715 lands
+                      // because we won't have onError calls when halting the prerender
                       if (finalServerController.signal.aborted) {
-                        serverIsDynamic = true
                         return
                       }
 
                       return serverComponentsErrorHandler(err)
                     },
                     signal: finalServerController.signal,
                   }
-                ),
+                )
+                prerenderIsPending = false
+                return prerenderResult
+              },
               () => {
+                if (
+                  finalServerController.signal.aborted ||
+                  prerenderIsPending
+                ) {
+                  // If the server controller is already aborted we must have called something
+                  // that required aborting the prerender synchronously such as with new Date()
+                  serverIsDynamic = true
+                  return
+                }
+
+                if (prerenderIsPending) {
+                  // If prerenderIsPending then we have blocked for longer than a Task and we assume
+                  // there is something unfinished.
+                  serverIsDynamic = true
+                }
                 finalServerController.abort()
               }
             )