chore: finish setting up linter (#21)

Author: ilyar

eslint.config.js

@@ -1,21 +1 @@
-// eslint-disable-next-line @typescript-eslint/no-require-imports
-const base = require('@ton/toolchain');
-// eslint-disable-next-line @typescript-eslint/no-require-imports
-const tsEslint = require('@ton/toolchain').tsEslint;
-
-module.exports = [
-    ...base,
-    {
-        plugins: {
-            '@typescript-eslint': tsEslint,
-        },
-        rules: {
-            'no-console': 'off',
-            '@typescript-eslint/no-explicit-any': 'off',
-            '@typescript-eslint/no-require-imports': 'warn',
-            '@typescript-eslint/no-namespace': 'warn',
-            'no-undef': 'warn',
-            'no-empty': 'warn',
-        },
-    },
-];
+module.exports = require('@ton/toolchain');

package.json

@@ -43,8 +43,8 @@
   },
   "prettier": "@ton/toolchain/prettier",
   "scripts": {
-    "lint": "eslint .",
-    "lint:fix": "eslint . --fix",
+    "lint": "eslint . --max-warnings 0",
+    "lint:fix": "eslint . --max-warnings 0 --fix",
     "test": "jest",
     "build": "rm -rf dist && tsc"
   },

src/test/chai.ts

@@ -4,7 +4,9 @@ import { compareAddressForTest, compareCellForTest, compareSliceForTest } from '
 import { CompareResult } from './interface';
 import { compareTransactionForTest, FlatTransactionComparable } from './transaction';
 
