fix(knowledge): judge a caller's reranker key by the resolver's policy, and let a custom window be cleared

- the credential check tries the platform key before a workspace lookup, and counts a caller's own key only off hosted Sim, as the resolver does
- a custom window with no days yet searches nothing and says so; another window drops the days; the picker's Clear reaches the URL through a forwarded `onClear`

Author: waleedlatif1

apps/sim/app/workspace/[workspaceId]/home/components/knowledge-search-results/knowledge-search-results.test.tsx

@@ -225,6 +225,47 @@ describe('result paging and the custom window', () => {
     expect(mocks.search.mock.calls.at(-1)![3]).toBe(20)
   })
 
+  it('drops the custom days when another window is chosen', async () => {
+    mocks.overview.mockReturnValue({ data: { providers: [], hasSearchableDocuments: true } })
+    mocks.search.mockReturnValue({
+      data: { query: 'launch', results: [], retrieval: { status: 'complete', timedOutLegs: [] } },
+      isPending: false,
+      isFetching: false,
+      isPlaceholderData: false,
+      isError: false,
+      refetch: mocks.retry,
+    })
+    await render(undefined, '?updated=custom&from=2026-09-01&to=2026-09-10')
+    expect(mocks.search.mock.calls.at(-1)![2]).toHaveProperty('modifiedBefore')
+    const anyTime = [...container.querySelectorAll('button')].find(
+      (b) => b.textContent === 'Any time'
+    )!
+    await act(async () => anyTime.click())
+    expect(mocks.search.mock.calls.at(-1)![2]).toEqual({})
+    await act(async () =>
+      [...container.querySelectorAll('button')]
+        .find((b) => b.textContent === 'Custom range')!
+        .click()
+    )
+    /** Back on the custom window, the old days are gone: nothing is searched until new ones are chosen. */
+    expect(mocks.search.mock.calls.at(-1)![1]).toBe('')
+  })
+
+  it('searches nothing while a custom window has no days yet', async () => {
+    mocks.overview.mockReturnValue({ data: { providers: [], hasSearchableDocuments: true } })
+    mocks.search.mockReturnValue({
+      data: undefined,
+      isPending: true,
+      isFetching: false,
+      isPlaceholderData: false,
+      isError: false,
+      refetch: mocks.retry,
+    })
+    await render(undefined, '?updated=custom')
+    expect(mocks.search.mock.calls.at(-1)![1]).toBe('')
+    expect(container.textContent).toContain('Choose the days to search.')
+  })
+
   it('searches a custom window as an inclusive range of days', async () => {
     mocks.overview.mockReturnValue({ data: { providers: [], hasSearchableDocuments: true } })
     mocks.search.mockReturnValue({

apps/sim/app/workspace/[workspaceId]/home/components/knowledge-search-results/knowledge-search-results.tsx

@@ -167,6 +167,8 @@ function SearchResults({ scope, query, onSummarize }: SearchResultsProps) {
   }
   const filtersKey = JSON.stringify(searchFilters)
   const expanded = expandedFor === filtersKey
+  /** A custom window with no days chosen yet is not "any time": nothing is searched until it has them. */
+  const awaitingRange = custom && !filters.from && !filters.to
   const {
     data: search,
     isPending,
@@ -176,7 +178,7 @@ function SearchResults({ scope, query, onSummarize }: SearchResultsProps) {
     refetch: refetchSearch,
   } = useWorkspaceKnowledgeSearch(
     scope,
-    query,
+    awaitingRange ? '' : query,
     searchFilters,
     expanded
       ? WORKSPACE_KNOWLEDGE_SEARCH_LIMITS.expanded
@@ -231,7 +233,11 @@ function SearchResults({ scope, query, onSummarize }: SearchResultsProps) {
     <div className='flex flex-col'>
       <div className='flex items-center gap-2 px-2 py-2'>
         <div className='min-w-0 flex-1'>
-          {fetching || (pending && !failed) ? (
+          {awaitingRange ? (
+            <p role='status' className='text-[var(--text-muted)] text-caption'>
+              Choose the days to search.
+            </p>
+          ) : fetching || (pending && !failed) ? (
             <ActivityStatus label={pending ? 'Searching…' : 'Updating results…'} isActive />
           ) : (
             <p role='status' className='text-[var(--text-muted)] text-caption'>
@@ -292,7 +298,13 @@ function SearchResults({ scope, query, onSummarize }: SearchResultsProps) {
               shape='round'
               active={filters.updated === window.id}
               aria-pressed={filters.updated === window.id}
-              onClick={() => setFilters({ updated: window.id })}
+              onClick={() =>
+                setFilters(
+                  window.id === 'custom'
+                    ? { updated: window.id }
+                    : { updated: window.id, from: null, to: null }
+                )
+              }
             >
               {window.label}
             </Chip>
@@ -301,11 +313,12 @@ function SearchResults({ scope, query, onSummarize }: SearchResultsProps) {
             <ChipDatePicker
               mode='range'
               label='Updated between'
-              startDate={filters.from?.toISOString()}
-              endDate={filters.to?.toISOString()}
+              startDate={filters.from?.toISOString().slice(0, 10)}
+              endDate={filters.to?.toISOString().slice(0, 10)}
               onRangeChange={(start, end) =>
                 void setFilters({ from: new Date(start), to: new Date(end) })
               }
+              onClear={() => void setFilters({ from: null, to: null })}
             />
           )}
         </div>

apps/sim/lib/knowledge/application/search.test.ts

@@ -873,6 +873,8 @@ describe('knowledge search application use case', () => {
 
       const result = await rerankedSearch(true)
 
+      /** A caller's own key is judged by the same policy the resolver applies, not taken on faith. */
+      expect(mocks.hasRerankerCredential).toHaveBeenLastCalledWith(expect.anything(), undefined)
       expect(mocks.rerank).not.toHaveBeenCalled()
       expect(result.rerankerStatus).toBe('unavailable')
       expect(result.results[0]).not.toHaveProperty('rerankerScore')

apps/sim/lib/knowledge/application/search.ts

@@ -394,7 +394,7 @@ const searchKnowledgeUseCase = defineAuthorizedKnowledgeUseCase({
     /** A surface may ask to rerank; without a key for the workspace or the platform there is nothing to ask. */
     const useReranker =
       Boolean(input.rerankerEnabled && hasQuery) &&
-      (Boolean(input.rerankerApiKey) || (await hasRerankerCredential(context.workspaceId)))
+      (await hasRerankerCredential(context.workspaceId, input.rerankerApiKey))
     const candidateTopK = useReranker
       ? input.rerankerInputCount !== undefined
         ? Math.min(

apps/sim/lib/knowledge/reranker.ts

@@ -34,9 +34,14 @@ const RERANK_OPERATION_TIMEOUT_MS = 30_000
  * Whether a search for this workspace could be reranked at all: a workspace key, or one of the
  * platform's. A surface that reranks "when configured" asks this before spending a call on it.
  */
-export async function hasRerankerCredential(workspaceId?: string): Promise<boolean> {
-  if (workspaceId && (await getBYOKKey(workspaceId, 'cohere'))) return true
-  return Boolean(env.COHERE_API_KEY) || hasRotatingApiKey('cohere')
+export async function hasRerankerCredential(
+  workspaceId?: string,
+  userApiKey?: string
+): Promise<boolean> {
+  /** The same policy as the key resolver: a caller's own key counts only off hosted Sim. */
+  if (!isHosted && userApiKey) return true
+  if (env.COHERE_API_KEY || hasRotatingApiKey('cohere')) return true
+  return Boolean(workspaceId && (await getBYOKKey(workspaceId, 'cohere')))
 }
 
 /**

packages/emcn/src/components/chip-date-picker/chip-date-picker.tsx

@@ -62,6 +62,8 @@ interface ChipDatePickerRangeProps extends ChipDatePickerBaseProps {
   showTime?: boolean
   /** Called on Apply with the ordered range bounds. */
   onRangeChange: (start: string, end: string) => void
+  /** Called on Clear, so a committed range can be dropped by whoever owns it. */
+  onClear?: () => void
 }
 
 export type ChipDatePickerProps = ChipDatePickerSingleProps | ChipDatePickerRangeProps
@@ -161,6 +163,7 @@ const ChipDatePicker = forwardRef<HTMLButtonElement, ChipDatePickerProps>(
                   setOpen(false)
                 }}
                 onCancel={() => setOpen(false)}
+                onClear={props.onClear}
               />
             ) : (
               <Calendar