feat(pi): add update PR mode - #6031
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
|
cursor review |
PR SummaryMedium Risk Overview The block gains Target Branch, PR State ( Host-side GitHub logic adds exact open-PR discovery for a branch ( Docs and tests cover the new mode, visibility, and edge cases (ambiguous PRs, non-fast-forward, duplicate PR avoidance). Reviewed by Cursor Bugbot for commit 5aaf5fd. Bugbot is set up for automated code reviews on this repo. Configure here. |
Greptile SummaryAdds Update PR (
Confidence Score: 5/5This follow-up pass found no remaining blocking failures and the PR appears safe to merge from the available review context. No blocking failure remains; no prior Greptile threads were available to re-validate, and no new follow-up-eligible findings were established.
|
| Filename | Overview |
|---|---|
| apps/sim/executor/handlers/pi/cloud-backend.ts | Shared cloud authoring path for Create/Update PR with push-before-PR, ensureUpdatePullRequest, and Babysit combine. |
| apps/sim/executor/handlers/pi/github-pr.ts | Exact open same-repo PR lookup/revalidation and GraphQL draft/ready state changes. |
| apps/sim/executor/handlers/pi/pi-handler.ts | Parses cloud_branch inputs (targetBranch, prState, babysit) and dispatches runCloudBranchPi. |
| apps/sim/blocks/blocks/pi.ts | Adds Update PR mode option, targetBranch/prState fields, and CLOUD_AUTHORING conditions for shared controls. |
| apps/docs/content/docs/en/workflows/blocks/pi.mdx | Documents Update PR behavior, permissions, fields, and Babysit parity with Create PR. |
Reviews (12): Last reviewed commit: "fix(pi): follow replacement update PRs" | Re-trigger Greptile
There was a problem hiding this comment.
✅ Bugbot reviewed your changes and found no new issues!
Comment @cursor review or bugbot run to trigger another review on this PR
Reviewed by Cursor Bugbot for commit 0fc8bd1. Configure here.
|
Demo: Screen.Recording.2026-07-28.at.5.26.08.PM.mov |
0fc8bd1 to
5fa94fe
Compare
8906140 to
217ea51
Compare
|
cursor review |
|
cursor review |
There was a problem hiding this comment.
✅ Bugbot reviewed your changes and found no new issues!
Comment @cursor review or bugbot run to trigger another review on this PR
Reviewed by Cursor Bugbot for commit 5aaf5fd. Configure here.
Stacked PR
This PR is stacked on #5962 and intentionally targets
feature/pi-babysit. PR #5962 remains unchanged and targetsstaging.Do not merge this PR before #5962. After #5962 merges, this branch will be rebased onto the merged Babysit tree on
staging, retargeted tostaging, and revalidated.Summary
Add Update PR (
cloud_branch) as the fourth Pi mode, with Create PR permission and Babysit parity.HEADback with a normal non-force refspec.preserve,draft,ready). Blank metadata preserves an existing PR and is generated for a new PR.Security Boundaries
Permissions
Create PR and Update PR require the same GitHub permissions:
Type of Change
Testing
Review focus: exact PR discovery/revalidation, push-before-PR sequencing, missing-PR creation, metadata/draft transitions, no-op finalization, non-fast-forward failure, Babysit reuse, and credential isolation.
Checklist
Screenshots/Videos
Not included: this is a cloud-gated block option and behavior is covered by block visibility and handler/backend tests.