Skip to main content

oven_cli/git/
mod.rs

1use std::path::{Path, PathBuf};
2
3use anyhow::{Context, Result};
4use tokio::process::Command;
5
6/// A git worktree created for an issue pipeline.
7#[derive(Debug, Clone)]
8pub struct Worktree {
9    pub path: PathBuf,
10    pub branch: String,
11    pub issue_number: u32,
12}
13
14/// Info about an existing worktree from `git worktree list`.
15#[derive(Debug, Clone)]
16pub struct WorktreeInfo {
17    pub path: PathBuf,
18    pub branch: Option<String>,
19}
20
21/// Generate a branch name for an issue: `oven/issue-{number}-{short_hex}`.
22fn branch_name(issue_number: u32) -> String {
23    let short_hex = &uuid::Uuid::new_v4().to_string()[..8];
24    format!("oven/issue-{issue_number}-{short_hex}")
25}
26
27/// Create a worktree for the given issue, branching from `base_branch`.
28pub async fn create_worktree(
29    repo_dir: &Path,
30    issue_number: u32,
31    base_branch: &str,
32) -> Result<Worktree> {
33    let branch = branch_name(issue_number);
34    let worktree_path =
35        repo_dir.join(".oven").join("worktrees").join(format!("issue-{issue_number}"));
36
37    // Ensure parent directory exists
38    if let Some(parent) = worktree_path.parent() {
39        tokio::fs::create_dir_all(parent).await.context("creating worktree parent directory")?;
40    }
41
42    run_git(
43        repo_dir,
44        &["worktree", "add", "-b", &branch, &worktree_path.to_string_lossy(), base_branch],
45    )
46    .await
47    .context("creating worktree")?;
48
49    Ok(Worktree { path: worktree_path, branch, issue_number })
50}
51
52/// Remove a worktree by path.
53pub async fn remove_worktree(repo_dir: &Path, worktree_path: &Path) -> Result<()> {
54    run_git(repo_dir, &["worktree", "remove", "--force", &worktree_path.to_string_lossy()])
55        .await
56        .context("removing worktree")?;
57    Ok(())
58}
59
60/// List all worktrees in the repository.
61pub async fn list_worktrees(repo_dir: &Path) -> Result<Vec<WorktreeInfo>> {
62    let output = run_git(repo_dir, &["worktree", "list", "--porcelain"])
63        .await
64        .context("listing worktrees")?;
65
66    let mut worktrees = Vec::new();
67    let mut current_path: Option<PathBuf> = None;
68    let mut current_branch: Option<String> = None;
69
70    for line in output.lines() {
71        if let Some(path_str) = line.strip_prefix("worktree ") {
72            // Save previous worktree if we have one
73            if let Some(path) = current_path.take() {
74                worktrees.push(WorktreeInfo { path, branch: current_branch.take() });
75            }
76            current_path = Some(PathBuf::from(path_str));
77        } else if let Some(branch_ref) = line.strip_prefix("branch ") {
78            // Extract branch name from refs/heads/...
79            current_branch =
80                Some(branch_ref.strip_prefix("refs/heads/").unwrap_or(branch_ref).to_string());
81        }
82    }
83
84    // Don't forget the last one
85    if let Some(path) = current_path {
86        worktrees.push(WorktreeInfo { path, branch: current_branch });
87    }
88
89    Ok(worktrees)
90}
91
92/// Prune stale worktrees and return the count pruned.
93pub async fn clean_worktrees(repo_dir: &Path) -> Result<u32> {
94    let before = list_worktrees(repo_dir).await?;
95    run_git(repo_dir, &["worktree", "prune"]).await.context("pruning worktrees")?;
96    let after = list_worktrees(repo_dir).await?;
97
98    let pruned = if before.len() > after.len() { before.len() - after.len() } else { 0 };
99    Ok(u32::try_from(pruned).unwrap_or(u32::MAX))
100}
101
102/// Delete a local branch.
103pub async fn delete_branch(repo_dir: &Path, branch: &str) -> Result<()> {
104    run_git(repo_dir, &["branch", "-D", branch]).await.context("deleting branch")?;
105    Ok(())
106}
107
108/// List merged branches matching `oven/*`.
109pub async fn list_merged_branches(repo_dir: &Path, base: &str) -> Result<Vec<String>> {
110    let output = run_git(repo_dir, &["branch", "--merged", base])
111        .await
112        .context("listing merged branches")?;
113
114    let branches = output
115        .lines()
116        .map(|l| l.trim().trim_start_matches("* ").to_string())
117        .filter(|b| b.starts_with("oven/"))
118        .collect();
119
120    Ok(branches)
121}
122
123/// Create an empty commit (used to seed a branch before PR creation).
124pub async fn empty_commit(repo_dir: &Path, message: &str) -> Result<()> {
125    run_git(repo_dir, &["commit", "--allow-empty", "-m", message])
126        .await
127        .context("creating empty commit")?;
128    Ok(())
129}
130
131/// Push a branch to origin.
132pub async fn push_branch(repo_dir: &Path, branch: &str) -> Result<()> {
133    run_git(repo_dir, &["push", "origin", branch]).await.context("pushing branch")?;
134    Ok(())
135}
136
137/// Force-push a branch to origin using `--force-with-lease` for safety.
138///
139/// Used after rebasing a pipeline branch onto the updated base branch.
140pub async fn force_push_branch(repo_dir: &Path, branch: &str) -> Result<()> {
141    let lease = format!("--force-with-lease=refs/heads/{branch}");
142    run_git(repo_dir, &["push", &lease, "origin", branch]).await.context("force-pushing branch")?;
143    Ok(())
144}
145
146/// Rebase the current branch onto the latest `origin/<base_branch>`.
147///
148/// Fetches the base branch first, then attempts a rebase. If the rebase fails
149/// (merge conflicts), it aborts the rebase and returns an error.
150pub async fn rebase_on_base(repo_dir: &Path, base_branch: &str) -> Result<()> {
151    run_git(repo_dir, &["fetch", "origin", base_branch])
152        .await
153        .context("fetching base branch before rebase")?;
154
155    let target = format!("origin/{base_branch}");
156    if run_git(repo_dir, &["rebase", &target]).await.is_ok() {
157        return Ok(());
158    }
159
160    let _ = run_git(repo_dir, &["rebase", "--abort"]).await;
161    anyhow::bail!("merge conflicts with {base_branch} that could not be automatically resolved")
162}
163
164/// Get the default branch name (main or master).
165pub async fn default_branch(repo_dir: &Path) -> Result<String> {
166    // Try symbolic-ref first
167    if let Ok(output) = run_git(repo_dir, &["symbolic-ref", "refs/remotes/origin/HEAD"]).await {
168        if let Some(branch) = output.strip_prefix("refs/remotes/origin/") {
169            return Ok(branch.to_string());
170        }
171    }
172
173    // Fallback: check if main exists, otherwise master
174    if run_git(repo_dir, &["rev-parse", "--verify", "main"]).await.is_ok() {
175        return Ok("main".to_string());
176    }
177    if run_git(repo_dir, &["rev-parse", "--verify", "master"]).await.is_ok() {
178        return Ok("master".to_string());
179    }
180
181    // Last resort: whatever HEAD points to
182    let output = run_git(repo_dir, &["rev-parse", "--abbrev-ref", "HEAD"])
183        .await
184        .context("detecting default branch")?;
185    Ok(output)
186}
187
188async fn run_git(repo_dir: &Path, args: &[&str]) -> Result<String> {
189    let output = Command::new("git")
190        .args(args)
191        .current_dir(repo_dir)
192        .kill_on_drop(true)
193        .output()
194        .await
195        .context("spawning git")?;
196
197    if !output.status.success() {
198        let stderr = String::from_utf8_lossy(&output.stderr);
199        anyhow::bail!("git {} failed: {}", args.join(" "), stderr.trim());
200    }
201
202    Ok(String::from_utf8_lossy(&output.stdout).trim().to_string())
203}
204
205#[cfg(test)]
206mod tests {
207    use super::*;
208
209    async fn init_temp_repo() -> tempfile::TempDir {
210        let dir = tempfile::tempdir().unwrap();
211
212        // Init a repo with an initial commit so we have a branch to work from
213        Command::new("git").args(["init"]).current_dir(dir.path()).output().await.unwrap();
214
215        Command::new("git")
216            .args(["config", "user.email", "test@test.com"])
217            .current_dir(dir.path())
218            .output()
219            .await
220            .unwrap();
221
222        Command::new("git")
223            .args(["config", "user.name", "Test"])
224            .current_dir(dir.path())
225            .output()
226            .await
227            .unwrap();
228
229        tokio::fs::write(dir.path().join("README.md"), "hello").await.unwrap();
230
231        Command::new("git").args(["add", "."]).current_dir(dir.path()).output().await.unwrap();
232
233        Command::new("git")
234            .args(["commit", "-m", "initial"])
235            .current_dir(dir.path())
236            .output()
237            .await
238            .unwrap();
239
240        dir
241    }
242
243    #[tokio::test]
244    async fn create_and_remove_worktree() {
245        let dir = init_temp_repo().await;
246
247        // Detect the current branch name
248        let branch = run_git(dir.path(), &["rev-parse", "--abbrev-ref", "HEAD"]).await.unwrap();
249
250        let wt = create_worktree(dir.path(), 42, &branch).await.unwrap();
251        assert!(wt.path.exists());
252        assert!(wt.branch.starts_with("oven/issue-42-"));
253        assert_eq!(wt.issue_number, 42);
254
255        remove_worktree(dir.path(), &wt.path).await.unwrap();
256        assert!(!wt.path.exists());
257    }
258
259    #[tokio::test]
260    async fn list_worktrees_includes_created() {
261        let dir = init_temp_repo().await;
262        let branch = run_git(dir.path(), &["rev-parse", "--abbrev-ref", "HEAD"]).await.unwrap();
263
264        let _wt = create_worktree(dir.path(), 99, &branch).await.unwrap();
265
266        let worktrees = list_worktrees(dir.path()).await.unwrap();
267        // Should have at least the main worktree + the one we created
268        assert!(worktrees.len() >= 2);
269        assert!(
270            worktrees
271                .iter()
272                .any(|w| { w.branch.as_deref().is_some_and(|b| b.starts_with("oven/issue-99-")) })
273        );
274    }
275
276    #[tokio::test]
277    async fn branch_naming_convention() {
278        let name = branch_name(123);
279        assert!(name.starts_with("oven/issue-123-"));
280        assert_eq!(name.len(), "oven/issue-123-".len() + 8);
281        // The hex part should be valid hex
282        let hex_part = &name["oven/issue-123-".len()..];
283        assert!(hex_part.chars().all(|c| c.is_ascii_hexdigit()));
284    }
285
286    #[tokio::test]
287    async fn default_branch_detection() {
288        let dir = init_temp_repo().await;
289        let branch = default_branch(dir.path()).await.unwrap();
290        // git init creates "main" or "master" depending on config
291        assert!(branch == "main" || branch == "master", "got: {branch}");
292    }
293
294    #[tokio::test]
295    async fn error_on_non_git_dir() {
296        let dir = tempfile::tempdir().unwrap();
297        let result = list_worktrees(dir.path()).await;
298        assert!(result.is_err());
299    }
300
301    #[tokio::test]
302    async fn rebase_on_base_clean() {
303        let dir = init_temp_repo().await;
304        let branch = run_git(dir.path(), &["rev-parse", "--abbrev-ref", "HEAD"]).await.unwrap();
305
306        // Create a feature branch with a non-conflicting change
307        run_git(dir.path(), &["checkout", "-b", "feature"]).await.unwrap();
308        tokio::fs::write(dir.path().join("feature.txt"), "feature work").await.unwrap();
309        run_git(dir.path(), &["add", "."]).await.unwrap();
310        run_git(dir.path(), &["commit", "-m", "feature commit"]).await.unwrap();
311
312        // Add a non-conflicting commit on the base branch
313        run_git(dir.path(), &["checkout", &branch]).await.unwrap();
314        tokio::fs::write(dir.path().join("base.txt"), "base work").await.unwrap();
315        run_git(dir.path(), &["add", "."]).await.unwrap();
316        run_git(dir.path(), &["commit", "-m", "base commit"]).await.unwrap();
317
318        // Go back to feature branch and rebase
319        run_git(dir.path(), &["checkout", "feature"]).await.unwrap();
320
321        // rebase_on_base fetches from origin, so we need a remote.
322        // Use the repo itself as origin for testing.
323        run_git(dir.path(), &["remote", "add", "origin", &dir.path().to_string_lossy()])
324            .await
325            .unwrap();
326
327        let result = rebase_on_base(dir.path(), &branch).await;
328        assert!(result.is_ok());
329
330        // Verify both files exist after rebase
331        assert!(dir.path().join("feature.txt").exists());
332        assert!(dir.path().join("base.txt").exists());
333    }
334
335    #[tokio::test]
336    async fn rebase_on_base_conflict_aborts() {
337        let dir = init_temp_repo().await;
338        let branch = run_git(dir.path(), &["rev-parse", "--abbrev-ref", "HEAD"]).await.unwrap();
339
340        // Create a feature branch with a conflicting change to README.md
341        run_git(dir.path(), &["checkout", "-b", "feature"]).await.unwrap();
342        tokio::fs::write(dir.path().join("README.md"), "feature version").await.unwrap();
343        run_git(dir.path(), &["add", "."]).await.unwrap();
344        run_git(dir.path(), &["commit", "-m", "feature conflict"]).await.unwrap();
345
346        // Add a conflicting commit on the base branch
347        run_git(dir.path(), &["checkout", &branch]).await.unwrap();
348        tokio::fs::write(dir.path().join("README.md"), "base version").await.unwrap();
349        run_git(dir.path(), &["add", "."]).await.unwrap();
350        run_git(dir.path(), &["commit", "-m", "base conflict"]).await.unwrap();
351
352        // Go back to feature and set up origin
353        run_git(dir.path(), &["checkout", "feature"]).await.unwrap();
354        run_git(dir.path(), &["remote", "add", "origin", &dir.path().to_string_lossy()])
355            .await
356            .unwrap();
357
358        let result = rebase_on_base(dir.path(), &branch).await;
359        assert!(result.is_err());
360        assert!(
361            result.unwrap_err().to_string().contains("merge conflicts"),
362            "error should mention merge conflicts"
363        );
364
365        // Verify rebase was aborted (no .git/rebase-merge directory)
366        assert!(!dir.path().join(".git/rebase-merge").exists());
367    }
368
369    #[tokio::test]
370    async fn force_push_branch_works() {
371        let dir = init_temp_repo().await;
372
373        // Set up a bare remote to push to
374        let remote_dir = tempfile::tempdir().unwrap();
375        Command::new("git")
376            .args(["clone", "--bare", &dir.path().to_string_lossy(), "."])
377            .current_dir(remote_dir.path())
378            .output()
379            .await
380            .unwrap();
381
382        run_git(dir.path(), &["remote", "add", "origin", &remote_dir.path().to_string_lossy()])
383            .await
384            .unwrap();
385
386        // Create a branch, push it, then amend and force-push
387        run_git(dir.path(), &["checkout", "-b", "test-branch"]).await.unwrap();
388        tokio::fs::write(dir.path().join("new.txt"), "v1").await.unwrap();
389        run_git(dir.path(), &["add", "."]).await.unwrap();
390        run_git(dir.path(), &["commit", "-m", "v1"]).await.unwrap();
391        push_branch(dir.path(), "test-branch").await.unwrap();
392
393        // Amend the commit (simulating a rebase)
394        tokio::fs::write(dir.path().join("new.txt"), "v2").await.unwrap();
395        run_git(dir.path(), &["add", "."]).await.unwrap();
396        run_git(dir.path(), &["commit", "--amend", "-m", "v2"]).await.unwrap();
397
398        // Regular push should fail, force push should succeed
399        assert!(push_branch(dir.path(), "test-branch").await.is_err());
400        assert!(force_push_branch(dir.path(), "test-branch").await.is_ok());
401    }
402}