from __future__ import annotations

import shutil
from dataclasses import dataclass

from evals import run_eval
from evals.patchers.base import ContextCompactionResult, ContextRequestResult, PatchResult


@dataclass(frozen=True)
class _NoDiffPatcher:
    name: str = "no_diff"

    def propose_patch(self, *, prompt: str, context_dir, cwd) -> PatchResult:
        return PatchResult(diff=None, meta={"reason": "no_diff"})


@dataclass(frozen=True)
class _RequestFailedNoDiffPatcher:
    name: str = "request_failed_no_diff"

    def propose_patch(self, *, prompt: str, context_dir, cwd) -> PatchResult:
        del prompt, context_dir, cwd
        return PatchResult(
            diff=None,
            meta={"error": "request_failed", "exc": "TimeoutError: timed out patcher"},
        )


@dataclass(frozen=True)
class _ContextLimitNoDiffPatcher:
    name: str = "context_limit_no_diff"

    def propose_patch(self, *, prompt: str, context_dir, cwd) -> PatchResult:
        del prompt, context_dir, cwd
        return PatchResult(
            diff=None,
            meta={
                "error": "http_error",
                "status": 400,
                "body": "Requested prompt is greater than the context length.",
            },
        )


@dataclass(frozen=True)
class _RetryableHttpNoDiffPatcher:
    name: str = "retryable_http_no_diff"

    def propose_patch(self, *, prompt: str, context_dir, cwd) -> PatchResult:
        del prompt, context_dir, cwd
        return PatchResult(
            diff=None,
            meta={
                "error": "http_error",
                "status": 503,
                "body": "backend temporarily unavailable",
            },
        )


@dataclass(frozen=True)
class _BadDiffPatcher:
    name: str = "bad_diff"

    def propose_patch(self, *, prompt: str, context_dir, cwd) -> PatchResult:
        diff = (
            "--- a/buggy.py\n"
            "+++ b/buggy.py\n"
            "@@ -99,1 +99,1 @@\n"
            "-missing_line = 1\n"
            "+missing_line = 2\n"
        )
        return PatchResult(diff=diff, meta={"reason": "bad_diff"})


@dataclass(frozen=True)
class _MalformedStageAPatcher:
    name: str = "malformed_stage_a"

    def request_context(self, *, prompt: str, context_dir, cwd) -> ContextRequestResult:
        del prompt
        del context_dir
        del cwd
        return ContextRequestResult(
            request_json=None,
            raw_response="not-json",
            meta={"error": "bad_json"},
        )

    def propose_patch(self, *, prompt: str, context_dir, cwd) -> PatchResult:
        return PatchResult(diff=None, meta={"reason": "no_diff"})


@dataclass(frozen=True)
class _TestFileDiffPatcher:
    name: str = "test_file_diff"

    def propose_patch(self, *, prompt: str, context_dir, cwd) -> PatchResult:
        diff = (
            "--- a/test_buggy.py\n"
            "+++ b/test_buggy.py\n"
            "@@ -3,3 +3,3 @@\n"
            " \n"
            " def test_f() -> None:\n"
            "-    assert f() == 2\n"
            "+    assert f() == 1\n"
        )
        return PatchResult(diff=diff, meta={"reason": "test_file_edit"})


@dataclass
class _CompactingStageAPatcher:
    name: str = "compacting_stage_a"
    compact_calls: int = 0
    last_patch_prompt: str = ""

    def request_context(self, *, prompt: str, context_dir, cwd) -> ContextRequestResult:
        del prompt, context_dir, cwd
        return ContextRequestResult(
            request_json={
                "need_more_context": True,
                "need_snapshot": True,
                "need_compact_context": True,
                "snapshot_justification": "need frame locals to compare actual vs expected values",
                "compact_context_reason": "snapshot has too much noise",
                "requests": [{"type": "file", "path": "buggy.py"}],
            },
            raw_response='{"need_more_context":true}',
            meta={},
        )

    def compact_context(
        self,
        *,
        prompt: str,
        context_text: str,
        context_dir,
        cwd,
        target_chars: int,
    ) -> ContextCompactionResult:
        del context_dir, cwd
        self.compact_calls += 1
        assert "snapshot has too much noise" in prompt
        assert "Pytest output" in context_text
        compacted = "Root cause hints: buggy.f returns 1 while test expects 2."
        return ContextCompactionResult(
            compacted_text=compacted[:target_chars],
            raw_response='{"compacted_context":"ok"}',
            meta={"llm_call_count": 1, "usage": {"prompt_tokens": 5, "completion_tokens": 3}},
        )

    def propose_patch(self, *, prompt: str, context_dir, cwd) -> PatchResult:
        del context_dir, cwd
        self.last_patch_prompt = prompt
        return PatchResult(diff=None, meta={"reason": "no_diff"})


