Get Resources improvements & error handling

Author: GTFalcao

components/cloudinary/actions/get-account-usage-details/get-account-usage-details.mjs

@@ -11,7 +11,7 @@ export default {
     dateInfo: {
       type: "alert",
       alertType: "info",
-      content: "If `Date` is not specified, it defaults to the current date."
+      content: "If `Date` is not specified, it defaults to the current date.",
     },
     date: {
       type: "string",
@@ -26,16 +26,16 @@ export default {
     };
 
     try {
-    const response = await this.cloudinary.getUsage(options);
+      const response = await this.cloudinary.getUsage(options);
 
-    if (response) {
-      $.export("$summary", "Successfully retrieved usage details");
-    }
+      if (response) {
+        $.export("$summary", "Successfully retrieved usage details");
+      }
 
-    return response;
-  }
-     catch (err) {
+      return response;
+    }
+    catch (err) {
       throw new Error(`Cloudinary error response: ${err.error?.message ?? JSON.stringify(err)}`);
-     }
+    }
   },
 };

components/cloudinary/actions/get-resources/get-resources.mjs

@@ -4,7 +4,7 @@ export default {
   key: "cloudinary-get-resources",
   name: "Get Resources",
   description: "Lists resources (assets) uploaded to your product environment. [See the documentation](https://cloudinary.com/documentation/admin_api#get_resources)",
-  version: "0.0.1",
+  version: "0.1.{{ts}}",
   type: "action",
   props: {
     cloudinary,
@@ -22,28 +22,35 @@ export default {
     },
     prefix: {
       type: "string",
-      label: "Prefix",
-      description: "Find all assets with a public ID that starts with the specified prefix",
+      label: "Filter by Prefix",
+      description: "Find all assets with a public ID that starts with the specified prefix.",
       optional: true,
     },
     tags: {
       type: "boolean",
-      label: "Tags",
-      description: "Whether to include the list of tag names assigned to each asset",
+      label: "Include Tags",
+      description: "Whether to include the list of tag names assigned to each asset.",
       default: false,
       optional: true,
     },
     context: {
       type: "boolean",
-      label: "Context",
-      description: "Whether to include key-value pairs of contextual metadata associated with each asset",
+      label: "Include Context",
+      description: "Whether to include key-value pairs of contextual metadata associated with each asset.",
+      default: false,
+      optional: true,
+    },
+    metadata: {
+      type: "boolean",
+      label: "Include Metadata",
+      description: "Whether to include the structured metadata fields and values assigned to each asset.",
       default: false,
       optional: true,
     },
     moderation: {
       type: "boolean",
-      label: "Moderation",
-      description: "Whether to include the image moderation status of each asset",
+      label: "Include Moderation",
+      description: "Whether to include the image moderation status of each asset.",
       default: false,
       optional: true,
     },
@@ -66,22 +73,27 @@ export default {
     };
 
     const resources = [];
-    let next;
-    do {
-      const response = await this.cloudinary.getResources(options);
-      resources.push(...response.resources);
-      next = response.next_cursor;
-      options.next_cursor = next;
-    } while (next && resources.length < this.maxResults);
+    try {
+      let next;
+      do {
+        const response = await this.cloudinary.getResources(options);
+        resources.push(...response.resources);
+        next = response.next_cursor;
+        options.next_cursor = next;
+      } while (next && resources.length < this.maxResults);
 
-    if (resources.length > this.maxResults) {
-      resources.length = this.maxResults;
-    }
+      if (resources.length > this.maxResults) {
+        resources.length = this.maxResults;
+      }
 
-    $.export("$summary", `Found ${resources.length} resource${resources.length === 1
-      ? ""
-      : "s"}.`);
+      $.export("$summary", `Retrieved ${resources.length} resource${resources.length === 1
+        ? ""
+        : "s"}`);
 
-    return resources;
+      return resources;
+    }
+    catch (err) {
+      throw new Error(`Cloudinary error response: ${err.error?.message ?? JSON.stringify(err)}`);
+    }
   },
 };

components/cloudinary/cloudinary.app.mjs

@@ -8,7 +8,7 @@ export default {
     resourceType: {
       type: "string",
       label: "Resource Type",
-      description: "The type of asset. Defaults to `image` if left blank",
+      description: "The type of asset. Defaults to `image` if not specified. `Note:` use video for all video and audio assets, such as `.mp3`. ",
       options: constants.RESOURCE_TYPE_OPTIONS,
       default: "image",
       optional: true,
@@ -32,7 +32,7 @@ export default {
     deliveryType: {
       type: "string",
       label: "Type",
-      description: "The delivery type. Defaults to `upload` if left blank",
+      description: "The delivery type. Defaults to `upload` if not specified.",
       options: constants.DELIVERY_TYPE_OPTIONS,
       default: "upload",
       optional: true,