fix(enrichment): report rejected optional inputs

Author: TheodoreSpeaks

apps/sim/enrichments/readiness.test.ts

@@ -51,6 +51,20 @@ describe('getEnrichmentReadiness', () => {
     expect(readiness.missingInputs.map((input) => input.id)).toEqual(['domain'])
   })
 
+  it('includes rejected optional values when another provider-specific input is blank', () => {
+    const readiness = getEnrichmentReadiness(workEmailEnrichment, {
+      fullName: 'John Doe',
+      companyDomain: 'https://',
+      linkedinUrl: '',
+    })
+
+    expect(readiness.ready).toBe(false)
+    expect(readiness.missingInputs.map((input) => input.id)).toEqual([
+      'companyDomain',
+      'linkedinUrl',
+    ])
+  })
+
   it('formats select-backed row inputs before checking provider request builders', () => {
     const columns: ColumnDefinition[] = [
       {

apps/sim/enrichments/readiness.ts

@@ -38,12 +38,10 @@ export function getEnrichmentReadiness(
   const ready = enrichment.providers.some((provider) => provider.buildParams(inputs) !== null)
   if (ready) return { ready: true, missingInputs: [] }
 
-  const missingInputs = enrichment.inputs.filter((input) => isEmpty(inputs[input.id]))
+  const providerSpecificInputs = enrichment.inputs.filter((input) => !input.required)
   return {
     ready: false,
-    /** If every input is present but every provider rejects the values, surface
-     * all fields as actionable rather than rendering a reasonless wait state. */
-    missingInputs: missingInputs.length > 0 ? missingInputs : enrichment.inputs,
+    missingInputs: providerSpecificInputs.length > 0 ? providerSpecificInputs : enrichment.inputs,
   }
 }