feat(eslint, prettier): Update eslint and prettier configurations (#69)

### Prettier

- Convert configuration to JSON
- Turn off semicolons
- Use tabs instead of spaces for indentation

### Eslint

- Use standard-with-typescript instead of xo
- Remove custom rule definitions
- Remove dependency on eslint-plugin-prettier

.changeset/healthy-planes-clap.md

@@ -0,0 +1,5 @@
+---
+'@jarsec/prettier-config': patch
+---
+
+Remove space indentation setting

.changeset/quick-turkeys-think.md

@@ -0,0 +1,7 @@
+---
+"@jarsec/prettier-config": major
+---
+
+- Convert configuration file to JSON
+- No semicolons allowed
+- Use tabs instead of spaces for indentation

.changeset/thin-grapes-matter.md

@@ -0,0 +1,8 @@
+---
+'@jarsec/eslint-config': major
+---
+
+- Remove xo and xo-typescript configurations
+- Add standard-with-typescript configuration
+- Remove eslint-plugin-prettier, let Prettier handle formatting in a separate step
+- Remove custom TypeScript rules

package.json

@@ -13,10 +13,12 @@
   "devDependencies": {
     "@changesets/cli": "2.26.2",
     "@jarsec/eslint-config": "workspace:*",
+    "@jarsec/prettier-config": "workspace:*",
     "@jarsec/tsconfig": "workspace:*",
     "eslint": "8.53.0",
     "prettier": "3.1.0",
     "turbo": "latest"
   },
-  "packageManager": "pnpm@8.10.5"
+  "packageManager": "pnpm@8.10.5",
+  "prettier": "@jarsec/prettier-config"
 }

packages/eslint-config/index.js

@@ -1,57 +1,36 @@
-/* eslint-disable @typescript-eslint/naming-convention */
-/* eslint-disable @typescript-eslint/no-require-imports */
-/* eslint-disable @typescript-eslint/no-unsafe-call */
-
-require("@rushstack/eslint-patch/modern-module-resolution");
-
-const rules = {
-  "comma-dangle": "off",
-  "import/first": "error",
-  "import/newline-after-import": "error",
-  "import/no-duplicates": "error",
-  "no-mixed-spaces-and-tabs": "off",
-  "no-warning-comments": "off",
-  "object-curly-spacing": "off",
-  "operator-linebreak": "off",
-};
+require('@rushstack/eslint-patch/modern-module-resolution')
 
 module.exports = {
-  env: {
-    node: true,
-  },
-  extends: [
-    "xo",
-    "xo-typescript",
-    "plugin:perfectionist/recommended-natural",
-    "plugin:prettier/recommended",
-  ],
-  overrides: [
-    {
-      files: ["**/*/*.{test,spec}.ts"],
-      rules: {
-        ...rules,
-        "@typescript-eslint/no-unsafe-assignment": "off",
-        "@typescript-eslint/no-unsafe-call": "off",
-        "@typescript-eslint/no-unsafe-member-access": "off",
-        "@typescript-eslint/unbound-method": "off",
-      },
-    },
-  ],
-  parser: "@typescript-eslint/parser",
-  parserOptions: {
-    project: "./tsconfig.json",
-  },
-  plugins: [
-    "@typescript-eslint/eslint-plugin",
-    "import",
-    "perfectionist",
-    "prettier",
-  ],
-  rules,
-  settings: {
-    "import/resolver": {
-      node: true,
-      typescript: true,
-    },
-  },
-};
+	env: {
+		node: true
+	},
+	extends: [
+		'eslint:recommended',
+		'plugin:@typescript-eslint/recommended',
+		'plugin:@typescript-eslint/stylistic',
+		'plugin:perfectionist/recommended-natural',
+		'prettier'
+	],
+	overrides: [
+		{
+			files: ['**/*/*.{test,spec}.ts'],
+			rules: {
+				'@typescript-eslint/no-unsafe-assignment': 'off',
+				'@typescript-eslint/no-unsafe-call': 'off',
+				'@typescript-eslint/no-unsafe-member-access': 'off',
+				'@typescript-eslint/unbound-method': 'off'
+			}
+		}
+	],
+	parser: '@typescript-eslint/parser',
+	parserOptions: {
+		project: './tsconfig.json'
+	},
+	plugins: ['import', 'perfectionist', '@typescript-eslint/eslint-plugin'],
+	settings: {
+		'import/resolver': {
+			node: true,
+			typescript: true
+		}
+	}
+}

packages/eslint-config/package.json

@@ -9,24 +9,26 @@
     "access": "public"
   },
   "dependencies": {
-    "@rushstack/eslint-patch": "1.5.1",
-    "@typescript-eslint/eslint-plugin": "6.11.0",
-    "@typescript-eslint/parser": "6.11.0",
-    "eslint-config-prettier": "9.0.0",
-    "eslint-config-xo": "0.43.1",
-    "eslint-config-xo-typescript": "1.0.1",
-    "eslint-import-resolver-typescript": "3.6.1",
-    "eslint-plugin-import": "2.29.0",
-    "eslint-plugin-perfectionist": "2.4.0",
-    "eslint-plugin-prettier": "5.0.1"
+    "@rushstack/eslint-patch": "^1.6.0",
+    "@types/node": "^20.9.3",
+    "@typescript-eslint/eslint-plugin": "^6.12.0",
+    "@typescript-eslint/parser": "^6.12.0",
+    "eslint": "^8.52.0",
+    "eslint-config-prettier": "^9.0.0",
+    "eslint-config-standard-with-typescript": "^40.0.0",
+    "eslint-plugin-import": "^2.29.0",
+    "eslint-plugin-n": "^16.3.1",
+    "eslint-plugin-perfectionist": "^2.4.0",
+    "eslint-plugin-promise": "^6.1.1",
+    "typescript": "^5.2.2"
   },
   "devDependencies": {
-    "@types/node": "20.9.1",
-    "eslint": "8.53.0",
-    "typescript": "5.2.2"
+    "@types/node": "^20.9.1",
+    "eslint": "^8.53.0",
+    "typescript": "^5.2.2"
   },
   "peerDependencies": {
-    "eslint": "8.53.0",
-    "typescript": "5.2.2"
+    "eslint": "^8.53.0",
+    "typescript": "^5.2.2"
   }
 }

packages/prettier/index.json

@@ -0,0 +1,11 @@
+{
+  "singleQuote": true,
+  "semi": false,
+  "arrowParens": "avoid",
+  "trailingComma": "none",
+  "bracketSpacing": true,
+  "endOfLine": "auto",
+  "printWidth": 100,
+  "tabWidth": 2,
+  "useTabs": true
+}

packages/prettier/package.json

@@ -5,7 +5,7 @@
   "license": "MIT",
   "author": "Jarrod Seccombe <jarrod.seccombe@icloud.com",
   "files": [
-    "index.js",
+    "index.json",
     ".prettierignore"
   ],
   "main": "index.js",