pages: Fix "worker called response.clone()" warnings in `wrangler pag…

…es dev`

Fixes #3259

fixtures/pages-functions-with-routes-app/functions/api/greet.ts

@@ -0,0 +1,8 @@
+export async function onRequestPost(context) {
+	try {
+		let data = await context.request.json();
+		return new Response(`[/functions/api/greet]: ${data.greeting}`);
+	} catch (err) {
+		return new Response("Error parsing JSON content", { status: 400 });
+	}
+}

fixtures/pages-functions-with-routes-app/functions/greeting/hello.ts

@@ -1,3 +1,13 @@
-export async function onRequest() {
-	return new Response("[/functions/greeting/hello]: Bonjour le monde!");
+export async function onRequest(context) {
+	const url = new URL(context.request.url);
+
+	const response = await fetch(`${url.origin}/api/greet`, {
+		method: "POST",
+		headers: {
+			"Content-Type": "application/json",
+		},
+		body: JSON.stringify({ greeting: "Bonjour le monde!" }),
+	});
+
+	return new Response(`[/functions/greeting/hello]: ${await response.text()}`);
 }

fixtures/pages-functions-with-routes-app/public/_routes.json

@@ -1,5 +1,5 @@
 {
 	"version": 1,
-	"include": ["/", "/greeting/*", "/greeting*", "/date"],
+	"include": ["/", "/greeting/*", "/greeting*", "/date", "/api/*"],
 	"exclude": ["/bye*", "/date", "/*.*"]
 }

fixtures/pages-functions-with-routes-app/tests/index.test.ts

@@ -42,7 +42,9 @@ describe("Pages Functions with custom _routes.json", () => {
 		// matches /greeting/* include rule
 		response = await fetch(`http://${ip}:${port}/greeting/hello`);
 		text = await response.text();
-		expect(text).toEqual("[/functions/greeting/hello]: Bonjour le monde!");
+		expect(text).toEqual(
+			"[/functions/greeting/hello]: [/functions/api/greet]: Bonjour le monde!"
+		);
 
 		// matches /greeting/* include rule
 		response = await fetch(`http://${ip}:${port}/greeting/bye`);

fixtures/pages-workerjs-with-routes-app/public/_routes.json

@@ -1,5 +1,5 @@
 {
 	"version": 1,
-	"include": ["/greeting/*", "/date", "/party*"],
+	"include": ["/greeting/*", "/date", "/party*", "/api/*"],
 	"exclude": ["/", "/party"]
 }

fixtures/pages-workerjs-with-routes-app/public/_worker.js

@@ -25,7 +25,17 @@ export default {
 		}
 
 		if (url.pathname === "/greeting/hello") {
-			return new Response("[/greeting/hello]: Bonjour le monde!");
+			const url = new URL(request.url);
+
+			const response = await fetch(`${url.origin}/api/greet`, {
+				method: "POST",
+				headers: {
+					"Content-Type": "application/json",
+				},
+				body: JSON.stringify({ greeting: "Bonjour le monde!" }),
+			});
+
+			return new Response(`[/greeting/hello]: ${await response.text()}`);
 		}
 
 		if (url.pathname === "/greeting/bye") {
@@ -36,6 +46,10 @@ export default {
 			return new Response("[/greetings]: Bonjour alligators!");
 		}
 
+		if (url.pathname === "/api/greet") {
+			return new Response("[/api/greet]: Bonjour le monde!");
+		}
+
 		return env.ASSETS.fetch(request);
 	},
 };

fixtures/pages-workerjs-with-routes-app/tests/index.test.ts

@@ -30,7 +30,7 @@ describe("Pages Advanced Mode with custom _routes.json", () => {
 		// matches /greeting/* include rule
 		let response = await fetch(`http://${ip}:${port}/greeting/hello`);
 		let text = await response.text();
-		expect(text).toEqual("[/greeting/hello]: Bonjour le monde!");
+		expect(text).toEqual("[/greeting/hello]: [/api/greet]: Bonjour le monde!");
 
 		// matches /greeting/* include rule
 		response = await fetch(`http://${ip}:${port}/greeting/bye`);

packages/wrangler/templates/pages-template-plugin.ts

@@ -146,7 +146,7 @@ export default function (pluginArgs: unknown) {
 			if (result.done === false) {
 				const { handler, params, path } = result.value;
 				const context = {
-					request: new Request(request.clone()),
+					request: new Request(request),
 					functionPath: workerContext.functionPath + path,
 					next: pluginNext,
 					params,

packages/wrangler/templates/pages-template-worker.ts

@@ -137,7 +137,7 @@ export default {
 			if (result.done === false) {
 				const { handler, params, path } = result.value;
 				const context = {
-					request: new Request(request.clone()),
+					request: new Request(request),
 					functionPath: path,
 					next,
 					params,