refactor: Upgrade ESlint to v9 & create shared config

paraspell · Jul 3, 2024 · 524161b · 524161b
1 parent 74180e1
commit 524161b
Show file tree

Hide file tree

Showing 139 changed files with 889 additions and 2,380 deletions.
diff --git a/apps/landing-page/.eslintrc.cjs b/apps/landing-page/.eslintrc.cjs
diff --git a/apps/landing-page/package.json b/apps/landing-page/package.json
@@ -6,8 +6,9 @@
   "scripts": {
     "dev": "vite",
     "build": "tsc && vite build",
-    "lint": "eslint . --ext ts,tsx --report-unused-disable-directives --max-warnings 0",
-    "preview": "vite preview"
+    "preview": "vite preview",
+    "lint:check": "eslint .",
+    "lint": "eslint --fix ."
   },
   "dependencies": {
     "@mantine/core": "^7.6.2",
@@ -23,16 +24,10 @@
   "devDependencies": {
     "@types/react": "^18.2.69",
     "@types/react-dom": "^18.2.22",
-    "@typescript-eslint/eslint-plugin": "^6.21.0",
-    "@typescript-eslint/parser": "^6.21.0",
     "@vitejs/plugin-react": "^4.2.1",
-    "eslint": "^8.57.0",
-    "eslint-plugin-react-hooks": "^4.6.0",
-    "eslint-plugin-react-refresh": "^0.4.6",
     "postcss": "^8.4.38",
     "postcss-preset-mantine": "^1.13.0",
     "postcss-simple-vars": "^7.0.1",
-    "typescript": "^5.2.2",
     "vite": "^5.2.4",
     "vite-plugin-node-polyfills": "^0.19.0"
   }

diff --git a/apps/playground/.eslintrc.cjs b/apps/playground/.eslintrc.cjs
diff --git a/apps/playground/package.json b/apps/playground/package.json
@@ -6,8 +6,9 @@
   "scripts": {
     "dev": "vite",
     "build": "tsc && vite build",
-    "lint": "eslint . --ext ts,tsx --report-unused-disable-directives --max-warnings 0",
-    "preview": "vite preview"
+    "preview": "vite preview",
+    "lint:check": "eslint .",
+    "lint": "eslint --fix ."
   },
   "dependencies": {
     "@mantine/core": "^7.10.1",
@@ -33,16 +34,10 @@
   "devDependencies": {
     "@types/react": "^18.3.3",
     "@types/react-dom": "^18.3.0",
-    "@typescript-eslint/eslint-plugin": "^7.12.0",
-    "@typescript-eslint/parser": "^7.12.0",
     "@vitejs/plugin-react": "^4.3.0",
-    "eslint": "^8.57.0",
-    "eslint-plugin-react-hooks": "^4.6.2",
-    "eslint-plugin-react-refresh": "^0.4.7",
     "postcss": "^8.4.38",
     "postcss-preset-mantine": "^1.15.0",
     "postcss-simple-vars": "^7.0.1",
-    "typescript": "^5.4.5",
     "vite": "^5.2.13",
     "vite-plugin-node-polyfills": "^0.22.0",
     "vite-plugin-wasm": "^3.3.0"

diff --git a/apps/playground/src/App.tsx b/apps/playground/src/App.tsx
@@ -55,6 +55,7 @@ const App = () => {
 
   const [accounts, setAccounts] = useState<InjectedAccountWithMeta[]>([]);
 
+  // eslint-disable-next-line @typescript-eslint/unbound-method
   const { selectedAccount, setSelectedAccount } = useWallet();
 
   const initAccounts = async () => {
@@ -114,10 +115,12 @@ const App = () => {
               <Image src="logo.png" h="100%" p={8} />
               {selectedAccount ? (
                 <Button
+                  // eslint-disable-next-line @typescript-eslint/no-misused-promises
                   onClick={onChangeAccountClick}
                   variant="outline"
                 >{`${selectedAccount.meta.name} (${selectedAccount.meta.source})`}</Button>
               ) : (
+                // eslint-disable-next-line @typescript-eslint/no-misused-promises
                 <Button onClick={onConnectWalletClick}>Connect wallet</Button>
               )}
             </Group>

diff --git a/apps/playground/src/components/RouterTransferForm.tsx b/apps/playground/src/components/RouterTransferForm.tsx
@@ -1,3 +1,4 @@
+/* eslint-disable @typescript-eslint/no-misused-promises */
 import { useForm } from "@mantine/form";
 import {
   EXCHANGE_NODES,
@@ -58,7 +59,7 @@ const RouterTransferForm: FC<Props> = ({ onSubmit, loading }) => {
   };
 
   useEffect(() => {
-    (async () => {
+    void (async () => {
       if (selectedAccount) {
         const injector = await web3FromAddress(selectedAccount.address);
         form.setFieldValue("evmSigner", injector.signer);

diff --git a/apps/playground/src/components/TransferInfo.tsx b/apps/playground/src/components/TransferInfo.tsx
@@ -1,3 +1,4 @@
+/* eslint-disable @typescript-eslint/no-misused-promises */
 import { Stack, Title, Box } from "@mantine/core";
 import ErrorAlert from "./ErrorAlert";
 import { useDisclosure, useScrollIntoView } from "@mantine/hooks";
@@ -38,6 +39,7 @@ const TransferInfo = () => {
     const { useApi } = formValues;
     const originAddress = selectedAccount?.address ?? "";
     if (useApi) {
+      // eslint-disable-next-line @typescript-eslint/no-unsafe-return
       return await fetchFromApi(
         {
           origin: formValues.from,
@@ -70,6 +72,7 @@ const TransferInfo = () => {
     setLoading(true);
 
     try {
+      // eslint-disable-next-line @typescript-eslint/no-unsafe-assignment
       const output = await getQueryResult(formValues);
       setOutput(JSON.stringify(output, null, 2));
       openOutputAlert();

diff --git a/apps/playground/src/components/XcmTransfer.tsx b/apps/playground/src/components/XcmTransfer.tsx
@@ -1,3 +1,4 @@
+/* eslint-disable @typescript-eslint/no-misused-promises */
 import { Stack, Title, Box } from "@mantine/core";
 import ErrorAlert from "./ErrorAlert";
 import TransferForm, { FormValues } from "./TransferForm";
@@ -42,11 +43,7 @@ const XcmTransfer = () => {
         .address(address)
         .build();
     } else if (to === "Polkadot" || to === "Kusama") {
-      return Builder(api)
-        .from(from as TNode)
-        .amount(amount)
-        .address(address)
-        .build();
+      return Builder(api).from(from).amount(amount).address(address).build();
     } else {
       return Builder(api)
         .from(from)

diff --git a/apps/playground/src/components/asset-claim/AssetClaim.tsx b/apps/playground/src/components/asset-claim/AssetClaim.tsx
@@ -1,3 +1,4 @@
+/* eslint-disable @typescript-eslint/no-misused-promises */
 import { Stack, Title, Box } from "@mantine/core";
 import { useDisclosure, useScrollIntoView } from "@mantine/hooks";
 import { Builder, createApiInstanceForNode } from "@paraspell/sdk";

diff --git a/apps/playground/src/components/assets/AssetsQueries.tsx b/apps/playground/src/components/assets/AssetsQueries.tsx
@@ -1,3 +1,4 @@
+/* eslint-disable @typescript-eslint/no-misused-promises */
 import { Stack, Title, Box, Alert, Text } from "@mantine/core";
 import ErrorAlert from "../ErrorAlert";
 import { useDisclosure, useScrollIntoView } from "@mantine/hooks";
@@ -101,6 +102,7 @@ const AssetsQueries = () => {
   const getQueryResult = async (formValues: FormValues) => {
     const { useApi } = formValues;
     if (useApi) {
+      // eslint-disable-next-line @typescript-eslint/no-unsafe-return
       return await fetchFromApi(
         formValues,
         `/assets/${getEndpoint(formValues)}`
@@ -114,6 +116,7 @@ const AssetsQueries = () => {
     setLoading(true);
 
     try {
+      // eslint-disable-next-line @typescript-eslint/no-unsafe-assignment
       const output = await getQueryResult(formValues);
       if (typeof output === "bigint") {
         setOutput(output.toString());

diff --git a/apps/playground/src/components/channels/ChannelsQueries.tsx b/apps/playground/src/components/channels/ChannelsQueries.tsx
@@ -1,3 +1,4 @@
+/* eslint-disable @typescript-eslint/no-misused-promises */
 import { Stack, Title, Box } from "@mantine/core";
 import ErrorAlert from "../ErrorAlert";
 import { useDisclosure, useScrollIntoView } from "@mantine/hooks";
@@ -40,7 +41,7 @@ const ChannelsQueries = () => {
     }
   }, [error, scrollIntoView]);
 
-  const submitUsingSdk = async (
+  const submitUsingSdk = (
     { func, from, to, maxSize, maxMessageSize, inbound, outbound }: FormValues,
     api: ApiPromise
   ) => {
@@ -66,13 +67,14 @@ const ChannelsQueries = () => {
   const getQueryResult = async (formValues: FormValues, api: ApiPromise) => {
     const { func, useApi } = formValues;
     if (useApi) {
+      // eslint-disable-next-line @typescript-eslint/no-unsafe-return
       return await fetchFromApi(
         formValues,
         `/hrmp/channels`,
         func === "OPEN_CHANNEL" ? "POST" : "DELETE"
       );
     } else {
-      return await submitUsingSdk(formValues, api);
+      return submitUsingSdk(formValues, api);
     }
   };
 
@@ -90,12 +92,14 @@ const ChannelsQueries = () => {
 
     try {
       const api = await createApiInstanceForNode(from);
+      // eslint-disable-next-line @typescript-eslint/no-unsafe-assignment
       const output = await getQueryResult(formValues, api);
       setOutput(JSON.stringify(output, null, 2));
       openOutputAlert();
       closeErrorAlert();
       await submitTransaction(
         api,
+        // eslint-disable-next-line @typescript-eslint/no-unsafe-argument
         buildTx(api, output),
         injector.signer,
         selectedAccount.address

diff --git a/apps/playground/src/components/pallets/PalletsQueries.tsx b/apps/playground/src/components/pallets/PalletsQueries.tsx
@@ -1,16 +1,20 @@
-import { Stack, Title, Box } from '@mantine/core';
-import ErrorAlert from '../ErrorAlert';
-import { useDisclosure, useScrollIntoView } from '@mantine/hooks';
-import { useState, useEffect } from 'react';
-import { fetchFromApi } from '../../utils/submitUsingApi';
-import { getDefaultPallet, getSupportedPallets } from '@paraspell/sdk';
-import PalletsForm, { FormValues } from './PalletsForm';
-import OutputAlert from '../OutputAlert';
+/* eslint-disable @typescript-eslint/no-misused-promises */
+import { Stack, Title, Box } from "@mantine/core";
+import ErrorAlert from "../ErrorAlert";
+import { useDisclosure, useScrollIntoView } from "@mantine/hooks";
+import { useState, useEffect } from "react";
+import { fetchFromApi } from "../../utils/submitUsingApi";
+import { getDefaultPallet, getSupportedPallets } from "@paraspell/sdk";
+import PalletsForm, { FormValues } from "./PalletsForm";
+import OutputAlert from "../OutputAlert";
 
 const PalletsQueries = () => {
-  const [errorAlertOpened, { open: openErrorAlert, close: closeErrorAlert }] = useDisclosure(false);
-  const [outputAlertOpened, { open: openOutputAlert, close: closeOutputAlert }] =
+  const [errorAlertOpened, { open: openErrorAlert, close: closeErrorAlert }] =
     useDisclosure(false);
+  const [
+    outputAlertOpened,
+    { open: openOutputAlert, close: closeOutputAlert },
+  ] = useDisclosure(false);
 
   const [error, setError] = useState<Error>();
   const [output, setOutput] = useState<string>();
@@ -27,37 +31,42 @@ const PalletsQueries = () => {
     }
   }, [error, scrollIntoView]);
 
-  const submitUsingSdk = async ({ func, node }: FormValues) => {
+  const submitUsingSdk = ({ func, node }: FormValues) => {
     switch (func) {
-      case 'ALL_PALLETS':
+      case "ALL_PALLETS":
         return getSupportedPallets(node);
-      case 'DEFAULT_PALLET':
+      case "DEFAULT_PALLET":
         return getDefaultPallet(node);
     }
   };
 
   const getEndpoint = ({ func, node }: FormValues) => {
     switch (func) {
-      case 'ALL_PALLETS':
+      case "ALL_PALLETS":
         return `${node}`;
-      case 'DEFAULT_PALLET':
+      case "DEFAULT_PALLET":
         return `${node}/default`;
     }
   };
 
   const getQueryResult = async (formValues: FormValues) => {
     const { useApi } = formValues;
     if (useApi) {
-      return await fetchFromApi(formValues, `/pallets/${getEndpoint(formValues)}`);
+      // eslint-disable-next-line @typescript-eslint/no-unsafe-return
+      return await fetchFromApi(
+        formValues,
+        `/pallets/${getEndpoint(formValues)}`
+      );
     } else {
-      return await submitUsingSdk(formValues);
+      return submitUsingSdk(formValues);
     }
   };
 
   const onSubmit = async (formValues: FormValues) => {
     setLoading(true);
 
     try {
+      // eslint-disable-next-line @typescript-eslint/no-unsafe-assignment
       const output = await getQueryResult(formValues);
       setOutput(JSON.stringify(output, null, 2));
       openOutputAlert();
@@ -90,11 +99,15 @@ const PalletsQueries = () => {
       </Stack>
       <Box ref={targetRef}>
         {errorAlertOpened && (
-          <ErrorAlert onAlertCloseClick={onErrorAlertCloseClick}>{error?.message}</ErrorAlert>
+          <ErrorAlert onAlertCloseClick={onErrorAlertCloseClick}>
+            {error?.message}
+          </ErrorAlert>
         )}
       </Box>
       <Box>
-        {outputAlertOpened && <OutputAlert onClose={onOutputAlertCloseClick}>{output}</OutputAlert>}
+        {outputAlertOpened && (
+          <OutputAlert onClose={onOutputAlertCloseClick}>{output}</OutputAlert>
+        )}
       </Box>
     </Stack>
   );

diff --git a/apps/playground/src/main.tsx b/apps/playground/src/main.tsx
@@ -3,8 +3,9 @@ import ReactDOM from "react-dom/client";
 import App from "./App";
 import WalletProvider from "./providers/WalletProvider";
 
-// eslint-disable-next-line @typescript-eslint/no-explicit-any
+// eslint-disable-next-line @typescript-eslint/no-explicit-any, @typescript-eslint/no-unsafe-member-access
 (BigInt.prototype as any)["toJSON"] = function () {
+  // eslint-disable-next-line @typescript-eslint/no-unsafe-return, @typescript-eslint/no-unsafe-call, @typescript-eslint/no-unsafe-member-access
   return this.toString();
 };
 

diff --git a/apps/playground/src/providers/WalletProvider.tsx b/apps/playground/src/providers/WalletProvider.tsx
@@ -14,7 +14,7 @@ const getWalletStateFromLocalStorage = ():
     return undefined;
   }
 
-  return JSON.parse(walletState);
+  return JSON.parse(walletState) as InjectedAccountWithMeta;
 };
 
 const WalletProvider: React.FC<PropsWithChildren<unknown>> = ({ children }) => {
@@ -28,7 +28,7 @@ const WalletProvider: React.FC<PropsWithChildren<unknown>> = ({ children }) => {
   }, [selectedAccount]);
 
   useEffect(() => {
-    web3Enable("SpellRouter");
+    void web3Enable("SpellRouter");
   }, []);
 
   return (