elliot-stack 1.0.29 → 1.0.33

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

@@ -1,78 +1,78 @@
-"""Tests for lib.search."""
-
-from datetime import datetime
-from pathlib import Path
-
-from lib import search as S
-
-
-def test_search_session_text(fixtures_dir):
-    matches = S.search_session(
-        fixtures_dir / "basic-session.jsonl", "Hello", role="both", in_channel="text"
-    )
-    assert len(matches) >= 1
-
-
-def test_search_session_assistant_only(fixtures_dir):
-    matches = S.search_session(
-        fixtures_dir / "basic-session.jsonl", "Hello",
-        role="assistant", in_channel="text",
-    )
-    # "Hello" is in the user message only
-    assert len(matches) == 0
-
-
-def test_search_session_user_only(fixtures_dir):
-    matches = S.search_session(
-        fixtures_dir / "basic-session.jsonl", "Hello",
-        role="user", in_channel="text",
-    )
-    assert len(matches) >= 1
-
-
-def test_search_in_tool_use(fixtures_dir):
-    matches = S.search_session(
-        fixtures_dir / "tool-zoo.jsonl", "ls -la",
-        role="both", in_channel="tool_use",
-    )
-    assert len(matches) >= 1
-    assert matches[0].where == "tool_use"
-
-
-def test_search_in_thinking(fixtures_dir):
-    matches = S.search_session(
-        fixtures_dir / "with-thinking.jsonl", "step by step",
-        role="both", in_channel="thinking",
-    )
-    assert len(matches) >= 1
-    assert matches[0].where == "thinking"
-
-
-def test_search_no_match(fixtures_dir):
-    matches = S.search_session(
-        fixtures_dir / "basic-session.jsonl", "this-string-not-present",
-        role="both", in_channel="text",
-    )
-    assert matches == []
-
-
-def test_search_with_time_filter(fixtures_dir):
-    # "Hello" appears in basic-session.jsonl at 2026-05-01T10:00:00Z
-    # Excluding that date should yield no matches
-    matches = S.search_session(
-        fixtures_dir / "basic-session.jsonl", "Hello",
-        role="both", in_channel="text",
-        since=datetime(2026, 6, 1),
-    )
-    assert matches == []
-
-
-def test_search_project(fixtures_dir, tmp_path):
-    # Copy 2 fixtures into a fake project dir and search
-    import shutil
-    pd = tmp_path / "fake-proj"
-    pd.mkdir()
-    shutil.copy(fixtures_dir / "basic-session.jsonl", pd / "session-a.jsonl")
-    shutil.copy(fixtures_dir / "tool-zoo.jsonl", pd / "session-b.jsonl")
-    matches = list(S.search_project(pd, "Hello", progress=False))
-    assert len(matches) >= 1
+"""Tests for lib.search."""
+
+from datetime import datetime
+from pathlib import Path
+
+from lib import search as S
+
+
+def test_search_session_text(fixtures_dir):
+    matches = S.search_session(
+        fixtures_dir / "basic-session.jsonl", "Hello", role="both", in_channel="text"
+    )
+    assert len(matches) >= 1
+
+
+def test_search_session_assistant_only(fixtures_dir):
+    matches = S.search_session(
+        fixtures_dir / "basic-session.jsonl", "Hello",
+        role="assistant", in_channel="text",
+    )
+    # "Hello" is in the user message only
+    assert len(matches) == 0
+
+
+def test_search_session_user_only(fixtures_dir):
+    matches = S.search_session(
+        fixtures_dir / "basic-session.jsonl", "Hello",
+        role="user", in_channel="text",
+    )
+    assert len(matches) >= 1
+
+
+def test_search_in_tool_use(fixtures_dir):
+    matches = S.search_session(
+        fixtures_dir / "tool-zoo.jsonl", "ls -la",
+        role="both", in_channel="tool_use",
+    )
+    assert len(matches) >= 1
+    assert matches[0].where == "tool_use"
+
+
+def test_search_in_thinking(fixtures_dir):
+    matches = S.search_session(
+        fixtures_dir / "with-thinking.jsonl", "step by step",
+        role="both", in_channel="thinking",
+    )
+    assert len(matches) >= 1
+    assert matches[0].where == "thinking"
+
+
+def test_search_no_match(fixtures_dir):
+    matches = S.search_session(
+        fixtures_dir / "basic-session.jsonl", "this-string-not-present",
+        role="both", in_channel="text",
+    )
+    assert matches == []
+
+
+def test_search_with_time_filter(fixtures_dir):
+    # "Hello" appears in basic-session.jsonl at 2026-05-01T10:00:00Z
+    # Excluding that date should yield no matches
+    matches = S.search_session(
+        fixtures_dir / "basic-session.jsonl", "Hello",
+        role="both", in_channel="text",
+        since=datetime(2026, 6, 1),
+    )
+    assert matches == []
+
+
+def test_search_project(fixtures_dir, tmp_path):
+    # Copy 2 fixtures into a fake project dir and search
+    import shutil
+    pd = tmp_path / "fake-proj"
+    pd.mkdir()
+    shutil.copy(fixtures_dir / "basic-session.jsonl", pd / "session-a.jsonl")
+    shutil.copy(fixtures_dir / "tool-zoo.jsonl", pd / "session-b.jsonl")
+    matches = list(S.search_project(pd, "Hello", progress=False))
+    assert len(matches) >= 1

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

