perf(mcp): thread test-connection authType through to create, drop duplicate probe

test-connection already probes the URL and returns `authType` as part
of its structured signal. Pass it through to the create call so
performCreateMcpServer skips its own probe — one network round-trip
per add instead of two.

server-lifecycle already short-circuits when params.authType is set,
so no route changes needed; this is purely UI plumbing. Falls back to
the existing probe if authType is omitted (e.g. server-to-server
callers or older clients), so behavior is unchanged for those paths.

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

Author: waleedlatif1

apps/sim/app/workspace/[workspaceId]/settings/components/mcp/components/mcp-server-form-modal/mcp-server-form-modal.tsx

@@ -52,6 +52,13 @@ export interface McpServerFormConfig {
   timeout: number
   oauthClientId?: string
   oauthClientSecret?: string
+  /**
+   * Auth type detected by the pre-submit `test-connection` probe. When set,
+   * the create route uses this directly and skips its own probe (the route
+   * still falls back to its own probe if omitted, so this is purely a perf
+   * win — one network round-trip per add instead of two).
+   */
+  authType?: 'none' | 'headers' | 'oauth'
 }
 
 export interface McpServerFormModalProps {
@@ -531,6 +538,7 @@ export function McpServerFormModal({
         url: formData.url!,
         headers,
         timeout: formData.timeout || 30000,
+        authType: connectionResult.authType,
         oauthClientId:
           mode === 'edit'
             ? oauthClientIdChanged
@@ -593,6 +601,7 @@ export function McpServerFormModal({
         url: config.url,
         headers: config.headers,
         timeout: 30000,
+        authType: connectionResult.authType,
       })
 
       onOpenChange(false)

apps/sim/hooks/queries/mcp.ts

@@ -54,6 +54,8 @@ export interface McpServerInput {
   enabled: boolean
   oauthClientId?: string
   oauthClientSecret?: string
+  /** Pre-detected by `test-connection`; lets the create route skip its own probe. */
+  authType?: 'none' | 'headers' | 'oauth'
 }
 
 async function fetchMcpServers(workspaceId: string, signal?: AbortSignal): Promise<McpServer[]> {