fix: don't fail on dynamic routes when PPR is enabled

Author: pieh

src/build/plugin-context.ts

@@ -10,11 +10,14 @@ import type {
   NetlifyPluginOptions,
   NetlifyPluginUtils,
 } from '@netlify/build'
-import type { PrerenderManifest, RoutesManifest } from 'next/dist/build/index.js'
 import type { MiddlewareManifest } from 'next/dist/build/webpack/plugins/middleware-plugin.js'
 import type { PagesManifest } from 'next/dist/build/webpack/plugins/pages-manifest-plugin.js'
 import type { NextConfigComplete } from 'next/dist/server/config-shared.js'
-import type { FunctionsConfigManifest } from 'next-with-cache-handler-v2/dist/build/index.js'
+import type {
+  FunctionsConfigManifest,
+  PrerenderManifest,
+  RoutesManifest,
+} from 'next-with-cache-handler-v2/dist/build/index.js'
 import { satisfies } from 'semver'
 
 const MODULE_DIR = fileURLToPath(new URL('.', import.meta.url))
@@ -374,7 +377,7 @@ export class PluginContext {
           //  - `string` - when user use pages router with `fallback: true`, and then it's html file path
           //  - `null` - when user use pages router with `fallback: 'block'` or app router with `export const dynamicParams = true`
           //  - `false` - when user use pages router with `fallback: false` or app router with `export const dynamicParams = false`
-          if (typeof meta.fallback === 'string') {
+          if (typeof meta.fallback === 'string' && meta.renderingMode !== 'PARTIALLY_STATIC') {
             for (const locale of locales) {
               const localizedRoute = posixJoin(locale, route.replace(/^\/+/g, ''))
               fallbacks.push(localizedRoute)

tests/fixtures/ppr/app/[dynamic]/page.js

@@ -0,0 +1,45 @@
+import { Suspense } from 'react'
+import { connection } from 'next/server'
+
+export async function generateStaticParams() {
+  return [{ dynamic: '1' }, { dynamic: '2' }]
+}
+
+async function getData(params) {
+  await connection()
+  const res = await fetch(`https://api.tvmaze.com/shows/${params.id}`, {
+    next: {
+      tags: [`show-${params.id}`],
+    },
+  })
+  await new Promise((res) => setTimeout(res, 3000))
+  return res.json()
+}
+
+async function Content(params) {
+  const data = await getData(params)
+
+  return (
+    <dl>
+      <dt>Show</dt>
+      <dd>{data.name}</dd>
+      <dt>Param</dt>
+      <dd>{params.id}</dd>
+      <dt>Time</dt>
+      <dd data-testid="date-now">{new Date().toISOString()}</dd>
+    </dl>
+  )
+}
+
+export default async function DynamicPage({ params }) {
+  const { dynamic } = await params
+
+  return (
+    <main>
+      <h1>Dynamic Page: {dynamic}</h1>
+      <Suspense fallback={<div>loading...</div>}>
+        <Content id={dynamic} />
+      </Suspense>
+    </main>
+  )
+}

tests/integration/simple-app.test.ts

@@ -394,6 +394,8 @@ test.skipIf(process.env.NEXT_VERSION !== 'canary')<FixtureTestContext>(
     const blobEntries = await getBlobEntries(ctx)
     expect(blobEntries.map(({ key }) => decodeBlobKey(key)).sort()).toEqual(
       [
+        '/1',
+        '/2',
         '/404',
         shouldHaveAppRouterGlobalErrorInPrerenderManifest() ? '/_global-error' : undefined,
         shouldHaveAppRouterNotFoundInPrerenderManifest() ? '/_not-found' : undefined,
@@ -407,6 +409,14 @@ test.skipIf(process.env.NEXT_VERSION !== 'canary')<FixtureTestContext>(
     const home = await invokeFunction(ctx)
     expect(home.statusCode).toBe(200)
     expect(load(home.body)('h1').text()).toBe('Home')
+
+    const dynamicPrerendered = await invokeFunction(ctx, { url: '/1' })
+    expect(dynamicPrerendered.statusCode).toBe(200)
+    expect(load(dynamicPrerendered.body)('h1').text()).toBe('Dynamic Page: 1')
+
+    const dynamicNotPrerendered = await invokeFunction(ctx, { url: '/3' })
+    expect(dynamicNotPrerendered.statusCode).toBe(200)
+    expect(load(dynamicNotPrerendered.body)('h1').text()).toBe('Dynamic Page: 3')
   },
 )