elliot-stack 1.0.18 → 1.0.19

package/skills/estack-read-claude-session-history/scripts/tests/conftest.py

@@ -0,0 +1,40 @@
+"""Shared pytest fixtures and import-path setup."""
+
+from __future__ import annotations
+
+import sys
+from pathlib import Path
+
+import pytest
+
+
+# Ensure UTF-8 output regardless of console code page (Windows)
+if sys.stdout.encoding and sys.stdout.encoding.lower() != "utf-8":
+    try:
+        sys.stdout.reconfigure(encoding="utf-8", errors="replace")
+    except (AttributeError, OSError):
+        pass
+
+
+THIS_DIR = Path(__file__).resolve().parent
+SCRIPTS_DIR = THIS_DIR.parent
+FIXTURES_DIR = THIS_DIR / "fixtures"
+
+# Make `from lib...` work in tests
+if str(SCRIPTS_DIR) not in sys.path:
+    sys.path.insert(0, str(SCRIPTS_DIR))
+
+
+@pytest.fixture
+def fixtures_dir() -> Path:
+    return FIXTURES_DIR
+
+
+@pytest.fixture
+def scripts_dir() -> Path:
+    return SCRIPTS_DIR
+
+
+@pytest.fixture
+def cli_path() -> Path:
+    return SCRIPTS_DIR / "read_transcript.py"

package/skills/estack-read-claude-session-history/scripts/tests/fixtures/README.md

@@ -0,0 +1,20 @@
+# Test fixtures
+
+Hand-crafted minimal JSONL files, one scenario per file. Keep them small (≤50 lines) — they're easier to reason about than real session snapshots and don't carry PII.
+
+| File | Purpose |
+|---|---|
+| `basic-session.jsonl` | One user + one assistant exchange. Sanity check for the parser. |
+| `with-compact.jsonl` | Conversation interrupted by a single `/compact` marker. |
+| `multi-compact.jsonl` | Two `/compact` markers — exercises "most recent" logic. |
+| `with-advisor.jsonl` | Contains an `advisor_tool_result` block. |
+| `with-thinking.jsonl` | Contains a `thinking` block plus normal text. |
+| `all-noise.jsonl` | Only `ai-title` + `attachment` entries — should look empty to signal queries. |
+| `subagent-parent.jsonl` | Parent session that spawns one subagent via the `Agent` tool. |
+| `subagent-no-meta.jsonl` | Parent session with a sibling subagent file but no `.meta.json` sidecar — `load_meta` must fall back. |
+| `tool-zoo.jsonl` | One call to each of Bash, Read, Edit, Write, Agent, Skill, Glob, Grep. |
+| `time-spread.jsonl` | Six messages over a known time range — exercises `--since`/`--until`. |
+| `truncated.jsonl` | Final line is missing its newline AND is malformed JSON — should be dropped silently. |
+| `unicode.jsonl` | Contains emoji + CJK characters — exercises UTF-8 decoding. |
+| `pending-user.jsonl` | Last assistant message ends with `?` — `infer_status` should return `pending-user`. |
+| `interrupted.jsonl` | Final assistant message has a `tool_use` block with no matching `tool_result` — status `interrupted`. |

package/skills/estack-read-claude-session-history/scripts/tests/fixtures/all-noise.jsonl

@@ -0,0 +1,4 @@
+{"type":"ai-title","aiTitle":"Test session"}
+{"type":"attachment","data":{"path":"foo.png"}}
+{"type":"permission-mode","mode":"plan"}
+{"type":"file-history-snapshot","content":{"file":"bar.py"}}

package/skills/estack-read-claude-session-history/scripts/tests/fixtures/basic-session.jsonl

@@ -0,0 +1,2 @@
+{"type":"user","timestamp":"2026-05-01T10:00:00Z","message":{"role":"user","content":"Hello Claude"}}
+{"type":"assistant","timestamp":"2026-05-01T10:00:05Z","message":{"role":"assistant","content":[{"type":"text","text":"Hi there. Here is help."}]}}

package/skills/estack-read-claude-session-history/scripts/tests/fixtures/interrupted.jsonl

@@ -0,0 +1,2 @@
+{"type":"user","timestamp":"2026-05-01T10:00:00Z","message":{"role":"user","content":"Run a tool"}}
+{"type":"assistant","timestamp":"2026-05-01T10:00:05Z","message":{"role":"assistant","content":[{"type":"text","text":"Running."},{"type":"tool_use","id":"toolu_999","name":"Bash","input":{"command":"sleep 999"}}]}}

