@lobehub/lobehub 2.0.0-next.324 → 2.0.0-next.326

package/src/services/models.ts

@@ -25,10 +25,8 @@ export type ProgressCallback = (progress: ModelProgressInfo) => void;
 export type ErrorCallback = (error: { message: string }) => void;
 
 export class ModelsService {
-  // Controller for aborting downloads
   private _abortController: AbortController | null = null;
 
-  // Get model list
   getModels = async (provider: string): Promise<ChatModelCard[] | undefined> => {
     const headers = await createHeaderWithAuth({
       headers: { 'Content-Type': 'application/json' },
@@ -66,7 +64,6 @@ export class ModelsService {
     { onProgress }: { onError?: ErrorCallback; onProgress?: ProgressCallback } = {},
   ): Promise<void> => {
     try {
-      // Create a new AbortController
       this._abortController = new AbortController();
       const signal = this._abortController.signal;
 
@@ -99,7 +96,6 @@ export class ModelsService {
         throw await getMessageError(res);
       }
 
-      // Process response stream
       if (res.body) {
         await this.processModelPullStream(res, { onProgress });
       }
@@ -112,14 +108,11 @@ export class ModelsService {
       console.error('download model error:', error);
       throw error;
     } finally {
-      // Clean up AbortController
       this._abortController = null;
     }
   };
 
-  // Abort model download
   abortPull = () => {
-    // Use AbortController to abort download
     if (this._abortController) {
       this._abortController.abort();
       this._abortController = null;
@@ -136,17 +129,14 @@ export class ModelsService {
     response: Response,
     { onProgress, onError }: { onError?: ErrorCallback; onProgress?: ProgressCallback },
   ): Promise<void> => {
-    // Process response stream
     const reader = response.body?.getReader();
     if (!reader) return;
 
-    // Read and process stream data
     // eslint-disable-next-line no-constant-condition
     while (true) {
       const { done, value } = await reader.read();
       if (done) break;
 
-      // Parse progress data
       const progressText = new TextDecoder().decode(value);
       // One line may contain multiple progress updates
       const progressUpdates = progressText.trim().split('\n');
@@ -162,10 +152,6 @@ export class ModelsService {
 
         if (progress.status === 'canceled') {
           console.log('progress:', progress);
-          // const abortError = new Error('abort');
-          // abortError.name = 'AbortError';
-          //
-          // throw abortError;
         }
 
         if (progress.status === 'error') {
@@ -173,7 +159,6 @@ export class ModelsService {
           throw new Error(progress.error);
         }
 
-        // Call progress callback
         if (progress.completed !== undefined || progress.status) {
           onProgress?.(progress);
         }

package/src/store/chat/agents/GroupOrchestration/createGroupOrchestrationExecutors.ts

@@ -120,11 +120,6 @@ export const createGroupOrchestrationExecutors = (
         };
       }
 
-      // Variable to capture the decision from tool handler
-      // let decision: ExecutorResult['type'] | undefined;
-      // let decisionParams: Record<string, unknown> = {};
-      // let skipCallSupervisor = false;
-
       // Execute Supervisor agent with the supervisor's agentId in context
       // Mark isSupervisor=true so assistant messages get metadata.isSupervisor for UI rendering
       // Note: Don't pass operationId - let it create a new child operation (same as call_agent)
@@ -139,10 +134,6 @@ export const createGroupOrchestrationExecutors = (
 
       log(`[${sessionLogId}] Supervisor agent finished`);
 
-      // Check what decision was made by the supervisor
-      // This is captured from the groupOrchestration callbacks registered by tools
-      // const orchestrationCallbacks = get().getGroupOrchestrationCallbacks();
-
       // If no tool was called (supervisor finished normally), end orchestration
       // The actual decision is captured via the afterCompletion callbacks
       // For now, return a finish decision if we reach here

package/src/store/chat/slices/aiChat/actions/conversationLifecycle.ts

@@ -399,9 +399,6 @@ export const conversationLifecycle: StateCreator<
         skipCreateFirstMessage: true,
       });
 
-      //
-      // // if there is relative files, then add files to agent
-      // // only available in server mode
       const userFiles = dbMessageSelectors
         .dbUserFiles(get())
         .map((f) => f?.id)

package/src/store/chat/slices/aiChat/actions/index.ts

@@ -14,9 +14,7 @@ export interface ChatAIChatAction
     ConversationLifecycleAction,
     ConversationControlAction,
     StreamingExecutorAction,
-    StreamingStatesAction {
-  /**/
-}
+    StreamingStatesAction {}
 
 export const chatAiChat: StateCreator<
   ChatStore,

package/src/store/file/slices/chat/action.test.ts

@@ -16,7 +16,6 @@ vi.mock('@/components/AntdStaticMethods', () => ({
   },
 }));
 
-//  mock the arrayBuffer
 beforeAll(() => {
   Object.defineProperty(File.prototype, 'arrayBuffer', {
     writable: true,
@@ -33,7 +32,6 @@ beforeAll(() => {
 });
 
 beforeEach(() => {
-  // Reset all mocks before each test
   vi.resetAllMocks();
 });
 
@@ -41,7 +39,6 @@ describe('useFileStore:chat', () => {
   it('clearChatUploadFileList should clear the inputFilesList', () => {
     const { result } = renderHook(() => useStore());
 
-    // Populate the list to clear it later
     act(() => {
       useStore.setState({ chatUploadFileList: [{ id: 'abc' }] as any });
     });
@@ -54,90 +51,4 @@ describe('useFileStore:chat', () => {
 
     expect(result.current.chatUploadFileList).toEqual([]);
   });
-
-  // it('removeFile should call fileService.removeFile and update the store', async () => {
-  //   const { result } = renderHook(() => useStore());
-  //
-  //   const fileId = 'test-id';
-  //
-  //   // Mock the fileService.removeFile to resolve
-  //   vi.spyOn(fileService, 'removeFile').mockResolvedValue(undefined);
-  //
-  //   // Populate the list to remove an item later
-  //   act(() => {
-  //     useStore.setState(({ inputFilesList }) => ({ inputFilesList: [...inputFilesList, fileId] }));
-  //     //   // result.current.inputFilesList.push(fileId);
-  //   });
-  //
-  //   await act(async () => {
-  //     await result.current.removeFile(fileId);
-  //   });
-  //
-  //   expect(fileService.removeFile).toHaveBeenCalledWith(fileId);
-  //   expect(result.current.inputFilesList).toEqual([]);
-  // });
-
-  // describe('uploadFile', () => {
-  //   it('uploadFile should handle errors', async () => {
-  //     const { result } = renderHook(() => useStore());
-  //     const testFile = new File(['content'], 'test.png', { type: 'image/png' });
-  //
-  //     // 模拟 fileService.uploadFile 抛出错误
-  //     const errorMessage = 'Upload failed';
-  //     vi.spyOn(uploadService, 'uploadFile').mockRejectedValue(new Error(errorMessage));
-  //
-  //     // Mock console.error for testing
-  //
-  //     await act(async () => {
-  //       await result.current.uploadFile(testFile);
-  //     });
-  //
-  //     expect(uploadService.uploadFile).toHaveBeenCalledWith({
-  //       createdAt: testFile.lastModified,
-  //       data: await testFile.arrayBuffer(),
-  //       fileType: testFile.type,
-  //       name: testFile.name,
-  //       saveMode: 'local',
-  //       size: testFile.size,
-  //     });
-  //     // 由于上传失败，inputFilesList 应该没有变化
-  //     expect(result.current.inputFilesList).toEqual([]);
-  //
-  //     // 确保错误提示被调用
-  //     expect(notification.error).toHaveBeenCalled();
-  //   });
-  //
-  //   it('uploadFile should upload the file and update inputFilesList', async () => {
-  //     const { result } = renderHook(() => useStore());
-  //     const testFile = new File(['content'], 'test.png', { type: 'image/png' });
-  //
-  //     // 模拟 fileService.uploadFile 返回的数据
-  //     const uploadedFileData = {
-  //       createdAt: testFile.lastModified,
-  //       data: await testFile.arrayBuffer(),
-  //       fileType: testFile.type,
-  //       name: testFile.name,
-  //       saveMode: 'local',
-  //       size: testFile.size,
-  //     };
-  //
-  //     // Mock the fileService.uploadFile to resolve with uploadedFileData
-  //     vi.spyOn(uploadService, 'uploadFile').mockResolvedValue(uploadedFileData as DB_File);
-  //     vi.spyOn(fileService, 'createFile').mockResolvedValue({ id: 'new-file-id', url: '' });
-  //
-  //     await act(async () => {
-  //       await result.current.uploadFile(testFile);
-  //     });
-  //
-  //     expect(fileService.createFile).toHaveBeenCalledWith({
-  //       createdAt: testFile.lastModified,
-  //       data: await testFile.arrayBuffer(),
-  //       fileType: testFile.type,
-  //       name: testFile.name,
-  //       saveMode: 'local',
-  //       size: testFile.size,
-  //     });
-  //     expect(result.current.inputFilesList).toContain('new-file-id');
-  //   });
-  // });
 });

package/src/store/file/slices/chunk/selectors.ts

@@ -1,4 +1,3 @@
-// import { FileStore } from '../../store';
 import { type FilesStoreState } from '@/store/file/initialState';
 
 const showSimilaritySearchResult = (s: FilesStoreState) => s.isSimilaritySearch;

package/src/store/file/slices/fileManager/selectors.ts

@@ -1,4 +1,3 @@
-// import { FileStore } from '../../store';
 import { type FilesStoreState } from '@/store/file/initialState';
 import { type FileUploadStatus } from '@/types/files/upload';
 

package/src/store/file/slices/tts/selectors.ts

@@ -1,3 +1 @@
-// import { FileStore } from '../../store';
-
 export const ttsFilesSelectors = {};

package/src/store/tool/slices/customPlugin/index.ts

@@ -1,3 +1,2 @@
 export * from './action';
 export * from './initialState';
-// export * from './selectors';

package/src/store/tool/slices/mcpStore/index.ts

@@ -1,3 +1,2 @@
 export * from './action';
 export * from './initialState';
-// export * from './selectors';

package/src/store/tool/slices/oldStore/index.ts

@@ -1,3 +1,2 @@
 export * from './action';
 export * from './initialState';
-// export * from './selectors';

package/src/store/tool/slices/plugin/index.ts

@@ -1,3 +1,2 @@
 export * from './action';
 export * from './initialState';
-// export * from './selectors';

package/src/styles/global.ts

@@ -59,4 +59,10 @@ export default ({ token }: { prefixCls: string; token: Theme }) => css`
   .${CLASSNAMES.DropdownMenuTrigger}[data-popup-open]:not([data-no-highlight]) {
     background: ${token.colorFillTertiary};
   }
+
+  .ant-form-item-control:has([role='combobox'][aria-controls^='base-ui-']),
+  .ant-form-item-control:has([role='combobox'][aria-haspopup='listbox']) {
+    width: min(70%, 800px);
+    min-width: min(70%, 800px) !important;
+  }
 `;

package/src/utils/router.tsx

@@ -145,13 +145,7 @@ export interface RouteConfig {
  */
 export function renderRoutes(routes: RouteConfig[]): ReactElement[] {
   return routes.map((route, index) => {
-    const { path, element, children, index: isIndex, loader } = route;
-
-    // Handle redirect loaders (convert to Navigate element)
-    if (loader && !element && isIndex) {
-      // Check if loader is a redirect by inspecting it
-      // For now, we'll handle this case in the config itself
-    }
+    const { path, element, children, index: isIndex } = route;
 
     const childRoutes = children ? renderRoutes(children) : undefined;
 

package/src/utils/server/parseModels.ts

@@ -26,7 +26,6 @@ export const parseModelString = async (
 
     if (withDeploymentName) {
       [id, deploymentName] = id.split('->');
-      // if (!deploymentName) deploymentName = id;
     }
 
     if (disable) {