style: clean eslintrc config (#2)

Author: originalix

.eslintrc

@@ -0,0 +1,25 @@
+{
+  "extends": [
+    "wesbos/typescript"
+  ],
+  "rules": {
+    "no-console": "off",
+    "prettier/prettier": [
+      "error",
+      {
+        "trailingComma": "es5",
+        "singleQuote": true,
+        "printWidth": 100,
+        "tabWidth": 2
+      }
+    ],
+    "no-plusplus": "off",
+    "no-shadow": "off",
+    "@typescript-eslint/no-shadow": "off",
+    "@typescript-eslint/no-unsafe-return": "off",
+    "@typescript-eslint/no-unsafe-assignment": "off",
+    "@typescript-eslint/no-unsafe-member-access": "off",
+    "@typescript-eslint/no-unsafe-call": "off",
+    "@typescript-eslint/no-unsafe-argument": "off"
+  }
+}

.eslintrc.js

@@ -11,27 +11,34 @@
     "packages/*"
   ],
   "scripts": {
-    "bootstrap": "lerna bootstrap"
+    "bootstrap": "lerna bootstrap",
+    "build": "lerna run build"
   },
   "devDependencies": {
+    "@babel/core": "^7.16.0",
+    "@babel/eslint-parser": "^7.16.3",
+    "@babel/preset-react": "^7.16.0",
     "@rollup/plugin-babel": "^5.3.1",
     "@rollup/plugin-typescript": "^8.3.2",
     "@types/jest": "^27.5.1",
-    "@types/node": "^17.0.32",
-    "@typescript-eslint/eslint-plugin": "^5.23.0",
-    "@typescript-eslint/parser": "^5.23.0",
-    "eslint": "^8.15.0",
-    "eslint-config-prettier": "^8.5.0",
-    "eslint-config-standard": "^17.0.0",
-    "eslint-plugin-import": "^2.26.0",
-    "eslint-plugin-n": "^15.2.0",
+    "@types/node": "^16.11.12",
+    "@typescript-eslint/eslint-plugin": "^5.6.0",
+    "@typescript-eslint/parser": "^5.6.0",
+    "eslint": "^8.4.1",
+    "eslint-config-airbnb": "^19.0.2",
+    "eslint-config-airbnb-typescript": "^16.1.0",
+    "eslint-config-prettier": "^8.3.0",
+    "eslint-config-wesbos": "3.0.2",
+    "eslint-plugin-html": "^6.2.0",
+    "eslint-plugin-import": "^2.25.3",
+    "eslint-plugin-jsx-a11y": "^6.5.1",
     "eslint-plugin-prettier": "^4.0.0",
-    "eslint-plugin-promise": "^6.0.0",
+    "eslint-plugin-react": "^7.27.1",
+    "eslint-plugin-react-hooks": "^4.3.0",
     "lerna": "^4.0.0",
-    "prettier": "^2.6.2",
-    "prettier-eslint": "^14.0.2",
+    "prettier": "^2.5.1",
     "rollup": "^2.73.0",
     "rollup-plugin-dts": "^4.2.1",
-    "typescript": "^4.6.4"
+    "typescript": "^4.5.2"
   }
 }

package.json

