fix lint

Author: Feiyang1

config/.eslintrc.js

@@ -1,3 +1,20 @@
+/**
+ * @license
+ * Copyright 2020 Google LLC
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *   http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
 const path = require('path');
 
 module.exports = {
@@ -128,12 +145,27 @@ module.exports = {
           'Object': "Use {} or 'object' instead.",
           'String': "Use 'string' instead.",
           'Number': "Use 'number' instead.",
-          'Boolean': "Use 'boolean' instead."
+          'Boolean': "Use 'boolean' instead.",
+          'Function': 'Avoid the Function type, as it provides little safety'
+        },
+        'extendDefaults': false
+      }
+    ],
+    '@typescript-eslint/naming-convention': [
+      'error',
+      {
+        'selector': 'class',
+        'format': ['PascalCase']
+      },
+      {
+        'selector': 'interface',
+        'format': ['PascalCase'],
+        'custom': {
+          'regex': '^I[A-Z]',
+          'match': false
         }
       }
     ],
-    '@typescript-eslint/class-name-casing': 'error',
-    '@typescript-eslint/interface-name-prefix': ['error', 'never'],
     '@typescript-eslint/consistent-type-definitions': ['error', 'interface'],
     '@typescript-eslint/explicit-member-accessibility': [
       'error',

packages-exp/functions-exp/src/error.ts

@@ -151,7 +151,7 @@ export function _errorForResponse(
 
       details = errorJSON.details;
       if (details !== undefined) {
-        details = decode(details as {} | null);
+        details = decode(details);
       }
     }
   } catch (e) {

packages-exp/functions-exp/src/serializer.ts

@@ -60,7 +60,7 @@ export function encode(data: unknown): unknown {
     return data.map(x => encode(x));
   }
   if (typeof data === 'function' || typeof data === 'object') {
-    return mapValues(data as object, x => encode(x));
+    return mapValues(data!, x => encode(x));
   }
   // If we got this far, the data is not encodable.
   throw new Error('Data cannot be encoded in JSON: ' + data);
@@ -99,7 +99,7 @@ export function decode(json: unknown): unknown {
     return json.map(x => decode(x));
   }
   if (typeof json === 'function' || typeof json === 'object') {
-    return mapValues(json as object, x => decode(x as {} | null));
+    return mapValues(json!, x => decode(x));
   }
   // Anything else is safe to return.
   return json;

packages-exp/functions-exp/src/service.ts

@@ -74,7 +74,7 @@ export class FunctionsService implements _FirebaseService {
   readonly contextProvider: ContextProvider;
   emulatorOrigin: string | null = null;
   cancelAllRequests: Promise<void>;
-  deleteService!: Function;
+  deleteService!: () => Promise<void>;
 
   /**
    * Creates a new Functions service for the given app.
@@ -90,7 +90,7 @@ export class FunctionsService implements _FirebaseService {
     // Cancels all ongoing requests when resolved.
     this.cancelAllRequests = new Promise(resolve => {
       this.deleteService = () => {
-        return resolve();
+        return Promise.resolve(resolve());
       };
     });
   }
@@ -154,7 +154,7 @@ export function httpsCallable(
  */
 async function postJSON(
   url: string,
-  body: {},
+  body: unknown,
   headers: Headers
 ): Promise<HttpResponse> {
   headers.append('Content-Type', 'application/json');
@@ -176,7 +176,7 @@ async function postJSON(
       json: null
     };
   }
-  let json: {} | null = null;
+  let json: HttpResponseBody | null = null;
   try {
     json = await response.json();
   } catch (e) {
@@ -254,7 +254,7 @@ async function call(
   }
 
   // Decode any special types, such as dates, in the returned data.
-  const decodedData = decode(responseData as {} | null);
+  const decodedData = decode(responseData);
 
   return { data: decodedData };
 }

packages/analytics/src/helpers.ts

