fix(underscore-redirects): respect trailingSlash config in redirects

.changeset/thin-memes-boil.md

@@ -0,0 +1,5 @@
+---
+'@astrojs/underscore-redirects': patch
+---
+
+Fixes redirect file generation to respect trailingSlash config

packages/integrations/netlify/test/functions/redirects.test.js

@@ -16,9 +16,8 @@ describe(
 		it('Creates a redirects file', async () => {
 			const redirects = await fixture.readFile('./_redirects');
 			const parts = redirects.split(/\s+/);
-			assert.deepEqual(parts, ['', '/other', '/', '301', '']);
-			// Snapshots are not supported in Node.js test yet (https://github.com/nodejs/node/issues/48260)
-			assert.equal(redirects, '\n/other    /       301\n');
+			// based on https://github.com/withastro/astro/issues/16030 for the default option `trailingSlash: 'ignore'` both variants should be generated
+			assert.deepEqual(parts, ['', '/other/', '/', '301', '/other', '/', '301', '']);
 		});
 
 		it('Does not create .html files', async () => {

packages/integrations/netlify/test/static/redirects.test.js

@@ -13,21 +13,24 @@ describe('SSG - Redirects', () => {
 	it('Creates a redirects file', async () => {
 		const redirects = await fixture.readFile('./_redirects');
 		const parts = redirects.split(/\s+/);
+		// based on https://github.com/withastro/astro/issues/16030 for the default option `trailingSlash: 'ignore'` both variants should be generated
 		assert.deepEqual(parts, [
 			'',
-
+			'/two/',
+			'/',
+			'302',
 			'/two',
 			'/',
 			'302',
-
+			'/other/',
+			'/',
+			'301',
 			'/other',
 			'/',
 			'301',
-
 			'/blog/*',
 			'/team/articles/*/index.html',
 			'301',
-
 			'',
 		]);
 	});

packages/underscore-redirects/src/astro.ts

@@ -17,7 +17,7 @@ function getRedirectStatus(route: IntegrationResolvedRoute): ValidRedirectStatus
 }
 
 interface CreateRedirectsFromAstroRoutesParams {
-	config: Pick<AstroConfig, 'build' | 'output' | 'base'>;
+	config: Pick<AstroConfig, 'build' | 'output' | 'base' | 'trailingSlash'>;
 	/**
 	 * Maps a `RouteData` to a dynamic target
 	 */
@@ -27,6 +27,35 @@ interface CreateRedirectsFromAstroRoutesParams {
 	assets: HookParameters<'astro:build:done'>['assets'];
 }
 
+/**
+ * Returns the path(s) to use for a redirect entry based on the trailingSlash config.
+ * - 'always': ensures the path ends with '/'
+ * - 'never': ensures the path does not end with '/'
+ * - 'ignore'(default): returns both with and without trailing slash variants
+ */
+function getTrailingSlashPaths(
+	inputPath: string,
+	trailingSlash: 'always' | 'never' | 'ignore',
+): string[] {
+	if (inputPath === '/') {
+		return ['/'];
+	}
+
+	const hasTrailingSlash = inputPath.endsWith('/');
+	const withoutSlash = hasTrailingSlash ? inputPath.slice(0, -1) : inputPath;
+	const withSlash = hasTrailingSlash ? inputPath : inputPath + '/';
+
+	switch (trailingSlash) {
+		case 'always':
+			return [withSlash];
+		case 'never':
+			return [withoutSlash];
+		case 'ignore':
+		default:
+			return [withoutSlash, withSlash];
+	}
+}
+
 /**
  * Takes a set of routes and creates a Redirects object from them.
  */
@@ -57,13 +86,20 @@ export function createRedirectsFromAstroRoutes({
 				// Use `entrypoint` when available to keep trailing slashes in _redirects.
 				const inputPath =
 					route.type === 'redirect' && route.entrypoint ? route.entrypoint : route.pathname;
-				redirects.add({
-					dynamic: false,
-					input: `${base}${inputPath}`,
-					target: typeof route.redirect === 'object' ? route.redirect.destination : route.redirect,
-					status: getRedirectStatus(route),
-					weight: 2,
-				});
+
+				// Generate redirect entries based on trailingSlash config.
+				const trailingSlash = config.trailingSlash ?? 'ignore';
+				const paths = getTrailingSlashPaths(inputPath, trailingSlash);
+				for (const path of paths) {
+					redirects.add({
+						dynamic: false,
+						input: `${base}${path}`,
+						target:
+							typeof route.redirect === 'object' ? route.redirect.destination : route.redirect,
+						status: getRedirectStatus(route),
+						weight: 2,
+					});
+				}
 				continue;
 			}