Merge branch 'main' into feat/Crokily/mdEditor

Author: Crokily

AGENT.md

@@ -1,5 +1,7 @@
 # AGENT Guidelines
 
+**你应该和用户说中文**
+
 Welcome! This repository hosts the Involution Hell documentation site, built with Next.js App Router, Fumadocs UI, TypeScript, and Tailwind CSS. Follow the instructions below whenever you contribute changes.
 
 ## General workflow

app/components/DocsAssistant.tsx

@@ -1,6 +1,6 @@
 "use client";
 
-import { useCallback, useEffect, useState, useRef } from "react";
+import { useCallback, useEffect, useMemo } from "react";
 
 import { AssistantRuntimeProvider } from "@assistant-ui/react";
 import { useAISDKRuntime } from "@assistant-ui/react-ai-sdk";
@@ -34,69 +34,44 @@ export function DocsAssistant({ pageContext }: DocsAssistantProps) {
 function DocsAssistantInner({ pageContext }: DocsAssistantProps) {
   const { provider, openaiApiKey, geminiApiKey } = useAssistantSettings();
 
-  // Use refs to ensure we always get the latest values
-  const providerRef = useRef(provider);
-  const openaiApiKeyRef = useRef(openaiApiKey);
-  const geminiApiKeyRef = useRef(geminiApiKey);
-
-  // Update refs whenever the values change
-  providerRef.current = provider;
-  openaiApiKeyRef.current = openaiApiKey;
-  geminiApiKeyRef.current = geminiApiKey;
-
-  const chat = useChat({
-    transport: new DefaultChatTransport({
-      api: "/api/chat",
-      body: () => {
-        // Use refs to get the current values at request time
-        const currentProvider = providerRef.current;
-        const currentApiKey =
-          currentProvider === "openai"
-            ? openaiApiKeyRef.current
-            : currentProvider === "gemini"
-              ? geminiApiKeyRef.current
-              : ""; // intern provider doesn't need API key
-
-        console.log("[DocsAssistant] useChat body function called with:", {
-          provider: currentProvider,
-          apiKeyLength: currentApiKey.length,
-          hasApiKey: currentApiKey.trim().length > 0,
-        });
-
-        return {
-          pageContext,
-          provider: currentProvider,
-          apiKey: currentApiKey,
-        };
-      },
-    }),
-  });
+  const transport = useMemo(
+    () =>
+      new DefaultChatTransport({
+        api: "/api/chat",
+        body: () => {
+          const apiKey =
+            provider === "openai"
+              ? openaiApiKey
+              : provider === "gemini"
+                ? geminiApiKey
+                : ""; // intern provider doesn't need API key
+
+          return { pageContext, provider, apiKey };
+        },
+      }),
+    [geminiApiKey, openaiApiKey, pageContext, provider],
+  );
+
+  const chat = useChat({ transport });
 
   const {
     error: chatError,
     status: chatStatus,
     clearError: clearChatError,
   } = chat;
-  const [assistantError, setAssistantError] =
-    useState<AssistantErrorState | null>(null);
-
-  useEffect(() => {
-    if (!chatError) {
-      return;
-    }
-
-    setAssistantError(deriveAssistantError(chatError, provider));
-    clearChatError();
-  }, [chatError, clearChatError, provider]);
 
   useEffect(() => {
     if (chatStatus === "submitted" || chatStatus === "streaming") {
-      setAssistantError(null);
+      clearChatError();
     }
-  }, [chatStatus]);
+  }, [chatStatus, clearChatError]);
+
+  const assistantError =
+    chatError && chatStatus !== "submitted" && chatStatus !== "streaming"
+      ? deriveAssistantError(chatError, provider)
+      : null;
 
   const handleClearError = useCallback(() => {
-    setAssistantError(null);
     clearChatError();
   }, [clearChatError]);
 

app/components/Footer.tsx

@@ -1,3 +1,4 @@
+import Link from "next/link";
 import { Github, MessageCircle, ExternalLink } from "lucide-react";
 import { BrandMark } from "./BrandMark";
 import { LicenseNotice } from "./LicenseNotice";
