Merge pull request #1757 from kleros/chore/eslint-and-more-version-bump

tractorss · web-flow · commit 88c1537e69d7 · 2024-11-22T13:50:12.000+05:30
Chore/eslint and more version bump
diff --git a/bot-pinner/package.json b/bot-pinner/package.json
@@ -19,6 +19,6 @@
     "node": ">=16.13.0"
   },
   "devDependencies": {
-    "@dappnode/dappnodesdk": "^0.3.11"
+    "@dappnode/dappnodesdk": "^0.3.35"
   }
 }
diff --git a/contracts/package.json b/contracts/package.json
@@ -60,7 +60,7 @@
     "build:types": "tsc --project tsconfig.json --module esnext --declarationDir ./dist/types --emitDeclarationOnly --declaration --declarationMap"
   },
   "devDependencies": {
-    "@defi-wonderland/natspec-smells": "^1.0.3",
+    "@defi-wonderland/natspec-smells": "^1.1.5",
     "@kleros/kleros-v2-eslint-config": "workspace:^",
     "@kleros/kleros-v2-prettier-config": "workspace:^",
     "@kleros/kleros-v2-tsconfig": "workspace:^",
@@ -71,18 +71,19 @@
     "@openzeppelin/contracts": "^5.1.0",
     "@typechain/ethers-v6": "^0.5.1",
     "@typechain/hardhat": "^9.1.0",
-    "@types/chai": "^4.3.11",
-    "@types/mocha": "^10.0.6",
-    "@types/node": "^20.17.1",
-    "@wagmi/cli": "^2.1.16",
+    "@types/chai": "^4.3.20",
+    "@types/mocha": "^10.0.10",
+    "@types/node": "^20.17.6",
+    "@wagmi/cli": "^2.1.18",
     "abitype": "^0.10.3",
-    "chai": "^4.4.1",
-    "dotenv": "^16.3.1",
+    "chai": "^4.5.0",
+    "dotenv": "^16.4.5",
+    "eslint": "^9.15.0",
     "ethereumjs-util": "^7.1.5",
     "ethers": "^6.13.4",
     "graphql": "^16.9.0",
-    "graphql-request": "^6.1.0",
-    "hardhat": "2.22.15",
+    "graphql-request": "^7.1.2",
+    "hardhat": "2.22.16",
     "hardhat-contract-sizer": "^2.10.0",
     "hardhat-deploy": "^0.14.0",
     "hardhat-deploy-ethers": "^0.4.2",
@@ -94,15 +95,17 @@
     "node-fetch": "^3.3.2",
     "pino": "^8.21.0",
     "pino-pretty": "^10.3.1",
+    "prettier": "^3.3.3",
+    "prettier-plugin-solidity": "^1.4.1",
     "shelljs": "^0.8.5",
     "solhint-plugin-prettier": "^0.1.0",
     "solidity-coverage": "^0.8.13",
     "ts-node": "^10.9.2",
     "typechain": "^8.3.2",
-    "typescript": "^5.3.3"
+    "typescript": "^5.6.3"
   },
   "dependencies": {
     "@kleros/vea-contracts": "^0.4.0",
-    "viem": "^2.21.35"
+    "viem": "^2.21.48"
   }
 }
diff --git a/eslint-config/package.json b/eslint-config/package.json
@@ -5,25 +5,26 @@
   "main": ".eslintrc.js",
   "license": "MIT",
   "dependencies": {
-    "@typescript-eslint/eslint-plugin": "^8.8.1",
-    "@typescript-eslint/parser": "^8.8.1",
-    "@typescript-eslint/utils": "^8.8.1",
+    "@typescript-eslint/eslint-plugin": "^8.15.0",
+    "@typescript-eslint/parser": "^8.15.0",
+    "@typescript-eslint/utils": "^8.15.0",
+    "eslint": "^9.15.0",
     "eslint-config-prettier": "^9.1.0",
-    "eslint-config-standard": "^17.1.0",
     "eslint-plugin-import": "^2.31.0",
     "eslint-plugin-node": "^11.1.0",
     "eslint-plugin-prettier": "^5.2.1",
-    "eslint-plugin-promise": "^6.0.0",
+    "eslint-plugin-promise": "^6.6.0",
     "eslint-plugin-security": "^3.0.1",
-    "eslint-utils": "^3.0.0"
+    "eslint-utils": "^3.0.0",
+    "prettier": "^3.3.3"
   },
   "devDependencies": {
-    "@eslint/eslintrc": "^3.1.0",
-    "@eslint/js": "^9.12.0",
-    "globals": "^15.11.0",
-    "typescript": "^5.3.3"
+    "@eslint/eslintrc": "^3.2.0",
+    "@eslint/js": "^9.15.0",
+    "globals": "^15.12.0",
+    "typescript": "^5.6.3"
   },
   "peerDependencies": {
-    "eslint": "8.x"
+    "eslint": "^8.0.0 || ^9.0.0"
   }
 }
