@sveltejs/adapter-netlify 6.0.2 → 6.0.4

package/index.js

@@ -237,13 +237,14 @@ function generate_lambda_functions({ builder, publish, split }) {
 
 			const routes = [route];
 
+			/** @type {string[]} */
 			const parts = [];
-			// Netlify's syntax uses '*' and ':param' as "splats" and "placeholders"
-			// https://docs.netlify.com/routing/redirects/redirect-options/#splats
+
+			// The parts should conform to URLPattern syntax
+			// https://docs.netlify.com/build/functions/get-started/?fn-language=ts&data-tab=TypeScript#route-requests
 			for (const segment of route.segments) {
 				if (segment.rest) {
 					parts.push('*');
-					break; // Netlify redirects don't allow anything after a *
 				} else if (segment.dynamic) {
 					// URLPattern requires params to start with letters
 					parts.push(`:param${parts.length}`);
@@ -252,13 +253,16 @@ function generate_lambda_functions({ builder, publish, split }) {
 				}
 			}
 
+			// Netlify handles trailing slashes for us, so we don't need to include them in the pattern
 			const pattern = `/${parts.join('/')}`;
 			const name =
 				FUNCTION_PREFIX + (parts.join('-').replace(/[:.]/g, '_').replace('*', '__rest') || 'index');
 
 			// skip routes with identical patterns, they were already folded into another function
 			if (seen.has(pattern)) continue;
-			seen.add(pattern);
+
+			const patterns = [pattern, `${pattern === '/' ? '' : pattern}/__data.json`];
+			patterns.forEach((p) => seen.add(p));
 
 			// figure out which lower priority routes should be considered fallbacks
 			for (let j = i + 1; j < builder.routes.length; j += 1) {
@@ -270,49 +274,28 @@ function generate_lambda_functions({ builder, publish, split }) {
 				}
 			}
 
-			const manifest = builder.generateManifest({
-				relativePath: '../server',
-				routes
+			generate_serverless_function({
+				builder,
+				routes,
+				patterns,
+				name
 			});
-
-			const fn = generate_serverless_function_module(manifest);
-			const config = generate_config_export(pattern);
-
-			if (builder.hasServerInstrumentationFile()) {
-				writeFileSync(`.netlify/functions-internal/${name}.mjs`, fn);
-				builder.instrument({
-					entrypoint: `.netlify/functions-internal/${name}.mjs`,
-					instrumentation: '.netlify/server/instrumentation.server.js',
-					start: `.netlify/functions-start/${name}.start.mjs`,
-					module: {
-						generateText: generate_traced_module(config)
-					}
-				});
-			} else {
-				writeFileSync(`.netlify/functions-internal/${name}.mjs`, `${fn}\n${config}`);
-			}
 		}
+
+		generate_serverless_function({
+			builder,
+			routes: [],
+			patterns: ['/*'],
+			name: `${FUNCTION_PREFIX}catch-all`,
+			exclude: Array.from(seen)
+		});
 	} else {
-		const manifest = builder.generateManifest({
-			relativePath: '../server'
+		generate_serverless_function({
+			builder,
+			routes: undefined,
+			patterns: ['/*'],
+			name: `${FUNCTION_PREFIX}render`
 		});
-
-		const fn = generate_serverless_function_module(manifest);
-		const config = generate_config_export('/*');
-
-		if (builder.hasServerInstrumentationFile()) {
-			writeFileSync(`.netlify/functions-internal/${FUNCTION_PREFIX}render.mjs`, fn);
-			builder.instrument({
-				entrypoint: `.netlify/functions-internal/${FUNCTION_PREFIX}render.mjs`,
-				instrumentation: '.netlify/server/instrumentation.server.js',
-				start: `.netlify/functions-start/${FUNCTION_PREFIX}render.start.mjs`,
-				module: {
-					generateText: generate_traced_module(config)
-				}
-			});
-		} else {
-			writeFileSync(`.netlify/functions-internal/${FUNCTION_PREFIX}render.mjs`, `${fn}\n${config}`);
-		}
 	}
 
 	// Copy user's custom _redirects file if it exists
@@ -394,6 +377,40 @@ function matches(a, b) {
 	}
 }
 
+/**
+ *
+ * @param {{
+ *   builder: import('@sveltejs/kit').Builder,
+ *   routes: import('@sveltejs/kit').RouteDefinition[] | undefined,
+ *   patterns: string[],
+ *   name: string,
+ *   exclude?: string[]
+ * }} opts
+ */
+function generate_serverless_function({ builder, routes, patterns, name, exclude }) {
+	const manifest = builder.generateManifest({
+		relativePath: '../server',
+		routes
+	});
+
+	const fn = generate_serverless_function_module(manifest);
+	const config = generate_config_export(patterns, exclude);
+
+	if (builder.hasServerInstrumentationFile()) {
+		writeFileSync(`.netlify/functions-internal/${name}.mjs`, fn);
+		builder.instrument({
+			entrypoint: `.netlify/functions-internal/${name}.mjs`,
+			instrumentation: '.netlify/server/instrumentation.server.js',
+			start: `.netlify/functions-start/${name}.start.mjs`,
+			module: {
+				generateText: generate_traced_module(config)
+			}
+		});
+	} else {
+		writeFileSync(`.netlify/functions-internal/${name}.mjs`, `${fn}\n${config}`);
+	}
+}
+
 /**
  * @param {string} manifest
  * @returns {string}
@@ -407,14 +424,16 @@ export default init(${manifest});
 }
 
 /**
- * @param {string} pattern
+ * @param {string[]} patterns
+ * @param {string[]} [exclude]
  * @returns {string}
  */
-function generate_config_export(pattern) {
+function generate_config_export(patterns, exclude = []) {
+	// TODO: add a human friendly name for the function https://docs.netlify.com/build/frameworks/frameworks-api/#configuration-options-2
 	return `\
 export const config = {
-	path: "${pattern}",
-	excludedPath: "/.netlify/*",
+	path: [${patterns.map((s) => JSON.stringify(s)).join(', ')}],
+	excludedPath: [${['/.netlify/*', ...exclude].map((s) => JSON.stringify(s)).join(', ')}],
 	preferStatic: true
 };
 `;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sveltejs/adapter-netlify",
-  "version": "6.0.2",
+  "version": "6.0.4",
   "description": "A SvelteKit adapter that creates a Netlify app",
   "keywords": [
     "adapter",
@@ -37,7 +37,7 @@
     "esbuild": "^0.25.4"
   },
   "devDependencies": {
-    "@netlify/dev": "^4.8.8",
+    "@netlify/dev": "^4.11.2",
     "@netlify/edge-functions": "^3.0.0",
     "@netlify/functions": "^5.0.0",
     "@netlify/node-cookies": "^0.1.0",
@@ -50,7 +50,7 @@
     "rollup": "^4.14.2",
     "typescript": "^5.3.3",
     "vitest": "^4.0.0",
-    "@sveltejs/kit": "^2.52.2"
+    "@sveltejs/kit": "^2.53.1"
   },
   "peerDependencies": {
     "@sveltejs/kit": "^2.31.0"