fix: correct vacuous-gate detection, import path, test mock, and symbol count

- Fix _is_vacuous_gate to skip the gate step itself when scanning for live
  capability references (gate_backend_write references its own capability
  ingredient in with_args, which was falsely preventing vacuous detection)
- Fix import path violation: use autoskillit.core gateway instead of
  autoskillit.core.types subpackage directly
- Fix deferred-recall test: add AsyncMock for ctx.disable_components to
  prevent "MagicMock can't be used in await" error
- Update symbol count from 93 to 94 for new CAPABILITY_INGREDIENT_TO_SKIP_GUARD

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

Author: Trecek

src/autoskillit/recipe/_recipe_composition.py

@@ -9,8 +9,7 @@
 
 import regex as re
 
-from autoskillit.core import YAMLError, load_yaml
-from autoskillit.core.types import CAPABILITY_INGREDIENT_TO_SKIP_GUARD
+from autoskillit.core import CAPABILITY_INGREDIENT_TO_SKIP_GUARD, YAMLError, load_yaml
 from autoskillit.recipe._contracts_types import INPUT_REF_RE
 from autoskillit.recipe.io import find_sub_recipe_by_name
 from autoskillit.recipe.io import load_recipe as _load_recipe_from_path
@@ -165,6 +164,7 @@ def _compute_capability_feasibility(
             continue
         if _is_vacuous_gate(
             gate_input_keys,
+            gate_step_name=step_name,
             skip_resolutions=skip_resolutions,
             pre_prune_steps=pre_prune_steps,
             post_prune_steps=steps,
@@ -178,6 +178,7 @@ def _compute_capability_feasibility(
 def _is_vacuous_gate(
     gate_input_keys: set[str],
     *,
+    gate_step_name: str,
     skip_resolutions: dict[str, bool | None] | None,
     pre_prune_steps: dict[str, Any] | None,
     post_prune_steps: dict[str, Any],
@@ -200,8 +201,8 @@ def _is_vacuous_gate(
         if not pruned_for_capability:
             return False
         live_uses_capability = False
-        for s in post_prune_steps.values():
-            if s is None:
+        for sn, s in post_prune_steps.items():
+            if s is None or sn == gate_step_name:
                 continue
             with_args = getattr(s, "with_args", None) or {}
             if cap_key in with_args:

tests/arch/test_subpackage_structure.py

@@ -60,8 +60,8 @@ def test_type_constants_split_completeness(self):
         assert len(combined) == len(remaining) + len(env) + len(features) + len(registries), (
             "Duplicate symbols across split modules"
         )
-        assert len(combined) == 93, (
-            f"Expected 93 symbols total, got {len(combined)} "
+        assert len(combined) == 94, (
+            f"Expected 94 symbols total, got {len(combined)} "
             f"(remaining={len(remaining)}, env={len(env)}, "
             f"features={len(features)}, registries={len(registries)})"
         )

tests/server/test_open_kitchen_deferred_recall.py

@@ -275,6 +275,7 @@ async def test_deferred_recall_returns_infeasible_when_dispatch_feasible_false()
     mock_recipe_obj = MagicMock()
     mock_recipe_obj.steps = {"gate_backend_write": MagicMock()}
     mock_ctx.recipes.load.return_value = mock_recipe_obj
+    mock_ctx.disable_components = AsyncMock()
 
     with patch("autoskillit.server._get_ctx", return_value=mock_ctx):
         result = await open_kitchen(name="test-recipe", ctx=mock_ctx)