elliot-stack 1.0.29 → 1.0.33

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

@@ -1,212 +1,212 @@
-"""Tests for timezone handling (set_timezone / --tz) and the --project filter."""
-
-import json
-import os
-import shutil
-import subprocess
-import sys
-from datetime import datetime
-from pathlib import Path
-
-import pytest
-
-from lib import parser as PR
-from lib import paths as P
-
-
-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(autouse=True)
-def _reset_tz():
-    yield
-    PR.set_timezone(None)
-
-
-# ── set_timezone / _parse_timestamp ─────────────────────────────────────────
-
-def test_parse_timestamp_default_is_naive():
-    dt = PR._parse_timestamp("2026-05-01T10:00:00Z")
-    assert dt is not None
-    assert dt.tzinfo is None
-
-
-def test_parse_timestamp_utc():
-    PR.set_timezone("UTC")
-    assert PR._parse_timestamp("2026-05-01T10:00:00Z") == datetime(2026, 5, 1, 10, 0)
-
-
-def test_parse_timestamp_offset():
-    PR.set_timezone("+2")
-    assert PR._parse_timestamp("2026-05-01T10:00:00Z") == datetime(2026, 5, 1, 12, 0)
-
-
-def test_parse_timestamp_utc_minus():
-    PR.set_timezone("UTC-4")
-    assert PR._parse_timestamp("2026-05-01T10:00:00Z") == datetime(2026, 5, 1, 6, 0)
-
-
-def test_set_timezone_half_hour_offset():
-    PR.set_timezone("+05:30")
-    assert PR._parse_timestamp("2026-05-01T10:00:00Z") == datetime(2026, 5, 1, 15, 30)
-
-
-def test_set_timezone_local_resets():
-    PR.set_timezone("UTC")
-    PR.set_timezone("local")
-    assert PR._TARGET_TZ is None
-
-
-def test_set_timezone_iana():
-    try:
-        PR.set_timezone("America/New_York")
-    except ValueError:
-        pytest.skip("IANA tz database not available on this machine")
-    dt = PR._parse_timestamp("2026-01-15T10:00:00Z")  # EST = UTC-5
-    assert dt == datetime(2026, 1, 15, 5, 0)
-
-
-def test_set_timezone_invalid_raises():
-    with pytest.raises(ValueError):
-        PR.set_timezone("Mars/Olympus_Mons")
-
-
-def test_epoch_to_display_utc():
-    PR.set_timezone("UTC")
-    # 2026-05-01T10:00:00Z as epoch
-    epoch = datetime(2026, 5, 1, 10, 0).replace(
-        tzinfo=__import__("datetime").timezone.utc
-    ).timestamp()
-    assert PR.epoch_to_display(epoch) == datetime(2026, 5, 1, 10, 0)
-
-
-def test_cli_tz_invalid(cli_path, fixtures_dir):
-    r = _run_cli(
-        cli_path, "--file", str(fixtures_dir / "basic-session.jsonl"),
-        "--mode", "changelog", "--tz", "Nope/Nowhere",
-    )
-    assert r.returncode == 1
-    assert "timezone" in r.stderr.lower()
-
-
-def test_display_to_epoch_roundtrip_offset_tz():
-    PR.set_timezone("+2")
-    epoch = 1772546400.0
-    assert PR.display_to_epoch(PR.epoch_to_display(epoch)) == epoch
-
-
-def test_display_to_epoch_roundtrip_local():
-    epoch = 1772546400.0
-    assert PR.display_to_epoch(PR.epoch_to_display(epoch)) == epoch
-
-
-def test_now_display_matches_utc_clock():
-    from datetime import timezone as _tzmod
-    PR.set_timezone("UTC")
-    delta = abs((PR.now_display() - datetime.now(_tzmod.utc).replace(tzinfo=None)).total_seconds())
-    assert delta < 5
-
-
-def test_parse_timespec_now_respects_tz():
-    from datetime import timezone as _tzmod
-    PR.set_timezone("UTC")
-    now_spec = P.parse_timespec("now")
-    delta = abs((now_spec - datetime.now(_tzmod.utc).replace(tzinfo=None)).total_seconds())
-    assert delta < 5
-
-
-def test_json_timestamps_are_display_tz(cli_path, fixtures_dir):
-    # basic-session events are 10:00:0xZ — with +2 the JSON timestamps read 12:00
-    r = _run_cli(
-        cli_path, "--file", str(fixtures_dir / "basic-session.jsonl"),
-        "--mode", "last", "--format", "json", "--tz", "+2",
-    )
-    assert r.returncode == 0
-    data = json.loads(r.stdout)
-    assert data[-1]["timestamp"].startswith("2026-05-01T12:00:05")
-
-
-def test_cli_changelog_respects_tz(cli_path, fixtures_dir):
-    # tool-zoo events are 10:00:0xZ — with +3 they display as 13:00
-    r = _run_cli(
-        cli_path, "--file", str(fixtures_dir / "tool-zoo.jsonl"),
-        "--mode", "changelog", "--tz", "+3",
-    )
-    assert r.returncode == 0
-    assert "13:00:01" in r.stdout
-
-
-# ── --project filter ─────────────────────────────────────────────────────────
-
-@pytest.fixture
-def multi_root(fixtures_dir, tmp_path):
-    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 / "tool-zoo.jsonl", other / "othersession.jsonl")
-    return root
-
-
-def test_filter_projects_substring(multi_root):
-    dirs = P.filter_projects(multi_root, "keel")
-    assert len(dirs) == 1
-    assert "Keel" in dirs[0].name
-
-
-def test_filter_projects_spaces_match_hyphens(multi_root):
-    dirs = P.filter_projects(multi_root, "Keel Project")
-    assert len(dirs) == 1
-
-
-def test_filter_projects_no_match(multi_root):
-    assert P.filter_projects(multi_root, "zzz") == []
-
-
-def test_cli_list_project_filter(cli_path, multi_root):
-    r = _run_cli(
-        cli_path, "--root", str(multi_root), "--mode", "list", "--project", "keel",
-    )
-    assert r.returncode == 0
-    assert "keelsess" in r.stdout
-    assert "othersess" not in r.stdout
-
-
-def test_cli_search_project_filter(cli_path, multi_root):
-    r = _run_cli(
-        cli_path, "--root", str(multi_root), "--mode", "search",
-        "--query", "help", "--project", "keel",
-    )
-    assert r.returncode == 0
-    assert "keelsession" in r.stdout
-
-
-def test_cli_journal_project_filter_json(cli_path, multi_root):
-    r = _run_cli(
-        cli_path, "--root", str(multi_root), "--mode", "journal",
-        "--since", "2020-01-01", "--project", "other claude", "--format", "json",
-    )
-    assert r.returncode == 0
-    data = json.loads(r.stdout)
-    assert len(data) == 1
-    assert data[0]["uuid"] == "othersession"
-
-
-def test_cli_project_no_match_exits_1(cli_path, multi_root):
-    r = _run_cli(
-        cli_path, "--root", str(multi_root), "--mode", "list", "--project", "zzz",
-    )
-    assert r.returncode == 1
+"""Tests for timezone handling (set_timezone / --tz) and the --project filter."""
+
+import json
+import os
+import shutil
+import subprocess
+import sys
+from datetime import datetime
+from pathlib import Path
+
+import pytest
+
+from lib import parser as PR
+from lib import paths as P
+
+
+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(autouse=True)
+def _reset_tz():
+    yield
+    PR.set_timezone(None)
+
+
+# ── set_timezone / _parse_timestamp ─────────────────────────────────────────
+
+def test_parse_timestamp_default_is_naive():
+    dt = PR._parse_timestamp("2026-05-01T10:00:00Z")
+    assert dt is not None
+    assert dt.tzinfo is None
+
+
+def test_parse_timestamp_utc():
+    PR.set_timezone("UTC")
+    assert PR._parse_timestamp("2026-05-01T10:00:00Z") == datetime(2026, 5, 1, 10, 0)
+
+
+def test_parse_timestamp_offset():
+    PR.set_timezone("+2")
+    assert PR._parse_timestamp("2026-05-01T10:00:00Z") == datetime(2026, 5, 1, 12, 0)
+
+
+def test_parse_timestamp_utc_minus():
+    PR.set_timezone("UTC-4")
+    assert PR._parse_timestamp("2026-05-01T10:00:00Z") == datetime(2026, 5, 1, 6, 0)
+
+
+def test_set_timezone_half_hour_offset():
+    PR.set_timezone("+05:30")
+    assert PR._parse_timestamp("2026-05-01T10:00:00Z") == datetime(2026, 5, 1, 15, 30)
+
+
+def test_set_timezone_local_resets():
+    PR.set_timezone("UTC")
+    PR.set_timezone("local")
+    assert PR._TARGET_TZ is None
+
+
+def test_set_timezone_iana():
+    try:
+        PR.set_timezone("America/New_York")
+    except ValueError:
+        pytest.skip("IANA tz database not available on this machine")
+    dt = PR._parse_timestamp("2026-01-15T10:00:00Z")  # EST = UTC-5
+    assert dt == datetime(2026, 1, 15, 5, 0)
+
+
+def test_set_timezone_invalid_raises():
+    with pytest.raises(ValueError):
+        PR.set_timezone("Mars/Olympus_Mons")
+
+
+def test_epoch_to_display_utc():
+    PR.set_timezone("UTC")
+    # 2026-05-01T10:00:00Z as epoch
+    epoch = datetime(2026, 5, 1, 10, 0).replace(
+        tzinfo=__import__("datetime").timezone.utc
+    ).timestamp()
+    assert PR.epoch_to_display(epoch) == datetime(2026, 5, 1, 10, 0)
+
+
+def test_cli_tz_invalid(cli_path, fixtures_dir):
+    r = _run_cli(
+        cli_path, "--file", str(fixtures_dir / "basic-session.jsonl"),
+        "--mode", "changelog", "--tz", "Nope/Nowhere",
+    )
+    assert r.returncode == 1
+    assert "timezone" in r.stderr.lower()
+
+
+def test_display_to_epoch_roundtrip_offset_tz():
+    PR.set_timezone("+2")
+    epoch = 1772546400.0
+    assert PR.display_to_epoch(PR.epoch_to_display(epoch)) == epoch
+
+
+def test_display_to_epoch_roundtrip_local():
+    epoch = 1772546400.0
+    assert PR.display_to_epoch(PR.epoch_to_display(epoch)) == epoch
+
+
+def test_now_display_matches_utc_clock():
+    from datetime import timezone as _tzmod
+    PR.set_timezone("UTC")
+    delta = abs((PR.now_display() - datetime.now(_tzmod.utc).replace(tzinfo=None)).total_seconds())
+    assert delta < 5
+
+
+def test_parse_timespec_now_respects_tz():
+    from datetime import timezone as _tzmod
+    PR.set_timezone("UTC")
+    now_spec = P.parse_timespec("now")
+    delta = abs((now_spec - datetime.now(_tzmod.utc).replace(tzinfo=None)).total_seconds())
+    assert delta < 5
+
+
+def test_json_timestamps_are_display_tz(cli_path, fixtures_dir):
+    # basic-session events are 10:00:0xZ — with +2 the JSON timestamps read 12:00
+    r = _run_cli(
+        cli_path, "--file", str(fixtures_dir / "basic-session.jsonl"),
+        "--mode", "last", "--format", "json", "--tz", "+2",
+    )
+    assert r.returncode == 0
+    data = json.loads(r.stdout)
+    assert data[-1]["timestamp"].startswith("2026-05-01T12:00:05")
+
+
+def test_cli_changelog_respects_tz(cli_path, fixtures_dir):
+    # tool-zoo events are 10:00:0xZ — with +3 they display as 13:00
+    r = _run_cli(
+        cli_path, "--file", str(fixtures_dir / "tool-zoo.jsonl"),
+        "--mode", "changelog", "--tz", "+3",
+    )
+    assert r.returncode == 0
+    assert "13:00:01" in r.stdout
+
+
+# ── --project filter ─────────────────────────────────────────────────────────
+
+@pytest.fixture
+def multi_root(fixtures_dir, tmp_path):
+    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 / "tool-zoo.jsonl", other / "othersession.jsonl")
+    return root
+
+
+def test_filter_projects_substring(multi_root):
+    dirs = P.filter_projects(multi_root, "keel")
+    assert len(dirs) == 1
+    assert "Keel" in dirs[0].name
+
+
+def test_filter_projects_spaces_match_hyphens(multi_root):
+    dirs = P.filter_projects(multi_root, "Keel Project")
+    assert len(dirs) == 1
+
+
+def test_filter_projects_no_match(multi_root):
+    assert P.filter_projects(multi_root, "zzz") == []
+
+
+def test_cli_list_project_filter(cli_path, multi_root):
+    r = _run_cli(
+        cli_path, "--root", str(multi_root), "--mode", "list", "--project", "keel",
+    )
+    assert r.returncode == 0
+    assert "keelsess" in r.stdout
+    assert "othersess" not in r.stdout
+
+
+def test_cli_search_project_filter(cli_path, multi_root):
+    r = _run_cli(
+        cli_path, "--root", str(multi_root), "--mode", "search",
+        "--query", "help", "--project", "keel",
+    )
+    assert r.returncode == 0
+    assert "keelsession" in r.stdout
+
+
+def test_cli_journal_project_filter_json(cli_path, multi_root):
+    r = _run_cli(
+        cli_path, "--root", str(multi_root), "--mode", "journal",
+        "--since", "2020-01-01", "--project", "other claude", "--format", "json",
+    )
+    assert r.returncode == 0
+    data = json.loads(r.stdout)
+    assert len(data) == 1
+    assert data[0]["uuid"] == "othersession"
+
+
+def test_cli_project_no_match_exits_1(cli_path, multi_root):
+    r = _run_cli(
+        cli_path, "--root", str(multi_root), "--mode", "list", "--project", "zzz",
+    )
+    assert r.returncode == 1

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