package/skills/estack-read-claude-session-history/scripts/tests/fixtures/multi-compact.jsonl

@@ -0,0 +1,8 @@
+{"type":"user","timestamp":"2026-05-01T10:00:00Z","message":{"role":"user","content":"First"}}
+{"type":"assistant","timestamp":"2026-05-01T10:00:05Z","message":{"role":"assistant","content":[{"type":"text","text":"R1"}]}}
+{"type":"user","timestamp":"2026-05-01T10:30:00Z","message":{"role":"user","content":"This session is being continued from a previous conversation. Round 1"}}
+{"type":"user","timestamp":"2026-05-01T11:00:00Z","message":{"role":"user","content":"Mid"}}
+{"type":"assistant","timestamp":"2026-05-01T11:00:05Z","message":{"role":"assistant","content":[{"type":"text","text":"R2"}]}}
+{"type":"user","timestamp":"2026-05-01T11:30:00Z","message":{"role":"user","content":"This session is being continued from a previous conversation. Round 2"}}
+{"type":"user","timestamp":"2026-05-01T12:00:00Z","message":{"role":"user","content":"Last"}}
+{"type":"assistant","timestamp":"2026-05-01T12:00:05Z","message":{"role":"assistant","content":[{"type":"text","text":"R3"}]}}

package/skills/estack-read-claude-session-history/scripts/tests/fixtures/pending-user.jsonl

@@ -0,0 +1,2 @@
+{"type":"user","timestamp":"2026-05-01T10:00:00Z","message":{"role":"user","content":"Hello"}}
+{"type":"assistant","timestamp":"2026-05-01T10:00:05Z","message":{"role":"assistant","content":[{"type":"text","text":"Sure, do you want me to A, B, or C?"}]}}

package/skills/estack-read-claude-session-history/scripts/tests/fixtures/subagent-no-meta/subagents/agent-aaa.jsonl

@@ -0,0 +1,2 @@
+{"type":"user","timestamp":"2026-05-01T10:00:10Z","message":{"role":"user","content":"Do thing"}}
+{"type":"assistant","timestamp":"2026-05-01T10:00:20Z","message":{"role":"assistant","content":[{"type":"text","text":"Done."}]}}

package/skills/estack-read-claude-session-history/scripts/tests/fixtures/subagent-no-meta.jsonl

@@ -0,0 +1,2 @@
+{"type":"user","timestamp":"2026-05-01T10:00:00Z","message":{"role":"user","content":"Run an agent"}}
+{"type":"assistant","timestamp":"2026-05-01T10:00:05Z","message":{"role":"assistant","content":[{"type":"text","text":"Agent ran."}]}}

package/skills/estack-read-claude-session-history/scripts/tests/fixtures/subagent-parent/subagents/agent-xyz123.jsonl

@@ -0,0 +1,2 @@
+{"type":"user","timestamp":"2026-05-01T10:00:10Z","message":{"role":"user","content":"Find the bug in foo.py"}}
+{"type":"assistant","timestamp":"2026-05-01T10:00:20Z","message":{"role":"assistant","content":[{"type":"text","text":"Found it: foo.py line 42 — off-by-one in loop bound."}]}}

package/skills/estack-read-claude-session-history/scripts/tests/fixtures/subagent-parent/subagents/agent-xyz123.meta.json

@@ -0,0 +1 @@
+{"agentType": "Explore", "description": "Find the bug in foo.py"}

package/skills/estack-read-claude-session-history/scripts/tests/fixtures/subagent-parent.jsonl

@@ -0,0 +1,4 @@
+{"type":"user","timestamp":"2026-05-01T10:00:00Z","message":{"role":"user","content":"Investigate the bug"}}
+{"type":"assistant","timestamp":"2026-05-01T10:00:05Z","message":{"role":"assistant","content":[{"type":"tool_use","id":"toolu_01abc","name":"Agent","input":{"description":"Find the bug","prompt":"Look at file X","subagent_type":"Explore"}}]}}
+{"type":"user","timestamp":"2026-05-01T10:00:30Z","message":{"role":"user","content":[{"type":"tool_result","tool_use_id":"toolu_01abc","content":"Found it in foo.py:42"}]}}
+{"type":"assistant","timestamp":"2026-05-01T10:00:35Z","message":{"role":"assistant","content":[{"type":"text","text":"Got the report, patching foo.py."}]}}

