fix: intercept routes being unintentionally stripped (#312)

.changeset/young-beers-buy.md

@@ -0,0 +1,5 @@
+---
+'@cloudflare/next-on-pages': patch
+---
+
+Prevent the route group stripping regex from removing intercept routes.

src/utils/routing.ts

@@ -6,11 +6,14 @@ import { normalizePath } from './fs';
  * The build output config does not rewrite requests to route groups, so we need to strip route
  * groups from the path name for file system matching.
  *
+ * Does not strip brackets containing `.` characters, as these are used for route intercepts.
+ * https://nextjs.org/docs/app/building-your-application/routing/intercepting-routes
+ *
  * @param path Path name to strip route groups from.
  * @returns Path name with route groups stripped.
  */
 export function stripRouteGroups(path: string) {
-	return path.replace(/\/\(([^)]+)\)/g, '');
+	return path.replace(/\/\(([^).]+)\)/g, '');
 }
 
 /**

tests/src/utils/routing.test.ts

@@ -14,6 +14,13 @@ describe('routing', () => {
 		expect(stripRouteGroups('/(route-group)/path')).toEqual('/path');
 	});
 
+	test('stripRouteGroups does not strip intercepts', () => {
+		expect(stripRouteGroups('/(.)intercept')).toEqual('/(.)intercept');
+		expect(stripRouteGroups('/(..)(..)intercept')).toEqual(
+			'/(..)(..)intercept'
+		);
+	});
+
 	test('stripIndexRoute', () => {
 		expect(stripIndexRoute('/index')).toEqual('/');
 		expect(stripIndexRoute('/path/index')).toEqual('/path');