test: add Windows CI coverage

Author: he-yufeng

.github/workflows/ci.yml

@@ -11,7 +11,7 @@ jobs:
     runs-on: ${{ matrix.os }}
     strategy:
       matrix:
-        os: [ubuntu-latest, macos-latest]
+        os: [ubuntu-latest, macos-latest, windows-latest]
         python-version: ["3.10", "3.12"]
 
     steps:

corecoder/__init__.py

@@ -1,6 +1,6 @@
 """CoreCoder - Minimal AI coding agent inspired by Claude Code's architecture."""
 
-__version__ = "0.2.0"
+__version__ = "0.3.0"
 
 from corecoder.agent import Agent
 from corecoder.llm import LLM

tests/test_core.py

@@ -2,7 +2,6 @@
 
 import os
 import pathlib
-import tempfile
 
 from corecoder import Agent, LLM, Config, ALL_TOOLS, __version__
 from corecoder.context import ContextManager, estimate_tokens
@@ -11,7 +10,7 @@
 
 
 def test_version():
-    assert __version__ == "0.2.0"
+    assert __version__ == "0.3.0"
 
 
 def test_public_api_exports():
@@ -122,25 +121,22 @@ def test_cost_estimation_unknown_model():
 
 # --- Changed files tracking ---
 
-def test_edit_tracks_changed_files():
+def test_edit_tracks_changed_files(tmp_path):
     from corecoder.tools.edit import _changed_files
     _changed_files.clear()
     edit = get_tool("edit_file")
-    with tempfile.NamedTemporaryFile(mode="w", suffix=".py", delete=False) as f:
-        f.write("aaa\nbbb\n")
-        f.flush()
-        edit.execute(file_path=f.name, old_string="aaa", new_string="zzz")
-        assert any(f.name in p for p in _changed_files)
-        os.unlink(f.name)
+    path = tmp_path / "sample.py"
+    path.write_text("aaa\nbbb\n")
+    edit.execute(file_path=str(path), old_string="aaa", new_string="zzz")
+    assert any(str(path) in p for p in _changed_files)
     _changed_files.clear()
 
 
-def test_write_tracks_changed_files():
+def test_write_tracks_changed_files(tmp_path):
     from corecoder.tools.edit import _changed_files
     _changed_files.clear()
     write = get_tool("write_file")
-    path = tempfile.mktemp(suffix=".txt")
-    write.execute(file_path=path, content="tracked\n")
-    assert any("tracked" not in p and path.split("/")[-1] in p for p in _changed_files) or len(_changed_files) > 0
-    os.unlink(path)
+    path = tmp_path / "tracked.txt"
+    write.execute(file_path=str(path), content="tracked\n")
+    assert any("tracked" not in p and path.name in p for p in _changed_files) or len(_changed_files) > 0
     _changed_files.clear()

tests/test_tools.py

@@ -1,6 +1,7 @@
 """Tests for the tool system."""
 
 import os
+import sys
 import tempfile
 from pathlib import Path
 
@@ -38,7 +39,7 @@ def test_bash_exit_code():
 
 def test_bash_timeout():
     bash = get_tool("bash")
-    r = bash.execute(command="sleep 10", timeout=1)
+    r = bash.execute(command=f'"{sys.executable}" -c "import time; time.sleep(10)"', timeout=1)
     assert "timed out" in r
 
 
@@ -62,21 +63,19 @@ def test_bash_blocks_curl_pipe():
 
 def test_bash_truncates_long_output():
     bash = get_tool("bash")
-    r = bash.execute(command="python3 -c \"print('x' * 20000)\"")
+    r = bash.execute(command=f'"{sys.executable}" -c "print(\'x\' * 20000)"')
     assert "truncated" in r
 
 
 # --- read_file ---
 
-def test_read_file():
+def test_read_file(tmp_path):
     read = get_tool("read_file")
-    with tempfile.NamedTemporaryFile(mode="w", suffix=".txt", delete=False) as f:
-        f.write("line1\nline2\nline3\n")
-        f.flush()
-        r = read.execute(file_path=f.name)
-        assert "line1" in r
-        assert "line2" in r
-        os.unlink(f.name)
+    path = tmp_path / "sample.txt"
+    path.write_text("line1\nline2\nline3\n")
+    r = read.execute(file_path=str(path))
+    assert "line1" in r
+    assert "line2" in r
 
 
 def test_read_file_not_found():
@@ -85,14 +84,12 @@ def test_read_file_not_found():
     assert "not found" in r.lower() or "Error" in r
 
 
-def test_read_file_offset_limit():
+def test_read_file_offset_limit(tmp_path):
     read = get_tool("read_file")
-    with tempfile.NamedTemporaryFile(mode="w", suffix=".txt", delete=False) as f:
-        f.write("\n".join(f"line{i}" for i in range(100)))
-        f.flush()
-        r = read.execute(file_path=f.name, offset=10, limit=5)
-        assert "line10" not in r or "line9" in r  # offset is 1-based
-        os.unlink(f.name)
+    path = tmp_path / "sample.txt"
+    path.write_text("\n".join(f"line{i}" for i in range(100)))
+    r = read.execute(file_path=str(path), offset=10, limit=5)
+    assert "line10" not in r or "line9" in r  # offset is 1-based
 
 
 # --- write_file ---
@@ -119,38 +116,32 @@ def test_write_file_creates_dirs():
 
 # --- edit_file ---
 
-def test_edit_file_basic():
+def test_edit_file_basic(tmp_path):
     edit = get_tool("edit_file")
-    with tempfile.NamedTemporaryFile(mode="w", suffix=".py", delete=False) as f:
-        f.write("def foo():\n    return 42\n")
-        f.flush()
-        r = edit.execute(file_path=f.name, old_string="return 42", new_string="return 99")
-        assert "Edited" in r
-        assert "---" in r  # unified diff
-        content = Path(f.name).read_text()
-        assert "return 99" in content
-        assert "return 42" not in content
-        os.unlink(f.name)
-
-
-def test_edit_file_not_found_string():
+    path = tmp_path / "sample.py"
+    path.write_text("def foo():\n    return 42\n")
+    r = edit.execute(file_path=str(path), old_string="return 42", new_string="return 99")
+    assert "Edited" in r
+    assert "---" in r  # unified diff
+    content = path.read_text()
+    assert "return 99" in content
+    assert "return 42" not in content
+
+
+def test_edit_file_not_found_string(tmp_path):
     edit = get_tool("edit_file")
-    with tempfile.NamedTemporaryFile(mode="w", suffix=".py", delete=False) as f:
-        f.write("hello\n")
-        f.flush()
-        r = edit.execute(file_path=f.name, old_string="NONEXISTENT", new_string="x")
-        assert "not found" in r.lower()
-        os.unlink(f.name)
+    path = tmp_path / "sample.py"
+    path.write_text("hello\n")
+    r = edit.execute(file_path=str(path), old_string="NONEXISTENT", new_string="x")
+    assert "not found" in r.lower()
 
 
-def test_edit_file_duplicate_string():
+def test_edit_file_duplicate_string(tmp_path):
     edit = get_tool("edit_file")
-    with tempfile.NamedTemporaryFile(mode="w", suffix=".py", delete=False) as f:
-        f.write("dup\ndup\n")
-        f.flush()
-        r = edit.execute(file_path=f.name, old_string="dup", new_string="x")
-        assert "2 times" in r
-        os.unlink(f.name)
+    path = tmp_path / "sample.py"
+    path.write_text("dup\ndup\n")
+    r = edit.execute(file_path=str(path), old_string="dup", new_string="x")
+    assert "2 times" in r
 
 
 # --- glob ---