kailash 0.5.0__py3-none-any.whl → 0.6.1__py3-none-any.whl

kailash/nodes/admin/permission_check.py

@@ -18,6 +18,7 @@ Features:
 
 import hashlib
 import json
+import logging
 from dataclasses import dataclass
 from datetime import UTC, datetime, timedelta
 from enum import Enum
@@ -40,9 +41,11 @@ from kailash.access_control_abac import (
     LogicalOperator,
 )
 from kailash.nodes.base import Node, NodeParameter, register_node
-from kailash.nodes.data import AsyncSQLDatabaseNode
+from kailash.nodes.data import SQLDatabaseNode
 from kailash.sdk_exceptions import NodeExecutionError, NodeValidationError
 
+from .schema_manager import AdminSchemaManager
+
 
 class PermissionCheckOperation(Enum):
     """Supported permission check operations."""
@@ -195,6 +198,8 @@ class PermissionCheckNode(Node):
         self._attribute_evaluator = None
         self._permission_cache = {}
         self._cache_timestamps = {}
+        self._schema_manager = None
+        self.logger = logging.getLogger(__name__)
 
     def get_parameters(self) -> Dict[str, NodeParameter]:
         """Define parameters for permission checking operations."""
@@ -286,6 +291,13 @@ class PermissionCheckNode(Node):
                     default=True,
                     description="Include timing information in results",
                 ),
+                NodeParameter(
+                    name="audit",
+                    type=bool,
+                    required=False,
+                    default=False,
+                    description="Enable audit logging for this operation",
+                ),
                 # Multi-tenancy
                 NodeParameter(
                     name="tenant_id",
@@ -308,6 +320,56 @@ class PermissionCheckNode(Node):
                     default=True,
                     description="Enable strict validation of inputs",
                 ),
+                # Additional validation options
+                NodeParameter(
+                    name="conditions",
+                    type=list,
+                    required=False,
+                    description="List of ABAC conditions to validate",
+                ),
+                NodeParameter(
+                    name="validate_syntax",
+                    type=bool,
+                    required=False,
+                    default=True,
+                    description="Validate condition syntax",
+                ),
+                NodeParameter(
+                    name="test_evaluation",
+                    type=bool,
+                    required=False,
+                    default=True,
+                    description="Test condition evaluation with provided context",
+                ),
+                NodeParameter(
+                    name="permission_type",
+                    type=str,
+                    required=False,
+                    default="all",
+                    choices=["all", "direct", "inherited"],
+                    description="Type of permissions to return",
+                ),
+                NodeParameter(
+                    name="check_inheritance",
+                    type=bool,
+                    required=False,
+                    default=True,
+                    description="Check hierarchical resource inheritance",
+                ),
+                NodeParameter(
+                    name="include_rules",
+                    type=bool,
+                    required=False,
+                    default=True,
+                    description="Include rule details in explanation",
+                ),
+                NodeParameter(
+                    name="include_hierarchy",
+                    type=bool,
+                    required=False,
+                    default=True,
+                    description="Include role hierarchy breakdown",
+                ),
             ]
         }
 
@@ -348,6 +410,10 @@ class PermissionCheckNode(Node):
 
     def _init_dependencies(self, inputs: Dict[str, Any]):
         """Initialize database and access manager dependencies."""
+        # Skip initialization if already initialized (for testing)
+        if hasattr(self, "_db_node") and self._db_node is not None:
+            return
+
         # Get database config
         db_config = inputs.get(
             "database_config",
@@ -361,8 +427,22 @@ class PermissionCheckNode(Node):
             },
         )
 
-        # Initialize async database node
-        self._db_node = AsyncSQLDatabaseNode(name="permission_check_db", **db_config)
+        # Initialize database node
+        self._db_node = SQLDatabaseNode(name="permission_check_db", **db_config)
+
+        # Initialize schema manager and ensure schema exists
+        if not self._schema_manager:
+            self._schema_manager = AdminSchemaManager(db_config)
+
+            # Validate schema exists, create if needed
+            try:
+                validation = self._schema_manager.validate_schema()
+                if not validation["is_valid"]:
+                    self.logger.info("Creating unified admin schema...")
+                    self._schema_manager.create_full_schema(drop_existing=False)
+                    self.logger.info("Unified admin schema created successfully")
+            except Exception as e:
+                self.logger.warning(f"Schema validation/creation failed: {e}")
 
         # Initialize enhanced access manager
         self._access_manager = AccessControlManager(strategy="abac")
