fix(pi): recreate closed update PRs

Author: Bill Leoutsakos

apps/docs/content/docs/en/workflows/blocks/pi.mdx

@@ -61,7 +61,7 @@ Update PR uses the same disposable authoring sandbox and host-controlled PR oper
 - Requires sandbox execution and **your own provider API key (BYOK)**.
 - Needs the same **GitHub token permissions as Create PR**: permission to clone, push, and create or update a pull request.
 - Never creates, rebases, merges, or force-pushes the branch. If another commit reaches the branch while Pi is working, the push fails rather than overwriting it.
-- Finds the exact open same-repository PR whose head is the target branch. One match is updated; no open match—including when an earlier PR was closed—creates a new PR after authoring; multiple open matches fail as ambiguous. A discovered PR that closes, moves, or no longer matches during validation aborts safely.
+- Finds the exact open same-repository PR whose head is the target branch and checks again after authoring. One match is updated; no open match—including when an earlier or preflight PR was closed—creates a new PR; multiple open matches fail as ambiguous.
 - When explicitly set, **Base Branch**, **PR Title**, **PR Body**, and **PR State** update the existing PR. Blank metadata preserves it. A newly created PR uses generated metadata and the repository default base when those fields are blank.
 - A no-change authoring pass still creates or updates the PR. A rejected initial push stops the run before any PR mutation or Babysit continuation.
 - The deliverable is the updated branch and its PR — read `prUrl`, `branch`, `changedFiles`, and `diff`.

apps/sim/executor/handlers/pi/cloud-backend.test.ts

@@ -108,6 +108,11 @@ function mockExistingBranchPullRequest(): void {
       output: { items: [{ number: 7 }], count: 1 },
     })
     .mockResolvedValueOnce(existingPullRequestOutput())
+    .mockResolvedValueOnce({
+      success: true,
+      output: { items: [{ number: 7 }], count: 1 },
+    })
+    .mockResolvedValueOnce(existingPullRequestOutput())
 }
 
 describe('runCloudPi', () => {
@@ -798,11 +803,16 @@ describe('runCloudPi', () => {
     })
 
     it('fails when a second PR for the branch appears during authoring', async () => {
-      mockExistingBranchPullRequest()
-      mockExecuteTool.mockResolvedValueOnce({
-        success: true,
-        output: { items: [{ number: 7 }, { number: 8 }], count: 2 },
-      })
+      mockExecuteTool
+        .mockResolvedValueOnce({
+          success: true,
+          output: { items: [{ number: 7 }], count: 1 },
+        })
+        .mockResolvedValueOnce(existingPullRequestOutput())
+        .mockResolvedValueOnce({
+          success: true,
+          output: { items: [{ number: 7 }, { number: 8 }], count: 2 },
+        })
 
       await expect(runCloudBranchPi(branchParams(), { onEvent: vi.fn() })).rejects.toThrow(
         /multiple open pull requests/
@@ -813,6 +823,32 @@ describe('runCloudPi', () => {
       expect(mockRunBabysit).not.toHaveBeenCalled()
     })
 
+    it('creates a replacement when the preflight PR is no longer open after authoring', async () => {
+      mockExecuteTool
+        .mockResolvedValueOnce({
+          success: true,
+          output: { items: [{ number: 7 }], count: 1 },
+        })
+        .mockResolvedValueOnce(existingPullRequestOutput())
+        .mockResolvedValueOnce({
+          success: true,
+          output: { items: [], count: 0 },
+        })
+
+      const result = await runCloudBranchPi(branchParams(), { onEvent: vi.fn() })
+
+      expect(mockExecuteTool).toHaveBeenCalledWith(
+        'github_create_pr',
+        expect.objectContaining({
+          head: 'feature/existing',
+          base: 'main',
+          draft: true,
+        }),
+        { signal: undefined }
+      )
+      expect(result.prUrl).toBe('https://github.com/octo/demo/pull/1')
+    })
+
     it('does not claim a push happened when no-op authoring is followed by a PR error', async () => {
       mockRun.mockImplementation((command: string) => {
         if (command.includes('git clone')) {

apps/sim/executor/handlers/pi/cloud-backend.ts

@@ -309,7 +309,6 @@ async function ensureUpdatePullRequest(
     if (currentPullRequest && currentPullRequest.pullNumber !== existingPullRequest.pullNumber) {
       throw new Error(`The open pull request for branch ${branch} changed during authoring`)
     }
-    return updatePullRequest(params, existingPullRequest, secrets, signal)
   }
   if (currentPullRequest) {
     return updatePullRequest(params, currentPullRequest, secrets, signal)