kailash 0.6.5__py3-none-any.whl → 0.7.0__py3-none-any.whl

kailash/servers/workflow_server.py

@@ -0,0 +1,290 @@
+"""Basic workflow server implementation.
+
+This module provides WorkflowServer - a renamed and improved version of
+WorkflowAPIGateway with clearer naming and better organization.
+"""
+
+import logging
+from concurrent.futures import ThreadPoolExecutor
+from contextlib import asynccontextmanager
+from typing import Any
+
+from fastapi import FastAPI, WebSocket
+from fastapi.middleware.cors import CORSMiddleware
+from pydantic import BaseModel, Field
+
+from ..api.workflow_api import WorkflowAPI
+from ..workflow import Workflow
+
+logger = logging.getLogger(__name__)
+
+
+class WorkflowRegistration(BaseModel):
+    """Registration details for a workflow."""
+
+    model_config = {"arbitrary_types_allowed": True}
+
+    name: str
+    type: str = Field(description="embedded or proxied")
+    workflow: Workflow | None = None
+    proxy_url: str | None = None
+    health_check: str | None = None
+    description: str | None = None
+    version: str = "1.0.0"
+    tags: list[str] = Field(default_factory=list)
+
+
+class WorkflowServer:
+    """Basic workflow server for hosting multiple Kailash workflows.
+
+    This server provides:
+    - Multi-workflow hosting with dynamic registration
+    - REST API endpoints for workflow execution
+    - WebSocket support for real-time updates
+    - MCP server integration
+    - Health monitoring
+    - CORS support
+
+    This is the base server class. For production deployments, consider
+    using EnterpriseWorkflowServer which includes durability, security,
+    and monitoring features.
+
+    Attributes:
+        app: FastAPI application instance
+        workflows: Registry of all registered workflows
+        executor: Thread pool for synchronous execution
+        mcp_servers: Registry of MCP servers
+    """
+
+    def __init__(
+        self,
+        title: str = "Kailash Workflow Server",
+        description: str = "Multi-workflow hosting server",
+        version: str = "1.0.0",
+        max_workers: int = 10,
+        cors_origins: list[str] = None,
+        **kwargs,
+    ):
+        """Initialize the workflow server.
+
+        Args:
+            title: Server title for documentation
+            description: Server description
+            version: Server version
+            max_workers: Maximum thread pool workers
+            cors_origins: Allowed CORS origins
+        """
+        self.workflows: dict[str, WorkflowRegistration] = {}
+        self.mcp_servers: dict[str, Any] = {}
+        self.executor = ThreadPoolExecutor(max_workers=max_workers)
+
+        # Create FastAPI app with lifespan
+        @asynccontextmanager
+        async def lifespan(app: FastAPI):
+            # Startup
+            logger.info(f"Starting {title} v{version}")
+            yield
+            # Shutdown
+            logger.info("Shutting down workflow server")
+            self.executor.shutdown(wait=True)
+
+        self.app = FastAPI(
+            title=title, description=description, version=version, lifespan=lifespan
+        )
+
+        # Add CORS middleware
+        if cors_origins:
+            self.app.add_middleware(
+                CORSMiddleware,
+                allow_origins=cors_origins,
+                allow_credentials=True,
+                allow_methods=["*"],
+                allow_headers=["*"],
+            )
+
+        # Register root endpoints
+        self._register_root_endpoints()
+
+    def _register_root_endpoints(self):
+        """Register server-level endpoints."""
+
+        @self.app.get("/")
+        async def root():
+            """Server information."""
+            return {
+                "name": self.app.title,
+                "version": self.app.version,
+                "workflows": list(self.workflows.keys()),
+                "mcp_servers": list(self.mcp_servers.keys()),
+                "type": "workflow_server",
+            }
+
+        @self.app.get("/workflows")
+        async def list_workflows():
+            """List all registered workflows."""
+            return {
+                name: {
+                    "type": reg.type,
+                    "description": reg.description,
+                    "version": reg.version,
+                    "tags": reg.tags,
+                    "endpoints": self._get_workflow_endpoints(name),
+                }
+                for name, reg in self.workflows.items()
+            }
+
+        @self.app.get("/health")
+        async def health_check():
+            """Server health check."""
+            health_status = {
+                "status": "healthy",
+                "server_type": "workflow_server",
+                "workflows": {},
+                "mcp_servers": {},
+            }
+
+            # Check workflow health
+            for name, reg in self.workflows.items():
+                if reg.type == "embedded":
+                    health_status["workflows"][name] = "healthy"
+                else:
+                    # TODO: Implement proxy health check
+                    health_status["workflows"][name] = "unknown"
+
+            # Check MCP server health
+            for name, server in self.mcp_servers.items():
+                # TODO: Implement MCP health check
+                health_status["mcp_servers"][name] = "unknown"
+
+            return health_status
+
+        @self.app.websocket("/ws")
+        async def websocket_endpoint(websocket: WebSocket):
+            """WebSocket for real-time updates."""
+            await websocket.accept()
+            try:
+                while True:
+                    # Basic WebSocket echo - subclasses can override
+                    data = await websocket.receive_text()
+                    await websocket.send_text(f"Echo: {data}")
+            except Exception as e:
+                logger.error(f"WebSocket error: {e}")
+            finally:
+                await websocket.close()
+
+    def register_workflow(
+        self,
+        name: str,
+        workflow: Workflow,
+        description: str = None,
+        tags: list[str] = None,
+    ):
+        """Register a workflow with the server.
+
+        Args:
+            name: Unique workflow identifier
+            workflow: Workflow instance to register
+            description: Optional workflow description
+            tags: Optional tags for categorization
+        """
+        if name in self.workflows:
+            raise ValueError(f"Workflow '{name}' already registered")
+
+        # Create workflow registration
+        registration = WorkflowRegistration(
+            name=name,
+            type="embedded",
+            workflow=workflow,
+            description=description or f"Workflow: {name}",
+            tags=tags or [],
+        )
+
+        self.workflows[name] = registration
+
+        # Create workflow API wrapper
+        workflow_api = WorkflowAPI(workflow)
+
+        # Register workflow endpoints with prefix
+        prefix = f"/workflows/{name}"
+        self.app.mount(prefix, workflow_api.app)
+
+        logger.info(f"Registered workflow '{name}' at {prefix}")
+
+    def register_mcp_server(self, name: str, mcp_server: Any):
+        """Register an MCP server with the workflow server.
+
+        Args:
+            name: Unique MCP server identifier
+            mcp_server: MCP server instance
+        """
+        if name in self.mcp_servers:
+            raise ValueError(f"MCP server '{name}' already registered")
+
+        self.mcp_servers[name] = mcp_server
+
+        # Mount MCP server endpoints
+        mcp_prefix = f"/mcp/{name}"
+        # TODO: Implement MCP mounting logic
+
+        logger.info(f"Registered MCP server '{name}' at {mcp_prefix}")
+
+    def proxy_workflow(
+        self,
+        name: str,
+        proxy_url: str,
+        health_check: str = "/health",
+        description: str = None,
+        tags: list[str] = None,
+    ):
+        """Register a proxied workflow running on another server.
+
+        Args:
+            name: Unique workflow identifier
+            proxy_url: Base URL of the proxied workflow
+            health_check: Health check endpoint path
+            description: Optional workflow description
+            tags: Optional tags for categorization
+        """
+        if name in self.workflows:
+            raise ValueError(f"Workflow '{name}' already registered")
+
+        # Create proxied workflow registration
+        registration = WorkflowRegistration(
+            name=name,
+            type="proxied",
+            proxy_url=proxy_url,
+            health_check=health_check,
+            description=description or f"Proxied workflow: {name}",
+            tags=tags or [],
+        )
+
+        self.workflows[name] = registration
+
+        # TODO: Implement proxy endpoint creation
+        logger.info(f"Registered proxied workflow '{name}' -> {proxy_url}")
+
+    def _get_workflow_endpoints(self, name: str) -> list[str]:
+        """Get available endpoints for a workflow."""
+        base = f"/workflows/{name}"
+        return [
+            f"{base}/execute",
+            f"{base}/status",
+            f"{base}/schema",
+            f"{base}/docs",
+        ]
+
+    def run(self, host: str = "0.0.0.0", port: int = 8000, **kwargs):
+        """Run the workflow server.
+
+        Args:
+            host: Host address to bind to
+            port: Port to listen on
+            **kwargs: Additional arguments passed to uvicorn
+        """
+        import uvicorn
+
+        uvicorn.run(self.app, host=host, port=port, **kwargs)
+
+    def execute(self, **kwargs):
+        """Execute the server (alias for run)."""
+        self.run(**kwargs)

