Merge pull request #140 from jhlee0409/codex/fix-missing-renderer-parsing

fix: handle missing codex renderer/parsing cases

Author: jhlee0409

src-tauri/src/providers/codex.rs

@@ -849,6 +849,66 @@ fn convert_codex_event(
             }));
             Some(msg)
         }
+        "agent_reasoning" => {
+            let text = payload.get("text").and_then(Value::as_str)?.trim();
+            if text.is_empty() {
+                return None;
+            }
+            *counter += 1;
+            let content = serde_json::json!([{
+                "type": "thinking",
+                "thinking": text
+            }]);
+            Some(build_codex_message(
+                format!("codex-event-{counter}"),
+                session_id,
+                line_timestamp.to_string(),
+                "assistant",
+                Some("assistant"),
+                Some(content),
+                None,
+            ))
+        }
+        "agent_message" => {
+            let text = payload.get("message").and_then(Value::as_str)?.trim();
+            if text.is_empty() {
+                return None;
+            }
+            *counter += 1;
+            let content = serde_json::json!([{
+                "type": "text",
+                "text": text
+            }]);
+            Some(build_codex_message(
+                format!("codex-event-{counter}"),
+                session_id,
+                line_timestamp.to_string(),
+                "assistant",
+                Some("assistant"),
+                Some(content),
+                None,
+            ))
+        }
+        "user_message" => {
+            let text = payload.get("message").and_then(Value::as_str)?.trim();
+            if text.is_empty() {
+                return None;
+            }
+            *counter += 1;
+            let content = serde_json::json!([{
+                "type": "text",
+                "text": text
+            }]);
+            Some(build_codex_message(
+                format!("codex-event-{counter}"),
+                session_id,
+                line_timestamp.to_string(),
+                "user",
+                Some("user"),
+                Some(content),
+                None,
+            ))
+        }
         // Unsupported/duplicated Codex events are intentionally ignored.
         _ => None,
     }
