From e968ece42c90e077d09adce6c5ebb7dded2bcc7e Mon Sep 17 00:00:00 2001 From: Bryan Helmkamp Date: Fri, 13 Mar 2026 09:56:59 -0400 Subject: [PATCH] rename git trailers from Arc-* to Fabro-* to match product rename Co-Authored-By: Claude Opus 4.6 --- lib/crates/fabro-workflows/src/engine.rs | 8 ++++---- .../fabro-workflows/src/handler/parallel.rs | 2 +- .../fabro-workflows/tests/daytona_integration.rs | 12 ++++++------ lib/crates/fabro-workflows/tests/integration.rs | 16 ++++++++-------- 4 files changed, 19 insertions(+), 19 deletions(-) diff --git a/lib/crates/fabro-workflows/src/engine.rs b/lib/crates/fabro-workflows/src/engine.rs index 9c16f4481..6437c4189 100644 --- a/lib/crates/fabro-workflows/src/engine.rs +++ b/lib/crates/fabro-workflows/src/engine.rs @@ -571,22 +571,22 @@ pub async fn git_checkpoint( } // Build commit message with trailers (same format as checkpoint_commit in git.rs) - let subject = format!("arc({run_id}): {node_id} ({status})"); + let subject = format!("fabro({run_id}): {node_id} ({status})"); let completed_str = completed_count.to_string(); let mut trailers = vec![ Trailer { - key: "Arc-Run", + key: "Fabro-Run", value: run_id, }, Trailer { - key: "Arc-Completed", + key: "Fabro-Completed", value: &completed_str, }, ]; let shadow_sha_ref = shadow_sha.as_deref().unwrap_or(""); if shadow_sha.is_some() { trailers.push(Trailer { - key: "Arc-Checkpoint", + key: "Fabro-Checkpoint", value: shadow_sha_ref, }); } diff --git a/lib/crates/fabro-workflows/src/handler/parallel.rs b/lib/crates/fabro-workflows/src/handler/parallel.rs index 4c69e902f..91a0c59c4 100644 --- a/lib/crates/fabro-workflows/src/handler/parallel.rs +++ b/lib/crates/fabro-workflows/src/handler/parallel.rs @@ -457,7 +457,7 @@ impl Handler for ParallelHandler { .exec_command(&add_cmd, 30_000, None, None, None) .await; if add_result.as_ref().is_ok_and(|r| r.exit_code == 0) { - let msg = format!("arc({rid}): {nid} ({status_str})"); + let msg = format!("fabro({rid}): {nid} ({status_str})"); let commit_cmd = format!( "{git_r} -c 'user.name={name}' -c 'user.email={email}' commit --allow-empty -m '{msg}'", name = git_author.name, diff --git a/lib/crates/fabro-workflows/tests/daytona_integration.rs b/lib/crates/fabro-workflows/tests/daytona_integration.rs index c0c634456..3227ef2b6 100644 --- a/lib/crates/fabro-workflows/tests/daytona_integration.rs +++ b/lib/crates/fabro-workflows/tests/daytona_integration.rs @@ -1058,7 +1058,7 @@ async fn daytona_cli_gemini() { use fabro_workflows::git::MetadataStore; /// End-to-end test: pipeline with git checkpointing enabled + `meta_branch` -/// writes shadow branch on the host repo and includes `Arc-Checkpoint` trailer in sandbox commits. +/// writes shadow branch on the host repo and includes `Fabro-Checkpoint` trailer in sandbox commits. #[tokio::test] #[ignore] async fn daytona_git_checkpoint_with_shadow_branch() { @@ -1191,7 +1191,7 @@ async fn daytona_git_checkpoint_with_shadow_branch() { "checkpoint should contain the 'work' node" ); - // Assert sandbox commit has Arc-Checkpoint trailer + // Assert sandbox commit has Fabro-Checkpoint trailer let log_result = env .exec_command("git log --format=%B -1", 10_000, None, None, None) .await @@ -1199,12 +1199,12 @@ async fn daytona_git_checkpoint_with_shadow_branch() { assert_eq!(log_result.exit_code, 0); let commit_msg = log_result.stdout.trim().to_string(); assert!( - commit_msg.contains("Arc-Checkpoint:"), - "sandbox commit should have Arc-Checkpoint trailer, got:\n{commit_msg}" + commit_msg.contains("Fabro-Checkpoint:"), + "sandbox commit should have Fabro-Checkpoint trailer, got:\n{commit_msg}" ); assert!( - commit_msg.contains("Arc-Run:"), - "sandbox commit should have Arc-Run trailer, got:\n{commit_msg}" + commit_msg.contains("Fabro-Run:"), + "sandbox commit should have Fabro-Run trailer, got:\n{commit_msg}" ); // Assert final.patch exists diff --git a/lib/crates/fabro-workflows/tests/integration.rs b/lib/crates/fabro-workflows/tests/integration.rs index 081e364bd..dc55fca51 100644 --- a/lib/crates/fabro-workflows/tests/integration.rs +++ b/lib/crates/fabro-workflows/tests/integration.rs @@ -10760,7 +10760,7 @@ async fn git_checkpoint_host_emits_events_and_diff_patch() { } /// End-to-end test: pipeline with git checkpointing enabled + `meta_branch` writes -/// shadow branch with checkpoint data and includes `Arc-Checkpoint` trailer in run-branch commits. +/// shadow branch with checkpoint data and includes `Fabro-Checkpoint` trailer in run-branch commits. #[tokio::test] async fn git_checkpoint_host_writes_shadow_branch() { use fabro_workflows::git::MetadataStore; @@ -10893,7 +10893,7 @@ async fn git_checkpoint_host_writes_shadow_branch() { "checkpoint should contain the 'work' node" ); - // 7. Assert run-branch commit has Arc-Checkpoint trailer pointing to shadow SHA + // 7. Assert run-branch commit has Fabro-Checkpoint trailer pointing to shadow SHA let output = std::process::Command::new("git") .args(["log", "--format=%B", "-1"]) .current_dir(&worktree_path) @@ -10901,16 +10901,16 @@ async fn git_checkpoint_host_writes_shadow_branch() { .unwrap(); let commit_msg = String::from_utf8_lossy(&output.stdout).trim().to_string(); assert!( - commit_msg.contains("Arc-Checkpoint:"), - "run-branch commit should have Arc-Checkpoint trailer, got:\n{commit_msg}" + commit_msg.contains("Fabro-Checkpoint:"), + "run-branch commit should have Fabro-Checkpoint trailer, got:\n{commit_msg}" ); assert!( - commit_msg.contains("Arc-Run:"), - "run-branch commit should have Arc-Run trailer, got:\n{commit_msg}" + commit_msg.contains("Fabro-Run:"), + "run-branch commit should have Fabro-Run trailer, got:\n{commit_msg}" ); assert!( - commit_msg.contains("Arc-Completed:"), - "run-branch commit should have Arc-Completed trailer, got:\n{commit_msg}" + commit_msg.contains("Fabro-Completed:"), + "run-branch commit should have Fabro-Completed trailer, got:\n{commit_msg}" ); // 8. Verify round-trip: shadow checkpoint's completed_nodes matches expected