fix(ci): omit null conclusion from check update

test/pr-e2e-gate-fork-skip.test.ts

@@ -1240,12 +1240,12 @@ describe("PR E2E controller fork credentialed E2E skip approval safety", () => {
       expect(restoredAuthorizations).toHaveLength(2);
       expect(restoredAuthorizations[0]?.body).toMatchObject({
         status: "in_progress",
-        conclusion: null,
         output: {
           title: "Maintainer authorization required to run E2E",
           summary: expect.stringContaining("launch a fresh first-attempt `run-control-plane`"),
         },
       });
+      expect(restoredAuthorizations[0]?.body).not.toHaveProperty("conclusion");
       expect(checkTitle).toBe("Maintainer authorization required to run E2E");
       expect(requests.some((request) => request.url.endsWith("/dispatches"))).toBe(false);
     } finally {

test/pr-e2e-gate-retry-history.test.ts

@@ -246,14 +246,14 @@ describe("PR E2E controller retry history", () => {
         .at(-1);
       expect(completion?.body).toMatchObject({
         status: "in_progress",
-        conclusion: null,
         output: {
           title: "Maintainer authorization required to run E2E",
           summary: expect.stringContaining(
             "No selected E2E job ran and no repository secret was exposed",
           ),
         },
       });
+      expect(completion?.body).not.toHaveProperty("conclusion");
       expect(checkRuns[0]).toEqual(completedCheck);
       expect(checkRuns[1]).toMatchObject({
         id: 18,

tools/e2e/pr-e2e-gate.mts

@@ -1057,7 +1057,7 @@ async function markCheckInProgress(
     token,
     {
       method: "PATCH",
-      body: { status: "in_progress", conclusion: null, output: { title, summary } },
+      body: { status: "in_progress", output: { title, summary } },
       userAgent: USER_AGENT,
     },
   );