Skip to content

Isolate report-only PR rebase from the configured checkout - #196

Merged
tzhouam merged 1 commit into
mainfrom
codex/pr-rebase-report-only-worktree-20260926
Sep 26, 2026
Merged

tzhouam merged 1 commit into
mainfrom
codex/pr-rebase-report-only-worktree-20260926

Conversation

@tzhouam

@tzhouam tzhouam commented Sep 26, 2026

Copy link
Copy Markdown
Collaborator

A report-only pr_rebase previously checked out and rebased the PR directly in the configured repository, even though the task was classified as read-only. Running it could move the user's current branch and rewrite checkout files.

Report-only runs now fetch the PR head into a detached worktree keyed to the run, perform the rebase there, and publish that private path through checkpoint state for resume. Downstream steps verify the path, repository ownership, detached HEAD, and liveness; a missing or redirected tree blocks instead of falling back to the configured checkout. Full-mode checkout and push behavior stays the same. The worktree reaper recognizes the new disposable key.

Validation: full Copilot pytest suite; focused PR/reaper tests after the final change; strict SPEC freshness, document links/citations, wheel build/import, and git diff --check.

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Sep 26, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-09-26T10:09:28.310697Z cd60892 PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@tzhouam
tzhouam merged commit 5a1069c into main Sep 26, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant