chore: trim restating comments and a dead coalesce

Cleanup-pass findings on this branch's own lines: comments that restated the
code they sat on, a four-line note whose sibling said the same in one, and a
`?? undefined` on a non-nullable value. The comment on the tool-picker filter
now explains the clause that actually needed it (an empty option list is not a
denied one) instead of narrating the filter.

Left alone as pre-existing and out of scope: the inline `staleTime` literal in
`useAllowedIntegrationsFromEnv`, and the Operation selector's raw label /
plain `Combobox` — both byte-identical to staging and matching the convention
of every sibling field in that panel.

Author: waleedlatif1

apps/sim/app/workspace/[workspaceId]/w/[workflowId]/components/panel/components/editor/components/sub-block/components/tool-input/tool-input.tsx

@@ -694,9 +694,8 @@ export const ToolInput = memo(function ToolInput({
   const customBlockOverlayVersion = useCustomBlockOverlayVersion()
   const toolBlocks = useMemo(() => {
     const allToolBlocks = getAllBlocks().filter(isAgentToolBlock)
-    /* A multi-operation block whose every operation is denied has nothing the
-       caller can run, so it leaves the picker alongside the blocks denied
-       outright by `filterBlocks`. */
+    /* An empty option list means the block declares no selectable operation, so
+       there is nothing to gate — only a wholly denied one leaves the picker. */
     return filterBlocks(allToolBlocks).filter((block) => {
       if (!hasMultipleOperations(block)) return true
       const { options, denied } = getOperationChoices(block)
@@ -818,7 +817,7 @@ export const ToolInput = memo(function ToolInput({
       if (isPreview || disabled) return
 
       const { options, denied } = hasMultipleOperations(toolBlock)
-        ? getOperationChoices(toolBlock ?? undefined)
+        ? getOperationChoices(toolBlock)
         : { options: [], denied: NO_DENIED_OPERATIONS }
       const defaultOperation = options.find((option) => !denied.has(option.id))?.id
 

apps/sim/app/workspace/[workspaceId]/w/[workflowId]/workflow.tsx

@@ -895,11 +895,6 @@ const WorkflowContent = React.memo(
         if (parentId) blockData.parentId = parentId
         if (extent) blockData.extent = extent
 
-        /**
-         * `undefined` until the permission config has resolved, so a declared
-         * default is never vetoed — or let through — on a guess. Blocks pre-fill
-         * two fields the group can restrict; both go through the same gate.
-         */
         const operationGate = resolveOperationGate(getBlock(type))
         const seedGate = operationGate
           ? (subBlockId: string, value: string) => {

apps/sim/hooks/use-permission-config.ts

@@ -145,11 +145,7 @@ export function usePermissionConfig(): PermissionConfigResult {
     }
   }, [isModelAllowed, isProviderAllowed])
 
-  /**
-   * Indexed rather than scanned: the operation gate calls this once per option
-   * of every block it offers, so a linear scan made the cost of a check scale
-   * with the length of the denylist.
-   */
+  /** Indexed so the per-tool check stays O(1) over a long denylist. */
   const deniedToolSet = useMemo(() => new Set(config.deniedTools), [config.deniedTools])
 
   const isToolAllowed = useMemo(() => {

apps/sim/lib/permission-groups/operation-access.ts

@@ -10,7 +10,6 @@ import type { BlockConfig } from '@/blocks/types'
  */
 export const OPERATION_SUBBLOCK_ID = 'operation'
 
-/** The subblock id that carries a block's model. */
 export const MODEL_SUBBLOCK_ID = 'model'
 
 /** Shared empty result, so a caller's memo sees a stable identity. */
@@ -56,8 +55,7 @@ export function resolveOperationToolId(
       const toolId = selectTool({ operation: operationId })
       if (toolId) return toolId
     } catch {
-      /* Falls through rather than guessing a tool for an operation the
-         selector could not resolve on its own. */
+      /* Unresolvable from the operation alone; see the TSDoc above. */
     }
   }