claude-dev-env 1.93.1 → 1.95.0

package/skills/log-audit/scripts/test_cluster_recurrences.py

@@ -1,160 +0,0 @@
-"""Tests for cluster_recurrences — signature grouping and timing regressions."""
-
-import io
-import sys
-from datetime import datetime, timedelta
-from pathlib import Path
-
-import pytest
-
-_script_directory = str(Path(__file__).resolve().parent)
-if _script_directory not in sys.path:
-    sys.path.insert(0, _script_directory)
-
-from cluster_recurrences import (  # noqa: E402
-    TimingSample,
-    detect_timing_regressions,
-    main,
-    normalize_signature,
-    recency_weight,
-    rank_signature_clusters,
-)
-from collect_log_window import LogRecord  # noqa: E402
-
-
-def _record(timestamp: datetime, message: str) -> LogRecord:
-    return LogRecord(
-        timestamp=timestamp, source="a_hook", level="block", message=message
-    )
-
-
-class TestNormalizeSignature:
-    def test_two_messages_differing_only_in_numbers_share_a_signature(self) -> None:
-        first = normalize_signature("blocked at line 12 with 3 issues")
-        second = normalize_signature("blocked at line 348 with 9 issues")
-        assert first == second
-
-    def test_messages_differing_only_in_a_path_share_a_signature(self) -> None:
-        first = normalize_signature(r"cannot read C:\a\b\one.py right now")
-        second = normalize_signature(r"cannot read C:\x\y\two.py right now")
-        assert first == second
-
-    def test_messages_differing_only_in_a_hash_share_a_signature(self) -> None:
-        first = normalize_signature("commit a1b2c3d failed to apply")
-        second = normalize_signature("commit deadbeef failed to apply")
-        assert first == second
-
-    def test_distinct_messages_keep_distinct_signatures(self) -> None:
-        assert normalize_signature("heavy word detected") != normalize_signature(
-            "missing type hint"
-        )
-
-
-class TestRecencyWeight:
-    def test_a_fresh_record_weighs_about_one(self) -> None:
-        now = datetime(2026, 7, 3, 12, 0, 0)
-        weight = recency_weight(now, now)
-        assert abs(weight - 1.0) < 0.001
-
-    def test_a_record_one_half_life_old_weighs_about_the_decay_base(self) -> None:
-        now = datetime(2026, 7, 3, 12, 0, 0)
-        one_half_life_ago = now - timedelta(hours=24)
-        weight = recency_weight(one_half_life_ago, now)
-        assert abs(weight - 0.5) < 0.001
-
-    def test_a_future_record_is_clamped_to_at_most_one(self) -> None:
-        now = datetime(2026, 7, 3, 12, 0, 0)
-        one_hour_ahead = now + timedelta(hours=1)
-        weight = recency_weight(one_hour_ahead, now)
-        assert abs(weight - 1.0) < 0.001
-
-
-class TestRankSignatureClusters:
-    def test_the_most_frequent_recent_signature_ranks_first(self) -> None:
-        now = datetime(2026, 7, 3, 12, 0, 0)
-        records = [
-            _record(now - timedelta(hours=1), "blocked at line 1"),
-            _record(now - timedelta(hours=2), "blocked at line 2"),
-            _record(now - timedelta(hours=1), "blocked at line 3"),
-            _record(now - timedelta(hours=200), "a rare stale failure"),
-        ]
-        clusters = rank_signature_clusters(records, now)
-        assert clusters[0].count == 3
-        assert clusters[0].signature == normalize_signature("blocked at line 1")
-
-    def test_each_distinct_signature_becomes_one_cluster(self) -> None:
-        now = datetime(2026, 7, 3, 12, 0, 0)
-        records = [
-            _record(now, "heavy word here"),
-            _record(now, "missing type hint"),
-        ]
-        clusters = rank_signature_clusters(records, now)
-        assert len(clusters) == 2
-
-
-class TestDetectTimingRegressions:
-    def test_flags_an_operation_whose_recent_runs_are_slower(self) -> None:
-        base = datetime(2026, 7, 3, 0, 0, 0)
-        samples = [
-            TimingSample("extract", base + timedelta(minutes=1), 100.0),
-            TimingSample("extract", base + timedelta(minutes=2), 110.0),
-            TimingSample("extract", base + timedelta(minutes=3), 105.0),
-            TimingSample("extract", base + timedelta(minutes=40), 300.0),
-            TimingSample("extract", base + timedelta(minutes=41), 320.0),
-            TimingSample("extract", base + timedelta(minutes=42), 310.0),
-        ]
-        regressions = detect_timing_regressions(samples)
-        assert len(regressions) == 1
-        assert regressions[0].operation == "extract"
-        assert regressions[0].ratio > 1.5
-
-    def test_ignores_a_steady_operation(self) -> None:
-        base = datetime(2026, 7, 3, 0, 0, 0)
-        samples = [
-            TimingSample("steady", base + timedelta(minutes=each_minute_offset), 100.0)
-            for each_minute_offset in range(6)
-        ]
-        assert detect_timing_regressions(samples) == []
-
-    def test_ignores_an_operation_with_too_few_samples(self) -> None:
-        base = datetime(2026, 7, 3, 0, 0, 0)
-        samples = [
-            TimingSample("sparse", base + timedelta(minutes=1), 100.0),
-            TimingSample("sparse", base + timedelta(minutes=2), 900.0),
-        ]
-        assert detect_timing_regressions(samples) == []
-
-
-class TestMain:
-    def test_rejects_non_json_stdin(
-        self,
-        monkeypatch: pytest.MonkeyPatch,
-        capsys: pytest.CaptureFixture[str],
-    ) -> None:
-        monkeypatch.setattr(sys, "stdin", io.StringIO("not json"))
-        exit_code = main()
-        captured = capsys.readouterr()
-        assert exit_code == 1
-        assert "not valid JSON" in captured.err
-
-    def test_rejects_a_non_list_payload(
-        self,
-        monkeypatch: pytest.MonkeyPatch,
-        capsys: pytest.CaptureFixture[str],
-    ) -> None:
-        monkeypatch.setattr(sys, "stdin", io.StringIO('{"not": "a list"}'))
-        exit_code = main()
-        captured = capsys.readouterr()
-        assert exit_code == 1
-        assert "must be a list" in captured.err
-
-    def test_rejects_a_malformed_record(
-        self,
-        monkeypatch: pytest.MonkeyPatch,
-        capsys: pytest.CaptureFixture[str],
-    ) -> None:
-        monkeypatch.setattr(sys, "stdin", io.StringIO('[{"missing": "fields"}]'))
-        exit_code = main()
-        captured = capsys.readouterr()
-        assert exit_code == 1
-        assert "malformed record" in captured.err

