chore: setup eslint

isaac-mason · isaac-mason · commit f5a10fe0309e · 2023-12-24T14:01:37.000+10:00
diff --git a/package.json b/package.json
@@ -14,10 +14,11 @@
     "build": "yarn build:packages && yarn build:apps",
     "build:packages": "yarn workspaces foreach -t --include @recast-navigation/wasm --include @recast-navigation/core --include @recast-navigation/generators --include @recast-navigation/three --include recast-navigation run build",
     "build:apps": "yarn workspaces foreach -t --include navmesh-website --include docs run build",
-    "test": "concurrently --kill-others-on-fail \"yarn test:packages\" \"yarn test:node-smoke-test\" \"yarn test:bundlers-smoke-test\"",
+    "test": "concurrently --kill-others-on-fail \"yarn test:packages\" \"yarn test:node-smoke-test\" \"yarn test:bundlers-smoke-test\" \"yarn lint\"",
     "test:packages": "yarn workspaces foreach -t --include recast-navigation --include @recast-navigation/core --include @recast-navigation/three --include @recast-navigation/generators run test",
     "test:node-smoke-test": "(cd ./examples/node-cjs-example && yarn start) && (cd ./examples/node-esm-example && yarn start)",
     "test:bundlers-smoke-test": "concurrently --kill-others-on-fail \"(cd ./examples/parcel-example && yarn build)\" \"(cd ./examples/react-scripts-example && yarn build)\" \"(cd ./examples/vite-example && yarn build)\"",
+    "lint": "yarn workspaces foreach -A -t run lint",
     "change": "yarn changeset",
     "release": "yarn build && yarn test",
     "publish": "changeset publish",
