refactor: Simplify function parameters and improve logging in Gitee login route

normal-coder · normal-coder · commit a9115c30dfe6 · 2025-04-18T02:42:13.000+08:00
- Removed unnecessary request parameter from GET function in Gitee login route.
- Updated console log messages for consistency in language.
- Streamlined icon component definitions in Calendar and adjusted color config filtering in Chart.
- Simplified action type definitions in use-toast by removing redundant constants.
diff --git a/app/api/auth/login/gitee/route.ts b/app/api/auth/login/gitee/route.ts
@@ -1,6 +1,6 @@
-import { type NextRequest, NextResponse } from "next/server"
+import { NextResponse } from "next/server"
 
-export async function GET(request: NextRequest) {
+export async function GET() {
   try {
     // 确保环境变量存在，否则使用硬编码的值（仅用于开发）
     const clientId = process.env.GITEE_CLIENT_ID || "5e96ca868817fa1b190d14e20ffd7b19f03f2c9aa6c064dda3bfe9e715ee8dd2"
@@ -14,12 +14,12 @@ export async function GET(request: NextRequest) {
     // 构建 Gitee 授权 URL
     const authUrl = `https://gitee.com/oauth/authorize?client_id=${clientId}&redirect_uri=${redirectUri}&response_type=code`
 
-    console.log("重定向到:", authUrl)
+    console.log("重定向到：", authUrl)
 
     // 重定向到 Gitee 授权页面
     return NextResponse.redirect(authUrl)
   } catch (error) {
-    console.error("Gitee 登录路由错误:", error)
+    console.error("Gitee 登录路由错误：", error)
 
     // 返回错误响应而不是重定向
     return new NextResponse(
diff --git a/components/ui/calendar.tsx b/components/ui/calendar.tsx
@@ -54,8 +54,8 @@ function Calendar({
         ...classNames,
       }}
       components={{
-        IconLeft: ({ ...props }) => <ChevronLeft className="h-4 w-4" />,
-        IconRight: ({ ...props }) => <ChevronRight className="h-4 w-4" />,
+        IconLeft: () => <ChevronLeft className="h-4 w-4" />,
+        IconRight: () => <ChevronRight className="h-4 w-4" />,
       }}
       {...props}
     />
diff --git a/components/ui/chart.tsx b/components/ui/chart.tsx
@@ -69,7 +69,7 @@ ChartContainer.displayName = "Chart"
 
 const ChartStyle = ({ id, config }: { id: string; config: ChartConfig }) => {
   const colorConfig = Object.entries(config).filter(
-    ([_, config]) => config.theme || config.color
+    ([, config]) => config.theme || config.color
   )
 
   if (!colorConfig.length) {
diff --git a/components/ui/use-toast.ts b/components/ui/use-toast.ts
@@ -15,37 +15,28 @@ type ToasterToast = ToastProps & {
   action?: ToastActionElement
 }
 
-const actionTypes = {
-  ADD_TOAST: "ADD_TOAST",
-  UPDATE_TOAST: "UPDATE_TOAST",
-  DISMISS_TOAST: "DISMISS_TOAST",
-  REMOVE_TOAST: "REMOVE_TOAST",
-} as const
-
 let count = 0
 
 function genId() {
   count = (count + 1) % Number.MAX_VALUE
   return count.toString()
 }
 
-type ActionType = typeof actionTypes
-
 type Action =
   | {
-      type: ActionType["ADD_TOAST"]
+      type: "ADD_TOAST"
       toast: ToasterToast
     }
   | {
-      type: ActionType["UPDATE_TOAST"]
+      type: "UPDATE_TOAST"
       toast: Partial<ToasterToast>
     }
   | {
-      type: ActionType["DISMISS_TOAST"]
+      type: "DISMISS_TOAST"
       toastId?: string
     }
   | {
-      type: ActionType["REMOVE_TOAST"]
+      type: "REMOVE_TOAST"
       toastId?: string
     }
 

Original file line number	Diff line number	Diff line change
`@@ -69,7 +69,7 @@ ChartContainer.displayName = "Chart"`
`69`	`69`
`70`	`70`	`const ChartStyle = ({ id, config }: { id: string; config: ChartConfig }) => {`
`71`	`71`	`const colorConfig = Object.entries(config).filter(`
`72`		`- ([_, config]) => config.theme \|\| config.color`
	`72`	`+ ([, config]) => config.theme \|\| config.color`
`73`	`73`	`)`
`74`	`74`
`75`	`75`	`if (!colorConfig.length) {`