Skip to content

Commit c8f4e2c

Browse files
ci: apply automated fixes
1 parent e6eae09 commit c8f4e2c

8 files changed

Lines changed: 69 additions & 33 deletions

File tree

src/components/TableComponents.tsx

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -73,8 +73,8 @@ export function TableHeaderCell({
7373
align === 'right'
7474
? 'text-right'
7575
: align === 'center'
76-
? 'text-center'
77-
: 'text-left'
76+
? 'text-center'
77+
: 'text-left'
7878
const paddingClass = compact ? 'px-2 py-1.5' : 'px-4 py-2'
7979
const textSizeClass = compact ? 'text-[10px]' : 'text-xs'
8080
return (
@@ -139,8 +139,8 @@ export function TableCell({
139139
align === 'right'
140140
? 'text-right'
141141
: align === 'center'
142-
? 'text-center'
143-
: 'text-left'
142+
? 'text-center'
143+
: 'text-left'
144144
const paddingClass = compact ? 'px-2 py-2' : 'px-4 py-3'
145145
return (
146146
<td

src/routes/_libraries/feed.index.tsx

Lines changed: 12 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -32,8 +32,18 @@ export const Route = createFileRoute('/_libraries/feed/')({
3232
includePrerelease: z.boolean().optional().catch(undefined),
3333
featured: z.boolean().optional().catch(undefined),
3434
search: z.string().optional().catch(undefined),
35-
page: z.number().optional().default(FEED_DEFAULTS.page).catch(FEED_DEFAULTS.page),
36-
pageSize: z.number().int().positive().optional().default(FEED_DEFAULTS.pageSize).catch(FEED_DEFAULTS.pageSize),
35+
page: z
36+
.number()
37+
.optional()
38+
.default(FEED_DEFAULTS.page)
39+
.catch(FEED_DEFAULTS.page),
40+
pageSize: z
41+
.number()
42+
.int()
43+
.positive()
44+
.optional()
45+
.default(FEED_DEFAULTS.pageSize)
46+
.catch(FEED_DEFAULTS.pageSize),
3747
viewMode: viewModeSchema,
3848
expanded: z.array(z.string()).optional().catch(undefined),
3949
})

src/routes/admin/roles.$roleId.tsx

Lines changed: 10 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -61,7 +61,10 @@ function RoleDetailPage() {
6161
})
6262
setSelectedUserIds(new Set())
6363
} catch (error) {
64-
console.error('Failed to remove users from role:', error instanceof Error ? error.message : 'Unknown error')
64+
console.error(
65+
'Failed to remove users from role:',
66+
error instanceof Error ? error.message : 'Unknown error',
67+
)
6568
alert(
6669
error instanceof Error
6770
? error.message
@@ -341,7 +344,12 @@ function RoleDetailPage() {
341344
})
342345
setConfirmRemove(null)
343346
} catch (error) {
344-
console.error('Failed to remove user from role:', error instanceof Error ? error.message : 'Unknown error')
347+
console.error(
348+
'Failed to remove user from role:',
349+
error instanceof Error
350+
? error.message
351+
: 'Unknown error',
352+
)
345353
alert(
346354
error instanceof Error
347355
? error.message

src/routes/admin/users.tsx

Lines changed: 12 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -533,7 +533,10 @@ function UsersPage() {
533533
setEditingUserId(null)
534534
setEditingRoleIds([])
535535
} catch (error) {
536-
console.error('Failed to update user:', error instanceof Error ? error.message : 'Unknown error')
536+
console.error(
537+
'Failed to update user:',
538+
error instanceof Error ? error.message : 'Unknown error',
539+
)
537540
alert(error instanceof Error ? error.message : 'Failed to update user')
538541
}
539542
}, [
@@ -607,7 +610,10 @@ function UsersPage() {
607610
setSelectedUserIds(new Set())
608611
setBulkActionRoleId(null)
609612
} catch (error) {
610-
console.error('Failed to assign role to users:', error instanceof Error ? error.message : 'Unknown error')
613+
console.error(
614+
'Failed to assign role to users:',
615+
error instanceof Error ? error.message : 'Unknown error',
616+
)
611617
alert(error instanceof Error ? error.message : 'Failed to assign role')
612618
}
613619
}, [selectedUserIds, bulkActionRoleId, bulkAssignRolesToUsers])
@@ -631,7 +637,10 @@ function UsersPage() {
631637
})
632638
setSelectedUserIds(new Set())
633639
} catch (error) {
634-
console.error('Failed to update user capabilities:', error instanceof Error ? error.message : 'Unknown error')
640+
console.error(
641+
'Failed to update user capabilities:',
642+
error instanceof Error ? error.message : 'Unknown error',
643+
)
635644
alert(
636645
error instanceof Error
637646
? error.message

src/routes/auth/signout.tsx

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,10 @@ export const Route = createFileRoute('/auth/signout')({
1818
await revokeUserSessions({ data: { userId: cookieData.userId } })
1919
} catch (error) {
2020
// Log but don't fail if revocation fails
21-
console.error('Failed to revoke sessions:', error instanceof Error ? error.message : 'Unknown error')
21+
console.error(
22+
'Failed to revoke sessions:',
23+
error instanceof Error ? error.message : 'Unknown error',
24+
)
2225
}
2326
}
2427
}

src/utils/cookies.server.ts

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -194,7 +194,10 @@ export async function verifyCookie(
194194
version,
195195
}
196196
} catch (error) {
197-
console.error('[verifyCookie] Error verifying cookie:', error instanceof Error ? error.message : 'Unknown error')
197+
console.error(
198+
'[verifyCookie] Error verifying cookie:',
199+
error instanceof Error ? error.message : 'Unknown error',
200+
)
198201
return null
199202
}
200203
}

src/utils/feedDefaults.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,4 +19,3 @@ export const FEED_DEFAULTS = {
1919
pageSize: 50,
2020
viewMode: 'table' as const,
2121
} as const
22-

src/utils/users.server.ts

Lines changed: 23 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -128,7 +128,10 @@ export const listUsers = createServerFn({ method: 'POST' })
128128
let allMatchingUsers: any[] = []
129129
let filteredUsers: any[] = []
130130

131-
if (useEffectiveCapabilities && (data.capabilityFilter?.length || data.noCapabilitiesFilter)) {
131+
if (
132+
useEffectiveCapabilities &&
133+
(data.capabilityFilter?.length || data.noCapabilitiesFilter)
134+
) {
132135
// Fetch all users matching other filters (without pagination)
133136
allMatchingUsers = await db
134137
.select()
@@ -139,9 +142,7 @@ export const listUsers = createServerFn({ method: 'POST' })
139142
// Get effective capabilities for all matching users
140143
const userIds = allMatchingUsers.map((u) => u.id)
141144
const effectiveCapabilitiesMap =
142-
userIds.length > 0
143-
? await getBulkEffectiveCapabilities(userIds)
144-
: {}
145+
userIds.length > 0 ? await getBulkEffectiveCapabilities(userIds) : {}
145146

146147
// Filter by effective capabilities
147148
filteredUsers = allMatchingUsers.filter((user) => {
@@ -174,22 +175,25 @@ export const listUsers = createServerFn({ method: 'POST' })
174175

175176
// Log performance metrics
176177
if (totalTime > 1000) {
177-
console.warn(`[listUsers] Slow query detected (effective capabilities):`, {
178-
authTime: `${authTime}ms`,
179-
selectTime: `${selectTime}ms`,
180-
totalTime: `${totalTime}ms`,
181-
filters: {
182-
emailFilter: !!data.emailFilter,
183-
nameFilter: !!data.nameFilter,
184-
capabilityFilter: !!data.capabilityFilter,
185-
noCapabilitiesFilter: data.noCapabilitiesFilter,
186-
useEffectiveCapabilities: true,
178+
console.warn(
179+
`[listUsers] Slow query detected (effective capabilities):`,
180+
{
181+
authTime: `${authTime}ms`,
182+
selectTime: `${selectTime}ms`,
183+
totalTime: `${totalTime}ms`,
184+
filters: {
185+
emailFilter: !!data.emailFilter,
186+
nameFilter: !!data.nameFilter,
187+
capabilityFilter: !!data.capabilityFilter,
188+
noCapabilitiesFilter: data.noCapabilitiesFilter,
189+
useEffectiveCapabilities: true,
190+
},
191+
pagination: { limit, pageIndex, offset },
192+
resultCount: page.length,
193+
totalMatching: allMatchingUsers.length,
194+
filteredCount,
187195
},
188-
pagination: { limit, pageIndex, offset },
189-
resultCount: page.length,
190-
totalMatching: allMatchingUsers.length,
191-
filteredCount,
192-
})
196+
)
193197
}
194198

195199
// Transform to match expected format

0 commit comments

Comments
 (0)