cloud: default to current branch in cloud exec#7460
Conversation
|
All contributors have signed the CLA ✍️ ✅ |
|
I have read the CLA Document and I hereby sign the CLA |
There was a problem hiding this comment.
Pull request overview
This PR adds branch override functionality for cloud-executed tasks, allowing users to specify a custom git reference via a --branch flag. The implementation introduces a shared git reference resolver that consolidates duplicate logic previously used in both codex cloud exec and TUI task submission.
Key Changes:
- Introduced a
resolve_git_reffunction with testable abstraction for git operations - Added
--branchflag to theExecCommandCLI - Replaced duplicate git reference resolution logic with the new shared function
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| codex-rs/cloud-tasks/src/lib.rs | Adds GitInfoProvider trait, resolve_git_ref functions, updates run_exec_command to use branch override, refactors TUI submission to use shared resolver, and adds comprehensive unit tests |
| codex-rs/cloud-tasks/src/cli.rs | Adds optional --branch parameter to ExecCommand struct |
| codex-rs/cloud-tasks/Cargo.toml | Moves async-trait from dev-dependencies to regular dependencies |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
tamird
left a comment
tamird
left a comment
There was a problem hiding this comment.
Could you please give me some context? I'm not sure why I was chosen as a reviewer for this PR. Thanks!
|
@tamird github thought you were relevant. |
|
It thought wrong! |
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
|
@squinlan-oai, PR has been reviewed and approved. Please address the merge conflicts, and then it should be ready. |
codex-rs/cloud-tasks/src/lib.rs
Outdated
| if let Some(branch) = git_info.default_branch_name(&cwd).await { | ||
| branch | ||
| } else if let Some(branch) = git_info.current_branch_name(&cwd).await { |
There was a problem hiding this comment.
should probably be the other way around? i.e. current branch first, otherwise default branch, otherwise "main"?
There was a problem hiding this comment.
Codex couldn't complete this request. Try again later.
nornagon-openai
changed the title
nornagon-openai
deleted the
codex/update-codex-cloud-exec-for-git-ref
branch
5 participants
Summary
codex cloud execand TUI task submission--branchflag to override the git ref passed to cloud tasksTesting
Codex Task