@@ -902,7 +962,7 @@ fn extract_last_token_usage(payload: &Value) -> Option<(u32, u32)> {
 
 fn map_codex_tool_name(name: &str) -> &str {
     match name {
-        "exec_command" | "shell" => "Bash",
+        "exec_command" | "shell" | "write_stdin" => "Bash",
         _ => name,
     }
 }
@@ -1215,6 +1275,7 @@ mod tests {
     fn map_exec_command_to_bash() {
         assert_eq!(map_codex_tool_name("exec_command"), "Bash");
         assert_eq!(map_codex_tool_name("shell"), "Bash");
+        assert_eq!(map_codex_tool_name("write_stdin"), "Bash");
         assert_eq!(map_codex_tool_name("batch_execute"), "batch_execute");
     }
 
@@ -1545,6 +1606,120 @@ mod tests {
         assert_eq!(msg.subtype.as_deref(), Some("microcompact_boundary"));
     }
 
+    #[test]
+    fn convert_agent_reasoning_event_to_thinking_message() {
+        let mut counter = 0u64;
+        let msg = convert_codex_event(
+            &json!({
+                "type": "agent_reasoning",
+                "text": "**Inspecting parsers**"
+            }),
+            "session-1",
+            "2026-02-19T12:00:00Z",
+            &mut counter,
+        )
+        .expect("agent_reasoning should be converted");
+
+        assert_eq!(msg.message_type, "assistant");
+        let arr = msg
+            .content
+            .as_ref()
+            .and_then(Value::as_array)
+            .expect("content should be an array");
+        assert_eq!(arr[0].get("type").and_then(Value::as_str), Some("thinking"));
+        assert_eq!(
+            arr[0].get("thinking").and_then(Value::as_str),
+            Some("**Inspecting parsers**")
+        );
+    }
+
+    #[test]
+    fn convert_agent_reasoning_event_skips_empty_text() {
+        let mut counter = 0u64;
+        let msg = convert_codex_event(
+            &json!({
+                "type": "agent_reasoning",
+                "text": "   "
+            }),
+            "session-1",
+            "2026-02-19T12:00:00Z",
+            &mut counter,
+        );
+
+        assert!(msg.is_none());
+        assert_eq!(counter, 0);
+    }
+
+    #[test]
+    fn convert_agent_message_event_to_assistant_text_message() {
+        let mut counter = 0u64;
+        let msg = convert_codex_event(
+            &json!({
+                "type": "agent_message",
+                "message": "Working on requested changes"
+            }),
+            "session-1",
+            "2026-02-19T12:00:00Z",
+            &mut counter,
+        )
+        .expect("agent_message should be converted");
+
+        assert_eq!(msg.message_type, "assistant");
+        let arr = msg
+            .content
+            .as_ref()
+            .and_then(Value::as_array)
+            .expect("content should be an array");
+        assert_eq!(arr[0].get("type").and_then(Value::as_str), Some("text"));
+        assert_eq!(
+            arr[0].get("text").and_then(Value::as_str),
+            Some("Working on requested changes")
+        );
+    }
+
+    #[test]
+    fn convert_agent_message_event_skips_missing_field() {
+        let mut counter = 0u64;
+        let msg = convert_codex_event(
+            &json!({
+                "type": "agent_message"
+            }),
+            "session-1",
+            "2026-02-19T12:00:00Z",
+            &mut counter,
+        );
+
+        assert!(msg.is_none());
+        assert_eq!(counter, 0);
+    }
+
+    #[test]
+    fn convert_user_message_event_to_user_text_message() {
+        let mut counter = 0u64;
+        let msg = convert_codex_event(
+            &json!({
+                "type": "user_message",
+                "message": "Please patch this file"
+            }),
+            "session-1",
+            "2026-02-19T12:00:00Z",
+            &mut counter,
+        )
+        .expect("user_message should be converted");
+
+        assert_eq!(msg.message_type, "user");
+        let arr = msg
+            .content
+            .as_ref()
+            .and_then(Value::as_array)
+            .expect("content should be an array");
+        assert_eq!(arr[0].get("type").and_then(Value::as_str), Some("text"));
+        assert_eq!(
+            arr[0].get("text").and_then(Value::as_str),
+            Some("Please patch this file")
+        );
+    }
+
     #[test]
     fn convert_compacted_line_to_system_message() {
         let mut counter = 0u64;

src/components/MessageViewer/components/ClaudeMessageNode.tsx

@@ -16,6 +16,7 @@ import {
   ToolExecutionResultRouter,
   ProgressRenderer,
   AgentProgressGroupRenderer,
+  FileHistorySnapshotRenderer,
 } from "../../messageRenderer";
 import { AgentTaskGroupRenderer, TaskOperationGroupRenderer } from "../../toolResultRenderer";
 import { extractClaudeMessageContent } from "../../../utils/messageUtils";
@@ -194,6 +195,32 @@ export const ClaudeMessageNode = React.memo(({
     );
   }
 
+  // File history snapshot messages
+  if (message.type === "file-history-snapshot") {
+    if (!message.snapshot) {
+      return null;
+    }
+
+    return (
+      <div
+        data-message-uuid={message.uuid}
+        className={cn(
+          "relative w-full px-4 py-2 transition-all duration-200",
+          isCaptureMode && CAPTURE_HOVER_BG
+        )}
+      >
+        {CaptureHideButton}
+        <div className="max-w-4xl mx-auto">
+          <FileHistorySnapshotRenderer
+            messageId={message.messageId ?? message.uuid}
+            snapshot={message.snapshot}
+            isSnapshotUpdate={Boolean(message.isSnapshotUpdate)}
+          />
+        </div>
+      </div>
+    );
+  }
+
   // Progress messages
   if (message.type === "progress" && message.data) {
     return (

src/components/MessageViewer/helpers/messageHelpers.ts

@@ -29,6 +29,11 @@ export const hasSystemCommandContent = (message: ClaudeMessage): boolean => {
  * ARE considered empty because they have no user-visible content.
  */
 export const isEmptyMessage = (message: ClaudeMessage): boolean => {
+  // Snapshot blocks have dedicated renderer and no standard "content" payload.
+  if (message.type === "file-history-snapshot") {
+    return false;
+  }
+
   // Messages with tool use or results should be shown
   if (
     (message.type === "assistant" && message.toolUse) ||