update linter rules and fix

.solhint.json

@@ -2,20 +2,28 @@
   "extends": "solhint:recommended",
   "plugins": ["prettier"],
   "rules": {
-    "constructor-syntax": "error",
-    "payable-fallback": "error",
+    "explicit-types": "error",
+    "max-states-count": "error",
+    "no-console": "error",
+    "no-empty-blocks": "error",
+    "no-global-import": "error",
+    "no-unused-import": "error",
     "no-unused-vars": "error",
-    "prettier/prettier": "error",
+    "payable-fallback": "error",
+    "constructor-syntax": "error",
     "comprehensive-interface": "warn",
+    "func-named-parameters": "warn",
+    "imports-on-top": "error",
+    "ordering": "warn",
+    "visibility-modifier-order": "error",
     "avoid-call-value": "error",
     "avoid-sha3": "error",
     "avoid-suicide": "error",
     "avoid-throw": "error",
     "avoid-tx-origin": "off",
     "check-send-result": "error",
-    "compiler-version": ["error", ">=0.6.0"],
+    "compiler-version": ["error", ">=0.8.0"],
     "func-visibility": ["error", { "ignoreConstructors": true }],
-    "mark-callable-contracts": "off",
     "multiple-sends": "error",
     "no-complex-fallback": "error",
     "no-inline-assembly": "warn",

contracts/AMMStrategy.sol

@@ -2,7 +2,6 @@
 pragma solidity 0.8.17;
 
 import { IERC20 } from "@openzeppelin/contracts/token/ERC20/IERC20.sol";
-import { ERC20 } from "@openzeppelin/contracts/token/ERC20/ERC20.sol";
 import { SafeERC20 } from "@openzeppelin/contracts/token/ERC20/utils/SafeERC20.sol";
 
 import { Ownable } from "./abstracts/Ownable.sol";

contracts/interfaces/IUniswapV3SwapRouter.sol

@@ -1,7 +1,7 @@
 // SPDX-License-Identifier: GPL-2.0-or-later
 pragma solidity >=0.8.0;
 
-import "./IUniswapV3SwapCallback.sol";
+import { IUniswapV3SwapCallback } from "./IUniswapV3SwapCallback.sol";
 
 /// @title Router token swapping functionality
 /// @notice Functions for swapping tokens via Uniswap V3