uipath-core 0.1.7__py3-none-any.whl → 0.1.9__py3-none-any.whl

uipath/core/guardrails/_deterministic_guardrails_service.py

@@ -40,7 +40,7 @@ class DeterministicGuardrailsService(BaseModel):
         if has_output_rule:
             return GuardrailValidationResult(
                 result=GuardrailValidationResultType.PASSED,
-                reason="Guardrail contains output-dependent rules that will be evaluated during post-execution",
+                reason="No rules to apply for input data.",
             )
         return self._evaluate_deterministic_guardrail(
             input_data=input_data,
@@ -66,7 +66,7 @@ class DeterministicGuardrailsService(BaseModel):
         if not has_output_rule:
             return GuardrailValidationResult(
                 result=GuardrailValidationResultType.PASSED,
-                reason="Guardrail contains only input-dependent rules that were evaluated during pre-execution",
+                reason="No rules to apply for output data.",
             )
 
         return self._evaluate_deterministic_guardrail(
@@ -117,7 +117,12 @@ class DeterministicGuardrailsService(BaseModel):
         output_data: dict[str, Any],
         guardrail: DeterministicGuardrail,
     ) -> GuardrailValidationResult:
-        """Evaluate deterministic guardrail rules against input and output data."""
+        """Evaluate deterministic guardrail rules against input and output data.
+
+        Validation fails only if ALL guardrail rules are violated.
+        """
+        validated_conditions: list[str] = []
+
         for rule in guardrail.rules:
             if isinstance(rule, WordRule):
                 passed, reason = evaluate_word_rule(rule, input_data, output_data)
@@ -132,14 +137,25 @@ class DeterministicGuardrailsService(BaseModel):
                     result=GuardrailValidationResultType.VALIDATION_FAILED,
                     reason=f"Unknown rule type: {type(rule)}",
                 )
-
-            if not passed:
+            validated_conditions.append(reason)
+            if passed:
                 return GuardrailValidationResult(
-                    result=GuardrailValidationResultType.VALIDATION_FAILED,
-                    reason=reason or "Rule validation failed",
+                    result=GuardrailValidationResultType.PASSED,
+                    reason=reason,
                 )
 
+        has_always_rule = any(
+            condition == "Always rule enforced" for condition in validated_conditions
+        )
+
+        validated_conditions_str = ", ".join(validated_conditions)
+        final_reason = (
+            "Always rule enforced"
+            if has_always_rule
+            else f"Data matched all guardrail conditions: [{validated_conditions_str}]"
+        )
+
         return GuardrailValidationResult(
-            result=GuardrailValidationResultType.PASSED,
-            reason="All deterministic guardrail rules passed",
+            result=GuardrailValidationResultType.VALIDATION_FAILED,
+            reason=final_reason,
         )

uipath/core/guardrails/_evaluators.py

