From 11c69fee43508bb35269d9f833998ed64eb083ba Mon Sep 17 00:00:00 2001 From: Cliff Date: Tue, 2 Jun 2026 04:23:42 +0000 Subject: [PATCH] fix(acp): make send_error available in release builds MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit thread_goal.rs:392 calls self.send_error() unconditionally, but the method was gated behind #[cfg(debug_assertions)], so it was compiled out of release builds and the call failed with E0599. CI only compiles with the ci-test profile (inherits test → debug-assertions on), so the method existed there and the break only surfaced in the release job's `cargo build --release`. Remove the gate so the method exists in all profiles; goal-update errors now surface to the user in release too. 🤖 Generated with [Nori](https://noriagentic.com) Co-Authored-By: Nori --- nori-rs/acp/src/backend/submit_and_ops.rs | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/nori-rs/acp/src/backend/submit_and_ops.rs b/nori-rs/acp/src/backend/submit_and_ops.rs index b7d90cd78..e65d5fc2a 100644 --- a/nori-rs/acp/src/backend/submit_and_ops.rs +++ b/nori-rs/acp/src/backend/submit_and_ops.rs @@ -293,8 +293,7 @@ impl AcpBackend { Ok(()) } - /// Send an error event to the TUI (only used in debug builds). - #[cfg(debug_assertions)] + /// Send an error event to the TUI. pub(super) async fn send_error(&self, message: &str) { let _ = self .event_tx