scriptoriumadmin/fabro
1
0
Fork 0
mirror of https://github.com/fabro-sh/fabro.git synced 2026-09-06 08:18:58 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 3

Cache start node ID before main loop to avoid repeated O(n) lookup

Browse source 
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This commit is contained in:
Bryan Helmkamp 2026-03-08 03:09:39 -04:00
parent a3abbaf191
commit 58e08bec60
1 changed files with 3 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
crates/arc-workflows/src/engine.rs
View file

@ -1410,6 +1410,8 @@ impl WorkflowRunEngine {
(token, shutdown)
});
let start_node_id = graph.find_start_node().map(|n| n.id.clone());
loop {
// Check for cancellation before processing each node
if let Some(ref token) = config.cancel_token {
@ -1894,11 +1896,7 @@ impl WorkflowRunEngine {
// Step 6b: Write shadow branch first, then run branch commit with trailer
// Skip git checkpoint for the start node — it's a no-op, so the commit is always empty.
let is_start_node = graph
.find_start_node()
.map(|n| n.id == node.id)
.unwrap_or(false);
if !is_start_node {
if start_node_id.as_deref() != Some(&*node.id) {
if let Some(ref mode) = config.git_checkpoint {
// Shadow commit (best-effort): extract repo path from either variant
let shadow_sha: Option<String> = if config.meta_branch.is_some() {