fix: address Copilot PR review - schema anyOf, encoding, tests

- Add anyOf to 5 build tool schemas enforcing code/source_path requirement
- Fix CHANGELOG: remove misleading GBK example for latin-1 fallback
- Add encoding="utf-8" to stress_test.py file reads
- Add tests for source_path (success, not found, neither provided)
- Add test for stress_test generator failure diagnostics

Author: SummerOneTwo

CHANGELOG.md

@@ -10,7 +10,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 ### Features
 
 - **source_path 参数**: 所有构建工具（solution_build, generator_build, validator_build, checker_build, interactor_build）新增 `source_path` 参数，可直接指定源文件路径，无需传入完整源码字符串。`code` 参数不再为必填，与 `source_path` 二选一。
-- **source_path 编码回退**: 自动处理非 UTF-8 编码的源文件（如 GBK），先尝试 UTF-8 读取，失败后回退到 latin-1。
+- **source_path 编码回退**: 自动处理非 UTF-8 编码的源文件，先尝试 UTF-8 读取，失败后回退到 latin-1（宽松解码，不会抛异常但可能产生乱码）。
 - **source_path 相对 include 支持**: 当 `source_path` 指向外部文件时，自动将源文件父目录加入编译 include 路径，确保 `#include "helper.h"` 等相对引用正常工作。
 
 ### Improvements

src/autocode_mcp/tools/checker.py

@@ -76,6 +76,10 @@ def input_schema(self) -> dict:
                 },
             },
             "required": ["problem_dir"],
+            "anyOf": [
+                {"required": ["code"]},
+                {"required": ["source_path"]},
+            ],
         }
 
     async def execute(

src/autocode_mcp/tools/generator.py

@@ -60,6 +60,10 @@ def input_schema(self) -> dict:
                 },
             },
             "required": ["problem_dir"],
+            "anyOf": [
+                {"required": ["code"]},
+                {"required": ["source_path"]},
+            ],
         }
 
     async def execute(

src/autocode_mcp/tools/interactor.py

@@ -68,6 +68,10 @@ def input_schema(self) -> dict:
                 },
             },
             "required": ["problem_dir"],
+            "anyOf": [
+                {"required": ["code"]},
+                {"required": ["source_path"]},
+            ],
         }
 
     async def execute(

src/autocode_mcp/tools/solution.py

@@ -62,6 +62,10 @@ def input_schema(self) -> dict:
                 },
             },
             "required": ["problem_dir", "solution_type"],
+            "anyOf": [
+                {"required": ["code"]},
+                {"required": ["source_path"]},
+            ],
         }
 
     async def execute(

src/autocode_mcp/tools/stress_test.py

@@ -169,7 +169,7 @@ async def execute(
 
                 # 2. 验证输入（如果有 validator）
                 if os.path.exists(val_exe):
-                    with open(input_path) as f:
+                    with open(input_path, encoding="utf-8") as f:
                         input_data = f.read()
                     val_result = await run_binary(val_exe, input_data, timeout=timeout)
                     if val_result.return_code != 0:
@@ -179,7 +179,7 @@ async def execute(
                         break
 
                 # 3. 运行 sol 和 brute，比较输出
-                with open(input_path) as f:
+                with open(input_path, encoding="utf-8") as f:
                     input_data = f.read()
                 last_input = input_data
 

src/autocode_mcp/tools/validator.py

@@ -75,6 +75,10 @@ def input_schema(self) -> dict:
                 },
             },
             "required": ["problem_dir"],
+            "anyOf": [
+                {"required": ["code"]},
+                {"required": ["source_path"]},
+            ],
         }
 
     async def execute(

tests/test_packaging.py

@@ -290,3 +290,103 @@ async def test_checker_fail_verdict():
         test_results = result.structuredContent.get("test_results", [])
         if test_results:
             assert test_results[0].get("actual_verdict") == "FAIL"
+
+
+# ============== source_path 参数测试 ==============
+
+
+@pytest.mark.asyncio
+async def test_solution_build_source_path():
+    """测试 solution_build 使用 source_path 参数。"""
+    from autocode_mcp.server import call_tool, register_all_tools
+
+    register_all_tools()
+
+    import tempfile
+
+    with tempfile.TemporaryDirectory() as tmpdir:
+        os.makedirs(os.path.join(tmpdir, "solutions"))
+        source_file = os.path.join(tmpdir, "solutions", "sol.cpp")
+        with open(source_file, "w", encoding="utf-8") as f:
+            f.write('#include <iostream>\nint main() { std::cout << 42; return 0; }')
+
+        result = await call_tool(
+            "solution_build",
+            {"problem_dir": tmpdir, "solution_type": "sol", "source_path": source_file},
+        )
+        assert result.isError is False
+
+
+@pytest.mark.asyncio
+async def test_solution_build_source_path_not_found():
+    """测试 source_path 文件不存在时报错。"""
+    from autocode_mcp.server import call_tool, register_all_tools
+
+    register_all_tools()
+
+    import tempfile
+
+    with tempfile.TemporaryDirectory() as tmpdir:
+        result = await call_tool(
+            "solution_build",
+            {
+                "problem_dir": tmpdir,
+                "solution_type": "sol",
+                "source_path": os.path.join(tmpdir, "nonexistent.cpp"),
+            },
+        )
+        assert result.isError is True
+        assert "not found" in result.structuredContent.get("error", "").lower()
+
+
+@pytest.mark.asyncio
+async def test_solution_build_neither_code_nor_source_path():
+    """测试既不提供 code 也不提供 source_path 时报错。"""
+    from autocode_mcp.server import call_tool, register_all_tools
+
+    register_all_tools()
+
+    import tempfile
+
+    with tempfile.TemporaryDirectory() as tmpdir:
+        result = await call_tool(
+            "solution_build",
+            {"problem_dir": tmpdir, "solution_type": "sol"},
+        )
+        assert result.isError is True
+        error = result.structuredContent.get("error", "").lower()
+        assert "either" in error or "must be provided" in error
+
+
+# ============== stress_test 错误诊断测试 ==============
+
+
+@pytest.mark.asyncio
+async def test_stress_test_generator_timeout_hint():
+    """测试 generator 超时时返回特定提示和数据字段。"""
+    from autocode_mcp.server import call_tool, register_all_tools
+
+    register_all_tools()
+
+    import tempfile
+
+    with tempfile.TemporaryDirectory() as tmpdir:
+        os.makedirs(os.path.join(tmpdir, "files"))
+        os.makedirs(os.path.join(tmpdir, "solutions"))
+
+        gen_code = '#include "testlib.h"\nint main(int argc, char* argv[]) { while(true); return 0; }'
+        gen_result = await call_tool("generator_build", {"problem_dir": tmpdir, "code": gen_code})
+        if gen_result.isError:
+            pytest.skip("Generator compilation failed (g++ not available)")
+
+        simple_code = '#include <iostream>\nint main() { int x; std::cin >> x; std::cout << x; return 0; }'
+        await call_tool("solution_build", {"problem_dir": tmpdir, "solution_type": "sol", "code": simple_code})
+        await call_tool("solution_build", {"problem_dir": tmpdir, "solution_type": "brute", "code": simple_code})
+
+        result = await call_tool("stress_test_run", {"problem_dir": tmpdir, "trials": 1, "timeout": 2})
+        assert result.isError is True
+        error_msg = result.structuredContent.get("error", "").lower()
+        assert "generator failed" in error_msg
+        data = result.structuredContent.get("data", {})
+        assert "seed" in data
+        assert "cmd_args" in data