refactor(connectors): tidy rehydrate flag + gate Full resync to supported connectors

Cleanup/simplify pass over the connector changes:
- use shared booleanQueryFlagSchema for the rehydrate query param (typed boolean
  at the boundary instead of a hand-rolled 'true'/'false' string enum)
- move rehydrateOnFullSync onto the client-safe ConnectorMeta so the UI can gate on it
- only Confluence (rehydrateOnFullSync) shows the Sync now / Full resync dropdown;
  every other connector keeps its original one-click sync button (Full resync is a
  no-op for them, and this restores the pre-change one-click UX)
- wrap the sync trigger in a span so its tooltip still shows while disabled (cooldown)

Author: waleedlatif1

apps/sim/app/api/knowledge/[id]/connectors/[connectorId]/sync/route.ts

@@ -29,7 +29,7 @@ export const POST = withRouteHandler(async (request: NextRequest, context: Route
   const parsed = await parseRequest(triggerKnowledgeConnectorSyncContract, request, context)
   if (!parsed.success) return parsed.response
   const { id: knowledgeBaseId, connectorId } = parsed.data.params
-  const rehydrate = parsed.data.query?.rehydrate === 'true'
+  const { rehydrate } = parsed.data.query
 
   try {
     const auth = await checkSessionOrInternalAuth(request, { requireWorkflowId: false })

apps/sim/app/workspace/[workspaceId]/knowledge/[id]/components/connectors-section/connectors-section.tsx

@@ -339,6 +339,14 @@ function ConnectorCard({
   )
   const syncLogs = detail?.syncLogs ?? []
 
+  const canFullResync = Boolean(connectorDef?.rehydrateOnFullSync)
+  const syncDisabled =
+    connector.status === 'syncing' ||
+    connector.status === 'disabled' ||
+    isSyncPending ||
+    syncCooldown
+  const syncTooltip = syncCooldown ? 'Sync recently triggered' : canFullResync ? 'Sync' : 'Sync now'
+
   return (
     <div
       className={cn(
@@ -421,20 +429,47 @@ function ConnectorCard({
         <div className='flex flex-shrink-0 items-center gap-0.5'>
           {canEdit && (
             <>
-              <DropdownMenu>
+              {canFullResync ? (
+                <DropdownMenu>
+                  <Tooltip.Root>
+                    <Tooltip.Trigger asChild>
+                      {/* span keeps the tooltip hoverable while the trigger button is disabled */}
+                      <span className='inline-flex'>
+                        <DropdownMenuTrigger asChild>
+                          <Button
+                            variant='ghost'
+                            aria-label='Sync options'
+                            className={CONNECTOR_ACTION_BUTTON_CLASSES}
+                            disabled={syncDisabled}
+                          >
+                            <RefreshCw
+                              className={cn(
+                                'size-3.5',
+                                connector.status === 'syncing' && 'animate-spin'
+                              )}
+                            />
+                          </Button>
+                        </DropdownMenuTrigger>
+                      </span>
+                    </Tooltip.Trigger>
+                    <Tooltip.Content>{syncTooltip}</Tooltip.Content>
+                  </Tooltip.Root>
+                  <DropdownMenuContent align='end'>
+                    <DropdownMenuItem onSelect={() => onSync(false)}>Sync now</DropdownMenuItem>
+                    <DropdownMenuItem onSelect={() => onSync(true)}>Full resync</DropdownMenuItem>
+                  </DropdownMenuContent>
+                </DropdownMenu>
+              ) : (
                 <Tooltip.Root>
                   <Tooltip.Trigger asChild>
-                    <DropdownMenuTrigger asChild>
+                    {/* span keeps the tooltip hoverable while the button is disabled */}
+                    <span className='inline-flex'>
                       <Button
                         variant='ghost'
-                        aria-label='Sync options'
+                        aria-label='Sync now'
                         className={CONNECTOR_ACTION_BUTTON_CLASSES}
-                        disabled={
-                          connector.status === 'syncing' ||
-                          connector.status === 'disabled' ||
-                          isSyncPending ||
-                          syncCooldown
-                        }
+                        disabled={syncDisabled}
+                        onClick={() => onSync(false)}
                       >
                         <RefreshCw
                           className={cn(
@@ -443,17 +478,11 @@ function ConnectorCard({
                           )}
                         />
                       </Button>
-                    </DropdownMenuTrigger>
+                    </span>
                   </Tooltip.Trigger>
-                  <Tooltip.Content>
-                    {syncCooldown ? 'Sync recently triggered' : 'Sync'}
-                  </Tooltip.Content>
+                  <Tooltip.Content>{syncTooltip}</Tooltip.Content>
                 </Tooltip.Root>
-                <DropdownMenuContent align='end'>
-                  <DropdownMenuItem onSelect={() => onSync(false)}>Sync now</DropdownMenuItem>
-                  <DropdownMenuItem onSelect={() => onSync(true)}>Full resync</DropdownMenuItem>
-                </DropdownMenuContent>
-              </DropdownMenu>
+              )}
 
               <Tooltip.Root>
                 <Tooltip.Trigger asChild>

apps/sim/connectors/confluence/confluence.ts

@@ -156,14 +156,6 @@ function cqlResultToStub(item: Record<string, unknown>, domain: string): Externa
 export const confluenceConnector: ConnectorConfig = {
   ...confluenceConnectorMeta,
 
-  /**
-   * Confluence pages can transclude other pages (Include Page / Excerpt macros).
-   * Editing an included page changes a container page's rendered `view` without
-   * bumping the container's version, so its version-based hash can't detect the
-   * change. A full resync re-hydrates and re-indexes to pick up that drift.
-   */
-  rehydrateOnFullSync: true,
-
   listDocuments: async (
     accessToken: string,
     sourceConfig: Record<string, unknown>,

apps/sim/connectors/confluence/meta.ts

@@ -22,6 +22,15 @@ export const confluenceConnectorMeta: ConnectorMeta = {
     ],
   },
 
+  /**
+   * Confluence pages can transclude other pages (Include Page / Excerpt macros).
+   * Editing an included page changes a container page's rendered `view` without
+   * bumping the container's version, so its version-based hash can't detect the
+   * change. A full resync re-hydrates and re-indexes to pick up that drift. This
+   * lives on the meta so the client can offer "Full resync" only where it applies.
+   */
+  rehydrateOnFullSync: true,
+
   configFields: [
     {
       id: 'domain',

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

@@ -215,7 +215,7 @@ async function triggerSync({
 }: TriggerSyncParams): Promise<void> {
   await requestJson(triggerKnowledgeConnectorSyncContract, {
     params: { id: knowledgeBaseId, connectorId },
-    query: rehydrate ? { rehydrate: 'true' } : {},
+    query: rehydrate ? { rehydrate: true } : {},
   })
 }
 

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

@@ -4,6 +4,7 @@ import {
   knowledgeConnectorParamsSchema,
   successResponseSchema,
 } from '@/lib/api/contracts/knowledge/shared'
+import { booleanQueryFlagSchema } from '@/lib/api/contracts/primitives'
 import { defineRouteContract } from '@/lib/api/contracts/types'
 
 export const createConnectorBodySchema = z.object({
@@ -155,11 +156,10 @@ export const triggerKnowledgeConnectorSyncQuerySchema = z.object({
    * Force re-hydration: for connectors whose rendered content can drift without a
    * hash change (e.g. Confluence transclusions), re-fetch and re-index every
    * already-synced document rather than only hash-changed ones. Listing and
-   * deletion reconciliation are unchanged from a normal sync. Omitted performs the
-   * normal hash-gated sync. Modeled as a string literal (not a boolean) so it
-   * round-trips cleanly through the query string.
+   * deletion reconciliation are unchanged from a normal sync. Defaults to the
+   * normal hash-gated sync.
    */
-  rehydrate: z.enum(['true', 'false']).optional(),
+  rehydrate: booleanQueryFlagSchema.optional().default(false),
 })
 
 export const triggerKnowledgeConnectorSyncContract = defineRouteContract({