package/skills/log-audit/scripts/test_collect_log_window.py

@@ -1,111 +0,0 @@
-"""Tests for collect_log_window — parsing and windowing hook-block records."""
-
-import json
-import sys
-from datetime import datetime, timedelta
-from pathlib import Path
-
-_script_directory = str(Path(__file__).resolve().parent)
-if _script_directory not in sys.path:
-    sys.path.insert(0, _script_directory)
-
-from collect_log_window import (  # noqa: E402
-    LogRecord,
-    collect_records,
-    parse_log_line,
-    read_log_window,
-)
-
-
-def _block_line(timestamp_text: str, hook_name: str, reason_text: str) -> str:
-    return json.dumps(
-        {
-            "timestamp": timestamp_text,
-            "hook": hook_name,
-            "event": "PreToolUse",
-            "reason": reason_text,
-        }
-    )
-
-
-class TestParseLogLine:
-    def test_parses_a_well_formed_block_record(self) -> None:
-        line = _block_line(
-            "2026-07-03T09:00:00", "plain_language_blocker", "Heavy word"
-        )
-        record = parse_log_line(line)
-        assert record == LogRecord(
-            timestamp=datetime(2026, 7, 3, 9, 0, 0),
-            source="plain_language_blocker",
-            level="block",
-            message="Heavy word",
-        )
-
-    def test_returns_none_for_a_blank_line(self) -> None:
-        assert parse_log_line("   ") is None
-
-    def test_returns_none_for_non_json(self) -> None:
-        assert parse_log_line("not json at all") is None
-
-    def test_returns_none_when_reason_is_missing(self) -> None:
-        line = json.dumps({"timestamp": "2026-07-03T09:00:00", "hook": "x"})
-        assert parse_log_line(line) is None
-
-    def test_returns_none_for_an_unparseable_timestamp(self) -> None:
-        line = _block_line("not-a-timestamp", "x", "y")
-        assert parse_log_line(line) is None
-
-
-class TestCollectRecords:
-    def test_keeps_only_records_at_or_after_the_window_start(self) -> None:
-        window_start = datetime(2026, 7, 3, 8, 0, 0)
-        log_text = "\n".join(
-            [
-                _block_line("2026-07-03T07:59:59", "old_hook", "before window"),
-                _block_line("2026-07-03T08:00:00", "edge_hook", "on the boundary"),
-                _block_line("2026-07-03T09:30:00", "new_hook", "inside window"),
-            ]
-        )
-        records = collect_records(log_text, window_start)
-        assert [each_record.source for each_record in records] == [
-            "edge_hook",
-            "new_hook",
-        ]
-
-    def test_skips_malformed_lines(self) -> None:
-        window_start = datetime(2026, 7, 3, 0, 0, 0)
-        log_text = "\n".join(
-            [
-                "garbage",
-                _block_line("2026-07-03T09:00:00", "good_hook", "kept"),
-                "",
-            ]
-        )
-        records = collect_records(log_text, window_start)
-        assert len(records) == 1
-        assert records[0].source == "good_hook"
-
-
-class TestReadLogWindow:
-    def test_returns_empty_list_when_the_log_is_absent(self, tmp_path: Path) -> None:
-        missing_path = tmp_path / "hook-blocks.log"
-        assert read_log_window(missing_path, 24, datetime(2026, 7, 3, 12, 0, 0)) == []
-
-    def test_reads_records_inside_the_window_from_a_real_file(
-        self, tmp_path: Path
-    ) -> None:
-        now = datetime(2026, 7, 3, 12, 0, 0)
-        recent_time = (now - timedelta(hours=1)).isoformat()
-        stale_time = (now - timedelta(hours=48)).isoformat()
-        log_path = tmp_path / "hook-blocks.log"
-        log_path.write_text(
-            "\n".join(
-                [
-                    _block_line(stale_time, "stale_hook", "too old"),
-                    _block_line(recent_time, "recent_hook", "in window"),
-                ]
-            ),
-            encoding="utf-8",
-        )
-        records = read_log_window(log_path, 24, now)
-        assert [each_record.source for each_record in records] == ["recent_hook"]

