chore(mcp): final audit pass — strip restated TSDoc and narration comments

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

Author: waleedlatif1

apps/sim/app/workspace/[workspaceId]/settings/components/mcp/mcp.tsx

@@ -152,10 +152,6 @@ interface MCPProps {
   initialServerId?: string | null
 }
 
-/**
- * MCP Settings component for managing Model Context Protocol servers.
- * Handles server CRUD operations, connection testing, and environment variable integration.
- */
 export function MCP({ initialServerId }: MCPProps) {
   const params = useParams()
   const workspaceId = params.workspaceId as string

apps/sim/lib/mcp/oauth/probe.ts

@@ -6,14 +6,6 @@ const logger = createLogger('McpOauthProbe')
 
 const PROBE_TIMEOUT_MS = 5000
 
-/**
- * Lightweight unauthenticated probe to determine the auth mode required by
- * a remote MCP server.
- *
- * - 200/2xx → server accepts unauthenticated requests; treat as `none`.
- * - 401 with `WWW-Authenticate: Bearer resource_metadata=...` → `oauth`.
- * - anything else → fall back to `headers` (caller can let the user paste a token).
- */
 export async function detectMcpAuthType(url: string): Promise<McpAuthType> {
   const controller = new AbortController()
   const timer = setTimeout(() => controller.abort(), PROBE_TIMEOUT_MS)

apps/sim/lib/mcp/oauth/provider.ts

@@ -148,10 +148,6 @@ export class SimMcpOauthProvider implements OAuthClientProvider {
   }
 }
 
-/**
- * Load workspace-shared pre-registered OAuth credentials for a server, if any.
- * Decrypts the stored client_secret. Returns undefined when DCR should be used.
- */
 export async function loadPreregisteredClient(
   serverId: string
 ): Promise<PreregisteredClient | undefined> {

apps/sim/lib/mcp/service.ts

@@ -162,8 +162,6 @@ class McpService {
   }
 
   /**
-   * Create and connect to an MCP client.
-   *
    * For `authType === 'oauth'` configs, a per-user `SimMcpOauthProvider` is
    * built and attached so the SDK can drive the standard MCP OAuth flow.
    */
@@ -412,9 +410,8 @@ class McpService {
           result.reason instanceof McpOauthAuthorizationRequiredError ||
           result.reason instanceof UnauthorizedError
         ) {
-          // Pending user authorization or expired refresh — not a hard failure.
-          // Force the row back to 'disconnected' so the settings UI surfaces the
-          // "Connect with OAuth" button instead of leaving stale 'connected' state.
+          // Force 'disconnected' so the settings UI surfaces the re-auth button
+          // instead of a stale 'connected' state when refresh has expired.
           logger.info(`[${requestId}] Skipping server ${server.name}: OAuth authorization pending`)
           statusUpdates.push(
             db