From 6db816e8b367526448ce5b6e4fdcb7a0e65e01e6 Mon Sep 17 00:00:00 2001 From: Bryan Helmkamp Date: Sat, 14 Mar 2026 15:00:40 -0400 Subject: [PATCH] Fix empty run_id on sandbox events in progress.jsonl The JSONL listener's run_id was initialized to "" and only populated when WorkflowRunStarted fired, but sandbox events emit before that. Seed it with the already-generated ULID so all events carry the run_id. Co-Authored-By: Claude Opus 4.6 (1M context) --- lib/crates/fabro-workflows/src/cli/run.rs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/crates/fabro-workflows/src/cli/run.rs b/lib/crates/fabro-workflows/src/cli/run.rs index 9cd2eeb3b..a429d7a5e 100644 --- a/lib/crates/fabro-workflows/src/cli/run.rs +++ b/lib/crates/fabro-workflows/src/cli/run.rs @@ -507,7 +507,7 @@ pub async fn run_command( { let jsonl_path = run_dir.join("progress.jsonl"); let live_path = run_dir.join("live.json"); - let run_id = Arc::new(Mutex::new(String::new())); + let run_id = Arc::new(Mutex::new(run_id.clone())); let run_id_clone = Arc::clone(&run_id); emitter.on_event(move |event| { if let crate::event::WorkflowRunEvent::WorkflowRunStarted { run_id, .. } = event {