fix(copilot): reject empty required multi-selects

Author: Bill Leoutsakos

apps/sim/lib/workflows/editing/validation.test.ts

@@ -309,6 +309,7 @@ describe('validateInputsForBlock', () => {
       id: 'eventTypes',
       type: 'dropdown' as const,
       multiSelect: true,
+      required: true,
       options: [
         { id: 'created', label: 'Created' },
         { id: 'updated', label: 'Updated' },
@@ -332,6 +333,9 @@ describe('validateInputsForBlock', () => {
         'quickbooks-1'
       ).valid
     ).toBe(false)
+    expect(
+      validateValueForSubBlockType(config, [], 'eventTypes', 'quickbooks', 'quickbooks-1').valid
+    ).toBe(false)
   })
 
   it('accepts condition-input arrays with arbitrary item ids', () => {

apps/sim/lib/workflows/editing/validation.ts

@@ -391,7 +391,9 @@ export function validateValueForSubBlockType(
       if (options && Array.isArray(options)) {
         const validIds = options.map((opt) => opt.id)
         const values = subBlockConfig.multiSelect && Array.isArray(value) ? value : [value]
-        if (values.some((item) => !validIds.includes(item))) {
+        const isEmptyRequiredMultiSelect =
+          subBlockConfig.multiSelect && subBlockConfig.required && values.length === 0
+        if (isEmptyRequiredMultiSelect || values.some((item) => !validIds.includes(item))) {
           return {
             valid: false,
             error: {