From d04f061120fc8adbc2ed22e39024757cb25033c0 Mon Sep 17 00:00:00 2001 From: RevIQ Date: Fri, 9 Jan 2026 17:13:14 +0800 Subject: [PATCH] Fix lint and formatting issues - Sort imports alphabetically in helpers.ts and router.ts - Simplify boolean comparison in users/update.ts Co-Authored-By: Claude Opus 4.5 --- apps/api-server/src/procedures/admin/helpers.ts | 2 +- apps/api-server/src/procedures/admin/users/update.ts | 5 +---- apps/api-server/src/router.ts | 8 ++++++-- 3 files changed, 8 insertions(+), 7 deletions(-) diff --git a/apps/api-server/src/procedures/admin/helpers.ts b/apps/api-server/src/procedures/admin/helpers.ts index 9043a46..d4614b8 100644 --- a/apps/api-server/src/procedures/admin/helpers.ts +++ b/apps/api-server/src/procedures/admin/helpers.ts @@ -2,8 +2,8 @@ * Admin procedure helpers - shared transformation functions */ +import type { OrgSites, Orgs, Users } from "@reviq/db-schema"; import type { Selectable } from "kysely"; -import type { Orgs, OrgSites, Users } from "@reviq/db-schema"; /** Transform org record to API response format */ export const toOrgResponse = (org: Selectable) => ({ diff --git a/apps/api-server/src/procedures/admin/users/update.ts b/apps/api-server/src/procedures/admin/users/update.ts index 8d6ef17..ddfc353 100644 --- a/apps/api-server/src/procedures/admin/users/update.ts +++ b/apps/api-server/src/procedures/admin/users/update.ts @@ -27,10 +27,7 @@ export const adminUsersUpdate = os.admin.users.update } // Prevent superuser from demoting themselves - if ( - isSuperuser === false && - normalizedEmail === context.user.email.toLowerCase() - ) { + if (!isSuperuser && normalizedEmail === context.user.email.toLowerCase()) { throw new ORPCError("BAD_REQUEST", { message: "Cannot remove your own superuser status", }); diff --git a/apps/api-server/src/router.ts b/apps/api-server/src/router.ts index e76f87f..a90fbfb 100644 --- a/apps/api-server/src/router.ts +++ b/apps/api-server/src/router.ts @@ -1,4 +1,5 @@ import { ORPCError } from "@orpc/server"; +import { adminRoutes } from "./procedures/admin/_routes.js"; import { createLoginRequest as createLoginRequestHandler } from "./procedures/auth/create-login-request.js"; import { forgotPassword as forgotPasswordHandler } from "./procedures/auth/forgot-password.js"; import { loginIfRequestIsCompleted as loginIfRequestIsCompletedHandler } from "./procedures/auth/login-if-completed.js"; @@ -9,8 +10,11 @@ import { resendVerificationEmail as resendVerificationHandler } from "./procedur import { resetPassword as resetPasswordHandler } from "./procedures/auth/reset-password.js"; import { signup as signupHandler } from "./procedures/auth/signup.js"; import { verifyEmail as verifyEmailHandler } from "./procedures/auth/verify-email.js"; -import { authMiddleware, loginRequestMiddleware, os } from "./procedures/base.js"; -import { adminRoutes } from "./procedures/admin/_routes.js"; +import { + authMiddleware, + loginRequestMiddleware, + os, +} from "./procedures/base.js"; import { meDelete } from "./procedures/me/delete.js"; import { getDeviceInfo,