package/skills/log-audit/scripts/test_mine_copilot_findings.py

@@ -1,126 +0,0 @@
-"""Tests for mine_copilot_findings — defect classification and clustering."""
-
-import sys
-from pathlib import Path
-
-_script_directory = str(Path(__file__).resolve().parent)
-if _script_directory not in sys.path:
-    sys.path.insert(0, _script_directory)
-
-from mine_copilot_findings import (  # noqa: E402
-    ReviewerComment,
-    _flatten_pages,
-    _pull_number_from_url,
-    _reviewer_comment_from_payload,
-    classify_defect,
-    cluster_defects,
-    proposal_for_defect_class,
-)
-from log_audit_constants.mine_copilot_findings_constants import (  # noqa: E402
-    PROPOSAL_BY_DEFECT_CLASS,
-)
-
-
-class TestClassifyDefect:
-    def test_recognizes_a_missing_type_hint_comment(self) -> None:
-        assert (
-            classify_defect("This function is missing a type hint on the return")
-            == "missing-type-hint"
-        )
-
-    def test_recognizes_a_broad_except_comment(self) -> None:
-        assert classify_defect("There is a broad except swallowing errors") == (
-            "broad-except"
-        )
-
-    def test_recognizes_a_magic_value_comment(self) -> None:
-        assert classify_defect("42 is a magic number, extract it") == "magic-value"
-
-    def test_returns_none_for_praise(self) -> None:
-        assert classify_defect("Looks great, nicely done!") is None
-
-
-class TestClusterDefects:
-    def test_groups_comments_by_class_and_ranks_by_count(self) -> None:
-        comments = [
-            ReviewerComment(1, "cursor[bot]", "missing type hint here"),
-            ReviewerComment(2, "cursor[bot]", "another missing annotation"),
-            ReviewerComment(3, "cursor[bot]", "this is a broad except"),
-            ReviewerComment(4, "cursor[bot]", "nice work"),
-        ]
-        clusters = cluster_defects(comments)
-        assert clusters[0].defect_class == "missing-type-hint"
-        assert clusters[0].count == 2
-        assert len(clusters) == 2
-
-    def test_caps_the_number_of_example_bodies(self) -> None:
-        comments = [
-            ReviewerComment(
-                each_index, "cursor[bot]", f"missing type hint number {each_index}"
-            )
-            for each_index in range(10)
-        ]
-        clusters = cluster_defects(comments)
-        assert clusters[0].count == 10
-        assert len(clusters[0].example_bodies) == 3
-
-
-class TestProposalForDefectClass:
-    def test_maps_a_class_to_its_skill_edit_proposal(self) -> None:
-        proposal = proposal_for_defect_class("broad-except")
-        assert proposal.defect_class == "broad-except"
-        assert proposal.proposal == PROPOSAL_BY_DEFECT_CLASS["broad-except"]
-
-
-class TestPullNumberFromUrl:
-    def test_reads_the_trailing_pull_number(self) -> None:
-        url = "https://api.github.com/repos/owner/name/pulls/867"
-        assert _pull_number_from_url(url) == 867
-
-    def test_returns_none_when_the_last_segment_is_not_a_number(self) -> None:
-        assert _pull_number_from_url("https://api.github.com/repos/owner/name") is None
-
-
-class TestReviewerCommentFromPayload:
-    def test_builds_a_comment_from_a_reviewer_bot_payload(self) -> None:
-        payload = {
-            "user": {"login": "cursor[bot]"},
-            "body": "missing type hint",
-            "pull_request_url": "https://api.github.com/repos/owner/name/pulls/12",
-        }
-        parsed_comment = _reviewer_comment_from_payload(payload)
-        assert parsed_comment == ReviewerComment(
-            pull_number=12, author="cursor[bot]", body="missing type hint"
-        )
-
-    def test_skips_a_comment_from_a_non_reviewer_login(self) -> None:
-        payload = {
-            "user": {"login": "some-human"},
-            "body": "looks good",
-            "pull_request_url": "https://api.github.com/repos/owner/name/pulls/12",
-        }
-        assert _reviewer_comment_from_payload(payload) is None
-
-    def test_skips_a_payload_that_is_not_a_dict(self) -> None:
-        assert _reviewer_comment_from_payload("not a payload") is None
-
-    def test_skips_a_payload_missing_the_pull_request_url(self) -> None:
-        payload = {"user": {"login": "cursor[bot]"}, "body": "missing type hint"}
-        assert _reviewer_comment_from_payload(payload) is None
-
-    def test_skips_a_payload_whose_url_has_no_numeric_pull_number(self) -> None:
-        payload = {
-            "user": {"login": "cursor[bot]"},
-            "body": "missing type hint",
-            "pull_request_url": "https://api.github.com/repos/owner/name/pulls",
-        }
-        assert _reviewer_comment_from_payload(payload) is None
-
-
-class TestFlattenPages:
-    def test_flattens_multiple_pages_into_one_list(self) -> None:
-        slurped_payload = [[{"id": 1}, {"id": 2}], [{"id": 3}]]
-        assert _flatten_pages(slurped_payload) == [{"id": 1}, {"id": 2}, {"id": 3}]
-
-    def test_returns_empty_list_for_a_non_list_payload(self) -> None:
-        assert _flatten_pages({"message": "not a list"}) == []

