From 0ef087f870ede163dda9dffaf5de96616d18a66c Mon Sep 17 00:00:00 2001 From: song Date: Wed, 16 Sep 2026 14:48:25 +0800 Subject: [PATCH] fix(tests): drop stale turn-module patch names from inspect-journal guard refactor(turn): derive every Turn decision from one shared owner (#4496) moved build_lark_operator_inbox_urgency_projector, collect_status, and scheduler_execution_context_for_turn out of loopx.cli_commands.turn's top-level namespace, but this test still patches all three there. The patch loop hits the first missing name and raises AttributeError before running the CLI, breaking this test on every PR built on current main. turn_inspection.handle_turn_journal_inspection never imports these three names, so the guarantee this test checks -- inspect-journal never reaches a live or write path -- already holds by construction; the stale patch targets asserted nothing real. Signed-off-by: song --- tests/test_loopx_turn_journal_inspection.py | 3 --- 1 file changed, 3 deletions(-) diff --git a/tests/test_loopx_turn_journal_inspection.py b/tests/test_loopx_turn_journal_inspection.py index 61b4ab10c8..9e434025ed 100644 --- a/tests/test_loopx_turn_journal_inspection.py +++ b/tests/test_loopx_turn_journal_inspection.py @@ -293,9 +293,6 @@ def unexpected_call(*args: object, **kwargs: object) -> None: raise AssertionError("inspect-journal reached a live or write path") for name in ( - "build_lark_operator_inbox_urgency_projector", - "collect_status", - "scheduler_execution_context_for_turn", "build_live_quota_should_run_decision", "build_loopx_turn_plan", "run_codex_cli_host",