From b3f3df63bfb344d890a22711457d8060efcf07ce Mon Sep 17 00:00:00 2001 From: huangruiteng <14976749+huangruiteng@users.noreply.github.com> Date: Wed, 16 Sep 2026 17:22:15 +0800 Subject: [PATCH] test(steward): point the machine-defaults guard at the current owner The guard asserts that the modules which resolve the steward's endpoint, model and effort are the ones that pass machine_defaults, but it still named `loopx/chat_server.py`. That module no longer holds such a call -- the manager readability projection moved to `loopx/chat_manager_context.py` -- so the test failed on every commit, including untouched ones, and a permanently red invariant check is exactly what hides a real regression later. The expected set now names the module that actually carries the call. The undocumented-call-site assertion, the call-site floor, and all other assertions are unchanged, so the check keeps its teeth. Verified: tests/test_manager_channel_binding.py 25 passed (was 1 failed), and a combined sweep of the steward surfaces touched today -- Lark topic connections and runtime, manager channel binding, manager context handoff, the Lark API contract, the manager guidance contract, the team-plan preview contract and the SSH evidence suite -- is 212 passed with no remaining failure. Signed-off-by: huangruiteng <14976749+huangruiteng@users.noreply.github.com> --- tests/test_manager_channel_binding.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/test_manager_channel_binding.py b/tests/test_manager_channel_binding.py index 171b777c49..1ec61900b6 100644 --- a/tests/test_manager_channel_binding.py +++ b/tests/test_manager_channel_binding.py @@ -836,8 +836,8 @@ def test_every_production_steward_caller_passes_the_machine_defaults() -> None: assert len(call_sites) >= 14, call_sites assert {path for path, _line, _name, _ok in call_sites} >= { "loopx/chat_manager.py", + "loopx/chat_manager_context.py", "loopx/chat_runtime.py", - "loopx/chat_server.py", "loopx/extensions/lark/goal_topic_runtime.py", } undocumented = [