From 6913de97de339f7a88ef66eba20e365d637c1100 Mon Sep 17 00:00:00 2001 From: xiaoan17 Date: Thu, 29 Oct 2026 16:01:13 +0800 Subject: [PATCH] fix(canary): lower the stale collaboration_mcp module ceiling `loopx/canary/module_metric_baseline.json` records a reviewed ceiling for every module that exceeds the coarse 2000-line default. `loopx/collaboration_mcp.py` was recorded at 2058 lines, but two later changes moved that decision into the typed owner and the module now measures 1924 lines. The entry still claims an exception the module no longer needs, and `tests/canary/test_maintainability_ratchet.py:: test_current_repository_debt_is_reviewed_without_line_count_pins` fails on main: ``` assert 1924 > 2000 ``` The entry is removed rather than lowered: with the module back under the default limit, the default limits are the honest bound and no reviewed exception is required. The module's other metrics are 0 for both the `Any` and the `dict[str, Any]` counts, so the default limits hold on every metric. Validation: tests/canary/ 89 passed on main 2a2323756, and examples/control_plane/control-plane-maintainability-ratchet-smoke.py is ok with unreviewed 0 and stale_exceptions 0. Signed-off-by: xiaoan17 --- loopx/canary/module_metric_baseline.json | 3 --- 1 file changed, 3 deletions(-) diff --git a/loopx/canary/module_metric_baseline.json b/loopx/canary/module_metric_baseline.json index 5b498482ac..ea1cb149cb 100644 --- a/loopx/canary/module_metric_baseline.json +++ b/loopx/canary/module_metric_baseline.json @@ -46,9 +46,6 @@ "loopx/chat_store.py": { "lines": 2066 }, - "loopx/collaboration_mcp.py": { - "lines": 2058 - }, "loopx/codex_cli_probe.py": { "any_count": 37, "dict_any_count": 0