chore(lint): delete unused variables and parameters across apps

Remove dead locals, constants, helpers, test mock aliases, and unused
function parameters (updating callers), drop the unused BlockConfig type
parameter and the response types that only fed it.

Author: waleedlatif1

apps/desktop/src/main/browser-agent/driver.ts

@@ -1738,7 +1738,7 @@ function requireSnapshotForElementAction(): void {
   )
 }
 
-function pageTargetForElement(contents: WebContents, elementId: number): PageExecutionTarget {
+function pageTargetForElement(elementId: number): PageExecutionTarget {
   requireSnapshotForElementAction()
   const target = driverScopeState().snapshotTargets.get(elementId)
   if (!target || ('isDestroyed' in target && target.isDestroyed())) {
@@ -2351,7 +2351,7 @@ async function captureSnapshot(
   if (tab.view.webContents !== contents) {
     throw new ToolError('The active tab changed before the snapshot started. Try again.')
   }
-  if (elementId !== undefined && pageTargetForElement(contents, elementId) !== contents) {
+  if (elementId !== undefined && pageTargetForElement(elementId) !== contents) {
     throw new ToolError(
       'Scoped snapshots require a top-page element. Omit elementId to capture framed content.'
     )
@@ -2745,7 +2745,7 @@ async function executeToolInner(
       const contents = session.requireAutomationTab().view.webContents
       const elementId = requireNum(params, 'elementId')
       const paths = uploadPaths(params)
-      const target = pageTargetForElement(contents, elementId)
+      const target = pageTargetForElement(elementId)
       assertCurrentExecution()
       const frame = 'getURL' in target ? target.mainFrame : target
       const expression = `(${String(resolveFileInputTarget)})(${elementId})`
@@ -2816,8 +2816,7 @@ async function executeToolInner(
       }
       const waitedTab = session.requireAutomationTab()
       const contents = waitedTab.view.webContents
-      const elementTarget =
-        elementId === undefined ? undefined : pageTargetForElement(contents, elementId)
+      const elementTarget = elementId === undefined ? undefined : pageTargetForElement(elementId)
       if (elementTarget && elementTarget !== contents) {
         throw new ToolError(
           'Element-state waits are limited to the top page. Use a text or URL condition for framed content.'
@@ -2946,7 +2945,7 @@ async function executeToolInner(
       const contents = session.requireAutomationTab().view.webContents
       const elementId = num(params, 'elementId')
       if (elementId === undefined) return await readWholePageText(contents, executionDeadline)
-      const target = pageTargetForElement(contents, elementId)
+      const target = pageTargetForElement(elementId)
       return unwrapPageResult(
         await execInPage(target, readPageText, [elementId], false, executionDeadline)
       )
@@ -2961,7 +2960,7 @@ async function executeToolInner(
       const elementId = num(params, 'elementId')
       let elementClip: Record<string, unknown> | undefined
       if (elementId !== undefined) {
-        const target = pageTargetForElement(contents, elementId)
+        const target = pageTargetForElement(elementId)
         if (target !== contents) {
           throw new ToolError(
             'Element screenshots are limited to the top page. Use browser_screenshot without elementId for framed content.'
@@ -3119,7 +3118,7 @@ async function executeToolInner(
       const contents = clickedTab.view.webContents
       const elementId = requireNum(params, 'elementId')
       const click = pointerClick(params)
-      const target = pageTargetForElement(contents, elementId)
+      const target = pageTargetForElement(elementId)
       const targetFrame = frameExecutionTarget(target, contents)
       let trusted = false
       let activation = 'synthetic-pointer'
@@ -3596,7 +3595,7 @@ async function executeToolInner(
       let stoppedIndex = 0
       let dispatchStarted = false
       const readField = async (field: FormField) => {
-        const target = pageTargetForElement(contents, field.elementId)
+        const target = pageTargetForElement(field.elementId)
         if (target !== contents)
           throw new ToolError(
             'Form batches require top-page fields; use individual tools for framed fields.'
@@ -3757,7 +3756,7 @@ async function executeToolInner(
       if (typeof text !== 'string') throw new ToolError('Missing required parameter "text"')
       const submit = params.submit === true
       const contents = session.requireAutomationTab().view.webContents
-      const target = pageTargetForElement(contents, elementId)
+      const target = pageTargetForElement(elementId)
       const targetFrame = frameExecutionTarget(target, contents)
 
       // Native path: focus + select current content, then insert through the
@@ -4307,9 +4306,7 @@ async function executeToolInner(
       const contents = session.requireAutomationTab().view.webContents
       const elementId = num(params, 'elementId')
       const target =
-        elementId !== undefined
-          ? pageTargetForElement(contents, elementId)
-          : focusedPageTarget(contents)
+        elementId !== undefined ? pageTargetForElement(elementId) : focusedPageTarget(contents)
       const targetFrame = frameExecutionTarget(target, contents)
       assertCurrentExecution()
       if (elementId !== undefined) assertElementActionCurrent(contents, elementId, target)
@@ -4355,7 +4352,7 @@ async function executeToolInner(
       const selection = values === undefined ? requireStr(params, 'value') : (values as string[])
       const contents = session.requireAutomationTab().view.webContents
       const elementId = requireNum(params, 'elementId')
-      const target = pageTargetForElement(contents, elementId)
+      const target = pageTargetForElement(elementId)
       const targetFrame = frameExecutionTarget(target, contents)
       assertCurrentExecution()
       assertElementActionCurrent(contents, elementId, target)
@@ -4434,8 +4431,7 @@ async function executeToolInner(
         throw new ToolError('Missing required boolean parameter "checked"')
       }
       const checked = params.checked
-      const contents = session.requireAutomationTab().view.webContents
-      const target = pageTargetForElement(contents, elementId)
+      const target = pageTargetForElement(elementId)
       const before = toRecord(
         unwrapPageResult(
           await execInPage(target, readCheckableElementState, [elementId], false, executionDeadline)
@@ -4562,7 +4558,7 @@ async function executeToolInner(
         )
       }
       const elementId = requireNum(params, 'elementId')
-      const target = pageTargetForElement(contents, elementId)
+      const target = pageTargetForElement(elementId)
       const targetFrame = frameExecutionTarget(target, contents)
       let beforePage = await pageActionState(target, true, elementId)
       let beforeElement = await activeElementState(target)
@@ -4939,7 +4935,7 @@ async function executeToolInner(
       ): Promise<{ x: number; y: number; element?: string }> => {
         const elementId = num(params, `${which}ElementId`)
         if (elementId !== undefined) {
-          const target = pageTargetForElement(contents, elementId)
+          const target = pageTargetForElement(elementId)
           if (frameExecutionTarget(target, contents)) {
             throw new ToolError(
               `Dragging elements inside embedded frames is not supported. Use ${which}X/${which}Y viewport coordinates instead.`

apps/sim/app/(interfaces)/resume/[workflowId]/[executionId]/resume-page-client.tsx

@@ -208,9 +208,7 @@ export default function ResumeExecutionPage({
   const queuePosition = selectedDetail?.pausePoint.queuePosition
   const resumeInputsRef = useRef<Record<string, string>>({})
   const [resumeInput, setResumeInput] = useState('')
-  const [formValuesByContext, setFormValuesByContext] = useState<
-    Record<string, Record<string, string>>
-  >({})
+  const formValuesByContextRef = useRef<Record<string, Record<string, string>>>({})
   const [formValues, setFormValues] = useState<Record<string, string>>({})
   const [formErrors, setFormErrors] = useState<Record<string, string>>({})
   const [loadingAction, setLoadingAction] = useState(false)
@@ -308,17 +306,6 @@ export default function ResumeExecutionPage({
     [formatValueForInputField]
   )
 
-  const formatStructureValue = useCallback((value: any): string => {
-    if (value === null || value === undefined) return '—'
-    if (typeof value === 'string') return value
-    if (typeof value === 'number' || typeof value === 'boolean') return String(value)
-    try {
-      return JSON.stringify(value, null, 2)
-    } catch {
-      return String(value)
-    }
-  }, [])
-
   const parseFormValue = useCallback(
     (field: NormalizedInputField, rawValue: string): { value: any; error?: string } => {
       const value = rawValue ?? ''
@@ -360,7 +347,7 @@ export default function ResumeExecutionPage({
       if (!selectedContextId) return
       setFormValues((prev) => {
         const updated = { ...prev, [fieldName]: newValue }
-        setFormValuesByContext((map) => ({ ...map, [selectedContextId]: updated }))
+        formValuesByContextRef.current[selectedContextId] = updated
         return updated
       })
       setFormErrors((prev) => {
@@ -516,12 +503,9 @@ export default function ResumeExecutionPage({
           : undefined
       if (operation === 'human' && fetchedInputFields.length > 0) {
         const baseValues = buildInitialFormValues(fetchedInputFields, submission)
-        let mergedValues = baseValues
-        setFormValuesByContext((prev) => {
-          const existingValues = prev[detail.pausePoint.contextId]
-          if (existingValues) mergedValues = { ...baseValues, ...existingValues }
-          return { ...prev, [detail.pausePoint.contextId]: mergedValues }
-        })
+        const existingValues = formValuesByContextRef.current[detail.pausePoint.contextId]
+        const mergedValues = existingValues ? { ...baseValues, ...existingValues } : baseValues
+        formValuesByContextRef.current[detail.pausePoint.contextId] = mergedValues
         setFormValues(mergedValues)
         setFormErrors({})
         if (resumeInputsRef.current[detail.pausePoint.contextId] !== undefined) {

apps/sim/app/api/auth/sso/register/route.test.ts

@@ -374,7 +374,7 @@ describe('POST /api/auth/sso/register', () => {
 
   it('does not SSRF-validate userInfoEndpoint when skipUserInfoEndpoint is requested', async () => {
     queueMembers([{ organizationId: 'org1', role: 'owner' }])
-    mockValidateUrlWithDNS.mockImplementation(async (url: string, label: string) => {
+    mockValidateUrlWithDNS.mockImplementation(async (_url: string, label: string) => {
       if (label === 'OIDC userInfoEndpoint') {
         return { isValid: false, error: 'resolves to a private IP address' }
       }

apps/sim/app/api/chat/[identifier]/otp/route.ts

@@ -60,7 +60,7 @@ async function deliverOtp(requestId: string, deploymentId: string, title: string
   const otp = generateOTP()
   await storeOTP('chat', deploymentId, email, otp)
 
-  const emailHtml = await renderOTPEmail(otp, email, 'email-verification', title)
+  const emailHtml = await renderOTPEmail(otp, 'email-verification', title)
   const emailResult = await sendEmail({
     to: email,
     subject: getOtpSubject(title),

apps/sim/app/api/copilot/feedback/route.ts

@@ -117,7 +117,7 @@ export const POST = withRouteHandler(async (req: NextRequest) => {
  * GET /api/copilot/feedback
  * Get feedback records for the authenticated user
  */
-export const GET = withRouteHandler(async (req: NextRequest) => {
+export const GET = withRouteHandler(async () => {
   const tracker = createRequestTracker()
 
   try {

apps/sim/app/api/environment/route.ts

@@ -115,7 +115,7 @@ export const POST = withRouteHandler(async (req: NextRequest) => {
   }
 })
 
-export const GET = withRouteHandler(async (request: Request) => {
+export const GET = withRouteHandler(async () => {
   const requestId = generateRequestId()
 
   try {

apps/sim/app/api/files/authorization.test.ts

@@ -142,7 +142,7 @@ describe('verifyKBFileWriteAccess (binding-only delete authorization)', () => {
 
 describe('public-context access (profile-pictures / og-images / workspace-logos)', () => {
   function write(cloudKey: string, context: 'profile-pictures' | 'og-images' | 'workspace-logos') {
-    return verifyFileAccess(cloudKey, USER_ID, undefined, context, false, { requireWrite: true })
+    return verifyFileAccess(cloudKey, USER_ID, undefined, context, { requireWrite: true })
   }
 
   it('allows organization logo reads and denies generic deletes even for the uploader', async () => {
@@ -152,10 +152,10 @@ describe('public-context access (profile-pictures / og-images / workspace-logos)
       true
     )
     await expect(
-      verifyFileAccess(key, USER_ID, undefined, 'organization-logos', false, { requireWrite: true })
+      verifyFileAccess(key, USER_ID, undefined, 'organization-logos', { requireWrite: true })
     ).resolves.toBe(false)
     await expect(
-      verifyFileAccess(key, USER_ID, undefined, 'general', false, { requireWrite: true })
+      verifyFileAccess(key, USER_ID, undefined, 'general', { requireWrite: true })
     ).resolves.toBe(false)
     expect(mockGetFileMetadata).not.toHaveBeenCalled()
     expect(mockGetUserEntityPermissions).not.toHaveBeenCalled()
@@ -203,7 +203,7 @@ describe('workspace-scoped access (workspace files and mothership attachments)',
   })
 
   function read(cloudKey: string, context: 'workspace' | 'mothership') {
-    return verifyFileAccess(cloudKey, USER_ID, undefined, context, false)
+    return verifyFileAccess(cloudKey, USER_ID, undefined, context)
   }
 
   interface BoundRow {
@@ -315,7 +315,7 @@ describe('organization connector cache access', () => {
     'denies the uploader a raw download even with a forged %s context',
     async (context) => {
       await expect(
-        verifyFileAccess(CLOUD_KEY, USER_ID, undefined, context, false, { knowledgeAccess: 'user' })
+        verifyFileAccess(CLOUD_KEY, USER_ID, undefined, context, { knowledgeAccess: 'user' })
       ).resolves.toBe(false)
       expect(mockGetFileMetadata).not.toHaveBeenCalled()
       expect(mockGetUserEntityPermissions).not.toHaveBeenCalled()
@@ -325,7 +325,7 @@ describe('organization connector cache access', () => {
   it('denies system reads after the cache loses its active document reference', async () => {
     dbChainMockFns.limit.mockResolvedValue([])
     await expect(
-      verifyFileAccess(CLOUD_KEY, USER_ID, undefined, 'knowledge-base', false, {
+      verifyFileAccess(CLOUD_KEY, USER_ID, undefined, 'knowledge-base', {
         knowledgeAccess: SYSTEM_ACCESS_SCOPE,
       })
     ).resolves.toBe(false)
@@ -338,7 +338,7 @@ describe('organization connector cache access', () => {
       deletedAt: null,
     })
     await expect(
-      verifyFileAccess(CLOUD_KEY, USER_ID, undefined, 'knowledge-base', false, {
+      verifyFileAccess(CLOUD_KEY, USER_ID, undefined, 'knowledge-base', {
         knowledgeAccess: SYSTEM_ACCESS_SCOPE,
       })
     ).resolves.toBe(false)
@@ -347,7 +347,7 @@ describe('organization connector cache access', () => {
 
   it('does not let a raw download endpoint delete organization caches', async () => {
     await expect(
-      verifyFileAccess(CLOUD_KEY, USER_ID, undefined, 'general', false, {
+      verifyFileAccess(CLOUD_KEY, USER_ID, undefined, 'general', {
         requireWrite: true,
         knowledgeAccess: SYSTEM_ACCESS_SCOPE,
       })
@@ -378,7 +378,7 @@ describe('KB file live source authorization', () => {
       queueTableRows(schemaMock.knowledgeConnector, [{ connectorId: 'confluence-source' }])
       queueTableRows(schemaMock.document, allowed ? [{ id: 'doc-1' }] : [])
       await expect(
-        verifyFileAccess(CLOUD_KEY, USER_ID, undefined, 'knowledge-base', false, {
+        verifyFileAccess(CLOUD_KEY, USER_ID, undefined, 'knowledge-base', {
           knowledgeAccess: access,
         })
       ).resolves.toBe(allowed)
@@ -415,7 +415,7 @@ describe('KB file live source authorization', () => {
     mockGetUserEntityPermissions.mockResolvedValue(null)
     const get = vi.fn()
     await expect(
-      verifyFileAccess(CLOUD_KEY, USER_ID, undefined, 'knowledge-base', false, {
+      verifyFileAccess(CLOUD_KEY, USER_ID, undefined, 'knowledge-base', {
         knowledgeAccess: { get, getForConnectors: vi.fn(), getForDocuments: vi.fn() },
       })
     ).resolves.toBe(false)