package/skills/logifix/CLAUDE.md

@@ -1,36 +0,0 @@
-# logifix skill
-
-Restores the Logitech Gaming Software (LCore) tray icon when it disappears on Windows. Runs a PowerShell script that reproduces the verified recovery procedure.
-
-**Trigger:** `/logifix`, "logitech tray icon missing", "LCore tray gone", "logitech is not loaded".
-
-## Key files
-
-| File | Purpose |
-|---|---|
-| `SKILL.md` | Full procedure description, invocation options, fallback steps |
-| `scripts/logifix.ps1` | PowerShell script that runs the recovery procedure |
-
-## Subdirectories
-
-| Directory | Role |
-|---|---|
-| `scripts/` | The PowerShell recovery script |
-
-## What the script does
-
-1. Takes a state snapshot (Explorer instances, Logitech services, LCore process).
-2. Stops LCore in user context.
-3. Runs one elevated UAC step: starts `LogiRegistryService`, stops `explorer.exe`, and lets Windows shell auto-respawn rebuild it (does **not** call `Start-Process explorer` from the elevated block).
-4. Waits for shell auto-respawn (default 5 seconds).
-5. Confirms exactly one Explorer in the user's session.
-6. Runs `LCoreRelaunchAttemptCount` full stop-then-launch cycles of `LCore.exe /minimized` (default 2). Always runs the full count.
-7. Takes a final state snapshot.
-
-## Invocation
-
-```
-/logifix
-```
-
-Optional parameters: `-ExplorerAutoRespawnWaitSeconds`, `-LCoreInitializationWaitSeconds`, `-LCoreRelaunchAttemptCount`.

package/skills/logifix/SKILL.md

