fix: improve error handling in bulk model deletion

- Added proper error serialization using serialize-error for better error logging
- Explicitly defined BulkDeleteModelsResponse type instead of relying on generated schema reference

Author: Pfannkuchensack

invokeai/frontend/web/src/features/modelManagerV2/subpanels/ModelManagerPanel/ModelList.tsx

@@ -2,6 +2,7 @@ import { Flex, Text, useDisclosure, useToast } from '@invoke-ai/ui-library';
 import { logger } from 'app/logging/logger';
 import { useAppDispatch, useAppSelector } from 'app/store/storeHooks';
 import ScrollableContent from 'common/components/OverlayScrollbars/ScrollableContent';
+import { serializeError } from 'serialize-error';
 import { MODEL_CATEGORIES_AS_LIST } from 'features/modelManagerV2/models';
 import {
   clearModelSelection,
@@ -114,8 +115,8 @@ const ModelList = () => {
       }
 
       log.info(`Bulk delete completed: ${result.deleted.length} deleted, ${result.failed.length} failed`);
-    } catch (error) {
-      log.error('Bulk delete error:', error);
+    } catch (err) {
+      log.error({ error: serializeError(err) }, 'Bulk delete error');
       toast({
         id: 'BULK_DELETE_ERROR',
         title: t('modelManager.modelsDeleteError', {

invokeai/frontend/web/src/services/api/endpoints/models.ts

@@ -46,8 +46,10 @@ type DeleteModelImageResponse = void;
 type BulkDeleteModelsArg = {
   keys: string[];
 };
-type BulkDeleteModelsResponse =
-  paths['/api/v2/models/i/bulk_delete']['post']['responses']['200']['content']['application/json'];
+type BulkDeleteModelsResponse = {
+  deleted: string[];
+  failed: string[];
+};
 
 type ConvertMainModelResponse =
   paths['/api/v2/models/convert/{key}']['put']['responses']['200']['content']['application/json'];

invokeai/frontend/web/src/services/api/schema.ts

@@ -167,6 +167,30 @@ export type paths = {
         patch: operations["update_model_image"];
         trace?: never;
     };
+    "/api/v2/models/i/bulk_delete": {
+        parameters: {
+            query?: never;
+            header?: never;
+            path?: never;
+            cookie?: never;
+        };
+        get?: never;
+        put?: never;
+        /**
+         * Bulk Delete Models
+         * @description Delete multiple model records from database.
+         *
+         *     The configuration records will be removed. The corresponding weights files will be
+         *     deleted as well if they reside within the InvokeAI "models" directory.
+         *     Returns a list of successfully deleted keys and failed deletions with error messages.
+         */
+        post: operations["bulk_delete_models"];
+        delete?: never;
+        options?: never;
+        head?: never;
+        patch?: never;
+        trace?: never;
+    };
     "/api/v2/models/install": {
         parameters: {
             query?: never;
@@ -3006,6 +3030,35 @@ export type components = {
              */
             type: "bounding_box_output";
         };
+        /**
+         * BulkDeleteModelsRequest
+         * @description Request body for bulk model deletion.
+         */
+        BulkDeleteModelsRequest: {
+            /**
+             * Keys
+             * @description List of model keys to delete
+             */
+            keys: string[];
+        };
+        /**
+         * BulkDeleteModelsResponse
+         * @description Response body for bulk model deletion.
+         */
+        BulkDeleteModelsResponse: {
+            /**
+             * Deleted
+             * @description List of successfully deleted model keys
+             */
+            deleted: string[];
+            /**
+             * Failed
+             * @description List of failed deletions with error messages
+             */
+            failed: {
+                [key: string]: unknown;
+            }[];
+        };
         /**
          * BulkDownloadCompleteEvent
          * @description Event model for bulk_download_complete
@@ -12940,14 +12993,14 @@ export type components = {
              * Convert Cache Dir
              * Format: path
              * @description Path to the converted models cache directory (DEPRECATED, but do not delete because it is needed for migration from previous versions).
-             * @default models/.convert_cache
+             * @default models\.convert_cache
              */
             convert_cache_dir?: string;
             /**
              * Download Cache Dir
              * Format: path
              * @description Path to the directory that contains dynamically downloaded models.
-             * @default models/.download_cache
+             * @default models\.download_cache
              */
             download_cache_dir?: string;
             /**
@@ -24943,6 +24996,39 @@ export interface operations {
             };
         };
     };
+    bulk_delete_models: {
+        parameters: {
+            query?: never;
+            header?: never;
+            path?: never;
+            cookie?: never;
+        };
+        requestBody: {
+            content: {
+                "application/json": components["schemas"]["BulkDeleteModelsRequest"];
+            };
+        };
+        responses: {
+            /** @description Models deleted (possibly with some failures) */
+            200: {
+                headers: {
+                    [name: string]: unknown;
+                };
+                content: {
+                    "application/json": components["schemas"]["BulkDeleteModelsResponse"];
+                };
+            };
+            /** @description Validation Error */
+            422: {
+                headers: {
+                    [name: string]: unknown;
+                };
+                content: {
+                    "application/json": components["schemas"]["HTTPValidationError"];
+                };
+            };
+        };
+    };
     list_model_installs: {
         parameters: {
             query?: never;