refactor: remove redundant ensureUserTableExists calls

Author: 夏一飞

app/api/users/route.ts

@@ -1,8 +1,12 @@
 import { NextRequest, NextResponse } from "next/server";
 import { query } from "@/lib/db/client";
+import { ensureUserTableExists } from "@/lib/db/users";
 
 export async function GET(req: NextRequest) {
   try {
+    // 确保表结构正确
+    await ensureUserTableExists();
+
     const { searchParams } = new URL(req.url);
     const page = parseInt(searchParams.get("page") || "1");
     const pageSize = parseInt(searchParams.get("pageSize") || "20");

lib/db/users.ts

@@ -134,7 +134,6 @@ async function ensureDeletedColumnExists() {
 }
 
 export async function deleteUser(userId: string) {
-  await ensureUserTableExists();
   await ensureDeletedColumnExists();
 
   const updateResult = await query(
@@ -163,7 +162,6 @@ export async function getUsers({
   sortOrder = null,
   search = null,
 }: GetUsersOptions = {}) {
-  await ensureUserTableExists();
   await ensureDeletedColumnExists();
 
   const offset = (page - 1) * pageSize;
@@ -221,9 +219,6 @@ export async function getUsers({
 }
 
 export async function getAllUsers(includeDeleted: boolean = false) {
-  await ensureUserTableExists();
-  await ensureDeletedColumnExists();
-
   const whereClause = includeDeleted
     ? ""
     : "WHERE (deleted = FALSE OR deleted IS NULL)";