From 71d5c49b829e28c15f6345154520f46acbd12d28 Mon Sep 17 00:00:00 2001
From: Claude <noreply@anthropic.com>
Date: Sat, 21 Mar 2026 09:11:52 +0000
Subject: [PATCH] fix: remove duplicated cfg(test) attribute and fix import
 order

Remove redundant #![cfg(test)] inner attribute from codex_test_helpers.rs
(already gated by #[cfg(test)] in mod.rs), fixing the clippy
duplicated_attributes warning. Also apply cargo fmt import reordering
in tools/mod.rs.

https://claude.ai/code/session_017ckCCurNiBL8uzE4dJg59K
---
 src/llm/codex_test_helpers.rs | 2 --
 src/tools/mod.rs              | 2 +-
 2 files changed, 1 insertion(+), 3 deletions(-)

diff --git a/src/llm/codex_test_helpers.rs b/src/llm/codex_test_helpers.rs
index 2368d6e6..64c0b3a3 100644
--- a/src/llm/codex_test_helpers.rs
+++ b/src/llm/codex_test_helpers.rs
@@ -1,7 +1,5 @@
 //! Shared test helpers for OpenAI Codex provider tests.
 
-#![cfg(test)]
-
 use crate::config::OpenAiCodexConfig;
 
 /// Build a minimal JWT for testing (header.payload.signature).
diff --git a/src/tools/mod.rs b/src/tools/mod.rs
index 83438b76..7e62a6fc 100644
--- a/src/tools/mod.rs
+++ b/src/tools/mod.rs
@@ -26,13 +26,13 @@ pub use autonomy::{
     AUTONOMOUS_TOOL_DENYLIST, autonomous_allowed_tool_names, autonomous_unavailable_error,
     autonomous_unavailable_message, is_autonomous_tool_denylisted,
 };
-pub use executor::{PtcError, PtcToolResult, ToolExecutor};
 pub use builder::{
     BuildPhase, BuildRequirement, BuildResult, BuildSoftwareTool, BuilderConfig, Language,
     LlmSoftwareBuilder, SoftwareBuilder, SoftwareType, Template, TemplateEngine, TemplateType,
     TestCase, TestHarness, TestResult, TestSuite, ValidationError, ValidationResult, WasmValidator,
 };
 pub(crate) use coercion::prepare_tool_params;
+pub use executor::{PtcError, PtcToolResult, ToolExecutor};
 pub use rate_limiter::RateLimiter;
 pub use registry::ToolRegistry;
 pub use tool::{