@@ -1,43 +1,43 @@
-"""Tests for lib.subagents."""
-
-from pathlib import Path
-
-from lib import subagents as SA
-from lib import paths as P
-
-
-def test_load_meta_hit(fixtures_dir):
-    agent_file = fixtures_dir / "subagent-parent" / "subagents" / "agent-xyz123.jsonl"
-    meta = SA.load_meta(agent_file)
-    assert meta["agentType"] == "Explore"
-    assert "bug" in meta["description"].lower()
-
-
-def test_load_meta_miss(fixtures_dir):
-    agent_file = fixtures_dir / "subagent-no-meta" / "subagents" / "agent-aaa.jsonl"
-    meta = SA.load_meta(agent_file)
-    assert meta["agentType"] == "unknown"
-    assert meta["description"] == ""
-
-
-def test_agent_finals(fixtures_dir):
-    parent = fixtures_dir / "subagent-parent.jsonl"
-    finals = SA.agent_finals(parent)
-    assert len(finals) == 1
-    agent_id, meta, text = finals[0]
-    assert agent_id == "agent-xyz123"
-    assert meta["agentType"] == "Explore"
-    assert "Found it" in text
-
-
-def test_agent_finals_no_subagents(fixtures_dir):
-    parent = fixtures_dir / "basic-session.jsonl"
-    finals = SA.agent_finals(parent)
-    assert finals == []
-
-
-def test_list_subagents(fixtures_dir):
-    parent = fixtures_dir / "subagent-parent.jsonl"
-    subs = P.list_subagents(parent)
-    assert len(subs) == 1
-    assert subs[0].stem == "agent-xyz123"
+"""Tests for lib.subagents."""
+
+from pathlib import Path
+
+from lib import subagents as SA
+from lib import paths as P
+
+
+def test_load_meta_hit(fixtures_dir):
+    agent_file = fixtures_dir / "subagent-parent" / "subagents" / "agent-xyz123.jsonl"
+    meta = SA.load_meta(agent_file)
+    assert meta["agentType"] == "Explore"
+    assert "bug" in meta["description"].lower()
+
+
+def test_load_meta_miss(fixtures_dir):
+    agent_file = fixtures_dir / "subagent-no-meta" / "subagents" / "agent-aaa.jsonl"
+    meta = SA.load_meta(agent_file)
+    assert meta["agentType"] == "unknown"
+    assert meta["description"] == ""
+
+
+def test_agent_finals(fixtures_dir):
+    parent = fixtures_dir / "subagent-parent.jsonl"
+    finals = SA.agent_finals(parent)
+    assert len(finals) == 1
+    agent_id, meta, text = finals[0]
+    assert agent_id == "agent-xyz123"
+    assert meta["agentType"] == "Explore"
+    assert "Found it" in text
+
+
+def test_agent_finals_no_subagents(fixtures_dir):
+    parent = fixtures_dir / "basic-session.jsonl"
+    finals = SA.agent_finals(parent)
+    assert finals == []
+
+
+def test_list_subagents(fixtures_dir):
+    parent = fixtures_dir / "subagent-parent.jsonl"
+    subs = P.list_subagents(parent)
+    assert len(subs) == 1
+    assert subs[0].stem == "agent-xyz123"

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

