fix: fix lint

Author: koxudaxi

.github/workflows/lint.yml

@@ -45,5 +45,5 @@ jobs:
         run: uv run ruff check . && uv run ruff format --check .
 
       - name: Run Pyright (Type Check)
-        run: uv run pyright .
+        run: uv run pyright sandbox
 

tests/test_manager.py

@@ -189,7 +189,7 @@ def test_sandbox_run_function_invalid_json_output(mocked_sandbox_factory):
     create_sandbox, mock_runner = mocked_sandbox_factory
     invalid_stdout = "This is not JSON { invalid"
     mock_runner.execute_script.return_value = Result(
-        stdout=invalid_stdout, stderr="", error=None
+        stdout=invalid_stdout, stderr="", error=None, result=None
     )
     with create_sandbox() as sandbox:
         result = sandbox.run_function(

tests/test_subprocess_runner.py

@@ -30,6 +30,9 @@ def runner() -> SubprocessSandboxRunner:
 def test_subprocess_runner_simple_script(runner: SubprocessSandboxRunner):
     """Test running a simple script."""
     result = runner.execute_script(SIMPLE_SCRIPT)
+    assert "stdout" in result
+    assert "stderr" in result
+    assert "error" in result
     assert "Hello from subprocess!" in result["stdout"]
     assert result["stderr"] == ""
     assert result["error"] is None
@@ -38,15 +41,21 @@ def test_subprocess_runner_simple_script(runner: SubprocessSandboxRunner):
 def test_subprocess_runner_json_output(runner: SubprocessSandboxRunner):
     """Test script outputting JSON."""
     result = runner.execute_script(JSON_OUTPUT_SCRIPT)
+    assert "stdout" in result
+    assert "stderr" in result
+    assert "error" in result
     assert '"result": 42' in result["stdout"]
     assert result["error"] is None
 
 
 def test_subprocess_runner_failing_script(runner: SubprocessSandboxRunner):
     """Test running a script that raises an exception."""
     result = runner.execute_script(FAILING_SCRIPT)
+    assert "stdout" in result
+    assert "stderr" in result
     assert result["stdout"] == ""
     assert "ValueError: Intentional fail" in result["stderr"]
+    assert "error" in result
     assert result["error"] is not None
     assert "exit code 1" in result["error"]
 
@@ -59,9 +68,10 @@ def test_subprocess_runner_timeout_expires(runner: SubprocessSandboxRunner):
     result = timeout_runner.execute_script(SLEEP_SCRIPT)  # This script takes 2s
     end_time = time.monotonic()
 
+    assert "error" in result
     assert result["error"] is not None
     assert "timed out" in result["error"]
-    assert "TimeoutExpired" in result["stderr"]
+    assert "stderr" in result and "TimeoutExpired" in result["stderr"]
     # Check if it actually timed out reasonably close to the limit
     assert end_time - start_time < 1.5  # Should be around 0.5s + overhead
 
@@ -73,8 +83,8 @@ def test_subprocess_runner_timeout_sufficient(runner: SubprocessSandboxRunner):
     result = runner.execute_script(SLEEP_SCRIPT)
     end_time = time.monotonic()
 
-    assert result["error"] is None
-    assert "Done sleeping" in result["stdout"]
+    assert "error" in result and result["error"] is None
+    assert "Done sleeping" in result["stdout"]  # pyright: ignore [reportTypedDictNotRequiredAccess]
     assert end_time - start_time >= 2.0  # Ensure it actually slept
 
 
@@ -102,8 +112,8 @@ def test_subprocess_runner_with_deps():
     print(json.dumps({"error": str(e)}))
 """
     result = runner.execute_script(script_with_header)
-    assert result["error"] is None, f"Execution failed: {result['stderr']}"
-    assert '"result": 200' in result["stdout"] or '"error":' in result["stdout"]
+    assert result["error"] is None, f"Execution failed: {result['stderr']}"  # pyright: ignore [reportTypedDictNotRequiredAccess]
+    assert '"result": 200' in result["stdout"] or '"error":' in result["stdout"]  # pyright: ignore [reportTypedDictNotRequiredAccess]
 
 
 def test_subprocess_runner_uv_not_found():