From 6da58254bc96e6b321f2896f88671984388debdc Mon Sep 17 00:00:00 2001 From: arabcoders Date: Sun, 18 Jan 2026 22:57:39 +0300 Subject: [PATCH] refactor: update validation logic to require cli or extras in Condition model --- app/features/conditions/schemas.py | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/app/features/conditions/schemas.py b/app/features/conditions/schemas.py index 08f552d2..b2abdfff 100644 --- a/app/features/conditions/schemas.py +++ b/app/features/conditions/schemas.py @@ -60,9 +60,9 @@ class Condition(BaseModel): return value @model_validator(mode="after") - def _validate_filter_or_extras(self) -> Condition: - if not self.filter and not self.extras: - msg: str = "Either filter or extras must be set." + def _validate_cli_or_extras(self) -> Condition: + if not self.cli and not self.extras: + msg: str = "Either cli or extras must be set." raise ValueError(msg) return self @@ -81,8 +81,7 @@ class ConditionPatch(Condition): description: str | None = None @model_validator(mode="after") - def _validate_filter_or_extras(self) -> ConditionPatch: - # Skip validation for PATCH - allow partial updates + def _validate_cli_or_extras(self) -> ConditionPatch: return self