bug:

- Fixed a script formatting bug that prevents the eslint commands from being run on windows
- Added an unused import plugin to eslint
- Updated the eslint script to automatically cleanup and organize imports on fix.

Author: Kailean O'Keefe

.eslintrc

@@ -18,12 +18,31 @@
     "plugin:import/recommended",
     "plugin:@react-three/recommended"
   ],
-  "plugins": ["@typescript-eslint", "react", "react-hooks", "import", "prettier", "@react-three"],
+  "plugins": [ "@typescript-eslint", "react", "react-hooks", "import", "simple-import-sort", "unused-imports", "prettier", "@react-three" ],
   "rules": {
     "import/no-unresolved": "off",
     "import/named": "off",
     "import/namespace": "off",
     "import/no-named-as-default-member": "off",
-    "import/extensions": ["error", "ignorePackages"]
+    "import/extensions": [ "error", "ignorePackages" ],
+
+    "import/order": ["error", {
+      "groups": [
+        "builtin",
+        "external",
+        "internal",
+        ["parent", "sibling", "index"],
+        "object",
+        "type"
+      ],
+      "newlines-between": "never",
+      "alphabetize": {
+        "order": "asc",
+        "caseInsensitive": true
+      }
+    }],
+
+    // "simple-import-sort/imports": "error",
+    "unused-imports/no-unused-imports": "error"
   }
-}
+}

package.json

@@ -3,9 +3,9 @@
   "license": "MIT",
   "type": "module",
   "devDependencies": {
+    "@react-three/drei": "rc",
     "@react-three/eslint-plugin": "^0.1.1",
     "@react-three/fiber": "rc",
-    "@react-three/drei": "rc",
     "@types/chai": "^5.0.1",
     "@types/eslint": "^8.56.10",
     "@types/mocha": "^10.0.10",
@@ -24,6 +24,7 @@
     "eslint-plugin-prettier": "^5.1.3",
     "eslint-plugin-react": "^7.34.2",
     "eslint-plugin-react-hooks": "^4.6.2",
+    "eslint-plugin-unused-imports": "^4.1.4",
     "json": "^11.0.0",
     "mocha": "^10.8.2",
     "playwright": "^1.45.1",

packages/handle/package.json

@@ -9,9 +9,9 @@
     "build": "tsc",
     "test": "mocha ./tests/*.spec.ts",
     "check:prettier": "prettier --check src",
-    "check:eslint": "eslint 'src/**/*.ts'",
+    "check:eslint": "eslint \"src/**/*.{ts,tsx}\"",
     "fix:prettier": "prettier --write src",
-    "fix:eslint": "eslint 'src/**/*.ts' --fix"
+    "fix:eslint": "eslint \"src/**/*.{ts,tsx}\" --fix"
   },
   "keywords": [
     "r3f",

packages/pointer-events/package.json

@@ -33,9 +33,9 @@
     "example": "vite example --host",
     "example:build": "vite build example",
     "check:prettier": "prettier --check src",
-    "check:eslint": "eslint 'src/**/*.ts'",
+    "check:eslint": "eslint \"src/**/*.{ts,tsx}\"",
     "fix:prettier": "prettier --write src",
-    "fix:eslint": "eslint 'src/**/*.ts' --fix"
+    "fix:eslint": "eslint \"src/**/*.{ts,tsx}\" --fix"
   },
   "dependencies": {},
   "devDependencies": {

packages/react/handle/package.json

@@ -40,8 +40,8 @@
   "scripts": {
     "build": "tsc",
     "check:prettier": "prettier --check src",
-    "check:eslint": "eslint 'src/**/*.ts'",
+    "check:eslint": "eslint \"src/**/*.{ts,tsx}\"",
     "fix:prettier": "prettier --write src",
-    "fix:eslint": "eslint 'src/**/*.ts' --fix"
+    "fix:eslint": "eslint \"src/**/*.{ts,tsx}\" --fix"
   }
 }

packages/react/xr/package.json

@@ -48,8 +48,8 @@
     "build": "tsc",
     "generate_docs": "typedoc",
     "check:prettier": "prettier --check src",
-    "check:eslint": "eslint 'src/**/*.ts'",
+    "check:eslint": "eslint \"src/**/*.{ts,tsx}\"",
     "fix:prettier": "prettier --write src",
-    "fix:eslint": "eslint 'src/**/*.ts' --fix"
+    "fix:eslint": "eslint \"src/**/*.{ts,tsx}\" --fix"
   }
 }

packages/xr/package.json

@@ -22,9 +22,9 @@
   "scripts": {
     "build": "tsc",
     "check:prettier": "prettier --check src",
-    "check:eslint": "eslint 'src/**/*.ts'",
+    "check:eslint": "eslint \"src/**/*.{ts,tsx}\"",
     "fix:prettier": "prettier --write src",
-    "fix:eslint": "eslint 'src/**/*.ts' --fix"
+    "fix:eslint": "eslint \"src/**/*.{ts,tsx}\" --fix"
   },
   "peerDependencies": {
     "three": "*"