@@ -1,80 +1,80 @@
-"""Tests for lib.tools."""
-
-from pathlib import Path
-
-from lib import parser as PR
-from lib import tools as T
-
-
-def _load(fixtures_dir, name):
-    return PR.parse_lines(fixtures_dir / name)
-
-
-def test_extract_tool_calls_count(fixtures_dir):
-    lines = _load(fixtures_dir, "tool-zoo.jsonl")
-    calls = T.extract_tool_calls(lines)
-    assert len(calls) == 9
-
-
-def test_format_bash():
-    call = {"name": "Bash", "input": {"command": "ls -la /tmp"}}
-    out = T.format_tool_call(call)
-    assert out == "Bash: ls -la /tmp"
-
-
-def test_format_powershell():
-    call = {"name": "PowerShell", "input": {"command": "Get-ChildItem"}}
-    assert "PowerShell" in T.format_tool_call(call)
-
-
-def test_format_read_with_lines():
-    call = {"name": "Read", "input": {"file_path": "C:\\foo.py", "offset": 1, "limit": 50}}
-    out = T.format_tool_call(call)
-    assert "C:\\foo.py" in out
-    assert "lines 1" in out
-
-
-def test_format_edit_with_edits():
-    call = {"name": "Edit", "input": {"file_path": "C:\\foo.py", "edits": [{}, {}, {}]}}
-    out = T.format_tool_call(call)
-    assert "3 edits" in out
-
-
-def test_format_write():
-    call = {"name": "Write", "input": {"file_path": "C:\\bar.py", "content": "print('hi')"}}
-    out = T.format_tool_call(call)
-    assert "chars" in out
-
-
-def test_format_agent():
-    call = {"name": "Agent", "input": {"subagent_type": "Explore", "description": "Find X"}}
-    out = T.format_tool_call(call)
-    assert "Agent[Explore]" in out
-    assert "Find X" in out
-
-
-def test_format_skill():
-    call = {"name": "Skill", "input": {"skill": "using-superpowers"}}
-    out = T.format_tool_call(call)
-    assert "using-superpowers" in out
-
-
-def test_tool_filter(fixtures_dir):
-    lines = _load(fixtures_dir, "tool-zoo.jsonl")
-    calls = T.extract_tool_calls(lines, tool_filter={"Bash"})
-    assert len(calls) == 1
-    assert calls[0]["name"] == "Bash"
-
-
-def test_files_touched(fixtures_dir):
-    lines = _load(fixtures_dir, "tool-zoo.jsonl")
-    files = T.files_touched(lines)
-    assert any("foo.py" in str(p) for p in files)
-    assert any("bar.py" in str(p) for p in files)
-
-
-def test_extract_tool_results(fixtures_dir):
-    lines = _load(fixtures_dir, "subagent-parent.jsonl")
-    result = T.extract_tool_results(lines, "toolu_01abc")
-    assert result is not None
-    assert "Found it" in result
+"""Tests for lib.tools."""
+
+from pathlib import Path
+
+from lib import parser as PR
+from lib import tools as T
+
+
+def _load(fixtures_dir, name):
+    return PR.parse_lines(fixtures_dir / name)
+
+
+def test_extract_tool_calls_count(fixtures_dir):
+    lines = _load(fixtures_dir, "tool-zoo.jsonl")
+    calls = T.extract_tool_calls(lines)
+    assert len(calls) == 9
+
+
+def test_format_bash():
+    call = {"name": "Bash", "input": {"command": "ls -la /tmp"}}
+    out = T.format_tool_call(call)
+    assert out == "Bash: ls -la /tmp"
+
+
+def test_format_powershell():
+    call = {"name": "PowerShell", "input": {"command": "Get-ChildItem"}}
+    assert "PowerShell" in T.format_tool_call(call)
+
+
+def test_format_read_with_lines():
+    call = {"name": "Read", "input": {"file_path": "C:\\foo.py", "offset": 1, "limit": 50}}
+    out = T.format_tool_call(call)
+    assert "C:\\foo.py" in out
+    assert "lines 1" in out
+
+
+def test_format_edit_with_edits():
+    call = {"name": "Edit", "input": {"file_path": "C:\\foo.py", "edits": [{}, {}, {}]}}
+    out = T.format_tool_call(call)
+    assert "3 edits" in out
+
+
+def test_format_write():
+    call = {"name": "Write", "input": {"file_path": "C:\\bar.py", "content": "print('hi')"}}
+    out = T.format_tool_call(call)
+    assert "chars" in out
+
+
+def test_format_agent():
+    call = {"name": "Agent", "input": {"subagent_type": "Explore", "description": "Find X"}}
+    out = T.format_tool_call(call)
+    assert "Agent[Explore]" in out
+    assert "Find X" in out
+
+
+def test_format_skill():
+    call = {"name": "Skill", "input": {"skill": "using-superpowers"}}
+    out = T.format_tool_call(call)
+    assert "using-superpowers" in out
+
+
+def test_tool_filter(fixtures_dir):
+    lines = _load(fixtures_dir, "tool-zoo.jsonl")
+    calls = T.extract_tool_calls(lines, tool_filter={"Bash"})
+    assert len(calls) == 1
+    assert calls[0]["name"] == "Bash"
+
+
+def test_files_touched(fixtures_dir):
+    lines = _load(fixtures_dir, "tool-zoo.jsonl")
+    files = T.files_touched(lines)
+    assert any("foo.py" in str(p) for p in files)
+    assert any("bar.py" in str(p) for p in files)
+
+
+def test_extract_tool_results(fixtures_dir):
+    lines = _load(fixtures_dir, "subagent-parent.jsonl")
+    result = T.extract_tool_results(lines, "toolu_01abc")
+    assert result is not None
+    assert "Found it" in result