@dataclass
class _ContextRecoveryDiffPatcher:
    name: str = "context_recovery_diff"
    calls: int = 0

    def propose_patch(self, *, prompt: str, context_dir, cwd) -> PatchResult:
        del prompt, context_dir, cwd
        self.calls += 1
        if self.calls == 1:
            return PatchResult(
                diff=None,
                meta={
                    "error": "http_error",
                    "status": 400,
                    "body": "Requested prompt is greater than the context length.",
                },
            )
        return PatchResult(
            diff=(
                "diff --git a/buggy.py b/buggy.py\n"
                "--- a/buggy.py\n"
                "+++ b/buggy.py\n"
                "@@ -1,2 +1,2 @@\n"
                " def f() -> int:\n"
                "-    return 1\n"
                "+    return 2\n"
            ),
            meta={"reason": "recovered_diff"},
        )


@dataclass
class _OpenAINoDiffRecoveryPatcher:
    name: str = "openai"
    calls: int = 0

    def propose_patch(self, *, prompt: str, context_dir, cwd) -> PatchResult:
        del prompt, context_dir, cwd
        self.calls += 1
        if self.calls == 1:
            return PatchResult(
                diff=None,
                meta={
                    "error": "no_json_payload",
                    "reason": "invalid_response",
                },
            )
        return PatchResult(
            diff=(
                "diff --git a/buggy.py b/buggy.py\n"
                "--- a/buggy.py\n"
                "+++ b/buggy.py\n"
                "@@ -1,2 +1,2 @@\n"
                " def f() -> int:\n"
                "-    return 1\n"
                "+    return 2\n"
            ),
            meta={"reason": "recovered_diff"},
        )


@dataclass(frozen=True)
class _MarkerOnlyDiffPatcher:
    name: str = "marker_only_diff"

    def propose_patch(self, *, prompt: str, context_dir, cwd) -> PatchResult:
        del prompt, context_dir, cwd
        return PatchResult(
            diff=(
                "diff --git a/buggy.py b/buggy.py\n"
                "--- a/buggy.py\n"
                "+++ b/buggy.py\n"
                "@@ -1,2 +1,3 @@\n"
                " def f() -> int:\n"
                "     return 1\n"
                "+# llmdebug-force-attempt-1\n"
            ),
            meta={"reason": "marker_only"},
        )


def _make_case(tmp_path) -> run_eval.CaseSpec:
    case_dir = tmp_path / "case_src"
    case_dir.mkdir(parents=True, exist_ok=True)
    (case_dir / "buggy.py").write_text("def f() -> int:\n    return 1\n", encoding="utf-8")
    (case_dir / "test_buggy.py").write_text(
        "from buggy import f\n\n\ndef test_f() -> None:\n    assert f() == 2\n",
        encoding="utf-8",
    )
    return run_eval.CaseSpec(
        case_id="force_patch_case",
        path=case_dir,
        category="traceback_controls",
        difficulty="easy",
        snapshot_dependency="none",
        expected_exception="AssertionError",
        tags=[],
        python_args=["-m", "pytest", "-q"],
        timeout_sec=30,
        description="force-patch policy test",
    )


