fix(ppr): fix prerender info matching for rewritten paths (#83359)

## What?

Fixed prerender info matching for rewritten paths in App Router by using
`resolvedPathname` instead of `parsedUrl.pathname` in the app-page
template.

## Why?

Previously, the app-page template used `parsedUrl.pathname` to match
against `prerenderManifest` routes. However, when middleware rewrites
paths, `parsedUrl.pathname` contains the original (unrewritten) path,
while `resolvedPathname` contains the actual path that should be used
for prerender lookup.

This mismatch caused incorrect prerender behavior for routes that were
rewritten by middleware, as the handler would look up prerender info
using the wrong path.

This handling is special cased for when not handling an interception
route when cache components is not enabled. This work is being tracked
in NAR-335.

## How?

- **Modified `packages/next/src/build/templates/app-page.ts`**: 
  - Removed unused `parsedUrl` parameter from handler function
- Updated prerender info matching to use `resolvedPathname` instead of
`parsedUrl.pathname`
- Added explanatory comment about why `resolvedPathname` is the correct
choice

- **Added comprehensive test suite** in
`test/e2e/app-dir/sub-shell-generation-middleware/`:
  - Tests various middleware rewrite scenarios
  - Validates sub-shell generation behavior with nested dynamic routes
  - Tests both static and dynamic prerender scenarios
- Includes middleware that rewrites paths to different route structures

- **Enhanced deploy testing** in `test/lib/next-modes/next-deploy.ts`:
  - Added `stderr: 'inherit'` to provide earlier deployment feedback
  - Improves debugging experience for deployment-related tests

The fix ensures that prerender info lookup works correctly for all
paths, whether they're rewritten by middleware or not, providing
consistent behavior across different routing scenarios.

Fixes #83354

Author: wyattjoh

packages/next/src/build/templates/app-page.ts

@@ -89,6 +89,7 @@ import * as entryBase from '../../server/app-render/entry-base' with { 'turbopac
 import { RedirectStatusCode } from '../../client/components/redirect-status-code'
 import { InvariantError } from '../../shared/lib/invariant-error'
 import { scheduleOnNextTick } from '../../lib/scheduler'
+import { isInterceptionRouteAppPath } from '../../shared/lib/router/utils/interception-routes'
 
 export * from '../../server/app-render/entry-base' with { 'turbopack-transition': 'next-server-utility' }
 
@@ -150,7 +151,6 @@ export async function handler(
     buildId,
     query,
     params,
-    parsedUrl,
     pageIsDynamic,
     buildManifest,
     nextFontManifest,
@@ -167,12 +167,24 @@ export async function handler(
     interceptionRoutePatterns,
   } = prepareResult
 
-  const pathname = parsedUrl.pathname || '/'
   const normalizedSrcPage = normalizeAppPath(srcPage)
 
   let { isOnDemandRevalidate } = prepareResult
 
-  const prerenderInfo = routeModule.match(pathname, prerenderManifest)
+  // We use the resolvedPathname instead of the parsedUrl.pathname because it
+  // is not rewritten as resolvedPathname is. This will ensure that the correct
+  // prerender info is used instead of using the original pathname as the
+  // source. If however PPR is enabled and cacheComponents is disabled, we
+  // treat the pathname as dynamic. Currently, there's a bug in the PPR
+  // implementation that incorrectly leaves %%drp placeholders in the output of
+  // parallel routes. This is addressed with cacheComponents.
+  const prerenderInfo =
+    nextConfig.experimental.ppr &&
+    !nextConfig.experimental.cacheComponents &&
+    isInterceptionRouteAppPath(resolvedPathname)
+      ? null
+      : routeModule.match(resolvedPathname, prerenderManifest)
+
   const isPrerendered = !!prerenderManifest.routes[resolvedPathname]
 
   const userAgent = req.headers['user-agent'] || ''

test/e2e/app-dir/sub-shell-generation-middleware/app/[first]/[second]/[third]/layout.tsx

@@ -0,0 +1,20 @@
+import { Suspense } from 'react'
+import SharedComponent from '../../../shared'
+
+export default async function ThirdLayout({
+  children,
+  params,
+}: {
+  children: React.ReactNode
+  params: Promise<Record<string, string>>
+}) {
+  const current = await params
+
+  return (
+    <>
+      <pre>{JSON.stringify(current)}</pre>
+      <SharedComponent layout={`/[first]/[second]/[third]`} />
+      <Suspense>{children}</Suspense>
+    </>
+  )
+}

test/e2e/app-dir/sub-shell-generation-middleware/app/[first]/[second]/[third]/page.tsx

@@ -0,0 +1,16 @@
+export default async function NotRewritePage({
+  params,
+}: {
+  params: Promise<{ first: string; second: string; third: string }>
+}) {
+  const { first, second, third } = await params
+  return (
+    <div data-slug={`${first}/${second}/${third}`}>
+      Page /{first}/{second}/{third}
+    </div>
+  )
+}
+
+export async function generateStaticParams() {
+  return [{ first: 'first', second: 'second', third: 'third' }]
+}

test/e2e/app-dir/sub-shell-generation-middleware/app/[first]/[second]/layout.tsx

@@ -0,0 +1,20 @@
+import { Suspense } from 'react'
+import SharedComponent from '../../shared'
+
+export default async function SecondLayout({
+  children,
+  params,
+}: {
+  children: React.ReactNode
+  params: Promise<Record<string, string>>
+}) {
+  const current = await params
+
+  return (
+    <>
+      <pre>{JSON.stringify(current)}</pre>
+      <SharedComponent layout={`/[first]/[second]`} />
+      <Suspense>{children}</Suspense>
+    </>
+  )
+}

test/e2e/app-dir/sub-shell-generation-middleware/app/[first]/layout.tsx

@@ -0,0 +1,20 @@
+import { Suspense } from 'react'
+import SharedComponent from '../shared'
+
+export default async function FirstLayout({
+  children,
+  params,
+}: {
+  children: React.ReactNode
+  params: Promise<Record<string, string>>
+}) {
+  const current = await params
+
+  return (
+    <>
+      <pre>{JSON.stringify(current)}</pre>
+      <SharedComponent layout={`/[first]`} />
+      <Suspense>{children}</Suspense>
+    </>
+  )
+}

test/e2e/app-dir/sub-shell-generation-middleware/app/layout.tsx

@@ -0,0 +1,13 @@
+import { type ReactNode, Suspense } from 'react'
+import SharedComponent from './shared'
+
+export default function Root({ children }: { children: ReactNode }) {
+  return (
+    <html>
+      <body>
+        <SharedComponent layout="/" />
+        <Suspense fallback={<div>Loading...</div>}>{children}</Suspense>
+      </body>
+    </html>
+  )
+}

test/e2e/app-dir/sub-shell-generation-middleware/app/page.tsx

@@ -0,0 +1,3 @@
+export default function Page() {
+  return <p>hello world</p>
+}

test/e2e/app-dir/sub-shell-generation-middleware/app/rewrite/[slug]/layout.tsx

@@ -0,0 +1,20 @@
+import { Suspense } from 'react'
+import SharedComponent from '../../shared'
+
+export default async function RewriteSlugLayout({
+  children,
+  params,
+}: {
+  params: Promise<Record<string, string>>
+  children: React.ReactNode
+}) {
+  const current = await params
+
+  return (
+    <>
+      <pre>{JSON.stringify(current)}</pre>
+      <SharedComponent layout={`/rewrite/[slug]`} />
+      <Suspense>{children}</Suspense>
+    </>
+  )
+}

test/e2e/app-dir/sub-shell-generation-middleware/app/rewrite/[slug]/page.tsx

@@ -0,0 +1,12 @@
+export default async function RewritePage({
+  params,
+}: {
+  params: Promise<{ slug: string }>
+}) {
+  const { slug } = await params
+  return <div data-rewrite-slug={slug}>Page /rewrite/{slug}</div>
+}
+
+export async function generateStaticParams() {
+  return []
+}

test/e2e/app-dir/sub-shell-generation-middleware/app/rewrite/layout.tsx

@@ -0,0 +1,20 @@
+import { Suspense } from 'react'
+import SharedComponent from '../shared'
+
+export default async function RewriteLayout({
+  children,
+  params,
+}: {
+  params: Promise<Record<string, string>>
+  children: React.ReactNode
+}) {
+  const current = await params
+
+  return (
+    <div>
+      <pre>{JSON.stringify(current)}</pre>
+      <SharedComponent layout={`/rewrite`} />
+      <Suspense>{children}</Suspense>
+    </div>
+  )
+}