diff --git a/kleros-app/package.json b/kleros-app/package.json
@@ -33,16 +33,16 @@
   ],
   "prettier": "@kleros/kleros-v2-prettier-config",
   "devDependencies": {
-    "@eslint/compat": "^1.2.2",
-    "@eslint/eslintrc": "^3.1.0",
-    "@eslint/js": "^9.14.0",
+    "@eslint/compat": "^1.2.3",
+    "@eslint/eslintrc": "^3.2.0",
+    "@eslint/js": "^9.15.0",
     "@kleros/kleros-v2-eslint-config": "workspace:^",
     "@kleros/kleros-v2-prettier-config": "workspace:^",
     "@types/react": "^18.3.12",
     "@types/react-dom": "^18.3.1",
     "@typescript-eslint/eslint-plugin": "^8.15.0",
     "@typescript-eslint/parser": "^8.15.0",
-    "eslint": "^9.14.0",
+    "eslint": "^9.15.0",
     "eslint-config-prettier": "^9.1.0",
     "eslint-plugin-import": "^2.31.0",
     "globals": "^15.12.0",
diff --git a/kleros-sdk/package.json b/kleros-sdk/package.json
@@ -36,18 +36,19 @@
   },
   "devDependencies": {
     "@types/mustache": "^4.2.5",
-    "@vitest/ui": "^1.1.3",
-    "mocha": "^10.2.0",
+    "@types/node": "^20.17.6",
+    "@vitest/ui": "^1.6.0",
+    "mocha": "^10.8.2",
     "rimraf": "^6.0.1",
     "ts-node": "^10.9.2",
-    "typescript": "^5.3.3",
-    "vitest": "^1.1.3"
+    "typescript": "^5.6.3",
+    "vitest": "^1.6.0"
   },
   "dependencies": {
     "@reality.eth/reality-eth-lib": "^3.2.44",
     "@urql/core": "^5.0.8",
     "mustache": "^4.2.0",
-    "viem": "^2.21.35",
-    "zod": "^3.22.4"
+    "viem": "^2.21.48",
+    "zod": "^3.23.8"
   }
 }
diff --git a/package.json b/package.json
@@ -36,14 +36,14 @@
   "devDependencies": {
     "@commitlint/cli": "^17.8.1",
     "@commitlint/config-conventional": "^17.8.1",
-    "assert": "^2.0.0",
+    "assert": "^2.1.0",
     "buffer": "^5.7.1",
     "conventional-changelog-cli": "^2.2.2",
-    "crypto-browserify": "^3.12.0",
+    "crypto-browserify": "^3.12.1",
     "husky": "^8.0.3",
     "lint-staged": "^13.3.0",
     "os-browserify": "^0.3.0",
-    "path-browserify": "^1.0.0",
+    "path-browserify": "^1.0.1",
     "process": "^0.11.10",
     "string_decoder": "^1.3.0"
   },
diff --git a/prettier-config/package.json b/prettier-config/package.json
@@ -4,9 +4,9 @@
   "main": "index.js",
   "license": "MIT",
   "dependencies": {
-    "eslint": "^8.57.1",
+    "eslint": "^9.15.0",
     "prettier": "^3.3.3",
-    "prettier-plugin-solidity": "^1.3.1"
+    "prettier-plugin-solidity": "^1.4.1"
   },
   "scripts": {
     "lint:w": "eslint --fix '**/*.{gql,graphql,js,jsx,ts,tsx,json,md}'",
diff --git a/subgraph/package.json b/subgraph/package.json
@@ -78,6 +78,7 @@
     "@graphprotocol/graph-cli": "0.64.1",
     "@kleros/kleros-v2-eslint-config": "workspace:^",
     "@kleros/kleros-v2-prettier-config": "workspace:^",
+    "eslint": "^9.15.0",
     "gluegun": "^5.2.0",
     "matchstick-as": "0.6.0"
   },