@@ -95,7 +95,7 @@ export function getOrCreateDataLayer(dataLayerName: string): DataLayer {
 function wrapGtag(
   gtagCore: Gtag,
   initializedIdPromisesMap: { [gaId: string]: Promise<void> }
-): Function {
+): Gtag {
   return (
     command: 'config' | 'set' | 'event',
     idOrNameOrParams: string | ControlParams,

packages/analytics/testing/get-fake-firebase-services.ts

@@ -42,6 +42,7 @@ export function getFakeApp(measurementId?: string): FirebaseApp {
 
 export function getFakeInstallations(
   fid: string = 'fid-1234',
+  // eslint-disable-next-line @typescript-eslint/ban-types
   onFidResolve?: Function
 ): FirebaseInstallations {
   return {

packages/database/.eslintrc.js

@@ -1,3 +1,20 @@
+/**
+ * @license
+ * Copyright 2020 Google LLC
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *   http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
 module.exports = {
   extends: '../../config/.eslintrc.js',
   parserOptions: {

packages/database/src/core/view/CompleteChildSource.ts

@@ -56,7 +56,7 @@ export interface CompleteChildSource {
  * @constructor
  * @implements CompleteChildSource
  */
-// eslint-disable-next-line @typescript-eslint/class-name-casing
+// eslint-disable-next-line @typescript-eslint/naming-convention
 export class NoCompleteChildSource_ implements CompleteChildSource {
   /**
    * @inheritDoc

packages/database/src/realtime/BrowserPollConnection.ts

@@ -400,7 +400,7 @@ export class BrowserPollConnection implements Transport {
   }
 }
 
-// eslint-disable-next-line @typescript-eslint/interface-name-prefix
+// eslint-disable-next-line @typescript-eslint/naming-convention
 export interface IFrameElement extends HTMLIFrameElement {
   doc: Document;
 }

packages/firestore/src/protos/firestore_proto_api.d.ts

@@ -19,7 +19,7 @@
 /*
    eslint-disable
    camelcase, @typescript-eslint/no-explicit-any,
-   @typescript-eslint/interface-name-prefix,  @typescript-eslint/class-name-casing
+   @typescript-eslint/naming-convention
 */
 export declare type ApiClientHookFactory = any;
 export declare type PromiseRequestService = any;

packages/functions/src/api/error.ts

@@ -158,7 +158,7 @@ export function _errorForResponse(
 
       details = errorJSON.details;
       if (details !== undefined) {
-        details = serializer.decode(details as {} | null);
+        details = serializer.decode(details);
       }
     }
   } catch (e) {

packages/functions/src/api/service.ts

@@ -85,7 +85,7 @@ export class Service implements FirebaseFunctions, FirebaseService {
   private readonly serializer = new Serializer();
   private emulatorOrigin: string | null = null;
   private cancelAllRequests: Promise<void>;
-  private deleteService!: Function;
+  private deleteService!: () => void;
 
   /**
    * Creates a new Functions service for the given app and (optional) region.
@@ -113,7 +113,7 @@ export class Service implements FirebaseFunctions, FirebaseService {
 
   INTERNAL = {
     delete: (): Promise<void> => {
-      return this.deleteService();
+      return Promise.resolve(this.deleteService());
     }
   };
 
@@ -183,7 +183,7 @@ export class Service implements FirebaseFunctions, FirebaseService {
         json: null
       };
     }
-    let json: {} | null = null;
+    let json: HttpResponseBody | null = null;
     try {
       json = await response.json();
     } catch (e) {
@@ -269,7 +269,7 @@ export class Service implements FirebaseFunctions, FirebaseService {
     }
 
     // Decode any special types, such as dates, in the returned data.
-    const decodedData = this.serializer.decode(responseData as {} | null);
+    const decodedData = this.serializer.decode(responseData);
 
     return { data: decodedData };
   }

packages/functions/src/serializer.ts

@@ -58,7 +58,7 @@ export class Serializer {
       return data.map(x => this.encode(x));
     }
     if (typeof data === 'function' || typeof data === 'object') {
-      return mapValues(data as object, x => this.encode(x));
+      return mapValues(data!, x => this.encode(x));
     }
     // If we got this far, the data is not encodable.
     throw new Error('Data cannot be encoded in JSON: ' + data);
@@ -93,7 +93,7 @@ export class Serializer {
       return json.map(x => this.decode(x));
     }
     if (typeof json === 'function' || typeof json === 'object') {
-      return mapValues(json as object, x => this.decode(x as {} | null));
+      return mapValues(json!, x => this.decode(x));
     }
     // Anything else is safe to return.
     return json;

packages/messaging/src/controllers/sw-controller.test.ts

@@ -79,6 +79,7 @@ const DATA_MESSAGE: MessagePayloadInternal = {
 
 describe('SwController', () => {
   let addEventListenerStub: Stub<typeof self.addEventListener>;
+  // eslint-disable-next-line @typescript-eslint/ban-types
   let eventListenerMap: Map<string, Function>;
   let swController: SwController;
   let firebaseDependencies: FirebaseInternalDependencies;

packages/messaging/src/controllers/sw-controller.ts

@@ -96,7 +96,7 @@ export class SwController implements FirebaseMessaging, FirebaseService {
   }
 
   onBackgroundMessage(
-    nextOrObserver: NextFn<object> | Observer<object>
+    nextOrObserver: NextFn<MessagePayload> | Observer<MessagePayload>
   ): Unsubscribe {
     this.isOnBackgroundMessageUsed = true;
     this.bgMessageHandler = nextOrObserver;

packages/performance/src/services/api_service.ts

@@ -21,6 +21,7 @@ import { consoleLogger } from '../utils/console_logger';
 declare global {
   interface Window {
     PerformanceObserver: typeof PerformanceObserver;
+    // eslint-disable-next-line @typescript-eslint/ban-types
     perfMetrics?: { onFirstInputDelay: Function };
   }
 }
@@ -45,6 +46,7 @@ export class Api {
   /** PreformanceObserver constructor function. */
   private readonly PerformanceObserver: typeof PerformanceObserver;
   private readonly windowLocation: Location;
+  // eslint-disable-next-line @typescript-eslint/ban-types
   readonly onFirstInputDelay?: Function;
   readonly localStorage?: Storage;
   readonly document: Document;

packages/storage/src/implementation/async.ts

@@ -20,6 +20,7 @@
  * microtask, i.e. as soon as possible after the current script returns back
  * into browser code.
  */
+// eslint-disable-next-line @typescript-eslint/ban-types
 export function async(f: Function): Function {
   return (...argsToForward: unknown[]) => {
     // eslint-disable-next-line @typescript-eslint/no-floating-promises

packages/storage/src/implementation/backoff.ts

@@ -31,7 +31,8 @@ export { id };
  */
 export function start(
   f: (p1: (success: boolean) => void, canceled: boolean) => void,
-  callback: Function,
+  // eslint-disable-next-line @typescript-eslint/no-explicit-any
+  callback: (...args: any[]) => unknown,
   timeout: number
 ): id {
   // TODO(andysoto): make this code cleaner (probably refactor into an actual

packages/storage/src/implementation/request.ts

@@ -64,8 +64,9 @@ class NetworkRequest<T> implements Request<T> {
   private additionalRetryCodes_: number[];
   private pendingXhr_: XhrIo | null = null;
   private backoffId_: backoff.id | null = null;
-  private resolve_: Function | null = null;
-  private reject_: Function | null = null;
+  private resolve_!: (value?: T | PromiseLike<T> | undefined) => void;
+  // eslint-disable-next-line @typescript-eslint/no-explicit-any
+  private reject_!: (reason?: any) => void;
   private canceled_: boolean = false;
   private appDelete_: boolean = false;
   private callback_: (p1: XhrIo, p2: string) => T;
@@ -170,8 +171,8 @@ class NetworkRequest<T> implements Request<T> {
       requestWentThrough: boolean,
       status: RequestEndStatus
     ): void {
-      const resolve = self.resolve_ as Function;
-      const reject = self.reject_ as Function;
+      const resolve = self.resolve_;
+      const reject = self.reject_;
       const xhr = status.xhr as XhrIo;
       if (status.wasSuccessCode) {
         try {

packages/storage/src/implementation/type.ts

@@ -26,6 +26,7 @@ export function isJustDef<T>(p: T | null | undefined): p is T | null {
   return p !== void 0;
 }
 
+// eslint-disable-next-line @typescript-eslint/ban-types
 export function isFunction(p: unknown): p is Function {
   return typeof p === 'function';
 }

packages/storage/test/unit/task.test.ts

@@ -328,6 +328,7 @@ describe('Firebase Storage > Upload Task', () => {
       blob
     );
 
+    // eslint-disable-next-line @typescript-eslint/ban-types
     let resolve: Function, reject: Function;
     const promise = new Promise<void>((innerResolve, innerReject) => {
       resolve = innerResolve;

packages/storage/test/unit/testshared.ts

@@ -52,9 +52,9 @@ export function makeFakeApp(bucketArg?: string): FirebaseApp {
   return app as FirebaseApp;
 }
 
-export function makeFakeAuthProvider(
-  token: {} | null
-): Provider<FirebaseAuthInternalName> {
+export function makeFakeAuthProvider(token: {
+  accessToken: string;
+}): Provider<FirebaseAuthInternalName> {
   const provider = new Provider(
     'auth-internal',
     new ComponentContainer('storage-container')
@@ -113,6 +113,7 @@ export function fakeXhrIo(headers: Headers, status: number = 200): XhrIo {
 /**
  * Binds ignoring types. Used to test calls involving improper arguments.
  */
+// eslint-disable-next-line @typescript-eslint/ban-types
 export function bind(f: Function, ctx: any, ...args: any[]): () => void {
   return () => {
     f.apply(ctx, args);

packages/util/index.ts

@@ -28,5 +28,4 @@ export * from './src/obj';
 export * from './src/query';
 export * from './src/sha1';
 export * from './src/subscribe';
-export * from './src/validation';
 export * from './src/utf8';

packages/util/src/subscribe.ts

@@ -126,7 +126,7 @@ class ObserverProxy<T> implements Observer<T> {
    *   call to subscribe().
    */
   subscribe(
-    nextOrObserver?: PartialObserver<T> | Function,
+    nextOrObserver?: NextFn<T> | PartialObserver<T>,
     error?: ErrorFn,
     complete?: CompleteFn
   ): Unsubscribe {
@@ -261,6 +261,7 @@ class ObserverProxy<T> implements Observer<T> {
 }
 
 /** Turn synchronous function into one called asynchronously. */
+// eslint-disable-next-line @typescript-eslint/ban-types
 export function async(fn: Function, onError?: ErrorFn): Function {
   return (...args: unknown[]) => {
     Promise.resolve(true)