+// eslint-disable-next-line @typescript-eslint/no-explicit-any
 function wrapComparer<T>(comparer: (subject: any, cmp: T) => CompareResult) {
+    // eslint-disable-next-line @typescript-eslint/no-explicit-any
     return function (this: any, cmp: T) {
         const result = comparer(this._obj, cmp);
         this.assert(result.pass, result.posMessage(), result.negMessage());

src/test/comparisons.ts

@@ -2,14 +2,17 @@ import { Address, Cell, Slice } from '@ton/core';
 
 import { CompareResult } from './interface';
 
+// eslint-disable-next-line @typescript-eslint/no-explicit-any
 export function compareCellForTest(subject: any, cmp: Cell): CompareResult {
     return {
         pass: cmp.equals(subject),
+        // eslint-disable-next-line @typescript-eslint/no-explicit-any
         posMessage: ((subject: any, cmp: Cell) => `Expected\n${subject}\nto equal\n${cmp}`).bind(
             undefined,
             subject,
             cmp,
         ),
+        // eslint-disable-next-line @typescript-eslint/no-explicit-any
         negMessage: ((subject: any, cmp: Cell) => `Expected\n${subject}\nNOT to equal\n${cmp}\nbut it does`).bind(
             undefined,
             subject,
@@ -18,14 +21,17 @@ export function compareCellForTest(subject: any, cmp: Cell): CompareResult {
     };
 }
 
+// eslint-disable-next-line @typescript-eslint/no-explicit-any
 export function compareAddressForTest(subject: any, cmp: Address): CompareResult {
     return {
         pass: cmp.equals(subject),
+        // eslint-disable-next-line @typescript-eslint/no-explicit-any
         posMessage: ((subject: any, cmp: Address) => `Expected ${subject} to equal ${cmp}`).bind(
             undefined,
             subject,
             cmp,
         ),
+        // eslint-disable-next-line @typescript-eslint/no-explicit-any
         negMessage: ((subject: any, cmp: Address) => `Expected ${subject} NOT to equal ${cmp}, but it does`).bind(
             undefined,
             subject,
@@ -34,14 +40,17 @@ export function compareAddressForTest(subject: any, cmp: Address): CompareResult
     };
 }
 
+// eslint-disable-next-line @typescript-eslint/no-explicit-any
 export function compareSliceForTest(subject: any, cmp: Slice): CompareResult {
     return {
         pass: cmp.asCell().equals(subject.asCell()),
+        // eslint-disable-next-line @typescript-eslint/no-explicit-any
         posMessage: ((subject: any, cmp: Slice) => `Expected\n${subject}\nto equal\n${cmp}`).bind(
             undefined,
             subject,
             cmp,
         ),
+        // eslint-disable-next-line @typescript-eslint/no-explicit-any
         negMessage: ((subject: any, cmp: Slice) => `Expected\n${subject}\nNOT to equal\n${cmp}\nbut it does`).bind(
             undefined,
             subject,

src/test/jest.ts

@@ -5,6 +5,7 @@ import { FlatTransactionComparable, compareTransactionForTest } from './transact
 import { CompareResult } from './interface';
 import { compareAddressForTest, compareCellForTest, compareSliceForTest } from './comparisons';
 
+// eslint-disable-next-line @typescript-eslint/no-explicit-any
 function wrapComparer<T>(comparer: (subject: any, cmp: T) => CompareResult): MatcherFunction<[cmp: T]> {
     return function (actual, cmp) {
         const result = comparer(actual, cmp);

src/test/transaction.ts

@@ -116,6 +116,7 @@ export function flattenTransaction(tx: Transaction): FlatTransaction {
     };
 }
 
+// eslint-disable-next-line @typescript-eslint/no-explicit-any
 function compareValue(a: any, b: any) {
     if (a instanceof Address) {
         if (!(b instanceof Address)) return false;
@@ -143,7 +144,9 @@ export function compareTransaction(tx: FlatTransaction, cmp: FlatTransactionComp
     for (const key in cmp) {
         if (!(key in tx)) throw new Error(`Unknown flat transaction object key ${key}`);
 
+        // eslint-disable-next-line @typescript-eslint/no-explicit-any
         const cmpv = (cmp as any)[key];
+        // eslint-disable-next-line @typescript-eslint/no-explicit-any
         const txv = (tx as any)[key];
         if (typeof cmpv === 'function') {
             if (!cmpv(txv)) return false;
@@ -155,16 +158,19 @@ export function compareTransaction(tx: FlatTransaction, cmp: FlatTransactionComp
     return true;
 }
 
+// eslint-disable-next-line @typescript-eslint/no-explicit-any
 export function compareTransactionForTest(subject: any, cmp: FlatTransactionComparable): CompareResult {
     if (Array.isArray(subject)) {
         return {
             pass: subject.some((tx) => compareTransaction(flattenTransaction(tx), cmp)),
+            // eslint-disable-next-line @typescript-eslint/no-explicit-any
             posMessage: ((subj: any[], cmp: FlatTransactionComparable) =>
                 `Expected ${inspect(subj.map((tx) => prettifyTransaction(tx)))} to contain a transaction that matches pattern ${inspect(cmp)}`).bind(
                 undefined,
                 subject,
                 cmp,
             ),
+            // eslint-disable-next-line @typescript-eslint/no-explicit-any
             negMessage: ((subj: any[], cmp: FlatTransactionComparable) =>
                 `Expected ${inspect(subj.map((tx) => prettifyTransaction(tx)))} NOT to contain a transaction that matches pattern ${inspect(cmp)}, but it does`).bind(
                 undefined,
@@ -176,12 +182,14 @@ export function compareTransactionForTest(subject: any, cmp: FlatTransactionComp
         try {
             return {
                 pass: compareTransaction(flattenTransaction(subject), cmp),
+                // eslint-disable-next-line @typescript-eslint/no-explicit-any
                 posMessage: ((subj: any, cmp: FlatTransactionComparable) =>
                     `Expected ${inspect(prettifyTransaction(subj))} to match pattern ${inspect(cmp)}`).bind(
                     undefined,
                     subject,
                     cmp,
                 ),
+                // eslint-disable-next-line @typescript-eslint/no-explicit-any
                 negMessage: ((subj: any, cmp: FlatTransactionComparable) =>
                     `Expected ${inspect(prettifyTransaction(subj))} NOT to match pattern ${inspect(cmp)}, but it does`).bind(
                     undefined,
@@ -191,6 +199,7 @@ export function compareTransactionForTest(subject: any, cmp: FlatTransactionComp
             };
         } catch (e) {
             if (subject.transactions !== undefined) {
+                // eslint-disable-next-line no-console
                 console.warn(
                     'It seems that a SendMessageResult is being used for this comparison. Please make sure to pass `result.transactions` instead of just `result` into the matcher.',
                 );