diff --git a/tsconfig/package.json b/tsconfig/package.json
@@ -9,6 +9,7 @@
   "devDependencies": {
     "@kleros/kleros-v2-eslint-config": "*",
     "@tsconfig/node18": "^18.2.4",
-    "@tsconfig/node20": "^20.1.4"
+    "@tsconfig/node20": "^20.1.4",
+    "eslint": "^9.15.0"
   }
 }
diff --git a/web-devtools/eslint.config.mjs b/web-devtools/eslint.config.mjs
@@ -30,7 +30,6 @@ export default [
       "plugin:react-hooks/recommended",
       "plugin:import/recommended",
       "plugin:import/react",
-      "plugin:security/recommended",
       "plugin:@typescript-eslint/recommended",
       "plugin:prettier/recommended",
       "prettier"
@@ -41,7 +40,6 @@ export default [
       react: fixupPluginRules(react),
       "react-hooks": fixupPluginRules(reactHooks),
       security: fixupPluginRules(security),
-      import: fixupPluginRules(_import),
     },
 
     languageOptions: {
diff --git a/web-devtools/package.json b/web-devtools/package.json
@@ -25,40 +25,47 @@
   },
   "prettier": "@kleros/kleros-v2-prettier-config",
   "devDependencies": {
-    "@graphql-codegen/cli": "^5.0.2",
-    "@graphql-codegen/client-preset": "^4.3.2",
+    "@graphql-codegen/cli": "^5.0.3",
+    "@graphql-codegen/client-preset": "^4.5.0",
     "@svgr/webpack": "^8.1.0",
-    "@types/node": "^20.17.1",
-    "@types/react": "18.2.0",
-    "@types/react-dom": "^18.2.18",
-    "@typescript-eslint/eslint-plugin": "^8.8.1",
-    "@typescript-eslint/parser": "^8.8.1",
-    "@typescript-eslint/utils": "^8.8.1",
-    "@wagmi/cli": "^2.1.16",
-    "eslint": "^8.57.1",
-    "eslint-config-next": "^14.2.15",
+    "@types/node": "^20.17.6",
+    "@types/react": "^18.3.12",
+    "@types/react-dom": "^18.3.1",
+    "@types/react-is": "^18.3.0",
+    "@types/styled-components": "^5.1.34",
+    "@typescript-eslint/eslint-plugin": "^8.15.0",
+    "@typescript-eslint/parser": "^8.15.0",
+    "@typescript-eslint/utils": "^8.15.0",
+    "@wagmi/cli": "^2.1.18",
+    "eslint": "^9.15.0",
+    "eslint-config-next": "^15.0.3",
     "eslint-config-prettier": "^9.1.0",
     "eslint-import-resolver-typescript": "^3.6.3",
-    "eslint-plugin-react": "^7.37.1",
-    "eslint-plugin-react-hooks": "^4.6.2",
+    "eslint-plugin-react": "^7.37.2",
+    "eslint-plugin-react-hooks": "^5.0.0",
     "rimraf": "^6.0.1",
     "ts-node": "^10.9.2",
-    "typescript": "^5.5.3"
+    "typescript": "^5.6.3"
   },
   "dependencies": {
     "@kleros/kleros-sdk": "workspace:^",
     "@kleros/ui-components-library": "^2.15.0",
+    "@tanstack/react-query": "^5.61.0",
+    "@wagmi/connectors": "^5.5.0",
+    "@wagmi/core": "^2.15.0",
     "@web3modal/wagmi": "^5.1.11",
     "graphql": "^16.9.0",
-    "graphql-request": "^7.1.0",
-    "next": "14.2.14",
-    "react": "^18.2.0",
-    "react-dom": "^18.2.0",
-    "react-markdown": "^8.0.7",
-    "react-toastify": "^10.0.5",
+    "graphql-request": "^7.1.2",
+    "next": "15.0.3",
+    "react": "^18.3.1",
+    "react-dom": "^18.3.1",
+    "react-is": "^18.3.1",
+    "react-markdown": "^9.0.1",
+    "react-toastify": "^10.0.6",
+    "styled-components": "^6.1.13",
     "typewriter-effect": "^2.21.0",
-    "vanilla-jsoneditor": "^0.21.4",
-    "viem": "^2.21.35",
-    "wagmi": "^2.12.25"
+    "vanilla-jsoneditor": "^0.21.6",
+    "viem": "^2.21.48",
+    "wagmi": "^2.13.0"
   }
 }
