elliot-stack 1.0.39 → 1.0.41

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

@@ -1,94 +0,0 @@
-"""Tests for lib.search."""
-
-from datetime import datetime, timedelta
-from pathlib import Path
-
-from lib import parser as PR
-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_until_is_exclusive(fixtures_dir):
-    # The "Hello" user message is stamped 2026-05-01T10:00:00Z. Derive the bound
-    # from the parser (it converts to local naive time) so this is tz-independent.
-    # Half-open [since, until): until at that instant excludes it; one second later includes it.
-    at = PR._parse_timestamp("2026-05-01T10:00:00Z")
-    assert S.search_session(
-        fixtures_dir / "basic-session.jsonl", "Hello", role="both", until=at
-    ) == []
-    # "Hello" matches only the user message at the boundary instant, so +1s yields exactly 1.
-    assert len(S.search_session(
-        fixtures_dir / "basic-session.jsonl", "Hello", role="both",
-        until=at + timedelta(seconds=1)
-    )) == 1
-
-
-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_search_output.py

@@ -1,161 +0,0 @@
-"""Tests for cross-scope search output budgeting: summary-by-default, --full,
-the char-budget degrade, and TTY-gated progress."""
-
-import json
-import os
-import shutil
-import subprocess
-import sys
-from pathlib import Path
-
-import pytest
-
-import read_transcript as RT
-from lib.search import Match
-
-
-def _run_cli(cli_path, *args):
-    env = dict(os.environ)
-    env["PYTHONIOENCODING"] = "utf-8"
-    return subprocess.run(
-        [sys.executable, str(cli_path), *args],
-        capture_output=True, text=True, encoding="utf-8", env=env,
-    )
-
-
-# A realistic epoch base so _fmt_mtime renders sane dates if output is printed.
-_BASE_MTIME = 1_700_000_000
-
-
-def _mk(stem, project, mtime, window, role="assistant", where="text"):
-    sp = Path("/root") / project / f"{stem}.jsonl"
-    return Match(session_path=sp, mtime=mtime, role=role, where=where,
-                 timestamp=None, window_text=window)
-
-
-@pytest.fixture
-def wide_root(fixtures_dir, tmp_path):
-    """A fake projects root with two projects, each holding a session."""
-    root = tmp_path / "projects"
-    keel = root / "C--Users-x-Keel-Project"
-    other = root / "C--Users-x-Other-Claude-Code"
-    keel.mkdir(parents=True)
-    other.mkdir(parents=True)
-    shutil.copy(fixtures_dir / "basic-session.jsonl", keel / "keelsession.jsonl")
-    shutil.copy(fixtures_dir / "basic-session.jsonl", other / "othersession.jsonl")
-    return root
-
-
-# ── unit: renderers ──────────────────────────────────────────────────────────
-
-def test_one_line_collapses_and_truncates():
-    assert RT._one_line("a\n  b   c", 100) == "a b c"
-    assert RT._one_line("x" * 200, 10) == "x" * 10 + "…"
-    assert RT._one_line("", 10) == ""
-
-
-def test_summary_counts_all_hits_and_orders_newest_first():
-    matches = [
-        _mk("aaaaaaaa11", "C--Users-x-Proj", _BASE_MTIME, "hello world"),
-        _mk("aaaaaaaa11", "C--Users-x-Proj", _BASE_MTIME, "hello again"),
-        _mk("bbbbbbbb22", "C--Users-x-Proj", _BASE_MTIME + 100, "hello there"),
-    ]
-    out = RT._render_search_summary("hello", matches)
-    assert '"hello": 3 matches across 2 sessions' in out
-    assert "2 hits" in out  # the aaaa session has 2
-    # bbbb (BASE+100, newer) sorts before aaaa (BASE, older) — newest first
-    assert out.index("bbbbbbbb") < out.index("aaaaaaaa")
-    assert "--- Match #" not in out  # no full windows in summary
-
-
-def test_summary_caps_sessions_but_counts_all(monkeypatch):
-    monkeypatch.setattr(RT, "SEARCH_SUMMARY_SESSION_CAP", 2)
-    matches = [_mk(f"sess{i:04d}xx", "C--Users-x-Proj", _BASE_MTIME + i, "hit") for i in range(5)]
-    out = RT._render_search_summary("hit", matches)
-    assert "5 matches across 5 sessions" in out  # header counts everything
-    assert "3 more session" in out  # 5 total - 2 shown
-
-
-def test_search_summary_json_has_no_windows():
-    matches = [_mk("aaaaaaaa11", "C--Users-x-My-Proj", _BASE_MTIME, "hello world snippet")]
-    js = RT._search_summary_json(matches)
-    assert js[0]["uuid"] == "aaaaaaaa11"
-    assert js[0]["hits"] == 1
-    assert "window" not in js[0]
-    assert js[0]["first_snippet"] == "hello world snippet"
-
-
-def test_single_file_full_degrades_to_summary_when_oversized(monkeypatch, fixtures_dir):
-    # Even a single-file search degrades if its full windows exceed the budget.
-    monkeypatch.setattr(RT, "SEARCH_CHAR_BUDGET", 10)
-    out = RT.mode_search_v2(
-        root=Path("."), cwd=None, all_projects=False,
-        file_path=fixtures_dir / "basic-session.jsonl",
-        query="Hello", role="both", in_channel="text",
-        since=None, until=None, fmt="text",
-    )
-    assert out.startswith("[note: full output")
-    assert '"Hello":' in out  # summary header is appended after the note
-
-
-def test_wide_full_degrades_to_summary_when_oversized(monkeypatch, wide_root):
-    # The advertised path: wide scope + --full, oversized, degrades to summary.
-    monkeypatch.setattr(RT, "SEARCH_CHAR_BUDGET", 10)
-    out = RT.mode_search_v2(
-        root=wide_root, cwd=None, all_projects=True, file_path=None,
-        query="Hello", role="both", in_channel="text",
-        since=None, until=None, fmt="text", full=True,
-    )
-    assert out.startswith("[note: full output")
-    assert "matches across" in out  # summary header appended
-    assert "--- Match #" not in out  # degraded — no windows survive
-
-
-# ── CLI: end-to-end ──────────────────────────────────────────────────────────
-
-def test_wide_search_summarizes_by_default(cli_path, wide_root):
-    r = _run_cli(cli_path, "--root", str(wide_root), "--mode", "search",
-                 "--query", "Hello", "--all-projects")
-    assert r.returncode == 0
-    assert '"Hello":' in r.stdout and "matches across" in r.stdout  # real summary header
-    assert "--- Match #" not in r.stdout  # summary, not windows
-
-
-def test_wide_search_full_shows_windows(cli_path, wide_root):
-    r = _run_cli(cli_path, "--root", str(wide_root), "--mode", "search",
-                 "--query", "Hello", "--all-projects", "--full")
-    assert r.returncode == 0
-    assert "--- Match #" in r.stdout  # full windows present
-
-
-def test_progress_suppressed_when_stderr_not_a_tty(cli_path, wide_root):
-    # stderr is captured (not a TTY) — the Searching i/N progress must not appear.
-    r = _run_cli(cli_path, "--root", str(wide_root), "--mode", "search",
-                 "--query", "Hello", "--all-projects")
-    assert "Searching" not in r.stderr
-
-
-def test_cwd_search_matches_user_messages(cli_path, wide_root):
-    # "Hello" lives only in the user message of basic-session.jsonl. A --cwd
-    # search now routes through mode_search_v2 (role=both), so it must be found —
-    # the old assistant-only --cwd path would have missed it.
-    found = _run_cli(cli_path, "--root", str(wide_root), "--mode", "search",
-                     "--query", "Hello", "--cwd", "Keel")
-    assert found.returncode == 0
-    assert '"Hello":' in found.stdout and "across" in found.stdout
-    assert "keelsess" in found.stdout
-
-    # Restricting to assistant role excludes the user-only hit.
-    asst = _run_cli(cli_path, "--root", str(wide_root), "--mode", "search",
-                    "--query", "Hello", "--cwd", "Keel", "--role", "assistant")
-    assert asst.returncode == 0
-    assert "No matches" in asst.stdout
-
-
-def test_json_role_filter_excludes_user_only_match(cli_path, wide_root):
-    # JSON path honors --role too: assistant-only over a user-only term → empty list.
-    r = _run_cli(cli_path, "--root", str(wide_root), "--mode", "search",
-                 "--query", "Hello", "--cwd", "Keel",
-                 "--role", "assistant", "--format", "json")
-    assert r.returncode == 0
-    assert json.loads(r.stdout) == []

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

@@ -1,43 +0,0 @@
-"""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 +0,0 @@
-"""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