fix lint

Author: icecrasher321

apps/sim/app/api/skills/[id]/members/route.ts

@@ -5,10 +5,7 @@ import { createLogger } from '@sim/logger'
 import { generateId } from '@sim/utils/id'
 import { and, eq, sql } from 'drizzle-orm'
 import { type NextRequest, NextResponse } from 'next/server'
-import {
-  removeSkillMemberContract,
-  upsertSkillMemberContract,
-} from '@/lib/api/contracts/skills'
+import { removeSkillMemberContract, upsertSkillMemberContract } from '@/lib/api/contracts/skills'
 import { parseRequest } from '@/lib/api/server'
 import { getSession } from '@/lib/auth'
 import { withRouteHandler } from '@/lib/core/utils/with-route-handler'
@@ -136,7 +133,11 @@ export const POST = withRouteHandler(async (request: NextRequest, context: Route
 
     const { userId, role } = parsed.data.body
 
-    const targetWorkspacePerm = await getUserEntityPermissions(userId, 'workspace', gate.workspaceId)
+    const targetWorkspacePerm = await getUserEntityPermissions(
+      userId,
+      'workspace',
+      gate.workspaceId
+    )
     if (targetWorkspacePerm === null) {
       return NextResponse.json({ error: 'User is not a member of this workspace' }, { status: 400 })
     }

apps/sim/lib/copilot/vfs/workspace-vfs.ts

@@ -104,6 +104,7 @@ import { BINARY_DOC_TASKS, MAX_DOCUMENT_PREVIEW_CODE_BYTES } from '@/lib/executi
 import { runSandboxTask, SandboxUserCodeError } from '@/lib/execution/sandbox/run-task'
 import { getKnowledgeBases } from '@/lib/knowledge/service'
 import { validateMermaidSource } from '@/lib/mermaid/validate'
+import { canUseSkill, getSkillAccessForUser } from '@/lib/skills/access'
 import { listTables } from '@/lib/table/service'
 import { listWorkspaceFileFolders } from '@/lib/uploads/contexts/workspace/workspace-file-folder-manager'
 import {
@@ -119,7 +120,6 @@ import {
   loadWorkflowFromNormalizedTables,
 } from '@/lib/workflows/persistence/utils'
 import { sanitizeForCopilot } from '@/lib/workflows/sanitization/json-sanitizer'
-import { canUseSkill, getSkillAccessForUser } from '@/lib/skills/access'
 import { getSkillById } from '@/lib/workflows/skills/operations'
 import { listFolders, listWorkflows } from '@/lib/workflows/utils'
 import {

apps/sim/lib/mothership/inbox/executor.ts

@@ -212,19 +212,14 @@ export async function executeInboxTask(taskId: string): Promise<void> {
 
     const workspaceAccess = await checkWorkspaceAccess(ws.id, userId)
     const userPermission = workspaceAccess.permission
-    const [
-      attachmentResult,
-      workspaceContext,
-      integrationTools,
-      billingAttribution,
-      entitlements,
-    ] = await Promise.all([
-      fetchAttachments(),
-      generateWorkspaceContext(ws.id, userId, { workspaceAccess }),
-      buildIntegrationToolSchemas(userId, undefined, undefined, ws.id),
-      resolveBillingAttribution({ actorUserId: userId, workspaceId: ws.id }),
-      computeWorkspaceEntitlements(ws.id, userId),
-    ])
+    const [attachmentResult, workspaceContext, integrationTools, billingAttribution, entitlements] =
+      await Promise.all([
+        fetchAttachments(),
+        generateWorkspaceContext(ws.id, userId, { workspaceAccess }),
+        buildIntegrationToolSchemas(userId, undefined, undefined, ws.id),
+        resolveBillingAttribution({ actorUserId: userId, workspaceId: ws.id }),
+        computeWorkspaceEntitlements(ws.id, userId),
+      ])
     const { attachments, fileAttachments, storedAttachments } = attachmentResult
 
     const truncatedTask = {

apps/sim/lib/skills/access.test.ts

@@ -5,30 +5,30 @@ import { beforeEach, describe, expect, it, vi } from 'vitest'
 
 const { mockCheckWorkspaceAccess, mockGetUsersWithPermissions, dbState, makeChain, dbMock } =
   vi.hoisted(() => {
-  const state = { results: [] as unknown[][] }
-  const chainFactory = () => {
-    const resolve = () => Promise.resolve(state.results.shift() ?? [])
-    const chain: any = {}
-    chain.from = vi.fn(() => chain)
-    chain.innerJoin = vi.fn(() => chain)
-    chain.where = vi.fn(() => chain)
-    chain.set = vi.fn(() => chain)
-    chain.limit = vi.fn(() => resolve())
-    chain.returning = vi.fn(() => resolve())
-    chain.then = (onFulfilled: any, onRejected: any) => resolve().then(onFulfilled, onRejected)
-    return chain
-  }
-  return {
-    mockCheckWorkspaceAccess: vi.fn(),
-    mockGetUsersWithPermissions: vi.fn(),
-    dbState: state,
-    makeChain: chainFactory,
-    dbMock: {
-      select: vi.fn(() => chainFactory()),
-      update: vi.fn(() => chainFactory()),
-    },
-  }
-})
+    const state = { results: [] as unknown[][] }
+    const chainFactory = () => {
+      const resolve = () => Promise.resolve(state.results.shift() ?? [])
+      const chain: any = {}
+      chain.from = vi.fn(() => chain)
+      chain.innerJoin = vi.fn(() => chain)
+      chain.where = vi.fn(() => chain)
+      chain.set = vi.fn(() => chain)
+      chain.limit = vi.fn(() => resolve())
+      chain.returning = vi.fn(() => resolve())
+      chain.then = (onFulfilled: any, onRejected: any) => resolve().then(onFulfilled, onRejected)
+      return chain
+    }
+    return {
+      mockCheckWorkspaceAccess: vi.fn(),
+      mockGetUsersWithPermissions: vi.fn(),
+      dbState: state,
+      makeChain: chainFactory,
+      dbMock: {
+        select: vi.fn(() => chainFactory()),
+        update: vi.fn(() => chainFactory()),
+      },
+    }
+  })
 
 vi.mock('@sim/db', () => ({
   db: dbMock,
@@ -246,7 +246,9 @@ describe('getSkillAccessForUser + resolveSkillRoleFromAccess', () => {
     expect(resolveSkillRoleFromAccess({ id: 's-admin', workspaceShared: false }, access)).toBe(
       'admin'
     )
-    expect(resolveSkillRoleFromAccess({ id: 's-revoked', workspaceShared: true }, access)).toBeNull()
+    expect(
+      resolveSkillRoleFromAccess({ id: 's-revoked', workspaceShared: true }, access)
+    ).toBeNull()
     expect(resolveSkillRoleFromAccess({ id: 's-shared', workspaceShared: true }, access)).toBe(
       'member'
     )

apps/sim/lib/skills/access.ts

@@ -302,10 +302,7 @@ export async function removeWorkspaceSkillMembershipsTx(
         eq(skillMember.status, 'active'),
         inArray(
           skillMember.skillId,
-          tx
-            .select({ id: skill.id })
-            .from(skill)
-            .where(inArray(skill.workspaceId, workspaceIds))
+          tx.select({ id: skill.id }).from(skill).where(inArray(skill.workspaceId, workspaceIds))
         )
       )
     )