@lobehub/lobehub 2.0.0-next.226 → 2.0.0-next.228

package/src/features/Conversation/Messages/AssistantGroup/Tool/Render/Intervention/ApprovalActions.tsx

@@ -1,5 +1,5 @@
-import { Button, Dropdown, Flexbox } from '@lobehub/ui';
-import { Input, Popover, Space } from 'antd';
+import { Button, Dropdown, Flexbox, Popover } from '@lobehub/ui';
+import { Input, Space } from 'antd';
 import { ChevronDown } from 'lucide-react';
 import { memo, useState } from 'react';
 import { useTranslation } from 'react-i18next';
@@ -80,7 +80,6 @@ const ApprovalActions = memo<ApprovalActionsProps>(
     return (
       <Flexbox gap={8} horizontal>
         <Popover
-          arrow={false}
           content={
             <Flexbox gap={12} style={{ width: 400 }}>
               <Flexbox align={'center'} horizontal justify={'space-between'}>

package/src/features/Conversation/Messages/components/Extras/Usage/UsageDetail/index.tsx

@@ -1,6 +1,6 @@
 import { type ModelPerformance, type ModelUsage } from '@lobechat/types';
-import { Center, Flexbox, Icon } from '@lobehub/ui';
-import { Divider, Popover } from 'antd';
+import { Center, Flexbox, Icon, Popover } from '@lobehub/ui';
+import { Divider } from 'antd';
 import { cssVar } from 'antd-style';
 import { BadgeCent, CoinsIcon } from 'lucide-react';
 import { memo } from 'react';
@@ -132,7 +132,6 @@ const TokenDetail = memo<TokenDetailProps>(({ usage, performance, model, provide
 
   return (
     <Popover
-      arrow={false}
       content={
         <Flexbox gap={8} style={{ minWidth: 200 }}>
           {modelCard && <ModelCard {...modelCard} provider={provider} />}
@@ -214,7 +213,7 @@ const TokenDetail = memo<TokenDetailProps>(({ usage, performance, model, provide
         </Flexbox>
       }
       placement={'top'}
-      trigger={['hover']}
+      trigger="hover"
     >
       <Center
         gap={2}

package/src/features/LocalFile/LocalFile.tsx

@@ -1,5 +1,5 @@
-import { Button, Flexbox } from '@lobehub/ui';
-import { Popover, Space } from 'antd';
+import { Button, Flexbox, Popover } from '@lobehub/ui';
+import { Space } from 'antd';
 import { createStaticStyles, cssVar } from 'antd-style';
 import { ExternalLink, FolderOpen } from 'lucide-react';
 import React from 'react';
@@ -98,12 +98,11 @@ export const LocalFile = ({ name, path, isDirectory = false }: LocalFileProps) =
 
   return (
     <Popover
-      arrow={false}
       content={popoverContent}
       styles={{
-        container: { padding: 0 },
+        content: { padding: 0 },
       }}
-      trigger={['hover']}
+      trigger="hover"
     >
       {fileContent}
     </Popover>

package/src/features/MCPPluginDetail/Deployment/index.tsx

@@ -1,7 +1,17 @@
 import { SiApple, SiLinux } from '@icons-pack/react-simple-icons';
 import { Microsoft } from '@lobehub/icons';
-import { ActionIcon, Block, Collapse, Empty, Flexbox, Icon, Snippet, Tag } from '@lobehub/ui';
-import { Divider, Popover, Steps } from 'antd';
+import {
+  ActionIcon,
+  Block,
+  Collapse,
+  Empty,
+  Flexbox,
+  Icon,
+  Popover,
+  Snippet,
+  Tag,
+} from '@lobehub/ui';
+import { Divider, Steps } from 'antd';
 import { createStaticStyles, cssVar } from 'antd-style';
 import { startCase } from 'es-toolkit/compat';
 import {
@@ -220,7 +230,6 @@ const Deployment = memo<{ mobile?: boolean }>(({ mobile }) => {
                               </span>
                               {dep.installInstructions && (
                                 <Popover
-                                  arrow={false}
                                   content={
                                     <Flexbox gap={8}>
                                       <Descriptions
@@ -265,7 +274,7 @@ const Deployment = memo<{ mobile?: boolean }>(({ mobile }) => {
                                       )}
                                     </Flexbox>
                                   }
-                                  trigger={['hover']}
+                                  trigger="hover"
                                 >
                                   <ActionIcon
                                     color={cssVar.colorTextDescription}

package/src/features/MCPPluginDetail/Score/TotalScore.tsx

@@ -1,5 +1,5 @@
-import { Block, Center, Flexbox } from '@lobehub/ui';
-import { Popover, Progress } from 'antd';
+import { Block, Center, Flexbox, Popover } from '@lobehub/ui';
+import { Progress } from 'antd';
 import { createStaticStyles, cssVar } from 'antd-style';
 import { memo } from 'react';
 import { useTranslation } from 'react-i18next';
@@ -248,10 +248,15 @@ const TotalScore = memo<TotalScoreProps>(({ scoreResult, scoreItems = [], isVali
 
       <div className={styles.progressContainer}>
         <Popover
-          arrow={false}
-          content={renderTooltipContent()}
+          content={
+            <div>
+              <div style={{ fontWeight: 'bold', marginBottom: 8 }}>
+                {t('mcp.details.totalScore.popover.title')}
+              </div>
+              {renderTooltipContent()}
+            </div>
+          }
           placement="bottom"
-          title={t('mcp.details.totalScore.popover.title')}
           trigger={['hover', 'click']}
         >
           <Progress

package/src/features/ModelSwitchPanel/index.tsx

@@ -1,5 +1,4 @@
-import { TooltipGroup } from '@lobehub/ui';
-import { Popover } from 'antd';
+import { Popover, TooltipGroup } from '@lobehub/ui';
 import { memo, useCallback, useState } from 'react';
 
 import { PanelContent } from './components/PanelContent';
@@ -32,9 +31,8 @@ const ModelSwitchPanel = memo<ModelSwitchPanelProps>(
     return (
       <TooltipGroup>
         <Popover
-          arrow={false}
           classNames={{
-            container: styles.container,
+            content: styles.container,
           }}
           content={
             <PanelContent

package/src/features/PageEditor/Copilot/Toolbar.tsx

@@ -1,5 +1,4 @@
-import { ActionIcon, Block, Flexbox } from '@lobehub/ui';
-import { Popover } from 'antd';
+import { ActionIcon, Block, Flexbox, Popover } from '@lobehub/ui';
 import { createStaticStyles, cx } from 'antd-style';
 import { ChevronsUpDownIcon, Clock3Icon, PanelRightCloseIcon, PlusIcon } from 'lucide-react';
 import { Suspense, memo, useMemo, useState } from 'react';
@@ -108,7 +107,6 @@ const AgentSelector = memo<AgentSelectorProps>(({ agentId, onAgentChange }) => {
 
   return (
     <Popover
-      arrow={false}
       content={
         <Suspense fallback={<SkeletonList rows={6} />}>
           <AgentModalProvider>
@@ -120,12 +118,12 @@ const AgentSelector = memo<AgentSelectorProps>(({ agentId, onAgentChange }) => {
       open={open}
       placement="bottomLeft"
       styles={{
-        container: {
+        content: {
           padding: 0,
           width: 240,
         },
       }}
-      trigger={['click']}
+      trigger="click"
     >
       <Block
         align={'center'}
@@ -197,7 +195,6 @@ const CopilotToolbar = memo<CopilotToolbarProps>(({ agentId, isHovered }) => {
             />
             {!hideHistory && (
               <Popover
-                arrow={false}
                 content={
                   <Flexbox
                     gap={4}
@@ -224,12 +221,12 @@ const CopilotToolbar = memo<CopilotToolbarProps>(({ agentId, isHovered }) => {
                 open={topicPopoverOpen}
                 placement="bottomRight"
                 styles={{
-                  container: {
+                  content: {
                     padding: 0,
                     width: 240,
                   },
                 }}
-                trigger={['click']}
+                trigger="click"
               >
                 <ActionIcon icon={Clock3Icon} size={DESKTOP_HEADER_ICON_SIZE} />
               </Popover>

package/src/server/routers/lambda/__tests__/file.test.ts

@@ -273,7 +273,7 @@ describe('fileRouter', () => {
       );
     });
 
-    it('should throw error when getFileMetadata fails and input size is less than 1', async () => {
+    it('should throw error when getFileMetadata fails and input size is negative', async () => {
       mockFileModelCheckHash.mockResolvedValue({ isExist: false });
       mockFileServiceGetFileMetadata.mockRejectedValue(new Error('File not found in S3'));
 
@@ -282,11 +282,11 @@ describe('fileRouter', () => {
           hash: 'test-hash',
           fileType: 'text',
           name: 'test.txt',
-          size: 0,
+          size: -1,
           url: 'files/non-existent.txt',
           metadata: {},
         }),
-      ).rejects.toThrow('File size must be at least 1 byte');
+      ).rejects.toThrow('File size cannot be negative');
     });
 
     it('should use input size when getFileMetadata returns contentLength less than 1', async () => {
@@ -315,10 +315,10 @@ describe('fileRouter', () => {
       );
     });
 
-    it('should throw error when both getFileMetadata contentLength and input size are less than 1', async () => {
+    it('should throw error when both getFileMetadata contentLength and input size are negative', async () => {
       mockFileModelCheckHash.mockResolvedValue({ isExist: false });
       mockFileServiceGetFileMetadata.mockResolvedValue({
-        contentLength: 0,
+        contentLength: -1,
         contentType: 'text/plain',
       });
 
@@ -327,11 +327,11 @@ describe('fileRouter', () => {
           hash: 'test-hash',
           fileType: 'text',
           name: 'test.txt',
-          size: 0,
+          size: -1,
           url: 'files/test.txt',
           metadata: {},
         }),
-      ).rejects.toThrow('File size must be at least 1 byte');
+      ).rejects.toThrow('File size cannot be negative');
     });
   });
 

package/src/server/routers/lambda/file.ts

@@ -1,6 +1,7 @@
 import { TRPCError } from '@trpc/server';
 import { z } from 'zod';
 
+import { businessFileUploadCheck } from '@/business/server/lambda-routers/file';
 import { checkFileStorageUsage } from '@/business/server/trpc-middlewares/lambda';
 import { serverDBEnv } from '@/config/db';
 import { AsyncTaskModel } from '@/database/models/asyncTask';
@@ -74,8 +75,16 @@ export const fileRouter = router({
         // If metadata fetch fails, use original size from input
       }
 
-      if (actualSize < 1) {
-        throw new TRPCError({ code: 'BAD_REQUEST', message: 'File size must be at least 1 byte' });
+      await businessFileUploadCheck({
+        actualSize,
+        clientIp: ctx.clientIp ?? undefined,
+        inputSize: input.size,
+        url: input.url,
+        userId: ctx.userId,
+      });
+
+      if (actualSize < 0) {
+        throw new TRPCError({ code: 'BAD_REQUEST', message: 'File size cannot be negative' });
       }
 
       const { id } = await ctx.fileModel.create(
@@ -367,7 +376,7 @@ export const fileRouter = router({
 
     if (!file) return;
 
-    // delele the file from remove from S3 if it is not used by other files
+    // delete the file from S3 if it is not used by other files
     await ctx.fileService.deleteFile(file.url!);
   }),