fix(kb): expire pending connector state

Author: Bill Leoutsakos

apps/sim/hooks/queries/kb/connectors.test.ts

@@ -7,13 +7,14 @@ import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest'
 const mocks = vi.hoisted(() => ({
   requestJson: vi.fn(),
   useInfiniteQuery: vi.fn(),
+  useQuery: vi.fn(),
 }))
 
 vi.mock('@tanstack/react-query', () => ({
   keepPreviousData: Symbol('keepPreviousData'),
   useInfiniteQuery: mocks.useInfiniteQuery,
   useMutation: vi.fn(),
-  useQuery: vi.fn(),
+  useQuery: mocks.useQuery,
   useQueryClient: vi.fn(() => ({ invalidateQueries: vi.fn() })),
 }))
 
@@ -26,7 +27,11 @@ import {
   listKnowledgeConnectorDocumentsContract,
 } from '@/lib/api/contracts/knowledge'
 import { MAX_KNOWLEDGE_CONNECTOR_DOCUMENT_PAGE_SIZE } from '@/lib/knowledge/constants'
-import { isConnectorSyncingOrPending, useConnectorDocuments } from '@/hooks/queries/kb/connectors'
+import {
+  isConnectorSyncingOrPending,
+  useConnectorDocuments,
+  useConnectorList,
+} from '@/hooks/queries/kb/connectors'
 
 const NOW_MS = new Date('2026-08-21T12:00:00.000Z').getTime()
 
@@ -67,6 +72,10 @@ interface ConnectorDocumentsQueryOptions {
   ) => number | undefined
 }
 
+interface ConnectorListQueryOptions {
+  notifyOnChangeProps?: 'all'
+}
+
 describe('isConnectorSyncingOrPending', () => {
   beforeEach(() => {
     vi.spyOn(Date, 'now').mockReturnValue(NOW_MS)
@@ -114,6 +123,19 @@ describe('isConnectorSyncingOrPending', () => {
   )
 })
 
+describe('useConnectorList', () => {
+  beforeEach(() => {
+    vi.clearAllMocks()
+  })
+
+  it('notifies consumers when identical polls complete so pending UI can expire', () => {
+    useConnectorList('knowledge-1')
+
+    const options = mocks.useQuery.mock.calls[0]?.[0] as ConnectorListQueryOptions
+    expect(options.notifyOnChangeProps).toBe('all')
+  })
+})
+
 describe('useConnectorDocuments', () => {
   beforeEach(() => {
     vi.clearAllMocks()

apps/sim/hooks/queries/kb/connectors.ts

@@ -86,6 +86,9 @@ export function useConnectorList(knowledgeBaseId?: string) {
     enabled: Boolean(knowledgeBaseId),
     staleTime: CONNECTOR_LIST_STALE_TIME,
     placeholderData: keepPreviousData,
+    // Pending state is time-based, so identical poll responses must still trigger a render
+    // for consumers to drop the pending UI when its two-minute window expires.
+    notifyOnChangeProps: 'all',
     refetchInterval: (query) => {
       const connectors = query.state.data
       if (!connectors?.length) return false