def test_runner_force_patch_attempt_applies_when_no_diff(tmp_path, monkeypatch) -> None:
    case = _make_case(tmp_path)
    workdir = tmp_path / "workdir"
    context_dir = tmp_path / "context"
    shutil.copytree(case.path, workdir)

    def _fake_run_pytest(cwd, python_args, condition, timeout_sec, output_format):
        return run_eval.RunResult(
            returncode=1,
            duration_sec=0.01,
            stdout="E   AssertionError: expected 2 got 1\n",
            stderr="",
        )

    monkeypatch.setattr(run_eval, "run_pytest", _fake_run_pytest)
    monkeypatch.setattr(run_eval, "apply_diff", lambda cwd, diff: (True, ""))

    records = run_eval.run_one_condition(
        case=case,
        condition=run_eval.Condition.TRACEBACK_ONLY,
        workdir=workdir,
        context_dir_base=context_dir,
        patcher=_NoDiffPatcher(),
        k=1,
        output_format="json_compact",
        run_id="run",
        include_files_context=False,
        force_patch_attempt=True,
    )

    assert len(records) == 1
    record = records[0]
    assert record["note"] == "forced_noop_patch"
    assert record["patch_meta"]["runner_forced_patch_attempt"] is True
    assert record["patch_meta"]["runner_forced_patch_target"] == "buggy.py"
    assert record["patch_meta"]["runner_forced_patch_marker_only"] is True
    assert record["patch_meta"]["runner_forced_patch_disabled_after_noop"] is True


def test_runner_no_force_patch_attempt_preserves_no_patch_proposed(tmp_path, monkeypatch) -> None:
    case = _make_case(tmp_path)
    workdir = tmp_path / "workdir_no_force"
    context_dir = tmp_path / "context_no_force"
    shutil.copytree(case.path, workdir)

    def _fake_run_pytest(cwd, python_args, condition, timeout_sec, output_format):
        return run_eval.RunResult(
            returncode=1,
            duration_sec=0.01,
            stdout="E   AssertionError: expected 2 got 1\n",
            stderr="",
        )

    apply_called = {"count": 0}

    def _fake_apply_diff(cwd, diff):
        apply_called["count"] += 1
        return True, ""

    monkeypatch.setattr(run_eval, "run_pytest", _fake_run_pytest)
    monkeypatch.setattr(run_eval, "apply_diff", _fake_apply_diff)

    records = run_eval.run_one_condition(
        case=case,
        condition=run_eval.Condition.TRACEBACK_ONLY,
        workdir=workdir,
        context_dir_base=context_dir,
        patcher=_NoDiffPatcher(),
        k=1,
        output_format="json_compact",
        run_id="run",
        include_files_context=False,
        force_patch_attempt=False,
    )

    assert len(records) == 1
    record = records[0]
    assert record["note"] == "no_patch_proposed"
    assert apply_called["count"] == 0


def test_runner_force_patch_attempt_skips_on_request_failure(tmp_path, monkeypatch) -> None:
    case = _make_case(tmp_path)
    workdir = tmp_path / "workdir_request_failed_skip"
    context_dir = tmp_path / "context_request_failed_skip"
    shutil.copytree(case.path, workdir)

    monkeypatch.setattr(
        run_eval,
        "run_pytest",
        lambda cwd, python_args, condition, timeout_sec, output_format: run_eval.RunResult(
            returncode=1,
            duration_sec=0.01,
            stdout="E   AssertionError: expected 2 got 1\n",
            stderr="",
        ),
    )
    apply_called = {"count": 0}

    def _fake_apply_diff(cwd, diff):
        del cwd, diff
        apply_called["count"] += 1
        return True, ""

    monkeypatch.setattr(run_eval, "apply_diff", _fake_apply_diff)

    records = run_eval.run_one_condition(
        case=case,
        condition=run_eval.Condition.TRACEBACK_ONLY,
        workdir=workdir,
        context_dir_base=context_dir,
        patcher=_RequestFailedNoDiffPatcher(),
        k=1,
        output_format="json_compact",
        run_id="run-request-failed-skip",
        include_files_context=False,
        force_patch_attempt=True,
    )

    assert len(records) == 1
    record = records[0]
    assert record["note"] == "patcher_request_failed"
    assert apply_called["count"] == 0
    assert record["patch_meta"]["patcher_request_failed"] is True
    assert record["patch_meta"]["patcher_request_failure_kind"] == "transport"
    assert record["patch_meta"]["runner_forced_patch_skipped"] is True
    assert record["patch_meta"]["runner_forced_patch_skip_reason"] == "transport_or_context_failure"
    assert "runner_forced_patch_attempt" not in record["patch_meta"]