package/skills/estack-read-claude-session-history/scripts/tests/fixtures/time-spread.jsonl

@@ -0,0 +1,6 @@
+{"type":"user","timestamp":"2026-04-15T08:00:00Z","message":{"role":"user","content":"Apr 15 morning"}}
+{"type":"assistant","timestamp":"2026-04-15T08:00:05Z","message":{"role":"assistant","content":[{"type":"text","text":"Apr 15 reply"}]}}
+{"type":"user","timestamp":"2026-05-01T12:00:00Z","message":{"role":"user","content":"May 1 noon"}}
+{"type":"assistant","timestamp":"2026-05-01T12:00:05Z","message":{"role":"assistant","content":[{"type":"text","text":"May 1 reply"}]}}
+{"type":"user","timestamp":"2026-05-15T20:00:00Z","message":{"role":"user","content":"May 15 evening"}}
+{"type":"assistant","timestamp":"2026-05-15T20:00:05Z","message":{"role":"assistant","content":[{"type":"text","text":"May 15 reply"}]}}

package/skills/estack-read-claude-session-history/scripts/tests/fixtures/timeline-day-test.jsonl

@@ -0,0 +1,5 @@
+{"type":"user","timestamp":"2026-05-01T10:00:00Z","message":{"role":"user","content":"Start the CRM cleanup"}}
+{"type":"assistant","timestamp":"2026-05-01T10:05:00Z","message":{"role":"assistant","content":[{"type":"text","text":"Working on it."}]}}
+{"type":"assistant","timestamp":"2026-05-01T10:08:00Z","message":{"role":"assistant","content":[{"type":"text","text":"First pass done."}]}}
+{"type":"user","timestamp":"2026-05-01T12:00:00Z","message":{"role":"user","content":"Back — continue"}}
+{"type":"assistant","timestamp":"2026-05-01T12:02:00Z","message":{"role":"assistant","content":[{"type":"text","text":"Resumed and finished."}]}}

package/skills/estack-read-claude-session-history/scripts/tests/fixtures/tool-zoo.jsonl

@@ -0,0 +1,10 @@
+{"type":"user","timestamp":"2026-05-01T10:00:00Z","message":{"role":"user","content":"Run all the tools"}}
+{"type":"assistant","timestamp":"2026-05-01T10:00:01Z","message":{"role":"assistant","content":[{"type":"tool_use","id":"t1","name":"Bash","input":{"command":"ls -la /tmp"}}]}}
+{"type":"assistant","timestamp":"2026-05-01T10:00:02Z","message":{"role":"assistant","content":[{"type":"tool_use","id":"t2","name":"PowerShell","input":{"command":"Get-ChildItem"}}]}}
+{"type":"assistant","timestamp":"2026-05-01T10:00:03Z","message":{"role":"assistant","content":[{"type":"tool_use","id":"t3","name":"Read","input":{"file_path":"C:\\foo.py","offset":1,"limit":50}}]}}
+{"type":"assistant","timestamp":"2026-05-01T10:00:04Z","message":{"role":"assistant","content":[{"type":"tool_use","id":"t4","name":"Edit","input":{"file_path":"C:\\foo.py","old_string":"a","new_string":"b"}}]}}
+{"type":"assistant","timestamp":"2026-05-01T10:00:05Z","message":{"role":"assistant","content":[{"type":"tool_use","id":"t5","name":"Write","input":{"file_path":"C:\\bar.py","content":"print('hi')"}}]}}
+{"type":"assistant","timestamp":"2026-05-01T10:00:06Z","message":{"role":"assistant","content":[{"type":"tool_use","id":"t6","name":"Agent","input":{"description":"Find X","subagent_type":"Explore"}}]}}
+{"type":"assistant","timestamp":"2026-05-01T10:00:07Z","message":{"role":"assistant","content":[{"type":"tool_use","id":"t7","name":"Skill","input":{"skill":"using-superpowers"}}]}}
+{"type":"assistant","timestamp":"2026-05-01T10:00:08Z","message":{"role":"assistant","content":[{"type":"tool_use","id":"t8","name":"Glob","input":{"pattern":"**/*.py"}}]}}
+{"type":"assistant","timestamp":"2026-05-01T10:00:09Z","message":{"role":"assistant","content":[{"type":"tool_use","id":"t9","name":"Grep","input":{"pattern":"TODO"}}]}}

