"""``config.set`` bound by ``session_id`` alone persists into the session's profile (issue #85669).

Desktop app-global remote mode and the Ink TUI send ``config.set`` with ``session_id`` and no
``profile`` param. The session record carries ``profile_home``; a write that ignores it lands in
the launch profile's config.yaml while the focused profile silently keeps its old value.
"""

from __future__ import annotations

from pathlib import Path
from unittest.mock import patch

import pytest
import hermes_yaml as yaml

import tui_gateway.server as server


def _write_cfg(home: Path, busy: str, approvals: str) -> None:
    home.mkdir(parents=True, exist_ok=True)
    (home / "config.yaml").write_text(
        yaml.safe_dump({"display": {"busy_input_mode": busy}, "approvals": {"mode": approvals}}),
        encoding="utf-8",
    )


def _read(home: Path) -> dict:
    return yaml.safe_load((home / "config.yaml").read_text(encoding="utf-8")) or {}


@pytest.fixture
def homes(tmp_path, monkeypatch):
    launch, worker = tmp_path / "launch", tmp_path / "profiles" / "worker"
    _write_cfg(launch, "queue", "manual")
    _write_cfg(worker, "queue", "manual")
    monkeypatch.setenv("HERMES_HOME", str(launch))
    monkeypatch.setattr(server, "_hermes_home", launch)
    monkeypatch.setattr(server, "_cfg_cache", None)
    monkeypatch.setattr(server, "_cfg_sig", None)
    monkeypatch.setattr(server, "_cfg_path", None)
    monkeypatch.setattr(server, "_emit", lambda *a, **k: None)
    return launch, worker


@pytest.mark.parametrize(
    ("key", "value", "section", "field"),
    [("busy", "steer", "display", "busy_input_mode"), ("approval_mode", "off", "approvals", "mode")],
)
def test_session_bound_config_set_writes_the_session_profile(homes, key, value, section, field):
    launch, worker = homes
    session = {"agent": None, "profile_home": str(worker), "session_key": "worker-session"}
    with patch.dict(server._sessions, {"s-worker": session}, clear=False):
        resp = server._methods["config.set"]("rid", {"session_id": "s-worker", "key": key, "value": value})
    assert resp["result"]["value"] == value
    assert _read(worker)[section][field] == value
    assert _read(launch)[section][field] != value


def test_unbound_config_set_still_writes_the_launch_profile(homes):
    launch, worker = homes
    resp = server._methods["config.set"]("rid", {"key": "busy", "value": "steer"})
    assert resp["result"]["value"] == "steer"
    assert _read(launch)["display"]["busy_input_mode"] == "steer"
    assert _read(worker)["display"]["busy_input_mode"] == "queue"