def test_runner_force_patch_attempt_skips_on_context_limit_failure(tmp_path, monkeypatch) -> None:
    case = _make_case(tmp_path)
    workdir = tmp_path / "workdir_context_limit_skip"
    context_dir = tmp_path / "context_context_limit_skip"
    shutil.copytree(case.path, workdir)

    monkeypatch.setattr(
        run_eval,
        "run_pytest",
        lambda cwd, python_args, condition, timeout_sec, output_format: run_eval.RunResult(
            returncode=1,
            duration_sec=0.01,
            stdout="E   AssertionError: expected 2 got 1\n",
            stderr="",
        ),
    )
    apply_called = {"count": 0}

    def _fake_apply_diff(cwd, diff):
        del cwd, diff
        apply_called["count"] += 1
        return True, ""

    monkeypatch.setattr(run_eval, "apply_diff", _fake_apply_diff)

    records = run_eval.run_one_condition(
        case=case,
        condition=run_eval.Condition.TRACEBACK_ONLY,
        workdir=workdir,
        context_dir_base=context_dir,
        patcher=_ContextLimitNoDiffPatcher(),
        k=1,
        output_format="json_compact",
        run_id="run-context-limit-skip",
        include_files_context=False,
        force_patch_attempt=True,
    )

    assert len(records) == 1
    record = records[0]
    assert record["note"] == "patcher_request_failed"
    assert apply_called["count"] == 0
    assert record["patch_meta"]["patcher_request_failed"] is True
    assert record["patch_meta"]["patcher_request_failure_kind"] == "context_limit"
    assert record["patch_meta"]["runner_forced_patch_skipped"] is True
    assert record["patch_meta"]["runner_forced_patch_skip_reason"] == "transport_or_context_failure"
    assert "runner_forced_patch_attempt" not in record["patch_meta"]


def test_runner_force_patch_attempt_skips_on_retryable_http_transport_failure(
    tmp_path, monkeypatch
) -> None:
    case = _make_case(tmp_path)
    workdir = tmp_path / "workdir_retryable_http_skip"
    context_dir = tmp_path / "context_retryable_http_skip"
    shutil.copytree(case.path, workdir)

    monkeypatch.setattr(
        run_eval,
        "run_pytest",
        lambda cwd, python_args, condition, timeout_sec, output_format: run_eval.RunResult(
            returncode=1,
            duration_sec=0.01,
            stdout="E   AssertionError: expected 2 got 1\n",
            stderr="",
        ),
    )
    apply_called = {"count": 0}

    def _fake_apply_diff(cwd, diff):
        del cwd, diff
        apply_called["count"] += 1
        return True, ""

    monkeypatch.setattr(run_eval, "apply_diff", _fake_apply_diff)

    records = run_eval.run_one_condition(
        case=case,
        condition=run_eval.Condition.TRACEBACK_ONLY,
        workdir=workdir,
        context_dir_base=context_dir,
        patcher=_RetryableHttpNoDiffPatcher(),
        k=1,
        output_format="json_compact",
        run_id="run-retryable-http-skip",
        include_files_context=False,
        force_patch_attempt=True,
    )

    assert len(records) == 1
    record = records[0]
    assert record["note"] == "patcher_request_failed"
    assert apply_called["count"] == 0
    assert record["patch_meta"]["patcher_request_failed"] is True
    assert record["patch_meta"]["patcher_request_failure_kind"] == "transport"
    assert record["patch_meta"]["runner_forced_patch_skipped"] is True
    assert record["patch_meta"]["runner_forced_patch_skip_reason"] == "transport_or_context_failure"
    assert "runner_forced_patch_attempt" not in record["patch_meta"]


