ci: add windows-latest to test matrix

Author: clean6378-max-it

.github/workflows/ci.yml

@@ -44,7 +44,11 @@ jobs:
           PY
 
   pytest:
-    runs-on: ubuntu-latest
+    runs-on: ${{ matrix.os }}
+    strategy:
+      fail-fast: false
+      matrix:
+        os: [ubuntu-latest, windows-latest]
     permissions:
       contents: read
     steps:
@@ -94,7 +98,11 @@ jobs:
 
   integration-tests:
     name: API integration tests + coverage
-    runs-on: ubuntu-latest
+    runs-on: ${{ matrix.os }}
+    strategy:
+      fail-fast: false
+      matrix:
+        os: [ubuntu-latest, windows-latest]
     permissions:
       contents: read
       actions: write
@@ -120,12 +128,16 @@ jobs:
 
       - uses: actions/upload-artifact@ea165f8d65b6e75b540449e92b4886f43607fa02 # v4
         with:
-          name: coverage-report
+          name: coverage-report-${{ matrix.os }}
           path: coverage.xml
 
   js-tests:
     name: Frontend unit tests (vitest)
-    runs-on: ubuntu-latest
+    runs-on: ${{ matrix.os }}
+    strategy:
+      fail-fast: false
+      matrix:
+        os: [ubuntu-latest, windows-latest]
     permissions:
       contents: read
     steps:

CONTRIBUTING.md

@@ -9,6 +9,8 @@ Thanks for considering a patch. This repo is a small Flask app plus a hash-route
 - **Python 3.12** (matches CI)
 - **Node 20+** (only if you change `static/js/` or run frontend unit tests)
 
+CI runs **`pytest`**, **integration tests**, and **Vitest** on **ubuntu-latest** and **windows-latest** (Python 3.12, Node 20). Type-check (`mypy`) and production install smoke run on Ubuntu only.
+
 ### Bootstrap (Windows PowerShell)
 
 ```powershell
@@ -105,7 +107,7 @@ npm run test:coverage   # optional
 - PR checklist:
   - [ ] `pytest -q` green locally
   - [ ] `npm test` green if JS changed
-  - [ ] CI jobs green (`pytest`, `integration-tests`, `js-tests`, `prod-install-smoke`)
+  - [ ] CI jobs green (`pytest`, `integration-tests`, `js-tests` on Ubuntu + Windows; `mypy`, `prod-install-smoke` on Ubuntu)
   - [ ] PR description includes a **Test plan** section
   - [ ] API changes update [`docs/api-reference.md`](docs/api-reference.md) if behavior or errors change
 

tests/test_export_state_store.py

@@ -3,9 +3,16 @@
 from __future__ import annotations
 
 import json
+import sys
 from pathlib import Path
 
-from utils.export_state_store import load_export_state_from_disk
+import pytest
+
+from utils.export_state_store import (
+    atomic_write_export_state,
+    export_state_lock,
+    load_export_state_from_disk,
+)
 
 
 def test_load_rejects_non_object_json(tmp_path: Path):
@@ -81,3 +88,22 @@ def locking(fd, mode, nbytes):
     with mod.export_state_lock(str(state_file)):
         assert (FakeMsvcrt.LK_LOCK, 1) in calls
     assert calls[-1] == (FakeMsvcrt.LK_UNLCK, 1)
+
+
+@pytest.mark.skipif(sys.platform != "win32", reason="requires Windows msvcrt")
+def test_export_state_lock_real_msvcrt_roundtrip(tmp_path: Path) -> None:
+    """Exercise real ``msvcrt.locking`` on a Windows runner (not FakeMsvcrt)."""
+    state_file = tmp_path / "export_state.json"
+    state_file.write_text("{}", encoding="utf-8")
+    payload = {
+        "sessions": {},
+        "lastExportTime": "2026-01-01T00:00:00",
+        "exportedCount": 0,
+    }
+
+    with export_state_lock(str(state_file)):
+        atomic_write_export_state(payload, str(state_file))
+
+    loaded = load_export_state_from_disk(str(state_file))
+    assert loaded.get("exportedCount") == 0
+    assert loaded.get("sessions") == {}

tests/test_session_path.py

@@ -0,0 +1,37 @@
+"""Tests for utils/session_path platform-specific home resolution."""
+
+from __future__ import annotations
+
+import os
+import sys
+from pathlib import Path
+
+import pytest
+
+from utils import session_path
+
+
+def test_get_claude_projects_dir_uses_userprofile_on_windows(
+    monkeypatch: pytest.MonkeyPatch, tmp_path: Path
+) -> None:
+    profile = tmp_path / "Users" / "chen"
+    profile.mkdir(parents=True)
+    monkeypatch.setattr(session_path.platform, "system", lambda: "Windows")
+    monkeypatch.setenv("USERPROFILE", str(profile))
+    monkeypatch.delenv("HOME", raising=False)
+
+    got = session_path.get_claude_projects_dir()
+    assert got == os.path.join(str(profile), ".claude", "projects")
+
+
+@pytest.mark.skipif(sys.platform != "win32", reason="native Windows runner")
+def test_get_claude_projects_dir_on_windows_runner(
+    monkeypatch: pytest.MonkeyPatch, tmp_path: Path
+) -> None:
+    profile = tmp_path / "profile"
+    profile.mkdir()
+    monkeypatch.setenv("USERPROFILE", str(profile))
+
+    got = session_path.get_claude_projects_dir()
+    assert got.startswith(str(profile))
+    assert got.endswith(os.path.join(".claude", "projects"))