From 88ac5e10ce811152b741fa2c3bdf5ba19655ddab Mon Sep 17 00:00:00 2001 From: Kent Overstreet Date: Wed, 8 Apr 2026 16:58:05 -0400 Subject: [PATCH] Log completed assistant node after parser finishes The parser mutates the AST directly but doesn't write to the conversation log. The turn loop now logs the completed assistant branch after the parser handle resolves successfully. Co-Authored-By: Proof of Concept --- src/agent/mod.rs | 10 +++++++++- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git a/src/agent/mod.rs b/src/agent/mod.rs index cfe0e78..d647ac4 100644 --- a/src/agent/mod.rs +++ b/src/agent/mod.rs @@ -362,7 +362,15 @@ impl Agent { return Err(e); } Err(e) => return Err(anyhow::anyhow!("parser task panicked: {}", e)), - Ok(Ok(())) => {} + Ok(Ok(())) => { + let node = agent.context.lock().await.conversation()[branch_idx].clone(); + let st = agent.state.lock().await; + if let Some(ref log) = st.conversation_log { + if let Err(e) = log.append_node(&node) { + eprintln!("warning: failed to log assistant response: {:#}", e); + } + } + } } // Empty response — nudge and retry