Align linting scripts, add turbo lint:fix task (#149)

Author: jeffsmale90

package.json

@@ -21,6 +21,7 @@
     "format": "prettier --write \"**/*.{ts,tsx,md}\"",
     "generate:abi": "cd ./shared/abi/scripts && ./generate-abi.sh",
     "lint": "turbo lint",
+    "lint:fix": "turbo lint:fix",
     "test": "turbo run test --filter='!./packages/delegator-e2e'",
     "test:e2e": "turbo run test:e2e --filter='./packages/delegator-e2e'",
     "test:external": "cd packages/delegator-e2e && yarn test:external",

packages/7715-permission-types/package.json

@@ -43,7 +43,9 @@
   "scripts": {
     "build": "yarn typecheck && tsup",
     "typecheck": "tsc --noEmit",
-    "lint": "eslint . --cache --ext js,ts",
+    "lint": "yarn lint:eslint",
+    "lint:eslint": "eslint . --cache --ext js,ts",
+    "lint:fix": "yarn lint:eslint --fix",
     "changelog:update": "../../scripts/update-changelog.sh @metamask/7715-permission-types",
     "changelog:validate": "../../scripts/validate-changelog.sh @metamask/7715-permission-types"
   },

packages/delegation-abis/package.json

@@ -57,7 +57,8 @@
     "typecheck": "tsc --noEmit",
     "changelog:update": "../../scripts/update-changelog.sh @metamask/delegation-abis",
     "changelog:validate": "../../scripts/validate-changelog.sh @metamask/delegation-abis",
-    "clean": "rm -rf dist"
+    "clean": "rm -rf dist",
+    "lint": ""
   },
   "publishConfig": {
     "access": "public",

packages/delegation-core/package.json

@@ -51,13 +51,8 @@
     "changelog:validate": "../../scripts/validate-changelog.sh @metamask/delegation-core",
     "clean": "rm -rf dist",
     "lint": "yarn lint:eslint",
-    "lint:complete": "yarn lint:eslint && yarn lint:constraints && yarn lint:misc --check && yarn lint:dependencies --check && yarn lint:changelog",
-    "lint:changelog": "auto-changelog validate --prettier",
-    "lint:constraints": "yarn constraints",
-    "lint:dependencies": "depcheck && yarn dedupe",
     "lint:eslint": "eslint . --cache --ext js,ts",
-    "lint:fix": "yarn lint:eslint --fix && yarn lint:constraints --fix && yarn lint:misc --write && yarn lint:dependencies && yarn lint:changelog",
-    "lint:misc": "prettier '**/*.json' '**/*.md' '**/*.yml' '!.yarnrc.yml' --ignore-path .gitignore --no-error-on-unmatched-pattern"
+    "lint:fix": "yarn lint:eslint --fix"
   },
   "publishConfig": {
     "access": "public",

packages/delegation-deployments/package.json

@@ -43,7 +43,9 @@
   "scripts": {
     "build": "yarn typecheck && tsup",
     "typecheck": "tsc --noEmit",
-    "lint": "eslint . --cache --ext js,ts",
+    "lint": "yarn lint:eslint",
+    "lint:eslint": "eslint . --cache --ext js,ts",
+    "lint:fix": "yarn lint:eslint --fix",
     "changelog:update": "../../scripts/update-changelog.sh @metamask/delegation-deployments",
     "changelog:validate": "../../scripts/validate-changelog.sh @metamask/delegation-deployments",
     "validate-latest-contracts": "yarn tsx script/validate-contract-deployments.ts"

packages/smart-accounts-kit/package.json

@@ -106,13 +106,8 @@
     "test:watch": "vitest watch",
     "format": "prettier --write \"src/**/*.{ts,tsx}\" --ignore-path .prettierignore",
     "lint": "yarn lint:eslint",
-    "lint:complete": "yarn lint:eslint && yarn lint:constraints && yarn lint:misc --check && yarn lint:dependencies --check && yarn lint:changelog",
-    "lint:changelog": "auto-changelog validate --prettier",
-    "lint:constraints": "yarn constraints",
-    "lint:dependencies": "depcheck && yarn dedupe",
     "lint:eslint": "eslint . --cache --ext js,ts",
-    "lint:fix": "yarn lint:eslint --fix && yarn lint:constraints --fix && yarn lint:misc --write && yarn lint:dependencies && yarn lint:changelog",
-    "lint:misc": "prettier '**/*.json' '**/*.md' '**/*.yml' '!.yarnrc.yml' --ignore-path .gitignore --no-error-on-unmatched-pattern"
+    "lint:fix": "yarn lint:eslint --fix"
   },
   "publishConfig": {
     "access": "public",

turbo.json

@@ -22,6 +22,9 @@
     "lint": {
       "dependsOn": ["^lint"]
     },
+    "lint:fix": {
+      "dependsOn": ["^lint:fix"]
+    },
     "dev": {
       "cache": false,
       "persistent": true