diff --git a/packages/recast-navigation-core/.eslintrc b/packages/recast-navigation-core/.eslintrc
@@ -0,0 +1,3 @@
+{
+  "extends": "@isaac-mason/eslint-config-typescript"
+}
diff --git a/packages/recast-navigation-core/.eslintrc.cjs b/packages/recast-navigation-core/.eslintrc.cjs
diff --git a/packages/recast-navigation-core/package.json b/packages/recast-navigation-core/package.json
@@ -21,32 +21,23 @@
     "LICENSE"
   ],
   "scripts": {
+    "build": "rollup --config rollup.config.js --bundleConfigAsCjs",
     "test": "tsc",
-    "build": "rollup --config rollup.config.js --bundleConfigAsCjs"
+    "lint": "eslint --ext .ts src"
   },
   "dependencies": {
     "@recast-navigation/wasm": "0.16.1"
   },
   "devDependencies": {
     "@babel/core": "^7.22.11",
+    "@isaac-mason/eslint-config-typescript": "^0.0.2",
     "@rollup/plugin-commonjs": "^25.0.7",
     "@rollup/plugin-node-resolve": "^15.0.1",
     "@rollup/plugin-terser": "^0.4.3",
     "@rollup/plugin-typescript": "^11.1.5",
     "@types/three": "^0.159.0",
-    "@typescript-eslint/eslint-plugin": "^6.10.0",
-    "@typescript-eslint/parser": "^6.7.3",
     "babel-loader": "^9.1.3",
-    "eslint": "^8.51.0",
-    "eslint-config-airbnb": "^18.2.1",
-    "eslint-config-airbnb-base": "^15.0.0",
-    "eslint-config-prettier": "^9.0.0",
-    "eslint-import-resolver-typescript": "^3.5.5",
-    "eslint-plugin-es": "^4.1.0",
-    "eslint-plugin-import": "^2.29.0",
-    "eslint-plugin-prettier": "^5.0.1",
-    "eslint-plugin-simple-import-sort": "^10.0.0",
-    "eslint-plugin-typescript-enum": "^2.1.0",
+    "eslint": "^8.56.0",
     "prettier": "^3.1.0",
     "rollup": "^4.6.0",
     "rollup-plugin-copy": "^3.4.0",
diff --git a/packages/recast-navigation-core/src/serdes/export-nav-mesh.ts b/packages/recast-navigation-core/src/serdes/export-nav-mesh.ts
@@ -8,6 +8,7 @@ export const exportNavMesh = (
 ): Uint8Array => {
   const navMeshExport = Raw.NavMeshExporter.exportNavMesh(
     navMesh.raw,
+    // eslint-disable-next-line @typescript-eslint/no-non-null-asserted-optional-chain
     tileCache?.raw!
   );
 
diff --git a/packages/recast-navigation-core/src/serdes/import-nav-mesh.ts b/packages/recast-navigation-core/src/serdes/import-nav-mesh.ts
@@ -34,6 +34,7 @@ export const importNavMesh = (
 
   const result = Raw.NavMeshImporter.importNavMesh(
     buf,
+    // eslint-disable-next-line @typescript-eslint/no-non-null-asserted-optional-chain
     tileCacheMeshProcess?.raw!
   );
 
diff --git a/packages/recast-navigation-generators/.eslintrc b/packages/recast-navigation-generators/.eslintrc
@@ -0,0 +1,3 @@
+{
+  "extends": "@isaac-mason/eslint-config-typescript"
+}
diff --git a/packages/recast-navigation-generators/.eslintrc.cjs b/packages/recast-navigation-generators/.eslintrc.cjs
diff --git a/packages/recast-navigation-generators/package.json b/packages/recast-navigation-generators/package.json
@@ -21,30 +21,21 @@
     "LICENSE"
   ],
   "scripts": {
+    "build": "rollup --config rollup.config.js --bundleConfigAsCjs",
     "test": "tsc",
-    "build": "rollup --config rollup.config.js --bundleConfigAsCjs"
+    "lint": "eslint --ext .ts src"
   },
   "dependencies": {
     "@recast-navigation/core": "0.16.1",
     "@recast-navigation/wasm": "0.16.1"
   },
   "devDependencies": {
+    "@isaac-mason/eslint-config-typescript": "^0.0.2",
     "@rollup/plugin-commonjs": "^25.0.7",
     "@rollup/plugin-node-resolve": "^15.0.1",
     "@rollup/plugin-terser": "^0.4.3",
     "@rollup/plugin-typescript": "^11.1.5",
-    "@typescript-eslint/eslint-plugin": "^6.10.0",
-    "@typescript-eslint/parser": "^6.7.3",
-    "eslint": "^8.51.0",
-    "eslint-config-airbnb": "^18.2.1",
-    "eslint-config-airbnb-base": "^15.0.0",
-    "eslint-config-prettier": "^9.0.0",
-    "eslint-import-resolver-typescript": "^3.5.5",
-    "eslint-plugin-es": "^4.1.0",
-    "eslint-plugin-import": "^2.29.0",
-    "eslint-plugin-prettier": "^5.0.1",
-    "eslint-plugin-simple-import-sort": "^10.0.0",
-    "eslint-plugin-typescript-enum": "^2.1.0",
+    "eslint": "^8.56.0",
     "prettier": "^3.1.0",
     "rollup": "^4.6.0",
     "rollup-plugin-copy": "^3.4.0",
diff --git a/packages/recast-navigation-three/.eslintrc b/packages/recast-navigation-three/.eslintrc
@@ -0,0 +1,3 @@
+{
+  "extends": "@isaac-mason/eslint-config-typescript"
+}
diff --git a/packages/recast-navigation-three/.eslintrc.cjs b/packages/recast-navigation-three/.eslintrc.cjs
diff --git a/packages/recast-navigation-three/package.json b/packages/recast-navigation-three/package.json
@@ -21,8 +21,9 @@
     "LICENSE"
   ],
   "scripts": {
+    "build": "rollup --config rollup.config.js --bundleConfigAsCjs",
     "test": "tsc",
-    "build": "rollup --config rollup.config.js --bundleConfigAsCjs"
+    "lint": "eslint --ext .ts src"
   },
   "dependencies": {
     "@recast-navigation/core": "0.16.1",
@@ -33,23 +34,13 @@
     "three": "0.x.x"
   },
   "devDependencies": {
+    "@isaac-mason/eslint-config-typescript": "^0.0.2",
     "@rollup/plugin-commonjs": "^25.0.7",
     "@rollup/plugin-node-resolve": "^15.0.1",
     "@rollup/plugin-terser": "^0.4.3",
     "@rollup/plugin-typescript": "^11.1.5",
     "@types/three": "^0.159.0",
-    "@typescript-eslint/eslint-plugin": "^6.10.0",
-    "@typescript-eslint/parser": "^6.7.3",
-    "eslint": "^8.51.0",
-    "eslint-config-airbnb": "^18.2.1",
-    "eslint-config-airbnb-base": "^15.0.0",
-    "eslint-config-prettier": "^9.0.0",
-    "eslint-import-resolver-typescript": "^3.5.5",
-    "eslint-plugin-es": "^4.1.0",
-    "eslint-plugin-import": "^2.29.0",
-    "eslint-plugin-prettier": "^5.0.1",
-    "eslint-plugin-simple-import-sort": "^10.0.0",
-    "eslint-plugin-typescript-enum": "^2.1.0",
+    "eslint": "^8.56.0",
     "prettier": "^3.1.0",
     "rollup": "^4.6.0",
     "rollup-plugin-copy": "^3.4.0",
diff --git a/packages/recast-navigation-three/src/helpers/tile-cache-helper.ts b/packages/recast-navigation-three/src/helpers/tile-cache-helper.ts
@@ -46,7 +46,7 @@ export class TileCacheHelper extends Object3D {
     const unseen = new Set(this.obstacleMeshes.keys());
 
     for (const [, obstacle] of this.tileCache.obstacles) {
-      let obstacleMesh = this.obstacleMeshes.get(obstacle);
+      const obstacleMesh = this.obstacleMeshes.get(obstacle);
 
       unseen.delete(obstacle);
 
diff --git a/packages/recast-navigation-three/src/utils/get-positions-and-indices.ts b/packages/recast-navigation-three/src/utils/get-positions-and-indices.ts
@@ -16,7 +16,7 @@ export const getPositionsAndIndices = (
     }
 
     let meshToMerge = mesh;
-    let index: ArrayLike<number> | undefined = mesh.geometry.getIndex()?.array;
+    const index: ArrayLike<number> | undefined = mesh.geometry.getIndex()?.array;
 
     if (!index) {
       meshToMerge = meshToMerge.clone();
diff --git a/packages/recast-navigation/.eslintrc b/packages/recast-navigation/.eslintrc
@@ -0,0 +1,3 @@
+{
+  "extends": "@isaac-mason/eslint-config-typescript"
+}
diff --git a/packages/recast-navigation/package.json b/packages/recast-navigation/package.json
@@ -62,7 +62,8 @@
     "build": "yarn clean && rollup --config rollup.config.js --bundleConfigAsCjs",
     "storybook": "storybook dev -p 6006",
     "build-storybook": "storybook build",
-    "test": "tsc && vitest run"
+    "test": "tsc && vitest run",
+    "lint": "eslint --ext .ts src"
   },
   "dependencies": {
     "@recast-navigation/core": "0.16.1",
@@ -71,6 +72,7 @@
   },
   "devDependencies": {
     "@babel/core": "^7.22.11",
+    "@isaac-mason/eslint-config-typescript": "^0.0.2",
     "@mdx-js/react": "^2.3.0",
     "@pmndrs/assets": "^1.6.0",
     "@react-three/csg": "^3.1.0",
@@ -90,6 +92,7 @@
     "@types/react": "^18.2.11",
     "@types/react-dom": "^18.2.8",
     "@types/three": "^0.159.0",
+    "eslint": "^8.56.0",
     "leva": "^0.9.35",
     "react": "^18.2.0",
     "react-dom": "^18.2.0",
diff --git a/yarn.lock b/yarn.lock

Original file line number	Diff line number	Diff line change
`@@ -0,0 +1,3 @@`
	`1`	`+{`
	`2`	`+ "extends": "@isaac-mason/eslint-config-typescript"`
	`3`	`+}`