Skip to content
This repository was archived by the owner on Mar 4, 2026. It is now read-only.

fix: filter guardrails based on the source#21

Merged
ctiliescuuipath merged 2 commits intomainfrom
fix_guardrails_filtering
Jan 14, 2026
Merged

fix: filter guardrails based on the source#21
ctiliescuuipath merged 2 commits intomainfrom
fix_guardrails_filtering

Conversation

@ctiliescuuipath
Copy link
Copy Markdown
Contributor

@ctiliescuuipath ctiliescuuipath commented Jan 14, 2026 •
edited
Loading

  • Enhanced _has_output_dependent_rule with configurable universal_rules_apply_to_values parameter for flexible pre/post execution handling
  • Refactored AllFieldsSelector from single source to sources list, enabling multiple field source specifications
  • Improved validation messages to clearly explain why evaluation was skipped (e.g., "output-dependent rules will be evaluated during post-execution" vs generic "all rules passed")

@ctiliescuuipath ctiliescuuipath force-pushed the fix_guardrails_filtering branch from 931e94c to a45477e Compare January 14, 2026 13:03
valentinabojan
valentinabojan reviewed Jan 14, 2026
View reviewed changes
Comment thread src/uipath/core/guardrails/_deterministic_guardrails_service.py Outdated
valentinabojan
valentinabojan reviewed Jan 14, 2026
View reviewed changes
Comment thread src/uipath/core/guardrails/_deterministic_guardrails_service.py Outdated
@ctiliescuuipath ctiliescuuipath merged commit d8bb43c into main Jan 14, 2026
13 checks passed
@cristipufu cristipufu deleted the fix_guardrails_filtering branch February 17, 2026 15:02
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@radu-mocanu radu-mocanu Awaiting requested review from radu-mocanu

1 more reviewer

@valentinabojan valentinabojan valentinabojan approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@ctiliescuuipath @valentinabojan