@adminforth/bulk-ai-flow 1.14.0 → 1.14.1

package/dist/index.js

@@ -80,6 +80,15 @@ export default class BulkAiFlowPlugin extends AdminForthPlugin {
                 return {};
             }
             else if (attachmentFiles.length !== 0) {
+                try {
+                    for (const fileUrl of attachmentFiles) {
+                        new URL(fileUrl);
+                    }
+                }
+                catch (e) {
+                    jobs.set(jobId, { status: 'failed', error: 'One of the image URLs is not valid' });
+                    return { ok: false, error: 'One of the image URLs is not valid' };
+                }
                 //create prompt for OpenAI
                 const compiledOutputFields = this.compileOutputFieldsTemplates(record);
                 const prompt = `Analyze the following image(s) and return a single JSON in format like: {'param1': 'value1', 'param2': 'value2'}. 
@@ -107,7 +116,14 @@ export default class BulkAiFlowPlugin extends AdminForthPlugin {
                         jobs.set(jobId, { status: 'failed', error: 'Unexpected AI response format' });
                     }
                     //parse response and update record
-                    const resData = JSON.parse(textOutput);
+                    let resData;
+                    try {
+                        resData = JSON.parse(textOutput);
+                    }
+                    catch (e) {
+                        jobs.set(jobId, { status: 'failed', error: 'AI response is not valid JSON. Probably attached invalid image URL' });
+                        return { ok: false, error: 'AI response is not valid JSON. Probably attached invalid image URL' };
+                    }
                     const result = resData;
                     jobs.set(jobId, { status: 'completed', result });
                     return { ok: true };
@@ -174,6 +190,15 @@ export default class BulkAiFlowPlugin extends AdminForthPlugin {
             }
             else {
                 attachmentFiles = yield this.options.attachFiles({ record });
+                try {
+                    for (const fileUrl of attachmentFiles) {
+                        new URL(fileUrl);
+                    }
+                }
+                catch (e) {
+                    jobs.set(jobId, { status: 'failed', error: 'One of the image URLs is not valid' });
+                    return { ok: false, error: 'One of the image URLs is not valid' };
+                }
             }
             const fieldTasks = Object.keys(((_b = this.options) === null || _b === void 0 ? void 0 : _b.generateImages) || {}).map((key) => __awaiter(this, void 0, void 0, function* () {
                 const prompt = this.compileGenerationFieldTemplates(record)[key];
@@ -563,10 +588,10 @@ export default class BulkAiFlowPlugin extends AdminForthPlugin {
                                         }
                                         catch (e) {
                                             // file might be e.g. already deleted, so we catch error
-                                            console.error(`Error setting tag to true for object ${oldRecord[value]}. File will not be auto-cleaned up`, e);
+                                            console.error(`Error setting tag to true for object ${oldRecord[value]}. File will not be auto-cleaned up`);
                                         }
                                     }
-                                    if (fieldsToUpdate[idx][key] && fieldsToUpdate[idx][key] !== null) {
+                                    if (fieldsToUpdate[idx][value] && fieldsToUpdate[idx][value] !== null) {
                                         // remove tag from new file
                                         // in this case we let it crash if it fails: this is a new file which just was uploaded. 
                                         yield columnPlugin.pluginOptions.storageAdapter.markKeyForNotDeletation(fieldsToUpdate[idx][value]);

package/index.ts

@@ -89,6 +89,14 @@ export default class  BulkAiFlowPlugin extends AdminForthPlugin {
       }
       return {};
     } else if (attachmentFiles.length !== 0) {
+      try {
+        for (const fileUrl of attachmentFiles) {
+          new URL(fileUrl);
+        }
+      } catch (e) {
+        jobs.set(jobId, { status: 'failed', error: 'One of the image URLs is not valid' });
+        return { ok: false, error: 'One of the image URLs is not valid' };
+      }
       //create prompt for OpenAI
       const compiledOutputFields = this.compileOutputFieldsTemplates(record);
       const prompt = `Analyze the following image(s) and return a single JSON in format like: {'param1': 'value1', 'param2': 'value2'}. 
@@ -118,7 +126,13 @@ export default class  BulkAiFlowPlugin extends AdminForthPlugin {
         }
 
         //parse response and update record
-        const resData = JSON.parse(textOutput);
+        let resData;
+        try {
+          resData = JSON.parse(textOutput);
+        } catch (e) {
+          jobs.set(jobId, { status: 'failed', error: 'AI response is not valid JSON. Probably attached invalid image URL' });
+          return { ok: false, error: 'AI response is not valid JSON. Probably attached invalid image URL' };
+        }
         const result = resData;
         jobs.set(jobId, { status: 'completed', result });
         return { ok: true };
@@ -180,6 +194,14 @@ export default class  BulkAiFlowPlugin extends AdminForthPlugin {
       attachmentFiles = [];
     } else {
       attachmentFiles = await this.options.attachFiles({ record });
+      try {
+        for (const fileUrl of attachmentFiles) {
+          new URL(fileUrl);
+        }
+      } catch (e) {
+        jobs.set(jobId, { status: 'failed', error: 'One of the image URLs is not valid' });
+        return { ok: false, error: 'One of the image URLs is not valid' };
+      }
     }
     const fieldTasks = Object.keys(this.options?.generateImages || {}).map(async (key) => {
       const prompt = this.compileGenerationFieldTemplates(record)[key];
@@ -591,10 +613,10 @@ export default class  BulkAiFlowPlugin extends AdminForthPlugin {
                       await columnPlugin.pluginOptions.storageAdapter.markKeyForDeletation(oldRecord[value]);
                     } catch (e) {
                       // file might be e.g. already deleted, so we catch error
-                      console.error(`Error setting tag to true for object ${oldRecord[value]}. File will not be auto-cleaned up`, e);
+                      console.error(`Error setting tag to true for object ${oldRecord[value]}. File will not be auto-cleaned up`);
                     }
                   }
-                  if (fieldsToUpdate[idx][key] && fieldsToUpdate[idx][key] !== null) {
+                  if (fieldsToUpdate[idx][value] && fieldsToUpdate[idx][value] !== null) {
                   // remove tag from new file
                   // in this case we let it crash if it fails: this is a new file which just was uploaded. 
                     await  columnPlugin.pluginOptions.storageAdapter.markKeyForNotDeletation(fieldsToUpdate[idx][value]);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@adminforth/bulk-ai-flow",
-  "version": "1.14.0",
+  "version": "1.14.1",
   "publishConfig": {
     "access": "public"
   },