Cache hook integration metadata

Author: nike-17

src/specify_cli/extensions.py

@@ -2515,6 +2515,9 @@ def __init__(self, project_root: Path):
         self.extensions_dir = project_root / ".specify" / "extensions"
         self.config_file = project_root / ".specify" / "extensions.yml"
         self._init_options_cache: Optional[Dict[str, Any]] = None
+        self._integration_invocation_cache: Optional[
+            tuple[Dict[str, Any] | None, str | None, Any | None]
+        ] = None
 
     def _load_init_options(self) -> Dict[str, Any]:
         """Load persisted init options used to determine invocation style.
@@ -2529,6 +2532,27 @@ def _load_init_options(self) -> Dict[str, Any]:
             self._init_options_cache = payload if isinstance(payload, dict) else {}
         return self._init_options_cache
 
+    def _load_integration_invocation_context(
+        self,
+    ) -> tuple[Dict[str, Any] | None, str | None, Any | None]:
+        """Load integration metadata used by hook rendering once per executor."""
+        if self._integration_invocation_cache is None:
+            try:
+                from .integration_state import default_integration_key, try_read_integration_json
+                from .integrations import get_integration
+
+                state, _ = try_read_integration_json(self.project_root)
+                key = default_integration_key(state) if state else None
+                integration = get_integration(key) if key else None
+                self._integration_invocation_cache = (state, key, integration)
+            except Exception:
+                # Hook rendering must keep working for projects with older or
+                # unreadable integration metadata; the init-options fallback
+                # below preserves the legacy behavior in that case.
+                self._integration_invocation_cache = (None, None, None)
+
+        return self._integration_invocation_cache
+
     @staticmethod
     def _skill_name_from_command(command: Any) -> str:
         """Map a command id like speckit.plan to speckit-plan skill name."""
@@ -2551,20 +2575,15 @@ def _render_hook_invocation(self, command: Any) -> str:
         if command_id.startswith("speckit."):
             try:
                 from .integration_runtime import user_command_invocation_for_integration
-                from .integration_state import default_integration_key, try_read_integration_json
-                from .integrations import get_integration
 
-                state, _ = try_read_integration_json(self.project_root)
-                if state:
-                    key = default_integration_key(state)
-                    integration = get_integration(key) if key else None
-                    if integration:
-                        return user_command_invocation_for_integration(
-                            integration,
-                            state,
-                            key,
-                            command_id,
-                        )
+                state, key, integration = self._load_integration_invocation_context()
+                if state and key and integration:
+                    return user_command_invocation_for_integration(
+                        integration,
+                        state,
+                        key,
+                        command_id,
+                    )
             except Exception:
                 # Hook rendering must keep working for projects with older or
                 # unreadable integration metadata; the init-options fallback

tests/test_extensions.py

@@ -5101,6 +5101,35 @@ def fake_load_init_options(_project_root):
         assert hook_executor._render_hook_invocation("speckit.tasks") == "/skill:speckit-tasks"
         assert calls["count"] == 1
 
+    def test_hook_executor_caches_integration_state_lookup(self, project_dir, monkeypatch):
+        """Integration metadata should be loaded once per executor instance."""
+        calls = {"count": 0}
+        state = {
+            "default_integration": "codex",
+            "integration": "codex",
+            "installed_integrations": ["codex"],
+            "integration_settings": {
+                "codex": {
+                    "invoke_separator": "-",
+                    "command_prefix": "$",
+                }
+            },
+        }
+
+        def fake_try_read_integration_json(_project_root):
+            calls["count"] += 1
+            return state, None
+
+        monkeypatch.setattr(
+            "specify_cli.integration_state.try_read_integration_json",
+            fake_try_read_integration_json,
+        )
+
+        hook_executor = HookExecutor(project_dir)
+        assert hook_executor._render_hook_invocation("speckit.plan") == "$speckit-plan"
+        assert hook_executor._render_hook_invocation("speckit.tasks") == "$speckit-tasks"
+        assert calls["count"] == 1
+
     def test_hook_message_falls_back_when_invocation_is_empty(self, project_dir):
         """Hook messages should still render actionable command placeholders."""
         init_options = project_dir / ".specify" / "init-options.json"