@@ -47,13 +48,13 @@ export function Footer() {
             <h3 className="font-semibold mb-4">快速链接</h3>
             <ul className="space-y-2">
               <li>
-                <a
+                <Link
                   href="/docs/ai"
                   className="text-muted-foreground hover:text-foreground transition-colors flex items-center"
                 >
                   知识库
                   <ExternalLink className="ml-1 h-3 w-3" aria-hidden="true" />
-                </a>
+                </Link>
               </li>
               <li>
                 <a

app/components/LicenseNotice.tsx

@@ -1,3 +1,4 @@
+import Image from "next/image";
 import { cn } from "@/lib/utils";
 
 interface LicenseNoticeProps {
@@ -19,44 +20,36 @@ export function LicenseNotice({ className }: LicenseNoticeProps) {
       <a href="https://creativecommons.org/licenses/by-nc-sa/4.0/">
         CC BY-NC-SA 4.0
       </a>
-      <img
+      <Image
         src="https://mirrors.creativecommons.org/presskit/icons/cc.svg"
-        alt=""
-        style={{
-          maxWidth: "1em",
-          maxHeight: "1em",
-          marginLeft: "0.2em",
-        }}
+        alt="Creative Commons"
+        width={16}
+        height={16}
+        className="ml-[0.2em] inline-block h-4 w-4"
         loading="lazy"
       />
-      <img
+      <Image
         src="https://mirrors.creativecommons.org/presskit/icons/by.svg"
-        alt=""
-        style={{
-          maxWidth: "1em",
-          maxHeight: "1em",
-          marginLeft: "0.2em",
-        }}
+        alt="Creative Commons BY"
+        width={16}
+        height={16}
+        className="ml-[0.2em] inline-block h-4 w-4"
         loading="lazy"
       />
-      <img
+      <Image
         src="https://mirrors.creativecommons.org/presskit/icons/nc.svg"
-        alt=""
-        style={{
-          maxWidth: "1em",
-          maxHeight: "1em",
-          marginLeft: "0.2em",
-        }}
+        alt="Creative Commons Non-Commercial"
+        width={16}
+        height={16}
+        className="ml-[0.2em] inline-block h-4 w-4"
         loading="lazy"
       />
-      <img
+      <Image
         src="https://mirrors.creativecommons.org/presskit/icons/sa.svg"
-        alt=""
-        style={{
-          maxWidth: "1em",
-          maxHeight: "1em",
-          marginLeft: "0.2em",
-        }}
+        alt="Creative Commons Share Alike"
+        width={16}
+        height={16}
+        className="ml-[0.2em] inline-block h-4 w-4"
         loading="lazy"
       />
     </p>

app/components/ThemeProvider.tsx

@@ -27,24 +27,21 @@ export function ThemeProvider({
   storageKey = "vite-ui-theme",
   ...props
 }: ThemeProviderProps) {
-  // SSR-safe: do not touch localStorage during render
-  const [theme, setTheme] = useState<Theme>(defaultTheme);
+  const [theme, setTheme] = useState<Theme>(() => {
+    if (typeof window === "undefined") {
+      return defaultTheme;
+    }
 
-  useEffect(() => {
-    // Read persisted theme on client only
     try {
-      const stored = (
-        typeof window !== "undefined"
-          ? (localStorage.getItem(storageKey) as Theme | null)
-          : null
-      ) as Theme | null;
-      if (stored) {
-        setTheme(stored);
-      }
+      const stored = localStorage.getItem(storageKey) as Theme | null;
+      return stored ?? defaultTheme;
     } catch {
       console.error("Error reading theme from localStorage");
+      return defaultTheme;
     }
+  });
 
+  useEffect(() => {
     const root = window.document.documentElement;
 
     root.classList.remove("light", "dark");
@@ -60,7 +57,7 @@ export function ThemeProvider({
     }
 
     root.classList.add(theme);
-  }, [theme, storageKey]);
+  }, [theme]);
 
   const value = {
     theme,

app/components/assistant-ui/attachment.tsx

@@ -1,6 +1,12 @@
 "use client";
 
-import { PropsWithChildren, useEffect, useState, type FC } from "react";
+import {
+  PropsWithChildren,
+  useEffect,
+  useMemo,
+  useState,
+  type FC,
+} from "react";
 import Image from "next/image";
 import { XIcon, PlusIcon, FileText } from "lucide-react";
 import {
@@ -31,21 +37,17 @@ import { TooltipIconButton } from "@/app/components/assistant-ui/tooltip-icon-bu
 import { cn } from "@/lib/utils";
 
 const useFileSrc = (file: File | undefined) => {
-  const [src, setSrc] = useState<string | undefined>(undefined);
+  const src = useMemo(
+    () => (file ? URL.createObjectURL(file) : undefined),
+    [file],
+  );
 
   useEffect(() => {
-    if (!file) {
-      setSrc(undefined);
-      return;
-    }
-
-    const objectUrl = URL.createObjectURL(file);
-    setSrc(objectUrl);
-
+    if (!src) return;
     return () => {
-      URL.revokeObjectURL(objectUrl);
+      URL.revokeObjectURL(src);
     };
-  }, [file]);
+  }, [src]);
 
   return src;
 };