codex: fix CI failure on PR #15211

Author: eternal-openai

codex-rs/hooks/src/events/pre_tool_use.rs

@@ -74,16 +74,20 @@ pub(crate) async fn run(
         };
     }
 
-    let input_json = match serde_json::to_string(&PreToolUseCommandInput::new(
-        request.session_id.to_string(),
-        request.turn_id.clone(),
-        request.transcript_path.clone(),
-        request.cwd.display().to_string(),
-        request.model.clone(),
-        request.permission_mode.clone(),
-        request.tool_use_id.clone(),
-        request.command.clone(),
-    )) {
+    let input_json = match serde_json::to_string(&PreToolUseCommandInput {
+        session_id: request.session_id.to_string(),
+        turn_id: request.turn_id.clone(),
+        transcript_path: crate::schema::NullableString::from_path(request.transcript_path.clone()),
+        cwd: request.cwd.display().to_string(),
+        hook_event_name: "PreToolUse".to_string(),
+        model: request.model.clone(),
+        permission_mode: request.permission_mode.clone(),
+        tool_name: "Bash".to_string(),
+        tool_input: crate::schema::PreToolUseToolInput {
+            command: request.command.clone(),
+        },
+        tool_use_id: request.tool_use_id.clone(),
+    }) {
         Ok(input_json) => input_json,
         Err(error) => {
             return serialization_failure_outcome(common::serialization_failure_hook_events(

codex-rs/hooks/src/schema.rs

@@ -138,34 +138,6 @@ pub(crate) struct PreToolUseCommandInput {
     pub tool_use_id: String,
 }
 
-impl PreToolUseCommandInput {
-    pub(crate) fn new(
-        session_id: impl Into<String>,
-        turn_id: impl Into<String>,
-        transcript_path: Option<PathBuf>,
-        cwd: impl Into<String>,
-        model: impl Into<String>,
-        permission_mode: impl Into<String>,
-        tool_use_id: impl Into<String>,
-        command: impl Into<String>,
-    ) -> Self {
-        Self {
-            session_id: session_id.into(),
-            turn_id: turn_id.into(),
-            transcript_path: NullableString::from_path(transcript_path),
-            cwd: cwd.into(),
-            hook_event_name: "PreToolUse".to_string(),
-            model: model.into(),
-            permission_mode: permission_mode.into(),
-            tool_name: "Bash".to_string(),
-            tool_input: PreToolUseToolInput {
-                command: command.into(),
-            },
-            tool_use_id: tool_use_id.into(),
-        }
-    }
-}
-
 #[derive(Debug, Clone, Serialize, Deserialize, JsonSchema)]
 #[serde(rename_all = "camelCase")]
 #[serde(deny_unknown_fields)]