@@ -7,16 +7,16 @@ export type HttpRequestOptions = {
 
 // slight hack to make Flow happy, but to allow Node to set its own fetch
 // Request, RequestOptions and Response are built-in types of Flow for fetch API
-let _fetch: (input: string | Request, init?: any) => Promise<Response> =
+let innerFetch: (input: string | Request, init?: any) => Promise<Response> =
   typeof window === 'undefined'
     ? () => Promise.reject(new Error('Not Browser environment'))
     : window.fetch;
 
-let _isNode = false;
+let isNode = false;
 
-export function setFetch(fetch: any, isNode?: boolean) {
-  _fetch = fetch;
-  _isNode = !!isNode;
+export function setFetch(fetch: any, node?: boolean) {
+  innerFetch = fetch;
+  isNode = !!node;
 }
 
 function contentType(body: any) {
@@ -61,14 +61,14 @@ export async function request(options: HttpRequestOptions) {
   }
 
   // Node applications must spoof origin for bridge CORS
-  if (_isNode) {
+  if (isNode) {
     fetchOptions.headers = {
       ...fetchOptions.headers,
       Origin: 'https://node.onekey.so',
     };
   }
 
-  const res = await _fetch(options.url, fetchOptions);
+  const res = await innerFetch(options.url, fetchOptions);
   const resText = await res.text();
   if (res.ok) {
     return parseResult(resText);

packages/hd-transport-http/src/http.ts

@@ -1,7 +1,7 @@
 import transport from '@onekeyfe/hd-transport';
+import type { AcquireInput, OneKeyDeviceInfoWithSession } from '@onekeyfe/hd-transport';
 import { request as http } from './http';
 import { DEFAULT_URL, DEFAULT_VERSION_URL } from './constants';
-import type { AcquireInput, OneKeyDeviceInfoWithSession } from '@onekeyfe/hd-transport';
 
 const { check, buildOne, receiveOne, parseConfigure } = transport;
 
@@ -12,13 +12,21 @@ type IncompleteRequestOptions = {
 
 export default class HttpTransport {
   _messages: ReturnType<typeof transport.parseConfigure> | undefined;
+
   bridgeVersion?: string;
+
   configured = false;
+
   debug = false;
+
   isOutdated?: boolean;
+
   newestVersionUrl: string;
+
   stopped = false;
+
   url: string;
+
   version = '';
 
   constructor(url?: string, newestVersionUrl?: string) {
@@ -58,7 +66,7 @@ export default class HttpTransport {
             await http({
               url: `${this.newestVersionUrl}?${Date.now()}`,
               method: 'GET',
-            }),
+            })
           );
     // TODO: 这里需要进行版本比较，可以使用 semver 比较，不引入其他工具类
     // this.isOutdated = versionUtils.isNewer(newVersion, this.version);
@@ -154,6 +162,7 @@ export default class HttpTransport {
     return check.call(jsonData);
   }
 
+  // eslint-disable-next-line class-methods-use-this
   requestDevice() {
     // eslint-disable-next-line prefer-promise-reject-errors
     return Promise.reject();

packages/hd-transport-http/src/index.ts

@@ -0,0 +1,21 @@
+{
+  "rules": {
+      "no-console": "error",
+      "@typescript-eslint/no-non-null-assertion": "off",
+      "@typescript-eslint/no-explicit-any": "off",
+      "@typescript-eslint/ban-ts-ignore": "off",
+      // We need empty functions for mocking modules for react-native
+      "@typescript-eslint/no-empty-function": "off",
+      "no-useless-constructor": "off",
+      "@typescript-eslint/no-useless-constructor": "error",
+      // valid case of class method overloads in typescript
+      "no-dupe-class-members": "off",
+      "@typescript-eslint/ban-ts-comment": "off",
+      //  Missing return type on function
+      "@typescript-eslint/explicit-module-boundary-types": "off",
+      // note you must disable the base rule as it can report incorrect errors
+      "no-use-before-define": "off",
+      "@typescript-eslint/no-use-before-define": ["error"],
+      "require-await": ["error"]
+  }
+}

packages/hd-transport/.eslintrc

@@ -1,7 +1,7 @@
-import { buildOne, receiveOne, parseConfigure } from './serialization';
-import * as check from './utils/highlevel-checks';
 import * as protobuf from 'protobufjs/light';
 import * as Long from 'long';
+import { buildOne, receiveOne, parseConfigure } from './serialization';
+import * as check from './utils/highlevel-checks';
 
 protobuf.util.Long = Long;
 protobuf.configure();

packages/hd-transport/src/index.ts

@@ -12,6 +12,7 @@ function encode(data: ByteBuffer, options: Options<true>): Buffer[];
 function encode(data: ByteBuffer, options: Options<false>): Buffer;
 function encode(data: any, options: any): any {
   const { addTrezorHeaders, chunked, messageType } = options;
+  // eslint-disable-next-line @typescript-eslint/restrict-plus-operands
   const fullSize = (addTrezorHeaders ? HEADER_SIZE : HEADER_SIZE - 2) + data.limit;
 
   const encodedByteBuffer = new ByteBuffer(fullSize);

packages/hd-transport/src/serialization/protocol/encode.ts

@@ -25,7 +25,7 @@ export type Transport = {
     session: string,
     name: string,
     data: Record<string, any>,
-    debugLink: boolean,
+    debugLink: boolean
   ): Promise<MessageFromOneKey>;
   post(session: string, name: string, data: Record<string, any>, debugLink: boolean): Promise<void>;
   read(session: string, debugLink: boolean): Promise<MessageFromOneKey>;