fix: use omit() to properly remove conditionals from ReadActionConfig

The .extend() method in Zod merges fields rather than replacing them.
When ActionConfigBase has conditionals defined, extending with conditionals: z.never()
creates a conflict. Using .omit({conditionals: true}) first removes the field completely,
then .extend() adds only the fields we want without the conditionals field.

This ensures READ actions truly cannot have conditionals, fixing the validation error:
'expected never, received array' at path actions[0].conditionals

Author: Siyasanga

packages/commons/src/conditionals/validate.ts

@@ -287,7 +287,7 @@ function isActionConditionMet(
   context: ValidatorContext,
   conditionalType: ConditionalType
 ) {
-  if (!actionConfig.conditionals) {
+  if (!('conditionals' in actionConfig) || !actionConfig.conditionals) {
     return true
   }
 

packages/commons/src/events/ActionConfig.ts

@@ -53,15 +53,11 @@ const DeclarationActionBase = ActionConfigBase.extend({
   deduplication: DeduplicationConfig.optional()
 })
 
-const ReadActionConfig = ActionConfigBase.extend({
+const ReadActionConfig = ActionConfigBase.omit({ conditionals: true }).extend({
   type: z.literal(ActionType.READ),
   review: DeclarationReviewConfig.describe(
     'Configuration of the review page for read-only view.'
-  ),
-  conditionals: z
-    .never()
-    .optional()
-    .describe('Read-action can not be disabled or hidden with conditionals.')
+  )
 })
 
 const DeclareConfig = DeclarationActionBase.extend({