def test_runner_force_patch_recovers_after_apply_failure(tmp_path, monkeypatch) -> None:
    case = _make_case(tmp_path)
    workdir = tmp_path / "workdir_recovery"
    context_dir = tmp_path / "context_recovery"
    shutil.copytree(case.path, workdir)

    def _fake_run_pytest(cwd, python_args, condition, timeout_sec, output_format):
        return run_eval.RunResult(
            returncode=1,
            duration_sec=0.01,
            stdout="E   AssertionError: expected 2 got 1\n",
            stderr="",
        )

    apply_attempts = {"count": 0}

    def _fake_apply_diff(cwd, diff):
        apply_attempts["count"] += 1
        if "llmdebug-force-attempt-1" in diff:
            return True, ""
        return False, "simulated hunk failure"

    monkeypatch.setattr(run_eval, "run_pytest", _fake_run_pytest)
    monkeypatch.setattr(run_eval, "apply_diff", _fake_apply_diff)

    records = run_eval.run_one_condition(
        case=case,
        condition=run_eval.Condition.TRACEBACK_ONLY,
        workdir=workdir,
        context_dir_base=context_dir,
        patcher=_BadDiffPatcher(),
        k=1,
        output_format="json_compact",
        run_id="run",
        include_files_context=False,
        force_patch_attempt=True,
    )

    assert len(records) == 1
    record = records[0]
    assert record["note"] == "tests_still_failing"
    assert record["patch_meta"]["runner_forced_patch_recovery"] is True
    assert record["patch_meta"]["runner_forced_patch_recovery_target"] == "buggy.py"
    assert apply_attempts["count"] == 2

    recovery_diff = (context_dir / "attempt_01" / "patch_recovery.diff").read_text(encoding="utf-8")
    assert "llmdebug-force-attempt-1" in recovery_diff


def test_runner_no_force_patch_keeps_apply_failure(tmp_path, monkeypatch) -> None:
    case = _make_case(tmp_path)
    workdir = tmp_path / "workdir_recovery_off"
    context_dir = tmp_path / "context_recovery_off"
    shutil.copytree(case.path, workdir)

    def _fake_run_pytest(cwd, python_args, condition, timeout_sec, output_format):
        return run_eval.RunResult(
            returncode=1,
            duration_sec=0.01,
            stdout="E   AssertionError: expected 2 got 1\n",
            stderr="",
        )

    monkeypatch.setattr(run_eval, "run_pytest", _fake_run_pytest)
    monkeypatch.setattr(run_eval, "apply_diff", lambda cwd, diff: (False, "simulated hunk failure"))

    records = run_eval.run_one_condition(
        case=case,
        condition=run_eval.Condition.TRACEBACK_ONLY,
        workdir=workdir,
        context_dir_base=context_dir,
        patcher=_BadDiffPatcher(),
        k=1,
        output_format="json_compact",
        run_id="run",
        include_files_context=False,
        force_patch_attempt=False,
    )

    assert len(records) == 1
    record = records[0]
    assert record["note"] == "patch_apply_failed"
    assert "runner_forced_patch_recovery" not in record["patch_meta"]


def test_staged_context_malformed_stage_a_uses_fallback_bundle(tmp_path, monkeypatch) -> None:
    case = _make_case(tmp_path)
    workdir = tmp_path / "workdir_stage_a_fallback"
    context_dir = tmp_path / "context_stage_a_fallback"
    shutil.copytree(case.path, workdir)

    def _fake_run_pytest(cwd, python_args, condition, timeout_sec, output_format):
        return run_eval.RunResult(
            returncode=1,
            duration_sec=0.01,
            stdout="E   AssertionError: expected 2 got 1\n",
            stderr="",
        )

    monkeypatch.setattr(run_eval, "run_pytest", _fake_run_pytest)
    monkeypatch.setattr(run_eval, "apply_diff", lambda cwd, diff: (True, ""))

    records = run_eval.run_one_condition(
        case=case,
        condition=run_eval.Condition.TRACEBACK_ONLY,
        workdir=workdir,
        context_dir_base=context_dir,
        patcher=_MalformedStageAPatcher(),
        k=1,
        output_format="json_compact",
        run_id="run-stage-a-fallback",
        include_files_context=False,
        force_patch_attempt=False,
        context_protocol="staged",
    )

    assert len(records) == 1
    record = records[0]
    assert record["attempt_metadata"]["stage_a_request_success"] is False
    assert record["attempt_metadata"]["stage_a_requests_count"] >= 1
    assert record["patch_meta"]["stage_a_used_fallback"] is True


