fix: remove deprecated --ext flag from ESLint scripts and improve pnpm detection in pre-push hook

olweraltuve · olweraltuve · commit ca41e239d4af · 2025-06-02T12:21:45.000-04:00
diff --git a/.husky/pre-push b/.husky/pre-push
@@ -7,7 +7,13 @@ fi
 
 # Detect if running on Windows and use pnpm.cmd, otherwise use pnpm.
 if [ "$OS" = "Windows_NT" ]; then
-  pnpm_cmd="pnpm.cmd"
+  if command -v pnpm.cmd >/dev/null 2>&1; then
+    pnpm_cmd="pnpm.cmd"
+  elif command -v pnpm >/dev/null 2>&1; then
+    pnpm_cmd="pnpm"
+  else
+    pnpm_cmd="npx pnpm"
+  fi
 else
   if command -v pnpm >/dev/null 2>&1; then
     pnpm_cmd="pnpm"
diff --git a/apps/vscode-e2e/package.json b/apps/vscode-e2e/package.json
@@ -2,7 +2,7 @@
 	"name": "@roo-code/vscode-e2e",
 	"private": true,
 	"scripts": {
-		"lint": "eslint src --ext=ts --max-warnings=0",
+		"lint": "eslint src --max-warnings=0",
 		"check-types": "tsc -p tsconfig.esm.json --noEmit",
 		"format": "prettier --write src",
 		"test:ci": "pnpm -w bundle && pnpm --filter @roo-code/vscode-webview build && pnpm test:run",
diff --git a/packages/build/package.json b/packages/build/package.json
@@ -6,7 +6,7 @@
 	"main": "./dist/index.js",
 	"types": "./src/index.ts",
 	"scripts": {
-		"lint": "eslint src --ext=ts --max-warnings=0",
+		"lint": "eslint src --max-warnings=0",
 		"check-types": "tsc --noEmit",
 		"test": "vitest run",
 		"build": "tsc",
diff --git a/packages/cloud/package.json b/packages/cloud/package.json
@@ -5,7 +5,7 @@
 	"type": "module",
 	"exports": "./src/index.ts",
 	"scripts": {
-		"lint": "eslint src --ext=ts --max-warnings=0",
+		"lint": "eslint src --max-warnings=0",
 		"check-types": "tsc --noEmit",
 		"test": "vitest run",
 		"clean": "rimraf dist .turbo"
diff --git a/packages/telemetry/package.json b/packages/telemetry/package.json
@@ -5,7 +5,7 @@
 	"type": "module",
 	"exports": "./src/index.ts",
 	"scripts": {
-		"lint": "eslint src --ext=ts --max-warnings=0",
+		"lint": "eslint src --max-warnings=0",
 		"check-types": "tsc --noEmit",
 		"test": "vitest run",
 		"clean": "rimraf dist .turbo"
diff --git a/packages/types/package.json b/packages/types/package.json
@@ -14,7 +14,7 @@
 		}
 	},
 	"scripts": {
-		"lint": "eslint src --ext=ts --max-warnings=0",
+		"lint": "eslint src --max-warnings=0",
 		"check-types": "tsc --noEmit",
 		"test": "vitest run",
 		"build": "tsup",
diff --git a/src/package.json b/src/package.json
@@ -339,7 +339,7 @@
 		}
 	},
 	"scripts": {
-		"lint": "eslint . --ext=ts --max-warnings=0",
+		"lint": "eslint . --max-warnings=0",
 		"check-types": "tsc --noEmit",
 		"pretest": "turbo run bundle --cwd ..",
 		"test": "jest -w=40% && vitest run",
diff --git a/webview-ui/package.json b/webview-ui/package.json
@@ -3,7 +3,7 @@
 	"private": true,
 	"type": "module",
 	"scripts": {
-		"lint": "eslint src --ext=ts,tsx --max-warnings=0",
+		"lint": "eslint src --max-warnings=0",
 		"check-types": "tsc",
 		"pretest": "turbo run bundle --cwd ..",
 		"test": "jest -w=40%",

Original file line number	Diff line number	Diff line change
`@@ -14,7 +14,7 @@`
`14`	`14`	`}`
`15`	`15`	`},`
`16`	`16`	`"scripts": {`
`17`		`- "lint": "eslint src --ext=ts --max-warnings=0",`
	`17`	`+ "lint": "eslint src --max-warnings=0",`
`18`	`18`	`"check-types": "tsc --noEmit",`
`19`	`19`	`"test": "vitest run",`
`20`	`20`	`"build": "tsup",`
Original file line number	Diff line number	Diff line change
`@@ -339,7 +339,7 @@`
`339`	`339`	`}`
`340`	`340`	`},`
`341`	`341`	`"scripts": {`
`342`		`- "lint": "eslint . --ext=ts --max-warnings=0",`
	`342`	`+ "lint": "eslint . --max-warnings=0",`
`343`	`343`	`"check-types": "tsc --noEmit",`
`344`	`344`	`"pretest": "turbo run bundle --cwd ..",`
`345`	`345`	`"test": "jest -w=40% && vitest run",`