From 0f8ef7756a0a9088bff4b980257dba3db1a75a45 Mon Sep 17 00:00:00 2001 From: Arjun Komath Date: Sat, 22 Aug 2026 11:32:42 +1000 Subject: [PATCH] Remove duplicate preview comment status --- web/lib/preview-deployments.ts | 8 ++++++-- web/tests/preview-deployments.test.ts | 3 +++ 2 files changed, 9 insertions(+), 2 deletions(-) diff --git a/web/lib/preview-deployments.ts b/web/lib/preview-deployments.ts index c7d26f61..015a1268 100644 --- a/web/lib/preview-deployments.ts +++ b/web/lib/preview-deployments.ts @@ -84,8 +84,12 @@ async function updatePreviewPullRequestComment(input: { input.previewUrl ? `**Preview:** [Open preview](${input.previewUrl})` : "**Preview:** No public URL configured", - "", - `${escapeGitHubCommentText(input.description.substring(0, 500))}`, + ...(input.state === "failure" + ? [ + "", + `${escapeGitHubCommentText(input.description.substring(0, 500))}`, + ] + : []), ].join("\n"); await upsertGitHubPullRequestComment( input.installationId, diff --git a/web/tests/preview-deployments.test.ts b/web/tests/preview-deployments.test.ts index 8e6efd0f..f4108b77 100644 --- a/web/tests/preview-deployments.test.ts +++ b/web/tests/preview-deployments.test.ts @@ -404,6 +404,9 @@ describe("preview service cloning", () => { }), ); expect(mocks.upsertGitHubPullRequestComment).toHaveBeenCalledTimes(2); + expect( + mocks.upsertGitHubPullRequestComment.mock.calls[0]?.[4], + ).not.toContain(""); expect(mocks.upsertGitHubPullRequestComment).toHaveBeenLastCalledWith( 101, "acme/app",