def test_patch_metrics_detect_test_file_edits(tmp_path, monkeypatch) -> None:
    case = _make_case(tmp_path)
    workdir = tmp_path / "workdir_test_metric"
    context_dir = tmp_path / "context_test_metric"
    shutil.copytree(case.path, workdir)

    monkeypatch.setattr(
        run_eval,
        "run_pytest",
        lambda *args, **kwargs: run_eval.RunResult(
            returncode=1,
            duration_sec=0.01,
            stdout="E   AssertionError: expected 2 got 1\n",
            stderr="",
        ),
    )
    monkeypatch.setattr(run_eval, "apply_diff", lambda cwd, diff: (True, ""))

    records = run_eval.run_one_condition(
        case=case,
        condition=run_eval.Condition.TRACEBACK_ONLY,
        workdir=workdir,
        context_dir_base=context_dir,
        patcher=_TestFileDiffPatcher(),
        k=1,
        output_format="json_compact",
        run_id="run-test-metric",
        include_files_context=False,
        force_patch_attempt=False,
    )

    assert len(records) == 1
    record = records[0]
    patch_meta = record["patch_meta"]
    assert patch_meta["patch_touches_tests"] is True
    assert int(patch_meta["patch_test_file_count"]) == 1
    assert int(patch_meta["patch_file_count"]) >= 1
    assert "test_buggy.py" in patch_meta["patch_test_file_paths"]


def test_context_compaction_stage_a_request_replaces_raw_sections(tmp_path, monkeypatch) -> None:
    case = _make_case(tmp_path)
    workdir = tmp_path / "workdir_context_compaction"
    context_dir = tmp_path / "context_context_compaction"
    shutil.copytree(case.path, workdir)
    patcher = _CompactingStageAPatcher()

    monkeypatch.setattr(
        run_eval,
        "run_pytest",
        lambda *args, **kwargs: run_eval.RunResult(
            returncode=1,
            duration_sec=0.01,
            stdout="tests/test_buggy.py::test_f\nE   AssertionError: expected 2 got 1\n",
            stderr="",
        ),
    )
    monkeypatch.setattr(run_eval, "apply_diff", lambda cwd, diff: (True, ""))
    monkeypatch.setattr(
        run_eval,
        "read_latest_snapshot",
        lambda *_args, **_kwargs: (
            {
                "exception": {"type": "AssertionError", "message": "expected 2 got 1"},
                "frames": [
                    {
                        "file_rel": "buggy.py",
                        "line": 2,
                        "function": "f",
                        "code": "return 1",
                        "locals": {},
                        "locals_meta": {},
                    }
                ],
            },
            None,
        ),
    )

    records = run_eval.run_one_condition(
        case=case,
        condition=run_eval.Condition.ADAPTIVE_GATED,
        workdir=workdir,
        context_dir_base=context_dir,
        patcher=patcher,
        k=1,
        output_format="json_compact",
        run_id="run-context-compaction",
        include_files_context=False,
        force_patch_attempt=False,
        context_protocol="staged",
        context_compaction_mode="auto_overflow",
        context_compaction_scope="snapshot,pytest,files",
        adaptive_gate_policy="aggressive",
    )

    assert len(records) == 1
    assert patcher.compact_calls == 1
    assert "Compacted context summary:" in patcher.last_patch_prompt
    assert "=== llmdebug Debug Snapshot ===" not in patcher.last_patch_prompt
    record = records[0]
    assert record["patch_meta"]["context_compaction_invoked"] is True
    assert record["patch_meta"]["context_compaction_success"] is True
    assert record["patch_meta"]["context_compaction_trigger"] == "stage_a_requested"
    assert "snapshot" in record["patch_meta"]["context_compaction_replaced_sections"]


