Add linting to CI

.github/workflows/ci.yml

@@ -27,3 +27,5 @@ jobs:
         npm run build
     - name: Run tests
       run: ETHERSJS_VERSION=${{ matrix.ethers }} npm run test
+    - name: Run lints
+      run: npm run lint

package.json

@@ -8,6 +8,7 @@
   "scripts": {
     "build": "lerna run build",
     "test": "lerna run --stream test",
+    "lint": "lerna run lint",
     "docs": "typedoc --plugin nlfurniss-typedoc-plugin-sourcefile-url --plugin typedoc-plugin-extras --sourcefile-url-prefix 'https://github.com/spruceid/siwe' --favicon ./favicon.svg --out docs packages/siwe/lib/siwe.ts"
   },
   "author": "Spruce Systems Inc.",

packages/siwe-parser/.eslintignore

@@ -0,0 +1,3 @@
+dist
+node_modules
+jest.config.js

packages/siwe-parser/.eslintrc

@@ -0,0 +1,11 @@
+{
+  "root": true,
+  "parser": "@typescript-eslint/parser",
+  "plugins": ["@typescript-eslint"],
+  "extends": [
+    "eslint:recommended",
+    "plugin:@typescript-eslint/eslint-recommended",
+    "plugin:@typescript-eslint/recommended",
+    "prettier"
+  ]
+}

packages/siwe-parser/lib/parsers.test.ts

@@ -1,7 +1,8 @@
 import { ParsedMessage } from "./abnf";
+import * as fs from "fs";
 
-const parsingPositive: Object = require("../../../test/parsing_positive.json");
-const parsingNegative: Object = require("../../../test/parsing_negative.json");
+const parsingPositive: object = JSON.parse(fs.readFileSync('../../test/parsing_positive.json', 'utf8'));
+const parsingNegative: object = JSON.parse(fs.readFileSync('../../test/parsing_negative.json', 'utf8'));
 
 //
 describe("Successfully parses with ABNF Client", () => {

packages/siwe-parser/lib/utils.ts

@@ -6,15 +6,15 @@ import { bytesToHex } from '@noble/hashes/utils';
  * @returns Either the return is or not in the EIP-55 format.
  */
 export const isEIP55Address = (address: string) => {
-    if(address.length != 42) {
+    if (address.length != 42) {
         return false;
     }
 
     const lowerAddress = `${address}`.toLowerCase().replace('0x', '');
-    var hash = bytesToHex(keccak_256(lowerAddress));
-    var ret = '0x';
+    const hash = bytesToHex(keccak_256(lowerAddress));
+    let ret = '0x';
 
-    for (var i = 0; i < lowerAddress.length; i++) {
+    for (let i = 0; i < lowerAddress.length; i++) {
         if (parseInt(hash[i], 16) >= 8) {
             ret += lowerAddress[i].toUpperCase();
         } else {
@@ -26,7 +26,7 @@ export const isEIP55Address = (address: string) => {
 
 export const parseIntegerNumber = (number: string): number => {
     const parsed = parseInt(number);
-    if(parsed === NaN) throw new Error("Invalid number.");
-    if(parsed === Infinity) throw new Error("Invalid number.");
+    if (isNaN(parsed)) throw new Error("Invalid number.");
+    if (parsed === Infinity) throw new Error("Invalid number.");
     return parsed;
-  }
+}

packages/siwe-parser/package.json

@@ -9,7 +9,9 @@
   },
   "scripts": {
     "test": "jest",
-    "build": "tsc"
+    "build": "tsc",
+    "lint": "eslint --ext .js,.ts .",
+    "format": "prettier --ignore-path .gitignore --write \"**/*.+(js|ts|json)\""
   },
   "author": "Spruce Systems Inc.",
   "keywords": [
@@ -27,6 +29,13 @@
     "valid-url": "^1.0.9",
     "@noble/hashes": "^1.1.2"
   },
+"devDependencies": {
+  "@typescript-eslint/eslint-plugin": "^5.23.0",
+  "@typescript-eslint/parser": "^5.23.0",
+  "eslint": "^8.15.0",
+  "eslint-config-prettier": "^8.5.0",
+  "prettier": "^2.6.2"
+},
   "repository": {
     "type": "git",
     "url": "git+https://github.com/spruceid/siwe.git"

packages/siwe/.eslintignore

@@ -1,2 +1,4 @@
 dist
-node_modules
+node_modules
+jest.config.js
+jest.setup.js

packages/siwe/lib/client.test.ts

@@ -1,10 +1,11 @@
-/* eslint @typescript-eslint/no-explicit-any: 0 */
-const parsingPositive = require('../../../test/parsing_positive.json');
-const parsingNegative = require('../../../test/parsing_negative.json');
-const parsingNegativeObjects = require('../../../test/parsing_negative_objects.json');
-const verificationPositive = require('../../../test/verification_positive.json');
-const verificationNegative = require('../../../test/verification_negative.json');
-const EIP1271 = require('../../../test/eip1271.json');
+import * as fs from "fs";
+
+const parsingPositive: object = JSON.parse(fs.readFileSync('../../test/parsing_positive.json', 'utf8'));
+const parsingNegative: object = JSON.parse(fs.readFileSync('../../test/parsing_negative.json', 'utf8'));
+const parsingNegativeObjects: object = JSON.parse(fs.readFileSync('../../test/parsing_negative_objects.json', 'utf8'));
+const verificationPositive: object = JSON.parse(fs.readFileSync('../../test/verification_positive.json', 'utf8'));
+const verificationNegative: object = JSON.parse(fs.readFileSync('../../test/verification_negative.json', 'utf8'));
+const EIP1271: object = JSON.parse(fs.readFileSync('../../test/eip1271.json', 'utf8'));
 
 import {
   providers,