@google/earthengine 1.6.0 → 1.6.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@google/earthengine",
-  "version": "1.6.0",
+  "version": "1.6.2",
   "description": "JavaScript client for Google Earth Engine API.",
   "author": "Google LLC",
   "license": "Apache-2.0",

package/src/apiclient.js

@@ -24,7 +24,7 @@ const {trustedResourceUrl} = goog.require('safevalues.index');
 /** @namespace */
 const apiclient = {};
 
-const API_CLIENT_VERSION = '1.6.0';
+const API_CLIENT_VERSION = '1.6.2';
 
 exports.VERSION = apiVersion.VERSION;
 exports.API_CLIENT_VERSION = API_CLIENT_VERSION;

package/src/eeapiclient/api_request_hook.ts

@@ -1,25 +1,54 @@
 import {GeneratedRequestParams} from './generated_types';
 
+/**
+ * Interface for a hook that can be used to intercept and modify API requests
+ * and responses.
+ */
 export interface ApiClientRequestHook {
-  /** Called once before each API send. */
+  /**
+   * Called once before each API send.
+   */
   onBeforeSend(): void;
-  /** Called after each successful API response. */
+  /**
+   * Called after each successful API response.
+   */
   onSuccess<T>(response: T): void;
-  /**  Called after each unsuccessful API response. */
+  /**
+   * Called after each unsuccessful API response.
+   */
   onError(error: {}): void;
+  /**
+   * Called after each API request is finalized.
+   * In some cases, the caller may cancel the API call which will not trigger
+   * any of the success or error callbacks. To handle such cases, the onFinalize
+   * callback can be used.
+   * OnFinalize will have equal number of calls as onBeforeSend.
+   */
+  onFinalize?: () => void;
 }
 
+/**
+ * Interface for a factory that can be used to create {@link
+ * ApiClientRequestHook} instances.
+ */
 export abstract class ApiClientHookFactory {
   abstract getRequestHook(
     requestParams: GeneratedRequestParams,
   ): ApiClientRequestHook;
 }
 
+/**
+ * Interface for a constructor that can be used to create {@link
+ * ApiClientHookFactory} instances.
+ */
 export interface ApiClientHookFactoryCtor {
   // tslint:disable-next-line:no-any ctor args are of any type
   new (...args: any[]): ApiClientHookFactory;
 }
 
+/**
+ * Default implementation of {@link ApiClientHookFactory} that does nothing.
+ */
 export class DefaultApiClientHookFactory implements ApiClientHookFactory {
   getRequestHook(requestParams: GeneratedRequestParams): ApiClientRequestHook {
     return {

package/src/eeapiclient/domain_object.ts

@@ -426,9 +426,7 @@ function deepCopyValue<T extends {}>(
   } else if (value instanceof NullClass) {
     deserialized = null as unknown as {};
   } else if (typeof value === 'object') {
-    // TODO(user): Assert as a type, declared interface, or `unknown`.
-    // tslint:disable-next-line:ban-types no-unnecessary-type-assertion
-    deserialized = JSON.parse(JSON.stringify(value)) as AnyDuringMigration;
+    deserialized = JSON.parse(JSON.stringify(value)) as object;
   } else {
     deserialized = value;
   }

package/src/eeapiclient/promise_api_client.ts

@@ -33,16 +33,20 @@ export class PromiseApiClient extends ApiClient {
     }
 
     hook.onBeforeSend();
-    return promise.then(
-      (response) => {
-        hook.onSuccess(response);
-        return response;
-      },
-      (error) => {
-        hook.onError(error);
-        throw error;
-      },
-    );
+    return promise
+      .then(
+        (response) => {
+          hook.onSuccess(response);
+          return response;
+        },
+        (error) => {
+          hook.onError(error);
+          throw error;
+        },
+      )
+      .finally(() => {
+        hook.onFinalize?.();
+      });
   }
 
   /** Converts a gapi.client.Request to an Promise<T>. */