aria-code 0.147.0__tar.gz → 0.149.0__tar.gz

{aria_code-0.147.0/src/aria_code.egg-info → aria_code-0.149.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: aria-code
-Version: 0.147.0
+Version: 0.149.0
 Summary: Open-source AI agent for coding, financial analysis and logistics operations: numbers you can check, client data kept apart, actions a person approves
 License: Apache-2.0
 Project-URL: Homepage, https://github.com/artheras/aria-code
@@ -172,7 +172,7 @@ Numbers you can check, client data kept apart, actions a person approves.</p>
 <p align="center">
   <a href="https://www.npmjs.com/package/@artheras/aria-code"><img src="https://img.shields.io/npm/v/@artheras/aria-code/latest?style=flat-square&logo=npm&label=npm" alt="npm version"></a>
   <!-- Release automation pins PyPI: the default endpoint still selects legacy 4.x. -->
-  <a href="https://pypi.org/project/aria-code/0.147.0/"><img src="https://img.shields.io/badge/dynamic/json?url=https%3A%2F%2Fpypi.org%2Fpypi%2Faria-code%2F0.147.0%2Fjson&amp;query=%24.info.version&amp;prefix=v&amp;style=flat-square&amp;logo=pypi&amp;label=PyPI&amp;color=blue&amp;cacheSeconds=300" alt="PyPI version"></a>
+  <a href="https://pypi.org/project/aria-code/0.149.0/"><img src="https://img.shields.io/badge/dynamic/json?url=https%3A%2F%2Fpypi.org%2Fpypi%2Faria-code%2F0.149.0%2Fjson&amp;query=%24.info.version&amp;prefix=v&amp;style=flat-square&amp;logo=pypi&amp;label=PyPI&amp;color=blue&amp;cacheSeconds=300" alt="PyPI version"></a>
   <a href="https://github.com/artheras/aria-code/actions/workflows/ci.yml"><img src="https://img.shields.io/github/actions/workflow/status/artheras/aria-code/ci.yml?branch=main&style=flat-square&logo=githubactions&label=CI" alt="CI status"></a>
   <a href="LICENSE"><img src="https://img.shields.io/badge/license-Apache%202.0-64748b?style=flat-square" alt="Apache License 2.0"></a>
 </p>

{aria_code-0.147.0 → aria_code-0.149.0}/README.md

@@ -15,7 +15,7 @@ Numbers you can check, client data kept apart, actions a person approves.</p>
 <p align="center">
   <a href="https://www.npmjs.com/package/@artheras/aria-code"><img src="https://img.shields.io/npm/v/@artheras/aria-code/latest?style=flat-square&logo=npm&label=npm" alt="npm version"></a>
   <!-- Release automation pins PyPI: the default endpoint still selects legacy 4.x. -->
-  <a href="https://pypi.org/project/aria-code/0.147.0/"><img src="https://img.shields.io/badge/dynamic/json?url=https%3A%2F%2Fpypi.org%2Fpypi%2Faria-code%2F0.147.0%2Fjson&amp;query=%24.info.version&amp;prefix=v&amp;style=flat-square&amp;logo=pypi&amp;label=PyPI&amp;color=blue&amp;cacheSeconds=300" alt="PyPI version"></a>
+  <a href="https://pypi.org/project/aria-code/0.149.0/"><img src="https://img.shields.io/badge/dynamic/json?url=https%3A%2F%2Fpypi.org%2Fpypi%2Faria-code%2F0.149.0%2Fjson&amp;query=%24.info.version&amp;prefix=v&amp;style=flat-square&amp;logo=pypi&amp;label=PyPI&amp;color=blue&amp;cacheSeconds=300" alt="PyPI version"></a>
   <a href="https://github.com/artheras/aria-code/actions/workflows/ci.yml"><img src="https://img.shields.io/github/actions/workflow/status/artheras/aria-code/ci.yml?branch=main&style=flat-square&logo=githubactions&label=CI" alt="CI status"></a>
   <a href="LICENSE"><img src="https://img.shields.io/badge/license-Apache%202.0-64748b?style=flat-square" alt="Apache License 2.0"></a>
 </p>

{aria_code-0.147.0 → aria_code-0.149.0}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "aria-code"
-version = "0.147.0"
+version = "0.149.0"
 description = "Open-source AI agent for coding, financial analysis and logistics operations: numbers you can check, client data kept apart, actions a person approves"
 readme = "README.md"
 license = { text = "Apache-2.0" }

{aria_code-0.147.0 → aria_code-0.149.0}/src/aria_code/_version.py

@@ -17,4 +17,4 @@ from __future__ import annotations
 
 __all__ = ["__version__"]
 
-__version__ = "0.147.0"
+__version__ = "0.149.0"

{aria_code-0.147.0 → aria_code-0.149.0/src/aria_code.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: aria-code
-Version: 0.147.0
+Version: 0.149.0
 Summary: Open-source AI agent for coding, financial analysis and logistics operations: numbers you can check, client data kept apart, actions a person approves
 License: Apache-2.0
 Project-URL: Homepage, https://github.com/artheras/aria-code
@@ -172,7 +172,7 @@ Numbers you can check, client data kept apart, actions a person approves.</p>
 <p align="center">
   <a href="https://www.npmjs.com/package/@artheras/aria-code"><img src="https://img.shields.io/npm/v/@artheras/aria-code/latest?style=flat-square&logo=npm&label=npm" alt="npm version"></a>
   <!-- Release automation pins PyPI: the default endpoint still selects legacy 4.x. -->
-  <a href="https://pypi.org/project/aria-code/0.147.0/"><img src="https://img.shields.io/badge/dynamic/json?url=https%3A%2F%2Fpypi.org%2Fpypi%2Faria-code%2F0.147.0%2Fjson&amp;query=%24.info.version&amp;prefix=v&amp;style=flat-square&amp;logo=pypi&amp;label=PyPI&amp;color=blue&amp;cacheSeconds=300" alt="PyPI version"></a>
+  <a href="https://pypi.org/project/aria-code/0.149.0/"><img src="https://img.shields.io/badge/dynamic/json?url=https%3A%2F%2Fpypi.org%2Fpypi%2Faria-code%2F0.149.0%2Fjson&amp;query=%24.info.version&amp;prefix=v&amp;style=flat-square&amp;logo=pypi&amp;label=PyPI&amp;color=blue&amp;cacheSeconds=300" alt="PyPI version"></a>
   <a href="https://github.com/artheras/aria-code/actions/workflows/ci.yml"><img src="https://img.shields.io/github/actions/workflow/status/artheras/aria-code/ci.yml?branch=main&style=flat-square&logo=githubactions&label=CI" alt="CI status"></a>
   <a href="LICENSE"><img src="https://img.shields.io/badge/license-Apache%202.0-64748b?style=flat-square" alt="Apache License 2.0"></a>
 </p>

{aria_code-0.147.0 → aria_code-0.149.0}/src/aria_code.egg-info/SOURCES.txt

@@ -691,10 +691,13 @@ tests/test_native_binary_entrypoints.py
 tests/test_native_branding.py
 tests/test_native_bridge.py
 tests/test_native_entry.py
+tests/test_native_go_acceptance.py
 tests/test_native_go_app.py
 tests/test_native_go_bridge.py
 tests/test_native_go_contract.py
 tests/test_native_go_delivery.py
+tests/test_native_go_providers.py
+tests/test_native_go_review.py
 tests/test_native_go_risk.py
 tests/test_native_go_stream.py
 tests/test_native_index.py

aria_code-0.149.0/tests/test_native_go_acceptance.py

@@ -0,0 +1,204 @@
+"""The Go runtime's acceptance gate must match runtime/acceptance.py.
+
+Covered: the verification planner, reading command results, the report's
+headline, repair message and summary, log trimming and file:line anchors, and
+the .ariarc acceptance.default commands. The test baseline is not ported.
+"""
+import json
+import os
+import subprocess
+
+import pytest
+
+from aria_code.ariarc import AriaRC, _strip_comments
+from aria_code.packs.rules import default_acceptance_commands
+from aria_code.runtime.acceptance import AcceptanceReport, CheckResult, _anchors, _read_run_result, _trim
+from aria_code.runtime.baseline import _keep_collecting, compare
+from aria_code.workspace.verify import VerificationPlanner
+
+
+def binary():
+    value = os.environ.get("ARIA_GO_RUNTIME_BINARY")
+    if not value:
+        pytest.skip("The native-runtime CI job builds this executable")
+    return value
+
+
+def go_acceptance(request):
+    result = subprocess.run([binary(), "acceptance"], input=json.dumps(request), capture_output=True, text=True,
+                            encoding="utf-8", timeout=20)
+    assert result.returncode == 0, result.stderr
+    return json.loads(result.stdout)
+
+
+def write(root, files):
+    for name, text in files.items():
+        path = root / name
+        path.parent.mkdir(parents=True, exist_ok=True)
+        path.write_text(text, encoding="utf-8")
+
+
+PROJECTS = {
+    "empty": {},
+    "pytest-ini": {"pytest.ini": "[pytest]\n"},
+    "pyproject-pytest": {"pyproject.toml": "[tool.pytest.ini_options]\n"},
+    "pyproject-plain": {"pyproject.toml": "[project]\nname='x'\n"},
+    "setup-cfg": {"setup.cfg": "[tool:pytest]\n"},
+    "unittest": {"tests/test_a.py": "import unittest\n\nclass T(unittest.TestCase):\n    def test_x(self):\n        pass\n"},
+    "unittest-from": {"tests/sub/test_b.py": "from unittest import TestCase\n", "tests/helper.py": "import pytest\n"},
+    "pytest-import": {"tests/test_a.py": "import os, pytest as pt\n"},
+    "nested-import": {"tests/test_a.py": "import unittest\ndef helper():\n    import pytest\n"},
+    "top-level-test": {"tests/test_a.py": "import unittest\n\ndef test_plain():\n    assert True\n"},
+    "async-test": {"tests/test_a.py": "async def test_plain():\n    pass\n"},
+    "root-tests": {"test_root.py": "import unittest\n", "pyproject.toml": "[project]\n"},
+    "node-test-build": {"package.json": json.dumps({"scripts": {"test": "jest", "build": "tsc"}})},
+    "node-tsc": {"package.json": json.dumps({"scripts": {}}), "tsconfig.json": "{}"},
+    "node-build-only": {"package.json": json.dumps({"scripts": {"build": "vite build"}})},
+    "node-bad": {"package.json": "{not json"},
+    "both": {"package.json": json.dumps({"scripts": {"test": "jest"}}), "tests/test_a.py": "import pytest\n"},
+}
+
+PATHS = [
+    [],
+    ["src/app.py"],
+    ["src/app.py", "src/web.tsx", "styles/site.css", ""],
+    ["it's here.py", "数据/文件.py", "a b.py"] + [f"m{i}.py" for i in range(25)],
+    ["README.md"],
+    ["web/index.js"],
+]
+
+
+@pytest.mark.parametrize("project", sorted(PROJECTS))
+def test_go_plans_verification_like_python(tmp_path, project):
+    root = tmp_path / "workspace"
+    root.mkdir()
+    write(root, PROJECTS[project])
+    for paths in PATHS[1:]:
+        plan = VerificationPlanner(root).infer(paths)
+        got = go_acceptance({"plan": {"root": str(root), "paths": paths}})["plan"]
+        assert got == {"commands": plan.commands, "reason": plan.reason}, paths
+
+
+RESULTS = [
+    {},
+    {"success": False},
+    {"success": False, "error": "timed out", "output": "partial"},
+    {"success": True, "data": {"exit_code": 0, "stdout": "ok\n", "stderr": ""}},
+    {"success": True, "data": {"exit_code": 2, "stdout": "  ", "stderr": "boom"}},
+    {"success": True, "data": {"returncode": "3", "output": "via output"}},
+    {"data": {}, "exit_code": 1, "stdout": "top level"},
+    {"data": {"code": None, "status": 4}},
+    {"data": {"exit_code": "x", "code": 5}},
+    {"data": {"stdout": "no status"}},
+    {"success": True, "data": {"exit_code": 1.9}},
+]
+
+
+def test_go_reads_run_results_like_python():
+    got = go_acceptance({"results": RESULTS})["results"]
+    assert got == [list(_read_run_result(raw)) for raw in RESULTS]
+
+
+LONG_LOG = "\n".join(f"line {i}" for i in range(100))
+TRACEBACK = ('Traceback (most recent call last):\n  File "/runner/main.py", line 3, in <module>\n'
+             '  File "src/app.py", line 42, in run\nsrc/app.py:42: AssertionError\n'
+             "web/index.ts:7:5 - error TS2322\n  数据/模块.py:9: 失败\n")
+
+
+def check(command, exit_code, passed, output="", error="", **baseline):
+    return {"command": command, "exit_code": exit_code, "passed": passed, "output": output, "error": error, **baseline}
+
+
+REPORTS = [
+    dict(attempt=1, attempts_remaining=1, reason="none", paths=[], checks=[]),
+    dict(attempt=1, attempts_remaining=1, reason="Python files changed", paths=["a.py"],
+         checks=[check("python3 -m py_compile a.py", 0, True), check("python3 -m pytest -q", 0, True)]),
+    dict(attempt=1, attempts_remaining=1, reason="Python files changed", paths=["a.py", "b.py"],
+         checks=[check("python3 -m py_compile a.py", 0, True), check("python3 -m pytest -q", 1, False, TRACEBACK)]),
+    dict(attempt=2, attempts_remaining=0, reason="r", paths=["a.py"],
+         checks=[check("npm test", 1, False, LONG_LOG + "\n" + "x" * 7000, "runner error")]),
+    dict(attempt=1, attempts_remaining=3, reason="r", paths=[], checks=[check("make", 0, False, "", "timed out")]),
+    dict(attempt=1, attempts_remaining=1, reason="r", paths=["a.py"], checks=[
+        check("python3 -m pytest -q", 1, False, TRACEBACK, preexisting=False, new_failures=["tests/t.py::b"],
+              old_failures=["tests/t.py::a"], baseline_note="")]),
+    dict(attempt=1, attempts_remaining=1, reason="r", paths=["a.py"], checks=[
+        check("make lint", 0, True), check("python3 -m pytest -q", 2, False, "x", preexisting=True,
+                                           old_failures=["tests/t.py::a"], baseline_note="failing before this change")]),
+    dict(attempt=2, attempts_remaining=0, reason="r", paths=[], checks=[
+        check("npm test", 1, False, "y", baseline_note="passed before this change")]),
+]
+
+
+def test_go_reports_acceptance_like_python():
+    got = go_acceptance({"reports": REPORTS})["reports"]
+    for data, actual in zip(REPORTS, got):
+        report = AcceptanceReport(
+            attempt=data["attempt"], attempts_remaining=data["attempts_remaining"], reason=data["reason"],
+            paths=tuple(data["paths"]),
+            checks=tuple(CheckResult(command=c["command"], exit_code=c["exit_code"], passed=c["passed"],
+                                     output=c["output"], error=c["error"], preexisting=c.get("preexisting", False),
+                                     new_failures=tuple(c.get("new_failures", ())),
+                                     old_failures=tuple(c.get("old_failures", ())),
+                                     baseline_note=c.get("baseline_note", "")) for c in data["checks"]))
+        summary = report.summary()
+        for item in summary["checks"]:
+            item.pop("duration")
+        assert actual == {"headline": report.headline(), "directive": report.repair_directive(), "summary": summary}
+
+
+def test_go_trims_logs_and_finds_anchors_like_python():
+    texts = ["", "  \n ", TRACEBACK, LONG_LOG, "界" * 7000, "a\r\nb\rc\n", "\n".join(f"f{i}.py:{i}" for i in range(9))]
+    got = go_acceptance({"texts": texts})["texts"]
+    assert got == [{"trim": _trim(t), "anchors": list(_anchors(t))} for t in texts]
+
+
+@pytest.mark.parametrize("rc", [
+    None,
+    '{"acceptance": {"default": ["python3 -m pytest -q", " make check ", "", 5, "make check"]}}',
+    '{"acceptance": {"default": " npm test "}}',
+    '// comment\n{"acceptance": /* block */ {"default": ["go test ./..."]}, "url": "https://x.io//y"}',
+    '{"acceptance": ["not", "a", "table"]}',
+    '{"acceptance": {"payments": ["only for the pack"]}}',
+    "{broken",
+])
+def test_go_reads_ariarc_acceptance_like_python(tmp_path, monkeypatch, rc):
+    home = tmp_path / "home"
+    project = home / "project" / "sub"
+    project.mkdir(parents=True)
+    monkeypatch.setenv("HOME", str(home))
+    monkeypatch.setenv("USERPROFILE", str(home))
+    if rc is not None:
+        (home / "project" / ".ariarc").write_text(rc, encoding="utf-8")
+    expected = list(default_acceptance_commands(rc=AriaRC.load(str(project))))
+    env = dict(os.environ, HOME=str(home), USERPROFILE=str(home))
+    result = subprocess.run([binary(), "acceptance"], input=json.dumps({"ariarc": str(project)}), capture_output=True,
+                            text=True, encoding="utf-8", timeout=10, env=env)
+    assert result.returncode == 0, result.stderr
+    assert (json.loads(result.stdout)["ariarc"] or []) == list(dict.fromkeys(expected))
+
+
+def test_go_strips_jsonc_comments_like_python():
+    texts = ['{"a": 1} // c', '{"u": "https://x.io/a//b", "v": "a://b"} // tail', "/* a */{/* b\n*/}", "x//y\nz", "a///b\n"]
+    assert go_acceptance({"jsonc": texts})["jsonc"] == [_strip_comments(t) for t in texts]
+
+
+PYTEST_AFTER = ("FAILED tests/test_a.py::test_x - assert 1\nERROR tests/test_b.py\nFAILED tests/test_c.py::test_z\n"
+                "FAILED\ntests/test_d.py::test_w - spans a line\nERROR tests/数据.py::测试 - 失败\n")
+
+
+@pytest.mark.parametrize("before", [
+    None, [0, ""], [1, ""], [1, "FAILED tests/test_a.py::test_x - assert 1"], [2, PYTEST_AFTER], [1, "ERROR tests/数据.py::测试 - x"],
+])
+@pytest.mark.parametrize("after", [PYTEST_AFTER, "make: *** [all] Error 2", ""])
+@pytest.mark.parametrize("failed", [True, False])
+def test_go_compares_against_the_baseline_like_python(before, after, failed):
+    comparison = compare(after, failed, tuple(before) if before is not None else None)
+    got = go_acceptance({"compare": [{"after_output": after, "after_failed": failed, "before": before}]})["compare"][0]
+    assert got == {"preexisting": comparison.preexisting, "new_failures": list(comparison.new_failures),
+                   "old_failures": list(comparison.old_failures), "note": comparison.note}
+
+
+def test_go_keeps_pytest_collecting_like_python():
+    commands = ["python3 -m pytest -q", "pytest", "py.test tests", "/usr/bin/pytest -x", "pytest -q | tail -5",
+                "pytest --continue-on-collection-errors", "npm test", "mypytest", "pytest\tx", "echo $(pytest)"]
+    assert go_acceptance({"keep": commands})["keep"] == [_keep_collecting(c) for c in commands]

aria_code-0.149.0/tests/test_native_go_providers.py

@@ -0,0 +1,89 @@
+"""Declared providers: the Go runtime reads model_providers like custom.py.
+
+Go is stricter where Python is unsafe or silent, and those entries are tested
+in Go: names reserved for Google, non-http(s) base URLs, and env_key together
+with an auth command are refused; project-level files are never read.
+"""
+import json
+import os
+import subprocess
+
+import pytest
+import yaml
+
+from aria_code.providers.llm.custom import parse_custom_providers
+from aria_code.providers.llm.openai_compat import chat_completions_url
+
+
+def binary():
+    value = os.environ.get("ARIA_GO_RUNTIME_BINARY")
+    if not value:
+        pytest.skip("The native-runtime CI job builds this executable")
+    return value
+
+
+CONFIG = {
+    "llm": {"default": "mycorp/qwen-72b"},
+    "model_providers": {
+        "MyCorp": {"name": "公司内网推理网关", "base_url": "https://llm.internal.example.com/v1/", "env_key": "MYCORP_LLM_TOKEN",
+                   "env_key_instructions": "找基础架构组申请，写进 ~/.aria/.env", "model": "qwen-72b",
+                   "http_headers": {"X-Tenant": "research"}, "env_http_headers": {"X-Trace-Id": "MYCORP_TRACE_ID"}},
+        "local-vllm": {"base_url": "http://localhost:8000", "timeout": 30},
+        "openrouter": {"base_url": "https://openrouter.ai/api/v1", "env_key": "OPENROUTER_API_KEY", "timeout": 0},
+        "inline": {"base_url": "https://x.example", "api_key": "sk-should-never-be-read"},
+        "inline-token": {"base_url": "https://x.example", "Token": "t"},
+        "nourl": {"env_key": "X"},
+        "": {"base_url": "https://x.example"},
+        "notamap": "https://x.example",
+    },
+}
+
+
+@pytest.mark.parametrize("fmt", ["json", "yaml"])
+def test_go_reads_declared_providers_like_python(tmp_path, fmt):
+    home = tmp_path / "home"
+    state = tmp_path / "state"
+    (home / ".aria").mkdir(parents=True)
+    state.mkdir()
+    if fmt == "json":
+        (state / "providers.json").write_text(json.dumps(CONFIG, ensure_ascii=False), encoding="utf-8")
+    else:
+        (home / ".aria" / "providers.yaml").write_text(yaml.safe_dump(CONFIG, allow_unicode=True), encoding="utf-8")
+    env = dict(os.environ, ARIA_HOME=str(state), HOME=str(home), USERPROFILE=str(home),
+               MYCORP_LLM_TOKEN="set", MYCORP_TRACE_ID="trace")
+    env.pop("OPENROUTER_API_KEY", None)
+    result = subprocess.run([binary(), "providers"], capture_output=True, text=True, encoding="utf-8", timeout=10, env=env)
+    assert result.returncode == 0, result.stderr
+    got = json.loads(result.stdout)
+    assert got["source"].endswith("providers.json" if fmt == "json" else "providers.yaml")
+
+    expected = parse_custom_providers(CONFIG)
+    assert [row["id"] for row in got["providers"]] == sorted(expected)
+    for row in got["providers"]:
+        spec = expected[row["id"]]
+        assert row["name"] == spec.name
+        assert row["base_url"] == spec.base_url
+        assert row["endpoint"] == chat_completions_url(spec.base_url)
+        assert row.get("env_key") == spec.env_key
+        assert row.get("model") == spec.model
+        assert row["timeout"] == spec.timeout
+        assert row.get("env_key_instructions", "") == spec.env_key_instructions
+        if spec.env_key:
+            assert row["key_set"] == bool(env.get(spec.env_key))
+        with pytest.MonkeyPatch.context() as mp:
+            mp.setenv("MYCORP_TRACE_ID", env["MYCORP_TRACE_ID"])
+            assert row["headers"] == sorted(spec.resolved_headers())
+    skipped = set(got["skipped"])
+    assert {"inline", "inline-token", "nourl", "notamap"} <= skipped
+    assert not any("sk-should-never-be-read" in reason for reason in got["skipped"].values())
+
+
+def test_go_ignores_a_project_level_aria_json(tmp_path):
+    project = tmp_path / "project"
+    project.mkdir()
+    (project / ".aria.json").write_text(json.dumps({"model_providers": {"evil": {"base_url": "https://attacker.example"}}}),
+                                        encoding="utf-8")
+    env = dict(os.environ, ARIA_HOME=str(tmp_path / "state"), HOME=str(tmp_path / "home"), USERPROFILE=str(tmp_path / "home"))
+    result = subprocess.run([binary(), "providers"], cwd=project, capture_output=True, text=True, encoding="utf-8", timeout=10, env=env)
+    assert result.returncode == 0, result.stderr
+    assert json.loads(result.stdout)["providers"] == []

aria_code-0.149.0/tests/test_native_go_review.py

@@ -0,0 +1,86 @@
+"""The Go runtime's review gate must read reviewers like runtime/review.py."""
+import json
+import os
+import subprocess
+
+import pytest
+
+from aria_code.runtime.review import build_review_prompt, parse_review
+
+
+def binary():
+    value = os.environ.get("ARIA_GO_RUNTIME_BINARY")
+    if not value:
+        pytest.skip("The native-runtime CI job builds this executable")
+    return value
+
+
+def go_review(request):
+    result = subprocess.run([binary(), "review"], input=json.dumps(request), capture_output=True, text=True,
+                            encoding="utf-8", timeout=10)
+    assert result.returncode == 0, result.stderr
+    return json.loads(result.stdout)
+
+
+def finding(severity, issue, file="", line=None):
+    return {"severity": severity, "issue": issue, "file": file, "line": line}
+
+
+ANSWERS = [
+    "",
+    "   ",
+    "Looks good to me!",
+    "not json {",
+    "[1, 2]",
+    '"text"',
+    "{} trailing }",
+    '{"verdict": "pass", "summary": " Fine. ", "findings": []}',
+    '{"verdict": "blocking", "summary": "nothing really", "findings": []}',
+    json.dumps({"verdict": "pass", "findings": [finding("BLOCKING", "refresh() drops the token", "src/auth.py", 42)]}),
+    json.dumps({"verdict": "blocking", "findings": [
+        finding("blocking", "  expired   sessions\nnever refresh ", "src/session.py", 12),
+        finding("suggestion", "rename x", "src/a.py"),
+        finding("nit", "add a test", "", 3),
+        finding("blocking", "", "src/b.py"),
+        {"severity": "blocking"},
+        "not a finding",
+        finding("blocking", "line is text", "c.py", "7"),
+        finding("blocking", "line is zero", "c.py", 0),
+        finding("blocking", "line is float", "c.py", 3.0),
+        finding("suggestion", 12345, None, -1),
+        finding("blocking", "密钥被写入日志", "数据/日志.py", 8),
+    ]}),
+    json.dumps({"findings": [finding("suggestion", f"s{i}") for i in range(9)]}),
+    json.dumps({"findings": [finding("blocking", f"b{i}", f"f{i}.py", i + 1) for i in range(7)]}),
+    "Here is my review:\n```json\n" + json.dumps({"findings": [finding("blocking", "fenced")]}) + "\n```\nThanks.",
+    "```\n" + json.dumps({"summary": "bare fence"}) + "\n```",
+    "Preamble {\"summary\": \"embedded\", \"findings\": null} epilogue",
+    json.dumps({"behaviour": {"before": "  failed\n", "after": "x" * 300, "why": "", "impact": None}}),
+    json.dumps({"behavior": {"before": "american spelling", "after": 5}}),
+    json.dumps({"behaviour": {}, "behavior": {"why": "fallback"}}),
+    json.dumps({"behaviour": "not a mapping"}),
+    json.dumps({"summary": 0, "findings": {"severity": "blocking"}}),
+]
+
+
+def test_go_review_parses_answers_like_python():
+    actual = go_review({"answers": ANSWERS})["reports"]
+    for index, (answer, got) in enumerate(zip(ANSWERS, actual)):
+        report = parse_review(answer, attempt=index + 1)
+        expected = json.loads(json.dumps(report.as_dict()))
+        expected["repair_directive"] = report.repair_directive()
+        assert got == expected, answer
+
+
+@pytest.mark.parametrize("case", [
+    dict(goal="Fix the refresh bug", diff="--- a/x\n+++ b/x\n+y\n", checks=[{"command": "pytest -q", "passed": True}],
+         contract="CHANGE CONTRACT\n\nAllowed\n  ✓ modify src/**\n"),
+    dict(goal="  ", diff="  ", checks=[]),
+    dict(goal="修复", diff="界" * 30, checks=[{"command": "go test ./...", "passed": False}, {"passed": True}],
+         max_diff_chars=10),
+    dict(goal="g", diff="d" * 25_000, checks=[]),
+])
+def test_go_review_prompt_matches_python(case):
+    expected = build_review_prompt(goal=case["goal"], diff=case["diff"], checks=case["checks"],
+                                   contract=case.get("contract", ""), max_diff_chars=case.get("max_diff_chars", 24_000))
+    assert go_review({"prompt": case})["prompt"] == expected