@@ -159,24 +159,44 @@ def get_fields_from_selector(
     return fields
 
 
-def format_guardrail_error_message(
+def format_guardrail_passed_validation_result_message(
     field_ref: FieldReference,
-    operator: str,
-    expected_value: str | None = None,
+    operator: str | None,
+    rule_description: str | None,
 ) -> str:
-    """Format a guardrail error message following the standard pattern."""
+    """Format a guardrail validation result message following the standard pattern."""
     source = "Input" if field_ref.source == FieldSource.INPUT else "Output"
-    message = f"{source} data didn't match the guardrail condition: [{field_ref.path}] comparing function [{operator}]"
-    if expected_value and expected_value.strip():
-        message += f" [{expected_value.strip()}]"
-    return message
+
+    if rule_description:
+        return (
+            f"{source} data didn't match the guardrail condition for field "
+            f"[{field_ref.path}]: {rule_description}"
+        )
+
+    return (
+        f"{source} data didn't match the guardrail condition: "
+        f"[{field_ref.path}] comparing function [{operator}]"
+    )
+
+
+def get_validated_conditions_description(
+    field_path: str,
+    operator: str | None,
+    rule_description: str | None,
+) -> str:
+    if rule_description:
+        return rule_description
+
+    return f"[{field_path}] comparing function [{operator}]"
 
 
 def evaluate_word_rule(
     rule: WordRule, input_data: dict[str, Any], output_data: dict[str, Any]
-) -> tuple[bool, str | None]:
+) -> tuple[bool, str]:
     """Evaluate a word rule against input and output data."""
     fields = get_fields_from_selector(rule.field_selector, input_data, output_data)
+    operator = _humanize_guardrail_func(rule.detects_violation) or "violation check"
+    field_paths = ", ".join({field_ref.path for _, field_ref in fields})
 
     for field_value, field_ref in fields:
         if field_value is None:
@@ -197,22 +217,28 @@ def evaluate_word_rule(
             # If function raises an exception, treat as failure
             violation_detected = True
 
-        if violation_detected:
-            operator = (
-                _humanize_guardrail_func(rule.detects_violation) or "violation check"
+        if not violation_detected:
+            reason = format_guardrail_passed_validation_result_message(
+                field_ref=field_ref,
+                operator=operator,
+                rule_description=rule.rule_description,
             )
-            reason = format_guardrail_error_message(field_ref, operator, None)
-            return False, reason
+            return True, reason
 
-    return True, "All word rule validations passed"
+    return False, get_validated_conditions_description(
+        field_path=field_paths,
+        operator=operator,
+        rule_description=rule.rule_description,
+    )
 
 
 def evaluate_number_rule(
     rule: NumberRule, input_data: dict[str, Any], output_data: dict[str, Any]
-) -> tuple[bool, str | None]:
+) -> tuple[bool, str]:
     """Evaluate a number rule against input and output data."""
     fields = get_fields_from_selector(rule.field_selector, input_data, output_data)
-
+    operator = _humanize_guardrail_func(rule.detects_violation) or "violation check"
+    field_paths = ", ".join({field_ref.path for _, field_ref in fields})
     for field_value, field_ref in fields:
         if field_value is None:
             continue
@@ -233,24 +259,30 @@ def evaluate_number_rule(
             # If function raises an exception, treat as failure
             violation_detected = True
 
-        if violation_detected:
-            operator = (
-                _humanize_guardrail_func(rule.detects_violation) or "violation check"
+        if not violation_detected:
+            reason = format_guardrail_passed_validation_result_message(
+                field_ref=field_ref,
+                operator=operator,
+                rule_description=rule.rule_description,
             )
-            reason = format_guardrail_error_message(field_ref, operator, None)
-            return False, reason
+            return True, reason
 
-    return True, "All number rule validations passed"
+    return False, get_validated_conditions_description(
+        field_path=field_paths,
+        operator=operator,
+        rule_description=rule.rule_description,
+    )
 
 
 def evaluate_boolean_rule(
     rule: BooleanRule,
     input_data: dict[str, Any],
     output_data: dict[str, Any],
-) -> tuple[bool, str | None]:
+) -> tuple[bool, str]:
     """Evaluate a boolean rule against input and output data."""
     fields = get_fields_from_selector(rule.field_selector, input_data, output_data)
-
+    operator = _humanize_guardrail_func(rule.detects_violation) or "violation check"
+    field_paths = ", ".join({field_ref.path for _, field_ref in fields})
     for field_value, field_ref in fields:
         if field_value is None:
             continue
@@ -270,20 +302,25 @@ def evaluate_boolean_rule(
             # If function raises an exception, treat as failure
             violation_detected = True
 
-        if violation_detected:
-            operator = (
-                _humanize_guardrail_func(rule.detects_violation) or "violation check"
+        if not violation_detected:
+            reason = format_guardrail_passed_validation_result_message(
+                field_ref=field_ref,
+                operator=operator,
+                rule_description=rule.rule_description,
             )
-            reason = format_guardrail_error_message(field_ref, operator, None)
-            return False, reason
+            return True, reason
 
-    return True, "All boolean rule validations passed"
+    return False, get_validated_conditions_description(
+        field_path=field_paths,
+        operator=operator,
+        rule_description=rule.rule_description,
+    )
 
 
 def evaluate_universal_rule(
     rule: UniversalRule,
     output_data: dict[str, Any],
-) -> tuple[bool, str | None]:
+) -> tuple[bool, str]:
     """Evaluate a universal rule against input and output data.
 
     Universal rules trigger based on the apply_to scope and execution phase:
@@ -302,18 +339,18 @@ def evaluate_universal_rule(
     if rule.apply_to == ApplyTo.INPUT:
         # INPUT: triggers in pre-execution, does not trigger in post-execution
         if is_pre_execution:
-            return False, "Universal rule validation triggered (pre-execution, input)"
+            return False, "Always rule enforced"
         else:
-            return True, "Universal rule validation passed (post-execution, input)"
+            return True, "No rules to apply for output data"
     elif rule.apply_to == ApplyTo.OUTPUT:
         # OUTPUT: does not trigger in pre-execution, triggers in post-execution
         if is_pre_execution:
-            return True, "Universal rule validation passed (pre-execution, output)"
+            return True, "No rules to apply for input data"
         else:
-            return False, "Universal rule validation triggered (post-execution, output)"
+            return False, "Always rule enforced"
     elif rule.apply_to == ApplyTo.INPUT_AND_OUTPUT:
         # INPUT_AND_OUTPUT: triggers in both phases
-        return False, "Universal rule validation triggered (input and output)"
+        return False, "Always rule enforced"
     else:
         return False, f"Unknown apply_to value: {rule.apply_to}"
 

uipath/core/guardrails/guardrails.py

@@ -102,6 +102,11 @@ class WordRule(BaseModel):
 
     rule_type: Literal["word"] = Field(alias="$ruleType")
     field_selector: FieldSelector = Field(alias="fieldSelector")
+    rule_description: str | None = Field(
+        default=None,
+        exclude=True,
+        description="Human-friendly description of the rule condition.",
+    )
     detects_violation: Callable[[str], bool] = Field(
         exclude=True,
         description="Function that returns True if the string violates the rule (validation should fail).",
@@ -124,6 +129,11 @@ class NumberRule(BaseModel):
 
     rule_type: Literal["number"] = Field(alias="$ruleType")
     field_selector: FieldSelector = Field(alias="fieldSelector")
+    rule_description: str | None = Field(
+        default=None,
+        exclude=True,
+        description="Human-friendly description of the rule condition.",
+    )
     detects_violation: Callable[[float], bool] = Field(
         exclude=True,
         description="Function that returns True if the number violates the rule (validation should fail).",
@@ -137,6 +147,11 @@ class BooleanRule(BaseModel):
 
     rule_type: Literal["boolean"] = Field(alias="$ruleType")
     field_selector: FieldSelector = Field(alias="fieldSelector")
+    rule_description: str | None = Field(
+        default=None,
+        exclude=True,
+        description="Human-friendly description of the rule condition.",
+    )
     detects_violation: Callable[[bool], bool] = Field(
         exclude=True,
         description="Function that returns True if the boolean violates the rule (validation should fail).",

uipath/core/tracing/decorators.py

@@ -9,7 +9,7 @@ from typing import Any, Callable, Optional
 from opentelemetry import context as context_api
 from opentelemetry import trace
 from opentelemetry.context import _SUPPRESS_INSTRUMENTATION_KEY
-from opentelemetry.trace import NonRecordingSpan, SpanContext, TraceFlags
+from opentelemetry.trace import SpanContext, TraceFlags
 from opentelemetry.trace.status import StatusCode
 
 from uipath.core.tracing._utils import (
@@ -17,7 +17,11 @@ from uipath.core.tracing._utils import (
     set_span_input_attributes,
     set_span_output_attributes,
 )
-from uipath.core.tracing.span_utils import UiPathSpanUtils
+from uipath.core.tracing.span_utils import (
+    ParentedNonRecordingSpan,
+    UiPathSpanUtils,
+    _span_registry,
+)
 
 logger = logging.getLogger(__name__)
 
@@ -50,7 +54,10 @@ def _opentelemetry_traced(
         trace_name = name or func.__name__
 
         def get_span():
+            ctx = UiPathSpanUtils.get_parent_context()
             if not recording:
+                parent_context = trace.get_current_span(ctx).get_span_context()
+
                 # Create a valid but non-sampled trace context
                 # Generate a valid trace ID (not INVALID)
                 trace_id = random.getrandbits(128)
@@ -62,20 +69,24 @@ def _opentelemetry_traced(
                     is_remote=False,
                     trace_flags=TraceFlags(0x00),  # NOT sampled
                 )
-                non_recording = NonRecordingSpan(non_sampled_context)
+                non_recording = ParentedNonRecordingSpan(
+                    non_sampled_context, parent=parent_context
+                )
 
                 # Make it active so children see it
                 span_cm = trace.use_span(non_recording)
                 span_cm.__enter__()
-                return span_cm, non_recording
 
-            # Normal recording span
-            ctx = UiPathSpanUtils.get_parent_context()
-            span_cm = trace.get_tracer(__name__).start_as_current_span(
-                trace_name, context=ctx
-            )
-            span = span_cm.__enter__()
-            return span_cm, span
+                _span_registry.register_span(non_recording)
+
+                return span_cm, non_recording
+            else:
+                # Normal recording span
+                span_cm = trace.get_tracer(__name__).start_as_current_span(
+                    trace_name, context=ctx
+                )
+                span = span_cm.__enter__()
+                return span_cm, span
 
         # --------- Sync wrapper ---------
         @wraps(func)

uipath/core/tracing/span_utils.py

@@ -4,11 +4,33 @@ import logging
 from typing import Callable, Optional
 
 from opentelemetry import context, trace
-from opentelemetry.trace import Span, set_span_in_context
+from opentelemetry.trace import NonRecordingSpan, Span, set_span_in_context
 
 logger = logging.getLogger(__name__)
 
 
+class ParentedNonRecordingSpan(NonRecordingSpan):
+    """Non-recording span with explicit parent tracking.
+
+    Extends NonRecordingSpan to include a parent attribute, allowing the SpanRegistry
+    to properly track parent-child relationships for non-recording spans.
+    This is necessary because NonRecordingSpan instances created directly don't have
+    their parent automatically set like normal recording spans do.
+    """
+
+    def __init__(
+        self, context: trace.SpanContext, parent: Optional[trace.SpanContext] = None
+    ):
+        """Initialize a parented non-recording span.
+
+        Args:
+            context: The SpanContext for this span
+            parent: Optional parent SpanContext
+        """
+        super().__init__(context)
+        self.parent = parent
+
+
 class SpanRegistry:
     """Registry to track all spans and their parent relationships."""
 
@@ -309,4 +331,4 @@ class UiPathSpanUtils:
         return UiPathSpanUtils._current_span_ancestors_provider
 
 
-__all__ = ["UiPathSpanUtils"]
+__all__ = ["ParentedNonRecordingSpan", "UiPathSpanUtils"]

{uipath_core-0.1.7.dist-info → uipath_core-0.1.9.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: uipath-core
-Version: 0.1.7
+Version: 0.1.9
 Summary: UiPath Core abstractions
 Project-URL: Homepage, https://uipath.com
 Project-URL: Repository, https://github.com/UiPath/uipath-core-python

{uipath_core-0.1.7.dist-info → uipath_core-0.1.9.dist-info}/RECORD

@@ -15,17 +15,17 @@ uipath/core/chat/tool.py,sha256=6e5pyX3hOWM5fIzr_fdG49Mbzz6XzJD3nsmha-yGa2k,2308
 uipath/core/errors/__init__.py,sha256=gjxdLibZ0fjwgzPuLJY04P8dIX9rbSM2wQ97jP34ucE,278
 uipath/core/errors/errors.py,sha256=5LajjuTfNW82ju07wT5mD3tXk0S-Ju7OqJqQpPN0F6g,486
 uipath/core/guardrails/__init__.py,sha256=hUCmD4y5te2iy01YnJlBuf2RWvqxmsNzoyOamXLXf2E,1028
-uipath/core/guardrails/_deterministic_guardrails_service.py,sha256=uX8f2HSGZ1bf5QFM9fnz0CvglxI3UIqY4OhHrKzemaU,5967
-uipath/core/guardrails/_evaluators.py,sha256=ovmVm-8iB8Pm9arjG7mHM9-GIRkrG3V6oHRPHKxZVO0,14601
-uipath/core/guardrails/guardrails.py,sha256=DraeFkoDKVDnc0EKdFYYP29lQu7U2hneTsj1dxveHU4,4935
+uipath/core/guardrails/_deterministic_guardrails_service.py,sha256=61ROXYmX3rjBfFUp7E83fm4Lk7h1DlJeukpZm7uzVqQ,6355
+uipath/core/guardrails/_evaluators.py,sha256=10tIRUufxoy9MkZPb-ytjsCSCIfIqQSOYXyEl4G7PSw,15649
+uipath/core/guardrails/guardrails.py,sha256=sTxsNHilEX908aF-WtQWVOQ4dCCwMgAepDebH35ngvo,5430
 uipath/core/tracing/__init__.py,sha256=1XNLYZ4J76XkRrizGO486mS6yxzVXUbrldpvxTyJe3E,483
 uipath/core/tracing/_utils.py,sha256=FiCFGOFa4czruhlSF87Q5Q4jX9KKPHZiw8k14K7W5v4,6636
-uipath/core/tracing/decorators.py,sha256=nE57gAb5Ul6c3ep2Nkkqr40SO_eiCoJL8_4VQ0AiHjY,11643
+uipath/core/tracing/decorators.py,sha256=JDNULkUu-Ufg8pJslG4i6Q2pmqaGNDS8NFRPgFs29Dw,11937
 uipath/core/tracing/exporters.py,sha256=FClouEEQfk3F8J7G_NFoarDJM3R0-gA5jUxA5xRHx5s,1562
 uipath/core/tracing/processors.py,sha256=R_652rtjPmfpUtaXoIcmfZrRZylVXFRNwjOmJUUxOQw,1408
-uipath/core/tracing/span_utils.py,sha256=WYBrd6ZbawAs7r1Js-Zvo9_8GzkD9LhHNOls00bK_xI,12235
+uipath/core/tracing/span_utils.py,sha256=LZXNdnI0-fhKe49CLPsvMJIfh9zdzk8rK4g4YN5RfDU,13064
 uipath/core/tracing/trace_manager.py,sha256=51rscJcepkTK4bWoCZdE-DFc9wt2F-aSuFBaSXmkHl0,3130
-uipath_core-0.1.7.dist-info/METADATA,sha256=vpXcfigfRmjzvhsyAm8fFs8dEu7BAJ6yAmv_ltKxhzY,938
-uipath_core-0.1.7.dist-info/WHEEL,sha256=WLgqFyCfm_KASv4WHyYy0P3pM_m7J5L9k2skdKLirC8,87
-uipath_core-0.1.7.dist-info/licenses/LICENSE,sha256=-KBavWXepyDjimmzH5fVAsi-6jNVpIKFc2kZs0Ri4ng,1058
-uipath_core-0.1.7.dist-info/RECORD,,
+uipath_core-0.1.9.dist-info/METADATA,sha256=wLk7EKaDR01IqA7RvbhRzw13dw2DRX6twknAm793_OY,938
+uipath_core-0.1.9.dist-info/WHEEL,sha256=WLgqFyCfm_KASv4WHyYy0P3pM_m7J5L9k2skdKLirC8,87
+uipath_core-0.1.9.dist-info/licenses/LICENSE,sha256=-KBavWXepyDjimmzH5fVAsi-6jNVpIKFc2kZs0Ri4ng,1058
+uipath_core-0.1.9.dist-info/RECORD,,