diff --git a/web-devtools/src/app/(main)/dispute-template/FetchDisputeRequestInput.tsx b/web-devtools/src/app/(main)/dispute-template/FetchDisputeRequestInput.tsx
@@ -115,7 +115,7 @@ const FetchDisputeRequestInput: React.FC<IFetchDisputeRequestInput> = ({ setPara
           value={chainId}
           placeholder="Enter chain Id"
           type="number"
-          onChange={(e) => setChainId(Number(e.target.value))}
+          onChange={(e: React.ChangeEvent<HTMLInputElement>) => setChainId(Number(e.target.value))}
         />
       </InputContainer>
       <StyledH3>Presets</StyledH3>
diff --git a/web-devtools/src/app/(main)/dispute-template/FetchFromIdInput.tsx b/web-devtools/src/app/(main)/dispute-template/FetchFromIdInput.tsx
@@ -60,7 +60,7 @@ const FetchFromIDInput: React.FC<IFetchFromID> = ({
         value={templateId}
         placeholder="Enter template Id"
         message={isLoading ? "fetching ..." : ""}
-        onChange={(e) => setTemplateId(e.target.value)}
+        onChange={(e: React.ChangeEvent<HTMLInputElement>) => setTemplateId(e.target.value)}
       />
     </Container>
   );
diff --git a/web-devtools/src/layout/Header/navbar/Explore.tsx b/web-devtools/src/layout/Header/navbar/Explore.tsx
@@ -38,12 +38,12 @@ const Title = styled.h1`
   )};
 `;
 
-const StyledLink = styled(Link)<{ isActive: boolean }>`
+const StyledLink = styled(Link)<{ $isActive: boolean }>`
   color: ${({ theme }) => theme.klerosUIComponentsPrimaryText};
   text-decoration: none;
   font-size: 16px;
 
-  font-weight: ${({ isActive }) => (isActive ? "600" : "normal")};
+  font-weight: ${({ $isActive }) => ($isActive ? "600" : "normal")};
 
   ${landscapeStyle(
     () => css`
@@ -70,7 +70,7 @@ const Explore: React.FC = () => {
           <StyledLink
             href={to}
             onClick={toggleIsOpen}
-            isActive={to === "/" ? pathname === "/" : pathname.startsWith(to)}
+            $isActive={to === "/" ? pathname === "/" : pathname.startsWith(to)}
           >
             {text}
           </StyledLink>
diff --git a/web-devtools/src/layout/Header/navbar/index.tsx b/web-devtools/src/layout/Header/navbar/index.tsx
@@ -8,16 +8,16 @@ import { useOpenContext } from "../MobileHeader";
 
 import Explore from "./Explore";
 
-const Wrapper = styled.div<{ isOpen: boolean }>`
-  visibility: ${({ isOpen }) => (isOpen ? "visible" : "hidden")};
+const Wrapper = styled.div<{ $isOpen: boolean }>`
+  visibility: ${({ $isOpen }) => ($isOpen ? "visible" : "hidden")};
   position: absolute;
   top: 100%;
   left: 0;
   width: 100vw;
   z-index: 30;
 `;
 
-const Container = styled.div<{ isOpen: boolean }>`
+const Container = styled.div<{ $isOpen: boolean }>`
   position: absolute;
   top: 0;
   left: 0;
@@ -29,8 +29,8 @@ const Container = styled.div<{ isOpen: boolean }>`
   border: 1px solid ${({ theme }) => theme.klerosUIComponentsStroke};
   box-shadow: 0px 2px 3px ${({ theme }) => theme.klerosUIComponentsDefaultShadow};
   transform-origin: top;
-  transform: scaleY(${({ isOpen }) => (isOpen ? "1" : "0")});
-  visibility: ${({ isOpen }) => (isOpen ? "visible" : "hidden")};
+  transform: scaleY(${({ $isOpen }) => ($isOpen ? "1" : "0")});
+  visibility: ${({ $isOpen }) => ($isOpen ? "visible" : "hidden")};
   transition-property: transform, visibility;
   transition-duration: ${({ theme }) => theme.klerosUIComponentsTransitionSpeed};
   transition-timing-function: ease;
@@ -47,8 +47,8 @@ const NavBar: React.FC = () => {
 
   return (
     <>
-      <Wrapper {...{ isOpen }}>
-        <Container {...{ isOpen }}>
+      <Wrapper {...{ $isOpen: isOpen }}>
+        <Container {...{ $isOpen: isOpen }}>
           <Explore />
         </Container>
       </Wrapper>
diff --git a/web-devtools/src/styles/landscapeStyle.ts b/web-devtools/src/styles/landscapeStyle.ts
@@ -1,8 +1,8 @@
-import { css, DefaultTheme, FlattenInterpolation, ThemeProps } from "styled-components";
+import { css, RuleSet } from "styled-components";
 
 export const BREAKPOINT_LANDSCAPE = 900;
 
-export const landscapeStyle = (styleFn: () => FlattenInterpolation<ThemeProps<DefaultTheme>>) => css`
+export const landscapeStyle = (styleFn: () => RuleSet<object>) => css`
   @media (min-width: ${BREAKPOINT_LANDSCAPE}px) {
     ${() => styleFn()}
   }
diff --git a/web-devtools/tsconfig.json b/web-devtools/tsconfig.json
@@ -1,5 +1,6 @@
 {
   "compilerOptions": {
+    "target": "ES2022",
     "lib": [
       "dom",
       "dom.iterable",
diff --git a/web-devtools/wagmi.config.ts b/web-devtools/wagmi.config.ts
@@ -6,8 +6,8 @@ import { react, actions } from "@wagmi/cli/plugins";
 import dotenv from "dotenv";
 import { Abi, Chain } from "viem";
 
-import IArbitrableV2 from "@kleros/kleros-v2-contracts/artifacts/src/arbitration/interfaces/IArbitrableV2.sol/IArbitrableV2.json" assert { type: "json" };
-import IHomeGateway from "@kleros/kleros-v2-contracts/artifacts/src/gateway/interfaces/IHomeGateway.sol/IHomeGateway.json" assert { type: "json" };
+import IArbitrableV2 from "@kleros/kleros-v2-contracts/artifacts/src/arbitration/interfaces/IArbitrableV2.sol/IArbitrableV2.json" with { type: "json" };
+import IHomeGateway from "@kleros/kleros-v2-contracts/artifacts/src/gateway/interfaces/IHomeGateway.sol/IHomeGateway.json" with { type: "json" };
 
 import { ArbitratorTypes, getArbitratorType } from "./src/consts/arbitratorTypes";
 import { arbitrum, arbitrumSepolia, gnosis, gnosisChiado, mainnet, sepolia } from "viem/chains";
diff --git a/web/package.json b/web/package.json
diff --git a/yarn.lock b/yarn.lock

Original file line number	Diff line number	Diff line change
`@@ -19,6 +19,6 @@`
`19`	`19`	`"node": ">=16.13.0"`
`20`	`20`	`},`
`21`	`21`	`"devDependencies": {`
`22`		`- "@dappnode/dappnodesdk": "^0.3.11"`
	`22`	`+ "@dappnode/dappnodesdk": "^0.3.35"`
`23`	`23`	`}`
`24`	`24`	`}`
Original file line number	Diff line number	Diff line change
`@@ -9,6 +9,7 @@`
`9`	`9`	`"devDependencies": {`
`10`	`10`	`"@kleros/kleros-v2-eslint-config": "*",`
`11`	`11`	`"@tsconfig/node18": "^18.2.4",`
`12`		`- "@tsconfig/node20": "^20.1.4"`
	`12`	`+ "@tsconfig/node20": "^20.1.4",`
	`13`	`+ "eslint": "^9.15.0"`
`13`	`14`	`}`
`14`	`15`	`}`