fix(knowledge): address PR feedback on Cohere reranker controls

- Drop required:true on apiKey field — server has BYOK→env→rotation fallback
  chain, so self-hosted users with COHERE_API_KEY env should not be blocked
- Drop .min(1) on rerankerApiKey contract field so empty strings coerce to
  undefined via the transform (matches the existing query field pattern)
- Log a warning when rerankerInputCount is clamped up to topK so users notice
  their setting was overridden

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

Author: waleedlatif1

apps/sim/app/api/knowledge/search/route.ts

@@ -251,9 +251,15 @@ export const POST = withRouteHandler(async (request: NextRequest) => {
      * Cap at 100 to bound Cohere request cost (1 search unit = ≤100 docs). When the caller
      * supplies `rerankerInputCount`, honor it but never let it drop below `topK`
      * (which would defeat the purpose) or exceed 100 (which would split into >1 search units). */
+    const rawInputCount = validatedData.rerankerInputCount
+    if (useReranker && rawInputCount !== undefined && rawInputCount < validatedData.topK) {
+      logger.warn(
+        `[${requestId}] rerankerInputCount (${rawInputCount}) is below topK (${validatedData.topK}); raising to topK`
+      )
+    }
     const candidateTopK = useReranker
-      ? validatedData.rerankerInputCount !== undefined
-        ? Math.min(100, Math.max(validatedData.topK, validatedData.rerankerInputCount))
+      ? rawInputCount !== undefined
+        ? Math.min(100, Math.max(validatedData.topK, rawInputCount))
         : Math.min(100, validatedData.topK * 4)
       : validatedData.topK
 

apps/sim/blocks/blocks/knowledge.ts

@@ -125,7 +125,6 @@ export const KnowledgeBlock: BlockConfig = {
       placeholder: 'Enter your Cohere API key',
       password: true,
       connectionDroppable: false,
-      required: true,
       condition: getCohereRerankerApiKeyCondition(),
     },
 

apps/sim/lib/api/contracts/knowledge/search.ts

@@ -51,7 +51,6 @@ export const knowledgeSearchBodySchema = z
       .transform((val) => val ?? undefined),
     rerankerApiKey: z
       .string()
-      .min(1, 'rerankerApiKey cannot be empty')
       .optional()
       .nullable()
       .transform((val) => val || undefined),