@adobe/helix-google-support 1.2.1 → 1.3.1

package/CHANGELOG.md

@@ -1,3 +1,24 @@
+## [1.3.1](https://github.com/adobe/helix-google-support/compare/v1.3.0...v1.3.1) (2022-04-28)
+
+
+### Bug Fixes
+
+* more fixes ([#14](https://github.com/adobe/helix-google-support/issues/14)) ([f52b01d](https://github.com/adobe/helix-google-support/commit/f52b01d575fb17602ea57b35f0e4c93dbbd41dd1))
+
+# [1.3.0](https://github.com/adobe/helix-google-support/compare/v1.2.2...v1.3.0) (2022-04-28)
+
+
+### Features
+
+* handle getItemForPath() with no path ([#12](https://github.com/adobe/helix-google-support/issues/12)) ([0e1b60d](https://github.com/adobe/helix-google-support/commit/0e1b60d6612b3628193b9188eadc230de2894aff))
+
+## [1.2.2](https://github.com/adobe/helix-google-support/compare/v1.2.1...v1.2.2) (2022-04-28)
+
+
+### Bug Fixes
+
+* include mimeType ([#11](https://github.com/adobe/helix-google-support/issues/11)) ([67ab999](https://github.com/adobe/helix-google-support/commit/67ab999a7fee111978a15c4393761e4b614ee2fc))
+
 ## [1.2.1](https://github.com/adobe/helix-google-support/compare/v1.2.0...v1.2.1) (2022-04-28)
 
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@adobe/helix-google-support",
-  "version": "1.2.1",
+  "version": "1.3.1",
   "description": "Helix Google Support",
   "main": "src/index.js",
   "type": "module",
@@ -47,7 +47,7 @@
     "eslint-plugin-import": "2.26.0",
     "husky": "7.0.4",
     "junit-report-builder": "3.0.0",
-    "lint-staged": "12.4.0",
+    "lint-staged": "12.4.1",
     "mocha": "9.2.2",
     "mocha-multi-reporters": "1.5.1",
     "nock": "13.2.4",

package/src/GoogleClient.js

@@ -25,19 +25,25 @@ let lru = new LRU({ max: 1000, ttl: 60000 });
  * @property {string} name
  * @property {string} url
  * @property {string} path
+ * @property {string} mimeType
+ * @property {number} lastModified
  */
 
 /**
- * Adds the last modified property if defined in item
+ * Adds the last modified property and mimeType if defined in item
  * @param {DriveItemInfo} itemInfo
  * @param item
  * @returns {DriveItemInfo}
  */
-function addLastModified(itemInfo, item) {
+function addFields(itemInfo, item) {
   if (item.modifiedTime) {
     // eslint-disable-next-line no-param-reassign
     itemInfo.lastModified = Date.parse(item.modifiedTime);
   }
+  if (item.mimeType) {
+    // eslint-disable-next-line no-param-reassign
+    itemInfo.mimeType = item.mimeType;
+  }
   return itemInfo;
 }
 
@@ -232,7 +238,7 @@ export class GoogleClient {
     if (!children) {
       return null;
     }
-    const pathItem = addLastModified({
+    const pathItem = addFields({
       name,
       path: itemPath,
       id: item.id,
@@ -292,6 +298,7 @@ export class GoogleClient {
       name: '',
       path: '/',
       id: parentId,
+      mimeType: 'application/vnd.google-apps.folder',
     }];
   }
 
@@ -310,6 +317,7 @@ export class GoogleClient {
       fields: [
         'name',
         'parents',
+        'mimeType',
         'modifiedTime',
       ].join(','),
     }));
@@ -317,7 +325,7 @@ export class GoogleClient {
     const root = roots[fileId];
     if (root) {
       // stop at mount root
-      return [addLastModified({
+      return [addFields({
         id: fileId,
         name: '',
         path: root,
@@ -327,7 +335,7 @@ export class GoogleClient {
     const parentId = data.parents ? data.parents[0] : '';
     if (!parentId) {
       // outside mountpoint
-      return [addLastModified({
+      return [addFields({
         id: fileId,
         name: data.name,
         path: `/root:/${data.name}`,
@@ -336,7 +344,7 @@ export class GoogleClient {
 
     const ancestors = await this.getItems(data.parents[0], roots);
     const parentPath = ancestors[0].path.replace(/\/+$/, '');
-    ancestors.unshift(addLastModified({
+    ancestors.unshift(addFields({
       id: fileId,
       name: data.name,
       path: `${parentPath}/${data.name}`,
@@ -367,11 +375,46 @@ export class GoogleClient {
    * Returns the (cached) item for the given path or {@code null} if the item cannot be found.
    * The item will contains a `invalidate()` method which can be used to remove it from the cache.
    *
-   * @param {string} parentId
-   * @param {string} path
+   * @param {string} parentId the parent id or the id of the item itself, if the path is missing
+   * @param {string} [path]
    * @returns {Promise<DriveItemInfo>}
    */
   async getItemFromPath(parentId, path) {
+    if (!path) {
+      try {
+        let item = lru.get(parentId);
+        if (!item) {
+          const { data } = (await this.drive.files.get({
+            fileId: parentId,
+            fields: [
+              'name',
+              'parents',
+              'mimeType',
+              'modifiedTime',
+            ].join(','),
+          }));
+          item = addFields({
+            id: parentId,
+            path: `/${data.name}`,
+          }, data);
+          // todo: extend caching to all items ?
+          Object.defineProperty(item, 'invalidate', {
+            enumerable: false,
+            value() {
+              lru.delete(parentId);
+            },
+          });
+          lru.set(parentId, item);
+        }
+        return item;
+      } catch (e) {
+        if (e.response && e.response.status === 404) {
+          return null;
+        }
+        throw e;
+      }
+    }
+
     const segs = createPathSegments(path);
     const items = await this.getDriveItemsFromSegments(parentId, segs, '');
     const item = items?.[0];