@netlify/plugin-nextjs 4.28.4-appdir.0 → 4.28.4

package/lib/helpers/edge.js

@@ -3,7 +3,7 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
     return (mod && mod.__esModule) ? mod : { "default": mod };
 };
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.writeEdgeFunctions = exports.writeDevEdgeFunction = exports.cleanupEdgeFunctions = exports.loadAppPathRoutesManifest = exports.loadMiddlewareManifest = void 0;
+exports.writeEdgeFunctions = exports.writeDevEdgeFunction = exports.cleanupEdgeFunctions = exports.loadMiddlewareManifest = void 0;
 /* eslint-disable max-lines */
 const fs_1 = require("fs");
 const path_1 = require("path");
@@ -13,15 +13,14 @@ const fs_extra_1 = require("fs-extra");
 const outdent_1 = require("outdent");
 const config_1 = require("./config");
 const matchers_1 = require("./matchers");
-const maybeLoadJson = (path) => {
-    if ((0, fs_1.existsSync)(path)) {
-        return (0, fs_extra_1.readJson)(path);
+const loadMiddlewareManifest = (netlifyConfig) => {
+    const middlewarePath = (0, path_1.resolve)(netlifyConfig.build.publish, 'server', 'middleware-manifest.json');
+    if (!(0, fs_1.existsSync)(middlewarePath)) {
+        return null;
     }
+    return (0, fs_extra_1.readJson)(middlewarePath);
 };
-const loadMiddlewareManifest = (netlifyConfig) => maybeLoadJson((0, path_1.resolve)(netlifyConfig.build.publish, 'server', 'middleware-manifest.json'));
 exports.loadMiddlewareManifest = loadMiddlewareManifest;
-const loadAppPathRoutesManifest = (netlifyConfig) => maybeLoadJson((0, path_1.resolve)(netlifyConfig.build.publish, 'app-path-routes-manifest.json'));
-exports.loadAppPathRoutesManifest = loadAppPathRoutesManifest;
 /**
  * Convert the Next middleware name into a valid Edge Function name
  */
@@ -96,7 +95,7 @@ const getMiddlewareBundle = async ({ edgeFunctionDefinition, netlifyConfig, }) =
 };
 const getEdgeTemplatePath = (file) => (0, path_1.join)(__dirname, '..', '..', 'src', 'templates', 'edge', file);
 const copyEdgeSourceFile = ({ file, target, edgeFunctionDir, }) => fs_1.promises.copyFile(getEdgeTemplatePath(file), (0, path_1.join)(edgeFunctionDir, target !== null && target !== void 0 ? target : file));
-const writeEdgeFunction = async ({ edgeFunctionDefinition, edgeFunctionRoot, netlifyConfig, pageRegexMap, appPathRoutesManifest = {}, nextConfig, }) => {
+const writeEdgeFunction = async ({ edgeFunctionDefinition, edgeFunctionRoot, netlifyConfig, nextConfig, }) => {
     const name = sanitizeName(edgeFunctionDefinition.name);
     const edgeFunctionDir = (0, path_1.join)(edgeFunctionRoot, name);
     const bundle = await getMiddlewareBundle({
@@ -124,14 +123,6 @@ const writeEdgeFunction = async ({ edgeFunctionDefinition, edgeFunctionRoot, net
     else {
         matchers.push(...edgeFunctionDefinition.matchers);
     }
-    // If the EF matches a page, it's an app dir page so needs a matcher too
-    // The object will be empty if appDir isn't enabled in the Next config
-    if (pageRegexMap && edgeFunctionDefinition.page in appPathRoutesManifest) {
-        const regexp = pageRegexMap.get(appPathRoutesManifest[edgeFunctionDefinition.page]);
-        if (regexp) {
-            matchers.push({ regexp });
-        }
-    }
     await (0, fs_extra_1.writeJson)((0, path_1.join)(edgeFunctionDir, 'matchers.json'), matchers);
     // We add a defintion for each matching path
     return matchers.map((matcher) => {
@@ -164,7 +155,7 @@ exports.writeDevEdgeFunction = writeDevEdgeFunction;
 /**
  * Writes Edge Functions for the Next middleware
  */
-const writeEdgeFunctions = async ({ netlifyConfig, routesManifest, }) => {
+const writeEdgeFunctions = async (netlifyConfig) => {
     const manifest = {
         functions: [],
         version: 1,
@@ -211,20 +202,12 @@ const writeEdgeFunctions = async ({ netlifyConfig, routesManifest, }) => {
         // Older versions of the manifest format don't have the functions field
         // No, the version field was not incremented
         if (typeof middlewareManifest.functions === 'object') {
-            // When using the app dir, we also need to check if the EF matches a page
-            const appPathRoutesManifest = await (0, exports.loadAppPathRoutesManifest)(netlifyConfig);
-            const pageRegexMap = new Map([...(routesManifest.dynamicRoutes || []), ...(routesManifest.staticRoutes || [])].map((route) => [
-                route.page,
-                route.regex,
-            ]));
             for (const edgeFunctionDefinition of Object.values(middlewareManifest.functions)) {
                 usesEdge = true;
                 const functionDefinitions = await writeEdgeFunction({
                     edgeFunctionDefinition,
                     edgeFunctionRoot,
                     netlifyConfig,
-                    pageRegexMap,
-                    appPathRoutesManifest,
                     nextConfig,
                 });
                 manifest.functions.push(...functionDefinitions);

package/lib/helpers/files.js

@@ -70,6 +70,7 @@ exports.getMiddleware = getMiddleware;
 const moveStaticPages = async ({ netlifyConfig, target, i18n, basePath, }) => {
     console.log('Moving static page files to serve from CDN...');
     const outputDir = (0, pathe_1.join)(netlifyConfig.build.publish, target === 'server' ? 'server' : 'serverless');
+    const root = (0, pathe_1.join)(outputDir, 'pages');
     const buildId = (0, fs_extra_1.readFileSync)((0, pathe_1.join)(netlifyConfig.build.publish, 'BUILD_ID'), 'utf8').trim();
     const dataDir = (0, pathe_1.join)('_next', 'data', buildId);
     await (0, fs_extra_1.ensureDir)((0, pathe_1.join)(netlifyConfig.build.publish, dataDir));
@@ -91,17 +92,14 @@ const moveStaticPages = async ({ netlifyConfig, target, i18n, basePath, }) => {
             }
         }
     });
-    let fileCount = 0;
+    const files = [];
     const filesManifest = {};
     const moveFile = async (file) => {
-        // Strip the initial 'app' or 'pages' directory from the output path
-        const pathname = file.split('/').slice(1).join('/');
-        // .rsc data files go next to the html file
         const isData = file.endsWith('.json');
-        const source = (0, pathe_1.join)(outputDir, file);
-        const targetFile = isData ? (0, pathe_1.join)(dataDir, pathname) : pathname;
+        const source = (0, pathe_1.join)(root, file);
+        const targetFile = isData ? (0, pathe_1.join)(dataDir, file) : file;
         const targetPath = basePath ? (0, pathe_1.join)(basePath, targetFile) : targetFile;
-        fileCount += 1;
+        files.push(file);
         filesManifest[file] = targetPath;
         const dest = (0, pathe_1.join)(netlifyConfig.build.publish, targetPath);
         try {
@@ -112,8 +110,8 @@ const moveStaticPages = async ({ netlifyConfig, target, i18n, basePath, }) => {
         }
     };
     // Move all static files, except error documents and nft manifests
-    const pages = await (0, globby_1.default)(['{app,pages}/**/*.{html,json,rsc}', '!**/(500|404|*.js.nft).{html,json}'], {
-        cwd: outputDir,
+    const pages = await (0, globby_1.default)(['**/*.{html,json}', '!**/(500|404|*.js.nft).{html,json}'], {
+        cwd: root,
         dot: true,
     });
     const matchingMiddleware = new Set();
@@ -151,7 +149,7 @@ const moveStaticPages = async ({ netlifyConfig, target, i18n, basePath, }) => {
         return limit(moveFile, filePath);
     });
     await Promise.all(promises);
-    console.log(`Moved ${fileCount} files`);
+    console.log(`Moved ${files.length} files`);
     if (matchedPages.size !== 0) {
         console.log((0, chalk_1.yellowBright)((0, outdent_1.outdent) `
         Skipped moving ${matchedPages.size} ${matchedPages.size === 1 ? 'file because it matches' : 'files because they match'} middleware, so cannot be deployed to the CDN and will be served from the origin instead.

package/lib/helpers/functions.js

@@ -3,7 +3,7 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
     return (mod && mod.__esModule) ? mod : { "default": mod };
 };
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.warnOnApiRoutes = exports.getApiRouteConfigs = exports.setupImageFunction = exports.generatePagesResolver = exports.generateFunctions = void 0;
+exports.warnOnApiRoutes = exports.getExtendedApiRouteConfigs = exports.getApiRouteConfigs = exports.setupImageFunction = exports.generatePagesResolver = exports.generateFunctions = void 0;
 const node_bridge_1 = __importDefault(require("@vercel/node-bridge"));
 const chalk_1 = __importDefault(require("chalk"));
 const destr_1 = __importDefault(require("destr"));
@@ -20,13 +20,10 @@ const utils_1 = require("./utils");
 const generateFunctions = async ({ FUNCTIONS_SRC = constants_1.DEFAULT_FUNCTIONS_SRC, INTERNAL_FUNCTIONS_SRC, PUBLISH_DIR }, appDir, apiRoutes) => {
     const publish = (0, pathe_1.resolve)(PUBLISH_DIR);
     const functionsDir = (0, pathe_1.resolve)(INTERNAL_FUNCTIONS_SRC || FUNCTIONS_SRC);
+    console.log({ functionsDir });
     const functionDir = (0, pathe_1.join)(functionsDir, constants_1.HANDLER_FUNCTION_NAME);
     const publishDir = (0, pathe_1.relative)(functionDir, publish);
     for (const { route, config, compiled } of apiRoutes) {
-        // Don't write a lambda if the runtime is edge
-        if (config.runtime === 'experimental-edge') {
-            continue;
-        }
         const apiHandlerSource = await (0, getApiHandler_1.getApiHandler)({
             page: route,
             config,
@@ -128,12 +125,21 @@ const getApiRouteConfigs = async (publish, baseDir) => {
     const pages = await (0, fs_extra_1.readJSON)((0, pathe_1.join)(publish, 'server', 'pages-manifest.json'));
     const apiRoutes = Object.keys(pages).filter((page) => page.startsWith('/api/'));
     const pagesDir = (0, pathe_1.join)(baseDir, 'pages');
-    return Promise.all(apiRoutes.map(async (apiRoute) => {
+    return await Promise.all(apiRoutes.map(async (apiRoute) => {
         const filePath = (0, files_1.getSourceFileForPage)(apiRoute, pagesDir);
         return { route: apiRoute, config: await (0, analysis_1.extractConfigFromFile)(filePath), compiled: pages[apiRoute] };
     }));
 };
 exports.getApiRouteConfigs = getApiRouteConfigs;
+/**
+ * Looks for extended API routes (background and scheduled functions) and extract the config from the source file.
+ */
+const getExtendedApiRouteConfigs = async (publish, baseDir) => {
+    const settledApiRoutes = await (0, exports.getApiRouteConfigs)(publish, baseDir);
+    // We only want to return the API routes that are background or scheduled functions
+    return settledApiRoutes.filter((apiRoute) => apiRoute.config.type !== undefined);
+};
+exports.getExtendedApiRouteConfigs = getExtendedApiRouteConfigs;
 /**
  * Warn the user of the caveats if they're using background or scheduled API routes
  */
@@ -152,7 +158,7 @@ const warnOnApiRoutes = async ({ FUNCTIONS_DIST, }) => {
     }
     if (functions.some((func) => func.schedule)) {
         console.warn((0, outdent_1.outdent) `
-        ${chalk_1.default.yellowBright `Using scheduled API routes`} 
+        ${chalk_1.default.yellowBright `Using scheduled API routes`}
         These are run on a schedule when deployed to production.
         You can test them locally by loading them in your browser but this will not be available when deployed, and any returned value is ignored.
       `);

package/lib/index.js

@@ -44,7 +44,7 @@ const plugin = {
         }
         const { publish } = netlifyConfig.build;
         (0, verification_1.checkNextSiteHasBuilt)({ publish, failBuild });
-        const { appDir, basePath, i18n, images, target, ignore, trailingSlash, outdir, experimental, routesManifest } = await (0, config_1.getNextConfig)({
+        const { appDir, basePath, i18n, images, target, ignore, trailingSlash, outdir, experimental } = await (0, config_1.getNextConfig)({
             publish,
             failBuild,
         });
@@ -92,7 +92,7 @@ const plugin = {
         }
         const buildId = (0, fs_extra_1.readFileSync)((0, path_1.join)(publish, 'BUILD_ID'), 'utf8').trim();
         await (0, config_1.configureHandlerFunctions)({ netlifyConfig, ignore, publish: (0, path_1.relative)(process.cwd(), publish) });
-        const apiRoutes = await (0, functions_1.getApiRouteConfigs)(publish, appDir);
+        const apiRoutes = await (0, functions_1.getExtendedApiRouteConfigs)(publish, appDir);
         await (0, functions_1.generateFunctions)(constants, appDir, apiRoutes);
         await (0, functions_1.generatePagesResolver)({ target, constants });
         await (0, files_1.movePublicFiles)({ appDir, outdir, publish });
@@ -118,7 +118,7 @@ const plugin = {
             buildId,
             apiRoutes,
         });
-        await (0, edge_1.writeEdgeFunctions)({ netlifyConfig, routesManifest });
+        await (0, edge_1.writeEdgeFunctions)(netlifyConfig);
     },
     async onPostBuild({ netlifyConfig: { build: { publish }, redirects, headers, }, utils: { status, cache, functions, build: { failBuild }, }, constants: { FUNCTIONS_DIST }, }) {
         await (0, cache_1.saveCache)({ cache, publish });

package/lib/templates/getApiHandler.js

@@ -105,7 +105,7 @@ const getApiHandler = ({ page, config, publishDir = '../../../.next', appDir = '
   const { config }  = require("${publishDir}/required-server-files.json")
   let staticManifest
   const path = require("path");
-  const pageRoot = path.resolve(path.join(__dirname, "${publishDir}", "server"));
+  const pageRoot = path.resolve(path.join(__dirname, "${publishDir}", "serverless", "pages"));
   const handler = (${makeHandler.toString()})(config, "${appDir}", pageRoot, ${JSON.stringify(page)})
   exports.handler = ${config.type === "experimental-scheduled" /* ApiRouteType.SCHEDULED */ ? `schedule(${JSON.stringify(config.schedule)}, handler);` : 'handler'}
 `;

package/lib/templates/getHandler.js

@@ -15,7 +15,6 @@ const { augmentFsModule, getMaxAge, getMultiValueHeaders, getNextServer } = requ
 // eslint-disable-next-line max-params
 const makeHandler = (conf, app, pageRoot, staticManifest = [], mode = 'ssr') => {
     var _a;
-    console.log('makeHandler', process.env);
     // Change working directory into the site root, unless using Nx, which moves the
     // dist directory and handles this itself
     const dir = path.resolve(__dirname, app);
@@ -38,7 +37,9 @@ const makeHandler = (conf, app, pageRoot, staticManifest = [], mode = 'ssr') =>
     for (const [key, value] of Object.entries(conf.env)) {
         process.env[key] = String(value);
     }
-    augmentFsModule({ promises, staticManifest, pageRoot });
+    // Set during the request as it needs the host header. Hoisted so we can define the function once
+    let base;
+    augmentFsModule({ promises, staticManifest, pageRoot, getBase: () => base });
     // We memoize this because it can be shared between requests, but don't instantiate it until
     // the first request because we need the host and port.
     let bridge;
@@ -48,6 +49,9 @@ const makeHandler = (conf, app, pageRoot, staticManifest = [], mode = 'ssr') =>
         }
         const url = new URL(event.rawUrl);
         const port = Number.parseInt(url.port) || 80;
+        const { host } = event.headers;
+        const protocol = event.headers['x-forwarded-proto'] || 'http';
+        base = `${protocol}://${host}`;
         const NextServer = getNextServer();
         const nextServer = new NextServer({
             conf,
@@ -72,8 +76,6 @@ const makeHandler = (conf, app, pageRoot, staticManifest = [], mode = 'ssr') =>
     };
     return async function handler(event, context) {
         var _a, _b, _c;
-        const baseUrl = process.env.CONTEXT === 'production' ? process.env.URL : process.env.DEPLOY_PRIME_URL;
-        console.log('Using baseUrl', baseUrl, process.env);
         let requestMode = mode;
         // Ensure that paths are encoded - but don't double-encode them
         event.path = new URL(event.rawUrl).pathname;
@@ -140,7 +142,7 @@ const getHandler = ({ isODB = false, publishDir = '../../../.next', appDir = '..
     staticManifest = require("${publishDir}/static-manifest.json")
   } catch {}
   const path = require("path");
-  const pageRoot = path.resolve(path.join(__dirname, "${publishDir}", "server"));
+  const pageRoot = path.resolve(path.join(__dirname, "${publishDir}", config.target === "server" ? "server" : "serverless", "pages"));
   exports.handler = ${isODB
     ? `builder((${makeHandler.toString()})(config, "${appDir}", pageRoot, staticManifest, 'odb'));`
     : `(${makeHandler.toString()})(config, "${appDir}", pageRoot, staticManifest, 'ssr');`}

package/lib/templates/getPageResolver.js

@@ -5,31 +5,26 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.getSinglePageResolver = exports.getPageResolver = void 0;
 const path_1 = require("path");
-const fs_extra_1 = require("fs-extra");
-const globby_1 = __importDefault(require("globby"));
 const outdent_1 = require("outdent");
 const pathe_1 = require("pathe");
 const slash_1 = __importDefault(require("slash"));
+const tiny_glob_1 = __importDefault(require("tiny-glob"));
 const constants_1 = require("../constants");
 const files_1 = require("../helpers/files");
 // Generate a file full of require.resolve() calls for all the pages in the
 // build. This is used by the nft bundler to find all the pages.
 const getPageResolver = async ({ publish, target }) => {
-    const functionDir = (0, path_1.resolve)((0, path_1.join)('.netlify', 'functions', constants_1.HANDLER_FUNCTION_NAME));
-    const root = (0, path_1.resolve)((0, slash_1.default)(publish), target === 'server' ? 'server' : 'serverless');
-    const pages = await (0, globby_1.default)('{pages,app}/**/*.js.nft.json', {
+    const functionDir = path_1.posix.resolve(path_1.posix.join('.netlify', 'functions', constants_1.HANDLER_FUNCTION_NAME));
+    const root = path_1.posix.resolve((0, slash_1.default)(publish), target === 'server' ? 'server' : 'serverless', 'pages');
+    const pages = await (0, tiny_glob_1.default)('**/*.js', {
         cwd: root,
         dot: true,
     });
-    const dependencies = await Promise.all(pages.map(async (page) => {
-        const dir = (0, path_1.dirname)(page);
-        const { files } = await (0, fs_extra_1.readJSON)((0, path_1.join)(root, page));
-        return files === null || files === void 0 ? void 0 : files.map((file) => (0, path_1.resolve)(root, dir, file));
-    }));
-    const deduped = [...new Set(dependencies.flat())];
-    const pageFiles = deduped.map((file) => `require.resolve('${(0, pathe_1.relative)(functionDir, file)}')`).sort();
+    const pageFiles = pages
+        .map((page) => `require.resolve('${path_1.posix.relative(functionDir, path_1.posix.join(root, (0, slash_1.default)(page)))}')`)
+        .sort();
     return (0, outdent_1.outdent) `
-    // This file is purely to allow nft to know about these pages.
+    // This file is purely to allow nft to know about these pages. It should be temporary.
     exports.resolvePages = () => {
         try {
             ${pageFiles.join('\n        ')}

package/lib/templates/handlerUtils.js

@@ -75,7 +75,7 @@ exports.getMultiValueHeaders = getMultiValueHeaders;
 /**
  * Monkey-patch the fs module to download missing files from the CDN
  */
-const augmentFsModule = ({ promises, staticManifest, pageRoot, }) => {
+const augmentFsModule = ({ promises, staticManifest, pageRoot, getBase, }) => {
     // Only do this if we have some static files moved to the CDN
     if (staticManifest.length === 0) {
         return;
@@ -93,24 +93,22 @@ const augmentFsModule = ({ promises, staticManifest, pageRoot, }) => {
     const statsOrig = promises.stat;
     // ...then money-patch it to see if it's requesting a CDN file
     promises.readFile = (async (file, options) => {
-        // In production use the public URL (e.g. https://example.com). Otherwise use the deploy URL, e.g. https://deploy-preview-123--example.netlify.app
-        const baseUrl = process.env.CONTEXT === 'production' ? process.env.URL : process.env.DEPLOY_PRIME_URL;
-        console.log('Using baseUrl', baseUrl);
+        const base = getBase();
         // We only care about page files
         if (file.startsWith(pageRoot)) {
-            // We only want the part after `.next/server/`
+            // We only want the part after `pages/`
             const filePath = file.slice(pageRoot.length + 1);
             // Is it in the CDN and not local?
             if (staticFiles.has(filePath) && !(0, fs_1.existsSync)(file)) {
                 // This name is safe to use, because it's one that was already created by Next
                 const cacheFile = path_1.default.join(cacheDir, filePath);
-                const url = `${baseUrl}/${staticFiles.get(filePath)}`;
+                const url = `${base}/${staticFiles.get(filePath)}`;
                 // If it's already downloading we can wait for it to finish
                 if (downloadPromises.has(url)) {
                     await downloadPromises.get(url);
                 }
                 // Have we already cached it? We download every time if running locally to avoid staleness
-                if ((!(0, fs_1.existsSync)(cacheFile) || process.env.NETLIFY_DEV) && baseUrl) {
+                if ((!(0, fs_1.existsSync)(cacheFile) || process.env.NETLIFY_DEV) && base) {
                     await promises.mkdir(path_1.default.dirname(cacheFile), { recursive: true });
                     try {
                         // Append the path to our host and we can load it like a regular page
@@ -131,7 +129,7 @@ const augmentFsModule = ({ promises, staticManifest, pageRoot, }) => {
     promises.stat = ((file, options) => {
         // We only care about page files
         if (file.startsWith(pageRoot)) {
-            // We only want the part after `.next/server`
+            // We only want the part after `pages/`
             const cacheFile = path_1.default.join(cacheDir, file.slice(pageRoot.length + 1));
             if ((0, fs_1.existsSync)(cacheFile)) {
                 return statsOrig(cacheFile, options);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@netlify/plugin-nextjs",
-  "version": "4.28.4-appdir.0",
+  "version": "4.28.4",
   "description": "Run Next.js seamlessly on Netlify",
   "main": "index.js",
   "files": [
@@ -36,7 +36,7 @@
   },
   "devDependencies": {
     "@delucis/if-env": "^1.1.2",
-    "@netlify/build": "^28.1.4",
+    "@netlify/build": "^28.1.7",
     "@types/fs-extra": "^9.0.13",
     "@types/jest": "^27.4.1",
     "@types/merge-stream": "^1.1.2",

package/src/templates/edge/runtime.ts

@@ -13,12 +13,6 @@ export interface FetchEventResult {
   waitUntil: Promise<any>
 }
 
-export interface I18NConfig {
-  defaultLocale: string
-  localeDetection?: false
-  locales: string[]
-}
-
 export interface RequestData {
   geo?: {
     city?: string
@@ -32,7 +26,7 @@ export interface RequestData {
   method: string
   nextConfig?: {
     basePath?: string
-    i18n?: I18NConfig | null
+    i18n?: Record<string, unknown>
     trailingSlash?: boolean
   }
   page?: {
@@ -56,10 +50,10 @@ declare global {
 globalThis.NFRequestContextMap ||= new Map()
 
 const handler = async (req: Request, context: Context) => {
-  // if (Deno.env.get('NETLIFY_DEV')) {
-  //   // Don't run in dev
-  //   return
-  // }
+  if (Deno.env.get('NETLIFY_DEV')) {
+    // Don't run in dev
+    return
+  }
 
   const url = new URL(req.url)