@@ -1,69 +0,0 @@
----
-name: logifix
-description: Restore the Logitech Gaming Software (LCore) tray icon when it disappears on Windows. Calls a PowerShell script that reproduces the verified Session 2 recovery procedure from 2026-04-25. Triggers on "logifix", "/logifix", "logitech tray icon missing", "LCore tray gone", "logitech is not loaded".
----
-
-# logifix
-
-## Overview
-
-Restore the Logitech Gaming Software (LCore) tray icon by reproducing the verified recovery procedure documented in `sessions/System Support/2. Logitech Tray Icon Fix Recurrence.md`.
-
-**Announce at start:** "Running /logifix to restore the LCore tray icon."
-
-## What the script does
-
-`scripts/logifix.ps1` runs this sequence (verified during Session 2 on 2026-04-25):
-
-1. **State snapshot.** Explorer instances per session, Logitech services state, LCore process state.
-2. **Stop LCore in user context.** `Stop-Process -Name LCore -Force`.
-3. **Single elevated UAC step:**
-   - `Start-Service -Name LogiRegistryService` (handles the recurrence case where the service is Stopped).
-   - `Stop-Process -Name explorer -Force`.
-   - **Does NOT** call `Start-Process explorer` from inside the elevated block. Windows shell auto-respawn handles the user-session explorer cleanly. Skipping this line is the operative fix discovered in Session 2 — including it created a duplicate admin-context `explorer.exe` (no resolvable owner, parent = the admin pwsh) that blocked LCore tray registration.
-4. **Wait for Windows shell auto-respawn** (default 5 seconds).
-5. **Verify exactly one explorer in the user's session.**
-6. **Perform `LCoreRelaunchAttemptCount` full stop-then-launch cycles of `LCore.exe /minimized`** (default 2). The full count always runs — a responsive LCore on the first attempt does NOT imply the tray icon registered. Per Session 2 gotcha #2, the first relaunch can leave LCore responsive with no tray icon; only a second stop+launch reliably triggers `Shell_NotifyIcon` registration.
-7. **Final state snapshot.**
-
-## Invocation
-
-From Claude Code:
-
-```
-/logifix
-```
-
-Direct PowerShell:
-
-```
-pwsh -File "$HOME\.claude\skills\logifix\scripts\logifix.ps1"
-```
-
-Optional parameters (all have safe defaults):
-
-- `-ExplorerAutoRespawnWaitSeconds <int>` — wait after the elevated kill (default 5).
-- `-LCoreInitializationWaitSeconds <int>` — wait after each LCore relaunch before checking responsiveness (default 5).
-- `-LCoreRelaunchAttemptCount <int>` — number of LCore stop+launch cycles to perform (default 2). Always runs the full count.
-
-## When to use
-
-- LCore tray icon missing (also confirmed absent from the overflow `^` chevron).
-- LCore process is running but the tray icon never appears.
-- After resume from sleep, system restart, or a Logitech service crash that leaves LCore in a half-loaded state.
-
-## Fallback (if the script does not restore the icon)
-
-If `/logifix` reports that UAC was canceled, or LCore is still not responding after both relaunch cycles:
-
-1. **Ctrl+Shift+Esc** → Task Manager.
-2. Find **Windows Explorer** → right-click → **Restart**.
-3. Re-run `/logifix`.
-
-The Task Manager restart path is guaranteed to hit the correct interactive session and elevation context regardless of how the calling shell was launched. In Session 2, elevated calls from Claude Code's PowerShell tool could not be confirmed to reach the user's interactive Session 1 — Task Manager bypasses that ambiguity.
-
-## Source
-
-Procedure verified during Session 2 (Logitech Tray Icon Fix Recurrence), 2026-04-25. The session log lists the verified command set, the gotcha catalog, and the final process/service state.
-
-The "always run the full relaunch count" behavior was added on 2026-04-26 after `/logifix` reported success on the first responsive attempt but the tray icon never appeared — the original early-break optimization conflicted with documented Session 2 gotcha #2 (responsive LCore, no tray icon).

package/skills/logifix/scripts/CLAUDE.md

@@ -1,16 +0,0 @@
-# logifix/scripts
-
-PowerShell recovery script for the `logifix` skill.
-
-## Key files
-
-| File | Purpose |
-|---|---|
-| `logifix.ps1` | Restores the LCore tray icon: stops LCore, runs one elevated UAC step (starts `LogiRegistryService`, stops explorer, lets Windows auto-respawn), then re-launches LCore a configurable number of times |
-
-## Conventions
-
-- The script accepts three optional parameters: `-ExplorerAutoRespawnWaitSeconds` (default 5), `-LCoreInitializationWaitSeconds` (default 5), `-LCoreRelaunchAttemptCount` (default 2).
-- The full relaunch count always runs — an early-exit on a responsive LCore is intentional behavior per the recovery procedure.
-- The elevated block does **not** call `Start-Process explorer`; Windows shell auto-respawn handles that to avoid a duplicate admin-context Explorer.
-- Run directly as: `pwsh -File "$HOME\.claude\skills\logifix\scripts\logifix.ps1"`.