@sveltejs/adapter-netlify 6.0.2 → 6.0.3

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,6 +253,7 @@ 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');
@@ -270,49 +272,22 @@ function generate_lambda_functions({ builder, publish, split }) {
 				}
 			}
 
-			const manifest = builder.generateManifest({
-				relativePath: '../server',
-				routes
+			generate_serverless_function({
+				builder,
+				routes,
+				patterns: [pattern, `${pattern === '/' ? '' : pattern}/__data.json`],
+				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}`);
-			}
 		}
+
+		// TODO: add a catch-all to display a 404 page if no other functions are invoked, similar to the Vercel adapter
 	} 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 +369,39 @@ function matches(a, b) {
 	}
 }
 
+/**
+ *
+ * @param {{
+ *   builder: import('@sveltejs/kit').Builder,
+ *   routes: import('@sveltejs/kit').RouteDefinition[] | undefined,
+ *   patterns: string[],
+ *   name: string
+ * }} opts
+ */
+function generate_serverless_function({ builder, routes, patterns, name }) {
+	const manifest = builder.generateManifest({
+		relativePath: '../server',
+		routes
+	});
+
+	const fn = generate_serverless_function_module(manifest);
+	const config = generate_config_export(patterns);
+
+	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,13 +415,14 @@ export default init(${manifest});
 }
 
 /**
- * @param {string} pattern
+ * @param {string[]} patterns
  * @returns {string}
  */
-function generate_config_export(pattern) {
+function generate_config_export(patterns) {
+	// 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}",
+	path: [${patterns.map((s) => JSON.stringify(s)).join(', ')}],
 	excludedPath: "/.netlify/*",
 	preferStatic: true
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sveltejs/adapter-netlify",
-  "version": "6.0.2",
+  "version": "6.0.3",
   "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.0"
   },
   "peerDependencies": {
     "@sveltejs/kit": "^2.31.0"