fix: update line-number allowlists for tools_kitchen.py additions

The gate_infrastructure_ready implementation added 29 lines to
tools_kitchen.py, shifting the atomic_write+json.dumps sites from
lines 918/978 to 947/1007. Update the _LEGACY_JSON_WRITES allowlist
to match. Bump tools_kitchen.py line-limit exemption from 1200 to
1250 lines and improve the close-then-reopen test to drain each
task independently.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: Trecek

tests/arch/test_subpackage_isolation.py

@@ -962,14 +962,16 @@ def test_data_directories_are_not_python_packages() -> None:
         "co-located with the execution engine that calls them",
     ),
     "tools_kitchen.py": (
-        1200,
+        1250,
         "REQ-CNST-010-E7: kitchen tool handlers — open_kitchen and lock_ingredients require "
         "inline validation helpers (_check_override_keys, _build_ingredient_key_suggestions) "
         "for ingredient key validation; splitting would cross import-layer boundaries; "
         "backend capability promotion delegated to _promote_capability_keys in _auto_overrides; "
         "fail-closed validity gate on both deferred-recall and normal paths adds structural "
         "error propagation from LoadRecipeResult; get_recipe validity guard adds 9 lines; "
-        "dispatch-feasibility preflight wiring on both paths with gate-close on failure",
+        "dispatch-feasibility preflight wiring on both paths with gate-close on failure; "
+        "gate_infrastructure_ready guard restructuring adds 22 lines for handler-skip path, "
+        "quota_refresh_loop deferred start, and all four gate.disable() rollback resets",
     ),
     "tools_execution.py": (
         1130,

tests/infra/test_schema_version_convention.py

@@ -122,8 +122,8 @@ def _is_yaml_dump(node: ast.expr) -> bool:
     ("src/autoskillit/server/tools/tools_kitchen.py", 171),
     ("src/autoskillit/server/tools/tools_kitchen.py", 190),
     ("src/autoskillit/server/tools/tools_kitchen.py", 224),
-    ("src/autoskillit/server/tools/tools_kitchen.py", 918),
-    ("src/autoskillit/server/tools/tools_kitchen.py", 978),
+    ("src/autoskillit/server/tools/tools_kitchen.py", 947),
+    ("src/autoskillit/server/tools/tools_kitchen.py", 1007),
     # tools_pipeline_tracker.py — tracker_data dict
     ("src/autoskillit/server/tools/tools_pipeline_tracker.py", 166),
     # tools_status.py — mcp_data dict

tests/server/test_kitchen_lifecycle.py

@@ -110,15 +110,19 @@ async def test_back_to_back_open_close_open_resets_infrastructure(monkeypatch, t
         result1 = await _open_kitchen_handler()
         assert result1 is None
         assert ctx.gate_infrastructure_ready is True
+        first_task = ctx.quota_refresh_task
 
         _close_kitchen_handler()
         assert ctx.gate_infrastructure_ready is False
+        await asyncio.sleep(0)
+        assert first_task.cancelled() or first_task.done()
 
         result2 = await _open_kitchen_handler()
         assert result2 is None
         assert ctx.gate_infrastructure_ready is True
+        second_task = ctx.quota_refresh_task
+        assert second_task is not first_task
 
-        task = ctx.quota_refresh_task
+        _close_kitchen_handler()
         await asyncio.sleep(0)
-        if task is not None:
-            assert task.cancelled() or task.done()
+        assert second_task.cancelled() or second_task.done()