package/skills/estack-read-claude-session-history/scripts/tests/fixtures/truncated.jsonl

@@ -0,0 +1,3 @@
+{"type":"user","timestamp":"2026-05-01T10:00:00Z","message":{"role":"user","content":"Valid line"}}
+{"type":"assistant","timestamp":"2026-05-01T10:00:05Z","message":{"role":"assistant","content":[{"type":"text","text":"Valid reply"}]}}
+{"type":"user","timestamp":"2026-05-01T10:01:00Z","message":{"role":"user","content":"Trunc

package/skills/estack-read-claude-session-history/scripts/tests/fixtures/unicode.jsonl

@@ -0,0 +1,2 @@
+{"type":"user","timestamp":"2026-05-01T10:00:00Z","message":{"role":"user","content":"Hello 🌍, 你好"}}
+{"type":"assistant","timestamp":"2026-05-01T10:00:05Z","message":{"role":"assistant","content":[{"type":"text","text":"Привет — 안녕하세요!"}]}}

package/skills/estack-read-claude-session-history/scripts/tests/fixtures/with-advisor.jsonl

@@ -0,0 +1,3 @@
+{"type":"user","timestamp":"2026-05-01T10:00:00Z","message":{"role":"user","content":"Get advice"}}
+{"type":"assistant","timestamp":"2026-05-01T10:00:05Z","message":{"role":"assistant","content":[{"type":"text","text":"Calling advisor..."},{"type":"advisor_tool_result","content":{"text":"The advisor says do X then Y."}}]}}
+{"type":"assistant","timestamp":"2026-05-01T10:00:10Z","message":{"role":"assistant","content":[{"type":"text","text":"OK, doing X."}]}}

package/skills/estack-read-claude-session-history/scripts/tests/fixtures/with-compact.jsonl

@@ -0,0 +1,5 @@
+{"type":"user","timestamp":"2026-05-01T10:00:00Z","message":{"role":"user","content":"First question"}}
+{"type":"assistant","timestamp":"2026-05-01T10:00:05Z","message":{"role":"assistant","content":[{"type":"text","text":"First answer"}]}}
+{"type":"user","timestamp":"2026-05-01T11:00:00Z","message":{"role":"user","content":"This session is being continued from a previous conversation. Summary follows..."}}
+{"type":"user","timestamp":"2026-05-01T11:00:10Z","message":{"role":"user","content":"After compact"}}
+{"type":"assistant","timestamp":"2026-05-01T11:00:15Z","message":{"role":"assistant","content":[{"type":"text","text":"Post-compact reply"}]}}

package/skills/estack-read-claude-session-history/scripts/tests/fixtures/with-thinking.jsonl

@@ -0,0 +1,2 @@
+{"type":"user","timestamp":"2026-05-01T10:00:00Z","message":{"role":"user","content":"Solve this"}}
+{"type":"assistant","timestamp":"2026-05-01T10:00:05Z","message":{"role":"assistant","content":[{"type":"thinking","thinking":"Let me think step by step..."},{"type":"text","text":"The answer is 42."}]}}

package/skills/estack-read-claude-session-history/scripts/tests/test_backup_roots.py

@@ -0,0 +1,56 @@
+"""Tests for backup-root resolution (--root mirror|snapshot-*|<abs-path>)."""
+
+from pathlib import Path
+
+import pytest
+
+from lib import paths as P
+
+
+def test_root_live_default():
+    assert P.resolve_root(None) == P.DEFAULT_LIVE_PROJECTS
+    assert P.resolve_root("live") == P.DEFAULT_LIVE_PROJECTS
+
+
+def test_root_mirror_path_shape():
+    r = P.resolve_root("mirror")
+    parts = r.parts
+    assert ".claude-backups" in parts
+    assert "mirror" in parts
+    assert parts[-1] == "projects"
+
+
+def test_root_snapshot_24h_path_shape():
+    r = P.resolve_root("snapshot-24h")
+    assert "snapshot-24h" in r.parts
+    assert r.name == "projects"
+
+
+def test_root_all_known():
+    for name in ("mirror", "snapshot-24h", "snapshot-1w", "snapshot-1mo"):
+        r = P.resolve_root(name)
+        assert name in r.parts
+
+
+def test_root_absolute_path(tmp_path: Path):
+    fake = tmp_path / "weird-root"
+    fake.mkdir()
+    assert P.resolve_root(str(fake)) == fake
+
+
+def test_root_unknown_relative_raises():
+    with pytest.raises(ValueError):
+        P.resolve_root("bogus")
+
+
+def test_find_project_dir_uses_root(tmp_path: Path):
+    # Build a fake root with a fake project dir
+    proj = tmp_path / "C--Users-foo-bar"
+    proj.mkdir()
+    found = P.find_project_dir("C:\\Users\\foo\\bar", root=tmp_path)
+    assert found == proj
+
+
+def test_find_project_dir_not_found(tmp_path: Path):
+    with pytest.raises(FileNotFoundError):
+        P.find_project_dir("C:\\does\\not\\exist", root=tmp_path)

package/skills/estack-read-claude-session-history/scripts/tests/test_json_format.py

@@ -0,0 +1,201 @@
+"""Tests for --format json across modes."""
+
+import json
+import os
+import shutil
+import subprocess
+import sys
+from pathlib import Path
+
+import pytest
+
+
+def _run_cli(cli_path, *args, env_overrides=None):
+    env = dict(os.environ)
+    env["PYTHONIOENCODING"] = "utf-8"
+    if env_overrides:
+        env.update(env_overrides)
+    return subprocess.run(
+        [sys.executable, str(cli_path), *args],
+        capture_output=True,
+        text=True,
+        encoding="utf-8",
+        env=env,
+    )
+
+
+def _json(r):
+    assert r.returncode == 0, r.stderr
+    return json.loads(r.stdout)
+
+
+def test_last_json(cli_path, fixtures_dir):
+    data = _json(_run_cli(
+        cli_path, "--file", str(fixtures_dir / "basic-session.jsonl"),
+        "--mode", "last", "--format", "json",
+    ))
+    assert isinstance(data, list)
+    assert data[-1]["n_from_end"] == 1
+    assert "Here is help" in data[-1]["text"]
+
+
+def test_json_alias_flag(cli_path, fixtures_dir):
+    data = _json(_run_cli(
+        cli_path, "--file", str(fixtures_dir / "basic-session.jsonl"),
+        "--mode", "last", "--json",
+    ))
+    assert isinstance(data, list)
+
+
+def test_advisor_json(cli_path, fixtures_dir):
+    data = _json(_run_cli(
+        cli_path, "--file", str(fixtures_dir / "with-advisor.jsonl"),
+        "--mode", "advisor", "--format", "json",
+    ))
+    assert data == ["The advisor says do X then Y."]
+
+
+def test_pre_compact_json(cli_path, fixtures_dir):
+    data = _json(_run_cli(
+        cli_path, "--file", str(fixtures_dir / "with-compact.jsonl"),
+        "--mode", "pre-compact", "--format", "json",
+    ))
+    assert data["found_compact"] is True
+    assert any("First answer" in m["text"] for m in data["messages"])
+
+
+def test_dump_json(cli_path, fixtures_dir):
+    data = _json(_run_cli(
+        cli_path, "--file", str(fixtures_dir / "basic-session.jsonl"),
+        "--mode", "dump", "--format", "json",
+    ))
+    assert isinstance(data, list)
+    assert data[0]["role"] == "user"
+
+
+def test_debug_json(cli_path, fixtures_dir):
+    data = _json(_run_cli(
+        cli_path, "--file", str(fixtures_dir / "with-advisor.jsonl"),
+        "--mode", "debug", "--format", "json",
+    ))
+    assert data["advisor_blocks"] == 1
+    assert "entry_types" in data
+
+
+def test_brief_json(cli_path, fixtures_dir):
+    data = _json(_run_cli(
+        cli_path, "--file", str(fixtures_dir / "tool-zoo.jsonl"),
+        "--mode", "brief", "--format", "json",
+    ))
+    assert data["exists"] is True
+    assert data["tool_counts"]["Bash"] == 1
+    assert isinstance(data["files_touched"], list)
+
+
+def test_changelog_json(cli_path, fixtures_dir):
+    data = _json(_run_cli(
+        cli_path, "--file", str(fixtures_dir / "tool-zoo.jsonl"),
+        "--mode", "changelog", "--format", "json",
+    ))
+    assert len(data) == 9
+    assert data[0]["tool"] == "Bash"
+    assert "input" not in data[0]
+
+
+def test_tool_calls_json_includes_input(cli_path, fixtures_dir):
+    data = _json(_run_cli(
+        cli_path, "--file", str(fixtures_dir / "tool-zoo.jsonl"),
+        "--mode", "tool-calls", "--format", "json", "--tool", "Bash",
+    ))
+    assert len(data) == 1
+    assert data[0]["input"]["command"] == "ls -la /tmp"
+
+
+def test_file_edits_json(cli_path, fixtures_dir):
+    data = _json(_run_cli(
+        cli_path, "--file", str(fixtures_dir / "tool-zoo.jsonl"),
+        "--mode", "file-edits", "--format", "json",
+    ))
+    paths = [row["path"] for row in data]
+    assert any("foo.py" in p for p in paths)
+
+
+def test_subagent_finals_json(cli_path, fixtures_dir):
+    data = _json(_run_cli(
+        cli_path, "--file", str(fixtures_dir / "subagent-parent.jsonl"),
+        "--mode", "subagent-finals", "--format", "json",
+    ))
+    assert data[0]["agentType"] == "Explore"
+    assert "Found it" in data[0]["text"]
+
+
+def test_search_json_single_file(cli_path, fixtures_dir):
+    data = _json(_run_cli(
+        cli_path, "--file", str(fixtures_dir / "basic-session.jsonl"),
+        "--mode", "search", "--query", "help", "--format", "json",
+    ))
+    assert len(data) == 1
+    assert data[0]["role"] == "assistant"
+
+
+def test_list_json(cli_path, fixtures_dir, tmp_path):
+    fake_root = tmp_path / "projects"
+    fake_proj = fake_root / "C--fake-proj"
+    fake_proj.mkdir(parents=True)
+    shutil.copy(fixtures_dir / "basic-session.jsonl", fake_proj / "abc.jsonl")
+    data = _json(_run_cli(
+        cli_path, "--root", str(fake_root), "--all-projects",
+        "--mode", "list", "--format", "json",
+    ))
+    assert len(data) == 1
+    assert data[0]["uuid"] == "abc"
+
+
+def test_journal_json(cli_path, fixtures_dir, tmp_path):
+    fake_root = tmp_path / "projects"
+    fake_proj = fake_root / "C--fake-proj"
+    fake_proj.mkdir(parents=True)
+    shutil.copy(fixtures_dir / "tool-zoo.jsonl", fake_proj / "def.jsonl")
+    data = _json(_run_cli(
+        cli_path, "--root", str(fake_root), "--all-projects",
+        "--mode", "journal", "--since", "2020-01-01", "--format", "json",
+    ))
+    assert data[0]["uuid"] == "def"
+    assert data[0]["tool_counts"]["Bash"] == 1
+
+
+def test_count_json(cli_path, fixtures_dir, tmp_path):
+    fake_root = tmp_path / "projects"
+    fake_proj = fake_root / "C--fake-proj"
+    fake_proj.mkdir(parents=True)
+    shutil.copy(fixtures_dir / "basic-session.jsonl", fake_proj / "abc.jsonl")
+    data = _json(_run_cli(
+        cli_path, "--root", str(fake_root), "--all-projects",
+        "--mode", "count", "--query", "help", "--format", "json",
+    ))
+    assert data == {"sessions": 1, "messages": 2, "matches": 1}
+
+
+def test_count_text_unchanged(cli_path, fixtures_dir, tmp_path):
+    fake_root = tmp_path / "projects"
+    fake_proj = fake_root / "C--fake-proj"
+    fake_proj.mkdir(parents=True)
+    shutil.copy(fixtures_dir / "basic-session.jsonl", fake_proj / "abc.jsonl")
+    r = _run_cli(
+        cli_path, "--root", str(fake_root), "--all-projects",
+        "--mode", "count", "--query", "help",
+    )
+    assert r.returncode == 0
+    assert r.stdout.strip() == "1"
+    assert "1 sessions" in r.stderr
+
+
+def test_diff_json(cli_path, fixtures_dir):
+    data = _json(_run_cli(
+        cli_path, "--mode", "diff",
+        "--file-a", str(fixtures_dir / "basic-session.jsonl"),
+        "--file-b", str(fixtures_dir / "with-thinking.jsonl"),
+        "--format", "json",
+    ))
+    sources = {m["source"] for m in data["messages"]}
+    assert sources == {"A", "B"}

package/skills/estack-read-claude-session-history/scripts/tests/test_modes.py

@@ -0,0 +1,199 @@
+"""End-to-end CLI tests via subprocess.run.
+
+Exercises mode dispatch + argument parsing. Library-level behavior is
+covered by the unit tests in test_paths/parser/tools/search/subagents.
+"""
+
+import json
+import os
+import shutil
+import subprocess
+import sys
+from pathlib import Path
+
+import pytest
+
+
+def _run_cli(cli_path, *args, env_overrides=None):
+    env = dict(os.environ)
+    env["PYTHONIOENCODING"] = "utf-8"
+    if env_overrides:
+        env.update(env_overrides)
+    return subprocess.run(
+        [sys.executable, str(cli_path), *args],
+        capture_output=True,
+        text=True,
+        encoding="utf-8",
+        env=env,
+    )
+
+
+def test_help(cli_path):
+    r = _run_cli(cli_path, "--help")
+    assert r.returncode == 0
+    assert "--mode" in r.stdout
+
+
+def test_last_mode(cli_path, fixtures_dir):
+    r = _run_cli(cli_path, "--file", str(fixtures_dir / "basic-session.jsonl"), "--mode", "last")
+    assert r.returncode == 0
+    assert "Here is help" in r.stdout
+
+
+def test_advisor_mode(cli_path, fixtures_dir):
+    r = _run_cli(cli_path, "--file", str(fixtures_dir / "with-advisor.jsonl"), "--mode", "advisor")
+    assert r.returncode == 0
+    assert "advisor" in r.stdout.lower()
+
+
+def test_pre_compact_mode(cli_path, fixtures_dir):
+    r = _run_cli(cli_path, "--file", str(fixtures_dir / "with-compact.jsonl"), "--mode", "pre-compact")
+    assert r.returncode == 0
+    assert "Pre-compact" in r.stdout or "First answer" in r.stdout
+
+
+def test_debug_mode(cli_path, fixtures_dir):
+    r = _run_cli(cli_path, "--file", str(fixtures_dir / "basic-session.jsonl"), "--mode", "debug")
+    assert r.returncode == 0
+    assert "Entry type distribution" in r.stdout
+
+
+def test_brief_mode(cli_path, fixtures_dir):
+    r = _run_cli(cli_path, "--file", str(fixtures_dir / "tool-zoo.jsonl"), "--mode", "brief")
+    assert r.returncode == 0
+    body = r.stdout
+    # 6 lines expected
+    assert "intent:" in body
+    assert "last:" in body
+    assert "edits:" in body
+    assert "tools:" in body
+    assert "subagents:" in body
+
+
+def test_brief_with_include_subagents(cli_path, fixtures_dir):
+    r = _run_cli(
+        cli_path, "--file", str(fixtures_dir / "subagent-parent.jsonl"),
+        "--mode", "brief", "--include-subagents",
+    )
+    assert r.returncode == 0
+    assert "subagent" in r.stdout.lower()
+    assert "Found it" in r.stdout
+
+
+def test_changelog(cli_path, fixtures_dir):
+    r = _run_cli(cli_path, "--file", str(fixtures_dir / "tool-zoo.jsonl"), "--mode", "changelog")
+    assert r.returncode == 0
+    assert "Bash" in r.stdout
+    assert "Read" in r.stdout
+
+
+def test_file_edits(cli_path, fixtures_dir):
+    r = _run_cli(cli_path, "--file", str(fixtures_dir / "tool-zoo.jsonl"), "--mode", "file-edits")
+    assert r.returncode == 0
+    assert "foo.py" in r.stdout
+    assert "bar.py" in r.stdout
+
+
+def test_tool_calls(cli_path, fixtures_dir):
+    r = _run_cli(cli_path, "--file", str(fixtures_dir / "tool-zoo.jsonl"), "--mode", "tool-calls")
+    assert r.returncode == 0
+    assert "Bash" in r.stdout
+
+
+def test_tool_calls_filter(cli_path, fixtures_dir):
+    r = _run_cli(
+        cli_path, "--file", str(fixtures_dir / "tool-zoo.jsonl"),
+        "--mode", "tool-calls", "--tool", "Bash",
+    )
+    assert r.returncode == 0
+    assert "Bash" in r.stdout
+    assert "Glob" not in r.stdout
+
+
+def test_subagent_list(cli_path, fixtures_dir):
+    r = _run_cli(
+        cli_path, "--file", str(fixtures_dir / "subagent-parent.jsonl"),
+        "--mode", "subagent-list",
+    )
+    assert r.returncode == 0
+    assert "agent-xyz123" in r.stdout
+
+
+def test_subagent_finals(cli_path, fixtures_dir):
+    r = _run_cli(
+        cli_path, "--file", str(fixtures_dir / "subagent-parent.jsonl"),
+        "--mode", "subagent-finals",
+    )
+    assert r.returncode == 0
+    assert "Found it" in r.stdout
+
+
+def test_diff_mode(cli_path, fixtures_dir):
+    r = _run_cli(
+        cli_path, "--mode", "diff",
+        "--file-a", str(fixtures_dir / "basic-session.jsonl"),
+        "--file-b", str(fixtures_dir / "with-thinking.jsonl"),
+    )
+    assert r.returncode == 0
+    assert "A>" in r.stdout
+    assert "B>" in r.stdout
+
+
+def test_lookup_no_match(cli_path, fixtures_dir, tmp_path):
+    # Point --root at an empty dir so lookup definitely misses
+    r = _run_cli(cli_path, "--root", str(tmp_path), "--mode", "lookup", "--uuid", "nope")
+    assert r.returncode == 1
+
+
+def test_list_legacy_format(cli_path, fixtures_dir, tmp_path):
+    # Build a fake project root + cwd that matches encoding
+    fake_root = tmp_path / "projects"
+    fake_proj = fake_root / "C--fake-proj"
+    fake_proj.mkdir(parents=True)
+    shutil.copy(fixtures_dir / "basic-session.jsonl", fake_proj / "abc.jsonl")
+    r = _run_cli(cli_path, "--root", str(fake_root), "--cwd", "C:\\fake\\proj", "--list")
+    assert r.returncode == 0
+    assert "abc.jsonl" in r.stdout
+
+
+def test_exclude_current(cli_path, fixtures_dir, tmp_path):
+    fake_root = tmp_path / "projects"
+    fake_proj = fake_root / "C--fake-proj"
+    fake_proj.mkdir(parents=True)
+    shutil.copy(fixtures_dir / "basic-session.jsonl", fake_proj / "abc.jsonl")
+    shutil.copy(fixtures_dir / "tool-zoo.jsonl", fake_proj / "def.jsonl")
+    r = _run_cli(
+        cli_path, "--root", str(fake_root), "--cwd", "C:\\fake\\proj",
+        "--mode", "list", "--exclude-current",
+        env_overrides={"CLAUDE_SESSION_ID": "abc"},
+    )
+    assert r.returncode == 0
+    assert "def" in r.stdout
+    assert "abc" not in r.stdout
+
+
+def test_dump_large_file_degrades(cli_path, fixtures_dir, tmp_path):
+    # Build a 6MB padded fixture by writing many valid lines
+    big = tmp_path / "big.jsonl"
+    line = '{"type":"user","timestamp":"2026-05-01T10:00:00Z","message":{"role":"user","content":"'
+    pad = "x" * 1000 + '"}}\n'
+    with open(big, "w", encoding="utf-8") as f:
+        # Each line ~1KB → write ~7000 to hit 6MB+
+        for _ in range(7000):
+            f.write(line + pad)
+    r = _run_cli(cli_path, "--file", str(big), "--mode", "dump")
+    assert r.returncode == 0
+    assert "degraded" in r.stderr.lower()
+
+
+def test_dump_large_file_force(cli_path, tmp_path):
+    big = tmp_path / "big.jsonl"
+    line = '{"type":"user","timestamp":"2026-05-01T10:00:00Z","message":{"role":"user","content":"'
+    pad = "x" * 1000 + '"}}\n'
+    with open(big, "w", encoding="utf-8") as f:
+        for _ in range(7000):
+            f.write(line + pad)
+    r = _run_cli(cli_path, "--file", str(big), "--mode", "dump", "--force-dump")
+    assert r.returncode == 0
+    # No degrade note when forced
+    assert "degraded" not in r.stderr.lower()