chore(perf): drop obvious inline comments, keep nuance as TSDoc

Author: waleedlatif1

apps/sim/lib/execution/preprocessing.ts

@@ -323,9 +323,8 @@ export async function preprocessExecution(
   }
 
   // ========== STEPS 3.5–6: Preflight Gates ==========
-  // Read-only gates run concurrently (ban + subscription, then usage). The
-  // rate-limit gate debits a token, so it runs sequentially only after ban and
-  // usage pass. Failures apply in fixed precedence: ban 403 → usage 402 → rate 429.
+  // Read-only gates (ban, subscription, usage) run concurrently; the stateful
+  // rate-limit gate runs after they pass. Precedence: ban 403 → usage 402 → rate 429.
 
   /**
    * A failing gate's deferred outcome: the response to return, plus an optional
@@ -643,7 +642,6 @@ export async function preprocessExecution(
   const usageResult = await usageCheckTask
   const usageSnapshot = usageResult.snapshot
 
-  // Precedence: ban (403) wins over usage (402).
   const readGateFailure = banFailure ?? usageResult.failure
   if (readGateFailure) {
     if (readGateFailure.recordError) {

apps/sim/providers/bedrock/index.ts

@@ -139,8 +139,6 @@ export const bedrockProvider: ProviderConfig = {
       }
     }
 
-    // AWS SDK clients own a per-client connection pool and are meant to be reused.
-    // Keyed by region + credential identity (a rotated key pair yields a new key).
     const client = getCachedProviderClient(
       `bedrock::${region}::${request.bedrockAccessKeyId ?? 'default-chain'}`,
       () => new BedrockRuntimeClient(clientConfig)

apps/sim/providers/client-cache.ts

@@ -3,8 +3,10 @@ import { LRUCache } from 'lru-cache'
 const CLIENT_CACHE_MAX_ENTRIES = 1_000
 const CLIENT_CACHE_TTL_MS = 30 * 60 * 1_000
 
-// updateAgeOnGet makes the TTL idle-based, so a continuously-used client keeps
-// its warm keep-alive connections while idle keys age out.
+/**
+ * `updateAgeOnGet` makes the TTL idle-based: a continuously-used client keeps its
+ * warm keep-alive connections, while idle keys age out.
+ */
 const clientCache = new LRUCache<string, object>({
   max: CLIENT_CACHE_MAX_ENTRIES,
   ttl: CLIENT_CACHE_TTL_MS,

apps/sim/providers/vllm/index.ts

@@ -135,8 +135,6 @@ export const vllmProvider: ProviderConfig = {
     }
 
     const apiKey = request.apiKey || env.VLLM_API_KEY || 'empty'
-    // A pinned endpoint gets its own undici Agent, so reuse keeps connections
-    // warm. DNS re-validation still runs every request; a new IP rekeys.
     const vllm = getCachedProviderClient(
       `vllm::${apiKey}::${baseUrl}::${pinnedIP ?? 'no-pin'}`,
       () =>