From 80ac3e63b3f5de1d4344e7f55489fc1d1ee5b497 Mon Sep 17 00:00:00 2001 From: Terraphim CI Date: Mon, 6 Apr 2026 00:27:24 +0100 Subject: [PATCH] fix(ci): resolve three pre-existing CI failures - Remove unused TerraphimAgent import in multi_agent test (clippy -D warnings) - Use sh -c instead of sh -lc for workspace hook execution (login shell sources /home/alex/tools/rust/cargo/env which doesn't exist on CI runners) - Add --features zlob to ci-check-format.sh clippy command (fff-search build.rs panics in CI without zlob feature enabled) Co-Authored-By: Claude Opus 4.6 --- crates/terraphim_multi_agent/tests/simple_agent_test.rs | 4 +--- crates/terraphim_workspace/src/lib.rs | 2 +- scripts/ci-check-format.sh | 2 +- 3 files changed, 3 insertions(+), 5 deletions(-) diff --git a/crates/terraphim_multi_agent/tests/simple_agent_test.rs b/crates/terraphim_multi_agent/tests/simple_agent_test.rs index 45d617f13..8b82000c9 100644 --- a/crates/terraphim_multi_agent/tests/simple_agent_test.rs +++ b/crates/terraphim_multi_agent/tests/simple_agent_test.rs @@ -1,6 +1,4 @@ -use terraphim_multi_agent::{ - agent::AgentStatus, test_utils::*, CommandInput, CommandType, TerraphimAgent, -}; +use terraphim_multi_agent::{agent::AgentStatus, test_utils::*, CommandInput, CommandType}; #[tokio::test] async fn test_agent_creation_simple() { diff --git a/crates/terraphim_workspace/src/lib.rs b/crates/terraphim_workspace/src/lib.rs index 549383484..a278fa772 100644 --- a/crates/terraphim_workspace/src/lib.rs +++ b/crates/terraphim_workspace/src/lib.rs @@ -266,7 +266,7 @@ impl WorkspaceManager { tracing::debug!(hook = hook_name, cwd = %cwd.display(), "running hook"); let mut cmd = Command::new("sh"); - cmd.arg("-lc") + cmd.arg("-c") .arg(script) .current_dir(cwd) .stdout(std::process::Stdio::piped()) diff --git a/scripts/ci-check-format.sh b/scripts/ci-check-format.sh index 8b3eec132..1daccc703 100755 --- a/scripts/ci-check-format.sh +++ b/scripts/ci-check-format.sh @@ -109,7 +109,7 @@ fi # - useless_vec, items_after_test_module, module_inception: test organization # - bool_comparison, nonminimal_bool: test boolean expressions # - redundant_clone: performance not critical in tests -if timeout 1200 cargo clippy --workspace --all-targets --message-format=short -- \ +if timeout 1200 cargo clippy --workspace --all-targets --features zlob --message-format=short -- \ -D clippy::all \ -A clippy::nursery -A clippy::pedantic \ -A dead_code -A unused \