def test_context_recovery_retries_once_with_compact_budget(tmp_path, monkeypatch) -> None:
    case = _make_case(tmp_path)
    workdir = tmp_path / "workdir_context_recovery_retry"
    context_dir = tmp_path / "context_context_recovery_retry"
    shutil.copytree(case.path, workdir)
    patcher = _ContextRecoveryDiffPatcher()

    monkeypatch.setattr(
        run_eval,
        "run_pytest",
        lambda *args, **kwargs: run_eval.RunResult(
            returncode=1,
            duration_sec=0.01,
            stdout="E   AssertionError: expected 2 got 1\n",
            stderr="",
        ),
    )
    monkeypatch.setattr(run_eval, "apply_diff", lambda cwd, diff: (True, ""))

    records = run_eval.run_one_condition(
        case=case,
        condition=run_eval.Condition.TRACEBACK_ONLY,
        workdir=workdir,
        context_dir_base=context_dir,
        patcher=patcher,
        k=1,
        output_format="json_compact",
        run_id="run-context-recovery-retry",
        include_files_context=False,
        force_patch_attempt=False,
    )

    assert len(records) == 1
    assert patcher.calls == 2
    record = records[0]
    assert record["note"] == "tests_still_failing"
    assert record["patch_meta"]["context_recovery_invoked"] is True
    assert record["patch_meta"]["context_recovery_retry_success"] is True


def test_openai_no_diff_recovery_retries_once_before_forced_patch(tmp_path, monkeypatch) -> None:
    case = _make_case(tmp_path)
    workdir = tmp_path / "workdir_openai_no_diff_recovery"
    context_dir = tmp_path / "context_openai_no_diff_recovery"
    shutil.copytree(case.path, workdir)
    patcher = _OpenAINoDiffRecoveryPatcher()

    monkeypatch.setattr(
        run_eval,
        "run_pytest",
        lambda *args, **kwargs: run_eval.RunResult(
            returncode=1,
            duration_sec=0.01,
            stdout="E   AssertionError: expected 2 got 1\n",
            stderr="",
        ),
    )
    monkeypatch.setattr(run_eval, "apply_diff", lambda cwd, diff: (True, ""))

    records = run_eval.run_one_condition(
        case=case,
        condition=run_eval.Condition.TRACEBACK_ONLY,
        workdir=workdir,
        context_dir_base=context_dir,
        patcher=patcher,
        k=1,
        output_format="json_compact",
        run_id="run-openai-no-diff-recovery",
        include_files_context=False,
        force_patch_attempt=False,
    )

    assert len(records) == 1
    assert patcher.calls == 2
    record = records[0]
    assert record["note"] == "tests_still_failing"
    assert record["patch_meta"]["no_diff_recovery_invoked"] is True
    assert record["patch_meta"]["no_diff_recovery_retry_success"] is True
    assert "runner_forced_patch_attempt" not in record["patch_meta"]


def test_marker_only_diff_is_treated_as_forced_noop(tmp_path, monkeypatch) -> None:
    case = _make_case(tmp_path)
    workdir = tmp_path / "workdir_marker_noop"
    context_dir = tmp_path / "context_marker_noop"
    shutil.copytree(case.path, workdir)

    monkeypatch.setattr(
        run_eval,
        "run_pytest",
        lambda *args, **kwargs: run_eval.RunResult(
            returncode=1,
            duration_sec=0.01,
            stdout="E   AssertionError: expected 2 got 1\n",
            stderr="",
        ),
    )
    monkeypatch.setattr(run_eval, "apply_diff", lambda cwd, diff: (True, ""))

    records = run_eval.run_one_condition(
        case=case,
        condition=run_eval.Condition.TRACEBACK_ONLY,
        workdir=workdir,
        context_dir_base=context_dir,
        patcher=_MarkerOnlyDiffPatcher(),
        k=1,
        output_format="json_compact",
        run_id="run-marker-noop",
        include_files_context=False,
        force_patch_attempt=True,
    )

    assert len(records) == 1
    record = records[0]
    assert record["note"] == "forced_noop_patch"
    assert record["patch_meta"]["runner_forced_patch_marker_only"] is True
    assert record["patch_meta"]["runner_forced_patch_disabled_after_noop"] is True
