[FIX] Google Sheets - Using the sub-agent to create and populate a sheet returns errors

Author: jcortes

components/google_drive/common/utils.mjs

@@ -104,15 +104,17 @@ function truncatePath(pathArr) {
  */
 function buildFilePaths(files = [], folders = []) {
   const folderIdToFolder = folders.reduce((acc, cur) => {
-    acc[cur.id] = cur;
+    if (cur?.id) {
+      acc[cur.id] = cur;
+    }
     return acc;
   }, {});
   const paths = {};
   // Recursive function that returns an array of file `id`s representing the path to a file if
   // requisite parent folders are available (in `file.parents`) to the requesting user, or an array
   // containing the file ID otherwise
   const pathToFile = (file) => {
-    if (!file) {
+    if (!file?.id) {
       // unretrieved folder or root folder
       return [];
     }
@@ -139,7 +141,9 @@ function buildFilePaths(files = [], folders = []) {
     ];
   };
   files.forEach((file) => {
-    paths[file.id] = pathToFile(file);
+    if (file?.id) {
+      paths[file.id] = pathToFile(file);
+    }
   });
   return paths;
 }
@@ -154,7 +158,9 @@ function buildFilePaths(files = [], folders = []) {
  */
 function buildFileNamePaths(files = [], folders = []) {
   const fileIdToFile = files.concat(folders).reduce((acc, cur) => {
-    acc[cur.id] = cur;
+    if (cur?.id) {
+      acc[cur.id] = cur;
+    }
     return acc;
   }, {});
   const fileIdToPath = buildFilePaths(files, folders);

components/google_drive/google_drive.app.mjs

@@ -127,16 +127,18 @@ export default {
           return [];
         }
         let parentFolders = await Promise.all(
-          file.parents.map((parentId) => this.getFile(parentId, {
+          (file.parents || []).map((parentId) => this.getFile(parentId, {
             fields: "id,name",
           })),
         );
-        return parentFolders.map(({
-          id, name,
-        }) => ({
-          value: id,
-          label: name,
-        }));
+        return parentFolders
+          .filter((folder) => folder?.id)
+          .map(({
+            id, name,
+          }) => ({
+            value: id,
+            label: name,
+          }));
       },
     },
     updateTypes: {
@@ -475,7 +477,7 @@ export default {
     },
     async _listDriveOptions(pageToken, myDrive = true) {
       const {
-        drives,
+        drives = [],
         nextPageToken,
       } = await this.listDrivesInPage(pageToken);
 
@@ -493,10 +495,12 @@ export default {
           ]
           : [];
       for (const d of drives) {
-        options.push({
-          label: d.name,
-          value: d.id,
-        });
+        if (d?.id) {
+          options.push({
+            label: d.name,
+            value: d.id,
+          });
+        }
       }
       return {
         options,
@@ -546,16 +550,18 @@ export default {
      */
     async listFilesOptions(pageToken, extraOpts = {}) {
       const {
-        files,
+        files = [],
         nextPageToken,
       } = await this.listFilesInPage(
         pageToken,
         extraOpts,
       );
-      const options = files.map((file) => ({
-        label: file.name,
-        value: file.id,
-      }));
+      const options = files
+        .filter((file) => file?.id)
+        .map((file) => ({
+          label: file.name,
+          value: file.id,
+        }));
       return {
         options,
         context: {
@@ -611,19 +617,21 @@ export default {
         opts,
       );
       const [
-        { files: folders },
+        { files: folders = [] },
         {
-          files, nextPageToken,
+          files = [], nextPageToken,
         },
       ] = await Promise.all([
         foldersPromise,
         filesPromise,
       ]);
       const filePaths = this.getFilePaths(files, folders);
-      const options = files.map((file) => ({
-        label: filePaths[file.id],
-        value: file.id,
-      }));
+      const options = files
+        .filter((file) => file?.id)
+        .map((file) => ({
+          label: filePaths[file.id],
+          value: file.id,
+        }));
       return {
         options,
         context: {

components/google_drive/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pipedream/google_drive",
-  "version": "1.0.3",
+  "version": "1.0.4",
   "description": "Pipedream Google_drive Components",
   "main": "google_drive.app.mjs",
   "keywords": [