@@ -1,179 +1,179 @@
-"""Tests for the timeline mode (build/render/gap parsing + CLI)."""
-
-import json
-import os
-import shutil
-import subprocess
-import sys
-from datetime import datetime, timedelta
-from pathlib import Path
-
-import pytest
-
-import read_transcript as RT
-
-
-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,
-    )
-
-
-@pytest.fixture
-def fake_root(fixtures_dir, tmp_path):
-    root = tmp_path / "projects"
-    proj = root / "C--fake-proj"
-    proj.mkdir(parents=True)
-    shutil.copy(fixtures_dir / "timeline-day-test.jsonl", proj / "abc12345.jsonl")
-    return root
-
-
-# ── unit: gap + duration helpers ─────────────────────────────────────────────
-
-def test_parse_gap_default():
-    assert RT._parse_gap(None) == 15
-
-
-def test_parse_gap_minutes():
-    assert RT._parse_gap("20m") == 20
-    assert RT._parse_gap("20") == 20
-
-
-def test_parse_gap_hours():
-    assert RT._parse_gap("1h") == 60
-
-
-def test_parse_gap_invalid():
-    with pytest.raises(ValueError):
-        RT._parse_gap("soon")
-
-
-def test_fmt_dur():
-    assert RT._fmt_dur(timedelta(minutes=8)) == "8m"
-    assert RT._fmt_dur(timedelta(minutes=72)) == "1h12m"
-    assert RT._fmt_dur(timedelta(seconds=30)) == "<1m"
-
-
-# ── unit: block grouping ─────────────────────────────────────────────────────
-
-def test_build_timeline_blocks(fake_root):
-    from lib import parser as PR
-    PR.set_timezone("UTC")
-    try:
-        data = RT.build_timeline(
-            [fake_root / "C--fake-proj"],
-            since=datetime(2026, 5, 1),
-            until=datetime(2026, 5, 2),
-            gap_minutes=15,
-            current_uuid=None,
-        )
-    finally:
-        PR.set_timezone(None)
-    blocks = data["blocks"]
-    assert len(blocks) == 2
-    assert blocks[0]["start"] == datetime(2026, 5, 1, 10, 0)
-    assert blocks[0]["end"] == datetime(2026, 5, 1, 10, 8)
-    assert blocks[1]["start"] == datetime(2026, 5, 1, 12, 0)
-    assert blocks[1]["end"] == datetime(2026, 5, 1, 12, 2)
-
-
-def test_build_timeline_wide_gap_merges(fake_root):
-    from lib import parser as PR
-    PR.set_timezone("UTC")
-    try:
-        data = RT.build_timeline(
-            [fake_root / "C--fake-proj"],
-            since=datetime(2026, 5, 1),
-            until=datetime(2026, 5, 2),
-            gap_minutes=180,  # 3h gap threshold swallows the 1h52m idle
-            current_uuid=None,
-        )
-    finally:
-        PR.set_timezone(None)
-    assert len(data["blocks"]) == 1
-
-
-# ── CLI ──────────────────────────────────────────────────────────────────────
-
-def test_timeline_cli_text(cli_path, fake_root):
-    r = _run_cli(
-        cli_path, "--root", str(fake_root), "--tz", "UTC",
-        "--mode", "timeline", "--date", "2026-05-01",
-    )
-    assert r.returncode == 0
-    assert "10:00" in r.stdout
-    assert "12:02" in r.stdout
-    assert "idle" in r.stdout
-    assert "2 block(s)" in r.stdout
-    # Timeline makes no attention claim — that's engagement mode's job.
-    assert "active" not in r.stdout
-
-
-def test_timeline_cli_json(cli_path, fake_root):
-    r = _run_cli(
-        cli_path, "--root", str(fake_root), "--tz", "UTC",
-        "--mode", "timeline", "--date", "2026-05-01", "--format", "json",
-    )
-    assert r.returncode == 0
-    data = json.loads(r.stdout)
-    assert data["totals"]["blocks"] == 2
-    assert data["totals"]["sessions"] == 1
-    assert data["totals"]["span_minutes"] == 122  # 10:00 → 12:02
-    assert "active_minutes" not in data["totals"]
-    assert data["blocks"][0]["start"].endswith("10:00:00")
-    assert data["blocks"][0]["sessions"][0]["uuid"] == "abc12345"
-
-
-def test_timeline_cli_empty_range(cli_path, fake_root):
-    r = _run_cli(
-        cli_path, "--root", str(fake_root), "--tz", "UTC",
-        "--mode", "timeline", "--date", "2020-01-01",
-    )
-    assert r.returncode == 0
-    assert "no activity" in r.stdout
-
-
-def test_timeline_cli_project_filter(cli_path, fake_root):
-    r = _run_cli(
-        cli_path, "--root", str(fake_root), "--tz", "UTC",
-        "--mode", "timeline", "--date", "2026-05-01", "--project", "fake",
-    )
-    assert r.returncode == 0
-    assert "10:00" in r.stdout
-
-
-def test_timeline_cli_exclude_current(cli_path, fake_root):
-    r = _run_cli(
-        cli_path, "--root", str(fake_root), "--tz", "UTC",
-        "--mode", "timeline", "--date", "2026-05-01", "--exclude-current",
-        env_overrides={"CLAUDE_SESSION_ID": "abc12345"},
-    )
-    assert r.returncode == 0
-    assert "no activity" in r.stdout
-
-
-def test_journal_cli_exclude_current(cli_path, fake_root):
-    r = _run_cli(
-        cli_path, "--root", str(fake_root),
-        "--mode", "journal", "--since", "2020-01-01", "--all-projects",
-        "--exclude-current",
-        env_overrides={"CLAUDE_SESSION_ID": "abc12345"},
-    )
-    assert r.returncode == 0
-    assert "abc12345" not in r.stdout
-
-
-def test_timeline_cli_project_no_match(cli_path, fake_root):
-    r = _run_cli(
-        cli_path, "--root", str(fake_root), "--tz", "UTC",
-        "--mode", "timeline", "--date", "2026-05-01", "--project", "zzz-nope",
-    )
-    assert r.returncode == 1
+"""Tests for the timeline mode (build/render/gap parsing + CLI)."""
+
+import json
+import os
+import shutil
+import subprocess
+import sys
+from datetime import datetime, timedelta
+from pathlib import Path
+
+import pytest
+
+import read_transcript as RT
+
+
+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,
+    )
+
+
+@pytest.fixture
+def fake_root(fixtures_dir, tmp_path):
+    root = tmp_path / "projects"
+    proj = root / "C--fake-proj"
+    proj.mkdir(parents=True)
+    shutil.copy(fixtures_dir / "timeline-day-test.jsonl", proj / "abc12345.jsonl")
+    return root
+
+
+# ── unit: gap + duration helpers ─────────────────────────────────────────────
+
+def test_parse_gap_default():
+    assert RT._parse_gap(None) == 15
+
+
+def test_parse_gap_minutes():
+    assert RT._parse_gap("20m") == 20
+    assert RT._parse_gap("20") == 20
+
+
+def test_parse_gap_hours():
+    assert RT._parse_gap("1h") == 60
+
+
+def test_parse_gap_invalid():
+    with pytest.raises(ValueError):
+        RT._parse_gap("soon")
+
+
+def test_fmt_dur():
+    assert RT._fmt_dur(timedelta(minutes=8)) == "8m"
+    assert RT._fmt_dur(timedelta(minutes=72)) == "1h12m"
+    assert RT._fmt_dur(timedelta(seconds=30)) == "<1m"
+
+
+# ── unit: block grouping ─────────────────────────────────────────────────────
+
+def test_build_timeline_blocks(fake_root):
+    from lib import parser as PR
+    PR.set_timezone("UTC")
+    try:
+        data = RT.build_timeline(
+            [fake_root / "C--fake-proj"],
+            since=datetime(2026, 5, 1),
+            until=datetime(2026, 5, 2),
+            gap_minutes=15,
+            current_uuid=None,
+        )
+    finally:
+        PR.set_timezone(None)
+    blocks = data["blocks"]
+    assert len(blocks) == 2
+    assert blocks[0]["start"] == datetime(2026, 5, 1, 10, 0)
+    assert blocks[0]["end"] == datetime(2026, 5, 1, 10, 8)
+    assert blocks[1]["start"] == datetime(2026, 5, 1, 12, 0)
+    assert blocks[1]["end"] == datetime(2026, 5, 1, 12, 2)
+
+
+def test_build_timeline_wide_gap_merges(fake_root):
+    from lib import parser as PR
+    PR.set_timezone("UTC")
+    try:
+        data = RT.build_timeline(
+            [fake_root / "C--fake-proj"],
+            since=datetime(2026, 5, 1),
+            until=datetime(2026, 5, 2),
+            gap_minutes=180,  # 3h gap threshold swallows the 1h52m idle
+            current_uuid=None,
+        )
+    finally:
+        PR.set_timezone(None)
+    assert len(data["blocks"]) == 1
+
+
+# ── CLI ──────────────────────────────────────────────────────────────────────
+
+def test_timeline_cli_text(cli_path, fake_root):
+    r = _run_cli(
+        cli_path, "--root", str(fake_root), "--tz", "UTC",
+        "--mode", "timeline", "--date", "2026-05-01",
+    )
+    assert r.returncode == 0
+    assert "10:00" in r.stdout
+    assert "12:02" in r.stdout
+    assert "idle" in r.stdout
+    assert "2 block(s)" in r.stdout
+    # Timeline makes no attention claim — that's engagement mode's job.
+    assert "active" not in r.stdout
+
+
+def test_timeline_cli_json(cli_path, fake_root):
+    r = _run_cli(
+        cli_path, "--root", str(fake_root), "--tz", "UTC",
+        "--mode", "timeline", "--date", "2026-05-01", "--format", "json",
+    )
+    assert r.returncode == 0
+    data = json.loads(r.stdout)
+    assert data["totals"]["blocks"] == 2
+    assert data["totals"]["sessions"] == 1
+    assert data["totals"]["span_minutes"] == 122  # 10:00 → 12:02
+    assert "active_minutes" not in data["totals"]
+    assert data["blocks"][0]["start"].endswith("10:00:00")
+    assert data["blocks"][0]["sessions"][0]["uuid"] == "abc12345"
+
+
+def test_timeline_cli_empty_range(cli_path, fake_root):
+    r = _run_cli(
+        cli_path, "--root", str(fake_root), "--tz", "UTC",
+        "--mode", "timeline", "--date", "2020-01-01",
+    )
+    assert r.returncode == 0
+    assert "no activity" in r.stdout
+
+
+def test_timeline_cli_project_filter(cli_path, fake_root):
+    r = _run_cli(
+        cli_path, "--root", str(fake_root), "--tz", "UTC",
+        "--mode", "timeline", "--date", "2026-05-01", "--project", "fake",
+    )
+    assert r.returncode == 0
+    assert "10:00" in r.stdout
+
+
+def test_timeline_cli_exclude_current(cli_path, fake_root):
+    r = _run_cli(
+        cli_path, "--root", str(fake_root), "--tz", "UTC",
+        "--mode", "timeline", "--date", "2026-05-01", "--exclude-current",
+        env_overrides={"CLAUDE_SESSION_ID": "abc12345"},
+    )
+    assert r.returncode == 0
+    assert "no activity" in r.stdout
+
+
+def test_journal_cli_exclude_current(cli_path, fake_root):
+    r = _run_cli(
+        cli_path, "--root", str(fake_root),
+        "--mode", "journal", "--since", "2020-01-01", "--all-projects",
+        "--exclude-current",
+        env_overrides={"CLAUDE_SESSION_ID": "abc12345"},
+    )
+    assert r.returncode == 0
+    assert "abc12345" not in r.stdout
+
+
+def test_timeline_cli_project_no_match(cli_path, fake_root):
+    r = _run_cli(
+        cli_path, "--root", str(fake_root), "--tz", "UTC",
+        "--mode", "timeline", "--date", "2026-05-01", "--project", "zzz-nope",
+    )
+    assert r.returncode == 1