kailash/utils/data_validation.py

@@ -0,0 +1,192 @@
+"""Data validation and type consistency utilities for workflow execution."""
+
+import logging
+from typing import Any, Dict, List, Union
+
+logger = logging.getLogger(__name__)
+
+
+class DataTypeValidator:
+    """Validates and fixes data type inconsistencies in workflow execution."""
+
+    @staticmethod
+    def validate_node_output(node_id: str, output: Dict[str, Any]) -> Dict[str, Any]:
+        """Validate and fix node output to ensure consistent data types.
+
+        Args:
+            node_id: ID of the node producing the output
+            output: Raw output from the node
+
+        Returns:
+            Validated and potentially fixed output
+        """
+        if not isinstance(output, dict):
+            logger.warning(
+                f"Node '{node_id}' output should be a dict, got {type(output)}. Wrapping in result key."
+            )
+            return {"result": output}
+
+        validated_output = {}
+
+        for key, value in output.items():
+            validated_value = DataTypeValidator._validate_value(node_id, key, value)
+            validated_output[key] = validated_value
+
+        return validated_output
+
+    @staticmethod
+    def _validate_value(node_id: str, key: str, value: Any) -> Any:
+        """Validate a single value and fix common type issues.
+
+        Args:
+            node_id: ID of the node producing the value
+            key: Key name for the value
+            value: The value to validate
+
+        Returns:
+            Validated value
+        """
+        # Common bug: Dictionary gets converted to list of keys
+        if isinstance(value, list) and key == "result":
+            # Check if this looks like dict keys
+            if all(isinstance(item, str) for item in value):
+                logger.warning(
+                    f"Node '{node_id}' output '{key}' appears to be dict keys converted to list: {value}. "
+                    "This is a known bug in some node implementations."
+                )
+                # We can't recover the original dict, so wrap the list properly
+                return value
+
+        # Ensure string data is not accidentally indexed as dict
+        if isinstance(value, str):
+            return value
+
+        # Validate dict structure
+        if isinstance(value, dict):
+            # Recursively validate nested dicts
+            validated_dict = {}
+            for subkey, subvalue in value.items():
+                validated_dict[subkey] = DataTypeValidator._validate_value(
+                    node_id, f"{key}.{subkey}", subvalue
+                )
+            return validated_dict
+
+        # Validate list structure
+        if isinstance(value, list):
+            # Ensure list elements are consistently typed
+            if len(value) > 0:
+                first_type = type(value[0])
+                inconsistent_types = [
+                    i for i, item in enumerate(value) if type(item) is not first_type
+                ]
+                if inconsistent_types:
+                    logger.warning(
+                        f"Node '{node_id}' output '{key}' has inconsistent list element types. "
+                        f"First type: {first_type}, inconsistent indices: {inconsistent_types[:5]}"
+                    )
+
+        return value
+
+    @staticmethod
+    def validate_node_input(node_id: str, inputs: Dict[str, Any]) -> Dict[str, Any]:
+        """Validate node inputs before execution.
+
+        Args:
+            node_id: ID of the node receiving the inputs
+            inputs: Input parameters for the node
+
+        Returns:
+            Validated inputs
+        """
+        if not isinstance(inputs, dict):
+            logger.error(f"Node '{node_id}' inputs must be a dict, got {type(inputs)}")
+            return {}
+
+        validated_inputs = {}
+
+        for key, value in inputs.items():
+            # Handle common data mapping issues
+            if key == "data" and isinstance(value, list):
+                # Check if this is the dict-to-keys bug
+                if all(isinstance(item, str) for item in value):
+                    logger.warning(
+                        f"Node '{node_id}' received list of strings for 'data' parameter: {value}. "
+                        "This may be due to a dict-to-keys conversion bug in upstream node."
+                    )
+
+            validated_inputs[key] = value
+
+        return validated_inputs
+
+    @staticmethod
+    def fix_string_indexing_error(data: Any, error_context: str = "") -> Any:
+        """Fix common 'string indices must be integers' errors.
+
+        Args:
+            data: Data that caused the error
+            error_context: Context information about the error
+
+        Returns:
+            Fixed data or None if unfixable
+        """
+        if isinstance(data, str):
+            logger.warning(
+                f"Attempting to index string as dict{' in ' + error_context if error_context else ''}. "
+                f"String value: '{data[:100]}...'"
+                if len(data) > 100
+                else f"String value: '{data}'"
+            )
+            return None
+
+        if isinstance(data, list) and all(isinstance(item, str) for item in data):
+            logger.warning(
+                f"Data appears to be list of dict keys{' in ' + error_context if error_context else ''}. "
+                f"Keys: {data}. Cannot recover original dict structure."
+            )
+            return None
+
+        return data
+
+    @staticmethod
+    def create_error_recovery_wrapper(
+        original_data: Any, fallback_data: Any = None
+    ) -> Dict[str, Any]:
+        """Create a recovery wrapper for problematic data.
+
+        Args:
+            original_data: The problematic data
+            fallback_data: Fallback data to use if original is unusable
+
+        Returns:
+            Recovery wrapper dict
+        """
+        return {
+            "data": fallback_data if fallback_data is not None else {},
+            "original_data": original_data,
+            "data_type_error": True,
+            "error_message": f"Data type conversion error. Original type: {type(original_data)}",
+        }
+
+
+def validate_workflow_data_flow(workflow_results: Dict[str, Any]) -> Dict[str, Any]:
+    """Validate entire workflow result data flow for consistency.
+
+    Args:
+        workflow_results: Results from workflow execution
+
+    Returns:
+        Validated workflow results
+    """
+    validated_results = {}
+
+    for node_id, result in workflow_results.items():
+        try:
+            validated_result = DataTypeValidator.validate_node_output(node_id, result)
+            validated_results[node_id] = validated_result
+        except Exception as e:
+            logger.error(f"Data validation failed for node '{node_id}': {e}")
+            validated_results[node_id] = (
+                DataTypeValidator.create_error_recovery_wrapper(result)
+            )
+
+    return validated_results