From ba9988d4ee09a964d4b5f2d1add25999a5b373ed Mon Sep 17 00:00:00 2001 From: Bryan Helmkamp Date: Fri, 20 Mar 2026 19:06:17 -0400 Subject: [PATCH] Destructure ModelInfo to avoid unnecessary clone Co-Authored-By: Claude Opus 4.6 (1M context) --- lib/crates/fabro-workflows/src/transform.rs | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/lib/crates/fabro-workflows/src/transform.rs b/lib/crates/fabro-workflows/src/transform.rs index 9f8898705..f9337951b 100644 --- a/lib/crates/fabro-workflows/src/transform.rs +++ b/lib/crates/fabro-workflows/src/transform.rs @@ -125,14 +125,16 @@ impl Transform for ModelResolutionTransform { .map(String::from); if let Some(model) = model { if let Some(info) = fabro_model::get_model_info(&model) { + let canonical_id = info.id; + let provider = info.provider; // Resolve alias to canonical model ID - if model != info.id { + if model != canonical_id { node.attrs - .insert("model".to_string(), AttrValue::String(info.id.clone())); + .insert("model".to_string(), AttrValue::String(canonical_id)); } if !node.attrs.contains_key("provider") { node.attrs - .insert("provider".to_string(), AttrValue::String(info.provider)); + .insert("provider".to_string(), AttrValue::String(provider)); } } }