fix(connectors): order Confluence spaces by name and stop implying a space is missing

Confluence defaults /spaces to id-ascending, which interleaves the site's
personal spaces (one per user) with team spaces in arbitrary order. The selector
drains pages in the background and filters client-side, so on a large site a
space can be absent from the dropdown purely because its id sorts late — it
reads as "this space does not exist" and sends users off to type the key by
hand. Sorting by name makes the drained prefix an alphabetical prefix.

The connector selector field also ignored the drain state the shared hook
already exposes, so a still-filling list showed "No spaces found". It now
reports that the list is still loading, and says so honestly when the drain
stops at its page cap.

Author: waleedlatif1

apps/sim/app/api/tools/confluence/selector-spaces/route.ts

@@ -21,6 +21,9 @@ export const dynamic = 'force-dynamic'
 
 const PAGE_LIMIT = 250
 
+/** Confluence v2 `/spaces` accepts only `id` or `name` (prefix `-` to reverse). */
+const SPACE_SORT_FIELD = 'name'
+
 type SpaceStatus = 'current' | 'archived'
 
 /**
@@ -101,7 +104,19 @@ export const POST = withRouteHandler(async (request: NextRequest) => {
     const baseUrl = `https://api.atlassian.com/ex/confluence/${cloudIdValidation.sanitized}/wiki/api/v2/spaces`
     const { status, inner } = parseCursor(cursor)
 
-    const params = new URLSearchParams({ limit: String(PAGE_LIMIT), status })
+    /**
+     * Confluence defaults to id-ascending, which interleaves the site's personal
+     * spaces (one per user) with team spaces in effectively arbitrary order. The
+     * dropdown drains pages in the background and filters client-side, so with an
+     * id-ordered stream a space can be absent from the list purely because its id
+     * sorts late. Sorting by name makes the drained prefix an alphabetical prefix,
+     * so a space the user is looking for appears where they expect it.
+     */
+    const params = new URLSearchParams({
+      limit: String(PAGE_LIMIT),
+      status,
+      sort: SPACE_SORT_FIELD,
+    })
     if (inner) params.set('cursor', inner)
     const url = `${baseUrl}?${params.toString()}`
 

apps/sim/app/workspace/[workspaceId]/knowledge/[id]/components/connector-selector-field/connector-selector-field.tsx

@@ -62,11 +62,31 @@ export function ConnectorSelectorField({
   }, [field.dependsOn, sourceConfig, configFields, canonicalModes])
 
   const isEnabled = !disabled && !!credentialId && depsResolved
-  const { data: options = [], isLoading } = useSelectorOptions(field.selectorKey, {
+  const {
+    data: options = [],
+    isLoading,
+    hasMore,
+    isFetchingMore,
+    truncated,
+  } = useSelectorOptions(field.selectorKey, {
     context,
     enabled: isEnabled,
   })
 
+  /**
+   * Paginated selectors drain in the background and the combobox filters
+   * client-side, so a not-yet-drained option is genuinely absent from the list.
+   * Saying "none found" there reads as "it does not exist" and sends users off to
+   * enter the value by hand — say the list is still filling instead, and be honest
+   * when the drain stopped at its page cap.
+   */
+  const emptyMessage = useMemo(() => {
+    const noun = field.title.toLowerCase()
+    if (hasMore || isFetchingMore) return `Still loading ${noun}…`
+    if (truncated) return `Too many ${noun} to list — enter the value directly`
+    return `No ${noun} found`
+  }, [field.title, hasMore, isFetchingMore, truncated])
+
   const comboboxOptions = useMemo<ComboboxOption[]>(
     () => options.map((opt) => ({ label: opt.label, value: opt.id })),
     [options]
@@ -99,7 +119,7 @@ export function ConnectorSelectorField({
               : field.placeholder || `Select ${field.title.toLowerCase()}`
         }
         disabled={disabled || !credentialId || !depsResolved}
-        emptyMessage={`No ${field.title.toLowerCase()} found`}
+        emptyMessage={emptyMessage}
       />
     )
   }
@@ -120,7 +140,7 @@ export function ConnectorSelectorField({
             : field.placeholder || `Select ${field.title.toLowerCase()}`
       }
       disabled={disabled || !credentialId || !depsResolved}
-      emptyMessage={`No ${field.title.toLowerCase()} found`}
+      emptyMessage={emptyMessage}
     />
   )
 }