From 8d472bb6ed72ed2f89a4e6e30f81c23d2b1511d7 Mon Sep 17 00:00:00 2001 From: Bryan Helmkamp Date: Thu, 23 Apr 2026 16:51:09 -0400 Subject: [PATCH] use run settings builder in manifest preflight --- lib/crates/fabro-server/src/run_manifest.rs | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/lib/crates/fabro-server/src/run_manifest.rs b/lib/crates/fabro-server/src/run_manifest.rs index a12ca2842..768a97b6c 100644 --- a/lib/crates/fabro-server/src/run_manifest.rs +++ b/lib/crates/fabro-server/src/run_manifest.rs @@ -4,7 +4,7 @@ use std::sync::Arc; use anyhow::{Result, anyhow, bail}; use fabro_api::types; -use fabro_config::{WorkflowSettingsBuilder, parse_settings_layer}; +use fabro_config::{RunSettingsBuilder, WorkflowSettingsBuilder, parse_settings_layer}; use fabro_graphviz::graph::{Graph, is_llm_handler_type}; use fabro_graphviz::render::apply_direction; use fabro_llm::Provider; @@ -389,12 +389,12 @@ async fn build_preflight_report( &configured_providers, ); let resolved_run = - WorkflowSettingsBuilder::from_layer(&materialized).map_err(|errors| anyhow!(errors))?; + RunSettingsBuilder::from_layer(&materialized).map_err(anyhow::Error::from)?; let server_settings = state.server_settings(); let github_integration = &server_settings.server.integrations.github; - let sandbox_provider = resolve_sandbox_provider(&resolved_run.run)?; + let sandbox_provider = resolve_sandbox_provider(&resolved_run)?; let sandbox_provider = - if resolved_run.run.execution.mode == RunMode::DryRun && !sandbox_provider.is_local() { + if resolved_run.execution.mode == RunMode::DryRun && !sandbox_provider.is_local() { SandboxProvider::Local } else { sandbox_provider @@ -414,7 +414,7 @@ async fn build_preflight_report( &mut checks, sandbox_provider, prepared, - &resolved_run.run, + &resolved_run, github_app.clone(), daytona_api_key, ) @@ -423,7 +423,7 @@ async fn build_preflight_report( state, &mut checks, graph, - &resolved_run.run, + &resolved_run, &configured_providers, ) .await;