@@ -379,6 +459,7 @@ class PermissionCheckNode(Node):
         cache_ttl = inputs.get("cache_ttl", 300)
         explain = inputs.get("explain", False)
         include_timing = inputs.get("include_timing", True)
+        audit = inputs.get("audit", False)
 
         start_time = datetime.now(UTC) if include_timing else None
 
@@ -458,6 +539,24 @@ class PermissionCheckNode(Node):
                 cache_data["explanation"] = explanation.to_dict()
             self._set_cache(cache_key, cache_data, cache_ttl)
 
+        # Log to audit trail if enabled
+        if audit:
+            self._create_audit_log(
+                user_id=user_id,
+                action="permission_check",
+                resource_type="resource",
+                resource_id=resource_id,
+                details={
+                    "permission": permission,
+                    "allowed": allowed,
+                    "context": context,
+                    "rbac_result": rbac_result,
+                    "abac_result": abac_result,
+                },
+                success=allowed,
+                tenant_id=tenant_id,
+            )
+
         result = {
             "result": {
                 "check": check_result.to_dict(),
@@ -623,30 +722,36 @@ class PermissionCheckNode(Node):
 
     def _get_user_context(self, user_id: str, tenant_id: str) -> Optional[UserContext]:
         """Get user context for permission evaluation."""
-        # Query user data from database
+        # Query user data from unified admin schema
         query = """
         SELECT user_id, email, roles, attributes, status, tenant_id
         FROM users
         WHERE user_id = $1 AND tenant_id = $2 AND status = 'active'
         """
 
-        self._db_node.config.update(
-            {"query": query, "params": [user_id, tenant_id], "fetch_mode": "one"}
-        )
+        try:
+            result = self._db_node.run(
+                query=query, parameters=[user_id, tenant_id], result_format="dict"
+            )
 
-        result = self._db_node.run()
-        user_data = result.get("result", {}).get("data")
+            # Handle the corrected result structure
+            user_rows = result.get("data", [])
+            if not user_rows:
+                return None
 
-        if not user_data:
-            return None
+            user_data = user_rows[0]
 
-        return UserContext(
-            user_id=user_data["user_id"],
-            tenant_id=user_data["tenant_id"],
-            email=user_data["email"],
-            roles=user_data.get("roles", []),
-            attributes=user_data.get("attributes", {}),
-        )
+            return UserContext(
+                user_id=user_data["user_id"],
+                tenant_id=user_data["tenant_id"],
+                email=user_data["email"],
+                roles=user_data.get("roles", []),
+                attributes=user_data.get("attributes", {}),
+            )
+        except Exception as e:
+            # Log the error and return None to indicate user not found
+            self.logger.warning(f"Failed to get user context for {user_id}: {e}")
+            return None
 
     def _check_rbac_permission(
         self, user_context: UserContext, resource_id: str, permission: str
@@ -687,9 +792,9 @@ class PermissionCheckNode(Node):
             # Convert permission string to NodePermission if possible
             node_permission = NodePermission.EXECUTE  # Default
             if permission.lower() in ["read", "view"]:
-                node_permission = NodePermission.VIEW
+                node_permission = NodePermission.READ_OUTPUT  # Map view to read_output
             elif permission.lower() in ["write", "edit", "update"]:
-                node_permission = NodePermission.EDIT
+                node_permission = NodePermission.WRITE_INPUT  # Map edit to write_input
             elif permission.lower() in ["execute", "run"]:
                 node_permission = NodePermission.EXECUTE
             elif permission.lower() in ["delete", "remove"]:
@@ -733,19 +838,25 @@ class PermissionCheckNode(Node):
 
             SELECT r.role_id, r.permissions, r.parent_roles
             FROM roles r
-            JOIN role_hierarchy rh ON r.role_id = ANY(rh.parent_roles)
+            JOIN role_hierarchy rh ON r.role_id = ANY(
+                SELECT jsonb_array_elements_text(rh.parent_roles)
+            )
             WHERE r.tenant_id = $2 AND r.is_active = true
         )
-        SELECT DISTINCT unnest(permissions) as permission
+        SELECT DISTINCT unnest(
+            CASE
+                WHEN jsonb_typeof(permissions) = 'array'
+                THEN ARRAY(SELECT jsonb_array_elements_text(permissions))
+                ELSE ARRAY[]::text[]
+            END
+        ) as permission
         FROM role_hierarchy
         """
 
-        self._db_node.config.update(
-            {"query": query, "params": [role_id, tenant_id], "fetch_mode": "all"}
+        result = self._db_node.run(
+            query=query, parameters=[role_id, tenant_id], result_format="dict"
         )
-
-        result = self._db_node.run()
-        permission_rows = result.get("result", {}).get("data", [])
+        permission_rows = result.get("data", [])
 
         return {row["permission"] for row in permission_rows}
 
@@ -841,24 +952,697 @@ class PermissionCheckNode(Node):
     # Additional operations would follow similar patterns
     def _check_node_access(self, inputs: Dict[str, Any]) -> Dict[str, Any]:
         """Check access to a specific node type."""
-        raise NotImplementedError("Check node access operation will be implemented")
+        user_id = inputs["user_id"]
+        node_type = inputs["resource_id"]  # node_type is passed as resource_id
+        permission = inputs.get("permission", "execute")  # Default to execute
+        tenant_id = inputs.get("tenant_id", "default")
+        context = inputs.get("context", {})
+
+        # Get user context
+        user_context = self._get_user_context(user_id, tenant_id)
+        if not user_context:
+            raise NodeValidationError(f"User not found: {user_id}")
+
+        # Map permission string to NodePermission enum
+        permission_mapping = {
+            "view": NodePermission.READ_OUTPUT,  # Map view to read_output
+            "edit": NodePermission.WRITE_INPUT,  # Map edit to write_input
+            "execute": NodePermission.EXECUTE,
+            "delete": NodePermission.SKIP,  # Map delete to skip (no direct delete permission)
+        }
+
+        node_permission = permission_mapping.get(
+            permission.lower(), NodePermission.EXECUTE
+        )
+
+        start_time = datetime.now(UTC)
+
+        # Use the access control manager for node access check
+        try:
+            decision = self._access_manager.check_node_access(
+                user=user_context,
+                resource_id=node_type,
+                permission=node_permission,
+                context=context,
+            )
+
+            evaluation_time_ms = (datetime.now(UTC) - start_time).total_seconds() * 1000
+
+            return {
+                "result": {
+                    "access_check": {
+                        "allowed": decision.allowed,
+                        "reason": decision.reason,
+                        "user_id": user_id,
+                        "node_type": node_type,
+                        "permission": permission,
+                        "evaluation_time_ms": evaluation_time_ms,
+                        "decision_id": decision.decision_id,
+                    },
+                    "operation": "check_node_access",
+                    "timestamp": datetime.now(UTC).isoformat(),
+                }
+            }
+
+        except Exception as e:
+            return {
+                "result": {
+                    "access_check": {
+                        "allowed": False,
+                        "reason": f"Access check failed: {str(e)}",
+                        "user_id": user_id,
+                        "node_type": node_type,
+                        "permission": permission,
+                        "evaluation_time_ms": (
+                            datetime.now(UTC) - start_time
+                        ).total_seconds()
+                        * 1000,
+                        "error": True,
+                    },
+                    "operation": "check_node_access",
+                    "timestamp": datetime.now(UTC).isoformat(),
+                }
+            }
 
     def _check_workflow_access(self, inputs: Dict[str, Any]) -> Dict[str, Any]:
         """Check access to workflow operations."""
-        raise NotImplementedError("Check workflow access operation will be implemented")
+        user_id = inputs["user_id"]
+        workflow_id = inputs["resource_id"]  # workflow_id is passed as resource_id
+        permission = inputs.get("permission", "execute")  # execute, view, edit, delete
+        tenant_id = inputs.get("tenant_id", "default")
+        context = inputs.get("context", {})
+
+        # Get user context
+        user_context = self._get_user_context(user_id, tenant_id)
+        if not user_context:
+            raise NodeValidationError(f"User not found: {user_id}")
+
+        # Map permission string to WorkflowPermission enum
+        permission_mapping = {
+            "view": WorkflowPermission.VIEW,
+            "execute": WorkflowPermission.EXECUTE,
+            "edit": WorkflowPermission.MODIFY,  # EDIT mapped to MODIFY
+            "delete": WorkflowPermission.DELETE,
+            "deploy": WorkflowPermission.DEPLOY,
+            "share": WorkflowPermission.SHARE,
+        }
+
+        workflow_permission = permission_mapping.get(
+            permission.lower(), WorkflowPermission.EXECUTE
+        )
+
+        start_time = datetime.now(UTC)
+
+        # Use the access control manager for workflow access check
+        try:
+            decision = self._access_manager.check_workflow_access(
+                user=user_context,
+                workflow_id=workflow_id,
+                permission=workflow_permission,
+                context=context,
+            )
+
+            evaluation_time_ms = (datetime.now(UTC) - start_time).total_seconds() * 1000
+
+            return {
+                "result": {
+                    "access_check": {
+                        "allowed": decision.allowed,
+                        "reason": decision.reason,
+                        "user_id": user_id,
+                        "workflow_id": workflow_id,
+                        "permission": permission,
+                        "evaluation_time_ms": evaluation_time_ms,
+                        "decision_id": decision.decision_id,
+                        "applied_rules": getattr(decision, "applied_rules", []),
+                    },
+                    "operation": "check_workflow_access",
+                    "timestamp": datetime.now(UTC).isoformat(),
+                }
+            }
+
+        except Exception as e:
+            return {
+                "result": {
+                    "access_check": {
+                        "allowed": False,
+                        "reason": f"Workflow access check failed: {str(e)}",
+                        "user_id": user_id,
+                        "workflow_id": workflow_id,
+                        "permission": permission,
+                        "evaluation_time_ms": (
+                            datetime.now(UTC) - start_time
+                        ).total_seconds()
+                        * 1000,
+                        "error": True,
+                    },
+                    "operation": "check_workflow_access",
+                    "timestamp": datetime.now(UTC).isoformat(),
+                }
+            }
 
     def _get_user_permissions(self, inputs: Dict[str, Any]) -> Dict[str, Any]:
         """Get all permissions for a user."""
-        raise NotImplementedError("Get user permissions operation will be implemented")
+        user_id = inputs["user_id"]
+        tenant_id = inputs.get("tenant_id", "default")
+        include_inherited = inputs.get("include_inherited", True)
+        permission_type = inputs.get("permission_type", "all")  # all, direct, inherited
+
+        # Get user context
+        user_context = self._get_user_context(user_id, tenant_id)
+        if not user_context:
+            raise NodeValidationError(f"User not found: {user_id}")
+
+        start_time = datetime.now(UTC)
+
+        # Get all effective permissions
+        all_permissions = self._get_user_effective_permissions(user_context)
+
+        # Get direct permissions from roles
+        direct_permissions = set()
+        role_permissions_breakdown = {}
+
+        for role in user_context.roles:
+            role_perms = self._get_role_direct_permissions(role, tenant_id)
+            direct_permissions.update(role_perms)
+            role_permissions_breakdown[role] = list(role_perms)
+
+        # Calculate inherited permissions
+        inherited_permissions = (
+            all_permissions - direct_permissions if include_inherited else set()
+        )
+
+        # Filter based on permission_type
+        result_permissions = set()
+        if permission_type == "all":
+            result_permissions = all_permissions
+        elif permission_type == "direct":
+            result_permissions = direct_permissions
+        elif permission_type == "inherited":
+            result_permissions = inherited_permissions
+
+        # Categorize permissions by resource type
+        categorized_permissions = {
+            "workflow": [],
+            "node": [],
+            "resource": [],
+            "admin": [],
+            "other": [],
+        }
+
+        for perm in result_permissions:
+            if perm.startswith("workflow:"):
+                categorized_permissions["workflow"].append(perm)
+            elif perm.startswith("node:"):
+                categorized_permissions["node"].append(perm)
+            elif perm.startswith("resource:"):
+                categorized_permissions["resource"].append(perm)
+            elif perm.startswith("admin:"):
+                categorized_permissions["admin"].append(perm)
+            else:
+                categorized_permissions["other"].append(perm)
+
+        evaluation_time_ms = (datetime.now(UTC) - start_time).total_seconds() * 1000
+
+        return {
+            "result": {
+                "user_permissions": {
+                    "user_id": user_id,
+                    "permissions": list(result_permissions),
+                    "categorized_permissions": categorized_permissions,
+                    "role_breakdown": role_permissions_breakdown,
+                    "permission_counts": {
+                        "total": len(all_permissions),
+                        "direct": len(direct_permissions),
+                        "inherited": len(inherited_permissions),
+                        "returned": len(result_permissions),
+                    },
+                    "evaluation_time_ms": evaluation_time_ms,
+                },
+                "options": {
+                    "include_inherited": include_inherited,
+                    "permission_type": permission_type,
+                },
+                "operation": "get_user_permissions",
+                "timestamp": datetime.now(UTC).isoformat(),
+            }
+        }
+
+    def _get_role_direct_permissions(self, role_id: str, tenant_id: str) -> Set[str]:
+        """Get direct permissions for a role (no inheritance)."""
+        query = """
+        SELECT permissions
+        FROM roles
+        WHERE role_id = $1 AND tenant_id = $2 AND is_active = true
+        """
+
+        result = self._db_node.run(
+            query=query, parameters=[role_id, tenant_id], result_format="dict"
+        )
+        role_rows = result.get("data", [])
+        role_data = role_rows[0] if role_rows else None
+
+        if not role_data:
+            return set()
+
+        return set(role_data.get("permissions", []))
 
     def _explain_permission(self, inputs: Dict[str, Any]) -> Dict[str, Any]:
         """Provide detailed explanation of permission logic."""
-        raise NotImplementedError("Explain permission operation will be implemented")
+        user_id = inputs["user_id"]
+        resource_id = inputs["resource_id"]
+        permission = inputs["permission"]
+        tenant_id = inputs.get("tenant_id", "default")
+        context = inputs.get("context", {})
+        include_rules = inputs.get("include_rules", True)
+        include_hierarchy = inputs.get("include_hierarchy", True)
+
+        # Get user context
+        user_context = self._get_user_context(user_id, tenant_id)
+        if not user_context:
+            raise NodeValidationError(f"User not found: {user_id}")
+
+        start_time = datetime.now(UTC)
+
+        # Perform detailed permission check with explanation
+        rbac_result = self._check_rbac_permission(user_context, resource_id, permission)
+        abac_result = self._check_abac_permission(
+            user_context, resource_id, permission, context
+        )
+        final_result = rbac_result and abac_result
+
+        # Build comprehensive explanation
+        explanation = {
+            "permission_granted": final_result,
+            "evaluation_steps": [],
+            "rbac_analysis": {},
+            "abac_analysis": {},
+            "decision_factors": [],
+        }
+
+        # RBAC Analysis
+        user_permissions = self._get_user_effective_permissions(user_context)
+        required_permission = f"{resource_id}:{permission}"
+        wildcard_resource = f"{resource_id}:*"
+        wildcard_permission = f"*:{permission}"
+        global_wildcard = "*:*"
+
+        rbac_matches = []
+        if required_permission in user_permissions:
+            rbac_matches.append({"type": "exact", "permission": required_permission})
+        if wildcard_resource in user_permissions:
+            rbac_matches.append(
+                {"type": "resource_wildcard", "permission": wildcard_resource}
+            )
+        if wildcard_permission in user_permissions:
+            rbac_matches.append(
+                {"type": "permission_wildcard", "permission": wildcard_permission}
+            )
+        if global_wildcard in user_permissions:
+            rbac_matches.append(
+                {"type": "global_wildcard", "permission": global_wildcard}
+            )
+
+        explanation["rbac_analysis"] = {
+            "result": rbac_result,
+            "required_permission": required_permission,
+            "matching_permissions": rbac_matches,
+            "user_roles": user_context.roles,
+            "total_permissions": len(user_permissions),
+        }
+
+        # ABAC Analysis
+        explanation["abac_analysis"] = {
+            "result": abac_result,
+            "context_provided": context,
+            "user_attributes": user_context.attributes,
+            "evaluation_method": "access_control_manager",
+        }
+
+        # Role hierarchy breakdown if requested
+        if include_hierarchy:
+            role_hierarchy_breakdown = {}
+            for role in user_context.roles:
+                direct_perms = self._get_role_direct_permissions(role, tenant_id)
+                inherited_perms = (
+                    self._get_role_permissions(role, tenant_id) - direct_perms
+                )
+
+                role_hierarchy_breakdown[role] = {
+                    "direct_permissions": list(direct_perms),
+                    "inherited_permissions": list(inherited_perms),
+                    "has_required_permission": required_permission
+                    in (direct_perms | inherited_perms),
+                }
+
+            explanation["role_hierarchy"] = role_hierarchy_breakdown
+
+        # Decision factors
+        if rbac_result:
+            explanation["decision_factors"].append(
+                "RBAC: User has required permission through role assignment"
+            )
+        else:
+            explanation["decision_factors"].append(
+                "RBAC: User lacks required permission"
+            )
+
+        if context:
+            if abac_result:
+                explanation["decision_factors"].append(
+                    "ABAC: Context attributes satisfy policy conditions"
+                )
+            else:
+                explanation["decision_factors"].append(
+                    "ABAC: Context attributes do not satisfy policy conditions"
+                )
+        else:
+            explanation["decision_factors"].append(
+                "ABAC: No context provided, defaulting to allow"
+            )
+
+        explanation["decision_factors"].append(
+            f"Final Decision: {'ALLOW' if final_result else 'DENY'}"
+        )
+
+        # Evaluation steps
+        explanation["evaluation_steps"] = [
+            f"1. Retrieved user context for {user_id}",
+            f"2. Evaluated RBAC permissions: {'PASS' if rbac_result else 'FAIL'}",
+            f"3. Evaluated ABAC conditions: {'PASS' if abac_result else 'FAIL'}",
+            f"4. Combined results: {'ALLOW' if final_result else 'DENY'}",
+        ]
+
+        evaluation_time_ms = (datetime.now(UTC) - start_time).total_seconds() * 1000
+
+        return {
+            "result": {
+                "explanation": explanation,
+                "summary": {
+                    "permission_granted": final_result,
+                    "user_id": user_id,
+                    "resource_id": resource_id,
+                    "permission": permission,
+                    "evaluation_time_ms": evaluation_time_ms,
+                },
+                "operation": "explain_permission",
+                "timestamp": datetime.now(UTC).isoformat(),
+            }
+        }
 
     def _validate_conditions(self, inputs: Dict[str, Any]) -> Dict[str, Any]:
         """Validate ABAC conditions and rules."""
-        raise NotImplementedError("Validate conditions operation will be implemented")
+        conditions = inputs.get("conditions", [])
+        context = inputs.get("context", {})
+        user_id = inputs.get("user_id")
+        tenant_id = inputs.get("tenant_id", "default")
+        validate_syntax = inputs.get("validate_syntax", True)
+        test_evaluation = inputs.get("test_evaluation", True)
+
+        start_time = datetime.now(UTC)
+        validation_results = {
+            "valid_conditions": [],
+            "invalid_conditions": [],
+            "syntax_errors": [],
+            "evaluation_errors": [],
+            "total_conditions": len(conditions),
+            "valid_count": 0,
+            "invalid_count": 0,
+        }
+
+        # Get user context for testing if user_id provided
+        user_context = None
+        if user_id:
+            user_context = self._get_user_context(user_id, tenant_id)
+
+        for i, condition in enumerate(conditions):
+            condition_result = {
+                "index": i,
+                "condition": condition,
+                "valid": True,
+                "errors": [],
+            }
+
+            # Validate syntax if requested
+            if validate_syntax:
+                try:
+                    # Basic structure validation
+                    if not isinstance(condition, dict):
+                        condition_result["errors"].append(
+                            "Condition must be a dictionary"
+                        )
+                        condition_result["valid"] = False
+                    else:
+                        required_fields = ["attribute", "operator", "value"]
+                        for field in required_fields:
+                            if field not in condition:
+                                condition_result["errors"].append(
+                                    f"Missing required field: {field}"
+                                )
+                                condition_result["valid"] = False
+
+                        # Validate operator
+                        valid_operators = [
+                            "eq",
+                            "ne",
+                            "lt",
+                            "le",
+                            "gt",
+                            "ge",
+                            "in",
+                            "not_in",
+                            "contains",
+                            "regex",
+                        ]
+                        if (
+                            "operator" in condition
+                            and condition["operator"] not in valid_operators
+                        ):
+                            condition_result["errors"].append(
+                                f"Invalid operator: {condition['operator']}. Valid operators: {valid_operators}"
+                            )
+                            condition_result["valid"] = False
+
+                except Exception as e:
+                    condition_result["errors"].append(
+                        f"Syntax validation error: {str(e)}"
+                    )
+                    condition_result["valid"] = False
+
+            # Test evaluation if requested and syntax is valid
+            if test_evaluation and condition_result["valid"] and context:
+                try:
+                    # Create an AttributeCondition for testing
+                    attr_condition = AttributeCondition(
+                        attribute=condition["attribute"],
+                        operator=AttributeOperator(condition["operator"]),
+                        value=condition["value"],
+                    )
+
+                    # Test evaluation with provided context
+                    test_context = context.copy()
+                    if user_context:
+                        test_context.update(user_context.attributes)
+
+                    # Use the attribute evaluator to test
+                    result = self._attribute_evaluator.evaluate_condition(
+                        attr_condition, test_context
+                    )
+                    condition_result["evaluation_result"] = result
+                    condition_result["test_context"] = test_context
+
+                except Exception as e:
+                    condition_result["errors"].append(f"Evaluation error: {str(e)}")
+                    condition_result["valid"] = False
+                    validation_results["evaluation_errors"].append(
+                        {"condition_index": i, "error": str(e)}
+                    )
+
+            # Categorize result
+            if condition_result["valid"]:
+                validation_results["valid_conditions"].append(condition_result)
+                validation_results["valid_count"] += 1
+            else:
+                validation_results["invalid_conditions"].append(condition_result)
+                validation_results["invalid_count"] += 1
+
+                if condition_result["errors"]:
+                    validation_results["syntax_errors"].extend(
+                        [
+                            {"condition_index": i, "error": error}
+                            for error in condition_result["errors"]
+                        ]
+                    )
+
+        evaluation_time_ms = (datetime.now(UTC) - start_time).total_seconds() * 1000
+
+        return {
+            "result": {
+                "validation": validation_results,
+                "summary": {
+                    "all_valid": validation_results["invalid_count"] == 0,
+                    "success_rate": validation_results["valid_count"]
+                    / max(validation_results["total_conditions"], 1)
+                    * 100,
+                    "evaluation_time_ms": evaluation_time_ms,
+                },
+                "options": {
+                    "validate_syntax": validate_syntax,
+                    "test_evaluation": test_evaluation,
+                    "context_provided": bool(context),
+                    "user_context_used": user_context is not None,
+                },
+                "operation": "validate_conditions",
+                "timestamp": datetime.now(UTC).isoformat(),
+            }
+        }
 
     def _check_hierarchical(self, inputs: Dict[str, Any]) -> Dict[str, Any]:
         """Check permissions with hierarchical resource access."""
-        raise NotImplementedError("Check hierarchical operation will be implemented")
+        user_id = inputs["user_id"]
+        resource_path = inputs["resource_id"]  # e.g., "org/team/project/workflow"
+        permission = inputs["permission"]
+        tenant_id = inputs.get("tenant_id", "default")
+        context = inputs.get("context", {})
+        check_inheritance = inputs.get("check_inheritance", True)
+
+        # Get user context
+        user_context = self._get_user_context(user_id, tenant_id)
+        if not user_context:
+            raise NodeValidationError(f"User not found: {user_id}")
+
+        start_time = datetime.now(UTC)
+
+        # Parse resource hierarchy
+        resource_parts = resource_path.split("/")
+        hierarchical_checks = []
+
+        # Check permission at each level if inheritance is enabled
+        if check_inheritance:
+            # Check from most specific to most general
+            for i in range(len(resource_parts), 0, -1):
+                partial_path = "/".join(resource_parts[:i])
+
+                # Check exact permission
+                exact_check = self._check_rbac_permission(
+                    user_context, partial_path, permission
+                )
+
+                # Check wildcard permission at this level
+                wildcard_check = self._check_rbac_permission(
+                    user_context, partial_path, "*"
+                )
+
+                hierarchical_checks.append(
+                    {
+                        "resource_level": partial_path,
+                        "depth": i,
+                        "exact_permission": exact_check,
+                        "wildcard_permission": wildcard_check,
+                        "grants_access": exact_check or wildcard_check,
+                    }
+                )
+
+                # If we found access at this level, we can stop (inheritance works)
+                if exact_check or wildcard_check:
+                    break
+        else:
+            # Only check the exact resource path
+            exact_check = self._check_rbac_permission(
+                user_context, resource_path, permission
+            )
+            wildcard_check = self._check_rbac_permission(
+                user_context, resource_path, "*"
+            )
+
+            hierarchical_checks.append(
+                {
+                    "resource_level": resource_path,
+                    "depth": len(resource_parts),
+                    "exact_permission": exact_check,
+                    "wildcard_permission": wildcard_check,
+                    "grants_access": exact_check or wildcard_check,
+                }
+            )
+
+        # Determine if access is granted
+        access_granted = any(check["grants_access"] for check in hierarchical_checks)
+
+        # Find the granting level
+        granting_level = None
+        for check in hierarchical_checks:
+            if check["grants_access"]:
+                granting_level = check["resource_level"]
+                break
+
+        # Perform ABAC check if context provided
+        abac_result = True
+        if context:
+            abac_result = self._check_abac_permission(
+                user_context, resource_path, permission, context
+            )
+
+        final_result = access_granted and abac_result
+
+        evaluation_time_ms = (datetime.now(UTC) - start_time).total_seconds() * 1000
+
+        return {
+            "result": {
+                "hierarchical_check": {
+                    "allowed": final_result,
+                    "rbac_result": access_granted,
+                    "abac_result": abac_result,
+                    "user_id": user_id,
+                    "resource_path": resource_path,
+                    "permission": permission,
+                    "granting_level": granting_level,
+                    "inheritance_used": check_inheritance
+                    and granting_level != resource_path,
+                    "hierarchy_checks": hierarchical_checks,
+                    "evaluation_time_ms": evaluation_time_ms,
+                },
+                "options": {
+                    "check_inheritance": check_inheritance,
+                    "context_provided": bool(context),
+                },
+                "operation": "check_hierarchical",
+                "timestamp": datetime.now(UTC).isoformat(),
+            }
+        }
+
+    def _create_audit_log(
+        self,
+        user_id: str,
+        action: str,
+        resource_type: str,
+        resource_id: str,
+        details: Dict[str, Any],
+        success: bool,
+        tenant_id: str,
+    ):
+        """Create an audit log entry for the permission check."""
+        try:
+            audit_query = """
+            INSERT INTO admin_audit_log (
+                user_id, action, resource_type, resource_id,
+                operation, details, success, tenant_id, created_at
+            ) VALUES ($1, $2, $3, $4, $5, $6, $7, $8, $9)
+            """
+
+            self._db_node.run(
+                query=audit_query,
+                parameters=[
+                    user_id,
+                    action,
+                    resource_type,
+                    resource_id,
+                    "permission_check",
+                    json.dumps(details),
+                    success,
+                    tenant_id,
+                    datetime.now(UTC),
+                ],
+            )
+        except Exception as e:
+            self.logger.warning(f"Failed to create audit log: {e}")
+            # Don't fail the permission check if audit logging fails