clear chinese comment and update model

Author: pengyu

backend/src/build-system/__tests__/test.frontend-code-generate.spec.ts

@@ -12,7 +12,7 @@ describe('FrontendCodeHandler', () => {
     name: 'Spotify-like Music Web',
     description: 'Users can play music',
     databaseType: 'SQLite',
-    model: 'o3-mini-high',
+    model: 'o4-mini',
     nodes: [
       {
         handler: FrontendCodeHandler,

backend/src/build-system/context.ts

@@ -115,7 +115,7 @@ export class BuilderContext {
           this.globalContext.set('projectSize', 'small');
           break;
         case 'gpt-4o':
-        case 'o3-mini-high':
+        case 'o4-mini':
           this.globalContext.set('projectSize', 'medium');
           break;
         default:

backend/src/build-system/handlers/frontend-code-generate/CodeReview.ts

@@ -198,7 +198,7 @@ export class FrontendQueueProcessor {
       let fixResponse = await chatSyncWithClocker(
         this.context,
         {
-          model: 'o3-mini-high',
+          model: 'o4-mini',
           messages: [
             { role: 'system', content: fixPrompt },
             {
@@ -270,7 +270,7 @@ export class FrontendQueueProcessor {
         fixResponse = await chatSyncWithClocker(
           this.context,
           {
-            model: 'o3-mini-high',
+            model: 'o4-mini',
             messages: [
               { role: 'system', content: fixPrompt },
               {

backend/src/build-system/handlers/frontend-code-generate/index.ts

@@ -366,8 +366,8 @@ export class FrontendCodeHandler implements BuildHandler<string> {
         context,
         {
           model: isSPAFlag
-            ? 'claude-3.7-sonnet' // Use Claude for SPAs
-            : 'o3-mini-high', // Use default or fallback for non-SPAs
+            ? 'gpt-4o-mini' // Use Claude for SPAs
+            : 'o4-mini', // Use default or fallback for non-SPAs
           messages,
         },
         'generate frontend code',

backend/src/build-system/handlers/ux/uiux-layout/index.ts

@@ -75,7 +75,7 @@ export class UIUXLayoutHandler implements BuildHandler<string> {
         context,
         {
           // model: context.defaultModel || 'gpt-4o-mini',
-          model: 'claude-3.7-sonnet',
+          model: 'gpt-4o-mini',
           messages,
         },
         'generateUIUXLayout',

backend/src/user/user.model.ts

@@ -22,7 +22,7 @@ export class User extends SystemBaseModel {
   id: string;
 
   @Field()
-  @Column()
+  @Column({ unique: true })
   username: string;
 
   @Column()

backend/src/user/user.service.ts

@@ -1,5 +1,6 @@
 import {
   BadRequestException,
+  ConflictException,
   Injectable,
   NotFoundException,
 } from '@nestjs/common';
@@ -107,4 +108,69 @@ export class UserService {
 
     return true;
   }
+
+  /**
+   * Checks if a username already exists in the database
+   * @param username Username to check
+   * @param excludeUserId Optional user ID to exclude from the check (for updates)
+   * @returns Boolean indicating if the username exists
+   */
+  async isUsernameExists(
+    username: string,
+    excludeUserId?: string,
+  ): Promise<boolean> {
+    const query = this.userRepository
+      .createQueryBuilder('user')
+      .where('LOWER(user.username) = LOWER(:username)', {
+        username: username.toLowerCase(),
+      });
+
+    if (excludeUserId) {
+      query.andWhere('user.id != :userId', { userId: excludeUserId });
+    }
+
+    const count = await query.getCount();
+    return count > 0;
+  }
+
+  /**
+   * Updates a user's username with uniqueness validation
+   * @param userId User ID
+   * @param newUsername New username to set
+   * @returns Updated user object
+   */
+  async updateUsername(userId: string, newUsername: string): Promise<User> {
+    if (!newUsername || newUsername.trim().length < 3) {
+      throw new BadRequestException(
+        'Username must be at least 3 characters long',
+      );
+    }
+
+    // Check if the username is already taken
+    const exists = await this.isUsernameExists(newUsername, userId);
+    if (exists) {
+      throw new ConflictException(
+        `Username '${newUsername}' is already taken. Please choose another one.`,
+      );
+    }
+
+    const user = await this.userRepository.findOneBy({ id: userId });
+    if (!user) {
+      throw new NotFoundException('User not found');
+    }
+
+    user.username = newUsername;
+
+    try {
+      return await this.userRepository.save(user);
+    } catch (error) {
+      // Check for unique constraint error (just in case of race condition)
+      if (error.code === '23505' || error.message.includes('duplicate')) {
+        throw new ConflictException(
+          `Username '${newUsername}' is already taken. Please choose another one.`,
+        );
+      }
+      throw error;
+    }
+  }
 }

frontend/src/app/api/screenshot/route.ts

@@ -33,59 +33,72 @@ export async function GET(req: Request) {
     );
   }
 
-  logger.info(`Starting screenshot for URL: ${url}`);
+  logger.info(`[SCREENSHOT] Starting screenshot for URL: ${url}`);
 
   try {
     // Get browser instance
+    logger.info(`[SCREENSHOT] Attempting to get browser instance`);
     const browser = await getBrowser();
-    logger.info('Browser instance acquired');
+    logger.info(`[SCREENSHOT] Browser instance acquired successfully`);
 
     // Create a new page
+    logger.info(`[SCREENSHOT] Creating new page`);
     page = await browser.newPage();
-    logger.info('New page created');
+    logger.info(`[SCREENSHOT] New page created successfully`);
 
     // Set viewport
+    logger.info(`[SCREENSHOT] Setting viewport`);
     await page.setViewport({ width: 1600, height: 900 });
-    logger.info('Viewport set to 1600x900');
+    logger.info(`[SCREENSHOT] Viewport set successfully`);
 
     // Navigate to the URL
-    logger.info(`Navigating to URL: ${url}`);
+    logger.info(`[SCREENSHOT] Navigating to URL: ${url}`);
     await page.goto(url, {
-      waitUntil: 'domcontentloaded',
-      timeout: 60000,
+      waitUntil: 'networkidle2',
+      timeout: 90000,
     });
-    logger.info('Page loaded (DOMContentLoaded)');
+    logger.info(`[SCREENSHOT] Page loaded successfully`);
 
     // Extra wait for rendering
-    await new Promise((resolve) => setTimeout(resolve, 2000));
-    logger.info('Waited 2 seconds for rendering');
+    logger.info(`[SCREENSHOT] Waiting additional time for rendering`);
+    await new Promise((resolve) => setTimeout(resolve, 5000));
+    logger.info(`[SCREENSHOT] Additional wait completed`);
 
     // Take screenshot
+    logger.info(`[SCREENSHOT] Taking screenshot`);
     const screenshot = await page.screenshot({
       type: 'png',
       fullPage: true,
     });
-    logger.info('Screenshot captured');
+    logger.info(`[SCREENSHOT] Screenshot captured successfully, size: ${screenshot.length} bytes`);
 
     // Clean up
-    if (page) await page.close();
-    logger.info('Page closed');
+    if (page) {
+      logger.info(`[SCREENSHOT] Closing page`);
+      await page.close();
+      logger.info(`[SCREENSHOT] Page closed successfully`);
+    }
 
+    logger.info(`[SCREENSHOT] Returning screenshot response`);
     return new Response(screenshot, {
       headers: {
         'Content-Type': 'image/png',
-        'Cache-Control': 's-maxage=3600',
+        'Cache-Control': 'no-cache, no-store, must-revalidate',
+        'Pragma': 'no-cache',
+        'Expires': '0'
       },
     });
   } catch (error: any) {
-    logger.error('Screenshot error:', error);
+    logger.error(`[SCREENSHOT] Error capturing screenshot: ${error.message}`, error);
+    logger.error(`[SCREENSHOT] Error stack: ${error.stack}`);
 
     if (page) {
       try {
+        logger.info(`[SCREENSHOT] Attempting to close page after error`);
         await page.close();
-        logger.info('Closed page after error');
+        logger.info(`[SCREENSHOT] Successfully closed page after error`);
       } catch (closeError) {
-        logger.error('Error closing page:', closeError);
+        logger.error(`[SCREENSHOT] Error closing page: ${closeError.message}`);
       }
     }
 
@@ -96,12 +109,13 @@ export async function GET(req: Request) {
     ) {
       try {
         if (browserInstance) {
+          logger.warn(`[SCREENSHOT] Resetting browser instance due to protocol error`);
           await browserInstance.close();
-          logger.warn('Browser instance was closed due to protocol error');
           browserInstance = null;
+          logger.warn(`[SCREENSHOT] Browser instance reset successfully`);
         }
       } catch (closeBrowserError) {
-        logger.error('Error closing browser:', closeBrowserError);
+        logger.error(`[SCREENSHOT] Error closing browser: ${closeBrowserError.message}`);
       }
     }
 

frontend/src/app/globals.css

@@ -82,7 +82,7 @@
   height: 1px;
 }
 
-/* 修改滚动条颜色为主题色调 */
+/* Modify scrollbar color to match theme */
 :root {
   --scrollbar-thumb-color: rgba(
     99,

frontend/src/app/media/[...path]/route.ts

@@ -4,7 +4,6 @@ import path from 'path';
 import { getMediaDir } from 'codefox-common';
 import { logger } from '@/app/log/logger';
 
-
 export async function GET(
   request: NextRequest,
   { params }: { params: { path: string[] } }
@@ -15,7 +14,7 @@ export async function GET(
     const filePath = path.join(mediaDir, ...params.path);
     const normalizedPath = path.normalize(filePath);
     logger.info(`📁 getMediaDir = ${mediaDir}`);
-logger.info(`📂 full filePath = ${filePath}`);
+    logger.info(`📂 full filePath = ${filePath}`);
     logger.debug(`Requested path: ${params.path.join('/')}`);
     logger.debug(`Full resolved path: ${filePath}`);