From 1cc3c8c4edcd8562944f965b248839cdf5f0defb Mon Sep 17 00:00:00 2001
From: Yang Liu <yang.l@envato.com>
Date: Sun, 31 May 2026 13:40:04 +1200
Subject: [PATCH] test(compat): verify codex-tui.log opt-in change in v0.134.0
 does not affect JSONL session parser
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Codex v0.134.0 PR #24081 made `codex-tui.log` opt-in. codex-trace reads
session data exclusively from JSONL files at ~/.codex/sessions/ — it does
not read the TUI log. The opt-in change therefore has no effect on session
parsing or session discovery.

Add a regression test in entry.rs that documents this boundary and verifies
all four standard JSONL entry types (session_meta, event_msg, response_item,
turn_context) continue to parse correctly for v0.134.0 sessions regardless
of whether the TUI log is present on disk.

Fixes #80
---
 src-tauri/src/parser/entry.rs | 41 +++++++++++++++++++++++++++++++++++
 1 file changed, 41 insertions(+)

diff --git a/src-tauri/src/parser/entry.rs b/src-tauri/src/parser/entry.rs
index 5d29441..a56e668 100644
--- a/src-tauri/src/parser/entry.rs
+++ b/src-tauri/src/parser/entry.rs
@@ -429,4 +429,45 @@ mod tests {
         assert_eq!(meta.payload["cli_version"], "0.131.0");
         assert_eq!(meta.payload["profile"], "default");
     }
+
+    // Codex v0.134.0 (PR #24081): `codex-tui.log` is now opt-in.
+    //
+    // Before v0.134.0, the TUI log file was written unconditionally at its default
+    // path. PR #24081 made this opt-in: the file no longer exists unless the user
+    // explicitly enables TUI logging.
+    //
+    // codex-trace reads session data exclusively from JSONL files at
+    // ~/.codex/sessions/ — it does not read `codex-tui.log`. The opt-in change
+    // therefore has no effect on session parsing or session discovery. Verify that
+    // all four standard JSONL entry types continue to parse correctly for v0.134.0
+    // sessions regardless of whether the TUI log is present on disk.
+
+    #[test]
+    fn v0134_tui_log_opt_in_does_not_affect_jsonl_session_parser() {
+        // Codex v0.134.0 PR #24081 made `codex-tui.log` opt-in. codex-trace reads
+        // session data from JSONL files at ~/.codex/sessions/, not from the TUI log,
+        // so the opt-in change has no effect on parsing. Verify all four standard
+        // entry types produced by a v0.134.0 session parse correctly.
+        let lines = [
+            r#"{"timestamp":"2026-05-26T10:00:00Z","type":"session_meta","payload":{"id":"v0134-session","timestamp":"2026-05-26T10:00:00Z","cwd":"/tmp","cli_version":"0.134.0","model_provider":"openai"}}"#,
+            r#"{"timestamp":"2026-05-26T10:00:01Z","type":"event_msg","payload":{"type":"task_started","turn_id":"turn-1"}}"#,
+            r#"{"timestamp":"2026-05-26T10:00:02Z","type":"response_item","payload":{"type":"message","role":"assistant","content":"Hello"}}"#,
+            r#"{"timestamp":"2026-05-26T10:00:03Z","type":"turn_context","payload":{"model":"gpt-5","cwd":"/tmp"}}"#,
+            r#"{"timestamp":"2026-05-26T10:00:04Z","type":"event_msg","payload":{"type":"task_complete","turn_id":"turn-1","completed_at":1748253604.0}}"#,
+        ];
+        let expected_types = [
+            "session_meta",
+            "event_msg",
+            "response_item",
+            "turn_context",
+            "event_msg",
+        ];
+        for (line, expected) in lines.iter().zip(expected_types.iter()) {
+            let entry = RawEntry::parse(line).expect("parse failed");
+            assert_eq!(entry.entry_type, *expected, "wrong type for: {line}");
+        }
+        let meta = RawEntry::parse(lines[0]).unwrap();
+        assert_eq!(meta.payload["cli_version"], "0.134.0");
+        assert_eq!(meta.payload["id"], "v0134-session");
+    }
 }