scriptoriumadmin/fabro
1
0
Fork 0
mirror of https://github.com/fabro-sh/fabro.git synced 2026-10-08 03:10:26 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 5

fabro(01KQT1V8JM80VEWG4ZQYJC885G): simplify_opus (succeeded)

Browse source 
Fabro-Run: 01KQT1V8JM80VEWG4ZQYJC885G
Fabro-Completed: 6
Fabro-Checkpoint: 91d11e2296

⚒️ Generated with [Fabro](https://fabro.sh)
This commit is contained in:
Fabro 2026-05-04 18:19:19 +00:00
parent d716e95d64
commit 616fdc3aa4
3 changed files with 16 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
lib/crates/fabro-workflow/src/handler/parallel.rs
View file

@ -210,7 +210,10 @@ impl Handler for ParallelHandler {
services.run.emitter.notice(
RunNoticeLevel::Warn,
"parallel_base_checkpoint_failed",
format!("Could not checkpoint base state before parallel branches: {e}"),
format!(
"Could not checkpoint base state before parallel branches: {}",
fabro_sandbox::display_for_log(&e)
),
);
None
}

13
lib/crates/fabro-workflow/src/handler/prompt.rs
View file

@ -193,6 +193,7 @@ mod tests {
use tempfile::TempDir;
use super::*;
use crate::event::Emitter;
fn make_services() -> EngineServices {
EngineServices::test_default()
@ -273,7 +274,7 @@ mod tests {
_prompt: &str,
_context: &Context,
_thread_id: Option<&str>,
_emitter: &Arc<crate::event::Emitter>,
_emitter: &Arc<Emitter>,
_sandbox: &Arc<dyn Sandbox>,
_tool_hooks: Option<Arc<dyn fabro_agent::ToolHookCallback>>,
) -> Result<CodergenResult, Error> {
@ -285,7 +286,7 @@ mod tests {
_node: &Node,
_prompt: &str,
_system_prompt: Option<&str>,
_emitter: &Arc<crate::event::Emitter>,
_emitter: &Arc<Emitter>,
_stage_scope: &StageScope,
) -> Result<CodergenResult, Error> {
Ok(CodergenResult::Text {
@ -335,7 +336,7 @@ mod tests {
_prompt: &str,
_context: &Context,
_thread_id: Option<&str>,
_emitter: &Arc<crate::event::Emitter>,
_emitter: &Arc<Emitter>,
_sandbox: &Arc<dyn Sandbox>,
_tool_hooks: Option<Arc<dyn fabro_agent::ToolHookCallback>>,
) -> Result<CodergenResult, Error> {
@ -347,7 +348,7 @@ mod tests {
_node: &Node,
_prompt: &str,
_system_prompt: Option<&str>,
_emitter: &Arc<crate::event::Emitter>,
_emitter: &Arc<Emitter>,
_stage_scope: &StageScope,
) -> Result<CodergenResult, Error> {
Ok(CodergenResult::Text {
@ -394,7 +395,7 @@ mod tests {
_prompt: &str,
_context: &Context,
_thread_id: Option<&str>,
_emitter: &Arc<crate::event::Emitter>,
_emitter: &Arc<Emitter>,
_sandbox: &Arc<dyn fabro_agent::Sandbox>,
_tool_hooks: Option<Arc<dyn fabro_agent::ToolHookCallback>>,
) -> Result<CodergenResult, Error> {
@ -406,7 +407,7 @@ mod tests {
_node: &Node,
prompt: &str,
system_prompt: Option<&str>,
_emitter: &Arc<crate::event::Emitter>,
_emitter: &Arc<Emitter>,
_stage_scope: &StageScope,
) -> Result<CodergenResult, Error> {
*self.captured_prompt.lock().unwrap() = Some(prompt.to_string());

5
lib/crates/fabro-workflow/src/pipeline/initialize.rs
View file

@ -1051,6 +1051,11 @@ mod tests {
move |event| seen.lock().unwrap().push(event.clone())
});
// The notice we assert below is emitted before any later sandbox/setup work
// that could fail in this minimal fixture (no real
// git/devcontainer/lifecycle), so the overall result is intentionally
// ignored — the assertion runs against the captured event stream
// regardless of how `initialize` ultimately resolves.
let _ = initialize(persisted, InitOptions {
run_id: test_run_id(),
run_store: {