WC-2965 Prevent selecting in-progress deployments when tailing (#9684)

If not provided with a deployment to tail, Wrangler will try to pull
the list of deployments and select the most recently created one. However,
we do not check whether this deployment has finished. This is causing
intermittent errors for users.

Co-authored-by: Mathias Lafeldt <[email protected]>

Author: WillTaylorDev

.changeset/weak-sites-itch.md

@@ -0,0 +1,5 @@
+---
+"wrangler": patch
+---
+
+Select only successfully deployed deployments when tailing.

packages/wrangler/src/__tests__/pages/pages-deployment-tail.test.ts

@@ -98,6 +98,23 @@ describe("pages deployment tail", () => {
 			await api.closeHelper();
 		});
 
+		it("only uses deployments with status=success and name=deploy", async () => {
+			setIsTTY(true);
+
+			api = mockTailAPIs("mock-deployment-id");
+			expect(api.requests.creation.length).toStrictEqual(0);
+
+			await runWrangler("pages deployment tail --project-name mock-project");
+
+			await expect(api.ws.connected).resolves.toBeTruthy();
+			expect(api.requests.creation.length).toStrictEqual(1);
+			expect(api.requests.deletion.count).toStrictEqual(0);
+
+			await api.closeHelper();
+			expect(api.requests.deletion.count).toStrictEqual(1);
+			await api.closeHelper();
+		});
+
 		it("creates and then deletes tails by deployment URL", async () => {
 			api = mockTailAPIs();
 			expect(api.requests.creation.length).toStrictEqual(0);
@@ -887,6 +904,24 @@ function mockListDeployments(): RequestLogger {
 						errors: [],
 						messages: [],
 						result: [
+							{
+								id: "mock-deployment-id-skipped",
+								url: "https://abc123.mock.pages.dev",
+								environment: "production",
+								created_on: "2020-01-17T14:52:26.133835Z",
+								latest_stage: {
+									ended_on: "2020-01-17T14:52:26.133835Z",
+									status: "skipped",
+									name: "deploy",
+								},
+								deployment_trigger: {
+									metadata: {
+										branch: "main",
+										commit_hash: "11122334c4cb32ad4f65b530b9424e8be5bec9d6",
+									},
+								},
+								project_name: "mock-project",
+							},
 							{
 								id: "mock-deployment-id",
 								url: "https://87bbc8fe.mock.pages.dev",
@@ -895,6 +930,7 @@ function mockListDeployments(): RequestLogger {
 								latest_stage: {
 									ended_on: "2021-11-17T14:52:26.133835Z",
 									status: "success",
+									name: "deploy",
 								},
 								deployment_trigger: {
 									metadata: {
@@ -921,11 +957,13 @@ function mockListDeployments(): RequestLogger {
  *
  * @returns a `RequestCounter` for counting how many times the API is hit
  */
-function mockCreateTailRequest(): RequestInit[] {
+function mockCreateTailRequest(
+	deploymentId: string = ":deploymentId"
+): RequestInit[] {
 	const requests: RequestInit[] = [];
 	msw.use(
 		http.post(
-			`*/accounts/:accountId/pages/projects/:projectName/deployments/:deploymentId/tails`,
+			`*/accounts/:accountId/pages/projects/:projectName/deployments/${deploymentId}/tails`,
 			async ({ request }) => {
 				requests.push((await request.json()) as RequestInit);
 				return HttpResponse.json(
@@ -1022,7 +1060,9 @@ const websocketURL = "ws://localhost:1234";
  * @param websocketURL a fake websocket URL for wrangler to connect to
  * @returns a mocked-out version of the API
  */
-function mockTailAPIs(): MockAPI {
+function mockTailAPIs(
+	expectedCreateDeploymentId: string = ":deploymentId"
+): MockAPI {
 	const api: MockAPI = {
 		requests: {
 			deletion: { count: 0 },
@@ -1054,7 +1094,7 @@ function mockTailAPIs(): MockAPI {
 	api.ws = new MockWebSocketServer(websocketURL);
 	mockWebSockets.push(api.ws);
 
-	api.requests.creation = mockCreateTailRequest();
+	api.requests.creation = mockCreateTailRequest(expectedCreateDeploymentId);
 	api.requests.deletion = mockDeleteTailRequest();
 	api.requests.deployments = mockListDeployments();
 

packages/wrangler/src/pages/deployment-tails.ts

@@ -177,7 +177,10 @@ export const pagesDeploymentTailCommand = createCommand({
 		);
 
 		const envDeployments = deployments.filter(
-			(d) => d.environment === environment
+			(d) =>
+				d.environment === environment &&
+				d.latest_stage.name === "deploy" &&
+				d.latest_stage.status === "success"
 		);
 
 		// Deployment is URL