Skip to content

Conversation

@ctiliescuuipath
Copy link
Contributor

No description provided.

model_config = ConfigDict(populate_by_name=True, extra="allow")


ValidatorParameter = Annotated[

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ValidatorParameter, EnumListParameterValue, MapEnumParameterValue, NumberParameterValue should stay in uipath-python. They are related to Out of the Box guardrails.

model_config = ConfigDict(populate_by_name=True, extra="allow")


class BuiltInValidatorGuardrail(BaseGuardrail):

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Shouldn't this also remain in uipath-python?

)


class CustomGuardrailsService(BaseModel):
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

let's find a better name for this, eg: GuardrailValidator, GuardrailEvaluator etc - also get rid of "custom" from models and everything

Copy link

@valentinabojan valentinabojan Dec 11, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@cristipufu We need a differentiator to distinguish between Custom|Deterministic guardrails and BuiltInValidator guardrails. If Custom sounds bad, what about DeterministicGuardrails?

Copy link
Contributor Author

@ctiliescuuipath ctiliescuuipath Dec 11, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@cristipufu we still need this : guardrail_type: Literal["custom"] = Field(alias="custom") for agent.json deserialization. All other references where renamed to DeterministicGuardrails

@ctiliescuuipath ctiliescuuipath force-pushed the AL-197_custom_guardrails branch 2 times, most recently from 396a1a3 to 6a8cfb7 Compare December 11, 2025 13:41
@ctiliescuuipath ctiliescuuipath merged commit 20261d1 into main Dec 11, 2025
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants