foundry-mcp 0.7.0__py3-none-any.whl → 0.8.10__py3-none-any.whl

foundry_mcp/tools/unified/review.py

@@ -27,7 +27,7 @@ from foundry_mcp.core.ai_consultation import (
 from foundry_mcp.core.prompts.fidelity_review import (
     FIDELITY_SYNTHESIZED_RESPONSE_SCHEMA,
 )
-from foundry_mcp.core.llm_config import get_consultation_config
+from foundry_mcp.core.llm_config import get_consultation_config, load_consultation_config
 from foundry_mcp.core.naming import canonical_tool
 from foundry_mcp.core.observability import get_metrics, mcp_tool
 from foundry_mcp.core.providers import get_provider_statuses
@@ -703,7 +703,10 @@ def _handle_fidelity(*, config: ServerConfig, payload: Dict[str, Any]) -> dict:
     preferred_providers = ai_tools if isinstance(ai_tools, list) else []
     first_provider = preferred_providers[0] if preferred_providers else None
 
-    orchestrator = ConsultationOrchestrator()
+    # Load consultation config from workspace path to get provider priority list
+    config_file = ws_path / "foundry-mcp.toml"
+    consultation_config = load_consultation_config(config_file=config_file)
+    orchestrator = ConsultationOrchestrator(config=consultation_config)
     if not orchestrator.is_available(provider_id=first_provider):
         return asdict(
             error_response(

foundry_mcp/tools/unified/review_helpers.py

@@ -193,6 +193,7 @@ def _run_ai_review(
             ConsultationRequest,
             ConsultationWorkflow,
         )
+        from foundry_mcp.core.llm_config import load_consultation_config
     except ImportError:
         return asdict(
             error_response(
@@ -203,7 +204,21 @@ def _run_ai_review(
             )
         )
 
-    orchestrator = ConsultationOrchestrator(default_timeout=ai_timeout)
+    # Load consultation config from workspace (fixes config discovery issue)
+    # Derive workspace from specs_dir - check parent directories for config
+    config_file = None
+    if specs_dir:
+        ws_path = specs_dir.parent if specs_dir.name == "specs" else specs_dir
+        for _ in range(5):  # Search up to 5 levels for foundry-mcp.toml
+            candidate = ws_path / "foundry-mcp.toml"
+            if candidate.exists():
+                config_file = candidate
+                break
+            if ws_path.parent == ws_path:  # Reached root
+                break
+            ws_path = ws_path.parent
+    consultation_config = load_consultation_config(config_file=config_file)
+    orchestrator = ConsultationOrchestrator(config=consultation_config, default_timeout=ai_timeout)
 
     if not orchestrator.is_available(provider_id=ai_provider):
         return asdict(

foundry_mcp/tools/unified/server.py

@@ -18,7 +18,6 @@ from mcp.server.fastmcp import FastMCP
 from foundry_mcp.config import ServerConfig
 from foundry_mcp.core.context import generate_correlation_id, get_correlation_id
 from foundry_mcp.core.discovery import get_capabilities, get_tool_registry
-from foundry_mcp.core.feature_flags import get_flag_service
 from foundry_mcp.core.naming import canonical_tool
 from foundry_mcp.core.observability import (
     get_metrics,
@@ -256,26 +255,15 @@ def _handle_tools(*, config: ServerConfig, payload: Dict[str, Any]) -> dict:
 
     start_time = time.perf_counter()
 
-    categories_list: list[str]
-    flag_service = get_flag_service()
-    if flag_service.is_enabled("unified_manifest"):
-        all_tools = _build_unified_manifest_tools()
-        if category:
-            all_tools = [tool for tool in all_tools if tool.get("category") == category]
-        if tag:
-            all_tools = [tool for tool in all_tools if tag in (tool.get("tags") or [])]
-        categories_list = sorted(
-            {tool.get("category", "general") for tool in all_tools}
-        )
-    else:
-        registry = get_tool_registry()
-        all_tools = registry.list_tools(
-            category=category,
-            tag=tag,
-            include_deprecated=include_deprecated,
-        )
-        categories = registry.list_categories()
-        categories_list = [c["name"] for c in categories]
+    # Always use unified manifest (feature flags removed)
+    all_tools = _build_unified_manifest_tools()
+    if category:
+        all_tools = [tool for tool in all_tools if tool.get("category") == category]
+    if tag:
+        all_tools = [tool for tool in all_tools if tag in (tool.get("tags") or [])]
+    categories_list = sorted(
+        {tool.get("category", "general") for tool in all_tools}
+    )
 
     start_idx = 0
     if cursor:
@@ -360,9 +348,6 @@ def _handle_tools(*, config: ServerConfig, payload: Dict[str, Any]) -> dict:
             tokens=manifest_tokens,
             tool_count=len(all_tools),
         )
-        exporter.record_feature_flag_state(
-            "unified_manifest", flag_service.is_enabled("unified_manifest")
-        )
 
     response["meta"]["request_id"] = request_id
     return response